Skip to content

docs(roadmap): capture deferred Explorer follow-ups - #58

Merged
tcballard merged 1 commit into
mainfrom
claude/explorer-followup-notes
Jun 10, 2026
Merged

docs(roadmap): capture deferred Explorer follow-ups#58
tcballard merged 1 commit into
mainfrom
claude/explorer-followup-notes

Conversation

@tcballard

@tcballard tcballard commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

Captures, as an informal future-scoping note under rac/roadmaps/future/, two design decisions made deliberately during the v0.8.x Explorer series and flagged for a later pass. Neither is a defect; this is housekeeping so the follow-ups are tracked rather than buried in merged PR descriptions (#54, #55).

What it records

  1. Recommendation "impact" copy lives in Explorer, not Core (deferred from v0.8.3). The impact sentence is Explorer-authored presentation copy keyed by the Core finding code; finding/severity/action/path all come from Core's review service. Follow-up option noted: add an additive impact field to ReviewIssue (a versioned JSON-contract change, ADR-007) so Core owns it and CLI/JSON consumers see it.
  2. Editor preference persistence + terminal-editor support (deferred from v0.8.4 / v0.8.6). The editor is resolved from $VISUAL/$EDITOR and launched fire-and-forget (GUI editors). Follow-up options noted: a persisted editor preference + first-run selection (the v0.8.6 preferences file already has a home for it), and TUI suspend/resume for terminal editors.

Verification

  • The note classifies as unknown (confidence 0.0), exactly like the existing rac/roadmaps/future/ notes, so it is skipped by validation and disturbs no contract.
  • rac validate rac/ exit 0 · rac relationships rac/ --validate exit 0 · rac review rac/ no priority 1–2 findings · tests/test_dogfood.py passes.

Notes For Reviewer

  • Single doc file; no code, no behavior change.
  • Branched off main after the full v0.8.x series merged — standalone, not stacked.

Implementation Process

Implemented with AI assistance.

Final scope, review, and acceptance decisions were made by the maintainer.

Record two design decisions made deliberately during the v0.8.x
Explorer series and deferred for a later pass: recommendation impact
text living in Explorer rather than Core's review service, and editor
preference persistence plus terminal-editor support. Informal future
note (classifies as unknown, like its siblings); neither is a defect.
@tcballard
tcballard merged commit f256166 into main Jun 10, 2026
2 checks passed
@tcballard
tcballard deleted the claude/explorer-followup-notes branch June 11, 2026 20:10
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