[BUG] Linter is_snake_case only matches ASCII — always fails for Devanagari identifiers
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 56/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- python
- Bereich
- compilers, internationalization
Rechercherichtung
The naming rule is is_snake_case in maithili_dsl/transpiler/linter.py, and its ASCII-only regex is the cause. Check where enforce_snake_case in LINT_CONFIG is read, and run 'नाम'.isidentifier() to confirm the identifier is valid. Done when Devanagari names pass the check, Latin snake_case is still enforced, and new cases in tests/test_linter.py cover both. The issue leaves the script-detection approach open, so confirm it with a maintainer before choosing one.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
File: maithili_dsl/transpiler/linter.py:11
def is_snake_case(name):
return bool(re.match(r'^[a-z_][a-z0-9_]*$', name))
The regex only matches ASCII lowercase, so every Devanagari variable name (e.g. नाम, गिनती) fails the check. Devanagari identifiers are valid Python 3 identifiers ('नाम'.isidentifier() is True), so the naming rule should accept them.
Currently mitigated because enforce_snake_case defaults to False in LINT_CONFIG, but any user who enables it gets a false positive on every line.
Fix: make the naming check script-aware — e.g. accept identifiers composed of Devanagari characters (and underscores/digits), and only apply the ASCII snake_case rule to Latin-script names. Add cases to tests/test_linter.py.
From BACKLOG.md item 10.
- Vorherrschende Sprache
- Python
- Sterne
- 1
- Forks
- 4
- Ø Merge
- 12 T. 21 Std.
- Gemergte PRs (30 T.)
- 1
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus alphacrack/python-maithili-dsl
-
help wanted infra P2 testing
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 80/100
-
documentation enhancement good first issue help wanted P3
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
documentation good first issue P3
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
-
area:linter enhancement good first issue P3 translation
Schwierigkeit 2/5 Ein halber Tag Anfängerfreundlichkeit 66/100
-
area:cli enhancement good first issue P3
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Alle Issues in alphacrack/python-maithili-dsl
Ähnliche Issues
-
ライブラリを追加Offenpriority middle
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 62/100
KATO-Hiro/AtCoderClans#13015 ·
Maintainer antworten meist innerhalb von 1 Tag
-
StringLiteral to IntegerType/LongType silently truncates fractional strings, changing predicate meaningEvtl. vergeben @Shahzeb-DataEngineer hat das heute übernommen. Offenkind:bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
apache/iceberg-python#4382 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
request-theme
Schwierigkeit 2/5 Unter einer Stunde Anfängerfreundlichkeit 62/100
LizardByte/ThemerrDB#10024 · 2 Kommentare ·
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
btclib-org/btclib-node#2505 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
open-webui/open-webui#34462 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag