Skip to content

Remove PR approval gate from CLAUDE.md workflow#76

Merged
juhgiyo merged 4 commits into
mainfrom
remove-pr-approval-gate
Mar 27, 2026
Merged

Remove PR approval gate from CLAUDE.md workflow#76
juhgiyo merged 4 commits into
mainfrom
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 4 commits March 25, 2026 09:36
Replace placeholder demo video with a real recording based on an actual
strawpot session run. The terminal scene replays captured output from
`strawpot start --task` showing ai-ceo delegation to implementer and
code-reviewer, matching the website's output block.

- Title cards generated with Pillow + ffmpeg (no drawtext filter needed)
- Terminal scene recorded with VHS (Catppuccin Mocha, 1920x1080)
- Scenes stitched with ffmpeg xfade crossfade transitions
- Final: 52.6s, 696KB, 1920x1080 @ 30fps, h264/yuv420p
- Simplified from 5 scenes to 3 (opening + terminal + closing CTA)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The previous demo showed raw ANSI escape codes as text because VHS's
Hide/Show + printf technique was broken. Replaced with a mock binary
approach: VHS runs .demo-video-work/bin/strawpot which outputs colored
text naturally via echo -e. Also suppresses gRPC debug noise.

- Terminal colors now render correctly (green checkmarks, yellow arrows,
  magenta/cyan role prefixes, bold text, dim timing)
- No raw \033 escape codes visible
- No gRPC WARNING/fork_posix log lines leaking
- 54.2s total, 551KB, 1920x1080, Catppuccin Mocha theme

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace fabricated demo with authentic CLI output that exactly matches
the real TerminalProgressRenderer (progress.py) in TTY mode.

Changes:
- Mock binary now replays exact progress renderer output (no fake
  sub-agent activity streaming, no invented code-reviewer delegation)
- Uses real markers: Unicode checkmark for TTY, correct indentation
- Includes Session Recap summary (changes table, test results, PR)
- Shortened from ~54s to ~23s (3s title + 18s terminal + 3s CTA)
- Updated demo-script.yaml with accurate scene descriptions

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 7ede530 into main Mar 27, 2026
1 check passed
@juhgiyo juhgiyo deleted the remove-pr-approval-gate branch March 27, 2026 23:41
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