Release: ndf v10.27.24 を main へ

.claude-plugin/marketplace.json

@@ -9,7 +9,7 @@
     {
       "name": "ndf",
       "source": "./plugins/ndf",
-      "description": "Claude Code plugin (v10.27.21): 8 specialized agents (plus 1 worker agent for the 3-layer workflow) and 49 focused NDF skills for PR/review workflows, cross-review, implementation planning, plan-to-spec, Docker container access, statusline, external AI delegation (Codex/agy), transcript retention guard, and optional Slack notifications.",
+      "description": "Claude Code plugin (v10.27.24): 8 specialized agents (plus 1 worker agent for the 3-layer workflow) and 49 focused NDF skills for PR/review workflows, cross-review, implementation planning, plan-to-spec, Docker container access, statusline, external AI delegation (Codex/agy), transcript retention guard, and optional Slack notifications.",
       "policy": {
         "installation": "AVAILABLE",
         "authentication": "ON_INSTALL"

.github/workflows/pytest.yml

@@ -95,10 +95,24 @@ jobs:
       #
       # 前提のコマンドの確認と git の身元の設定は、根の conftest.py が行う。前提を
       # ワークフローの側にも置くと、増えたときに片方だけが更新される
-      - run: uv run --frozen --project . --all-extras pytest . -q -n auto
+      # 統括役の結合テスト（test_relay_hub.py::TestWithTmux）が tmux を使う。runner のイメージは tmux を
+      # 載せると約束していないため、無いときだけ入れる（ある間は時間を足さない。#2346）。apt の索引が古いと
+      # 取得で落ちるため、入れる経路では update を install の直前に打つ。版はログに残す
+      - name: tmux
+        run: |
+          command -v tmux || { sudo apt-get update && sudo apt-get install -y tmux; }
+          tmux -V
+      # 項目ごとの名前と結果を JUnit XML に残す（ログは -q で件数だけ。#2346）
+      - run: uv run --frozen --project . --all-extras pytest . -q -n auto --junitxml=pytest-junit-${{ matrix.shard }}.xml
         env:
           SHARD_TOTAL: 2
           SHARD_INDEX: ${{ matrix.shard }}
+      - uses: actions/upload-artifact@v7
+        if: always()
+        with:
+          name: pytest-junit-${{ matrix.shard }}
+          path: pytest-junit-${{ matrix.shard }}.xml
+          if-no-files-found: warn
 
   # ruleset の必須のチェックは `pytest` という名前の結果を 1 つ待つ。分けたジョブの結果を
   # ここで 1 つにまとめる。名前を変えると ruleset の変更が要る。

CHANGELOG.md

@@ -9,6 +9,68 @@
 **開発版（接尾辞の付いた版）は載せない。** `9.8.0` は `9.8.0-dev.1` までしか出ておらず、
 その内容は `10.0.0` で届いている。
 
+## [ndf 10.27.24] - 2026-11-05
+
+- 設計の承認に「承認」と答えると、その設計の PR がマージされる（#2542）（#2543）
+- 開発ループの周回の境で、承認済みの設計の PR がマージし直される（#2542）（#2543）
+- 承認に答えた後、その時点の整理結果が一覧で表示される（#2542）（#2543）
+- 承認の答えの `--by` 引数が契約に含まれ、検査の対象になる（#2542）（#2543）
+- 差分の棚卸しで見積の無い経路に出る Issue は、起点より後に作られたものだけになる（#2546）（#2547）
+- 差分の棚卸しの各行に、プランの進行の記録から写した所要秒数・費用（ドル）・言語モデルが載る（#2546）（#2547）
+- `depth compare` で、所要秒数などの中央値を基準と比べられる（#2546）（#2547）
+- スプリントのまとめでは、プランの深さを記録するステップが、プランの進行の記録（`--progress`）と開発ループ・周回の番号（`--loop`・`--lap`）を受け取る（#2546）（#2547）
+- NDF の `--budget-*` に `default` を渡すと、その単位の予算がリポジトリの予算の既定に戻ります（#2559）（#2561）
+- Factory でループの予算の単位を空にすると、次のジョブからその単位にリポジトリの既定が使われます（#2559）（#2561）
+- Factory のループの画面では、空の予算の単位が「リポジトリの既定」と表示されます（#2559）（#2561）
+- 既定へ戻せない版では、Factory は予算の単位を空にする変更を受け付けず、画面に理由を表示します（#2559）（#2561）
+- リポジトリの設定で本番のブランチ（`production_branch`）を宣言していなくても、作業用の worktree をまだ作っていない段階で、リポジトリの既定ブランチを本番のブランチとして実行計画を組む（#2562）（#2565）
+- ndf の design で、設計の判断を下す前に外部の調査を行います。これが手順 2 になり、以降の手順は番号が 1 つずつ繰り下がります（#2564）（#2574）
+- 設計文書のテンプレートに「外部の調査」の節があります。自前で作ると決めた事項には、比べた既存の候補を書く行があります（#2564）（#2574）
+- 自前で作ると決めた事項に比べた候補が書かれていない設計文書は、ndf の設計の検査で指摘されます（#2564）（#2574）
+- スプリントの設計の工程では、外部の調査の検査、指摘の修正、修正後の再検査が順に自動で実行されます（#2564）（#2574）
+- ndf の pr-review で、レビューの観点に外部の調査の確認が入る（#2572）（#2574）
+- ndf の cross-review で、設計 PR と実装 PR を見る担当へ外部の調査の確認の観点が渡る（#2572）（#2574）
+- ndf の cross-review で、既定の分類を外すレビューのプロファイルを選んでも、外部の調査の確認の観点は残る（#2572）（#2574）
+- ndf プラグインの開発ループは、人が送った停止のときだけ止める指示を置く（#2580）（#2581）
+
+## [ndf 10.27.23] - 2026-11-05
+
+- 設計の承認に「承認」と答えると、その設計の PR がマージされる（#2542）（#2543）
+- 開発ループの周回の境で、承認済みの設計の PR がマージし直される（#2542）（#2543）
+- 承認に答えた後、その時点の整理結果が一覧で表示される（#2542）（#2543）
+- 承認の答えの `--by` 引数が契約に含まれ、検査の対象になる（#2542）（#2543）
+- 差分の棚卸しで見積の無い経路に出る Issue は、起点より後に作られたものだけになる（#2546）（#2547）
+- 差分の棚卸しの各行に、プランの進行の記録から写した所要秒数・費用（ドル）・言語モデルが載る（#2546）（#2547）
+- `depth compare` で、所要秒数などの中央値を基準と比べられる（#2546）（#2547）
+- スプリントのまとめでは、プランの深さを記録するステップが、プランの進行の記録（`--progress`）と開発ループ・周回の番号（`--loop`・`--lap`）を受け取る（#2546）（#2547）
+- NDF の `--budget-*` に `default` を渡すと、その単位の予算がリポジトリの予算の既定に戻ります（#2559）（#2561）
+- Factory でループの予算の単位を空にすると、次のジョブからその単位にリポジトリの既定が使われます（#2559）（#2561）
+- Factory のループの画面では、空の予算の単位が「リポジトリの既定」と表示されます（#2559）（#2561）
+- 既定へ戻せない版では、Factory は予算の単位を空にする変更を受け付けず、画面に理由を表示します（#2559）（#2561）
+- リポジトリの設定で本番のブランチ（`production_branch`）を宣言していなくても、作業用の worktree をまだ作っていない段階で、リポジトリの既定ブランチを本番のブランチとして実行計画を組む（#2562）（#2565）
+- ndf の design で、設計の判断を下す前に外部の調査を行います。これが手順 2 になり、以降の手順は番号が 1 つずつ繰り下がります（#2564）（#2574）
+- 設計文書のテンプレートに「外部の調査」の節があります。自前で作ると決めた事項には、比べた既存の候補を書く行があります（#2564）（#2574）
+- 自前で作ると決めた事項に比べた候補が書かれていない設計文書は、ndf の設計の検査で指摘されます（#2564）（#2574）
+- スプリントの設計の工程では、外部の調査の検査、指摘の修正、修正後の再検査が順に自動で実行されます（#2564）（#2574）
+- ndf の pr-review で、レビューの観点に外部の調査の確認が入る（#2572）（#2574）
+- ndf の cross-review で、設計 PR と実装 PR を見る担当へ外部の調査の確認の観点が渡る（#2572）（#2574）
+- ndf の cross-review で、既定の分類を外すレビューのプロファイルを選んでも、外部の調査の確認の観点は残る（#2572）（#2574）
+
+## [ndf 10.27.22] - 2026-11-05
+
+- 設計の承認に「承認」と答えると、その設計の PR がマージされる（#2542）（#2543）
+- 開発ループの周回の境で、承認済みの設計の PR がマージし直される（#2542）（#2543）
+- 承認に答えた後、その時点の整理結果が一覧で表示される（#2542）（#2543）
+- 承認の答えの `--by` 引数が契約に含まれ、検査の対象になる（#2542）（#2543）
+- 差分の棚卸しで見積の無い経路に出る Issue は、起点より後に作られたものだけになる（#2546）（#2547）
+- 差分の棚卸しの各行に、プランの進行の記録から写した所要秒数・費用（ドル）・言語モデルが載る（#2546）（#2547）
+- `depth compare` で、所要秒数などの中央値を基準と比べられる（#2546）（#2547）
+- スプリントのまとめでは、プランの深さを記録するステップが、プランの進行の記録（`--progress`）と開発ループ・周回の番号（`--loop`・`--lap`）を受け取る（#2546）（#2547）
+- NDF の `--budget-*` に `default` を渡すと、その単位の予算がリポジトリの予算の既定に戻ります（#2559）（#2561）
+- Factory でループの予算の単位を空にすると、次のジョブからその単位にリポジトリの既定が使われます（#2559）（#2561）
+- Factory のループの画面では、空の予算の単位が「リポジトリの既定」と表示されます（#2559）（#2561）
+- 既定へ戻せない版では、Factory は予算の単位を空にする変更を受け付けず、画面に理由を表示します（#2559）（#2561）
+
 ## [ndf 10.27.21] - 2026-11-05
 
 - 設計の承認に「承認」と答えると、その設計の PR がマージされる（#2542）（#2543）

README.md

@@ -48,7 +48,7 @@ Claude Code では `/ndf:<Skill 名>` と打って呼びます。**Skill** は
 
 | プラグイン名 | バージョン | できること | 詳細 |
 | --- | --- | --- | --- |
-| **ndf** | 10.27.21 | 開発の作業手順（Skill）と専門のサブエージェント。Claude Code と Codex へ同じ名前で入る | [README](./plugins/ndf/README.md) |
+| **ndf** | 10.27.24 | 開発の作業手順（Skill）と専門のサブエージェント。Claude Code と Codex へ同じ名前で入る | [README](./plugins/ndf/README.md) |
 | **playwright-kit** | 4.0.0 | Playwright による E2E テストの計画・スクリプト作成・証跡のレポート | [README](./plugins/playwright-kit/README.md) |
 
 外部サービスへつなぐ MCP サーバーの設定のプラグインが 10 個あります（Claude Code / Codex 向け）。
@@ -69,7 +69,7 @@ Claude Code では `/ndf:<Skill 名>` と打って呼びます。**Skill** は
 
 ### NDF プラグインの Skill の一覧
 
-**NDFプラグイン v10.27.21** に入る Skill の数は、AI ツールごとに違います。
+**NDFプラグイン v10.27.24** に入る Skill の数は、AI ツールごとに違います。
 Claude Code向け core 49個、Codex向け core 45個です。
 元Skills（49個）の内訳は次のとおりです。
   - Pull Request とレビュー (7): pr, pr-tests, fix, pr-review, cherry-pick-pr, deploy, merged

conftest.py

@@ -55,8 +55,9 @@
 # 増えた）。**生成物は収集の対象から外す。**
 collect_ignore_glob = ["plugins/*/dev.agy"]
 
-# 束ごとに要る外部コマンド。**束によって前提が違う**ため、一覧は 1 か所に置きつつ
-# 収集した束にだけ適用する。全体へ課すと、必要のないコマンドを求めることになる。
+# 束ごとに要る外部コマンド（前提のコマンドの一覧）。**束によって前提が違う**ため、一覧は
+# 1 か所に置きつつ収集した束にだけ適用する。全体へ課すと、必要のないコマンドを求めることになる。
+# キーはリポジトリの根からの相対パスで、ファイルの中のクラスまで絞るときは `::` でつなぐ。
 REQUIRED_COMMANDS: dict[str, tuple[str, ...]] = {
     "scripts/tests": ("bash", "jq", "git"),
     "plugins/ndf/skills/worktree/tests": ("bash", "jq", "git"),
@@ -72,6 +73,9 @@
     # スライドの生成スクリプト（pptxgenjs）を node で実行し、依存が無ければ `npm ci` で入れる（#2065）
     "plugins/ndf/skills/release/tests/test_slide_template.py": ("node", "npm"),
     "tests/test_presentation_decks.py": ("node", "npm"),
+    # 名前つきのラッパーを使い捨ての tmux サーバで動かす結合テスト（#2346）。同じファイルの
+    # ほかのテストは tmux を使わないため、クラスまで絞る。
+    "plugins/ndf/scripts/tests/test_relay_hub.py::TestWithTmux": ("tmux",),
 }
 
 # 読み飛ばしを選ぶ指定。**既定は失敗**である。読み飛ばしたい実行環境のために残すが、
@@ -83,16 +87,27 @@ def _allow_missing() -> bool:
     return os.environ.get(ALLOW_MISSING_ENV, "") not in ("", "0")
 
 
-def _bundle_of(path: Path) -> str | None:
-    """その項目が属する束を返す。一覧に無ければ `None`。"""
-    try:
-        rel = path.resolve().relative_to(ROOT).as_posix()
-    except ValueError:
+def _item_id(item) -> str | None:
+    """項目の識別子（根からの相対パス、続けてファイルの中の位置を `::` でつないだもの）を返す。
+
+    根の外のファイルは `None`。
+    """
+    path = Path(str(item.fspath)).resolve()
+    if not path.is_relative_to(ROOT):
         return None
-    for bundle in REQUIRED_COMMANDS:
-        if rel == bundle or rel.startswith(bundle + "/"):
-            return bundle
-    return None
+    _, sep, inner = item.nodeid.partition("::")
+    return path.relative_to(ROOT).as_posix() + sep + inner
+
+
+def _bundles_of(item_id: str | None) -> list[str]:
+    """識別子が当たる一覧のキーを返す。
+
+    当たるのは、識別子がキーと一致するか、キーの直後が `/` か `::` のときだけである
+    （`::TestWithTmux` は `::TestWithTmuxOld` に当たらない）。
+    """
+    if item_id is None:
+        return []
+    return [k for k in REQUIRED_COMMANDS if item_id == k or item_id.startswith((k + "/", k + "::"))]
 
 
 def _missing(bundles: set[str]) -> dict[str, list[str]]:
@@ -213,15 +228,15 @@ def _select_shard(config, items) -> None:
 
 def pytest_collection_modifyitems(config, items) -> None:
     _select_shard(config, items)
-    bundles = {b for item in items if (b := _bundle_of(Path(str(item.fspath)))) is not None}
+    bundles = {b for item in items for b in _bundles_of(_item_id(item))}
     missing = _missing(bundles)
     if not missing:
         return
     lines = [f"{bundle}: {' '.join(commands)}" for bundle, commands in missing.items()]
     if _allow_missing():
         mark = pytest.mark.skip(reason=f"外部コマンドが無い（{ALLOW_MISSING_ENV} の指定により読み飛ばす）")
         for item in items:
-            if _bundle_of(Path(str(item.fspath))) in missing:
+            if any(b in missing for b in _bundles_of(_item_id(item))):
                 item.add_marker(mark)
         return
     raise pytest.UsageError(