feat: ReviewSessionMeta for structured review results (closes #455)#456
Merged
RyderFreeman4Logos merged 3 commits intomainfrom Mar 22, 2026
Merged
feat: ReviewSessionMeta for structured review results (closes #455)#456RyderFreeman4Logos merged 3 commits intomainfrom
RyderFreeman4Logos merged 3 commits intomainfrom
Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…sults
Write machine-readable review metadata (session_id, head_sha, verdict,
decision, tool, scope, exit_code, fix_attempted, fix_rounds) to
{session_dir}/review_meta.json after csa review completes.
Meta is updated after each --fix round, enabling downstream consumers
(pr-codex-bot, commit skill, orchestration scripts) to programmatically
access review results without parsing free-form text output.
Closes part of #455 — provides the structured review session metadata
that enables the reviewer-fixes-own-findings workflow.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…iew and pr-codex-bot - csa-review: Add "Review-then-Fix via --fix" section documenting the CLI --fix flag, review session resume, and review_meta.json format - pr-codex-bot: Update Steps 2 and 8 to use --fix for reviewer-fixes-own-findings pattern, referencing review_meta.json for SHA-verified fast-path Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ReviewSessionMetastruct tocsa-session— written to{session_dir}/review_meta.jsonafter everycsa reviewrun--fixround for downstream consumers--fixworkflow andreview_meta.jsonincsa-reviewandpr-codex-botskills--fixalready existed — clarified and closedKey changes
csa-session/src/state.rsReviewSessionMetastruct +write_review_meta()csa-session/src/state_tests.rscli-sub-agent/src/review_cmd.rspersist_review_meta()called after review + each fix roundcsa-review/SKILL.mdpr-codex-bot/SKILL.md--fixandreview_meta.jsonTest plan
just test— 2766 tests passjust clippy— cleancsa review --branch main— PASSCloses #455
🤖 Generated with Claude Code