TypeScript 7 [BLOCKED]

[rkistner]

Blocked until typescript 7.1.0 - see comments.

This upgrades to TypeScript 7.

Build times on my machine, running tsc -b --clean then time tsc -b:

# Before:
tsc -b  14.86s user 0.70s system 172% cpu 9.020 total
# After:
tsc -b  9.07s user 0.74s system 748% cpu 1.312 total

The main changes appear to be from increasing the concurrency, but it does also have a reduction in overall CPU usage.

This also removes nodemon and ts-node, since ts-node is not directly compatible with typescript 7. Possible options:

1. Just remove the pnpm run watch command (current PR state).
2. tsx might be compatible with typescript 7 as a drop-in replacement.
3. Install typescript 6.x next to 7.x.
4. Use node --watch for restarting the process, along with tsc -b -w in a separate tab.

No changeset since this does not change the build output.

AI Usage

None.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 578347f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[rkistner]

@stevensJourney I believe you originally added the pnpm watch command - are you still actively using it? If not, I think the simplest option is to just remove it.

[Rentacookie]

@stevensJourney I believe you originally added the pnpm watch command - are you still actively using it? If not, I think the simplest option is to just remove it.

I still use the watch command to start up the service locally. But if there is an alternative to that I'd gladly use it.

[rkistner]

It looks like in theory node --watch along with a separate tsc -b -w can work quite well. @Rentacookie would that be fine for you (needing two commands instead of one)?

That said, during testing I found that tsc -b -w is unusable on my machine right now, blocking this PR. That should be resolved in 7.1.0, so we can wait until then before merging this: microsoft/typescript-go#4795

[Rentacookie]

It looks like in theory node --watch along with a separate tsc -b -w can work quite well. @Rentacookie would that be fine for you (needing two commands instead of one)?

That said, during testing I found that tsc -b -w is unusable on my machine right now, blocking this PR. That should be resolved in 7.1.0, so we can wait until then before merging this: microsoft/typescript-go#4795

That should be fine then. I am not stressed about auto triggering a rebuild for file changes while running the service. 🙇

[stevensJourney]

@stevensJourney I believe you originally added the pnpm watch command - are you still actively using it? If not, I think the simplest option is to just remove it.

I do still use the watch command in my dev setup - it's quite convenient, but I'm happy to work around it.

I think we already use concurrently in the watch:service command, so I think we could run both node --watch and tsc -b -w in the single script. I'm also happy if I have to run two commands.