Skip to content

agy のエージェント定義: agy 1.4 系でも 11 本すべてが読み込まれ、agy の入った手元で push が通る(#2087) - #2146

Merged
takemi-ohama merged 3 commits into
developfrom
design/issue-2087
Oct 25, 2026
Merged

takemi-ohama merged 3 commits into
developfrom
design/issue-2087

Conversation

@takemi-ohama

Copy link
Copy Markdown
Contributor

#2087 の要求・受け入れ条件の写しと設計文書を追加しました(スプリント m2087)。設計の内容は、agy に配る定義を生成した写しにし、model を tier 名へ変換するものです。この PR は文書だけを変更し、実装は含みません。

利用者向けの変化

  • 無し

移行の手順

  • 無し

変更の要点

テスト

supervise.py の run のステップの結果です。

ステップ exit 結果
glossary 0 ok。宣言と用語集が揃っている
requirements-check 2 stopped。issues/issue-2087-requirements.md を読めない([Errno 2] No such file o…)
  • requirements-check は失敗のままです。このファイルは変更の統計では追加済みです。
  • ステップが走った時点でファイルを読めなかった理由は、材料からは分かりません。マージの前に再実行して確認が要ります。

モード: standard / 通した工程: 設計 → 要求と受け入れ条件 → ドキュメントレビュー

🤖 Generated with Claude Code

決めたこと

issues/issue-2087-design.md

  • 決定 1: Claude Code の定義を変えずに agy だけの値を持たせるため、agy に配る定義を symlink から生成した写しへ変える
  • 決定 2: Claude Code で選び分けた費用の意図を agy でも保つため、model は外さずに対応表で tier 名へ変換する(承認ゲート 1 で人が決める)
  • 決定 3: 写しが元の定義と model の他で一致することを確かめやすくするため、frontmatter を YAML として読み直さず model: の 1 行だけを置き換える
  • 決定 4: 対応表に無い値を黙って通さないため、型の名前でない model は生成を止める

takemi-ohama and others added 2 commits October 25, 2026 10:27
agy 1.4 系がエージェント定義 8 本の model(sonnet / opus)を拒む件の要求。
agy に配る定義の形と model の扱いは未決として設計へ渡す。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
dev.agy/agents の symlink をやめ、scripts/lib/agy_agents.py が plugins/ndf/agents から
写しを生成する。model は対応表で sonnet→flash・opus→pro に変換する(外すかは承認ゲート 1 で決める)。
agy 1.4.7 と 1.3.4 で変換・外す両方の形が validate を通ることを実測した。
用語集に「agy に配る定義」「model の対応表」を足す。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 1 | claude | APPROVE

ドメインモデルの節(コンテキスト・集約・不変条件・ドメインイベント・用語)と用語集の差分に、基準に当たる修正点は無い。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 1 | claude-2 | APPROVE

ドメインモデルの節(コンテキスト・集約・不変条件 I1〜I6・ドメインイベント E1〜E5・用語)と用語集の差分に、critical / major に当たる修正点はありません。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 2 | claude | REQUEST_CHANGES

設計の「触るファイル」に、一時ツリーへ生成スクリプトを複製するテスト補助(scripts/tests/agy_build_helpers.py)が抜けている。指摘 1 件。

Comment thread issues/issue-2087-design.md Resolved

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 2 | claude-2 | APPROVE

要求・設計・用語集の差分(受け入れ条件 1〜9 とテスト設計の対応、symlink から生成した写しへの移行の順序、build-runtime-plugins.sh・validate-runtime-plugins.sh・README・確定仕様との整合)に、critical / major に当たる修正点はありません。

一時ツリーへ生成スクリプトだけを複製する build_tree では agy_agents.py が
見つからず生成テストが落ちるため、構成要素・構成・設計の結果へ足す。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@takemi-ohama

Copy link
Copy Markdown
Contributor Author

🔧 /ndf:fix サマリ | round 2 | commit 91ff98f

対応件数: critical=0 / major=1 / minor=0(合計 1 件)
決着: 1 件 / 見送り: 0 件 / 却下: 0 件
CI: PENDING

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 3 | claude | APPROVE

round 2 の指摘(テスト補助 agy_build_helpers.py の一時ツリーへの agy_agents.py の複製)が、構成要素・ディレクトリ構成・設計の結果の触るファイルの 3 か所にそろって反映されている。差分に critical / major に当たる修正点は無い。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 3 | claude-2 | APPROVE

round 2 の指摘への対応(触るファイルへの scripts/tests/agy_build_helpers.py の追加。構成要素の表・ファイル配置・設計の結果の 3 か所)に、critical / major に当たる修正点はありません。

@takemi-ohama

Copy link
Copy Markdown
Contributor Author

🔧 /ndf:fix サマリ | commit (なし)

対応件数: critical=0 / major=0 / minor=0(合計 0 件)
決着: 0 件 / 見送り: 0 件 / 却下: 0 件
CI: NONE

@takemi-ohama
takemi-ohama marked this pull request as ready for review October 25, 2026 10:38
@takemi-ohama takemi-ohama added the design-approved 設計 Pull Request の承認の印 label Oct 25, 2026
@takemi-ohama

Copy link
Copy Markdown
Contributor Author

承認ゲート 1: 利用者が 2026-10-25 に承認(設計 PR #2146・#2145 をまとめて 1 回で承認)。#2087 の未決 1 は決定 2(model を対応表で tier 名へ変換)で確定。

@takemi-ohama
takemi-ohama merged commit 373290f into develop Oct 25, 2026
23 checks passed
@takemi-ohama
takemi-ohama deleted the design/issue-2087 branch October 25, 2026 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

design-approved 設計 Pull Request の承認の印

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant