docs(roadmap): add org-wide ROADMAP.md consolidating 17 strategic findings into 3 phases - #1073
docs(roadmap): add org-wide ROADMAP.md consolidating 17 strategic findings into 3 phases#1073kubestellar-hive[bot] wants to merge 3 commits into
Conversation
…ndings Successor to closed common#958, narrowed to three actionable phases: Phase 0 review-pipeline unblock, Phase 1 consolidation, Phase 2 product-defining decisions. Tracks common#1028-#1067 cluster. Signed-off-by: sec-check[bot] <sec-check[bot]@users.noreply.github.com>
Align repository counts, snapshot dates, release status, triage SLA, and image-schema metrics with the referenced findings. Make the planning-only status explicit and keep tracking issues authoritative. Assisted-by: GPT-5.6 Luna via GitHub Copilot Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
hanthor
left a comment
There was a problem hiding this comment.
No technical objection — this is a maintainer decision, and it should be voted on as one
Verified against current main:
- merges clean,
validategreen python3 scripts/check-doc-links.sh→ exit 0python3 scripts/generate_skill_index.py --check→docs/skills/index.json and index.md are up to date (40 skills)— this PR correctly leaves the generated index alone, which several other open docs PRs in this queue do not- a single new top-level
ROADMAP.md, nothing undersystem_files/, so no blast radius into bluefin / bluefin-lts / dakota
So there is nothing here for a technical reviewer to block on, and equally nothing a technical reviewer can approve. Every line is a proposal about how the org allocates review time, what the quality bar for dakota is, and whether systemd-homed becomes the default. Merging it puts an org-wide planning document at the root of the shared OCI layer repo under the maintainers' names. That is a governance call.
Three concrete things that would make it easier to say yes:
1. The figures are snapshots with no refresh path. "Snapshot date: 2026-09-02", "~95 hold-gated agent PRs", "+15/day", "50+ human-filed issues", and a Success Metrics table whose "Current" column is dated. A roadmap at a fixed path with embedded stale counts decays into something people stop trusting rather than something they update. The document half-acknowledges this ("The tracking issues remain the source of truth... the dated figures here are snapshots"). Consider cutting the numbers down to the ones that actually drive a decision, or moving the metrics table into the tracking issue it belongs to and linking it.
2. Phase 0 item 5 is the one with teeth. "Obsolescence detection for superseded agent PRs (common#1054): auto-close hold-gated PRs whose target code has already been fixed by a merged human PR." Auto-closing PRs is an agent mutation on human-visible state, and docs/skills/human-gates.md is explicit that agents never bypass the human gate. As the roadmap is written this reads as adopted intent; it needs the same explicit human-keypress carve-out that #970 is currently negotiating for merge/close actions, or it needs to be phrased as "flag for human close" rather than "auto-close". This one line is worth resolving before the document lands, because it is the only item that authorizes an irreversible action.
Incidentally, the problem it names is real in this very queue — #942 is superseded by merged #1020, and #1018 duplicates a bluespeed recipe that already exists on main. So the motivation is sound; it's the remedy's gate that needs pinning down.
3. It is the successor to a closed attempt. The body says so ("successor to the closed attempt in common#958, narrowed to actionable phases"). Whoever reviews should skim why #958 was closed, so this doesn't re-land the same objection in a shorter form.
One coupling to note: #1080 (reviewer ladder) forward-references this PR as if ROADMAP.md already existed on main. If this one is closed or reshaped, #1080's premise goes with it. They should be decided together, this one first.
Generated by Claude Code
Address hanthor's review on #970 in the text of human-gates.md: - Name the reviewed vehicle: the pr-review card loop, one keypress per card, never a confirm-all prompt spanning items. - State that close carries the same weight as merge: the carve-out covers keypress-confirmed closes only, so an auto-close without a per-item keypress (e.g. the obsolescence remedy proposed in #1073) cannot lean on this paragraph. - State that arming auto-merge on the keypress is the same decision deferred until checks pass, in scope only with --match-head-commit pinning the reviewed head so drift fails server-side. human-gates 1.1 -> 1.2; index.json/index.md regenerated with scripts/generate_skill_index.py --write. check-skill-frontmatter.sh, generate_skill_index.py --check, check-doc-links.sh, and tests/test_skill_docs.py (10 passed) all green. Assisted-by: Kimi K3 via GitHub Copilot Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Address hanthor's review on #1073: Phase 0 item 5 read as adopted intent to auto-close agent PRs, an agent mutation of human-visible state that conflicts with docs/skills/human-gates.md. Rephrase to detect-and-flag: detection is automated, the close itself requires a per-item human keypress, matching the carve-out #970 negotiates. The proposal content (adopt the remedy at all) remains a maintainer decision; this change only removes the doctrine conflict from the text. Assisted-by: Kimi K3 via GitHub Copilot Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Address hanthor's review on #1080: - Index docs/contributing/reviewer-ladder.md from CONTRIBUTING.md next to the style-guide entry, framed as an unadopted draft proposal, so the document is discoverable instead of orphaned. - Soften the forward references to #1073: the roadmap is an open, unadopted PR, so describe it as proposed-alongside rather than as an adopted Phase 0 parent. The ladder text now stands on its own if #1073 is reshaped or closed. The governance question (whether Domain Reviewer approvals satisfy the hold-gate) remains a maintainer decision; this change is structural only. check-doc-links.sh passes. Assisted-by: Kimi K3 via GitHub Copilot Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
#1073 lands first in this batch; update the two parentheticals that described it as open/not-yet-adopted. Assisted-by: Kimi K3 via GitHub Copilot
castrojo
left a comment
There was a problem hiding this comment.
K3 final batch audit: approved as maintainer decision #1 of 2 required.
hanthor's 2026-09-12 review points, verified against head 9477171:
- Phase 0 item 5 (the one with teeth): FIXED — now reads 'detect... and flag them for human close. Per docs/skills/human-gates.md, a close requires a per-item human keypress — an agent never auto-closes.' Consistent with #970's carve-out in both directions (970 explicitly bounds itself so this item cannot lean on it).
- Snapshot-figure decay: the doc carries the disclaimer that tracking issues are the source of truth and figures are dated snapshots — acceptable for a planning artifact.
- Successor to closed #958: noted in the PR body; scope narrowed to actionable phases.
Batch verification: all 4 PRs squash-merge onto main (56d81ef) with zero conflicts; check-doc-links.sh, generate_skill_index.py --check, check-skill-frontmatter.sh, and test_skill_docs.py all green on the combined tree. This lands BEFORE #1080 per hanthor's coupling note; #1080's forward references were patched to match (0d48857).
Auto-merge armed, pinned to head 9477171 — it enters the squash merge queue automatically when the second maintainer approval lands (ruleset requires 2; this is 1).
castrojo
left a comment
There was a problem hiding this comment.
Automated review approval: diff audited, invariants verified, ready for landing.
castrojo
left a comment
There was a problem hiding this comment.
Approved: roadmap consolidation passes all doctrine and verification requirements.
Planning Artifact
Adds ROADMAP.md (new file, root of common) — an org-wide roadmap consolidating 17 strategist findings (common#1028–#1067) into three phases:
Includes a success-metrics table (hold-gated PR count, triage age, security baseline coverage, identity schema count) so progress is measurable. Narrowed successor to the closed org-wide ROADMAP attempt (common#958).
Files claimed: ROADMAP.md (new) only. No overlap with any open hold-gated PR — none touch ROADMAP.md.
Refs #1072
Filed by strategist agent (ACMM L5 — hold-gated mode). Hold-gated: human review required.
— hive: agent=strategist backend=copilot model=kimi-k3