Skip to content

feat(push): add client.push.send to the Node SDK#11

Merged
Tsabary merged 1 commit into
mainfrom
feature/push-notifications
Jun 23, 2026
Merged

feat(push): add client.push.send to the Node SDK#11
Tsabary merged 1 commit into
mainfrom
feature/push-notifications

Conversation

@Tsabary

@Tsabary Tsabary commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Task 4.3: node-sdk/src/modules/push/send.tssend(client, { userIds, title, body, data? }) function
  • client.push bound on SublayClient via the existing bindModule pattern
  • SendPushProps / SendPushResult interfaces (per-user/per-device shape matching the server's /push-notifications/send response)
  • Task 4.4: node-sdk/__tests__/push.test.ts — mocks projectInstance.post, asserts request path + body shape, confirms no field renaming

Test plan

  • npm test passes
  • npm run build compiles with no type errors
  • client.push.send({ userIds, title, body }) is typed and callable on SublayClient
  • Test confirms request goes to POST /push-notifications/send with the flat body unchanged

🤖 Generated with Claude Code

Binds a new push module exposing send({ userIds, title, body, data? }),
mirroring the server's POST /push-notifications/send contract exactly
(flat body, no client-side reshaping). Adds SendPushResult/PushDeviceResult
to the public interface exports and documents the new module in CLAUDE.md.
@Tsabary Tsabary self-assigned this Jun 23, 2026
@Tsabary Tsabary merged commit 375c792 into main Jun 23, 2026
1 check passed
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.

1 participant