Skip to content

Add UI extension errors to app logs-next - #9104

Open
grollest wants to merge 1 commit into
grollest/app-logs-sidekickfrom
grollest/app-logs-ui-extension
Open

grollest wants to merge 1 commit into
grollest/app-logs-sidekickfrom
grollest/app-logs-ui-extension

Conversation

@grollest

@grollest grollest commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

UI extension error records are not yet available through historical CLI retrieval.

WHAT is this pull request doing?

Adds --type ui-extension, reusing the target and result-status filters. Default queries include UI extension errors, and the human table shows the recorded target under Details.

Part 6 of the six-PR log-type stack, based on #9103. Tracking: EXTLOGS-108.

Validation

  • 269 focused tests passed, including native status/error/extension fields and human Details/result rendering.
  • Targeted ESLint, app and CLI type-checks, CLI bundle, manifest refresh, command snapshot, and diff checks passed.
  • Production tophat — 2026-11-09, tested head 7952ae96fadb, against billing-event-tophat-oct27.
    • Shared seven-day UTC window: 2026-11-02T17:03:32Z to 2026-11-09T17:03:32Z.
    • --type ui-extension --limit 20: 10 records returned. JSON and human commands both exited 0; returned types and timestamps matched the request.
    • Human table: 10 rows, populated Details values, expected headings, and no GraphQL-only API column. Result/status counts matched the native JSON sample.
    • Live exact filters --target: 10 matching records, using values from the returned records; all returned values matched the predicates.
    • The first UI target-filter request was rate-limited; one retry after a 65-second cooldown passed.
    • Record contents and filter values kept private. No source or service changes. This replaces the earlier production validation notes.
  • Prerequisite Rename the App Logs type from WEBHOOK_DELIVERY to WEBHOOK #9098 is merged; the WEBHOOK schema-freshness check now passes.
  • Command remains hidden; no changeset for this pre-release stack.

How to manually test your changes?

Using this branch's CLI with access to an app that has recent ui extension errors:

shopify app logs-next --type ui-extension --since 1h --limit 5
shopify app logs-next --type ui-extension --target purchase.checkout.block.render --status failure --since 1h --limit 5 --json

Confirm the recorded target and result appear in the human table, and JSON preserves the native extension and error fields.

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • The change is user-facing — I've identified the correct bump type and added a changeset with pnpm changeset add

@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Nov 6, 2026
@grollest
grollest added this pull request to stack #9105 November 6, 2026 21:17
@isaacroldan
isaacroldan force-pushed the grollest/app-logs-ui-extension branch from 137271a to 7952ae9 Compare November 9, 2026 12:44
@grollest
grollest marked this pull request as ready for review November 9, 2026 17:52
@grollest
grollest requested review from a team as code owners November 9, 2026 17:52

This branch has not been deployed

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

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant