Skip to content

Renamed offline_mode to runs_offline#6888

Open
js-goupil wants to merge 3 commits intomainfrom
jsgoupil/add-runs-offline-support
Open

Renamed offline_mode to runs_offline#6888
js-goupil wants to merge 3 commits intomainfrom
jsgoupil/add-runs-offline-support

Conversation

@js-goupil
Copy link
Contributor

@js-goupil js-goupil commented Feb 23, 2026

WHY are these changes introduced?

We are renaming offline_mode to runs_offline after some feedback from a GSD review. All of the internal API is left as offline_mode in order to ship this and reduce the amount of friction for now. We can migrate/refactor later if we decide it's worthwhile.

WHAT is this pull request doing?

Just a rename for the developer facing toml

How to test your changes?

  1. Create an extension with supported_features.runs_offline set to true or false
  2. Run pnpm shopify app dev --path {Path to extension} to run this app locally and deploy the draft to your dev shop

Measuring impact

How do we know this change was effective? Please choose one:

  • n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
  • Existing analytics will cater for this addition
  • PR includes analytics changes to measure impact

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@js-goupil js-goupil changed the title Add runs_offline support for UI extensions Renamed offline_mode to runs_offline Feb 23, 2026
@js-goupil js-goupil marked this pull request as ready for review February 23, 2026 20:16
@js-goupil js-goupil requested a review from a team as a code owner February 23, 2026 20:16
@github-actions
Copy link
Contributor

We detected some changes at packages/*/src and there are no updates in the .changeset.
If the changes are user-facing, run pnpm changeset add to track your changes and include them in the next release CHANGELOG.

Caution

DO NOT create changesets for features which you do not wish to be included in the public changelog of the next CLI release.

js-goupil and others added 2 commits February 25, 2026 12:06
Add supported_features.runs_offline configuration option for UI extensions
that enables offline mode. The TOML config uses `runs_offline` but it is
transformed to `offline_mode` when sent to the backend API.

- Add runs_offline to SupportedFeaturesSchema
- Transform runs_offline -> offline_mode in deployConfig for backend
- Add offlineMode to dev server payload for local development
- Add comprehensive tests for all scenarios

Co-authored-by: Cursor <cursoragent@cursor.com>
Update remaining test fixtures to use the new runs_offline field name
in the configuration schema.

Co-authored-by: Cursor <cursoragent@cursor.com>
@js-goupil js-goupil force-pushed the jsgoupil/add-runs-offline-support branch from 448e354 to dbd1a35 Compare February 25, 2026 17:11
Backend now accepts runs_offline directly, no need to transform to
offline_mode. Pass supported_features through as-is.

Co-authored-by: Cursor <cursoragent@cursor.com>
@js-goupil js-goupil force-pushed the jsgoupil/add-runs-offline-support branch from dbd1a35 to b31410f Compare February 25, 2026 17:13
@github-actions
Copy link
Contributor

github-actions bot commented Feb 25, 2026

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements 78.78% 14487/18389
🟡 Branches 73.12% 7202/9850
🟡 Functions 79.01% 3686/4665
🟡 Lines 79.12% 13695/17309

Test suite run success

3768 tests passing in 1448 suites.

Report generated by 🧪jest coverage report action from b31410f

Copy link
Contributor Author

@aaronschubert0 I'll merge this once the backend PR hits prod, that way we can tophat this full flow against core

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants