Remove some deprecated dependencies

[gonzaloriestra]

WHY are these changes introduced?

There's a warning about @shopify/function-runner and some subdependencies being deprecated:

❯ pnpm install
Scope: all 14 workspace projects
packages/app                             |  WARN  deprecated @shopify/function-runner@4.1.1
packages/ui-extensions-dev-console       |  WARN  deprecated @shopify/react-i18n@7.14.0
packages/ui-extensions-dev-console       |  WARN  deprecated @shopify/react-testing@5.4.0
 WARN  19 deprecated subdependencies found: @babel/plugin-proposal-class-properties@7.18.6, @babel/plugin-proposal-object-rest-spread@7.20.7, @shopify/dates@2.1.1, @shopify/function-enhancers@3.1.0, @shopify/i18n@2.1.0, @shopify/name@1.3.0, @shopify/react-effect@5.2.0, @shopify/react-hooks@4.1.2, @shopify/useful-types@5.3.0, abab@2.0.6, boolean@3.2.0, domexception@4.0.0, glob@10.5.0, glob@7.2.3, glob@8.1.0, inflight@1.0.6, node-domexception@1.0.0, rimraf@3.0.2, whatwg-encoding@2.0.0
Packages: +1
+
Progress: resolved 2005, reused 1855, downloaded 0, added 1, done
Done in 2.7s using pnpm v10.11.1

WHAT is this pull request doing?

Removes @shopify/function-runner (unused) and a few subdependencies:

❯ pnpm install
Scope: all 14 workspace projects
packages/ui-extensions-dev-console       |  WARN  deprecated @shopify/react-testing@5.4.0
packages/ui-extensions-dev-console       |  WARN  deprecated @shopify/react-i18n@7.14.0
 WARN  14 deprecated subdependencies found: @shopify/dates@2.1.1, @shopify/function-enhancers@3.1.0, @shopify/i18n@2.1.0, @shopify/name@1.3.0, @shopify/react-effect@5.2.0, @shopify/react-hooks@4.1.2, @shopify/useful-types@5.3.0, boolean@3.2.0, glob@10.5.0, glob@7.2.3, glob@8.1.0, inflight@1.0.6, node-domexception@1.0.0, whatwg-encoding@3.1.1
Packages: +12
++++++++++++
Progress: resolved 1991, reused 1841, downloaded 0, added 1, done
Done in 3.8s using pnpm v10.11.1

How to test your changes?

pnpm install

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

[gonzaloriestra]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• Remove @shopify/react-testing #6920
• Remove @shopify/react-i18n #6919
• Remove some deprecated dependencies #6918 👈 (View in Graphite)
• Fix peer dependencies issues #6898
• main

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

[github-actions]

Coverage report

St.❔	Category	Percentage	Covered / Total
🟡	Statements	76.91%	14518/18876
🟡	Branches	70.63%	7221/10224
🟡	Functions	75.93%	3693/4864
🟡	Lines	78.42%	13717/17491

Test suite run success

3788 tests passing in 1448 suites.

Report generated by 🧪jest coverage report action from 24611bb

[github-actions]

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.

[binks-code-reviewer]

rmdir() behavior change may break callers (non-empty dirs, error semantics)

The implementation of rmdir changed from del(path, {force}) to fs.promises.rm(path, {recursive: true, force: options.force ?? false}).

This is not equivalent:

• del() is a glob-aware deletion library and historically has different semantics (it will delete contents via globbing and tends to “just work” even with tricky path patterns).
• fs.rm(..., {recursive:true}) has stricter Node/filesystem semantics and can throw in cases del may have handled (permissions, readonly files on Windows, symlinks, etc.).
• Also, force default is now effectively false (unless passed), which can change behavior vs prior usage if any callers relied on del being more forgiving.

Impact: Could cause CLI operations that clean up temp/build directories to fail unexpectedly (especially on Windows), leaving stale directories behind, breaking follow-up steps (installs/builds), or causing flaky CI/feature tests.

[binks-code-reviewer]

🤖 Code Review · #projects-dev-ai for questions
React with 👍/👎 or reply — all feedback helps improve the agent.

✅ Complete - 1 findings

📋 History

✅ 1 findings