fix: return typing of global functions

[daibhin]

Changes

Seeing these errors locally

src/extensions/replay/sessionrecording.ts:644:9 - error TS2322: Type 'Timeout' is not assignable to type 'number'.

644         this._fullSnapshotTimer = setInterval(() => {
            ~~~~~~~~~~~~~~~~~~~~~~~

src/request-queue.ts:50:9 - error TS2322: Type 'Timeout' is not assignable to type 'number'.

50         this.flushTimeout = setTimeout(() => {
           ~~~~~~~~~~~~~~~~~

src/request.ts:159:13 - error TS2322: Type 'Timeout' is not assignable to type 'number'.

159             timeout: setTimeout(() => controller.abort(), options.timeout),
                ~~~~~~~


Found 3 errors.

Don't know why this only happens in my environment but TS suggests the return type should be NodeJS.Timeout. Depending on the environment posthog-js is running in we can support the global functions return type.

More context: https://stackoverflow.com/questions/45802988/typescript-use-correct-version-of-settimeout-node-vs-window

Checklist

• Tests for new code (see advice on the tests we use)
• Accounted for the impact of any changes across different browsers
• Accounted for backwards compatibility of any changes (no breaking changes in posthog-js!)

[pauldambra]

🚀

[github-actions]

Size Change: 0 B

Total Size: 853 kB

ℹ️ View Unchanged

Filename	Size
dist/array.full.js	183 kB
dist/array.js	124 kB
dist/es.js	124 kB
dist/exception-autocapture.js	12.1 kB
dist/module.js	124 kB
dist/recorder-v2.js	106 kB
dist/recorder.js	58.7 kB
dist/surveys-module-previews.js	62 kB
dist/surveys.js	58.3 kB

_{compressed-size-action}