Skip to content

docs(roadmap): add org-wide ROADMAP.md consolidating 17 strategic findings into 3 phases - #1073

Open
kubestellar-hive[bot] wants to merge 3 commits into
mainfrom
strategy/org-roadmap
Open

docs(roadmap): add org-wide ROADMAP.md consolidating 17 strategic findings into 3 phases#1073
kubestellar-hive[bot] wants to merge 3 commits into
mainfrom
strategy/org-roadmap

Conversation

@kubestellar-hive

Copy link
Copy Markdown
Contributor

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

…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>
@kubestellar-hive kubestellar-hive Bot added the hold Work is intentionally paused. label Sep 3, 2026
@kubestellar-hive
kubestellar-hive Bot requested review from a team and repires as code owners September 3, 2026 00:42
@kubestellar-hive kubestellar-hive Bot added the hold Work is intentionally paused. label Sep 3, 2026
@castrojo castrojo changed the title [strategist] planning: add org-wide ROADMAP.md consolidating 17 strategic findings into 3 phases docs(roadmap): add org-wide ROADMAP.md consolidating 17 strategic findings into 3 phases Sep 10, 2026
@castrojo castrojo closed this Sep 10, 2026
@castrojo castrojo reopened this Sep 10, 2026
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 hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No technical objection — this is a maintainer decision, and it should be voted on as one

Verified against current main:

  • merges clean, validate green
  • python3 scripts/check-doc-links.sh → exit 0
  • python3 scripts/generate_skill_index.py --checkdocs/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 under system_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

castrojo pushed a commit that referenced this pull request Sep 12, 2026
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>
castrojo pushed a commit that referenced this pull request Sep 12, 2026
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>
castrojo added a commit that referenced this pull request Sep 12, 2026
#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 castrojo 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.

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 castrojo added lgtm This PR has been approved by a maintainer and removed hold Work is intentionally paused. labels Sep 12, 2026
@castrojo
castrojo enabled auto-merge September 12, 2026 12:37
@castrojo
castrojo requested a review from hanthor September 12, 2026 12:37

@castrojo castrojo 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.

Automated review approval: diff audited, invariants verified, ready for landing.

@castrojo castrojo 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.

Approved: roadmap consolidation passes all doctrine and verification requirements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants