Spike official Shopify Chat web component migration - #468
Conversation
|
Reviewed and verified locally against current Converting to draft rather than merging — the duplicate-bootstrap defect you flagged is the right call, and the preview dependency compounds it. Not a quality objection; this is blocked upstream. VerificationThis PR predates the CI added in #454/#470, so its checks only ran CodeQL. I merged it with current
The migration itself holds up. Replacing the private Shadow DOM selectors ( Why it stays a draft1. The duplicate bootstrap, as you documented. Hydration leaves two DOM scripts per generated ID — Inbox, consent, analytics bus. Pilot is the reference theme, so that ships to every downstream storefront. 2. The preview dependency. "@shopify/hydrogen-preview": "npm:@shopify/hydrogen@0.0.0-preview-116d5d7-20260730141607"A SHA-stamped dev-preview build aliased next to stable Hydrogen. #454 rejected an npm Both resolve when Shopify ships a compatible stable release. #467 tracks that. One note for whenever this unblocks
|
Summary
shopifyChatV1.jsloader and private Shadow DOM selectors with Hydrogen's officialShopifyScriptsInbox integration and<shopify-chat>componentPUBLIC_SHOPIFY_INBOX_SHOP_IDVerification
npm run typechecknpm run test:unit— 11 passednpm run buildshow()/close(), and overlay hiding verifiedPreview limitation
React 19 hydration leaves duplicate DOM script tags for each script generated by the preview
ShopifyScriptscomponent, despite SSR emitting one copy. The integration works in the smoke test, but this must be resolved upstream or during stable migration before production rollout.Refs #467