Skip to content

fix(vue): correct typo in resolveCodeFeatures patch#7

Merged
auvred merged 1 commit intoauvred:mainfrom
keesvanlierop:fix/patch-language-tools-typo
Mar 10, 2026
Merged

fix(vue): correct typo in resolveCodeFeatures patch#7
auvred merged 1 commit intoauvred:mainfrom
keesvanlierop:fix/patch-language-tools-typo

Conversation

@keesvanlierop
Copy link
Contributor

@keesvanlierop keesvanlierop commented Mar 10, 2026

Hi @auvred.

Ran golar on our project, and we get the following (non-blocking) run-time error:

ReferenceError: feature is not defined
    at Object.resolveCodeFeatures

It seems like a small typo, see below more context 🙏


Summary

  • Fix ReferenceError: feature is not defined in patch-language-tools.ts
  • The resolveCodeFeatures wrapper spreads ...feature (undefined) instead of ...features (the local variable declared on the line above)
  • This error triggers whenever golar processes a .vue file with a @vue-expect-error directive in its template

Reproduction

  1. Run golar --noEmit on a Vue project that has .vue files using @vue-expect-error in templates
  2. Observe: ReferenceError: feature is not defined at patch-language-tools.js

Fix

One-character change: ...feature,...features, in the patched resolveCodeFeatures function.

The spread uses `...feature` (undefined) instead of `...features`
(the local variable), causing a ReferenceError at runtime when
processing .vue files with @vue-expect-error template directives.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@auvred
Copy link
Owner

auvred commented Mar 10, 2026

Thank you!!

@auvred auvred merged commit 63cfe37 into auvred:main Mar 10, 2026
3 checks 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.

2 participants