Split `forgelm/cli/_parser.py` into a sub-package (`_train`, `_inspect`, `_data`, `_run`) — deferred until Phase 16 S5, S7 and S11 have landed
まだ誰も着手していません。
評価
調査の方向性
Start by confirming that #891, #893 and #897 have merged, then read the split seams recorded in docs/roadmap/risks-and-decisions.md near line 80. The work touches forgelm/cli/_parser.py, the _DEFERRED_SPLITS entry in tools/check_module_size.py, and check_cli_help_consistency.py, which pins every --help string. Done means the sub-package keeps every public name importable, the module-size check passes without the deferral, and the CHANGELOG names F-PR29-A1-06/07/08.
索引モデルが issue の本文から書いたものです。
説明
Deferred work ·
F-PR29-A1-06 / A1-07 / A1-08: _parser.py· severity Low · cost ~4-8 h · recorded indocs/roadmap/risks-and-decisions.md:80
Summary
forgelm/cli/_parser.py is a deferred module split: every subcommand's argparse surface lives in one file, and check_cli_help_consistency.py pins every --help string, so the diff is wide and must be taken once. The module-size guard holds it to a fatal no-growth budget, but nothing pays the debt down until someone takes the split.
Condition or budget
Phase 16 S5, S7 and S11 have all landed (each edits this file); they are #891, #893 and #897
State today (re-checked 2026-10-09)
1370 code lines against a budget of 1372 (deferred at 1320).
Related
#667 and #668 track how the module-size guard counts lines and accepts budget raises; decide them before relying on a module's budget.
Acceptance criteria
- The module becomes a sub-package along the recorded seams, behaviour-neutral, with every public name still importable from its current path.
- Its
_DEFERRED_SPLITSentry is removed fromtools/check_module_size.pyand every resulting module is under the normal ceiling. - Test patch targets that named the old module are retargeted to the module that resolves the name at call time (the
forgelm/verify.pysplit found five that silently moved). - The row is removed from the deferral table and the CHANGELOG names its ID (the table's removal contract).
Recorded in docs/roadmap/risks-and-decisions.md (read at f94595f) and re-checked against the code on 2026-10-09. The row leaves that table only when the fix is on main, the CHANGELOG names its ID and any promised guard exists.
- 主要言語
- Python
- スター
- 9
- フォーク
- 1
- 平均マージ
- 8時間 10分
- マージ済み PR（30日）
- 5
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
HodeTech/ForgeLM のほかの issue
-
area: dev-tooling bug severity: low source: roadmap wave: 4
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
Site: the theme-locked --code-comment grey (#6a6f80) on the dark code surface is 3.8:1, below WCAG AA, in both themes (F-W1-SITE-06)対応中かも @vortsghost2025 が今日担当しました。 オープンarea: site bug good first issue severity: low source: review-2026-09 wave: 4
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
-
documentation good first issue severity: medium source: review-2026-09 wave: 3
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
documentation good first issue severity: low source: review-2026-09 wave: 4
難易度 1/5 1〜3時間 初心者へのやさしさ 80/100
-
documentation severity: medium source: review-2026-09 wave: 3
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
HodeTech/ForgeLM の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
uniswap-python/uniswap-python#487 ·
-
[Bug]: agent_final_response still returns the abandoned branch's answer after navigate moves HEAD back対応中かも @enyst が今日担当しました。 オープンbug priority:low ready-for-dev
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
OpenHands/software-agent-sdk#5822 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
-
area: server enhancement good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
scs0209/laya-guard#52 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 2 日以内に返信