Skip to content

fix: update default Angular LS and TypeScript versions#84

Merged
notpeter merged 1 commit intonathansbradshaw:mainfrom
pmig:fix/update-default-versions
Mar 14, 2026
Merged

fix: update default Angular LS and TypeScript versions#84
notpeter merged 1 commit intonathansbradshaw:mainfrom
pmig:fix/update-default-versions

Conversation

@pmig
Copy link
Copy Markdown
Contributor

@pmig pmig commented Feb 25, 2026

This actually a series a fixes to make that extension work again. You can try out the extension locally over at https://github.com/pmig/zed-angular. Although most of the code was written by claude, I (@pmig) manually reviewed and tested it.

Summary

  • Updates @angular/language-server default from 19.0.4 to 21.1.5
  • Updates typescript default from 5.7.3 to 5.9.3

Angular v21 moved type declarations from @angular/core/core.d.ts to @angular/core/types/core.d.ts. The v19 language server's project detection uses path.endsWith("@angular/core/core.d.ts") which no longer matches, causing it to disable the language service entirely with the message: "project is not an Angular project ('@angular/core' could not be found)".

The v21 language server uses a regex (/@angular\/core\/.+\.d\.ts$/) that correctly matches the new path structure.

This is the root cause of autocompletion and diagnostics not working for Angular v21+ projects.

Closes #77
Relates to #69

Update @angular/language-server from 19.0.4 to 21.1.5 and TypeScript
from 5.7.3 to 5.9.3.

Angular v21 moved type declarations from @angular/core/core.d.ts to
@angular/core/types/core.d.ts. The v19 language server's project
detection checks for path.endsWith("@angular/core/core.d.ts") which
no longer matches, causing it to disable the language service with
"project is not an Angular project ('@angular/core' could not be
found)". The v21 language server uses a regex that matches any .d.ts
file under @angular/core/.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 25, 2026 13:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request updates the default versions of Angular Language Server and TypeScript dependencies to support Angular v21+ projects. Angular v21 changed the location of type declarations from @angular/core/core.d.ts to @angular/core/types/core.d.ts, which breaks project detection in the v19 language server. The v21 language server uses a regex pattern that correctly handles the new path structure.

Changes:

  • Updates @angular/language-server default version from 19.0.4 to 21.1.5
  • Updates typescript default version from 5.7.3 to 5.9.3

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Collaborator

@notpeter notpeter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems sane to me.

@notpeter notpeter merged commit 95a8d1a into nathansbradshaw:main Mar 14, 2026
4 of 5 checks passed
github-merge-queue bot pushed a commit to zed-industries/extensions that referenced this pull request Mar 18, 2026
Includes: 

- Diff:
nathansbradshaw/zed-angular@v0.0.4...v0.0.5
- nathansbradshaw/zed-angular#84
- nathansbradshaw/zed-angular#80
- nathansbradshaw/zed-angular#82
- nathansbradshaw/zed-angular#67
- nathansbradshaw/zed-angular#60

Update tree-sitter-angular to v0.8.3:

- Diff:
dlvandenberg/tree-sitter-angular@8b4f7e6...f0d0685
- Complex CSS classes breaking parsing
- Template string as primitive
- Hyphen in identifier
- Percentage sign as unit for style property binding
- Class attribute modifier starting with hyphen

Closes: nathansbradshaw/zed-angular#71
cramhead pushed a commit to cramhead/extensions that referenced this pull request Mar 31, 2026
Includes: 

- Diff:
nathansbradshaw/zed-angular@v0.0.4...v0.0.5
- nathansbradshaw/zed-angular#84
- nathansbradshaw/zed-angular#80
- nathansbradshaw/zed-angular#82
- nathansbradshaw/zed-angular#67
- nathansbradshaw/zed-angular#60

Update tree-sitter-angular to v0.8.3:

- Diff:
dlvandenberg/tree-sitter-angular@8b4f7e6...f0d0685
- Complex CSS classes breaking parsing
- Template string as primitive
- Hyphen in identifier
- Percentage sign as unit for style property binding
- Class attribute modifier starting with hyphen

Closes: nathansbradshaw/zed-angular#71
pataruco pushed a commit to pataruco/extensions that referenced this pull request Apr 15, 2026
Includes: 

- Diff:
nathansbradshaw/zed-angular@v0.0.4...v0.0.5
- nathansbradshaw/zed-angular#84
- nathansbradshaw/zed-angular#80
- nathansbradshaw/zed-angular#82
- nathansbradshaw/zed-angular#67
- nathansbradshaw/zed-angular#60

Update tree-sitter-angular to v0.8.3:

- Diff:
dlvandenberg/tree-sitter-angular@8b4f7e6...f0d0685
- Complex CSS classes breaking parsing
- Template string as primitive
- Hyphen in identifier
- Percentage sign as unit for style property binding
- Class attribute modifier starting with hyphen

Closes: nathansbradshaw/zed-angular#71
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.

No error reporting / autocompletion

3 participants