Skip to content

docs(plans): C4 exit record — NOT REACHED on not_configured; exit criterion is admin installs once, stranger grants + approves - #1714

Merged
lilyshen0722 merged 1 commit into
mainfrom
wren/c4-exit-record
Sep 16, 2026
Merged

lilyshen0722 merged 1 commit into
mainfrom
wren/c4-exit-record

Conversation

@lilyshen0722

Copy link
Copy Markdown
Contributor

Summary

C4 exit record in docs/plans/production-readiness-and-selling-2026-09-11.md, same shape as the C3 record (#1708), plus the C4 exit criterion rewritten to the wording Sam accepted in pod message 68649.

  • Run: 2026-09-16, throwaway account, commonly.me on 00e7ebc9. Tools opened 04:55:23Z, blocked 04:55:33Z (~10 s). NOT REACHED.
  • Why: the catalogue returns available: false, unavailableReason: not_configured, connections: []. The chart maps no GITHUB_APP_ID / GITHUB_APP_PRIVATE_KEY (Sam verified them unset in the running pod), and a github-app Integration row is admin-only, so no fresh account gets past the not-yet row.
  • Rows: TASK-033 GitHub App credentials (Sam; Kai's chart-mapping PR held like fix(helm): wire Discord webhook public key #1685 until the Secret Manager values exist), TASK-034 accepted as a product fact (member self-install waits on piece 6 §10), TASK-035 the inert not-enabled row gets the Ask affordance (Kai builds, UX Lead gate).
  • Exit criterion now: an admin installs the GitHub App once; a stranger grants a tool to a room and approves an action, timed from opening Tools to the approved action landing.

Docs only, one row. No code.

🤖 Generated with Claude Code

…terion is admin installs once, stranger grants + approves

Records the 2026-09-16 stranger run on the Tools page (Sam 68637, rulings 68649):
blocked in ~10 s because the GitHub tool is not_configured on commonly.me and
github-app rows are admin-only. Files TASK-033/034/035 in the row and rewrites
the C4 exit criterion to the redefinition Sam accepted.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@lilyshen0722
lilyshen0722 merged commit d390ca0 into main Sep 16, 2026
13 checks passed
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