Skip to content

feat(mcp): surface highlight range and FIFO comment queue - #115

Merged
scott merged 1 commit into
fix/112-never-delete-from-commentfrom
feat/113-highlight-range-fifo
Sep 12, 2026
Merged

scott merged 1 commit into
fix/112-never-delete-from-commentfrom
feat/113-highlight-range-fifo

Conversation

@scott

@scott scott commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Unmarshal quote_index, quote_start/quote_end, md_start/md_end, queue_position, and queue_length on threadView. Missing keys from an old server stay zero.
  • Nest target.md_start, target.md_end, target.text, and target.path on path-scoped inbox threads.
  • MCP instructions: path-scoped inbox is FIFO; address only queue_position 1 this cycle; re-list before the next thread; do not apply two highlight edits in one pass.
  • gander comments prints queue index and highlight, e.g. [1/2] "rollout_v2" md:812-822.

Related Issue

Closes #113

Testing

go test ./...
go vet ./...
go build ./...

All passed.

Notes

Stacked on #114 (issue #112 no-delete guardrails). Retarget main after that merges.

Populated md_* depends on gandermd/gandermd#122 on :latest. An old server still works: offsets stay 0 and the CLI falls back to [index/count].

@scott
scott merged commit bc1246f into fix/112-never-delete-from-comment Sep 12, 2026
@scott
scott deleted the feat/113-highlight-range-fifo branch September 12, 2026 20:00
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