Skip to content

Sync the shared review stopping rules - #49

Merged
raghubetina merged 1 commit into
mainfrom
review-amendment-sync
Oct 1, 2026
Merged

raghubetina merged 1 commit into
mainfrom
review-amendment-sync

Conversation

@raghubetina

@raghubetina raghubetina commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The commit, pull request, review, and landing rules are shared across firstdraft/firstdraft, firstdraft/cli, and firstdraft/skills. The service repository is changing two of them to cut duplicate review sessions:

  • An amendment that only applies accepted findings no longer needs a re-review.
  • A review whose only trigger is an instruction-policy change runs once at --effort high instead of max.

This keeps this copy in step, so following it does not bring the extra sessions back. This repository's own review triggers are unchanged.

Docs: updated CONTRIBUTING.md.

Review: codex-review 0.7.3, high effort, session 35ddc7e9-d56a-41e6-9f82-4e91b2c58ef3, approve.

The commit, pull request, review, and landing rules are shared with
firstdraft/firstdraft and firstdraft/skills, and the service
repository is changing two of them to cut duplicate review sessions.
An amendment that only applies accepted findings no longer needs a
re-review, and a review whose only trigger is an instruction-policy
change runs once at high effort instead of max. Keep this copy in
step so following it does not bring the extra sessions back.
@raghubetina
raghubetina merged commit d9ae7e4 into main Oct 1, 2026
4 checks passed
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