361 Commits

This Branch

This Branch

All Branches

Author	SHA1	Message	Date
jpmschweitzerandClaude Opus 4.7	a0bd459b7a	test sweep: cover widgets/src markdown / code-block / accordion / scrollbar / pty-view (T-91) ... 12 widget tests in test/widgets/more_widgets_test.dart covering the next set of zero-coverage widgets: - ClideMarkdown: headings + paragraphs + lists + code-block + hr + record-id link rendering through the md.Document → Widget compiler. - ClideCodeBlock: plain source (no language), unavailable grammar fallback to plain text, didUpdateWidget re-highlight. - ClideAccordion: collapsed (children hidden) vs expanded, tap toggles, leading-widget slot renders. - ClideScrollbar: wraps a scrollable child, ScrollbarTheme inherited widget + its updateShouldNotify contract. - ClidePtyView: Semantics live region wrapping a TerminalView. Coverage: jumps from 86.58% to 89.08%. Floor bumped to 89. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 19:04:21 +02:00
jpmschweitzerandClaude Opus 4.7	86c3325de2	test sweep: cover widgets/src zero-coverage primitives (T-91) ... 12 widget tests in test/widgets/zero_coverage_widgets_test.dart covering six widgets that had 0% coverage: - ClidePalette: closed → SizedBox.shrink; open → input + command list rendered; tap-to-invoke. - ClideFilterBox: debounced onChanged (timer-flushed), clear button visibility + reset, onSubmitted forward. - ColumnHat: left / center / right factories all render; center label joins project + branch with " > " separator; falls back to "clide" when neither set. - ClideIconRail: one button per item, semantics labels, onSelect fires on tap. - ClideSpine: label rotation + badge dot + onExpand tap; right-side variant. - ClideResizeBorder: wraps child + renders 8 resize zones. Coverage (totals lifted by these + the prior stale-lcov merge): jumps from 83.92% to 86.58%. Floor bumped to 86. Target reset to 90% per user direction: keep grinding sub-areas until total clears 90%, then keep pushing as reasonable. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 16:45:07 +02:00
jpmschweitzerandClaude Opus 4.7	dc86fac85e	test sweep: cover SettingsStore project + ext scopes + YAML emitter (T-91) ... Seven new tests in test/kernel/src/settings_test.dart covering the SettingsStore branches the existing 6-test suite didn't reach: - project-scoped set + get round-trip when projectDir is configured (covers the project-file readFile branch + the project-scope set path). - setting a project key without a project throws StateError. - ext.* keys default to app scope; project overrides app when both are set (covers the project-overrides-app branch in _lookup). - setProjectDir(null) clears the in-memory project values. - YAML emitter handles every scalar branch: bool, num, string (simple + special-chars + empty), list (mixed types incl null), and round-trips through reload (covers _emitScalar + _emit branches for null / List / unknown types). - load tolerates a malformed YAML file (_readFile catch path). - load returns empty when file is missing or blank. Coverage: kernel/src/settings.dart 104/123 -> 112/123 (91%). Remaining lines are a couple of corner-case formatting paths in _emit when the top-level value isn't a Map (the public API always serializes a Map, so they're effectively unreachable through normal use). Total coverage 83.85% -> 83.92%. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 08:27:04 +02:00
jpmschweitzerandClaude Opus 4.7	e2d98445f5	test sweep: cover ExtensionManager contribution branches + ctx (T-91) ... Five new tests in test/kernel/src/extensions_manager_test.dart covering the contribution-type case branches and the _ExtensionContext passthrough surface that the existing 8-test suite didn't reach: - TrayItemContribution lands in TrayRegistry on activate; deactivate removes it. - StatusItem + ToolbarButton contributions activate/deactivate cleanly through PanelRegistry. - LayoutPresetContribution exercises the no-kernel-side-wiring case branch (consumed by default-layout's own activate()). - Every _ExtensionContext getter (log / events / messages / settings / theme / i18n / panels / arrangement / commands / palette / clipboard / files / notify / dialog / tray / secrets / os / net / focus / project / ipc) returns the same instance the kernel exposes — locks in the passthrough contract. Coverage: kernel/src/extensions_manager.dart 85/130 -> 122/130 (94%). Remaining 8 lines are the disabled-by-default + persisted- disabled paths plus the deactivate-during-failed-activate cleanup — edge cases reachable only when extensions throw. Total coverage 83.39% -> 83.85%. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 08:22:33 +02:00
jpmschweitzerandClaude Opus 4.7	2a7875e1a9	remove OsBridge openURL/reveal tests — they spawn OS popups ... services_bigger_test.dart's openURL + reveal tests called the real OsBridge methods, which Process.run xdg-open / open / explorer. On a desktop session the OS dispatcher (KIO on KDE, etc.) surfaces a "Could not read file …" error dialog for the bogus URL, owned by the desktop session not the test process — it persists after the test exits and a fresh run spawns another one. The user has to manually close each popup, blocking CI from running unattended. Coverage value was minimal (anyOf(true, false)). Leaving fire() and the rest of the OsBridge bus-event surface; command-shape coverage of openURL/reveal can come from integration tests where a real OS dispatcher is desired. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 08:02:37 +02:00
jpmschweitzerandClaude Opus 4.7	7ba0500f75	test sweep: cover kernel toolchain + medium services (T-91) ... Two test files chasing the bigger kernel residuals: - test/kernel/src/toolchain_test.dart (10 tests): Toolchain defaults + missing list, applyResolved with full / partial paths, waitForResolution sync + async, Toolchain.resolvePaths against the current workspace + dugite detection + PATH fallback, resolveToolchainPaths top-level matches the static. - test/kernel/src/services_bigger_test.dart (15 tests): DialogRouter show/dismiss/queue/notify, FileServices.pick* UnimplementedError trio + notifyDropped event, OsBridge openURL / reveal / fire, WindowControls setStyle idempotency + MissingPlugin-safe platform-channel methods + isMaximized success path, SchedulerTier intervals + SchedulerTick payload + start/dispose. Coverage: kernel/src/toolchain.dart 37/95 -> 67/95 (71%); dialog.dart 20/47 -> 27/47 (remaining is the DialogHost widget, needs a real overlay tree); files.dart 1/16 -> 14/16 (88%); os.dart 1/26 -> 19/26 (73%); window_controls.dart 2/25 -> 25/25 (100%); scheduler.dart 14/41 -> 18/41 (remaining is the isolate ticker entry point, only fires after a real project-open event). Total coverage 82.43% -> 83.57%; floor bumped to 83. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 08:00:23 +02:00
jpmschweitzerandClaude Opus 4.7	8efcf186e6	test sweep: cover kernel ProjectManager + RecentProject (T-91) ... 18 tests in test/kernel/src/project_test.dart driving ProjectManager with an injected onValidateProject hook so the manager doesn't shell out to git in tests. Covers: - RecentProject: relativePath HOME collapsing, timeAgo bucketing (minutes / hours / yesterday / days / weeks / months), toJson / fromJson round-trip, fromJson tolerance for missing fields. - ProjectManager.open: non-git path (returns false), valid path (sets current, emits ProjectOpened, registers in recents), re-open deduplication, recents capped at 10. - ProjectManager.close: resets current, emits ProjectClosed, no-op when already closed. - ProjectManager.loadRecents: happy path, malformed-value tolerance, no-setting clears to empty. - ProjectManager.openLast: no-stored, stored-but-missing, stored-and-exists. - ProjectManager.resolveProject without injected validator falls back to Process.run. Coverage: kernel/src/project.dart 2/72 -> 66/72 (92%). Remaining 6 lines are the _currentBranch helper + ProjectOpened-event-time branch shake (need a real git in the sandbox to materialise the branch). Total coverage 81.64% -> 82.42%; floor bumped to 82. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 07:40:12 +02:00
jpmschweitzerandClaude Opus 4.7	2d123681be	test sweep: cover kernel Tier-0 service stubs (T-91) ... 19 tests in test/kernel/src/services_stubs_test.dart covering six small Tier-0 service classes: ClideClipboard (typed write + readAs, bounded LIFO history, typed history miss, writePlain/readPlain via mocked platform channel, write-with-toPlain OS sync, clear), FocusTracker (setActive flip + same-value no-op + clear), NetworkStatus (default + setState idempotency), SecretsVault (per-extension isolation, read/write/delete/deleteAll), TrayRegistry (add/remove + priority sort + unknown-id no-op), Notifications (info/warn/error/ success levels, dismiss + unknown-id no-op). Coverage: kernel/src/clipboard.dart 1/23 -> 23/23; focus.dart 1/12 -> 12/12; net.dart 0/6 -> 6/6; secrets.dart 0/8 -> 8/8; tray.dart 0/7 -> 7/7; notify.dart 4/23 -> 23/23. All six files now at 100%. Total coverage 80.81% -> 81.64%; floor bumped to 81. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 07:36:22 +02:00
jpmschweitzerandClaude Opus 4.7	3339d3fcb5	test sweep: cover default_layout commands (T-91) ... Eight new tests in test/builtin/default_layout/widget_test.dart covering the DefaultLayoutExtension command handlers the existing four-test suite didn't reach: - All commands return the _notActivated() error pre-activate (sweeps every command's defensive null-context branch in one loop). - palette.toggle flips the palette's open state. - sidebar.collapse + context.collapse toggle their slots. - panel.focus.left / .middle / .right — including the auto-expand branches when a focused side is collapsed. - panel.focusMode toggles focus mode on the active slot. - panel.focusMode.exit unwinds in order: focus mode → editor → palette, with the empty-data no-op fallback. - editor.open + editor.close including the close-already-closed no-op. - sidebar.section.N: auto-expand sidebar + no-op when no tabs are contributed. Coverage: builtin/default_layout/src/extension.dart 60/167 -> 139/167 (83%). Remaining 28 lines are the _restoreLayout / _persistLayout persistence paths and the section-activation happy path, which need a populated tab list + a projectDir on settings to exercise — out of scope for this batch. Total coverage 79.80% -> 80.81%; floor bumped to 80. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 07:30:52 +02:00
jpmschweitzerandClaude Opus 4.7	3a6e906dc1	test sweep: cover welcome_view tail paths (T-91) ... Eight new widget tests in test/builtin/welcome/widget_test.dart covering the WelcomeView paths the existing render-it test didn't reach: - TIPS card visibility: shown when viewport height > 640, hidden when shorter (LayoutBuilder branch). - Status line tri-state: "checking…" before toolchain resolution, "application ok" when all tools present, missing-tools listing when some are absent. - Theme-name link tap fires the theme.pick command via the registry. - Open folder tap with no native picker (returns null, function returns early) and with MissingPluginException (falls through to the path dialog via kernel.dialog.show). Coverage: builtin/welcome/src/welcome_view.dart 84/221 -> 120/221 (54%). Remaining 101 lines are inside the _OpenProjectDialog, _NotARepoDialog, and recent-project rows — they need either a DialogHost in the test harness or a populated recents list, which is materially more setup than the rest of this batch's pattern. Total coverage 79.34% -> 79.80%. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 07:25:17 +02:00
jpmschweitzerandClaude Opus 4.7	7b9feba987	test sweep: cover src/pty/env + native_pty error paths (T-91) ... Two test additions for the pty subsystem: - test/pty/env_test.dart (9 tests): PtyException.toString with and without errno, expandedPath on every platform branch, mergePtyEnv override precedence (clide defaults > process env > explicit overrides), clidePtyEnvDefaults shape. - test/pty/session_test.dart extended (4 new tests): bare-command PATH resolution, non-existent workingDirectory triggering the chdir-failed diagnostic via the child branch, non-existent executable triggering the exec-failed diagnostic, resize on a live PTY. Coverage: pty/env.dart 5/19 -> 9/19 (remaining 10 lines are the macOS-only PATH-merge branch, only reachable when Platform.isMacOS). pty/errors.dart 0/4 -> 3/4 (remaining 1 is a const-ctor phantom). The new pty session tests run under `dart test --tags forkpty` so their branch coverage doesn't surface via lcov, but the code paths (chdir failure, execve failure, PATH resolution) are now verified. Total coverage 79.26% -> 79.34%. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 07:18:33 +02:00
jpmschweitzerandClaude Opus 4.7	b1e9abd6aa	test sweep: cover daemon editor / files / pane / pql commands (T-91) ... Extends the four existing daemon command suites with the verbs + error paths the originals didn't reach: - editor_commands_test (12 new): unreadable-path FileSystemException catch, editor.active with no buffer, editor.activate requires + validates id, editor.read no-active / unknown-id, editor.set-selection no-active / clamped, editor.set-content with + without selection, editor.save no-active, editor.close requires + validates id. - files_commands_test (8 new): files.read happy + missing-path + empty-path + outside-root + missing-file, files.ls outside-root, files.watch idempotent, FilesService.atCwd resolver. - pane_commands_test (10 new): argv-non-string rejection, unknown kind rejection, env passthrough, close / write / focus / resize missing-id and unknown-id validations, write requires bytes_b64 or text, malformed base64 rejection. - pql_commands_test (14 new): pql.files glob + limit, pql.backlinks happy, pql.outlinks missing, pql.tags, pql.query + pql.search happy paths + missing-arg user_error, pql.decisions.read missing + happy, pql.decisions.show with --with-refs / --with-tickets, pql.decisions.list domain filter, pql.tickets.list multi-filter, pql.tickets.show missing + happy, pql.tickets.status missing + partial-args, pql.tickets.board with team. Coverage: src/daemon/editor_commands.dart 64/100 -> 88/100; files_commands.dart 33/70 -> 64/70 (91%); pane_commands.dart 66/92 -> 78/92 (85%); pql_commands.dart 62/149 -> 105/149 (70% — remaining 44 lines are the per-command PqlException catch branches that only fire when the pql subprocess itself fails mid-call). Total coverage 77.92% -> 79.26%; floor bumped to 79. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 07:13:38 +02:00
jpmschweitzerandClaude Opus 4.7	8eaf7446a1	test sweep: cover daemon dispatcher + git_commands (T-91) ... Two test additions toward finishing src/daemon/: - test/daemon/dispatcher_test.dart (6 tests): the entire DaemonDispatcher surface — ping + version default handlers, unknown-command not-found error, register routing, isEmpty before/after registration, clear preserving ping + version. - test/daemon/git_commands_test.dart extended (14 new tests): the git.* commands the existing suite didn't reach — git.diff with paths, git.stage-hunk + git.unstage-hunk (happy + missing-patch + bad-patch GitException), git.branches, git.checkout (happy + missing + unknown), git.log with count, git.push + git.pull both with and without a local bare remote, git.stage accepting a String single-path arg via _pathList. Coverage: src/daemon/dispatcher.dart 8/22 -> 22/22 (100%); src/daemon/git_commands.dart 73/146 -> 122/146 (84%). The remaining 24 lines in git_commands are mid-call GitException catch branches that need the git client to fail after the dispatcher accepted the request. Total coverage 77.05% -> 77.92%. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 07:09:33 +02:00
jpmschweitzerandClaude Opus 4.7	035491f7db	test sweep: finish src/git/ (T-91) ... Closes out the three remaining files in src/git/ following the client.dart batch: operations.dart (25 new tests): - GitException.toString - GitLogEntry.toJson (body present + absent) - gitStage / gitUnstage / gitStashPop / gitPush / gitPull error paths (no remote, empty stash, bogus path) - gitUnstage with empty paths (unstage everything) - gitStageHunk + gitUnstageHunk via _applyPatch - _applyPatch surfaces stderr in GitException on a bad patch - gitBranches happy + non-repo - gitCheckout happy + unknown-branch error - gitPull + gitPush round-trip against a local bare remote - gitLog / gitCurrentBranch / gitBranches non-repo fallbacks - gitDiscard with empty list short-circuit - gitBin resolver diff.dart (7 new tests): - gitDiff with paths argument and non-repo fallback - GitHunk.toPatch round-tripping header + every DiffLineKind - GitDiff.toJson with / without oldPath - parseDiffOutput edge cases: pre-diff garbage skipped, "\ No newline" marker captured as a header line, malformed @@ header skipped via _parseHunk null return status.dart (4 new tests): - branch.upstream + branch.ab populated from a local bare remote - non-git directory returns an empty branchless status - rename in porcelain output captures the original path - parsePorcelainV1 short-part / empty-part / empty-input edge cases Coverage: src/git/diff.dart 106/124 -> 124/124 (100%); src/git/operations.dart 88/155 -> 148/155 (95% — remaining 7 lines are gitPull/gitPush variants that need credentials, and the gitBranches non-zero stderr path); src/git/status.dart 86/98 -> 95/98 (97% — remaining 3 are the ProcessException-during-call catch branches, only reachable if git crashes mid-call). Total coverage 76.04% -> 77.05%; floor bumped to 77. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 07:04:30 +02:00
jpmschweitzerandClaude Opus 4.7	7c8114ecbe	test sweep: cover src/git/client.dart (T-91) ... 23 integration tests in test/git/client_test.dart driving a real git binary against per-test sandbox repos. Covers every GitClient public method that wasn't already exercised through the daemon command suite: - Queries: status (clean + with upstream tracking ahead/behind), diff with explicit paths, log, currentBranch, branches (with the current-marker check), repoRoot (in-repo + non-repo paths). - Mutations: unstage with paths, commit-returns-hash, discard, discard with empty list, stash + stashPop with includeUntracked, checkout, stageHunk + unstageHunk via _applyPatch. - Error surface: mutations throw GitException on non-zero exit, _applyPatch surfaces stderr from a bad patch, bad git binary path → ProcessException → GitException, queries return empty fallbacks on a non-git workDir. - parseLog standalone parser: empty input, short-record skip, full record with body, 5-field record without body. Coverage: src/git/client.dart 76/141 -> 124/141 (88%). The remaining 17 lines are the pull / push paths (would need a real remote with matching credentials — out of scope) and the status GitException catch branches that require an in-flight git process failure mid-call. Total coverage 75.42% -> 76.04%; floor bumped to 76. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 06:46:13 +02:00
jpmschweitzerandClaude Opus 4.7	3175b7a633	test sweep: cover kernel/src/syntax (T-91) ... Two test files covering the pure-Dart and fallback paths of the tree-sitter integration. The native-FFI parsing depth stays untested — DynamicLibrary.open('libtree-sitter.so') doesn't resolve under the flutter test runner because the bundled lib sits at native/linux-x64/ rather than on the linker search path. Real parsing coverage would need an integration-test harness that copies or symlinks the lib into the runner's working directory. - test/kernel/src/syntax/language_map_test.dart (6 tests): every branch of grammarForPath — common extensions, case-insensitive matching, special filenames (Makefile / Dockerfile / .gitignore / justfile), no-dot/no-special-match fallthrough, unknown extension. - test/kernel/src/syntax/tree_sitter_service_test.dart (9 tests): graceful-fallback paths of TreeSitterService when the library can't load — hasGrammar / languageFor / highlight all short- circuit cleanly, plus SyntaxResult.empty and SyntaxSpan field storage. Coverage: language_map.dart 0/7 -> 7/7; tree_sitter_service.dart 1/131 -> 22/131 (the rest is deep FFI work); tree_sitter_ffi.dart 0/15 -> 1/15 (init entry only). Total coverage 75.11% -> 75.42%. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 06:41:14 +02:00
jpmschweitzerandClaude Opus 4.7	331201e287	test sweep: cover src/editor/ tail paths (T-91) ... Adds 6 tests in test/editor/registry_test.dart closing out the EditorRegistry methods the existing suite didn't reach: - activate(id): unknown id is a no-op (no event emitted); known id flips the active buffer and emits editor.active-changed. - setContent(id, content, selection?): explicit selection clamped to new content length, missing selection falls back to clamping the existing one, missing id is a silent no-op. Each emits the editor.edited replace event. - contentFromArgs(args): the content_b64 fallback when text is absent, plus the empty-args default. - Selection.hashCode + .toString. Coverage: src/editor/buffer.dart 20/24 -> 24/24; src/editor/registry .dart 87/105 -> 105/105. Both at 100%. Total coverage 74.79% -> 75.11%; floor bumped to 75. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-13 06:37:14 +02:00
jpmschweitzerandClaude Opus 4.7	7ad427bd88	test sweep: cover I18n locale getters (T-91) ... Adds one test covering the currentLocale / defaultLocale / availableLocales getters on the I18n class. Existing suite exercised the lookup behaviour but never read back the three locale fields, leaving them as the last reachable gaps in lib/kernel/src/i18n/. Coverage: kernel/src/i18n/i18n.dart 54/58 -> 57/58. The 1 remaining line is the I18nReplacer const-constructor body (lcov phantom). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-12 11:21:24 +02:00
jpmschweitzerandClaude Opus 4.7	eeb2ee4b82	test sweep: mop up kernel/src/theme residuals (T-91) ... 10 small tests in test/kernel/src/theme/mop_up_test.dart covering the scattered last-mile gaps across the theme subsystem: - ClideTheme.of and ClideTheme.controllerOf inherited-widget lookups (both happy path + the FlutterError-throw branch when no ancestor). - contrast._composite alpha-mix branch via a partially-transparent foreground in contrastRatio. - ContrastFailure.toString format with embedded pair name / ratio / minimum. - ThemeLoader.fromYamlString FormatException on a non-map root. - ThemeLoader.fromFile happy path with a tempdir YAML. - Palette.names and SemanticRoles.roles iterables. Coverage: theme/controller.dart 37/42 -> 42/42; theme/contrast.dart 54/64 -> 62/64 (remaining 2 lines are the failingPairs add-to-list path, needs a constructed-broken SurfaceTokens to fire); theme/loader.dart 48/54 -> 53/54 (1 const-ctor phantom); theme/palette.dart 9/10 -> 10/10; theme/semantic.dart 2/3 -> 3/3. Total coverage 74.55% -> 74.79%. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-12 09:13:43 +02:00
jpmschweitzerandClaude Opus 4.7	c3033fe304	test sweep: cover i18n catalog loaders + theme_picker _pick (T-91) ... Two test files / batches: - test/kernel/src/i18n/catalog_loader_test.dart (11 tests): all three CatalogLoader implementations end-to-end. AssetCatalogLoader against an in-memory AssetBundle covers happy path, FlutterError catch (missing asset), FormatException catch (malformed JSON), blank asset, non-object JSON. FileCatalogLoader against a tempdir covers happy path, missing file, malformed, blank. InMemoryCatalogLoader covers lookup + namespace-miss. - Extends test/builtin/theme_picker/widget_test.dart with two tests for the _pick command callback: defensive not-activated branch (drive the contribution's run() before activate runs, _ctx is still null) and the dialog-resolution happy path (register + activate, invoke theme.pick, manually drive dialog.dismiss('forest'), assert IpcResponse data['selected']). Coverage: kernel/src/i18n/catalog_loader.dart 14/26 -> 26/26; builtin/theme_picker/src/extension.dart 9/18 -> 15/18 (remaining 3 lines are the dialog builder body — needs a DialogHost in the test harness, out of scope). Total coverage 74.08% -> 74.55%; floor bumped to 74. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-12 09:03:29 +02:00
jpmschweitzerandClaude Opus 4.7	5dcd76a9a9	test sweep: cover builtin/ipc_status (T-91) ... Four new widget tests in test/builtin/ipc_status/widget_test.dart covering the toolchain-state branches of ToolStatusItem and the StatusItemContribution build callback: - unresolved toolchain → SizedBox.shrink (no chip text rendered) - all tools resolved → single "application ok" chip - some tools missing → one warning chip per missing tool - StatusItemContribution.build returns a ToolStatusItem widget Coverage: builtin/ipc_status/src/status_item.dart 9/22 -> 22/22; builtin/ipc_status/src/extension.dart 6/7 -> 7/7. Both at 100%. Total 73.88% -> 73.99%. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 22:23:17 +02:00
jpmschweitzerandClaude Opus 4.7	8d98efc900	test sweep: cover src/pql/client.dart (T-91) ... 18 integration tests in test/pql/client_test.dart driving the real pql 1.4.30 binary against the clide vault. Covers the methods the existing test/daemon/pql_commands_test suite didn't reach: backlinks, tags, query, search, decisionValidate, decisionRead, decisionList with filters, decisionShow with --with-refs / --with-tickets, ticketList with status / team / assigned / decision filters, ticketShow with --with-context / --with-blockers, ticketBoard with --team. Plus PqlException.toString and the two error paths in _run (ProcessException from a bad binary path, non-zero exit code from the validator). Coverage: src/pql/client.dart 43/92 -> 89/92 (97%). The 3 remaining lines are an unreachable null-coalesce branch in _runList and PqlException's field initialisers (lcov phantom). Total coverage 73.34% -> 73.88%. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 19:11:52 +02:00
jpmschweitzerandClaude Opus 4.7	889058db1b	test sweep: cover kernel/src/{events,ipc} (T-91) ... Three new test files + a small DaemonClient dispose-safety fix: - test/kernel/src/events/types_test.dart (7 tests): every ClideEvent subclass's subsystem / kind / payload contract + the ClideEventEnvelope v1 JSON shape. - test/kernel/src/events/message_bus_test.dart (6 tests): Message shape, MessageBus publish/subscribe/dispose, filter-by-publisher, filter-by-channel, intersection. - test/kernel/src/ipc/client_test.dart (9 tests): real Unix-socket roundtrip via a _TestDaemon helper — connect + correlate request/ response, event forwarding to the DaemonBus, malformed-line skip, daemon-disconnect failing pending requests, stop cleanup, dispose, connect-failure-then-reconnect, daemon-sent-Request warn-and-skip, DaemonConnectionChanged emission. Fix in lib/kernel/src/ipc/client.dart: _setConnected now skips notifyListeners / event emit when _disposed. The socket stream's onDone can fire after dispose runs, which previously hit ChangeNotifier's "used after disposed" assertion. State flip stays unconditional so stop()'s explicit transition still works. Coverage: ipc/client.dart 14% -> 92% (79/86; remaining 7 lines are the socket onError callback + 1 const ctor phantom); events/types .dart 95% (37/39 — 2 const-ctor phantoms); events/message_bus.dart 100%; events/bus.dart stays 100%. Total coverage 71.93% -> 73.34%; floor bumped to 73. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 19:06:17 +02:00
jpmschweitzerandClaude	b3ed7db75c	.gitignore: drop dead pql un-ignores ... Companion to add028e. The hook-tracking removal commit only staged the .pql/hooks/* file deletions, missing the .gitignore changes themselves. This commit drops: - !.pql/hooks/ (hooks are now per-clone, never tracked) - !.pql/pql-plan.json (legacy snapshot was deleted in the changelog migration; the un-ignore was already pointing at nothing) pql init re-adds !.pql/pql-plan.json on every run even though the file no longer exists post-changelog-migration — minor pql-side finding to flag; harmless until then. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-11 18:52:01 +02:00
jpmschweitzerandClaude	add028eed0	stop tracking .pql/hooks — finish T-28 decommission ... pql 1.4.x bakes the absolute path to the pql binary into each hook when init plants them. Tracking the hooks made every developer's init produce a permanent dirty diff against the committed paths. The right fix per T-28 is to never track them: each clone's 'pql init' plants its own hooks locally, nothing tracked, no drift. Drops: - !.pql/hooks/ from .gitignore (now ignored alongside the rest of .pql/) - !.pql/pql-plan.json from .gitignore (the legacy snapshot was deleted in the changelog migration; this un-ignore was already dead) - The four tracked hook files (git rm --cached); local copies stay on disk and continue working. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-11 18:49:42 +02:00
jpmschweitzerandClaude	12b2f33572	retire Q-21 + Q-22, add D-67 changelog persistence ... Two open questions have been answered by reality and should be formally resolved: - Q-21 (pql planning home): pql 1.4.30 ships the full surface and clide consumes it via shell-out. Resolves to D-3 + D-39, which already encoded the intent. - Q-22 (ticket persistence): pql 1.4.x's append-only per-month SQL changelog with inline LWW guards is option (3) of Q-22 evolved into a form that merges by default. Clide migrated to it on 2026-05-09. Adds D-67 to formalise clide's adoption: which directories carry changelog data, what stays gitignored, which hooks run when. Q-22 points at D-67; Q-21 points at the pre-existing D-3 + D-39. Pql state: 106 decisions → 107 (D-67 added); 134 refs → 139; 0 broken; validate ok. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-11 18:43:40 +02:00
jpmschweitzerandClaude	f90ddc345f	update decisions/ → governance/ refs after D-21 migration ... Stale path references from the move in 63195d1: - CLAUDE.md: 8 D-record links, the layout tree, the open-questions pointer, all rewritten to governance/. - docs/design/multitab-pane.md + docs/claude-design/README.md: cross-references updated. - lib/clide.dart: doc-comment refs. - lib/builtin/problems: user-facing message string. - Makefile: decisions-validate target docstring. Note: lib/builtin/decisions/ (the in-app decisions panel package) keeps its name — it's a feature name, not a filesystem-path mirror. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-11 18:38:39 +02:00
jpmschweitzerandClaude	63195d1ec8	migrate DQR tree to governance/ layout (pql D-21) ... pql 1.4.30 introduces a per-type subdirectory layout for decision records: governance/{decisions,questions,rejected}/<domain>.md. The parser infers record type from the parent subdir and domain from the filename stem, replacing the flat decisions/<file>.md layout with questions-X and rejected prefixes. Moves: - 6 D-files to governance/decisions/ (accessibility, architecture, extensions, process, testing, tooling) - 5 Q-files to governance/questions/ (stripped the questions- prefix) - rejected.md split by domain to governance/rejected/{architecture (5), process (4), accessibility (1), testing (1)}.md — split per cross-reference target. Drops: - decisions/README.md (auto-regenerated as governance/README.md by pql init below the records marker) - decisions/questions.md (hand-curated index pointing at the per- domain files; redundant once pql generates the index, and the "load-bearing" curation was already present in each Q-record's Context field). Convention-driven; not a clide-specific change. Counts unchanged across migration: 106 decisions, 95 tickets, 134 refs, 0 broken. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-11 18:34:20 +02:00
jpmschweitzerandClaude Opus 4.7	ae8e43c298	lower coverage floor to 71 after dead-widget deletion ... The previous commit deleted three dead widgets + their tests under lib/src/terminal/src/ui/ — 585 lines net. The deleted test suite was contributing more lcov hits than the deleted code was contributing DA lines, so total coverage drops from 72.22% to 71.93%. Intentional: deleting tested-but-unused code is the right move; the absolute coverage number isn't. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 18:22:14 +02:00
jpmschweitzerandClaude Opus 4.7	94ff623708	remove dead Scrollable-era widgets + simulateScroll param ... Three widgets under lib/src/terminal/src/ui/ and one TerminalView parameter were leftovers from the era when TerminalView wrapped its viewport in a real Scrollable. The Scrollable path was replaced with PointerScrollEvent → PgUp/PgDown translation (alive, well-tested in terminal_view_test.dart); these helpers stayed behind with tests but zero production callers. Drops: - TerminalScrollGestureHandler (scroll_handler.dart, 100 LOC) - InfiniteScrollView (infinite_scroll_view.dart, 117 LOC) - KeyboardVisibilty (keyboard_visibility.dart, 59 LOC) — last production caller was removed in 048e835 - TerminalView.simulateScroll parameter — declared, never read - The matching test groups + imports in ui_widget_test.dart - The KeyboardVisibilty export from the terminal barrel Net: -585 lines from lib/ + test/, no behavior change, and infinite_scroll_view.dart stops being the 90%-coverage outlier we were apologising for in the previous test sweep. Same shape as T-93 (dead onTapUp wiring), T-95 (dead tertiary tap), and 048e835 (dead scrollController plumbing). Public-or-tested surface that no caller exercised. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 18:20:50 +02:00
jpmschweitzerandClaude Opus 4.7	387a336520	test sweep: cover escape parser + render _onScroll listener (T-91) ... Six tests for the last reachable coverage gaps in the terminal tree: - EscapeParser CSI handlers that read params[0] and normalise 0 → 1: CUF (ESC [ Ps C), CUB (ESC [ Ps D), CPL (ESC [ Ps F). - insertLines with an explicit param (ESC [ Ps L). - DEC private mode 1001 (HighlightMouseTracking) toggles MouseMode .upDownScroll on, none off. - RenderTerminal._onScroll driven via a test ViewportOffset that exposes notifyListeners — verifies the listener body runs without throwing when an offset is swapped in via the setter. Coverage: escape/parser.dart 503/513 -> 510/513 (the 3 remaining lines are _CSI.toString — only reachable from a debug log call that no production path makes); render.dart 245/249 -> 249/249. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 18:12:31 +02:00
jpmschweitzerandClaude Opus 4.7	048e835b31	remove dead Scrollable plumbing from TerminalView ... TerminalView's build tree doesn't wrap content in a Scrollable — scroll is handled by translating PointerScrollEvent into PgUp/PgDown keyInput. The ScrollController parameter, _scrollableKey, internal _scrollController, _scrollToBottom helper, and its five call sites were all dead: _scrollableKey.currentState was always null because no Scrollable in the tree carried the key, so _scrollToBottom's jumpTo never fired. Drops: - public scrollController parameter on TerminalView - _scrollableKey + _scrollController fields - the didUpdateWidget swap block and dispose call - _scrollToBottom + the five call sites - KeyboardVisibilty wrapper (its only callback was _scrollToBottom, now a no-op; the widget remains a reusable primitive under ui/ for future use) - the matching tests in terminal_view_test.dart Same shape as T-93 (dead onTapUp wiring) and T-95 (dead tertiary tap surface) — public API that no caller used + internal state that no path executed. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 18:12:23 +02:00
jpmschweitzerandClaude Opus 4.7	f46ab50ff9	test sweep: cover terminal_view IME / focus tail paths (T-91) ... Three tests for the lines the existing TerminalView suite didn't quite touch: the deleteDetection backspace flow (CustomTextEdit.onDelete → scrollToBottom + Terminal.keyInput), the hardwareKeyboardOnly tap that requests focus directly (_onTapUp's else-branch when there's no CustomTextEdit), and the single-char IME insert path where the character maps to a TerminalKey (_onInsert's key != null branch). Coverage: terminal_view.dart 180/188 -> 187/188 (99.5%). The 1 remaining line is _scrollToBottom's jumpTo call — unreachable in current wiring since the tree has no Scrollable; _scrollableKey .currentState is always null and the guard short-circuits. Note: infinite_scroll_view.dart stays at 90% — the 4 uncovered lines are the position-setter's value-changed branch, only reachable when the inner Scrollable swaps its ViewportOffset. The widget doesn't expose ScrollController or physics, so there's no public surface to drive that path from a test. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 18:04:48 +02:00
jpmschweitzerandClaude Opus 4.7	0a43a1f7d2	remove dead tertiary-tap surface (T-95) ... The middle-click ("tertiary tap") path in TerminalGestureHandler was wired wrong: build() bound onTertiaryTapDown to the secondary state method, so a middle-click fired as if it were a right-click. The state's onTertiaryTapDown/Up methods were unreachable, and the onTertiaryTapUp body had a copy-paste bug (button=right instead of middle). No production caller passed onTertiaryTapDown / onTertiaryTapUp through, and TerminalView didn't expose them either, so the public parameters were dead too. Drops both layers of dead surface — option B of T-95. Same shape as T-93's resolution (delete unused, restore later when a real consumer needs it). Also collapses the unreachable onDragStart selectWord branch (PanGestureRecognizer is mouse-only, so the touch path can't fire) into a single selectCharacters call with a comment. Companion: refines the reflow-padding test in coverage_trivials_test to use narrow→wide reflow setup (more honest about intent, also actually exercises the padding branch — Buffer.resize now 100%) and clears two unnecessary_import warnings surfaced by the deletion. Coverage: gesture_handler 55/59 -> 59/59; gesture_detector 50/50; buffer/buffer 260/261 -> 261/261. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 17:56:24 +02:00
jpmschweitzerandClaude Opus 4.7	155af91e66	test sweep: close small terminal coverage gaps (T-91) ... Four targeted tests covering the last single-line/short-tail gaps in the terminal tree: the PointerInputs.none / .all const constructors, the abstract TerminalMouseHandler const constructor (reached via a private subclass), the reflow-output-padding branch in Buffer.resize (line < newHeight), and the wide-char skip in TerminalPainter.paintLine. Coverage: pointer_input 1/3 -> 3/3; mouse/handler 33/34 -> 34/34; buffer/buffer 260/261 -> 261/261; painter 120/121 -> 121/121. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 17:03:47 +02:00
jpmschweitzerandClaude Opus 4.7	eb32422e05	drop dead pql.decisions.coverage IPC command ... The 'pql decisions coverage' subcommand was removed in pql 1.4.x — the replacement is the 'coverage_gaps' field on 'pql plan status'. No production callers used this IPC surface; only the unit test referenced it. Removes the IPC registration, the PqlClient helper, and the matching test case. Net negative LOC. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 17:03:40 +02:00
jpmschweitzerandClaude Opus 4.7	d2e65d8821	test sweep: cover ui/render.dart (T-91) ... 10 widget tests driving RenderTerminal through a hosted TerminalView: the reactive setters via updateRenderObject (theme, textStyle, textScaler, padding, autoResize, cursorType, alwaysShowCursor) and direct setter calls on the render box (padding, onEditableRect, composingText) to cover the value-changed branches; getOffset for non-origin cells; systemFontsDidChange; the terminal listener via write(); the viewport-offset listener via scrollback overflow; and the paint paths for composingText and controller-driven highlights. Coverage: render.dart 179/249 -> 245/249 (98%). The 4 still-uncovered lines are the _onScroll body — reachable only when the inner ViewportOffset notifies, which doesn't happen with the current ViewportOffset.zero() wiring (scroll lives in PointerScrollEvent → PgUp/PgDown). Total 71.23% -> 72.02%; floor bumped to 72. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-11 09:15:29 +02:00
jpmschweitzerandClaude Opus 4.7	1bb5d8d534	test sweep: cover base/disposable.dart (T-91) ... Three pure-Dart tests covering the Disposable mixin: the disposed getter flipping after dispose(), onDisposed firing once, and register propagating dispose to child disposables. Coverage: disposable.dart 12/17 -> 17/17. Total 71.20% -> 71.23%. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-10 20:23:00 +02:00
jpmschweitzerandClaude Opus 4.7	8ae5e7a9e8	test sweep: cover ui/custom_text_edit.dart (T-91) ... Adds 17 widget tests covering CustomTextEdit's focus / input-connection lifecycle (autofocus, focus loss, readOnly toggling both directions, focusNode swap), the keyboard helpers (requestKeyboard / closeKeyboard both with and without an active connection, setEditingState, setEditableRect early-return + active path), and the TextInputClient surface (updateEditingValue insert / delete / composing branches, performAction, plus the no-op stubs — updateFloatingCursor, showAutocorrectionPromptRect, connectionClosed, performPrivateCommand, insertTextPlaceholder, removeTextPlaceholder, showToolbar — and the two getters). Coverage: custom_text_edit.dart 66/96 -> 96/96. Total 70.86% -> 71.20%; floor bumped to 71. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-10 17:29:13 +02:00
jpmschweitzerandClaude Opus 4.7	ca46bf4518	test sweep: cover ui/painter.dart (T-91) ... Adds 22 unit tests in test/terminal/painter_test.dart covering the constructor, the three reactive setters (textStyle, textScaler, theme — both same-value early-return and different-value paths), clearFontCache, paintCursor for all three cursor types plus the no-focus stroked-rect branch, paintHighlight, paintLine end-to-end, paintCellForeground (codepoint-0 short-circuit, faint, inverse, underline-on-space, bold+italic, cache hit), paintCellBackground (normal early-return, inverse, named/palette, double-width), and the foreground / background colour resolvers across normal / named / palette / rgb colour types. Coverage: painter.dart 72/120 -> 120/121. Total 70.30% -> 70.86%. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-10 09:08:57 +02:00
jpmschweitzerandClaude Opus 4.7	3ea49421bb	test sweep: cover shortcut/ + base/event.dart (T-91) ... Adds 9 tests in test/terminal/shortcut_event_test.dart covering Event / EventEmitter / EventSubscription, the platform branch in defaultTerminalShortcuts, and the three TerminalActions intent handlers (copy / paste / select-all, plus the empty-selection no-op path). Coverage: base/event.dart 2/13 -> 13/13; shortcut/actions.dart 8/24 -> 24/24; shortcut/shortcuts.dart 8/18 -> 18/18. Total 69.87% -> 70.30%; floor bumped to 70. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-10 01:40:07 +02:00
jpmschweitzerandClaude Opus 4.7	d162ba205e	install pql 1.4.26 changelog-mode hooks ... Replaces the legacy pql-plan.json export hooks with the new changelog-aware set: pre-commit now stages incremental changelog deltas via 'pql plan export --stage'; post-merge replays new changelog files into pql.db; post-checkout / post-rewrite rebuild pql.db when branch state changes. Companion shims under .githooks/ keep core.hooksPath = .githooks the single activation point. Hook bodies are de-baked — pql 1.4.26 ships them with absolute paths to the local pql binary, which doesn't survive cross-machine tracking. Restored 'pql' on PATH form so the tracked copy stays portable. Filed back-channel for pql to keep portable form. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-10 01:31:06 +02:00
jpmschweitzer	01a99edd52	migrate to changelog replication	2026-05-10 01:25:32 +02:00
jpmschweitzerandClaude Opus 4.7	2216092ef4	test sweep: cover gesture_handler / gesture_detector selection paths (T-91) ... Adds three widget tests under TerminalView — selection gestures: double-tap selects a word; long-press (touch) selects a word and keeps the selection across move-update; mouse drag selects characters across drag-start and drag-update. Each verifies the side-effect on the externally-supplied TerminalController. Coverage: gesture_detector.dart 42/50 -> 50/50; gesture_handler.dart 39/59 -> 55/59. The 4 remaining uncovered lines (147, 148, 151, 152) are the dead tertiary-tap state methods filed as T-95 — same shape as T-93 but on middle-click. Floor stays at 69 (69.79% measured; integer threshold unchanged). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-08 18:16:07 +02:00
jpmschweitzerandClaude	e966836886	test sweep: cover ui/ widget-level helpers (T-91) ... Adds test/terminal/ui/ui_widget_test.dart — 14 widget tests covering the lib/src/terminal/src/ui/ helpers that need a Flutter widget tree (CustomKeyboardListener, KeyboardVisibilty, InfiniteScrollView, TerminalScrollGestureHandler). Files reaching ~100% (the 4 missing lines in infinite_scroll_view sit in the render-object's `position` setter, only triggered when Scrollable feeds a different ViewportOffset between rebuilds — not reachable through normal widget plumbing without internal access): - scroll_handler.dart: 0 / unmeasured → 39/39 (100%) — main-buffer passthrough vs. alt-buffer interception, mouse-mode forwarding vs. simulateScroll fallback, simulateScroll=false drop, alt-flip + didUpdateWidget rebinding, onPointerDown tracking the cursor for the next scroll event. - infinite_scroll_view.dart: 0 / unmeasured → 36/40 (90%) — onScroll fires on viewport position change, callback identity update via updateRenderObject. - keyboard_listener.dart: 7/12 → 12/12 (100%) — character-key fallthrough into onInsert when onKeyEvent returns ignored, short-circuit to onKeyEvent's "handled" return, no-op on a key with no character. - keyboard_visibility.dart: 18/19 → 19/19 (100%) — show + hide callbacks paired against view-insets transitions; no fire on same-inset metrics events. Coverage delta: - Total project: 68.80% → 69.12%; coverage_floor bumped 68 → 69. Tests use a `_host()` helper that wraps the widget under test in Directionality + MediaQuery + Center + a sized SizedBox. The TerminalScrollGestureHandler tests use ColoredBox as the child because Listener.onPointerSignal needs a hit-testable render object below it, and SizedBox.expand alone doesn't paint anything. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-08 12:40:03 +02:00
jpmschweitzerandClaude	ecf8ee56af	test sweep: cover ui/ pure-Dart files (T-91) ... Adds test/terminal/ui/ui_pure_test.dart — 39 unit tests covering the lib/src/terminal/src/ui/ files that don't need a Flutter widget harness. Files at 100%: - palette_builder.dart: 0/52 → 52/52 (named-colour mapping for indices 0..15, RGB cube walk for 16..231 with the 95-then-+40 step pattern, grayscale ramp 232..255, out-of-range clamp). - paragraph_cache.dart: 2/22 → 22/22 (LRU promotion on read, eviction at maxSize, miss returns null, clear empties). - terminal_text_style.dart: 7/19 → 19/19 (default ctor, fromTextStyle fontFamily fallback chain + size/height pass- through, toTextStyle attribute threading, copyWith with override + no-args clone). - input_map.dart: 8/20 → 20/20 (keyToTerminalKey hits + null on unmapped LogicalKeyboardKey id, charToTerminalKey case-fold + multi-char/unknown null). - terminal_size.dart: 4/8 → 8/8 (operator==/hashCode/toString, identity + type-mismatch branches). - pointer_input.dart: 1/3 → 3/3 (default ctor, .none(), .all()). - terminal_theme.dart: 1/1 unchanged. - char_metrics.dart: 0/10 → 10/10 (positive width/height, monotonic text-scaler scaling). - ui/controller.dart: 18/52 → 52/52 (TerminalController default state, setSelection + mode override, attached/detached anchor selection-getter behaviour, prior-anchor disposal on second setSelection, setSelectionMode no-op short-circuit, clearSelection, setPointerInputs / setSuspendPointerInput gating, highlight register/dispose lifecycle, TerminalHighlight.range null on detached anchors). Enums (selection_mode, cursor_type) and themes constants are covered transitively — they have no executable lines that lcov instruments. Coverage delta: - Total project: 67.06% → 68.80%; coverage_floor bumped 66 → 68. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-08 11:28:17 +02:00
jpmschweitzerandClaude	2432207e5b	fix stale coverage/floor.txt reference in gate script ... The "consider bumping" hint pointed to `coverage/floor.txt`, but the floor moved to `pubspec.yaml`'s `coverage_floor:` key when the gate was first folded together. Updates the message to match the actual source. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-07 21:05:16 +02:00
Jeroen SchweitzerandClaude Opus 4.6	a6eca2561b	remove dissolved daemon, retire ptyc, fix golden cross-platform ... Complete three overdue cleanups discovered during macOS health check: D-56 daemon dissolution: delete bin/clide.dart, DaemonServer, and orphaned tests (test/cli/, subprocess_test, in_process_test). Update stale "clide --daemon" references in i18n catalogs, error messages, editor_commands, CI scripts, and decision records. ptyc retirement: delete ptyc/ source tree, PtySession, scm_rights. Remove from Toolchain resolution, ToolCheck gate, backend serialization, testmode harness, Makefile, CI, and sandbox entitlements. PTY spawning uses NativePty (Dart FFI forkpty) since the terminal was absorbed in-tree. D-5 amended. Golden tests: wire the existing but never-applied clideGoldenConfig via flutter_test_config.dart. Disable CI goldens (Skia anti-aliasing differs between macOS/Linux even with Ahem). Keep platform-keyed goldens only — goldens/linux/ and goldens/macos/ each run on their own OS. Test suite: 826 pass, 0 fail on macOS (was 829 pass, 11 fail). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>	2026-05-07 18:40:01 +02:00
jpmschweitzerandClaude	6b7290dc42	fix dead-wired TerminalView.onTapUp callback (T-93) ... `TerminalView.onTapUp` was documented as "Callback for when the user taps on the terminal" but was wired to a code path nothing ever invoked: - `TerminalView.build` passed it via `onTapUp:` to `TerminalGestureHandler`, - which forwarded via `onTapUp:` to `TerminalGestureDetector`, - whose `onTapUp` field was declared and accepted but never called by `_handleTapUp` (which only fires `onSingleTapUp`). Net: every caller that registered an `onTapUp` callback on `TerminalView` got silent failure. zero in-tree callers depended on it (clide_pty_view.dart is the only TerminalView callsite and doesn't pass any tap callback), but the public API said one thing and did another. Fix: wire `_onTapUp` (the cell-resolving state-method) through the detector's working `onSingleTapUp` slot. The user-facing semantics ("fires on confirmed single tap with the resolved cell offset") match the only sane interpretation of the docstring, and don't overlap with the existing `onSecondaryTap*` (which were already correctly wired through TapGestureRecognizer's secondary callbacks). Also drops the dead surface that surfaced the bug: - `TerminalGestureHandler.onTapUp` parameter + field — no caller passes it after the fix; was only used to forward into the dead detector field. - `TerminalGestureDetector.onTapUp` parameter + field — never invoked by `_handleTapUp`. Pure dead code. Tests: extends `terminal_view_test.dart` with a primary-tap regression case + paired tests for selection-clearing and secondary-tap callback routing. The double-tap recognizer's 300 ms timer is flushed via `pump(const Duration(seconds: 1))` (pumpAndSettle waits for animations, not arbitrary timers). Coverage delta: - terminal_view.dart: 151/188 → 180/188 (95.7%; remaining gaps are IME `_onComposing`/`_onEditableRect`/`_onKeyboardShow` body branches that need deeper IME mocking). - gesture_handler.dart: 18/60 → 39/59. - gesture_detector.dart: 30/50 → 42/50. - Total project: 65.76% → 66.97%; coverage_floor 65 → 66. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-07 08:49:09 +02:00
jpmschweitzerandClaude	fb85dfa6f9	test sweep: cover TerminalView widget (T-91) ... Adds test/terminal/terminal_view_test.dart — 14 widget tests covering the StatefulWidget that wires gesture / keyboard / scroll plumbing around a `Terminal`: - Construction smoke tests (default, externally-supplied controller / focusNode / scrollController, hardwareKeyboardOnly, readOnly + hardwareKeyboardOnly). - Pointer-signal handling: PointerScrollEvent → PgUp/PgDown keyInput; non-scroll PointerSignalEvent ignored. - didUpdateWidget swap of focusNode / controller / scrollController (auto-created previous instance gets disposed cleanly). - Hardware key event flowing through to Terminal.keyInput. - cursorRect / globalCursorRect after layout. - requestKeyboard / closeKeyboard as no-ops when no edit state is mounted; hasInputConnection false when no input connection is open. - Selection survival when the widget unmounts but an externally-owned controller stays alive. Coverage delta: - terminal_view.dart: 0/188 → 151/188 (80.3%). - Total project: 59.82% → 65.76%; coverage_floor bumped 59 → 65. The remaining ~20% in terminal_view.dart sits in gesture / IME / keyboard-event plumbing (`_onTapUp`, `_onTapDown`, secondary-tap callbacks, `_onInsert`, `_onComposing`, `_handleKeyEvent` shortcut path, `_onKeyboardShow`, `_onEditableRect`, `_scrollToBottom`). These are reachable only through full pointer / IME simulation that's better suited to `integration_test/` than widget tests — leaving them for a later integration-test pass rather than papering over with brittle gesture mocking. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-07 08:25:44 +02:00