5 Commits

Include renames

Author	SHA1	Message	Date
jpmschweitzerandClaude Opus 4.8	2150a85b2f	fix(cli): make clide draw reachable as an umbrella command (T-318) ... draw is single-token (no verb), so `clide draw --file x` mis-parsed as cmd "draw.--file" — the dispatcher registered draw but the argv parser never learned it was umbrella (unlike icon/image which use subsystem.verb). Found driving the live install; the handler unit tests bypassed the argv path so they missed it. Added draw to _umbrellaCommands + a parse regression test. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-30 00:28:28 +02:00
jpmschweitzerandClaude Opus 4.8	7171a09fc2	gate clide:// deep links: paranoid allowlist + confirmation prompt (T-56, D-90) ... A clide:// link is an untrusted external vector (any webpage can fire one), so it no longer translates to a command in parseArgv. It routes the raw URL to a new builtin.deeplink handler that is doubly defensive: a default-deny allowlist (kDeepLinkSafeActions — only the read-only 'open' verb; run/git/write/passthrough rejected) AND a mandatory 'an external link wants to: … allow?' confirmation before anything runs. Records the security boundary as D-90. The earlier silent editor.open passthrough is replaced; open still works, now behind the prompt. Tests cover the allowlist (the boundary) + the gating. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-09 22:57:43 +02:00
jpmschweitzerandClaude Opus 4.8	4057286632	clide:// deep link handler — open files at a line (T-56) ... clide://open?path=&line= translates (in parseArgv) to editor.open, so an OS scheme invocation routes through the existing CLI→IPC path into the running window — single-instance for free, no new code path. Registered the scheme: linux/clide.desktop MimeType x-scheme-handler/clide (Exec already passes %U) and macOS Info.plist CFBundleURLTypes. Parser validates the action/path/line. Linux works end to end (the OS passes the URL as argv). macOS URL DELIVERY (the AppDelegate openURLs callback → forward into this path) is a follow-up — the scheme is registered, but the native hook needs a real macOS machine to verify, so it's not shipped blind. Also drops a stray import in clide_markdown_test. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-09 22:41:00 +02:00
jpmschweitzerandClaude Opus 4.8	d6dccd841b	add cursor-based pull events: clide events --since <cursor> (T-223) ... The one-shot request/response complement to the never-returning `tail --events` stream — what an agent poll loop wants. The IPC server now keeps a single global, arrival-ordered event log keyed by a monotonic cursor (alongside the per-subsystem replay ring), bounded by eventLogDepth with drop-oldest back-pressure (D-85: producer never blocks). `clide events [--since <cursor>] [--filter X]` returns events after the cursor, a high-water `cursor` to poll from next, and `gap: true` (+ oldestCursor) when the requested cursor predates the retained window so a caller detects loss instead of silently missing dropped events. Repeated polls neither drop nor duplicate. No on-disk persistence. `events` is handled in the IPC server like `tail` (not the dispatcher); added to the argv umbrella set. bindingWhen/CommandContribution untouched. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-06 21:24:34 +02:00
jpmschweitzerandClaude	1147bfac0e	T-125: argv→IpcRequest translator (CLI grammar in Dart) ... Second slice of T-99. Pure Dart function that takes the argv tail of a `clide ...` invocation and returns either an IpcRequest ready to dispatch or an ArgvError carrying a pre-built userError response. The grammar — `SUBSYSTEM VERB [pos...] [--flag value] [--flag=val] [-- passthrough...]` plus the umbrella commands `status`, `tail`, `version`, `ping` — sits here so the C client (T-126) is a dumb pipe: it sends argv as JSON and the server runs the translator before dispatch. Wire envelope: cmd is `subsystem.verb` (or just `subsystem` for umbrella commands). Args is a generic envelope — `positional: [...]`, `flags: {...}`, `passthrough: [...]` — none required, all omitted when empty so the dispatch surface stays minimal. Per-command typed schemas land later as each CLI verb gets wired end-to-end. Co-Authored-By: Claude <noreply@anthropic.com>	2026-05-18 17:51:17 +02:00