Skip to content

Remove dashboard managed extension support flag#8034

Open
isaacroldan wants to merge 1 commit into
remove-ui-extensions-dev-consolefrom
remove-dashboard-managed-extension-support
Open

Remove dashboard managed extension support flag#8034
isaacroldan wants to merge 1 commit into
remove-ui-extensions-dev-consolefrom
remove-dashboard-managed-extension-support

Conversation

@isaacroldan

@isaacroldan isaacroldan commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Stacked on #8033.

Summary:

  • Remove the supportsDashboardManagedExtensions developer platform capability now that dashboard-managed extensions are supported by default.
  • Preserve the Partner Dashboard managed extension warning/error path for apps with unimported dashboard-managed extensions.
  • Use current force-equivalent flags (--no-release, --allow-updates, --allow-deletes) to avoid prompting and fail with the dashboard-extension error instead.
  • Treat release --force as the equivalent of --allow-updates --allow-deletes when using the shared release confirmation prompt.
  • Stop rendering the deprecated include_config_on_deploy state in the config summary while still removing the field from TOML.

Validation:

  • pnpm vitest run packages/app/src/cli/prompts/deploy-release.test.ts packages/app/src/cli/services/context/identifiers.test.ts packages/app/src/cli/services/deploy.test.ts
  • pnpm --filter app type-check
  • pnpm --filter app lint
  • pnpm knip
  • git diff --check

Copy link
Copy Markdown
Contributor Author

@isaacroldan isaacroldan marked this pull request as ready for review July 8, 2026 10:19
@isaacroldan isaacroldan requested a review from a team as a code owner July 8, 2026 10:19
@isaacroldan isaacroldan force-pushed the remove-dashboard-managed-extension-support branch 2 times, most recently from 59e941e to 4bffb48 Compare July 8, 2026 10:48
Assisted-By: devx/a2560e25-a597-4af7-92ea-24c6fa943415
@isaacroldan isaacroldan force-pushed the remove-dashboard-managed-extension-support branch from 4bffb48 to 6986839 Compare July 8, 2026 10:55
@github-actions

github-actions Bot commented Jul 8, 2026

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
@@ -6,7 +6,6 @@ export declare const environmentVariables: {
     doctor: string;
     enableCliRedirect: string;
     env: string;
-    firstPartyDev: string;
     noAnalytics: string;
     optOutInstrumentation: string;
     appAutomationToken: string;
packages/cli-kit/dist/public/node/context/local.d.ts
@@ -63,13 +63,6 @@ export declare function alwaysLogAnalytics(env?: NodeJS.ProcessEnv): boolean;
  * @returns True if SHOPIFY_CLI_ALWAYS_LOG_METRICS is truthy.
  */
 export declare function alwaysLogMetrics(env?: NodeJS.ProcessEnv): boolean;
-/**
- * Returns true if the CLI User is 1P.
- *
- * @param env - The environment variables from the environment of the current process.
- * @returns True if SHOPIFY_CLI_1P is truthy.
- */
-export declare function firstPartyDev(env?: NodeJS.ProcessEnv): boolean;
 /**
  * Returns true if the CLI can run the "doctor-release" command.
  *

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