Skip to content

docs(adr): ADR-022 reasoning is streamed and shown, never fed back - #19

Merged
SaketMunda merged 1 commit into
masterfrom
plan/adr-022-reasoning
Oct 10, 2026
Merged

SaketMunda merged 1 commit into
masterfrom
plan/adr-022-reasoning

Conversation

@SaketMunda

Copy link
Copy Markdown
Owner

Adds the reasoning work to MOD-3 (now M) and UI-3.

Module

Brief: roadmap/modules/<ID>-*.md

What changed

Definition of Done

  • Every item in the brief's Acceptance criteria is met and demonstrated (test or recorded manual check).
  • scripts/verify.sh is green (typecheck, lint, unit tests).
  • New logic in packages/* has unit tests (target ≥80% line coverage on new files). UI logic (state, reducers, parsers) is tested. Pure presentation isn't required to be.
  • No any / as unknown as at package boundaries. Every external input is validated with zod.
  • No new network destinations except ones the user configured. No telemetry.
  • Errors are handled: user-facing failures show an actionable message, and nothing fails silently.
  • Brief updated: Status, Handoff notes (what was built, deviations, known gaps, follow-ups).
  • PR opened from mod/<ID>-<slug>, description links the brief, CI green.

Acceptance criteria evidence

# Criterion Evidence

Adds the reasoning work to MOD-3 (now M) and UI-3.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@SaketMunda
SaketMunda merged commit 846a036 into master Oct 10, 2026
1 check passed
@SaketMunda
SaketMunda deleted the plan/adr-022-reasoning branch October 10, 2026 12:33
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