Skip to content

Remove PR approval gate + add stream-json output#26

Closed
juhgiyo wants to merge 2 commits intomainfrom
remove-pr-approval-gate
Closed

Remove PR approval gate + add stream-json output#26
juhgiyo wants to merge 2 commits intomainfrom
remove-pr-approval-gate

Conversation

@juhgiyo
Copy link
Copy Markdown
Contributor

@juhgiyo juhgiyo commented Apr 3, 2026

Summary

  • Remove PR approval gate from CLAUDE.md workflow
  • Add --output-format stream-json to Claude Code command for structured output

Test plan

  • Verify Claude Code CLI uses stream-json format
  • Verify CI workflow runs without approval gate

🤖 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
Copy link
Copy Markdown
Contributor Author

juhgiyo commented Apr 4, 2026

Duplicate — already merged as PR #25.

@juhgiyo juhgiyo closed this Apr 4, 2026
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