chore: update nx packages

[toddbaert]

This PR:

• updates to nx@latest (had to do 17, then latest to get new lint modules to work). I did:
  • npx nx migrate 17
  • npx nx migrate latest
  • then just run the lint fix and manually fixed a couple lint issues
  • removed deprecated workspace-lint from CI (this validates the project config, but it's automatic now): feat(core): remove deprecated workspace-lint nrwl/nx#16212
  • added the new lint rule below
• updates @types/node (required to fix new compilation issues)
• updates the generators

⚠️

There is one significant change in how this repo works: we no longer automatically compute the deps for all packages, BUT we instead throw a lint error if nx calculates that you have failed to include a dep in your package.json which you import in your code.

For example, flagd uses the lru-cache package. Because we import it in our ts files, if we DON'T include it in our pacakge.json, we see this lint error (which causes the build to fail):

nx run providers-flagd:lint
      Linting "providers-flagd"...
      
      /home/todd/git/js-sdk-contrib/libs/providers/flagd/package.json
        9:3  error  The "providers-flagd" project uses the following packages, but they are missing from "peerDependencies":
          - lru-cache  @nx/dependency-checks
      
      ✖ 1 problem (1 error, 0 warnings)
        1 error and 0 warnings potentially fixable with the `--fix` option.

I think this is actually an improvement over the last behavior, since it will cause devs to think carefully about their deps.

⚠️

I have updated all the deps for all the packages appropriately according to this check. Please check the deps of your package.json.

[lukas-reining]

Good that we update that!

[toddbaert]

Unfortunately, the nx-computed dependencies created by buildableProjectDepsInPackageJsonType no longer work (it was removed); this means we aren't properly computing deps at package time anymore. There's a new way to do this I need to get working.

Marking draft until I figure this out.

[toddbaert]

Unfortunately, the nx-computed dependencies created by buildableProjectDepsInPackageJsonType no longer work (it was removed); this means we aren't properly computing deps at package time anymore. There's a new way to do this I need to get working.

Marking draft until I figure this out.

Looks like we can use @nx/dependency-checks for a comparable feature. I almost have it working.

[toddbaert]

⚠️

There is one significant change in how this repo works: we no longer automatically compute the deps for all packages, BUT we instead throw a lint error if nx calculates that you have failed to include a dep in your package.json which you import in your code.

For example, flagd uses the lru-cache package. Because we import it in our ts files, if we DON'T include it in our package.json, we see this lint error (which causes the build to fail):

nx run providers-flagd:lint
      Linting "providers-flagd"...
      
      /home/todd/git/js-sdk-contrib/libs/providers/flagd/package.json
        9:3  error  The "providers-flagd" project uses the following packages, but they are missing from "peerDependencies":
          - lru-cache  @nx/dependency-checks
      
      ✖ 1 problem (1 error, 0 warnings)
        1 error and 0 warnings potentially fixable with the `--fix` option.

I think this is actually an improvement over the last behavior, since it will cause devs to think carefully about their deps.

⚠️

I have updated all the deps for all the packages appropriately according to this check. Please check the deps of your package.json.

[toddbaert]

This is the new lint rule which ensures projects have all the dependencies they need.

[toddbaert]

We should avoid direct imports from core in server/web libs. It technically isn't a problem, but it adds a direct dep to core which is not needed.

[toddbaert]

we we're importing from server in a web lib 😨

This was caught because the new lint tried to add the server-sdk to the deps.

[toddbaert]

We now use TestContainers to start/stop all containers during testing.

[lukas-reining]

Looks good to me! Nice update!