Skip to content

Extract operational-repo rules into one scoped section (#368, PR-3) - #371

Merged
ptr727 merged 2 commits into
developfrom
operational-section-368
Jul 20, 2026
Merged

Extract operational-repo rules into one scoped section (#368, PR-3)#371
ptr727 merged 2 commits into
developfrom
operational-section-368

Conversation

@ptr727

@ptr727 ptr727 commented Jul 20, 2026

Copy link
Copy Markdown
Owner

What and why

Third of the staged PRs for #368 — the first doc surgery. The operational-vs-release carve-out was scattered inline across Foundational Principles, Branching Model, and Release Model, so those all-downstream sections mixed two scopes a reader (and the audit) had to piece apart. This makes the operational rules one granular single-scope piece.

Change

  • New ## Operational Repositories section consolidates the operational workflow rules — direct-to-develop, advisory CI, the develop/main ruleset differences, dispatch-only release, dependabot/merge-bot on every tier. It is tagged appliesTo: ["operational"] in spec/files.json, so PR-2's section-presence check requires it only on the 4 operational repos.
  • The three inline carve-outs slim to one-line pointers, leaving Foundational, Branching, and Release single-scope (the release model). No operational detail is lost — it all moves into the new section.
  • The operational line-ending rule stays in ## Line Endings (findable where EOL rules live) with a cross-pointer, rather than moving into the new section.

Scope

Deliberately just the operational extraction. The other two contaminants from the plan — hub-only pointers inside carried sections, and the csharp/python/docker examples baked into Release Model / Workflow YAML — are separate follow-up PRs, to keep each a small composable piece.

Verified

  • Operational repos require the new section; release repos do not.
  • The hub's own AGENTS.md is complete (no missing sections in either view).
  • spec/validate.py, markdownlint, editorconfig-checker clean.

Once merged, the 4 operational repos will DRIFT on this section until they adopt it — the intended convergence signal, not a regression. Held on develop — no main promotion.

Doc surgery on the granular-single-scope model. The operational-vs-release
carve-out was scattered inline across Foundational Principles, Branching Model,
and Release Model, so those all-downstream sections mixed two scopes a reader
had to piece apart.

- New "## Operational Repositories" section consolidates the operational
  workflow rules (direct-to-develop, advisory CI, the develop/main ruleset
  differences, dispatch-only release, dependabot/merge-bot on every tier). It
  is tagged appliesTo ["operational"] in spec/files.json, so PR-2's
  section-presence check requires it only on operational repos.
- The three inline carve-outs slim to one-line pointers, leaving Foundational,
  Branching, and Release single-scope (the release model). No operational
  detail is lost - it all moves into the new section.
- The operational line-ending rule stays in the Line Endings section (findable
  where EOL rules live) with a cross-pointer, rather than moving into the new
  section.

Verified: operational repos require the new section and release repos do not,
the hub's own AGENTS.md is complete, validate.py and the doc linters are clean.
Once merged, the 4 operational repos will DRIFT on this section until they adopt
it - the intended convergence signal.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 20, 2026 22:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR refactors governance documentation by consolidating operational-repository (workflowModel: operational) rules into a single dedicated section, and updates the spec baseline so that section is required only for operational repos.

Changes:

  • Add a new ## Operational Repositories section to AGENTS.md, moving previously scattered operational carve-outs into one scoped place.
  • Slim the operational references in Foundational Principles / Branching Model / Release Model down to short pointers to the new section.
  • Update spec/files.json to require the new Operational Repositories section only when appliesTo: ["operational"].

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
AGENTS.md Extracts and centralizes operational repo rules into a dedicated section, leaving other models single-scope with pointers.
spec/files.json Makes Operational Repositories a section-level appliesTo: ["operational"] requirement under AGENTS.md.

Comment thread AGENTS.md Outdated
)

Copilot round-1 finding: the Release Model intro line (edited in this PR) joined
two clauses with a semicolon. Split into two sentences.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 20, 2026 22:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@ptr727
ptr727 merged commit 8578e12 into develop Jul 20, 2026
7 checks passed
@ptr727
ptr727 deleted the operational-section-368 branch July 20, 2026 22:58
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.

2 participants