[codex] Revert non-compliant PR #10 - #11
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
EvoZeus Labeler ReportPR type/risk labels were updated from changed files. Surfaces
Labels
Changed lines
|
EvoZeus Queue GuardMode: dry-run Author open PRs: 1 Branch family: codex/docs/20260618-governance-revert-pr10 Branch family open PRs: 1 Violations
Next action
|
EvoZeus Privacy ScanMode: dry-run Findings
Next action
|
EvoZeus Dirty PR CheckMode: dry-run Surfaces
Reasons
Next action
|
EvoZeus Proof GateMode: dry-run Proof labels
Missing or weak fields
Next action
|
|
Process correction note: This PR intentionally reverts #10 because #10 was merged after automation marked it as Current state:
After this rollback is approved and merged, the reverted content should be resubmitted as smaller PRs using the appropriate specialized templates:
|
|
Process blocker update: Attempted to request CODEOWNERS review from GitHub rejected the request:
Observed state:
Correct next step is repository maintenance: make the CODEOWNERS team a collaborator / valid review target, or update CODEOWNERS to a valid maintainer reviewer. The authoring agent should not admin-merge this rollback PR. |
Summary
What problem does this PR solve?
This PR reverts #10 because it was merged without following the repository development rules for protected skill and governance surfaces.
Why does this matter now?
#10 touched
SKILL.md,skills/, governance docs, README docs, and factor docs in one PR. The bot labeled ittriage:dirty-pr,triage:owner-only,proof:needed, andrisk:skill-entry, but it was admin-merged before owner review.What is the intended outcome?
Restore
mainto the pre-#10 state so the changes can be resubmitted as smaller PRs using the correct templates and review gates.What is intentionally out of scope?
This PR does not reintroduce the factor registry governance docs or the
EvoZeus-...local skill naming change. Those should be separate follow-up PRs.What should reviewers focus on?
Confirm this is a clean revert of #10 and that it does not revert unrelated work from #9 or the factor repo split merge.
Linked Context
Closes #
Related #10
Was this requested in an issue, review, or maintainer discussion?
Requested in maintainer discussion after #10 was identified as non-compliant with EvoZeus development rules.
PR Scope
Primary layer:
Primary artifact kind:
Rollback of a non-compliant protected-path PR.
One thing this PR does:
Reverts merge commit
0d09febfrom #10.What this PR does not do:
It does not re-submit the reverted changes in a compliant split form.
EvoZeus Evidence Proof
triage:dirty-pr,triage:owner-only,proof:needed, andrisk:skill-entry.codex/docs/20260618-governance-revert-pr10.git revert -m 1 0d09feb --no-edit;git diff --check origin/main..HEAD;python3 -m py_compile scripts/check_pr_ready.py;python3 scripts/check_pr_ready.py --base origin/main --allow-cross-layer.6513cca.Verdict / Review Gate
Proposed verdict, if this PR adds or changes a Case or Candidate:
Review gates checked:
Tests and Validation
Which commands did you run?
git diff --check origin/main..HEADpython3 -m py_compile scripts/check_pr_ready.pypython3 scripts/check_pr_ready.py --base origin/main --allow-cross-layerWhat changed after running those checks?
No additional edits were needed after validation.
If no test or script was run, why not?
Not applicable.
Risk Checklist
Did user-visible behavior change? (
Yes/No)Yes. This removes the #10 user-facing docs and skill naming changes from
main.Did SKILL.md, public instructions, or agent behavior change? (
Yes/No)Yes. This reverts the #10 instruction changes.
Did privacy, security, network, auth, or tool permission behavior change? (
Yes/No)No.
Did schema, storage, branch, PR, or governance behavior change? (
Yes/No)Yes. This reverts governance documentation introduced by #10.
Highest-risk area:
Protected-path rollback touching
SKILL.mdandskills/.How is that risk mitigated?
This PR is a mechanical Git revert of #10 and is left for owner review. It should not be admin-merged by the authoring agent.
AI-Assisted Work
Was this PR written or materially changed by an AI coding agent? (
Yes/No)Yes.
If yes, which agent/tool and what did it do?
Codex generated the revert commit and prepared this PR after verifying the repository rules.
Confirm the author understands the changed behavior:
Current Review State
Next action:
Owner review. Do not automerge. Do not admin-merge by the authoring agent.
Still waiting on author, maintainer, CI, or external proof:
Waiting on GitHub checks and maintainer/owner decision.
Reviewer or bot comments addressed:
This PR addresses the bot feedback on #10 by reverting the non-compliant merge.
Privacy Checklist
Operational Checklist
python3 scripts/check_pr_ready.pypassesgit diff --checkpassesGovernance surface changed
Rollback of a non-compliant governance and skill-instruction PR.
Linked RFC
Maintainer discussion in the Codex session; related PR #10.
Problem with current rule
The rule was not applied: a protected-path, dirty, owner-only PR was admin-merged.
Proposed rule
Restore main and require follow-up changes to use split PRs with the closest specialized template.
Before / after example
Before: #10 merged factor governance and skill naming in one large protected-path PR.
After: main is restored; follow-up work can be split into separate PRs.
Who is affected
Maintainers, contributors, and agents using
SKILL.mdor governance docs.Abuse case prevented
Prevents an agent from treating a successful admin merge as compliance with repository review rules.
New risk introduced
Temporarily removes useful #10 content until it is resubmitted correctly.
Rollout plan
Merge this rollback after owner review, then resubmit smaller PRs.
Rollback plan
Revert this rollback PR if maintainers decide to keep #10 as-is.
Evidence proof
git revert -m 1 0d09feb --no-edit.6513ccaand passing local checks.Owner review
Instruction surface changed
This PR reverts #10 changes to
SKILL.mdand scenario skills.Agent behavior before / after
Before: main contained #10 instructions requiring visible local scenario skill names to use
EvoZeus-....After: main returns to the instruction surface before #10.
Safety boundary
This rollback does not add new instructions. It removes a non-compliant instruction change pending split review.
Prompt injection risk
No new prompt or external instruction source is added.
Problem
A protected-path PR was merged without following required review flow.
Scope
Mechanical revert of #10 only.
Real behavior proof
Command run:
git revert -m 1 0d09feb --no-edit.Tests
git diff --check origin/main..HEAD;python3 -m py_compile scripts/check_pr_ready.py;python3 scripts/check_pr_ready.py --base origin/main --allow-cross-layer.