samclarke / robots-parser

100%

push

github

fix: type the CommonJS export with `export =` (#47)

`index.js` is CommonJS (`module.exports = fn`), but `index.d.ts` typed it
as an ES `export default` alongside an empty `declare module 'robots-parser';`.

The empty ambient declaration types the whole module as `any`, so the
`Robot` interface is never actually applied. And `export default` does not
describe `module.exports = fn` under `moduleResolution: node16`/`nodenext`.

Replace both with `export = robotsParser`, matching the runtime. No code
changes. Since the module currently resolves to `any`, this only adds
accurate types and breaks nothing that type-checks today.

Co-authored-by: Mark Edwards <mark@bluefishai.com>

99 of 99 branches covered (100.0%)

Branch coverage included in aggregate %.

146 of 146 relevant lines covered (100.0%)

347781.51 hits per line