chore(CI): add Node.js 22 to matrix

[danielpeintner]

fixes #1273

[danielpeintner]

npm ERR! code 1
npm ERR! path /home/runner/work/node-wot/node-wot/node_modules/node-mbus

mbus modbus seems to cause issues on Node.js 22

[danielpeintner]

BTW, I created Apollon77/node-mbus#123

[danielpeintner]

I messed up mbus with modbus 🙈

[danielpeintner]

The new error seems to stem from a wrong node installation!?!

https://github.com/eclipse-thingweb/node-wot/actions/runs/8878604576/job/24374716602?pr=1274

[danielpeintner]

Seems to be related with nodejs/node#52682 and a possible fix can be found here nodejs/node#52682 (comment)

Let's wait a bit. I assume it gets fixed with a new minor version soon.

[relu91]

Should we go with this suggestion ?

[danielpeintner]

Should we go with this suggestion ?

No longer needed since the GH workflow should pick the latest Node.js 22.x version.
Re-running the workflow seems to show that

https://github.com/eclipse-thingweb/node-wot/actions/runs/8878604576/job/24558179052?pr=1274#step:3:91

Anyhow, we are now running into other issues 😒

Now, I think they are really mbus-related.
https://github.com/eclipse-thingweb/node-wot/actions/runs/8878604576/job/24558179052?pr=1274#step:5:25

I will take a closer look next week...

[danielpeintner]

Created Apollon77/node-mbus#124

[danielpeintner]

mbus issue should be resolved (see Apollon77/node-mbus#124 (comment)). We just need to wait till 2.24 is published.

[danielpeintner]

We now have some other issues in the step "Test with coverage report"

I do not really understand "what" is failing. I just see "passing" or "pending"

Anyhow, it fails with Error: Process completed with exit code 1.
see https://github.com/eclipse-thingweb/node-wot/actions/runs/9252944897/job/25847856047?pr=1274#step:9:6944

[danielpeintner]

I am seeing logs like this for Node.js 22

(node:4256) [DEP0040] DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.

This might be solved by updating AJV. see https://stackoverflow.com/a/77929792

Grr: There are more dependencies: see

+-- @node-wot/[email protected] -> .\packages\binding-http
| `-- [email protected]
|   `-- [email protected]
|     `-- [email protected]
|       `-- [email protected]
|         `-- [email protected] deduped
+-- @node-wot/[email protected] -> .\packages\browser-bundle
| `-- [email protected]
|   +-- [email protected]
|   `-- [email protected]
|     `-- [email protected]
`-- @node-wot/[email protected] -> .\packages\cli
  `-- [email protected]
    `-- [email protected]
      `-- [email protected]

OR

2024-06-05T15:17:05.296Z node-wot:cli:cli-default-servient:error Servient found error in privileged script 'script' Bad synchronous error in Servient sandbox
(node:6876) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 uncaughtException listeners added to [process]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit

Here I am not sure. Is it a "vm2" issue?
@relu91

[relu91]

Tested locally and I was able to reproduce the error. All the packages are working correctly but mbus:

npm run test -w packages/binding-mbus/

> @node-wot/[email protected] test
> mocha --require ts-node/register --extension ts



  mbus client test
    ✔ should override form values with URL
    read resource
      ✔ should throw exception for missing offset

  mbus connection test
npm error Lifecycle script `test` failed with error:
npm error Error: command failed
npm error   in workspace: @node-wot/[email protected]
npm error   at location: E:\Progetti\reluWoT\packages\binding-mbus

I don't have that much time to investigate but it is the only package that is falling (you can verify yourself by excluding it by the workspaces list).