reactivando / use-pubsub-js / 28117362938

100%

master: 80%

push

github

feat(v2-M4)!: semver-major type cleanups (#73)

* feat(v2-M4)!: semver-major type cleanups

BREAKING type changes bundled for v2.0.0:
- useSubscribe handler payload `Message` any -> unknown (handlers must now
  narrow/guard the payload).
- rename exported interfaces IUsePublishResponse/IUsePublishParams ->
  UsePublishResponse/UsePublishParams (match the UseSubscription* naming).
- remove the undocumented default export from src/index.ts (named exports
  only); also clears the rolldown MIXED_EXPORTS warning. Added a barrel test
  asserting the named API exists and there is no default export.
- debounce.ts: drop @ts-nocheck; narrow the generic to
  `<T extends (...args: any[]) => any>` and type the internals (args:
  Parameters<T>|null, result: ReturnType<T>|undefined); clear/flush are arrows
  now. Slimmed its Biome override to just noExplicitAny.

Also surfaces the public types and `createPubSub` from the barrel (additive).
`debounceMs: number | string` kept as-is (decided). 107 tests, 100% coverage,
tsc/lint/build/e2e green, attw all-green, publint clean.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* fix(v2-M4): narrow unknown payload in the example (any->unknown fallout)

The example-build CI gate (added in M2) correctly caught that the example's
useSubscribe handler typed `data: string` no longer compiles now that the
payload is `unknown`. Narrow with String(data) — this is exactly the consumer
migration the any->unknown change requires, demonstrated in the example.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

25 of 25 branches covered (100.0%)

Branch coverage included in aggregate %.

9 of 9 new or added lines in 1 file covered. (100.0%)

162 of 162 relevant lines covered (100.0%)

30.2 hits per line