refactor: convert events module to TS#1258
Conversation
PR SummaryMedium Risk Overview Expands commerce APIs to accept additional input shapes: Includes supporting type tweaks (e.g., Reviewed by Cursor Bugbot for commit 12da570. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 14dfc68. Configure here.
…ression input safely
|



Background
What Has Changed
src/events.jstosrc/events.tswith full type annotations on all method parameters, return types, and internal variablestest/src/tests-event-logging.jstotest/src/tests-event-logging.tswith proper global type declarations and type casts for intentionally invalid test inputssrc/events.interfaces.tsto expand thelogImpressionEventsignature to acceptSDKImpression | SDKImpression[] | SDKProductImpression | SDKProductImpression[]src/sdkRuntimeModels.ts,src/serverModel.ts, andsrc/store.tsScreenshots/Video
Checklist
Additional Notes
Reference Issue (For employees only. Ignore if you are an outside contributor)