Adopt the `minimal_config` test factory across the suite and add a CI guard on its adoption rate

オープン

#910 2026年10月9日に作成コメント 0 件リアクション 0 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 4/5
見積もり時間: 3〜5日
初心者へのやさしさ: 45/100

issue の種類: リファクタリング
明瞭さ: おおむね明確
活発さ: 活発
技術スタック: github-actions, python
領域: ci-cd, testing

調査の方向性

Start by reading minimal_config in tests/_helpers/factories, then grep tests/test_*.py for inline "name_or_path": dicts to see which modules still build configs by hand. The guard script tools/check_test_factory_adoption.py does not exist yet; it needs to fail below 70% adoption and be wired into ci.yml and tests/test_guard_wiring.py. Done means the migrated modules pass, the guard runs in CI, and the CHANGELOG names the deferral ID.

索引モデルが issue の本文から書いたものです。

説明

area: dev-tooling severity: low source: roadmap wave: 4

Deferred work · F-PR29-A4-03 · severity Low · cost ~3-4 h · recorded in docs/roadmap/risks-and-decisions.md:81

Summary

tests/_helpers/factories.minimal_config is documented as the single source of a minimal valid ForgeConfig, but only 21 of 128 test modules use it and 21 modules build inline config dicts without it, so a schema change has to be applied by hand to every inline copy. The PR #29 plan was a bulk migration plus tools/check_test_factory_adoption.py failing CI below 70 % adoption; adoption has since fallen from 30 % to about 16 % as the suite grew.

Condition or budget

None recorded (the row still names v0.6.x, long past). Guard threshold: 70 % adoption.

State today (re-checked 2026-10-09)

21 of 128 tests/test_*.py modules use the factory; 36 modules hold 116 inline "name_or_path": config dicts; tools/check_test_factory_adoption.py does not exist.

Acceptance criteria

Inline ForgeConfig dicts in tests use minimal_config(...) (or document why they cannot).
tools/check_test_factory_adoption.py fails below the agreed rate and is wired into ci.yml, the CLAUDE.md / AGENTS.md / CONTRIBUTING.md gauntlet and tests/test_guard_wiring.py.
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
平均マージ: 4時間 3分
マージ済み PR（30日）: 3

環境構築

Dockerfile または Docker Compose ファイルあり
プルリクエストのテンプレートあり
コントリビューションガイドを読む

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
issue 番号を参照したプルリクエストを送ります。

HodeTech/ForgeLM のほかの issue

`sonar-project.properties` names a SonarCloud project that does not exist (`HodeTech_ForgeLM`), so its sources and exclusions are probably not applied
オープン
- 何か問題がありますか？
area: dev-tooling bug severity: low source: roadmap wave: 4

難易度 2/5 1〜3時間初心者へのやさしさ 70/100

HodeTech/ForgeLM#927 · 2026年10月9日に作成
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)
オープン
- 何か問題がありますか？
area: site bug good first issue severity: low source: review-2026-09 wave: 4

難易度 2/5 1〜3時間初心者へのやさしさ 66/100

HodeTech/ForgeLM#887 · 2026年10月9日に作成
Choosing-a-trainer table documents KTO rows as {prompt, response, label}, but the KTO loader requires a completion column (F-W1-DOCS-TRAIN-02)
オープン
- 何か問題がありますか？
documentation good first issue severity: medium source: review-2026-09 wave: 3

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

HodeTech/ForgeLM#885 · 2026年10月9日に作成
Trend-tracking jq recipe prints evaluation_completed=false as 'n/a', because jq's // operator treats false like null (F-W1-DOCS-TRAIN-16)
オープン
- 何か問題がありますか？
documentation good first issue severity: low source: review-2026-09 wave: 4

難易度 1/5 1〜3時間初心者へのやさしさ 80/100

HodeTech/ForgeLM#884 · 2026年10月9日に作成
Ingestion guide's AWS Textract recipe reads only the first GetDocumentTextDetection page and never checks JobStatus, silently truncating multi-page OCR text (F-W1-DOCS-TRAIN-13)
オープン
- 何か問題がありますか？
documentation severity: medium source: review-2026-09 wave: 3

難易度 2/5 1〜3時間初心者へのやさしさ 72/100

HodeTech/ForgeLM#883 · 2026年10月9日に作成

HodeTech/ForgeLM の issue をすべて見る

似ている issue

[workshop-sim] Repair: Fix checkpoint_quality rubric detection to recognize :white_check_mark: emoji shortcode
オープン
- 何か問題がありますか？
feedback simulation workshop

難易度 2/5 1〜3時間初心者へのやさしさ 73/100

githubnext/gh-aw-workshop#4455 · 2026年10月11日に作成

メンテナーはふだん 1 日以内に返信
`highly_variable_genes(flavor="cell_ranger", n_top_genes=n)` selects 0 genes when the top genes have infinite normalized dispersion
オープン
- 何か問題がありますか？
Triage 🩺

難易度 2/5 1〜3時間初心者へのやさしさ 76/100

scverse/scanpy#4415 · 2026年10月11日に作成

メンテナーはふだん 1 日以内に返信
[BUG] Container scenario crashes without expected_recovery_time, kube DNS example uses retry_wait
オープン
- 何か問題がありますか？
needs-triage

難易度 2/5 1〜3時間初心者へのやさしさ 77/100

krkn-chaos/krkn#1627 · コメント 1 件 · 2026年10月11日に作成

メンテナーはふだん 1 日以内に返信
[Bug]: Desktop catalog deep links (hermes://plugin/install?catalog=…) all fail — plugins.json fetch blocked by a 301 without CORS
オープン
- 何か問題がありますか？
難易度 2/5 1〜3時間初心者へのやさしさ 72/100

NousResearch/hermes-agent#136483 · 2026年10月11日に作成

メンテナーはふだん 1 日以内に返信
BM25SearchStrategy (the default) never matches underscore-joined terms in tool descriptions or param descriptions
オープン
- 何か問題がありますか？
難易度 1/5 1時間未満初心者へのやさしさ 88/100

livekit/agents#7727 · 2026年10月11日に作成

メンテナーはふだん 1 日以内に返信

Python の issue をもっと見る

Adopt the `minimal_config` test factory across the suite and add a CI guard on its adoption rate

何か問題がありますか？

評価

調査の方向性

説明

Summary

Condition or budget

State today (re-checked 2026-10-09)

Acceptance criteria

環境構築

はじめの一歩

HodeTech/ForgeLM のほかの issue

何か問題がありますか？

何か問題がありますか？

何か問題がありますか？

何か問題がありますか？

何か問題がありますか？

似ている issue

何か問題がありますか？

何か問題がありますか？

何か問題がありますか？

何か問題がありますか？

何か問題がありますか？

新しい issue をメールで受け取る