Skip to content

Let permission hooks resolve strict auto-review requests - #32232

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/18e9d76baee6cbae9d35ae517250f2f82270a9d2
Jul 10, 2026
Merged

Let permission hooks resolve strict auto-review requests#32232
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/18e9d76baee6cbae9d35ae517250f2f82270a9d2

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 10, 2026

Copy link
Copy Markdown

Let permission hooks resolve strict auto-review requests

What changed

  • Run permission-request hooks before routing approvals to the automated reviewer or user, including when strict_auto_review is enabled.
  • Centralize approval resolution so hook, automated-reviewer, and user decisions share rejection handling and report the correct telemetry source.

Testing

  • Add an integration test showing that an allow hook can approve a shell command during strict auto-review without invoking the automated reviewer.

## What changed

- Run permission-request hooks before routing approvals to the automated reviewer or user, including when `strict_auto_review` is enabled.
- Centralize approval resolution so hook, automated-reviewer, and user decisions share rejection handling and report the correct telemetry source.

## Testing

- Add an integration test showing that an allow hook can approve a shell command during strict auto-review without invoking the automated reviewer.

GitOrigin-RevId: 18e9d76baee6cbae9d35ae517250f2f82270a9d2
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/18e9d76baee6cbae9d35ae517250f2f82270a9d2 branch from d1f853c to 2da1b12 Compare July 10, 2026 18:13
@copyberry
copyberry Bot merged commit 2da1b12 into main Jul 10, 2026
14 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/18e9d76baee6cbae9d35ae517250f2f82270a9d2 branch July 10, 2026 18:14
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants