Typescript 7
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- Half a day
- Newbie friendliness
- 48/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- typescript, vim
- Domain
- devtools
Research direction
The problem is the tsserver linter and fixer in ALE, which still launches a tsserver binary that TypeScript 7 removed. Start by finding where ALE defines the tsserver command, then check whether the tsc --lsp --stdio invocation from the issue can replace it, and run :ALEInfo on a TypeScript 7 project to confirm the executable check. Done means linting and fixing work on TypeScript 7 without breaking TypeScript 6 projects.
Written by the indexing model from the issue text.
Description
Information
VIM version
NVIM v0.12.5
Build type: RelWithDebInfo
Operating System: Arch Linux
What went wrong
Updating a project from typescript 6 to typescript 7 causes the tsserver ALE linter/fixer to stop working.
My understanding is this is because typescript 7 has removed the tsserver binary and incorporated the functionality into tsc. So you are supposed to run tsc --lsp --stdio (source).
I guess its meant to be a drop-in, but I was unable to verify that. I tried creating a tsserver script that just execd that command, but that didn't work. ALE found it and executed it, but no linting/fixing occurred.
Reproducing the bug
Have a typescript 6 project using tsserver successfully. Upgrade the project to typescript 7.
:ALEInfo
I can provide this if necessary, but I downgraded my project now so I would have to re-upgrade to do so. The only relevant bit was that it showed an executable check failure for tssever, as I'd expect.
- Dominant language
- Vim Script
- Stars
- 14k
- Forks
- 1.5k
- PR merge metrics
- No merged PRs in 30d
Getting set up
- Ships a Dockerfile or Docker Compose file
- Has a pull request template
- Read 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.
More from dense-analysis/ale
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
dense-analysis/ale#5181 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
dense-analysis/ale#5180 · 1 comment ·
-
new tool
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
dense-analysis/ale#5176 ·
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 75/100
dense-analysis/ale#5174 ·
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
dense-analysis/ale#5158 ·
All issues in dense-analysis/ale
Similar issues
-
bug parity
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
WenyuChiou/agent-board#1489 ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 72/100
ScoopInstaller/Extras#19364 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
osrep/Runes2P.jl#279 ·
Maintainers usually reply within 1 day
-
Incorrect Ruff version demandedPossibly taken @FrancescoPezzella claimed this today. Open
Difficulty 1/5 Under an hour Newbie friendliness 88/100
zarr-developers/VirtualiZarr#1160 ·
Maintainers usually reply within 3 days
-
area: sdk bug complexity: trivial
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
accensa/x402-facilitator-stellar#697 ·
Maintainers usually reply within 1 day