Skip to content

feat(dart): phase 2 task detail, comments and New task - #10

Merged
Interstellar-code merged 124 commits into
mainfrom
dart/phase-2-integration
Nov 6, 2026
Merged

Interstellar-code merged 124 commits into
mainfrom
dart/phase-2-integration

Conversation

@Interstellar-code

@Interstellar-code Interstellar-code commented Nov 6, 2026 •

Copy link
Copy Markdown
Owner

Summary

Dart phase 2 (plan docs/plans/dart-phase-2.md, tasks T1–T8): Dart task detail editing, comments (T5), New task (T7), web provider.v1 update-required gate, remote/web transport parity, and end-to-end tests over fakes (T8).

Testing (fakes only, no live Dart calls)

  1. Vitest: 2,627 passed; Dart UI suites 426 passed.
  2. Typecheck (web + node), translation checks (4), changed-code quality gate, renderer boot-graph check: all green.
  3. Focus-read ratchet: baselines bumped for deliberate live-host re-checks around writes/creates (config/focus-setting-read-baseline.txt, each with a reason).
  4. Separate Opus review of the cleanup round: approved, low findings only.

Live Dart API check (backend)

Orca main-process Dart code run against the real Dart API: reads, create, edit of every field, comments/replies/paging/duplicate guard, error mapping all pass. It found 3 mismatches, fixed in 3308df4 and 4b58d01 (comment errors on unknown tasks; priority casing in the select chip). Test task created then deleted. Rate-limit (429) path not exercised.

Known out-of-diff failures

  1. tests/e2e/cross-version-wire/*: same failure on clean main; needs release tags locally (git fetch --tags origin).
  2. src/cli/handlers/dart.test.ts: fails on clean main too. (Orchestration CLI tests pass after a fresh CLI build; earlier failures were a stale build.)

Known limits

  1. Locked New task fields keep standard disabled styling.
  2. Same-author same-text comment duplicate is blocked until "Post anyway".
  3. Manual UI checklist in a dev build still to run.

🤖 Generated with Claude Code

Interstellar-code and others added 30 commits November 4, 2026 09:46
…its, writes.v2 and write outcome

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
….v2 probe

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…patch-refused message, header notes

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
… required requestSent for writes

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
… and traces

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…t-size admission

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…aries

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…drop description from field updates

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…t disconnected

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…o the frozen contracts

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…wn returns to confirmed; refusals head the notices

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…se description at the type

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e write

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Interstellar-code and others added 27 commits November 5, 2026 12:38
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Resolves en.json by union, drops 3 unused keys, and gives the no-viewer
assignee test a host without a viewer now that each host reports its own.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…y cases

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…e import cycle

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…ping

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…nd fix lint findings

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…ng disabled fields

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
… fields

Reverts a636894 and 2121a3b except the typed reply helper in the
recovery-attempts test. Locked fields keep the standard disabled look.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Listing comments for an unknown task is a 200 empty page, not a 400, so the
400 to taskNotFound mapping is removed. addComment on an unknown task returns
400 with "taskId: No Task found", now mapped narrowly to dart.taskNotFound
(still notApplied).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Real Dart returns High/Medium while dart.config lists high/medium. The chip now
resolves the applied value to the config option case-insensitively, so there is
no duplicate entry and writes keep the config spelling.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@Interstellar-code
Interstellar-code merged commit 5fe2edc into main Nov 6, 2026
2 of 3 checks passed
@Interstellar-code
Interstellar-code deleted the dart/phase-2-integration branch November 7, 2026 16:09
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