Skip to content

fix: ignore empty relative file links - #247

Merged
vycdev merged 2 commits into
vycdev:developfrom
vycdev2:fix/ignore-empty-relative-file-links
Oct 14, 2026
Merged

vycdev merged 2 commits into
vycdev:developfrom
vycdev2:fix/ignore-empty-relative-file-links

Conversation

@vycdev2

@vycdev2 vycdev2 commented Oct 1, 2026

Copy link
Copy Markdown

Summary

  • Ignore relative file links that resolve only to . or .. after trimming punctuation and decoding the path. These previously linked to the Todo document's directory instead of a named file.
  • Add regression cases for wrapped, dot-segment, and URL-encoded empty links; update the changelog.

Verification

  • RED on unchanged implementation: npm test failed the new wrapped-empty-link assertion (414 passing, 1 failing).
  • npm test passed (415 passing).
  • npm run compile passed.
  • npm run format:check passed.
  • npm run typecheck passed.
  • git diff --check passed.

Risk

  • Low: only suppresses relative links ending in dot path components; named file and directory links retain their existing behavior.

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

@vycdev
vycdev merged commit e42fc5a into vycdev:develop Oct 14, 2026
4 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.

2 participants