[cold-review] feat(cli): Cursor-only display_links MCP (#1516) - #130
Closed
heavygee wants to merge 3 commits into
Closed
[cold-review] feat(cli): Cursor-only display_links MCP (#1516)#130heavygee wants to merge 3 commits into
heavygee wants to merge 3 commits into
Conversation
Cursor-routed agents drop doubled letters when recalling URLs; store href bytes built outside the model (MCP display_links + CLI fallback) and render a tappable SessionChat card without a fake user turn. Fixes tiann#1516 Co-authored-by: Cursor <cursoragent@cursor.com>
Doubled-letter URL recall is a Cursor-routed failure mode. Keep the paint path and CLI fallback, but do not advertise display_links to Claude/Codex/OpenCode/Grok tool lists or system prompts. Co-authored-by: Cursor <cursoragent@cursor.com>
Estate hosts bind Cursor state under /var/lib/hapi/cursor via a home symlink; refusing the symlink directory left overlay install dead and pushed agents toward stale project mcp.json sidecars. Resolve to the real directory (and honor HAPI_CURSOR_MCP_CONFIG_DIR); keep refusing symlinked mcp.json files. Co-authored-by: Cursor <cursoragent@cursor.com>
Owner
Author
|
Cold-review staging only. Claude Sonnet review: Ready to push yes (1 cosmetic nit). Promoted to tiann#1564. Closing this fork PR (not for merge). |
Owner
Author
|
Superseded by tiann#1564. Fork-stage only. |
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
1 participant
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.
Fork-side cold-review stage. Not for merge.
Upstream target:
tiann/hapiissue 1516 (display_links).Spin for reviewers: this is a Cursor-ACP-only MCP tool we did not want. Cursor drops doubled letters in visible assistant text; remote operators tap 404s. Claude/Codex/OpenCode do not get the tool. Please comment on the upstream PR if you run Cursor ACP and have never seen the drop.
See body that will go to
tiann/hapiin the worktree filelocaldocs/pr-1516-upstream.md(gitignored) / the upstream PR once fork review is clean.