Skip to content

Add REST requests to app logs-next - #9102

Open
grollest wants to merge 2 commits into
grollest/app-logs-app-billing-eventfrom
grollest/app-logs-rest
Open

grollest wants to merge 2 commits into
grollest/app-logs-app-billing-eventfrom
grollest/app-logs-rest

Conversation

@grollest

@grollest grollest commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

REST API requests are not yet available through historical CLI retrieval.

WHAT is this pull request doing?

Adds --type rest, with --method and --request-path filters. Reuses the request-ID, HTTP status-code, and result-status filters, updates their applicability help, and shows the recorded method and path under Details.

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

Validation

  • 253 focused tests passed, covering new flag/env wiring, exact filters, native JSON, and Details rendering.
  • Targeted ESLint, app type-check, CLI bundle, manifest refresh, command snapshot, and diff checks passed.
  • Production tophat — 2026-11-09, tested head d844f6701495, against billing-event-tophat-oct27.
    • Shared seven-day UTC window: 2026-11-02T17:03:32Z to 2026-11-09T17:03:32Z.
    • --type rest --limit 20: 20 records returned (requested limit reached; this is a sample, not a total). JSON and human commands both exited 0; returned types and timestamps matched the request.
    • Human table: 20 rows, populated Details values, expected headings, and no GraphQL-only API column. Result/status counts matched the native JSON sample.
    • Live exact filters --method + --request-path: 20 matching records, using values from the returned records; all returned values matched the predicates.
    • 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 rest requests:

shopify app logs-next --type rest --since 1h --limit 5
shopify app logs-next --type rest --method GET --request-path /admin/api/2026-10/orders.json --since 1h --limit 5 --json
shopify app logs-next --type rest --request-id REQUEST_ID --status-code 500 --since 1h --limit 5 --json

Confirm Details shows the method and recorded path, exact filters constrain the query, and JSON preserves the native request 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-rest branch from 3e37643 to d844f67 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