Switch deploy to the active-app-version classification

[isaacroldan]

Part 7 of 8 — splitting #8040. The switch-over.

WHY are these changes introduced?

With the classification module in place (#8055), deploy can stop using the interactive matching flow.

WHAT is this pull request doing?

• ensureDeployContext now calls ensureDeployIdentifiersFromAppVersion and returns DeployIdentifiers.
• The bundle/upload path consumes the resulting maps; upload's extensionIds becomes appModuleRegistrationIds.
• deploy no longer writes identifiers back to the .env file.

The interactive matching flow is now unused; it's removed in #8049.

⚠️ User-facing behavior change (deploy classification is non-interactive; identifiers are no longer persisted to .env) — needs a changeset.

How to test your changes?

pnpm --filter @shopify/app exec vitest run src/cli/services/deploy.test.ts src/cli/services/context.test.ts src/cli/services/deploy/upload.test.ts

[isaacroldan]

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 the Partners API developer platform client from app commands #8079
• Remove the legacy extension identifier-matching code #8049
• Switch deploy to the active-app-version classification #8056 👈 (View in Graphite)
• Add active-app-version deploy classification module #8055
• Rework release config diff to compare version against active version #8054
• Make deploy consumers speak app-module UUID/registration-ID maps #8047 : 1 other dependent PR (#8048 )
• Fix webhook subscription UID to omit the literal "undefined" for absent filters #8046
• Classify webhook subscription modules as configuration experience #8045
• Add order-insensitive deep object comparison to cli-kit #8044
• main

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

[github-actions]

Differences in type declarations

We detected differences in the type declarations generated by Typescript for this branch compared to the baseline ('main' branch). Please, review them to ensure they are backward-compatible. Here are some important things to keep in mind:

• Some seemingly private modules might be re-exported through public modules.
• If the branch is behind main you might see odd diffs, rebase main into this branch.

New type declarations

We found no new type declarations in this PR

Existing type declarations

packages/cli-kit/dist/public/common/version.d.ts

@@ -1 +1 @@
-export declare const CLI_KIT_VERSION = "4.5.0";
\ No newline at end of file
+export declare const CLI_KIT_VERSION = "4.4.0";
\ No newline at end of file

[gonzaloriestra]

⚠️ User-facing behavior change (deploy classification is non-interactive; identifiers are no longer persisted to .env) — needs a changeset.

Agree! The changeset is missing

[isaacroldan]

Actually, this wasn't happening anymore! only the Partners client forced this, and it is unused now. So this PR doesn't really modify that

[isaacroldan]

updateAppIdentifiers , if you look at the implementation, was ignoring calls from deploy. It only persisted identifiers from import-extensionscommand