31 lines
1.1 KiB
Markdown
31 lines
1.1 KiB
Markdown
# PR#153: The Codex Review Advises on Werkator
|
|
|
|
> **WARNING:** This document describes only the change applied in this PR.
|
|
> It may already be outdated once the next PR is merged.
|
|
> Historic PR documentation is not maintained along with new PRs; treat it as a snapshot.
|
|
|
|
## Related Links
|
|
|
|
- [werkrevisor PR#88](https://git.javagil.de/mi/werkrevisor/pulls/88): the runner of the Codex review and `[codex] mode`.
|
|
|
|
## Problem
|
|
|
|
werkrevisor can let a runner on the workstation of the user review PRs with the Codex CLI and a frontier model.
|
|
On 10 merged hs.hsadmin.ng PRs, the Codex review flagged all 5 PRs that the user had classified as critical;
|
|
in two of these 5 it found a real RBAC defect.
|
|
A repository takes part only if its `.werkrevisor.toml` chooses it; the user chose Werkator to start.
|
|
|
|
## Non-Goals
|
|
|
|
- No `gate`: the Codex review only advises and holds no merge.
|
|
|
|
## Solution
|
|
|
|
- `.werkrevisor.toml`: `[codex] mode = "advise"`; werkrevisor 0.34.0 on mih09 accepts the section.
|
|
- Werkbaum: the node "Codex review by the runner of werkrevisor, advise only" below `#process`, merged (`[x]`).
|
|
- Version v1.41.4, internal.
|
|
|
|
## Open Questions
|
|
|
|
None.
|