Skip to content

cross-review: ai-plugins 以外のリポジトリで SKILL.md のとおりに単独起動すると、Skill のディレクトリが git リポジトリでないため PR を取れずに止まる #1707

Open

Description

@takemi-ohama

何を見つけたか

cross-review の SKILL.md は「Skill のディレクトリで」python3 scripts/drive.py <PR> を打つと書くが、実装は対象のリポジトリを現在のディレクトリ(cwd)から決める。インストール先の Skill のディレクトリ(~/.claude/plugins/cache/ai-plugins/ndf/*/skills/cross-review)は git リポジトリではないため、ai-plugins 以外のリポジトリで書かれたとおりに打つと、origin を読めずに止まる。ai-plugins では Skill のディレクトリが対象のリポジトリの中にあるため、偶然動く。

どこで見つけたか

  • plugins/ndf/skills/cross-review/SKILL.md:133(「Skill のディレクトリで次の 1 行を打ち」)
  • plugins/ndf/skills/cross-review/scripts/review_lib/github.py:53(_git_remote_url が cwd の origin を読む)
  • plugins/ndf/skills/cross-review/scripts/review_lib/store.py:52、review_lib/participants.py:167

#1655(cross-refactoring の単独起動の打つ場所)の設計の中で、同じ形の依存として確かめた。

なぜこの変更の範囲外なのか

#1655 の要求の前提 5 と対象範囲(含まない)が、cross-review の単独起動の打つ場所を別の課題で扱うと決めている。#1655 は cross-refactoring の単独起動だけを直し、最終ゲートが打つ cross-review の駆動は対象のリポジトリを cwd にして打つ形で回避する。根拠: Value 5(MVV 版 2)

直さないと何が起きるか

ai-plugins 以外のリポジトリで /ndf:cross-review を単独で使う利用者が、SKILL.md のとおりに打つと PR を取れずに止まる。#1655 の設計が足す「cwd が git リポジトリでなければ対象のリポジトリを決められないと示して止まる」検査(共通ライブラリの proc.git_root)と、SKILL.md の入口の書き方(解決の入口 resolve.sh scripts cross-review)を同じ形で当てれば直る見込みである。

由来

#1655 の設計(issues/issue-1655-design.md)

Metadata

Metadata

Assignees

No one assigned

    Labels

    area: ndf-skillNDF の Skill 本体bugSomething isn't workingpriority: medium保守性・設計一貫性など、計画的に対応する

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions