RobotWebTools / rclnodejs / 27263579073

91%

push

github

fix(types): add type declarations for the rclnodejs/rosocket subpath (#1533)

The `./rosocket` subpath export shipped **no type declarations**, so under `node16`/`nodenext` module resolution:

- `import { startRosocket } from 'rclnodejs/rosocket'` failed with **TS7016** (the module implicitly had an `any` type).
- `@arethetypeswrong` reported **No types** for the subpath across every resolution mode.

### Changes

- **Add `rosocket/index.d.ts`** — types the public entry point (`startRosocket`, `StartRosocketOptions`, `RosocketServer`), matched against `rosocket/index.js` (`port=9000`, `host='0.0.0.0'`, required `node`, returns `{ wss, close, port }`). `wss` is typed as its `EventEmitter` supertype to avoid a hard dependency on `ws` types.
- **Add `rosocket/index.d.cts`** — a CommonJS-format companion (re-exporting the `.d.ts`) so the `require` condition resolves to a CJS type file and is not flagged as "Masquerading as ESM".
- **Wire per-condition `types` into the `./rosocket` export** in `package.json` (`import` → `index.d.ts`, `require` → `index.d.cts`).


Fix: #1532

2044 of 2402 branches covered (85.1%)

Branch coverage included in aggregate %.

16688 of 18133 relevant lines covered (92.03%)

227.22 hits per line