セキュリティ調査で検出された問題を直すpull requestの状態を、トリアージページと調査レポートに反映する。codepatrol skillが起動したsubagentが実行する。ユーザーが直接呼び出す事は想定していない。
The English summary is being prepared. Showing the original description.
See how to installReview the instructions the agent will receive before you install.
codepatrol skillの指揮役から任されて、問題を直すpull requestの実際の状態を、トリアージページと調査レポートに反映する。
修正の作業は、複数の人間とAIが並行して進める。ページに書かれた状態は、実際の状態からずれていく。mergeされたのにレビュー希望のまま残る、他の人が着手したのにページに書かれていない、片方のページにだけ書かれている、といったずれである。ずれが残ると、次の調査やトリアージが、対応済みの問題を未対応だと誤認する。
起点は、指揮役から渡されたトリアージページである。レポートは、トリアージページの各行からリンクされている物を対象にする。
トリアージページの冒頭に書かれた編集ルールを、着手する前に読む。 状態アイコンの意味と、書いてよい事・書いてはいけない事は、そこで決まっている。この手順書と食い違う時は、ページの記述を優先する。
トリアージページが無い時は、hubページに集約された、各領域の最新のレポートを対象にする。
ページにあるpull requestの行について、pull requestの実際の状態を確かめ、状態アイコンを合わせる。印が表すのはpull requestの状態で、問題が解決したかどうかではない。
| その行のpull requestの状態 | 印 |
|---|---|
| 全てmergeされている | 完了 |
| draftの物を含む | 作業中 |
| 上のどちらでもなく、openである | レビュー希望 |
問題を直すpull requestのうち、ページに載っていない物を探す。見落としは、この方向で起きる。実装者がページに書かずに修正を出す事がある。
載っていないpull requestは、概要欄を読み、どの問題の修正かを判断する。問題の修正であれば、状態アイコンを付けて、トリアージページとレポートの両方に足す。問題の修正でなければ、足さない。
前回の同期の時期は、探す範囲を絞る手掛かりになる。ただし、前回より前に作られたpull requestが、後から問題の修正になる事がある。日付で絞る時は、タイムゾーンにも気を付ける。日付の境目の前後に作られたpull requestが漏れる事がある。
完了の印が付いた行について、残タスクが無いかを確かめる。pull requestがmergeされても、問題が全て解決したとは限らない。一部だけを直した、別の課題として残した、既存のデータの棚卸しが要る、実環境の確認が要る、といった場合である。残タスクは、完了の印を外さずに、1行のサマリーで表す。
人間が並行してページを編集している。
次の事を見つけた時は、ページを直さず、指揮役に報告する。
次の3つに分けて報告する。
加えて、次の事を報告する。
No reviews yet. Tell others what you thought.
No login needed. Share how it worked for you, ask a question, or note where it did not run.
No comments yet. Be the first to write one.
Agent Skillを書く時の心得。SKILL.mdに何を書き、何を書かないかの判断基準を定める。AIが読んで従う文書に共通の心得は、kuden:common-rules-for-agent-instructionsにある。 Agent Skillを作成する時、編集する時、レビュー指摘の採否を決める時に読み込む。
The English summary is being prepared. Showing the original description.
ソースコード中のコメントを書く時の心得。コメントに何を書き、何を書かないかの判断基準と、置く位置、日本語の文体を定める。 ソースコードのコメントを書く時、直す時、レビューする時に読み込む。
The English summary is being prepared. Showing the original description.
リポジトリのセキュリティ調査を領域ごとに進める。同じサービスを構成する複数のリポジトリを束ねて調査できる。 このsessionは調査を指揮し、領域ごとに起動したsubagentが調査してレポートを出力する。 レポートの問題のトリアージと、問題の自動修正、問題を直すpull requestの状態の同期も指揮する。 複数sessionにまたがる長期作業を想定し、実行するたびに現状を確認して続きの作業を行う。 ユーザーが手動で起動する。
The English summary is being prepared. Showing the original description.
セキュリティ調査で検出された問題を1つ修正し、pull requestをready for reviewまで仕上げる。 codepatrol skillが起動したsubagentが実行する。ユーザーが直接呼び出す事は想定していない。
The English summary is being prepared. Showing the original description.
本番に出る前の修正が積まれたrelease pull requestに、デプロイの前後に人間がやる事をまとめたコメントを投稿・更新する。 codepatrol skillが起動したsubagentが実行する。ユーザーが直接呼び出す事は想定していない。
The English summary is being prepared. Showing the original description.
リポジトリの1つの領域をセキュリティ観点で調査し、Codexの批判的レビューを経てレポートを出力する。 codepatrol skillが起動したsubagentが実行する。ユーザーが直接呼び出す事は想定していない。
The English summary is being prepared. Showing the original description.