Skip to content

fix: harden comment-aware hover and folding#74

Open
tzzs wants to merge 2 commits into
masterfrom
codex/hover-method-doc-comments
Open

fix: harden comment-aware hover and folding#74
tzzs wants to merge 2 commits into
masterfrom
codex/hover-method-doc-comments

Conversation

@tzzs
Copy link
Copy Markdown
Owner

@tzzs tzzs commented Jun 1, 2026

Summary

  • normalize method hover doc comments for triple-slash comments so rendered text does not keep an extra leading slash
  • support hash comments as hover documentation and avoid treating inline block comments on previous code lines as docs for the next method
  • make folding range bracket and parenthesis scans reuse the core tokenizer comment masking so line comments, hash comments, and block comments do not create fake list/map/parameter folds
  • export ThriftTokenizer from thrift-core so VS Code providers can share the same comment and token handling instead of hand-rolling it

Verification

  • npm run lint:fix
  • npm run lint
  • npm run build
  • npm test (1164 passing)

@github-actions github-actions Bot added the bug Something isn't working label Jun 1, 2026
@tzzs tzzs changed the title fix: normalize hover doc comments fix: harden comment-aware hover and folding Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant