Replace hosted_app_home build mode with copy_files mode

[alfonso-noriega]

WHY are these changes introduced?

Fixes https://github.com/shop/issues-admin-extensibility/issues/2240

WHAT is this pull request doing?

How to test your changes?

Post-release steps

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

[alfonso-noriega]

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

• Add a build manifest build step #6869
• Replace hosted_app_home build mode with copy_files mode #6868 👈 (View in Graphite)
• Refactor extension build process to use modular build steps #6867
• Abstract build steps to externalize the build configuration #6866
• Asset pipeline flow for hosted static app #6806 : 2 other dependent PRs (#6842 , #6895 )
• main

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

[github-actions]

Coverage report

St.❔	Category	Percentage	Covered / Total
🟡	Statements	78.92%	14626/18533
🟡	Branches	73.18%	7270/9935
🟡	Functions	79.13%	3711/4690
🟡	Lines	79.27%	13831/17448

Test suite run success

3850 tests passing in 1491 suites.

Report generated by 🧪jest coverage report action from f042eff

[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]

BuildConfig is now a breaking change: mode: 'none' requires steps

Previously BuildConfig was a union where {mode: 'none'} did not include steps. This PR changes it to an interface with mandatory steps for all modes, including none. Existing specs or runtime payloads that omit steps for none will fail type-checking or force dummy steps: [] everywhere.

Evidence: helper defaults updated to {mode: 'none', steps: []} only fixes callers using helpers; other constructors/deserializers remain broken.

Impact: TS compile failures and downstream breakage; may block builds/releases; affects any extension spec using {mode:'none'} without steps.

[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