Skip to content

docs(roadmap): re-scope Phase 2.6 to Conversational Authoring + First-Class CLI#70

Merged
cemililik merged 6 commits into
mainfrom
development
Jul 8, 2026
Merged

docs(roadmap): re-scope Phase 2.6 to Conversational Authoring + First-Class CLI#70
cemililik merged 6 commits into
mainfrom
development

Conversation

@cemililik

@cemililik cemililik commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

What

Phase 2.6'yı orijinal "authoring-and-parity" kapsamından, CLI'yı bir ürün olarak bitiren faza yeniden kapsamlandırıyoruz: full-screen Home-merkezli bir yüzey (providers/keys, models, MCP, settings, workflows, agents, gates — hepsi Home'dan), üç seviyeli run drill-down (list → run → node), competitor-breadth toolbelt (edit/search/find/todo/ask-user/working web search) governance floor altında, settings + theming + en/tr localization, onboarding v2 (Relavium-account stub), ve run-ops resume follow-up — @relavium/authoring omurgası üzerine.

Re-scope üç girdiyi harmanlar: maintainer CLI-experience bulguları, bir competitor araştırması (opencode, Claude Code, Codex CLI, Gemini CLI, gh run/Aider/Goose/Amp/Cursor/Copilot), ve deferred-tasks.md'nin satır satır triajı (21 now-doable item 2.6.A–M workstream'lerine pull-in edildi). 2.6.A–E kimliklerini korur (ADR-0058/0059/0060 referansları); 2.6.F–M yeni.

Scope

Package(s): docs (roadmap, README, CLAUDE.md). Phase/workstream: Phase 2.6 (Planned).

6 dosya tek bir tutarlı anlatıda senkronize:

  • phase-2.6-conversational-authoring.md — rewritten (13 workstream A–M, milestones M2.6-1..5, pull-in table, sequencing Mermaid, risks)
  • current.md — Phase 2.5 complete → Phase 2.6 next-up
  • roadmap/README.md — phase index + Mermaid graph updated
  • root README.md + CLAUDE.md — status sections
  • deferred-tasks.md — 21 Scheduled → 2.6.X markers + stale live-nightly note updated

Review corrections (applied in this PR)

  • 2.6.K ref belirsizliği → input'un stable identifier'ı (workflow YAML'deki) olarak tanımlandı
  • 2.6.L NO_COLOR/ansi theme → NO_COLOR tüm temaları aynı şekilde override eder (ansi bir color-free bypass değil)
  • tr → CJK-ready → factually wrong (Türkçe ≠ CJK); "laying the groundwork for future CJK locales" olarak düzeltildi
  • nodeCostMicrocents optional vs exact → optional backward-compat için; mevcut olduğunda exact
  • ink-7 adoption gate → "adopted only if floor >=22; otherwise ships on current ink major"
  • Sora 2 + live-nightly → canonical deferred-tasks.md entries referans verildi (restated değil)
  • British spelling (realises) → American English (realizes) ile normalize edildi
  • perf pins jargon → "performance regression thresholds (frame time / render count)"
  • Required ADRs → 2.6.E'nin koşullu markdown-dependency ADR footnote'u eklendi
  • current.md → "desktop\napp" word-split düzeltildi
  • deferred-tasks.md → "defer to 2.K" stale note → "2.K shipped; Phase-2.6 in-window obligation"

Checklist

  • Conventional Commits, one scope per commit, Refs: ADR-0058/0059/0060
  • No code change (docs-only) — lint/typecheck/test unaffected
  • Cross-document consistency: dates (2026-07-08), PR Phase 2.5 (CLI Consolidation) complete — 2.5.I/2.5.J close-out + Batch A–E → main #69, M2.5-4, "re-scoped 2026-07-08" identical across all 6 files
  • Pull-in table (21 items) ↔ Scheduled → 2.6.X markers in deferred-tasks.md — birebir eşleşiyor
  • ADR references (0058 Proposed, 0059 Accepted, 0060 Proposed, 0021 supersede) correct
  • One-canonical-home: Sora 2 + live-nightly referenced from deferred-tasks.md, not restated
  • No vendor SDK type / engine platform imports (docs-only)
  • No secret in logs/events/IPC/exported YAML

Refs: ADR-0058, ADR-0059, ADR-0060

🤖 Generated with Claude Code

Summary by Sourcery

Re-scope and expand the Phase 2.6 roadmap into a full first-class, Home-centric CLI phase, align all roadmap and status docs with Phase 2.5 completion and the new Phase 2.6 scope, and synchronize deferred-task scheduling and ADR references accordingly.

Enhancements:

  • Expand related-doc references and ADR requirements around Phase 2.6 to cover new CLI foundations, management browsers, MCP management, toolbelt rendering, and i18n/theming architecture.

Documentation:

  • Rewrite the Phase 2.6 roadmap to describe a broader CLI-focused phase covering full-screen Home UX, management surfaces, toolbelt expansion, theming/localization, onboarding v2, and run-ops follow-ups, while preserving the original authoring spine.
  • Update high-level roadmap, current-status, README, and CLAUDE.md documentation to mark Phase 2.5 as complete (PR Phase 2.5 (CLI Consolidation) complete — 2.5.I/2.5.J close-out + Batch A–E → main #69) and position the re-scoped Phase 2.6 as the next planned phase.
  • Refresh deferred-tasks.md to map specific deferred items into Phase 2.6 workstreams with explicit Scheduled → 2.6.X markers and to record already-completed items accurately.

Summary by CodeRabbit

  • Documentation
    • Updated roadmap and status docs to show Phase 2.5 as complete and published.
    • Added a new, expanded Phase 2.6 roadmap covering the next CLI and conversational authoring experience.
    • Refreshed deferred-tasks tracking with updated statuses, scheduling, and follow-up items.
    • Expanded phase planning details, including milestones, scope, sequencing, risks, and exit criteria.

cemililik and others added 5 commits July 8, 2026 09:10
Post-merge traceability: the 2.5.I/2.5.J + Batch A–E close-out landed via PR #69
(development → main, merged 2026-07-08). Add the durable PR reference to every
status surface (phase-2.5 doc, current.md, CLAUDE.md, README) — the per-commit
`Refs:` had pointed at the now-deleted scratch close-plan, so the PR number is
the durable pointer — and note Phase 2.5 is merged to `main`.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
… + PR #69 refs

Rigorous audit of deferred-tasks.md against what the 2.5 close-out actually
shipped — mark done ONLY what is genuinely in the merged code (verified each
against the source), and leave the intentionally-deferred items open:

Newly marked [x] DONE (were still [ ] — done this round but missed):
- Profile-unaware advertise-filter → `wiredToolIds({ readOnly })` (Step 15, E).
- Compact abort hint during streaming → `streamingAbortHint` (Step 12, B).
- Bound the EXPANDED reasoning panel by rendered LINES → `MAX_REASONING_PANEL_LINES`
  row-tail (Step 12, B).
- Allow Ctrl+T / /thinking during a pending approval → the whitelist in
  `reduceApprovalKey` (Step 14, D-4).
- Two transitively-covered test gaps → the git_status union pin was ADDED
  (session-host.test.ts) and the failed-turn persister fold was CONFIRMED already
  pinned (persister.test.ts) (Step 11, A).

Also added the durable `PR #69` merge reference to the six earlier close-out
entries, so all eleven closed items consistently cite the merge.

Left [ ] on purpose (NOT closed by this round — verified): the two gate.ts-resume
follow-ups (budget resume + secret re-provide), the session {{ctx.*}} interpolation
(Proposed ADR-0060), extraRoots (blocked on its config source), the two Step-14
review follow-ups (approval-consent zero-width; extract the [c] reducer), the
AgentParseError chat-surface diagnostic gap, and the cross-process gate-resolve
TOCTOU (needs a store-level uniqueness constraint — BEGIN IMMEDIATE does not close it).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
All ten workstreams 2.5.A–J shipped (milestone M2.5-4, merged to `main` via
PR #69); the go/no-go exit criteria hold and Phase 2.6 is unblocked. Reconcile
the phase-status surfaces that still framed 2.5 as in-progress:

- phase-2.5-cli-consolidation.md: banner "In progress" → "✅ COMPLETE"; mark the
  go/no-go exit criteria "all five met" at phase close.
- current.md: retire the "active phase = Phase 2.5" pointer (2.5 is historical);
  name Phase 2.6 (Conversational Authoring, Planned/unblocked) as the next phase
  in both the intro and the "Not started yet" section.
- phase-2.6-conversational-authoring.md: note the 2.5 dependency is satisfied —
  status "Planned — next up", now unblocked.
- CLAUDE.md: the Phase-2.5 status paragraph lead "is now underway" → "✅ complete"
  (its tail already read complete), resolving the lead/tail contradiction.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Phase 2.5 already read "complete" in the README; add the forward pointer to
Phase 2.6 (Conversational Authoring — the shared `@relavium/authoring` package
that lets a chat author standards-valid `.relavium.yaml`), matching the roadmap
surfaces closed in the previous commit.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…-Class CLI

Rewrite Phase 2.6 from the original authoring-and-parity scope into the phase
that finishes the CLI as a product: a full-screen Home-centric surface managing
the entire product (providers/keys, models, MCP, settings, workflows, agents,
gates), a three-level run drill-down over attributed durable history, competitor-
breadth tools (edit/search/find/todo/ask-user/working web search) under the
governance floor, settings + theming + en/tr localization, onboarding v2 with
the Relavium-account stub, and the run-ops resume follow-up — on top of the
original @relavium/authoring spine.

The re-scope folds three inputs: maintainer CLI-experience findings, a competitor
research pass (opencode, Claude Code, Codex CLI, Gemini CLI, gh run/Aider/Goose/
Amp/Cursor/Copilot), and a line-by-line triage of deferred-tasks.md (21 now-doable
items pulled into workstreams 2.6.A–M). Workstreams 2.6.A–E keep their identities
(referenced by ADR-0058/0059/0060); 2.6.F–M are new.

Six files stay in sync: the phase doc (rewritten), current.md (status → next-up),
roadmap/README.md (phase index + Mermaid), root README.md + CLAUDE.md (status
sections), and deferred-tasks.md (Scheduled → 2.6.X markers on the pulled-in items).

Review corrections applied during the re-scope: the 2.6.K input ref is clarified
(stable identifier in workflow YAML), the 2.6.L NO_COLOR/ansi theme interaction is
explicit (all themes overridden identically), the tr→CJK-ready phrasing is corrected
(Turkish ≠ CJK — now "laying the groundwork for future CJK locales"), the
nodeCostMicrocents optional-vs-exact tension is resolved (optional for backward-
compat; exact when present), the ink-7 adoption gate is unambiguous (adopted only
if floor >=22), Sora 2 + live-nightly are referenced (not restated) from their
canonical deferred-tasks entries, and British spellings (realises) are normalized
to the codebase's American English.

Refs: ADR-0058, ADR-0059, ADR-0060

Co-Authored-By: Claude <noreply@anthropic.com>
@sourcery-ai

sourcery-ai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Reviewer's Guide

Re-scopes Phase 2.6 from a narrow authoring/parity focus to a broader "Conversational Authoring + First-Class CLI" phase, rewriting the phase doc and synchronizing all roadmap/status docs and deferred-task mappings to reflect Phase 2.5 completion, the new 2.6 scope, and clarified ADR/behavioral details.

Flow diagram for re-scoped Phase 2.6 workstreams

flowchart LR
    P25["Phase 2.5<br/>CLI consolidation<br/>(complete)"] --> F["2.6.F<br/>platform_floor<br/>full_screen_TUI"]

    F --> A["2.6.A<br/>authoring_package"]
    F --> G["2.6.G<br/>home_browsers"]
    F --> L["2.6.L<br/>settings_theme_i18n"]
    F --> E["2.6.E<br/>chat_polish"]

    A --> B["2.6.B<br/>conversational_authoring"]
    D["2.6.D<br/>ctx_interpolation"] --> B

    H["2.6.H<br/>run_detail_data"] --> G
    K["2.6.K<br/>run_ops_resume"] --> G
    I["2.6.I<br/>providers_MCP_mgmt"] --> G

    G --> M26["M2.6_4<br/>home_managed_CLI"]
    B --> M23["M2.6_3<br/>conversational_authoring"]
    F --> M21["M2.6_1<br/>foundation"]
    A --> M22["M2.6_2<br/>authoring_core_shared"]
    L --> M25["M2.6_5<br/>first_class_experience"]
    M["2.6.M<br/>toolbelt"] --> M25
Loading

File-Level Changes

Change Details Files
Rewrite Phase 2.6 plan to cover a full-screen, Home-centric first-class CLI plus conversational authoring, with new workstreams, milestones, sequencing, and ADR requirements.
  • Replace the previous Phase 2.6 description (authoring and parity) with a much more detailed plan that centers on making the CLI a first-class, Home-centric product while retaining the authoring spine.
  • Define 13 workstreams (2.6.A–M) including authoring core promotion, conversational/wizard authoring, mid-session model reseat residuals, ctx interpolation, chat polish, full-screen TUI foundation, management browsers, durable run-detail, provider/MCP management, onboarding v2, run-ops resume follow-up, settings/theming/localization, and toolbelt breadth + rendering.
  • Expand outcomes, scope, out-of-scope, maintenance obligations, milestones (M2.6-1..5), sequencing (Mermaid graph), dependencies, exit criteria, ADR list, and risks to match the widened 2.6 remit.
  • Clarify that 2.6.C mid-session model reseat largely shipped in Phase 2.5 and now only carries residual tasks and cross-references.
  • Document explicit security and compatibility constraints (e.g., no breaking of ADR-0049 machine-output contract, secret-taint rules, governance floors) across new workstreams.
docs/roadmap/phases/phase-2.6-conversational-authoring.md
Align deferred-tasks with the new Phase 2.6 workstreams and record what was already completed in Phase 2.5 close-out.
  • Update last-updated metadata and note that all open items were triaged against Phase 2.6, adding explicit "Scheduled → 2.6.X" markers that map each now-doable item to a specific 2.6 workstream.
  • Mark several previously open items as completed in PR Phase 2.5 (CLI Consolidation) complete — 2.5.I/2.5.J close-out + Batch A–E → main #69 (e.g., media_surface wiring, advisory .gitignore matcher, approval reason capture behavior, test gaps, abort hints, reasoning panel bounds, Ctrl+T during approval) with explicit dates and PR references.
  • Assign concrete Phase 2.6 homes for security and correctness follow-ups such as MCP stdio consent gate, MCP header auth, tool-list cache and mid-call abort, run-resume consistency fixes, gate TOCTOU constraints, session budget pause/resume, approval-line zero-width hardening, extraRoots allowlist, and web_search credential resolver.
  • Clarify that some host-wiring and media-related obligations are now handled by earlier phases (e.g., 2.S) and adjust language accordingly.
  • Tighten or correct several explanations (e.g., schedule for live-nightly lane, mapping of budget resume and secret re-provide work to 2.6.K, run-submission idempotency to 2.6.H).
docs/roadmap/deferred-tasks.md
Update current roadmap status to mark Phase 2.5 as complete, declare Phase 2.6 as next, and briefly describe the re-scoped Phase 2.6.
  • Mark Phase 2.5 as complete (milestone M2.5-4, PR Phase 2.5 (CLI Consolidation) complete — 2.5.I/2.5.J close-out + Batch A–E → main #69, 2026-07-08) and describe its consolidation outcomes as historical.
  • Add Phase 2.6 as the immediate next phase, referencing the re-scoped Phase 2.6 doc and summarizing its goals: full-screen Home-managed CLI, management browsers, provider/MCP/settings management, onboarding v2, localization, and the authoring spine.
  • Fix a minor word-splitting issue in the "desktop app" wording.
  • Ensure the narrative around "Not started yet" now begins with Phase 2.6 before the desktop, VS Code, and managed/cloud phases.
docs/roadmap/current.md
Mark Phase 2.5 as complete and contextualize Phase 2.6 in the phase-2.5 CLI consolidation doc. docs/roadmap/phases/phase-2.5-cli-consolidation.md
Refresh top-level README and CLAUDE.md narrative to reflect Phase 2.5 completion and the re-scoped Phase 2.6.
  • In the main README, mark Phase 2.5 as complete with milestone M2.5-4 and PR Phase 2.5 (CLI Consolidation) complete — 2.5.I/2.5.J close-out + Batch A–E → main #69, and introduce Phase 2.6 as "Conversational Authoring and the First-Class CLI" with a concise description of its full-screen Home, management features, tools, localization, and authoring goals.
  • In CLAUDE.md, update the phase narrative to state that Phase 2.5 is complete (M2.5-4, PR Phase 2.5 (CLI Consolidation) complete — 2.5.I/2.5.J close-out + Batch A–E → main #69) and that Phase 2.6 has been re-scoped on 2026-07-08 to a broader "First-Class CLI" phase, linking to the Phase 2.6 doc.
  • Add cross-references in CLAUDE.md pointing readers to docs/roadmap/current.md for live status and to the Phase 2.6 doc for detailed planning.
  • Normalize spelling (e.g., "realizes" vs "realises") as part of the documentation clean-up.
  • Ensure the project overview describes the roadmap progression accurately with explicit mention of Phase 2.6 as the next phase.
README.md
CLAUDE.md
Adjust roadmap index and graph to show the new Phase 2.6 scope and keep the roadmap spine consistent with the detailed phase docs.
  • Update the Phase 2.6 label in the Mermaid roadmap graph to "conversational authoring + first-class CLI" instead of a narrower authoring-only description.
  • Rewrite the Phase 2.6 table entry to describe the widened scope: full-screen Home-managed CLI, management and drill-down features, toolbelt breadth, localization, onboarding v2, run-ops follow-up, plus the @relavium/authoring spine.
  • Tighten the Phase 2.6 acceptance criteria in the table row to match the new phase doc: Home managing everything, toolbelt shipped behind a reviewed render contract, attributed run history, localization, and ADR acceptance.
  • Keep surrounding phase descriptions (Phase 2.5, Phase 3, etc.) intact while aligning linking to the updated Phase 2.6 doc.
docs/roadmap/README.md

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey - I've left some high level feedback:

  • The Phase 2.6 roadmap doc has become very long and dense; consider adding a short front-loaded TL;DR/summary section (e.g. 5–7 bullets with the key outcomes and workstreams) so maintainers and contributors can quickly re-orient before diving into the detailed workstream text.
  • You repeat ADR status and acceptance conditions in multiple sections (workstreams, exit criteria, risks); consolidating these into a single small table or list of ADRs + status + owning workstream and then linking to it would reduce future drift between sections and make it easier to keep the plan current.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- The Phase 2.6 roadmap doc has become very long and dense; consider adding a short front-loaded TL;DR/summary section (e.g. 5–7 bullets with the key outcomes and workstreams) so maintainers and contributors can quickly re-orient before diving into the detailed workstream text.
- You repeat ADR status and acceptance conditions in multiple sections (workstreams, exit criteria, risks); consolidating these into a single small table or list of ADRs + status + owning workstream and then linking to it would reduce future drift between sections and make it easier to keep the plan current.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

An error occurred during the review process. Please try again later.

📝 Walkthrough

Walkthrough

Documentation-only update marking Phase 2.5 (CLI Consolidation) complete with PR #69/M2.5-4 milestone details, and rewriting Phase 2.6 (Conversational Authoring and the First-Class CLI) scope across README, CLAUDE.md, roadmap index, current-state, phase docs, and deferred-tasks scheduling markers.

Changes

Roadmap documentation update

Layer / File(s) Summary
Top-level status summaries
README.md, CLAUDE.md
Status sections updated to state Phase 2.5 is feature-complete (v0.1.1, PR #69, M2.5-4) and introduce the Phase 2.6 conversational-authoring/first-class-CLI re-scope.
Roadmap index and current-state docs
docs/roadmap/README.md, docs/roadmap/current.md
Phase dependency graph and index table updated for Phase 2.6's Home-centric scope; current-state narrative confirms Phase 2.5 completion and expands the "not started" Phase 2.6 description.
Phase 2.5 close-out document
docs/roadmap/phases/phase-2.5-cli-consolidation.md
Status marked COMPLETE with PR #69 closeout framing, updated workstream statuses, milestone table row, and exit-criteria confirmation.
Phase 2.6 rewritten roadmap document
docs/roadmap/phases/phase-2.6-conversational-authoring.md
Full rewrite expanding goals, scope, workstreams 2.6.A–2.6.M, deferred-task pull-in mapping, milestones/sequencing diagram, dependencies, exit criteria, and required ADRs.
Deferred-tasks scheduling refresh
docs/roadmap/deferred-tasks.md
"Last updated" note refreshed; multiple items marked DONE with PR references, and remaining items tagged with 2.6.B–2.6.M scheduling markers.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • HodeTech/Relavium#59: Updates the same Phase 2.5/2.6 roadmap docs, including phase-2.6-conversational-authoring.md and README/roadmap status narrative.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: re-scoping Phase 2.6 around Conversational Authoring and the First-Class CLI.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch development

Comment @coderabbitai help to get the list of available commands.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request officially marks Phase 2.5 (CLI Consolidation) as complete and introduces a comprehensive rewrite of the Phase 2.6 roadmap, re-scoping it from "Conversational Authoring and Parity" to "Conversational Authoring and the First-Class CLI". The updated plan expands the scope to turn the CLI into a full-screen, Home-centric product with robust management browsers, provider/MCP controls, settings, theming, localization, and an expanded toolbelt. The single review comment correctly identifies a potential ambiguity in the risk table where a reference to "ADR #9" could be confused with the existing ADR-0009, and provides a clear suggestion to clarify it as the new Toolbelt ADR.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

| Package promotion over-engineers a CLI-only need | The maintainer decision is multi-surface (desktop/VS Code also consume); the add-package ADR records the rationale |
| **Scope breadth** — thirteen workstreams invite drift | Milestone gating (M2.6-1..5); the additive arms (E/L parts, M render-v2) can defer individual items without breaking the spine; the pull-in table keeps deferred-tasks as the single overflow home |
| Full-screen renderer performance/fragility on ink | The floor bump unlocks ink 7; the component harness carries performance regression thresholds; the inline renderer is retained as a first-class fallback, and non-TTY paths never change |
| Rendering args/diffs leaks sensitive data (reverses a 2.5 posture) | Sanctioned only by ADR #9 + a mandatory security review; every string passes the shared sanitize floor; bounded previews; secrets structurally excluded (keychain-only, never in tool args by construction) |

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Risk tablosundaki 'Sanctioned only by ADR #9' ifadesi, projedeki mevcut ADR-0009 (Git-native workflow YAML) ile karıştırılabilir. Burada kastedilen yeni önerilen 9. ADR (Toolbelt additions + tool-render/approval-preview contract) olduğu için, kafa karışıklığını önlemek adına bu ifadenin netleştirilmesi veya geçici bir tanımlayıcı (örneğin 'ADR-0067' veya 'New ADR #9') kullanılması daha sağlıklı olacaktır.

Suggested change
| Rendering args/diffs leaks sensitive data (reverses a 2.5 posture) | Sanctioned only by ADR #9 + a mandatory security review; every string passes the shared sanitize floor; bounded previews; secrets structurally excluded (keychain-only, never in tool args by construction) |
| Rendering args/diffs leaks sensitive data (reverses a 2.5 posture) | Sanctioned only by the new Toolbelt ADR (New ADR #9) + a mandatory security review; every string passes the shared sanitize floor; bounded previews; secrets structurally excluded (keychain-only, never in tool args by construction) |

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 2

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
docs/roadmap/phases/phase-2.5-cli-consolidation.md (1)

550-580: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Keep the color-policy wording consistent. The status block says --color / --no-color > NO_COLOR > FORCE_COLOR is done, but the Tasks line still says only --no-color is honoured today.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/roadmap/phases/phase-2.5-cli-consolidation.md` around lines 550 - 580,
The roadmap note has conflicting color-policy status text: the completed status
block says `--color` / `--no-color` overrides `NO_COLOR` and `FORCE_COLOR`, but
the Tasks line still says only `--no-color` is honored. Update the wording in
this section so it consistently reflects the implemented color-resolution order
across the roadmap narrative, using the existing roadmap markers and references
in the phase document.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@CLAUDE.md`:
- Around line 74-75: The release-status text in CLAUDE.md is inconsistent with
the root README/current docs, so update the status wording to match the same
phrasing used elsewhere for v0.1.1 rather than saying it is already published.
Locate the release note sentence near the Phase 2.5 / M2.5-4 history entry and
revise only the status phrase so it reflects the
cut-but-not-yet-publicly-published state consistently across docs.

In `@docs/roadmap/phases/phase-2.6-conversational-authoring.md`:
- Around line 93-95: The Required ADRs section is missing the placeholder for
the new validator dependency introduced by output_schema deep JSON-Schema
conformance. Update the roadmap entry in the conversational authoring phase doc
to add a validator ADR placeholder alongside the existing required ADR
references, and keep it aligned with the mention of output_schema so it’s easy
to find later.

---

Outside diff comments:
In `@docs/roadmap/phases/phase-2.5-cli-consolidation.md`:
- Around line 550-580: The roadmap note has conflicting color-policy status
text: the completed status block says `--color` / `--no-color` overrides
`NO_COLOR` and `FORCE_COLOR`, but the Tasks line still says only `--no-color` is
honored. Update the wording in this section so it consistently reflects the
implemented color-resolution order across the roadmap narrative, using the
existing roadmap markers and references in the phase document.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 67e3f6ed-a932-40f6-b861-ae22f77f3d55

📥 Commits

Reviewing files that changed from the base of the PR and between 7fac102 and abd4393.

📒 Files selected for processing (7)
  • CLAUDE.md
  • README.md
  • docs/roadmap/README.md
  • docs/roadmap/current.md
  • docs/roadmap/deferred-tasks.md
  • docs/roadmap/phases/phase-2.5-cli-consolidation.md
  • docs/roadmap/phases/phase-2.6-conversational-authoring.md

Comment thread CLAUDE.md Outdated
Comment thread docs/roadmap/phases/phase-2.6-conversational-authoring.md
@cemililik
cemililik merged commit a58f3ae into main Jul 8, 2026
9 of 10 checks passed
@sonarqubecloud

sonarqubecloud Bot commented Jul 8, 2026

Copy link
Copy Markdown

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.

1 participant