feat(cli): polish command surface (remote group, bare note, note-search dedup, examples) - #78
Open
olddognewflex wants to merge 1 commit into
Open
feat(cli): polish command surface (remote group, bare note, note-search dedup, examples)#78olddognewflex wants to merge 1 commit into
olddognewflex wants to merge 1 commit into
Conversation
CLI edge inconsistencies (closes #61): - Group the cloud-queue verbs under `qi remote status|drain`, matching every other command family. The hyphenated `qi remote-status` / `qi remote-drain` survive as deprecated aliases (same builders, no duplicated logic) that still run but point at the grouped form. - Bare `qi note` now shows help instead of silently creating an untitled inbox note and opening Obsidian — an undocumented, surprising side effect. Use `qi note new` to create. - Deprecate `qi note search` in favor of `qi search <q> --kind note` (the canonical unified-search path); it still runs. - Add cobra Example blocks for the non-obvious date/repeat syntax (task add/schedule/list) plus search/capture/note new.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #61.
CLI surface inconsistencies at the edges (§2.2, §2.6).
1.
qi remote status|drainThe cloud-queue verbs were hyphenated top-levels (
remote-status/remote-drain) — inconsistent with every other command group. Grouped under aqi remoteparent. The old spellings survive as deprecated aliases (built by the samenewRemote*Command(cfg, use)builders, so no logic is duplicated) that still run but print a one-line pointer to the grouped form.2. Bare
qi noteshows helpIt used to silently create an untitled inbox note and open Obsidian — an undocumented, side-effecting surprise where every sibling group shows help. Now it shows help;
qi note newis the create path. (Removed the now-deadopenURLcall fromnote;openURLitself stays —daily.gouses it.)3.
note search→ deprecatedqi note search <q>duplicatedqi search <q> --kind note. Kept working but markedDeprecated, pointing at the canonical unified-search path.4. Cobra
ExampleblocksAdded where the syntax is non-obvious — especially date/repeat on
task add/schedule/list, plussearch/capture/note new. e.g.:Tests
remote_test.go: theremotegroup hasstatus/drain; the deprecated aliases carry a pointer; bareqi noteis non-runnable (help) andnote searchis deprecated. Fullgo test ./...,go vet,go build ./...green; smoke-tested each surface change.Merge-order note
Touches
internal/commands/task.go(Examples) andcapture.go. Overlaps #60 and #62 on those files — trivial rebase for whichever merges later.