chore: update TS versions for TD and TM

[danielpeintner]

No description provided.

[danielpeintner]

We have issues in browser-bundle... grr
I had the locally too but was hoping it is a caching issue... need to look into

[relu91]

What about this error: https://github.com/eclipse/thingweb.node-wot/actions/runs/3263939568/jobs/5363790618 ?

[danielpeintner]

What about this error: https://github.com/eclipse/thingweb.node-wot/actions/runs/3263939568/jobs/5363790618 ?

Thanks! I missed to store an opened file ...
I will provide an update.

[danielpeintner]

The issue with browserify remains :-(
https://github.com/eclipse/thingweb.node-wot/actions/runs/3273332391/jobs/5385413069

[danielpeintner]

The error message is as follows (at least locally)

> build
> tsc -b && npm run build -w packages/browser-bundle

error TS2688: Cannot find type definition file for 'bl'.
  The file is in the program because:
    Entry point for implicit type library 'bl'

error TS2688: Cannot find type definition file for 'bl'.
  The file is in the program because:
    Entry point for implicit type library 'bl'

Found 2 errors.

https://www.npmjs.com/package/@types/bl seems to be used by coap.

@JKRhb maybe you know whether those types are used somewhere not properly... I might be wrong but maybe you have a hint..

[JKRhb]

Hmm, I tried to reproduce the error, @danielpeintner, but I haven't seen any issues regarding bl locally :/ Instead I am getting the same errors as in the CI job.

However, one of the issues linked in browserify/browserify#1986 seems to have a similar problem (see rdfjs/N3.js#310), related to readable-stream. That module got recently updated to the latest version in node-coap, so my guess would be that this could be the root cause here :/

Edit: One comment in the last issue I linked suggests a workaround until the bug is fixed in browserify: rdfjs/N3.js#310 (comment)

[danielpeintner]

Hmm, I tried to reproduce the error, @danielpeintner, but I haven't seen any issues regarding bl locally :/ Instead I am getting the same errors as in the CI job.

Very strange... the issue locally does not go away. If I call npm install I also see the following warning

npm WARN deprecated @types/bl@5.1.0: This is a stub types definition. bl provides its own type definitions, so you do not need this installed.

According to package-lock.json it is just used in node coap. I found also
https://github.com/coapjs/node-coap/blob/b66eac03a45cc1c6f1ee8b942bdc5f2f45e06dc4/package.json#L60

According to my understanding and https://www.npmjs.com/package/@types/bl it should not be installed directly. Am I wrong?

However, one of the issues linked in browserify/browserify#1986 seems to have a similar problem (see rdfjs/N3.js#310), related to readable-stream. That module got recently updated to the latest version in node-coap, so my guess would be that this could be the root cause here :/

Edit: One comment in the last issue I linked suggests a workaround until the bug is fixed in browserify: rdfjs/N3.js#310 (comment)

I tried to add the workaround but I cannot check locally whether it works (see issue before)
However, I noticed that now also the CI fails with the same problem.
see https://github.com/eclipse/thingweb.node-wot/actions/runs/3530119538/jobs/5921767506
@JKRhb any idea?

[JKRhb]

Thanks for the hint regarding the bl types! I will remove them from node-coap shortly -- maybe that already resolves the problem :)

[danielpeintner]

node-coap v1.2.2 seems to have resolved the coap issue. Thanks @JKRhb!

[danielpeintner]

firestore fails. Other than that I think this PR is finally fine now.

@relu91 ?

[relu91]

Talked offline, it seems that also the browser examples work! merging.