Skip to content

Show App Events extension status in dev sessions - #8319

Closed
Barefoot0 wants to merge 2 commits into
mainfrom
analytics-app-events-cli-output
Closed

Show App Events extension status in dev sessions#8319
Barefoot0 wants to merge 2 commits into
mainfrom
analytics-app-events-cli-output

Conversation

@Barefoot0

@Barefoot0 Barefoot0 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

TL;DR

Show analytics_app_events Extension loaded after Shopify CLI successfully creates or updates a dev session containing an Analytics App Events extension.

Context

analytics_app_events is currently a remote-only extension specification. Shopify CLI includes its TOML configuration in the dev-session flow, but the extension has neither client build steps nor a getDevSessionUpdateMessages hook, so it is silent after Ready.

Changes

  • Register a thin local, contract-based analytics_app_events specification that returns Extension loaded through the existing dev-session message hook.
  • Let contract-based specifications opt into that existing hook without changing the default behavior of any other extension type.
  • Retain the remote platform JSON schema as the source of truth for App Events configuration validation.
  • Add coverage for the exact handle-prefixed message, remote-schema merging, and silence for a representative existing contract-based extension.

@github-actions github-actions Bot added the Area: @shopify/app @shopify/app package issues label Aug 12, 2026
@Barefoot0
Barefoot0 force-pushed the analytics-app-events-cli-output branch from c1a3460 to 07a4568 Compare August 12, 2026 20:48
@Barefoot0
Barefoot0 marked this pull request as ready for review August 13, 2026 12:53
@Barefoot0
Barefoot0 requested a review from a team as a code owner August 13, 2026 12:53
@isaacroldan

Copy link
Copy Markdown
Contributor

/snapit

@github-actions

Copy link
Copy Markdown
Contributor

🫰✨ Thanks @isaacroldan! Your snapshot has been published to npm.

Test the snapshot by installing your package globally:

pnpm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20260814142720

Caution

After installing, validate the version by running shopify version in your terminal.
If the versions don't match, you might have multiple global instances installed.
Use which shopify to find out which one you are running and uninstall it.

Assisted-By: devx/b10cb676-29cc-4a4f-8b91-853b8aa9b5a7
Assisted-By: devx/b10cb676-29cc-4a4f-8b91-853b8aa9b5a7
@Barefoot0
Barefoot0 force-pushed the analytics-app-events-cli-output branch from 07a4568 to e668509 Compare August 17, 2026 18:05
@Barefoot0

Copy link
Copy Markdown
Contributor Author
Screenshot 2026-08-17 at 2 57 29 PM Image of my local testing. It logs a message when it is loaded, but doesn't repeat that message every time it is changed.

@amcaplan

Copy link
Copy Markdown
Contributor

This code passes through a very specific value (session status) for a very specific purpose, which I don't love as it emphasizes special casing in shared code. ‏ ‏I can see an argument for passing through the entire dev session result, though, as it opens up more possibilities for various extensions in the future.

@Barefoot0

Copy link
Copy Markdown
Contributor Author

Closed by #8350 (Dev Platform decided to make it a general implementation, not specific to one extension).

@Barefoot0 Barefoot0 closed this Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: @shopify/app @shopify/app package issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants