Skip to content

feat: Heartbeat the Makeswift preview session - #1415

Merged
parthshahp merged 1 commit into
parthshah/ltrac-1794-stencil-cli-sign-every-proxied-request-with-an-hmac-prooffrom
parthshah/ltrac-1796-stencil-cli-session-heartbeat-and-re-registration-on-expiry
Oct 19, 2026
Merged

parthshahp merged 1 commit into
parthshah/ltrac-1794-stencil-cli-sign-every-proxied-request-with-an-hmac-prooffrom
parthshah/ltrac-1796-stencil-cli-session-heartbeat-and-re-registration-on-expiry

Conversation

@parthshahp

@parthshahp parthshahp commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

What?

Send a heartbeat to keep the Makeswift preview session alive, and register again when the session expires.

Tickets / Documentation

cc @bigcommerce/storefront-team

@parthshahp
parthshahp added this pull request to stack #1417 October 8, 2026 15:17
@parthshahp
parthshahp force-pushed the parthshah/ltrac-1796-stencil-cli-session-heartbeat-and-re-registration-on-expiry branch from b319e43 to bf8fded Compare October 8, 2026 15:18
@parthshahp
parthshahp marked this pull request as ready for review October 8, 2026 17:56

@rtalvarez rtalvarez left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we can just use a heartbeat library to alleviate some of the stuff in this PR

@parthshahp
parthshahp force-pushed the parthshah/ltrac-1796-stencil-cli-session-heartbeat-and-re-registration-on-expiry branch from bf8fded to 32da3fb Compare October 9, 2026 15:06
@parthshahp

Copy link
Copy Markdown
Contributor Author

Couldn't find one that fit our use case well without also requiring more boiler plate than we already have.

I refactored the PreviewSessionHeartbeat.js file to be more readable (and remove some extraneous logic).

@parthshahp
parthshahp force-pushed the parthshah/ltrac-1796-stencil-cli-session-heartbeat-and-re-registration-on-expiry branch from 32da3fb to 87dad53 Compare October 16, 2026 14:31
@parthshahp
parthshahp force-pushed the parthshah/ltrac-1796-stencil-cli-session-heartbeat-and-re-registration-on-expiry branch from 87dad53 to d502d13 Compare October 19, 2026 19:06
@parthshahp
parthshahp merged commit df4fd46 into master Oct 19, 2026
4 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 10.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants