Refactor extension build process to use modular build steps

[alfonso-noriega]

WHY are these changes introduced?

Fixes #0000

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
• Refactor extension build process to use modular build steps #6867 👈 (View in Graphite)
• 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.94%	14634/18539
🟡	Branches	73.19%	7273/9937
🟡	Functions	79.16%	3714/4692
🟡	Lines	79.28%	13837/17453

Test suite run success

3851 tests passing in 1492 suites.

Report generated by 🧪jest coverage report action from 61f4d35

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

Hosted app home build is effectively disabled (behavior change)

The PR changes hosted_app_home from buildConfig: {mode: 'hosted_app_home'} to {mode: 'none'}. Previously, ExtensionInstance.build() had a hosted_app_home branch that called copyStaticAssets(). After this refactor, mode: 'none' produces steps = [], so no build-time actions happen for hosted app home extensions.

Evidence (new behavior):

• ExtensionInstance.build() now does: const steps = buildConfig.mode === 'none' ? [] : buildConfig.steps and runs only those steps.
• HostedAppHome spec now sets mode: 'none', so nothing runs.

Impact: Hosted app home extensions can ship without static assets being copied into the build output. End users (merchants) could see missing UI/resources; deployments may succeed but produce broken runtime behavior. This affects all users of that extension type.

[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