Skip to content

docs: record centralize-claude-assistant plan with post-deployment fixes - #15

Merged
twistedmelonman merged 1 commit into
mainfrom
claude/docs-plan-postmortem-20260309
Mar 9, 2026
Merged

twistedmelonman merged 1 commit into
mainfrom
claude/docs-plan-postmortem-20260309

Conversation

@twistedmelonman

Copy link
Copy Markdown
Member

Commits the plan document for the centralize-claude-assistant effort, updated to reflect what actually shipped including the two post-deployment fixes (v1.2.1 and v1.2.2).

Key corrections from the original draft:

  • File contents updated to match the final working versions
  • Constraints section corrected: reusable workflow permissions: overrides (not inherits) caller permissions
  • Documents actions: read → startup_failure failure mode
  • Documents id-token: write must be in the reusable workflow block, not just the caller

🤖 Generated with Claude Code

Documents the two issues discovered after initial deployment:
- actions: read causing startup_failure (fixed in v1.2.1)
- id-token: write missing from reusable workflow (fixed in v1.2.2)

Updates file contents to match what actually shipped and corrects the
misleading constraint note about permissions inheritance.
@twistedmelonman
twistedmelonman merged commit 630ae23 into main Mar 9, 2026
4 checks passed
@twistedmelonman
twistedmelonman deleted the claude/docs-plan-postmortem-20260309 branch March 9, 2026 17:16
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