docs(roadmap): restructure around v1 objectives#708
Conversation
Replace task-list format with theme-based scope organized around the four v1 objectives: coverage, stability, UX, and security. Removes shipped items, individual issue references, and internal program terminology to keep the document high-level and externally consumable.
This comment was marked as resolved.
This comment was marked as resolved.
Coverage Report ✅
Coverage BadgeNo Go source files changed in this PR. |
Address CodeRabbit feedback on PR #708: - Spell out CSP and replace 'NCP' with 'partner-cloud offerings' on first use of the service axis description. - Correct the sustained-validation claim: KWOK is advisory on PRs (Tier 1 generic plus diff-aware Tier 2) with the full matrix on merge and nightly. v1 adds live-cluster UAT and tightens these signals into the merge path.
Address remaining CodeRabbit feedback on PR #708: - Replace 'defining, perpetual theme' / 'keeps it honest' / 'cannot silently rot' with neutral operational language. - Soften 'are all in place' for build-time provenance to a concrete 'are shipped' status, with specific surfaces named.
There was a problem hiding this comment.
♻️ Duplicate comments (2)
ROADMAP.md (2)
100-102:⚠️ Potential issue | 🟠 MajorAvoid unconditional shipped-status assertions without explicit qualification.
This line still reads as a hard guarantee across multiple security surfaces. If any item is partial, scope-limited, or still being hardened, split into “implemented” vs “in progress” to avoid over-commitment.
As per coding guidelines: documentation should “clearly distinguish current behavior from future intent” and “must not invent features, guarantees, timelines, or roadmap commitments.”
Suggested wording pattern
-**Build-time provenance.** SLSA build attestation, cosign signing, trust-level -evaluation in `aicr bundle verify`, and bundle attestation for the `argocd-helm` -output are shipped. +**Build-time provenance.** SLSA build attestation and cosign signing are implemented. +Trust-level evaluation in `aicr bundle verify` and bundle attestation for the +`argocd-helm` output are implemented where currently supported, with remaining +hardening tracked in v1 scope.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@ROADMAP.md` around lines 100 - 102, The current ROADMAP entry "Build-time provenance. SLSA build attestation, cosign signing, trust-level evaluation in `aicr bundle verify`, and bundle attestation for the `argocd-helm` output are shipped." reads as an unconditional shipped guarantee; change the wording to clearly separate what is implemented today from what is in progress or partial. Edit the "Build-time provenance" sentence to (a) enumerate implemented items (e.g., SLSA build attestation, cosign signing) under an "Implemented" clause and (b) list any partial/hardening items (e.g., trust-level evaluation in `aicr bundle verify`, bundle attestation for `argocd-helm`) under an "In progress" or "Planned" clause, avoiding words like "are shipped" or absolute guarantees so the statement reflects exact current status.
74-75:⚠️ Potential issue | 🟡 MinorSoften the causal claim about community growth bottlenecks.
“primary bottleneck to community growth” reads as an asserted diagnosis. Consider neutral wording like “a key bottleneck in contributor onboarding and throughput.”
As per coding guidelines: documentation must “use neutral tone” and “must not invent features, guarantees, timelines, or roadmap commitments.”
Suggested wording
-Friction in the contribution path is the primary bottleneck to community growth. +Friction in the contribution path is a key bottleneck in contributor onboarding and review throughput.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@ROADMAP.md` around lines 74 - 75, Replace the assertive phrase in ROADMAP.md "Friction in the contribution path is the primary bottleneck to community growth." with a neutral formulation (e.g., "Friction in the contribution path is a key bottleneck in contributor onboarding and throughput.") so the document uses neutral tone and avoids asserting a definitive diagnosis; update the sentence around that exact text to match the suggested wording.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Duplicate comments:
In `@ROADMAP.md`:
- Around line 100-102: The current ROADMAP entry "Build-time provenance. SLSA
build attestation, cosign signing, trust-level evaluation in `aicr bundle
verify`, and bundle attestation for the `argocd-helm` output are shipped." reads
as an unconditional shipped guarantee; change the wording to clearly separate
what is implemented today from what is in progress or partial. Edit the
"Build-time provenance" sentence to (a) enumerate implemented items (e.g., SLSA
build attestation, cosign signing) under an "Implemented" clause and (b) list
any partial/hardening items (e.g., trust-level evaluation in `aicr bundle
verify`, bundle attestation for `argocd-helm`) under an "In progress" or
"Planned" clause, avoiding words like "are shipped" or absolute guarantees so
the statement reflects exact current status.
- Around line 74-75: Replace the assertive phrase in ROADMAP.md "Friction in the
contribution path is the primary bottleneck to community growth." with a neutral
formulation (e.g., "Friction in the contribution path is a key bottleneck in
contributor onboarding and throughput.") so the document uses neutral tone and
avoids asserting a definitive diagnosis; update the sentence around that exact
text to match the suggested wording.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Enterprise
Run ID: c9b4b2d1-afc6-494c-a0ca-021f032e1a7d
📒 Files selected for processing (1)
ROADMAP.md
Summary
Restructures
ROADMAP.mdaround the four v1 objectives (coverage, stability, UX, security), prunes shipped items, and removes individual issue references and internal program terminology to keep the document high-level and externally consumable.Motivation / Context
The previous roadmap mixed shipped MVP work, in-flight tasks, and post-launch backlog in a flat task-list format that had drifted out of date. With AICR converging on v1, the document needs to communicate the GA bar at a theme level rather than serve as a per-issue tracker (the project board handles that).
Fixes: N/A
Related: N/A
Type of Change
Component(s) Affected
docs/,examples/) —ROADMAP.mdonlyImplementation Notes
Objectives+Scope(Coverage / Stability / UX / Security) instead of a task list.Acceptancedefinition.Testing
Documentation-only change (no code or build files touched). Rendered locally to verify formatting.
Risk Assessment
Rollout notes: N/A
Checklist
git commit -S)