[BUG] translate_imports uses blind str.replace — corrupts identifiers and string contents
まだ誰も着手していません。
評価
調査の方向性
Start in translate_imports in maithili_dsl/cli.py, where module names are swapped with plain str.replace, then read _make_keyword_pattern in transpile.py to see the word-boundary approach to reuse. Switch the module rewrite to word boundaries applied only outside string literals and comments, and record translated_modules from actual import statements. Done when both reproductions (the substring case and the string-literal case) produce unchanged output in tests/test_cli.py or tests/test_security.py, and आयात गणित followed by गणित.sqrt(...) still works.
索引モデルが issue の本文から書いたものです。
説明
File: maithili_dsl/cli.py (translate_imports)
translate_imports() still uses blind str.replace() — the exact class of bug that was fixed in the transpiler for keywords (see v0.3.0 hardening). Two verified reproductions:
Substring corruption:
Input: उच्चगणित.कोष = १
Output: उच्चmath.कोष = १ # गणित matched inside उच्चगणित
Rewrites inside string literals:
Input: छपाउ("गणित.")
Output: छपाउ("math.") # user-visible output changed
Side effect: because "{module}." appearing anywhere (including strings/comments) adds the module to translated_modules, the import whitelist gets looser than intended.
Fix: reuse the word-boundary pattern machinery from transpile.py (_make_keyword_pattern) and run module translation on code regions only (after/with tokenization), not on the raw source. Track translated_modules from actual आयात statements rather than substring hits.
Tests: both repros above in tests/test_cli.py or tests/test_security.py, plus regression tests that legitimate आयात गणित + गणित.sqrt(...) still work.
- 主要言語
- Python
- スター
- 1
- フォーク
- 4
- 平均マージ
- 12日 21時間
- マージ済み PR（30日）
- 1
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
alphacrack/python-maithili-dsl のほかの issue
-
help wanted infra P2 testing
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
-
documentation enhancement good first issue help wanted P3
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
documentation good first issue P3
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
-
area:linter enhancement good first issue P3 translation
難易度 2/5 半日 初心者へのやさしさ 66/100
-
area:cli enhancement good first issue P3
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
alphacrack/python-maithili-dsl の issue をすべて見る
似ている issue
-
documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
JheysonJhordyHuarauyaRiquelme/sprocguard#18 ·
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
dlq/synthpopcan#59 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
mdn/translated-content#40220 ·
メンテナーはふだん 4 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
ETSInitiative/PETSIRD#243 ·
メンテナーはふだん 1 日以内に返信
-
deferred p2
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
cuadrao-ai/argus#2160 ·
メンテナーはふだん 1 日以内に返信