Syntax highlight error: Did not highlight comment after string value

Open

#531 opened on Nov 23, 2025 1 comment 2 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 3/5
Estimated time: 1-2 days
Newbie friendliness: 45/100

Issue type: Bug
Clarity: Mostly clear
Activity status: Stale
Tech stack: typescript, yaml
Domain: tooling

Research direction

Start with the YAML reproduction in the issue and verify the behavior using the extension version v1.28.1 in VS Code. Trace the syntax-highlighting behavior for the inline comment after the quoted value; done means the # Cache causes errors on macOS text is highlighted as a comment.

Written by the indexing model from the issue text.

Description

bug

Describe the bug

In the screenshot below, the comment # Cache causes errors on macOS wasn't highlighted as comment.

      - name: Cache
        uses: actions/cache@v4
        timeout-minutes: 2
        continue-on-error: true
        if: matrix.os != 'macos-latest' # Cache causes errors on macOS
        with:
          path: |
            ~/.cargo
            target
          key: ${{ github.job }}-${{ runner.os }}-${{ hashFiles('rust-toolchain') }}-${{ hashFiles('**/Cargo.lock') }}
          restore-keys: |
            ${{ github.job }}-${{ runner.os }}-${{ hashFiles('rust-toolchain') }}-${{ hashFiles('**/Cargo.lock') }}
            ${{ github.job }}-${{ runner.os }}-${{ hashFiles('rust-toolchain') }}-

To Reproduce

Already provided

Expected behavior

Highlight comment as comment

Screenshots

Already provided

Extension Version
v1.28.1

Additional context
Add any other context about the problem here.

Dominant language: TypeScript
Stars: 661
Forks: 214
PR merge metrics: No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Open

Difficulty 2/5 1-3 hours Newbie friendliness 70/100

tiann/hapi#1918 · 1 comment · Opened Sep 24, 2026
Crush Open

Difficulty 1/5 Under an hour Newbie friendliness 85/100

catppuccin/catppuccin#3125 · Opened Sep 24, 2026
Add a SECURITY.md Open

Difficulty 1/5 Under an hour Newbie friendliness 90/100

ElementsProject/cln-application#167 · 1 comment · 1 reaction · Opened Sep 24, 2026
Dependencies whose license text can't be resolved are silently dropped from the output Open

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

Quantco/pnpm-licenses#17 · Opened Sep 24, 2026
EV search minPowerKW filters out every station that reports connectors Open

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

tomtom-international/tomtom-maps-mcp#284 · Opened Sep 24, 2026

More TypeScript issues