Skip to content

Remove PR approval gate from CLAUDE.md workflow#25

Merged
juhgiyo merged 2 commits intomainfrom
remove-pr-approval-gate
Mar 27, 2026
Merged

Remove PR approval gate from CLAUDE.md workflow#25
juhgiyo merged 2 commits intomainfrom
remove-pr-approval-gate

Conversation

@juhgiyo
Copy link
Copy Markdown
Contributor

@juhgiyo juhgiyo commented Mar 27, 2026

Summary

  • Removes the manual PR approval gate from the CLAUDE.md workflow
  • Agents now proceed directly to rebase/commit/PR after completing code changes, without waiting for explicit approval of the diff
  • The 4-step workflow (rebase → push to open PR or create new branch → open PR → never reuse merged branches) is preserved, just without the approval gate

Test plan

  • Verify CLAUDE.md renders correctly in each repo
  • Confirm agents follow the updated workflow without waiting for approval

🤖 Generated with Claude Code

juhgiyo and others added 2 commits March 23, 2026 16:33
Enable stream-json output format so Claude Code emits JSONL to stdout,
including a final "result" message with token usage data. This is the
prerequisite for cost tracking in StrawPot sessions.

Part of strawpot#422 (Cost Observability Phase 1).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@juhgiyo juhgiyo merged commit 3e61a74 into main Mar 27, 2026
1 check passed
@juhgiyo juhgiyo deleted the remove-pr-approval-gate branch March 27, 2026 23:42
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