Skip to content

Conversation

@hakatashi
Copy link
Member

@hakatashi hakatashi commented Jul 7, 2025

https://x.com/aidenybai/status/1923781810820968857

AI instruction file の命名を統一しようとするAmpの取り組みにならい、指示内容をAGENT.mdに移動させ、各ファイルからはこちらを参照するように修正した。

@hakatashi hakatashi force-pushed the single-instructions-file branch from 43a74fe to 39b68e6 Compare July 7, 2025 08:34
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 7, 2025

@hakatashi
Copy link
Member Author

@gemini-cli /review

@tsg-ut-gemini-assistant
Copy link

📋 概要

このPRは、各種AIアシスタント向けの指示を新しく作成されたAGENT.mdファイルに集約し、メンテナンス性と一貫性を向上させるものです。全体として、AIを活用した開発体験を標準化するための素晴らしい改善であり、非常によく構成されています。

🔍 フィードバック

AI開発の指示をAGENT.mdに一元化するというアプローチは非常に優れています。これにより、指示の重複が排除され、将来の更新が容易になります。また、関連するワークフローやnpmスクリプトの変更も、この目的を達成するために適切に実装されています。

🎯 具体的なフィードバック

今回のレビューでは、修正が必須となるような重大な問題は特に見つかりませんでした。

🔵 Low

  • File: .github/workflows/agent-pr-review.yml:29 - 現在の実装では、curldpkgを使ってgh CLIをインストールしていますが、より標準的な方法としてgithub/gh-actionのような公式セットアップアクションの利用を検討しても良��かもしれません。必須の変更ではありませんが、ワークフローの可読性と将来のメンテナンス性が向上する可能性があります。

✅ ハイライト

  • AGENT.mdの導入: AIアシスタントとの対話方法を標準化するためのAGENT.mdの作成は、非常に優れた取り組みです。これにより、プロジェクトへの新規参画者(AIアシスタントを含む)が迅速にキャッチアップできるようになります。
  • ワークフローの改善: agent-pr-review.ymlで、レビューコメントを環境変数から--body-file経由で渡すように変更した点は、堅牢性を高める良い改善です。
  • npmスクリプトの追加: agent:pr-reviewというnpmスクリプトを追加したことで、AIによるレビュープロセスが形式化され、手動での実行が容易になりました。
  • bin/slack-verify.tsの改善: 検証成功時のメッセージにアプリ設定へのリンクを追加した点は、開発者体験を向上させる細やかな配慮であり、素晴らしいです。

@hakatashi
Copy link
Member Author

そういえばこれマージしたいんだけど、またプロジェクトのルートディレクトリが汚くなるからマージしたくないんだよな どうしよう

@hakatashi
Copy link
Member Author

@copilot 上のコメントで言及した懸念点について、アドバイスはありますか?

Copy link
Contributor

Copilot AI commented Dec 21, 2025

@hakatashi I've opened a new pull request, #1111, to work on those changes. Once the pull request is ready, I'll request review from you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

2 participants