Skip to content

hy/lsp gtd in command block - #84

Merged
hon-gyu merged 3 commits into
mainfrom
hy/lsp-gtd-in-command-block
Jul 30, 2026
Merged

hy/lsp gtd in command block#84
hon-gyu merged 3 commits into
mainfrom
hy/lsp-gtd-in-command-block

Conversation

@hon-gyu

@hon-gyu hon-gyu commented Jul 30, 2026

Copy link
Copy Markdown
Owner
  • lsp: go-to-definition on command-block lines
  • lsp/config: a way to turn the server off
  • bump oymarkit

hon-gyu added 3 commits July 30, 2026 21:13
A line in an oysterlsp block now answers the editor's navigation key with
the note its command would open, resolved exactly as its lens resolves it.
A calendar command whose note is missing has no definition: a jump may not
create a file, so creation stays on the lens and the code action.
`"disable": true`, in oysterlsp.json or initializationOptions, makes the
server adopt no vault: nothing is indexed and every request is answered the
way it is before a root is known. It is for the directory of Markdown that
is a vault only by accident, and living in the file means the directory
carries that verdict rather than each contributor's editor settings.

The switch is one decision rather than a flag threaded through the
features — leaving the vault unbuilt is the whole of it. Two handlers did
not honour that: the daily-note code actions need only a clock, and
execute_command resolved a path without a root, so both now refuse the
rootless case, which also fixes them before initialize.

A disabled server says so once, at Info rather than Warning: it answers
nothing, and silence is indistinguishable from a broken server. The
warnings its other settings would have produced are suppressed instead.

Capabilities are still advertised — requests are answered emptily, not
refused — which the spec records as a possible refinement.

close #83
@hon-gyu
hon-gyu merged commit 837ae03 into main Jul 30, 2026
0 of 2 checks passed
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