Skip to content

fix: preserve repeated embedded marker prefixes - #188

Merged
vycdev merged 2 commits into
vycdev:developfrom
vycdev2:fix/repeated-embedded-marker-prefix
Aug 10, 2026
Merged

fix: preserve repeated embedded marker prefixes#188
vycdev merged 2 commits into
vycdev:developfrom
vycdev2:fix/repeated-embedded-marker-prefix

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 9, 2026

Copy link
Copy Markdown

Summary

  • Parse embedded todo match metadata in one shared helper used by both live-document and ag-provider paths.
  • Use each regex match position for the source-code prefix, so repeated identical markers on one line no longer point at the first marker.
  • Add regression coverage and document the user-visible fix.

Verification

  • npm test — passed: 135 tests.
  • npm run compile — passed: webpack development build completed.
  • npm run format:check — passed: all matched files use Prettier style.
  • git diff --check upstream/develop...HEAD — passed.

Risk

  • Low: the parsing fields are unchanged except for the corrected prefix position; both provider paths now share the same tested mapping.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

@vycdev
vycdev merged commit f6a8f7f into vycdev:develop Aug 10, 2026
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.

2 participants