Skip to content

Show schema defaults in hovers - #380

Open
Ching Wei Kang (WilliamK112) wants to merge 1 commit into
microsoft:mainfrom
WilliamK112:codex/default-hover-143
Open

Ching Wei Kang (WilliamK112) wants to merge 1 commit into
microsoft:mainfrom
WilliamK112:codex/default-hover-143

Conversation

@WilliamK112

Copy link
Copy Markdown
Contributor

Summary

  • include JSON-serialized schema default values in property and value hover content
  • localize the default label and retain existing title, description, and enum-value details
  • cover string, numeric zero, false, null, and structured defaults

This addresses the hover portion of #143. The proposed redundant-default diagnostic remains outside this PR because it needs separate policy/design discussion.

Validation

  • npm run compile
  • node --test ./lib/esm/test/hover.test.js (8 passed)
  • full test suite with Node 24.19.0 (5,352 passed)
  • npm run lint (9 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.

1 participant