Skip to content

Remove Partners client selection#8083

Open
isaacroldan wants to merge 1 commit into
mainfrom
cleanup/remove-partners-client-selection
Open

Remove Partners client selection#8083
isaacroldan wants to merge 1 commit into
mainfrom
cleanup/remove-partners-client-selection

Conversation

@isaacroldan

Copy link
Copy Markdown
Contributor

What

Remove the remaining Partners client selection path and use App Management as the default developer platform client.

Testing

  • pnpm test packages/app/src/cli/commands/app/init.test.ts packages/app/src/cli/services/context.test.ts packages/app/src/cli/services/dev/fetch.test.ts packages/app/src/cli/services/app/config/link-service.test.ts packages/cli-kit/src/public/node/api/partners.test.ts

Assisted-By: devx/082b4d86-7362-4c41-a55c-65a391c27853
@isaacroldan isaacroldan requested a review from a team as a code owner July 14, 2026 14:34
@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Jul 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

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/private/node/constants.d.ts
@@ -30,7 +30,6 @@ export declare const environmentVariables: {
     otelURL: string;
     themeKitAccessDomain: string;
     json: string;
-    neverUsePartnersApi: string;
     skipNetworkLevelRetry: string;
     maxRequestTimeForNetworkCalls: string;
     disableImportScanning: string;
packages/cli-kit/dist/public/node/environment.d.ts
@@ -45,12 +45,6 @@ export declare function getIdentityTokenInformation(): {
  * @returns True if the JSON output is enabled, false otherwise.
  */
 export declare function jsonOutputEnabled(environment?: NodeJS.ProcessEnv): boolean;
-/**
- * If true, the CLI should not use the Partners API.
- *
- * @returns True when the CLI should not use the Partners API.
- */
-export declare function blockPartnersAccess(): boolean;
 /**
  * If true, the CLI should not use the network level retry.
  *

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant