Skip to content

Add file icons for bsconfig.json and brsconfig.json#801

Open
TwitchBronBron wants to merge 2 commits into
masterfrom
bsconfig-file-icons
Open

Add file icons for bsconfig.json and brsconfig.json#801
TwitchBronBron wants to merge 2 commits into
masterfrom
bsconfig-file-icons

Conversation

@TwitchBronBron

@TwitchBronBron TwitchBronBron commented May 20, 2026

Copy link
Copy Markdown
Member

Summary

  • Add custom file icons for bsconfig.json (BrighterScript) and brsconfig.json (BrightScript), styled to match the built-in tsconfig.json icon
  • Define bsconfig / brsconfig as languages so VSCode can attach the icons via filename patterns
  • Add a stub syntaxes/jsonc.tmLanguage.json that re-uses VSCode's built-in source.json.comments scope — without this, the new languages would render as plain text and jsonValidation would never apply the schema

Here's how they look, in practice (and the tsconfig.json and jsconfig.json for reference)
image

Test plan

  • Reload the extension dev host
  • Confirm bsconfig.json and brsconfig.json show the new colored "BS"/"BRS" icons in the file explorer
  • Confirm JSON syntax highlighting works in both files
  • Confirm bsconfig.schema.json validation kicks in (hover, completions, error squigglies on bad values)

🤖 Generated with Claude Code

Defines `bsconfig` and `brsconfig` as languages so VSCode can attach
custom file icons. Both languages reuse the built-in JSON-with-comments
grammar via a stub `.tmLanguage.json`, which restores syntax
highlighting and lets `jsonValidation` apply `bsconfig.schema.json`.
chrisdp
chrisdp previously approved these changes May 20, 2026
@chrisdp chrisdp dismissed their stale review June 10, 2026 13:18

Found issues like editor.bracketPairColorization.enabled native settings stop working, json schema editor feedback no longer applying.

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