Skip to content

Restructure parser/decorator, Vitest, CI, deps, and link/E2E hardening#94

Merged
SeardnaSchmid merged 9 commits into
SeardnaSchmid:mainfrom
bircni:feature/restructuration
May 11, 2026
Merged

Restructure parser/decorator, Vitest, CI, deps, and link/E2E hardening#94
SeardnaSchmid merged 9 commits into
SeardnaSchmid:mainfrom
bircni:feature/restructuration

Conversation

@bircni
Copy link
Copy Markdown
Contributor

@bircni bircni commented May 2, 2026

This branch modernizes the extension codebase (parser/decorator split, Vitest, CI, changelog tooling), bumps dependencies, and adds follow-up fixes/tests for link document ranges and E2E release coverage.

Commit Description
07c66b5 refactor: split parser and decorator, streamline extension wiring
344e22e test: migrate from Jest to Vitest
a439cd8 ci: reorganize CI workflow
977726b docs: restructure Changelog generation
e1ce613 chore: update dependencies
7bcf32e fix(link-provider): expand zero-width document link ranges for empty image alt
4295a8a test(link-provider): assert non-collapsed range for empty-alt images
5c2d78b test(e2e): broaden release smoke for hovers, links, anchors, and parse

User-visible / behavioral

  • Link provider: ![](https://…) (empty alt) no longer produces an invalid zero-width DocumentLink range (VS Code “Illegal argument: range”). Ranges are expanded by one UTF-16 offset when needed; emptiness is detected without relying on Range.isEmpty so mocks and runtime match.
  • E2E: Additional coverage for image hover, relative ./ file links, mailto: / https image links, ## anchor navigation, unknown-anchor no-op, kitchen-sink markdown smoke, ordered lists, and inline code parse checks.

Replaces #78 #85 #84 #81 #91

@bircni bircni force-pushed the feature/restructuration branch 2 times, most recently from 5c2d78b to b40160d Compare May 2, 2026 12:18
@bircni bircni marked this pull request as ready for review May 2, 2026 13:55
@bircni bircni force-pushed the feature/restructuration branch from b40160d to 6ae9390 Compare May 8, 2026 16:49
@bircni
Copy link
Copy Markdown
Contributor Author

bircni commented May 8, 2026

@SeardnaSchmid ready with the latest updates

Copy link
Copy Markdown
Contributor

@IrishBruse IrishBruse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good seems to have fixed some corruption when the underlying file was modified externally nice

@bircni
Copy link
Copy Markdown
Contributor Author

bircni commented May 11, 2026

rebasing

@bircni bircni force-pushed the feature/restructuration branch from 6ae9390 to a35fb08 Compare May 11, 2026 19:51
@bircni bircni marked this pull request as draft May 11, 2026 19:52
@bircni
Copy link
Copy Markdown
Contributor Author

bircni commented May 11, 2026

need 2 minutes

@bircni bircni force-pushed the feature/restructuration branch from a35fb08 to d024222 Compare May 11, 2026 19:54
bircni added 9 commits May 11, 2026 21:55
Extract parser and decorator into focused modules, add registration and
link-interaction helpers, improve Mermaid coordination and observability,
and tighten ESLint configuration.

Made-with: Cursor
Replace Jest with Vitest, update test files and mocks, and refresh docs
and tooling for the new runner.

Made-with: Cursor
…image alt

VS Code rejects DocumentLink when start equals end (e.g. ![](url)). Detect empty ranges without Range.isEmpty so mocks and runtime agree.
Regression for ![](url) document links after zero-width range expansion.
- Kitchen-sink markdown document smoke test

- Image hover; relative ./ links; mailto and https image document links

- navigateToAnchor for ## headings and unknown anchors

- Parse assertions for ordered lists and inline code
@bircni bircni force-pushed the feature/restructuration branch from d024222 to e935004 Compare May 11, 2026 19:55
@bircni bircni marked this pull request as ready for review May 11, 2026 19:55
@bircni
Copy link
Copy Markdown
Contributor Author

bircni commented May 11, 2026

Ready

@SeardnaSchmid SeardnaSchmid merged commit c89d6dc into SeardnaSchmid:main May 11, 2026
9 checks passed
@bircni bircni deleted the feature/restructuration branch May 11, 2026 20:04
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.

3 participants