Follow-up from the Phase 7 (PR #2) merge review.
Gap
Phase 7 stopping criterion: "requires-auth MCPs surface a hint to run the relevant login flow."
render-row currently appends only " [connect]" to requires-auth rows — it does not name what login flow / command the user should run. The hint is present but uninformative.
Constraint
Per PR #2 notes, MCPServerUpdatedParams carries no per-MCP provider mapping, so there is no direct field to derive the flow name from. Resolving this may need a name→provider convention or an ECA protocol addition.
Acceptance
requires-auth rows tell the user the concrete action (e.g. the /login command or the provider name), not a bare [connect].- Test asserts the row text names the flow.
Follow-up from the Phase 7 (PR #2) merge review.
Gap
Phase 7 stopping criterion: "
requires-authMCPs surface a hint to run the relevant login flow."render-rowcurrently appends only" [connect]"torequires-authrows — it does not name what login flow / command the user should run. The hint is present but uninformative.Constraint
Per PR #2 notes,
MCPServerUpdatedParamscarries no per-MCP provider mapping, so there is no direct field to derive the flow name from. Resolving this may need a name→provider convention or an ECA protocol addition.Acceptance
requires-authrows tell the user the concrete action (e.g. the/logincommand or the provider name), not a bare[connect].