Skip to content

AGENTS.md にレビュー指摘の再発防止方針を追加する - #94

Merged
tokuhirom merged 1 commit into
mainfrom
docs/agent-review-prevention
Oct 9, 2026
Merged

tokuhirom merged 1 commit into
mainfrom
docs/agent-review-prevention

Conversation

@tokuhirom

Copy link
Copy Markdown
Member

概要

レビューで確認された問題の再発を防ぐため、AGENTS.md の開発・検証ルールを補います。実装コードの変更は含みません。

  • ヘルプと実際のフラグ・JSON 専用項目・入力制約の照合を追加します。
  • ドメイン単体テストの正常系・エラー系と、生成型名全体の衝突検証を明記します。
  • PLAN.md の対応範囲の記載と、ADR 番号・参照の確認を補います。
  • JSON・ファイル入力、通信の期限・キャンセル、モックのエラー分類の検証方針を追加します。
  • GitHub Issue のタイトルと本文を日本語で記述するルールを追加します。

関連 Issue: #75、#82、#83、#84、#85、#86、#87、#88、#89、#90、#91、#92、#93。各実装対応は別途行います。この PR ではこれらの Issue を閉じません。

確認

  • make lint-text
  • git diff --check

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Tokuhiro Matsuno <tokuhirom@gmail.com>
Copilot AI balanced review requested due to automatic review settings October 9, 2026 09:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The documentation-only changes are internally consistent and match the stated scope and related issues.

0 open findings

What changed in this PR

レビュー指摘の再発防止に向け、開発・検証ルールを明文化する文書変更です。

Changes:

  • CLI 入力、ヘルプ、PLAN・ADR の確認ルールを追加
  • 生成型名の衝突検証とドメイン単体テスト方針を追加
  • JSON 入力、通信、API モックの検証方針を追加
File Description
AGENTS.md 開発・検証時の再発防止ルールを追加

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@tokuhirom
tokuhirom merged commit 2948726 into main Oct 9, 2026
9 checks passed
@tokuhirom
tokuhirom deleted the docs/agent-review-prevention branch October 9, 2026 13:52
@github-actions github-actions Bot mentioned this pull request Oct 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants