Skip to content

feat: a missing GitHub App installation gets the install link, not a console page - #5

Closed
spacedragon wants to merge 1 commit into
mainfrom
dev/yulong/install-url-guidance
Closed

spacedragon wants to merge 1 commit into
mainfrom
dev/yulong/install-url-guidance

Conversation

@spacedragon

Copy link
Copy Markdown
Collaborator

What

When the create-agent flow (code-reviewer template) finds no GitHub App installation covering the repository's owner, it now outputs the actual install URL instead of directing the user to a console page.

  • references/templates/code-reviewer.md P1: call the admin MCP's new getGithubApp and hand the user its installUrl verbatim, with a one-line explanation (bound to this org, single-use, valid ~15 minutes, GitHub returns them to the console). Offer to wait via a URL-mode card, then re-run listGithubInstallations and continue. Spells out the fallbacks: 403 → an editor/owner must open the link; tool absent (older Control Plane) → the console's Workspace → GitHub editor shows the same link. Forbids hand-building github.com/apps/<slug>/installations/new, which lacks the signed state the setup callback requires.
  • SKILL.md: getGithubApp joins the tool list and the prerequisite step.
  • agentconnect-platform/references/admin-operations.md: catalog row for getGithubApp.

The rest of the diff is Prettier re-padding two tables.

Why

Seen in a webchat session: with the App not yet installed, the agent could only point at a console tab, because no admin tool returned the install link. The link is minted per call with a signed org-bound state, so the skill cannot construct it; it needs the tool.

Depends on

agentconnect-md/agentconnect PR adding the getGithubApp MCP read tool (GET /github/app). Until that ships, the skill's documented fallback (console Workspace → GitHub) applies.

🤖 Generated with Claude Code

…console page

When listGithubInstallations shows nothing covering the repository's owner,
the code-reviewer template now calls the admin MCP's getGithubApp and hands
the user its installUrl verbatim: the org-bound, single-use, ~15-minute link
GitHub needs to bind the installation to this organization. It spells out
the 403 (viewer cannot mint) and tool-absent (older Control Plane) fallbacks,
and forbids hand-building the github.com/apps/<slug> URL, which would lack
the signed state. The platform skill's tool catalog gains the same row.

Pairs with agentconnect-md/agentconnect: the getGithubApp MCP read tool.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
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.

1 participant