Skip to content

chore(governance): simplify agent workflow - #847

Merged
proerror77 merged 2 commits into
mainfrom
codex/process-first-principles
Aug 12, 2026
Merged

chore(governance): simplify agent workflow#847
proerror77 merged 2 commits into
mainfrom
codex/process-first-principles

Conversation

@proerror77

@proerror77 proerror77 commented Aug 12, 2026

Copy link
Copy Markdown
Owner

Change

Replace process-heavy agent governance with direct technical constraints:

  • one Issue per behavior or runtime outcome, with attempts and cleanup recorded on it;
  • skills, specifications, branches, and worktrees only when they reduce uncertainty or isolate concurrent writes;
  • repeated runtime attempts require a changed cause/input and an explicit hypothesis;
  • remove the non-required Issue Lifecycle workflows and their 3,623-line implementation/test surface;
  • simplify Issue and PR templates while retaining immutable runtime identities, one controller, stop/cleanup rules, and direct readback.

Issue relationship

None

Validation

  • .github/scripts/test-select-rust-ci-scope.sh
  • Issue-template YAML parse
  • git diff --check
  • repository search confirms no remaining Issue Lifecycle references
  • live branch protection readback confirms required checks remain Monorepo CI gate, Prediction Markets CI gate, and Security Summary Report

Runtime impact and rollback

None. No business code, deployment configuration, credentials, runtime service, Gate, cutover, or Live state changes. Revert this commit to restore the removed governance automation.

Summary by CodeRabbit

  • New Features

    • Issue and pull request templates now use simpler, outcome-focused fields.
    • Blank issue creation is now enabled.
    • CI scope selection now includes broader workflow and script checks.
  • Documentation

    • Updated contribution, issue-tracking, triage, evidence, and runtime guidance.
  • Chores

    • Removed automated issue-lifecycle workflows, reconciliation tooling, and related validation tests.
    • Simplified workflow linting by removing obsolete lifecycle checks.

@proerror77
proerror77 enabled auto-merge (squash) August 12, 2026 00:37
@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@proerror77, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 48 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 2b5ce935-e907-461d-ab45-774da746977a

📥 Commits

Reviewing files that changed from the base of the PR and between 6ede500 and 148a131.

📒 Files selected for processing (8)
  • .claude/commands/pm/issue-close.md
  • .claude/commands/pm/issue-start.md
  • .claude/rules/agent-coordination.md
  • .claude/rules/branch-operations.md
  • .claude/rules/worktree-operations.md
  • .claude/scripts/pm/test-github-first-issue-mutations.sh
  • .github/ISSUE_TEMPLATE/runtime-rollout.yml
  • docs/agents/issue-tracker.md

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 225e6b79-d4e1-401a-987d-1d7997898e25

📥 Commits

Reviewing files that changed from the base of the PR and between eda042c and 6ede500.

📒 Files selected for processing (23)
  • .github/ISSUE_TEMPLATE/bug-report.yml
  • .github/ISSUE_TEMPLATE/config.yml
  • .github/ISSUE_TEMPLATE/engineering-change.yml
  • .github/ISSUE_TEMPLATE/prd.yml
  • .github/ISSUE_TEMPLATE/runtime-rollout.yml
  • .github/pull_request_template.md
  • .github/scripts/issue-lifecycle-audit.rb
  • .github/scripts/issue-lifecycle-preflight.rb
  • .github/scripts/issue-lifecycle-status-reconcile.rb
  • .github/scripts/select-rust-ci-scope.sh
  • .github/scripts/test-issue-lifecycle-audit.sh
  • .github/scripts/test-issue-lifecycle-contract.sh
  • .github/scripts/test-issue-lifecycle-preflight.sh
  • .github/scripts/test-issue-lifecycle-reconcile-workflow.rb
  • .github/scripts/test-issue-lifecycle-status-reconcile.sh
  • .github/scripts/test-issue-lifecycle-workflow.rb
  • .github/scripts/test-select-rust-ci-scope.sh
  • .github/workflows/issue-lifecycle-reconcile.yml
  • .github/workflows/issue-lifecycle.yml
  • .github/workflows/ploy-ci.yml
  • AGENTS.md
  • docs/agents/issue-tracker.md
  • docs/agents/triage-labels.md
💤 Files with no reviewable changes (13)
  • .github/workflows/issue-lifecycle-reconcile.yml
  • .github/scripts/test-issue-lifecycle-contract.sh
  • .github/workflows/issue-lifecycle.yml
  • .github/scripts/issue-lifecycle-status-reconcile.rb
  • .github/scripts/test-issue-lifecycle-preflight.sh
  • .github/scripts/test-issue-lifecycle-reconcile-workflow.rb
  • .github/scripts/test-issue-lifecycle-status-reconcile.sh
  • .github/scripts/issue-lifecycle-audit.rb
  • .github/scripts/test-issue-lifecycle-audit.sh
  • .github/workflows/ploy-ci.yml
  • .github/scripts/test-issue-lifecycle-workflow.rb
  • .github/scripts/issue-lifecycle-preflight.rb
  • .github/scripts/test-select-rust-ci-scope.sh

📝 Walkthrough

Walkthrough

The PR simplifies issue and pull-request templates, removes issue-lifecycle automation and its tests, updates CI scope selection, and revises agent, issue-tracker, and triage-label guidance.

Changes

Issue intake and pull-request contracts

Layer / File(s) Summary
Simplified issue templates
.github/ISSUE_TEMPLATE/*
Templates now capture concise outcomes, evidence, constraints, identities, stop rules, and proof. Blank issue creation is enabled.
Simplified pull-request template
.github/pull_request_template.md
The template now contains change, issue relationship, validation, and runtime impact and rollback sections.

Lifecycle automation removal

Layer / File(s) Summary
Removed lifecycle workflows and tooling
.github/workflows/issue-lifecycle*.yml, .github/scripts/issue-lifecycle-*.rb, .github/scripts/test-issue-lifecycle-*
Issue-lifecycle audits, preflight, reconciliation, status publication, workflow checks, and their test harnesses were deleted.
Updated CI scope and lint steps
.github/scripts/select-rust-ci-scope.sh, .github/scripts/test-select-rust-ci-scope.sh, .github/workflows/ploy-ci.yml
Special-case lifecycle paths and lifecycle test assertions were removed. Workflow lint no longer runs lifecycle checks.

Operating guidance

Layer / File(s) Summary
Revised agent guidance
AGENTS.md
Guidance now covers minimal changes, evidence freshness, fail-closed behavior, publishing readback, cutovers, ownership, and focused validation.
Revised issue and label guidance
docs/agents/issue-tracker.md, docs/agents/triage-labels.md
The documents now define outcome-based issues, native relationships, terminal evidence, cleanup, retry conditions, flat triage labels, and controller-based runtime authorization.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Possibly related issues

  • proerror77/monday#455 — The PR removes and simplifies the issue-lifecycle workflows, templates, audits, and enforcement mechanisms described by the issue.

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers the change, issue relationship, validation, and rollback, but omits required out-of-scope, dependency, and scope-exception details. Add the missing template sections, including dependencies, deliberate out-of-scope work, and the required split or inseparability rationale with named reviewer approval.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the primary change: simplifying agent governance and workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/process-first-principles

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6ede500481

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/agents/issue-tracker.md Outdated
Comment thread .github/ISSUE_TEMPLATE/bug-report.yml
Comment thread .github/ISSUE_TEMPLATE/runtime-rollout.yml Outdated
Comment thread .github/ISSUE_TEMPLATE/runtime-rollout.yml Outdated
Comment thread AGENTS.md
Comment thread .github/ISSUE_TEMPLATE/engineering-change.yml
@proerror77
proerror77 disabled auto-merge August 12, 2026 00:56
@proerror77
proerror77 merged commit 588270d into main Aug 12, 2026
50 of 51 checks passed
@proerror77
proerror77 deleted the codex/process-first-principles branch August 12, 2026 01:45
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