Skip to content

Dedupe setAppUserId identify posts (0.3.3) - #6

Merged
cursor[bot] merged 1 commit into
mainfrom
cursor/dedupe-identify-0-3-3-174a
Oct 13, 2026
Merged

cursor[bot] merged 1 commit into
mainfrom
cursor/dedupe-identify-0-3-3-174a

Conversation

@KennyYe

@KennyYe KennyYe commented Oct 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

Keep Maps and similar apps saw /ingestIdentify fire three times per launch because setAppUserId always POSTed. 0.3.3 skips when the id matches the last successfully identified value (persisted like trackInstall), coalesces concurrent identical calls into one in-flight promise, and still posts when the id changes.

Changes

  • Persist successful appUserId in install state; hydrate restores it
  • Skip /ingestIdentify when appUserId equals the last successful identify
  • Coalesce concurrent identical setAppUserId calls into one POST
  • A different appUserId still posts
  • Add typed TapAppLinkRedeemResult with optional codeMatches and offer.creatorName
  • Bump SDK_VERSION / package to 0.3.3 with changelog

Test plan

  • Same id twice in one session posts once and returns skipped
  • Cold start with persisted id skips POST
  • Concurrent identical calls share one in-flight promise
  • Different id posts again
  • Failed identify is not persisted and retries
  • CI green

After merge: tag v0.3.3 on the merge commit to trigger release.yml npm publish.

Open in Web Open in Cursor 

Skip /ingestIdentify when the app user id matches the last successful
identify (persisted like trackInstall), coalesce concurrent identical
calls, type optional redeem codeMatches and offer.creatorName, and bump
to 0.3.3.

Co-authored-by: Kenny Ye <KennyYe@users.noreply.github.com>
@KennyYe
KennyYe marked this pull request as ready for review October 13, 2026 13:08
@cursor
cursor Bot merged commit f7e30ea into main Oct 13, 2026
2 checks passed
@cursor
cursor Bot deleted the cursor/dedupe-identify-0-3-3-174a branch October 13, 2026 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants