Skip to content

fix: make Ctrl-click link capture opt-in (#69) - #71

Merged
backnotprop merged 4 commits into
mainfrom
fix/69-link-capture-opt-in
Oct 7, 2026
Merged

backnotprop merged 4 commits into
mainfrom
fix/69-link-capture-opt-in

Conversation

@backnotprop

@backnotprop backnotprop commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Ctrl-click on a file:// Markdown link is no longer captured by Annotate. Herdr matches [[link_handlers]] from the manifest before plugin code runs, so the handler moves out of the Full and Windows Full manifests into a separate opt-in add-on:

herdr plugin install plannotator/herdr-annotate/links   # opt in
herdr plugin uninstall annotate-links                   # opt out

The add-on (annotate-links) ships no binary. It looks up the annotate plugin root with herdr plugin list --plugin annotate --json and runs that plugin's plannotator-tui herdr open with HERDR_PLUGIN_ID=annotate, as the old open-link action did. macOS/Linux use open-link.sh; Windows (with Windows Full) uses open-link-windows -> open-link.ps1. If Annotate is missing, disabled, or lacks plannotator-tui, it shows a notification saying what to install.

Plugin 0.9.0. README notes the change for existing users. Tests: manifest check, Windows PS checks (the windows-full pane test and path matrix now run the add-on's Windows action end to end), smoke.

refs #69

….9.0

The Full and Windows Full manifests no longer declare the markdown-file link handler or the open-link action, so Ctrl-click on file:// Markdown links keeps Herdr's default behavior. The new links/ add-on (annotate-links) declares the handler and runs the installed annotate plugin's plannotator-tui, without a binary of its own.

refs #69
@backnotprop
backnotprop force-pushed the fix/69-link-capture-opt-in branch from 4df6a6a to 7d40597 Compare October 7, 2026 16:30
The add-on gains an open-link-windows action and markdown-file-windows handler that run links/open-link.ps1, which resolves the annotate plugin root and runs its staged plannotator-tui. The windows-full pane test now drives that action end to end.

refs #69
@backnotprop
backnotprop merged commit e3ca7e8 into main Oct 7, 2026
9 of 10 checks passed
@backnotprop
backnotprop deleted the fix/69-link-capture-opt-in branch October 7, 2026 16:40
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