node-zwave-js icon indicating copy to clipboard operation
node-zwave-js copied to clipboard

I'm getting compilation errors in the zwave-js library..

Open iklein99 opened this issue 2 years ago • 3 comments

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?

NO, my problem is NOT within Home Assistant or the ZWave JS integration

Is your problem within ZWaveJS2MQTT?

NO, my problem is NOT within ZWaveJS2MQTT

Checklist

Describe the bug

What causes the bug? Getting compilation errors after trying to upgrade to the latest version of the library. What do you observe? Here are the errors: node_modules/zwave-js/build/lib/controller/MockControllerBehaviors.d.ts:1:45 - error TS2307: Cannot find module '@zwave-js/testing' or its corresponding type declarations.

1 import type { MockControllerBehavior } from "@zwave-js/testing"; ~~~~~~~~~~~~~~~~~~~

node_modules/zwave-js/build/lib/driver/Driver.d.ts:76:22 - error TS2420: Class 'Driver' incorrectly implements interface 'ZWaveHost'. Type 'Driver' is missing the following properties from type 'ZWaveHost': controllerLog, securityManager, securityManager2, ownNodeId, and 3 more.

76 export declare class Driver extends TypedEventEmitter<DriverEventCallbacks> implements ZWaveHost { ~~~~~~

Found 2 errors in 2 files.

Device information

NA

How are you using node-zwave-js?

  • [ ] zwavejs2mqtt Docker image (latest)
  • [ ] zwavejs2mqtt Docker image (dev)
  • [ ] zwavejs2mqtt Docker manually built (please specify branches)
  • [ ] ioBroker.zwave2 adapter (please specify version)
  • [ ] HomeAssistant zwave_js integration (please specify version)
  • [ ] pkg
  • [ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
  • [ ] Manually built from GitHub (please specify branch)
  • [X] Other (please describe)

Which branches or versions?

version: `node-zwave-js 9.3.0

Did you change anything?

no

If yes, what did you change?

No response

Did this work before?

Yes (please describe)

If yes, where did it work?

Was working well until I tried to update the library.

Attach Driver Logfile

N/A

iklein99 avatar May 27 '22 18:05 iklein99

👋 Hey @iklein99!

Thanks for opening an issue! It doesn't look like you provided a logfile though. While not strictly necessary for every issue, having a driver log is required to diagnose most issues.

Please consider uploading a logfile that captures your problem.

zwave-js-bot avatar May 27 '22 18:05 zwave-js-bot

You'll have to enable skipLibCheck for now in tsconfig.json. I'll look into fixing the errors when I'm done with the current refactor.

AlCalzone avatar May 30 '22 09:05 AlCalzone

Thanks, Al!

iklein99 avatar May 31 '22 14:05 iklein99