Skip to content

Point the Fleet Map at the Shipped Diagrams and Current Tooling - #703

Merged
ptr727 merged 1 commit into
developfrom
feature/fleet-map-realign
Aug 14, 2026
Merged

Point the Fleet Map at the Shipped Diagrams and Current Tooling#703
ptr727 merged 1 commit into
developfrom
feature/fleet-map-realign

Conversation

@ptr727

@ptr727 ptr727 commented Aug 14, 2026

Copy link
Copy Markdown
Owner

Why

A re-analysis of docs/fleet-map.md, applying the same gap-finding method that created it, after the P0-P4 closures, the #689/#701 install-model changes, and the #702 procedure-doc diagrams.

What

Flag, not a task

docs/fleet-map.md and the #702 diagrams exist only on develop (main is 22 commits behind and lacks the file entirely). A develop to main promotion is a maintainer decision and is not part of this change.

Verification

All nine CI prose_lint checks pass tree-wide and --diff develop is clean, markdownlint reports 0 issues on both edited files, cspell is clean on README and leaves only the six pre-existing fleet-map findings (the file is not CI spell-gated), editorconfig-checker passes (CRLF preserved, diff is line-scoped), spec/validate.py and jq pass, all 6 remaining mermaid fences (5 in fleet-map, README doors) parse OK via mermaid@11, and a two-way reference-link check (every use defined, every relative target exists) passes.

🤖 Generated with Claude Code

A re-analysis of docs/fleet-map.md after the P0-P4 closures and the #702
procedure-doc diagrams found the map's own copies of the STANDUP and RESYNC
flows superseded at finer grain by the diagrams the procedure docs carry,
with no cross-reference in either direction. By the map's own rule the
procedure doc wins, so the derivative copies are removed in favor of
pointers, the maintenance rule is extended to cover drawn flows, the stale
Checked-anchor sentence and cold-start tool list are corrected after #701,
OPERATIONS.md, repo_gate.py, pr_review.py, and spec/audit.py are surfaced
as the entry points they are, #699 and #700 are cross-linked, and README
gains the one link that makes the map discoverable from the front door.
Copilot AI lite review requested due to automatic review settings August 14, 2026 17:06

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

Updates fleet documentation cross-links so docs/fleet-map.md points at the authoritative, shipped procedure diagrams and current tooling references, and exposes the fleet map from README for easier discovery.

Changes:

  • Add a README reference-link to the hub-only fleet map behind the entry doors diagram.
  • Replace duplicated STANDUP/RESYNC flow diagrams in docs/fleet-map.md with pointer sentences, and refresh stale/incorrect prose (maintenance rule scope, cold-start tooling reference, and additional runnable entry-point links).
  • Add worktree to cspell.json to support new cross-link wording.

Reviewed changes

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

File Description
README.md Adds a reference-style link to docs/fleet-map.md from the entry doors section.
docs/fleet-map.md Removes duplicated procedure diagrams, strengthens/extends the maintenance rule, refreshes stale references, and adds/uses reference links for key entry-point tooling/docs.
cspell.json Adds worktree to the dictionary to avoid false positives.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ptr727
ptr727 merged commit 43c6184 into develop Aug 14, 2026
7 checks passed
@ptr727
ptr727 deleted the feature/fleet-map-realign branch August 14, 2026 17:12
ptr727 added a commit that referenced this pull request Aug 15, 2026
…Python CI Gates (#718)

Thirty-one squashes, `56f4d7d..d54862a`. 115 files, +20436/-5298.

**Merge with a merge commit, never a squash, and never with
`--delete-branch`.** This pull request's head is `develop` itself.

## What lands

**Fleet Skills.** The `.agents/skills/` source tree, the generated
`.claude-plugin/` distribution, `scripts/build_dist.py` with its
`--check` gate, and `scripts/skills_install.py` with its host stamp
(#676). Packaged as skills on top of the scaffold: PR review conduct and
Copilot instructions upkeep (#677), comment and doc style (#678),
resync-a-repo and fleet-conformance-check (#679), the per-language
codestyles (#680), git commit conventions and operational vs release
workflow (#681), stand up a repo (#683), and repo-worktree (#717).
Coverage gaps closed in three passes (#690, #691, #692) plus the P4
sentence-length opt-in (#697).

**Host setup.** The Windows host-setup tooling and its PowerShell gate
(#674), the Windows bootstrap loader (#682), Docker install and upgrade
on Linux and Windows with a version floor (#701, #705), a `uv` floor in
`spec/host-tools.json` (#698), self-healing of a shadowing `uv`, `jq`,
or `git-restore-mtime` copy (#689), node's real winget package id
(#696), and a README for the Linux host-setup nuances (#710).

**Python and CI.** Python tooling in CI with the script tests moved to
`scripts/tests` (#704), `ruff format` adopted and gated (#709), and the
PSScriptAnalyzer claim conditioned on repos that carry `.ps1` files
(#686).

**Conduct rules.** Triage-order and scope guardrails in
pr-review-conduct (#684), `pr_review.py wait` requesting a review rather
than only polling for one (#685), a tech-agnostic signed-commit
verification (#708), execution rather than analogy to verify
platform-specific code (#715), and a unique worktree for every task
(#717).

**Docs.** The fleet map and gap register with peer messaging declared
(#687), mermaid flow diagrams in the kept-authority docs (#702), and the
map pointed at the shipped diagrams and current tooling (#703).

## Issues this promotion closes

Each landed on `develop` on its own pull request. The keyword fires only
on a merge into `main`, so it sits here rather than on the feature pull
requests.

Closes #700
Closes #707
Closes #711
Closes #712
Closes #714
Closes #688

#699 stays open on purpose: #717 shipped the layout convention and the
skill, and the physical migration of existing checkouts is still tracked
there.

## Review record

Every squash closed its own Copilot loop on its own pull request before
merging to `develop`. This promotion carries no new content of its own,
so its review is the merged tree as a whole.

## Consequence worth stating

The `GOVERNANCE.md` and `AGENTS.md` sections these squashes changed
become the canonical the moment this reaches `main`, and every carrying
repository reads as drifted from that point until it resyncs. That is
the ordinary consequence of a canonical moving rather than a defect. The
Skills installer added here is also how a machine picks the new skills
up, so a session that keeps restating a rule already packaged as a skill
is the signal to run it.
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