6 Commits

Include renames

Author	SHA1	Message	Date
jpmschweitzerandClaude Fable 5	5d52694889	serve IPC clients with one await-for read loop (T-372) ... The async onData handler never paused its subscription, so pipelined requests interleaved mid-handler — violating D-72's serial-dispatch contract — while the shared StringBuffer could re-frame underneath an in-flight await and the per-chunk utf8.decode corrupted runes split across reads. One `await for` over a persistent Utf8Decoder + LineSplitter fixes framing, decoding, and serialization at once. Tests: two frames pipelined in one write dispatch strictly in order; a frame split mid-rune across writes decodes intact. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>	2026-06-12 01:13:43 +02:00
jpmschweitzerandClaude Opus 4.8	6d0ebab721	chore: adopt Dart 3.9 toolchain — honest floor + tall-style reformat (T-353) ... Raise the declared minimums in pubspec.yaml to what our deps already require: Flutter >=3.35.0 / Dart >=3.9.0 (was 3.19.0 / 3.5.0). alchemist 0.12 needs Flutter 3.32; Dart 3.9 first ships in Flutter 3.35, so 3.35 is the binding floor. Pin the exact build toolchain in .fvmrc (Flutter 3.44.1). Moving to the Dart 3.9 language level switches `dart format` to the new "tall" style and enables two new lints. This commit is the resulting mechanical churn, isolated from any behaviour change: - whole-tree `dart format` reformat (tall style) - `dart fix` for unnecessary_underscores + use_null_aware_elements No runtime behaviour change; `make test` green. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-11 12:11:53 +02:00
jpmschweitzerandClaude	d460e8e7d5	T-124 follow-up: actually delete the parent dir before testing create ... Previous follow-up only proved the existing-dir branch on this host because the dir was already there. Now the test deletes the parent when it's safe to do so (exists + empty) so the create-if-missing branch fires and counts toward coverage. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-18 14:57:10 +02:00
jpmschweitzerandClaude	2aed695cd9	T-124 follow-up: two more server tests to clear the gate ... socketPath-before-start + prepareParentDir-creates-missing — adds two assertions that net a few more covered lines and put the coverage gate back over 95%. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-18 14:55:29 +02:00
jpmschweitzerandClaude	5428b5a2db	T-124 follow-up: cover server.dart error paths ... Net new uncovered lines in lib/src/ipc/server.dart from the T-124 commit (handler-throw → toolError, non-request → userError, in-flight client teardown on stop, multi-request on one connection) are covered here so the coverage gate holds at 95%. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-18 14:53:36 +02:00
jpmschweitzerandClaude	c4bccd35a3	T-124: unix-domain IPC server, wired into Flutter app boot ... First slice of T-99 (the D-56-path-a IPC server). What this lands: * lib/src/ipc/paths.dart rewritten — `workspaceSocketPath(root)` returns the per-workspace path per D-70 (FNV-1a 64-bit hash, hex, no crypto dep — D-70 amended in this commit to record the hash choice). Old `defaultSocketPath()` removed; the lone fallback in facade.dart kept with a clear placeholder pending T-127. * lib/src/ipc/server.dart — IpcServer class. ServerSocket.listen accept loop (D-72), 0600 socket + 0700 parent (D-71), stale-node probe + unlink on start, refuses to clobber a live listener. * lib/main.dart — IpcServer started after the first dispatcher is built and swapped on project open (workspace path changes). Failure logged but non-fatal so the UI still works without IPC. * 11 server tests + 5 path tests cover socket modes, multi-conn, stale unlink, live-conflict, idempotent start/stop. T-99 children downstream of T-124 (T-125 / T-126 / T-127 / T-130) are now unblocked. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-18 14:51:35 +02:00