docs: record Claude App + cross-account prereqs in caller template#15
Conversation
…ler template The reviewer's tag mode posts via the Claude GitHub App, so the API key alone is not enough — the app must be installed on the account. Also note that a private reusable workflow can only be called within one account; a repo in another org needs a standalone copy. Both gaps surfaced while wiring up the first adopters. ADR 0021 left as-is (append-only); the operational prereqs live in the template. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Claude finished @vpciii's task in 51s —— View job ReviewChecked the change against ADR 0021, the reusable workflow it documents, and the repo's Conventional Commits convention. Both new prerequisites are accurate. The reusable workflow sets Process is clean. Doc-only change, single Conventional Commit ( One optional follow-up (non-blocking): ADR 0021's Consequences ( Nice first dogfood PR for the reviewer. 🙂 LGTM. |
Fourth run: design-mode review of the dedup PR returned PUSH BACK — 2 real bugs the removal introduced (dropped settings, stale README) plus a fair generate-vs-remove design challenge. Bugs fixed; design answered, not reversed (removal held on scale-to-work grounds, ADR 0008 made honest). 4 runs / 2 repos / 0 false positives / verdict scale discriminating honestly — meets the README's graduation bar. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Fixes the two gaps that surfaced wiring up the first adopters:
ANTHROPIC_API_KEYalone isn't enough (the first smoke test 401'd until the app was installed).vpciii/methodology's; it needs a standalone copy.Updates
templates/github/claude-review.ymlprereqs accordingly. ADR 0021 left as-is (append-only); the operational prereqs belong in the template.This is also the first PR the methodology repo's own reviewer should comment on. 🙂
🤖 Generated with Claude Code