Skip to content

docs: retire mandatory minimal-orchestration - #984

Merged
mk3008 merged 3 commits into
mainfrom
codex/retire-minimal-orchestration-20261022
Oct 22, 2026
Merged

mk3008 merged 3 commits into
mainfrom
codex/retire-minimal-orchestration-20261022

Conversation

@mk3008

@mk3008 mk3008 commented Oct 22, 2026 •

Copy link
Copy Markdown
Owner

Problem and result

The repository required a globally installed minimal-orchestration skill, which can block authorized work when that old skill is unavailable. Retire that prerequisite in AGENTS.md, the orchestration adapter, and rawsql-task-orchestrator. Keep repository impact routing, tmp/PLAN.md, durable handoff evidence, verification, review, branch isolation, and authorization requirements.

This is repository-only workflow maintenance (no-release; no Changeset). It does not change packages, dependencies, SQL behavior, MCP, security controls, or historical reports. The user explicitly requested this retirement separately from the concurrent security fixes: control task.

Verification and readiness

  • done: all three edited files read back exactly from the isolated feature branch; compare against current main shows only the intended policy files, with behind_by=0.
  • done: maintained guidance paths resolve in the tracked tree; Markdown whitespace and fenced-block checks pass.
  • done: consistency/coverage review, retro triage, and human-acceptance wording review completed; independent security, checks, review and task-tracking requirements retained.
  • done: CI, corrected PR Check, and Release Readiness all completed successfully at this head. CI includes Node 22/24 core lint/build/tests and Windows/Linux/macOS consumer guards. The initial PR Check failed because required body sections were omitted; those sections were corrected, the real checker passed locally, and the replacement run passed remotely.
  • Limits: full local workspace build/typecheck/lint/tests, manual integration tests, and docs:build were not run. Fresh local clone failed before checkout (getaddrinfo thread startup error); local gh authentication was invalid. No credentials were changed. The Docs workflow builds and deploys from main; it was not manually dispatched. No standalone Markdown lint is configured. Direct Markdown/fence/path checks cover this prose-only policy diff, and CI supplies the existing PR checks; this is not a claim that every optional or main-only workflow ran.

Next: review the policy diff. Merge/release is not authorized by this PR.

Merge Readiness

  • No baseline exception requested.
  • Baseline exception requested and linked below.

The required checks remain required. This PR does not claim that unrun local or main-only workflows passed and does not request a waiver.

Self Review

Self-review workflow: .codex/guidance/review.md consistency review, retro triage, and human-acceptance review against root#145.
Self-review result: no unresolved policy-diff blockers; preservation checks and remote PR/CI/Release Readiness passed. Unrun optional/local/main-only verification is disclosed above, without changing the existing checks.
Concept-review workflow: explicit no-concept-impact review; only repository task guidance changes, with no packages, SQL contracts, public API, CLI, scaffold, generated product output, or human-owned business meaning affected.
Concept-review result: no package or concept-boundary violations found in the three-file diff; no product/release scope change.

@coderabbitai

coderabbitai Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

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

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 746a3f59-7216-477d-b8e1-73cf1d0e1a0d
📥 Commits

Reviewing files that changed from the base of the PR and between 1999874 and 3bb9de7.

📒 Files selected for processing: 3
  • .agents/skills/rawsql-task-orchestrator/SKILL.md
  • AGENTS.md
  • docs/codex-orchestration.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@mk3008
mk3008 marked this pull request as ready for review October 22, 2026 06:21
@mk3008
mk3008 merged commit 2aec5cd into main Oct 22, 2026
11 of 12 checks passed
@mk3008
mk3008 deleted the codex/retire-minimal-orchestration-20261022 branch October 22, 2026 06:28
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