Shrink PartnersClient and remove the GraphQL it exclusively used

[isaacroldan]

PartnersClient cleanup — 2 of 3. The keystone. No behavior change on the live migration path.

WHY are these changes introduced?

PartnersClient is only ever instantiated to run the three legacy extension migrations, yet it implements the full DeveloperPlatformClient. Reducing it to a migration-only client lets us delete the large surface of code — methods, GraphQL documents, and shared query consts — that only it used.

The shrink and the deletions are one PR on purpose: flipping PartnersClient to implements MigrationDeveloperPlatformClient orphans all of that code at once, and the Knip (unused-code) check runs per-PR, so the removal has to land in the same PR that orphans it.

WHAT is this pull request doing?

Three commits:

1. Shrink PartnersClient to the migration surface — implement only MigrationDeveloperPlatformClient; delete the dead methods and every import they exclusively pulled in, keeping the migrate mutations plus the session/token/request infrastructure. session() no longer makes a redundant Partners currentAccountInfo round-trip (the migration path only reads the token). Test trimmed to the surviving behavior.
2. Delete orphaned partners-era GraphQL documents — 16 now-unreferenced files (app_active_version, app_version_by_tag, create_app, find_app, find_org, find_org_basic, find_store_by_domain, generate_signed_upload_url, subscribe_to_app_logs, template_specifications, the two functions/*_schema_definition, and the partners/{all-orgs,dev-stores-by-org} query + generated pairs) plus their no-inline-graphql allowlist entries and the matching app.test-data.ts stubs.
3. Trim dead partners-era consts — remove the PartnersClient-only query/mutation consts from shared GraphQL files (app_deploy, app_release, app_versions_diff, get_versions_list, update_urls, all_app_extension_registrations, extension_specifications, and the webhook request-* services), keeping the schema/variable types the App Management client still uses; remove generateFetchAppLogUrl from cli-kit; drop the corresponding allowlist entries.

Net ~1,830 lines removed.

How to test your changes?

pnpm nx type-check app cli-kit, lint, and Knip pass. Migration commands still work — only dead code and a redundant network round-trip were removed.

Checklist

• I've considered possible cross-platform impacts (Mac, Linux, Windows)
• I've considered possible documentation changes
• I've considered analytics changes to measure impact
• The change is user-facing — I've identified the correct bump type (patch for bug fixes · minor for new features · major for breaking changes) and added a changeset with pnpm changeset add

[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

• Shrink PartnersClient and remove the GraphQL it exclusively used #8111 : 2 dependent PRs (#8112 , #8115 ) 👈 (View in Graphite)
• Extract MigrationDeveloperPlatformClient interface #8109 : 1 other dependent PR (#8110 )
• 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/node/api/partners.d.ts

@@ -14,10 +14,6 @@ import { TypedDocumentNode } from '@graphql-typed-document-node/core';
  * @returns The response of the query of generic type <T>.
  */
 export declare function partnersRequest<T>(query: string, token: string, variables?: GraphQLVariables, cacheOptions?: CacheOptions, preferredBehaviour?: RequestModeInput, unauthorizedHandler?: UnauthorizedHandler): Promise<T>;
-export declare const generateFetchAppLogUrl: (cursor?: string, filters?: {
-    status?: string;
-    source?: string;
-}) => Promise<string>;
 /**
  * Executes a GraphQL query against the Partners API. Uses typed documents.
  *