@loopress/cli 0.29.0 → 0.30.0

package/dist/commands/project/switch.js

@@ -1,9 +1,8 @@
-import { select, Separator } from '@inquirer/prompts';
-import { Command, ux } from '@oclif/core';
+import { select } from '@inquirer/prompts';
+import { Command } from '@oclif/core';
 import { configManager } from '../../config/project-config.manager.js';
-const c = ux.colorize;
 export default class Switch extends Command {
-    static description = 'Switch the active project and environment';
+    static description = 'Switch the active project (pass --env to target a non-local environment)';
     static examples = ['$ lps project switch'];
     async run() {
         await this.parse(Switch);
@@ -11,37 +10,18 @@ export default class Switch extends Command {
         if (projects.length === 0) {
             this.error('No projects configured. Run `lps project config` first.');
         }
-        const { envName, projectId, projectName } = await this.resolveSelection(projects);
-        configManager.setCurrent(projectId, envName);
-        this.log(`✓ Switched to "${projectName}/${envName}"`);
-    }
-    async resolveSelection(projects) {
-        const groups = projects
-            .map((project) => ({ envs: configManager.listEnvironments(project.id), project }))
-            .filter(({ envs }) => envs.length > 0);
-        if (groups.length === 0) {
-            this.error('No environments configured. Run `lps project config` first.');
-        }
-        const entries = groups.flatMap(({ envs, project }) => envs.map((env) => ({ env, project })));
-        if (entries.length === 1) {
-            const { env, project } = entries[0];
-            return { envName: env.name, projectId: project.id, projectName: project.name };
-        }
-        const choices = groups.flatMap(({ envs, project }) => [
-            new Separator(c(project.isCurrent ? 'green' : 'dim', `─── ${project.name} ───`)),
-            ...envs.map((env) => ({
-                name: `${env.name.padEnd(20)} ${env.url}${env.isCurrent ? ' [current]' : ''}`,
-                value: `${project.id}::${env.name}`,
-            })),
-        ]);
-        const current = entries.find(({ env }) => env.isCurrent);
-        const chosen = await select({
-            choices,
-            default: current && `${current.project.id}::${current.env.name}`,
-            message: 'Select project / environment',
-        });
-        const [projectId, envName] = chosen.split('::', 2);
-        const { project } = groups.find((group) => group.project.id === projectId);
-        return { envName, projectId, projectName: project.name };
+        const projectId = projects.length === 1
+            ? projects[0].id
+            : await select({
+                choices: projects.map((project) => ({
+                    name: `${project.name}${project.isCurrent ? ' [current]' : ''}`,
+                    value: project.id,
+                })),
+                default: projects.find((project) => project.isCurrent)?.id,
+                message: 'Select project',
+            });
+        configManager.setCurrent(projectId);
+        const project = projects.find((candidate) => candidate.id === projectId);
+        this.log(`✓ Switched to "${project.name}"`);
     }
 }

package/dist/commands/pull.js

@@ -7,17 +7,21 @@ import { pluralize } from '../utils/pluralize.js';
 const PULL_TARGETS = [
     { commandId: 'composer:pull', label: 'composer' },
     { commandId: 'plugin:pull', label: 'plugins' },
+    { commandId: 'cpt:pull', label: 'post types', supportsYes: true },
+    { commandId: 'taxonomy:pull', label: 'taxonomies', supportsYes: true },
+    { commandId: 'meta:pull', label: 'post meta', supportsYes: true },
+    { commandId: 'role:pull', label: 'roles', supportsYes: true },
     { commandId: 'acf:pull', label: 'ACF', supportsYes: true },
     { commandId: 'api:pull', label: 'API routes', supportsYes: true },
     { commandId: 'hook:pull', label: 'hooks', supportsYes: true },
     { commandId: 'form:pull', label: 'forms', supportsYes: true },
-    { commandId: 'seo:pull', label: 'SEO', supportsYes: true },
+    { commandId: 'pattern:pull', label: 'patterns', supportsYes: true },
     { commandId: 'menu:pull', label: 'menus', supportsYes: true },
     { commandId: 'option:pull', label: 'options', supportsYes: true },
     { commandId: 'snippet:pull', label: 'snippets', supportsYes: true },
 ];
 export default class Pull extends LoopressCommand {
-    static description = 'Pull all content, plugins, composer dependencies, ACF, API routes, hooks, forms, SEO, menus, options, and snippets, from WordPress';
+    static description = 'Pull all content, plugins, composer dependencies, post types, taxonomies, post meta, roles, ACF, API routes, hooks, forms, patterns, menus, options, and snippets, from WordPress';
     static enableJsonFlag = true;
     static examples = ['$ lps pull', '$ lps pull --env staging', '$ lps pull --dry-run'];
     static flags = {
@@ -27,6 +31,7 @@ export default class Pull extends LoopressCommand {
     failedCount = 0;
     async run() {
         const results = [];
+        this.warnRemovedSeoModule();
         for (const target of PULL_TARGETS) {
             this.log(`\n→ Pulling ${target.label}...`);
             try {

package/dist/commands/push.js

@@ -12,20 +12,26 @@ import { pluralize } from '../utils/pluralize.js';
 const PUSH_TARGETS = [
     { commandId: 'plugin:push', label: 'plugins' },
     { commandId: 'composer:push', label: 'composer' },
+    // Before ACF: a field group's location rule can target a Loopress post type or taxonomy.
+    { commandId: 'cpt:push', label: 'post types' },
+    { commandId: 'taxonomy:push', label: 'taxonomies' },
+    { commandId: 'meta:push', label: 'post meta' },
+    { commandId: 'role:push', label: 'roles' },
     { commandId: 'acf:push', label: 'ACF' },
     { commandId: 'api:push', label: 'API routes' },
     { commandId: 'hook:push', label: 'hooks' },
     { commandId: 'form:push', label: 'forms' },
+    // Before templates and pages, which can reference them (`wp:pattern`).
+    { commandId: 'pattern:push', label: 'patterns' },
     // Before pages: WordPress refuses a page whose `template` it doesn't know yet.
     { commandId: 'theme:template:push', label: 'templates and parts' },
     { commandId: 'page:push', label: 'pages' },
-    { commandId: 'seo:push', label: 'SEO' },
     { commandId: 'menu:push', label: 'menus' },
     { commandId: 'option:push', label: 'options' },
     { commandId: 'snippet:push', label: 'snippets' },
 ];
 export default class Push extends LoopressCommand {
-    static description = 'Push all local content, plugins, composer dependencies, ACF, API routes, hooks, forms, templates, pages, SEO, menus, options, and snippets, to WordPress';
+    static description = 'Push all local content, plugins, composer dependencies, post types, taxonomies, post meta, roles, ACF, API routes, hooks, forms, patterns, templates, pages, menus, options, and snippets, to WordPress';
     static enableJsonFlag = true;
     static examples = ['$ lps push', '$ lps push --env staging', '$ lps push --dry-run'];
     static flags = {
@@ -35,6 +41,7 @@ export default class Push extends LoopressCommand {
     failedCount = 0;
     async run() {
         await this.guardProductionPush();
+        this.warnRemovedSeoModule();
         const argv = this.buildArgv();
         const results = [];
         const composerPushedByPlugins = await this.composerDeclaresPlugins();

package/dist/commands/role/diff.d.ts

@@ -0,0 +1,2 @@
+declare const _default: typeof import("../../lib/diff-command.js").DiffCommand;
+export default _default;

package/dist/commands/role/diff.js

@@ -0,0 +1,5 @@
+import { resourceDiffCommand } from '../../lib/diff-command.js';
+export default resourceDiffCommand('role', {
+    description: 'Show what differs in roles between your local roles/ files and a WordPress environment, or between two environments',
+    pathNoun: 'roles directory',
+});

package/dist/commands/role/list.d.ts

@@ -0,0 +1,2 @@
+declare const _default: import("../../lib/php-files-command.js").CommandClass<import("../../lib/declared-type-command.js").DeclaredListCommand>;
+export default _default;

package/dist/commands/role/list.js

@@ -0,0 +1,4 @@
+import { declaredListCommand } from '../../lib/declared-type-command.js';
+import { ROLE_SPEC } from '../../utils/declared-type-format.js';
+import { pluralize } from '../../utils/pluralize.js';
+export default declaredListCommand(ROLE_SPEC, 'List the user roles on WordPress, where each comes from (Loopress, WordPress, a plugin) and how many users have it. Only Loopress ones are pulled and pushed.', (count) => pluralize(count, 'user'));

package/dist/commands/role/pull.d.ts

@@ -0,0 +1,2 @@
+declare const _default: import("../../lib/php-files-command.js").CommandClass<import("../../lib/declared-type-command.js").DeclaredPullCommand>;
+export default _default;

package/dist/commands/role/pull.js

@@ -0,0 +1,4 @@
+import { declaredPullCommand } from '../../lib/declared-type-command.js';
+import { ROLE_SPEC } from '../../utils/declared-type-format.js';
+export default declaredPullCommand(ROLE_SPEC, 'Pull the roles Loopress manages on WordPress into roles/<slug>.json files. ' +
+    'Built-in roles and roles created by a plugin are never pulled, see `lps role list`.');

package/dist/commands/role/push.d.ts

@@ -0,0 +1,2 @@
+declare const _default: import("../../lib/php-files-command.js").CommandClass<import("../../lib/declared-type-command.js").DeclaredPushCommand>;
+export default _default;

package/dist/commands/role/push.js

@@ -0,0 +1,4 @@
+import { declaredPushCommand } from '../../lib/declared-type-command.js';
+import { ROLE_SPEC } from '../../utils/declared-type-format.js';
+export default declaredPushCommand(ROLE_SPEC, 'Push local roles (roles/<slug>.json, a "name" and its "capabilities") to WordPress. ' +
+    'Create or update only, never deletes a role (use `lps role rm`), never changes which users have it.');

package/dist/commands/role/rm.d.ts

@@ -0,0 +1,2 @@
+declare const _default: import("../../lib/php-files-command.js").CommandClass<import("../../lib/declared-type-command.js").DeclaredRmCommand>;
+export default _default;

package/dist/commands/role/rm.js

@@ -0,0 +1,3 @@
+import { declaredRmCommand } from '../../lib/declared-type-command.js';
+import { ROLE_SPEC } from '../../utils/declared-type-format.js';
+export default declaredRmCommand(ROLE_SPEC, 'Stop registering a Loopress role on WordPress. Its users keep it in their profile, with no capability until the role is pushed again.', 'Its users keep it in their profile, with no capability until it is pushed again.');

package/dist/commands/role/rollback.d.ts

@@ -0,0 +1,2 @@
+declare const _default: typeof import("../../lib/base.js").LoopressCommand;
+export default _default;

package/dist/commands/role/rollback.js

@@ -0,0 +1,5 @@
+import { resourceRollbackCommand } from '../../lib/rollback-command.js';
+export default resourceRollbackCommand('role', {
+    description: 'Restore roles to a snapshot taken automatically before an earlier `lps role push`',
+    pathNoun: 'roles directory',
+});

package/dist/commands/status.d.ts

@@ -18,8 +18,7 @@ export default class Status extends Command {
         env: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
     };
     run(): Promise<StatusResult>;
-    private reportActiveProject;
-    private reportEnvOverride;
-    private reportPinnedProject;
+    private logTarget;
+    private report;
 }
 export {};

package/dist/commands/status.js

@@ -11,52 +11,26 @@ export default class Status extends Command {
     };
     async run() {
         const { flags } = await this.parse(Status);
-        const localConfig = await readLocalConfig();
-        let info;
-        if (flags.env) {
-            info = this.reportEnvOverride(localConfig.projectId, flags.env);
-        }
-        else if (localConfig.projectId) {
-            info = this.reportPinnedProject(localConfig.projectId);
-        }
-        else {
-            info = this.reportActiveProject();
-        }
+        const { projectId } = await readLocalConfig();
+        const info = this.report(projectId, flags.env);
         this.log('');
         this.log(`Config dir: ${this.config.configDir}`);
         this.log(`Data dir:   ${this.config.dataDir}`);
         return { ...info, configDir: this.config.configDir, dataDir: this.config.dataDir };
     }
-    reportActiveProject() {
-        const env = configManager.getCurrentEnv();
-        const project = env ? configManager.getCurrentProject() : undefined;
-        if (!env || !project) {
-            const note = 'No project configured. Run `lps project config` first.';
-            this.log(note);
-            return { note };
-        }
-        this.log(`Project:  ${project.name} (${env.name})`);
+    logTarget(label, env) {
+        this.log(`Project:  ${label}`);
         this.log(`URL:      ${env.url}`);
-        return { environment: env.name, project: `${project.name} (${env.name})`, url: env.url };
+        return { environment: env.name, project: label, url: env.url };
     }
-    // Mirrors base.ts:resolveEnvironment with --env: the targeted project comes from
-    // loopress.json when pinned, from the globally active project otherwise.
-    reportEnvOverride(pinnedProjectId, envName) {
-        const project = pinnedProjectId ? configManager.getProject(pinnedProjectId) : configManager.getCurrentProject();
-        if (!project) {
+    // Mirrors base.ts:resolveEnvironment, reporting instead of failing where it can.
+    report(pinnedProjectId, envName) {
+        const projectId = pinnedProjectId ?? configManager.getCurrentProject()?.id;
+        if (!projectId) {
             const note = 'No project configured. Run `lps project config` first.';
             this.log(note);
             return { note };
         }
-        const env = project.environments[envName];
-        if (!env) {
-            this.error(`Environment "${envName}" not found in project "${project.name}". Available: ${Object.keys(project.environments).join(', ')}`);
-        }
-        this.log(`Project:  ${project.name} (${env.name}, via --env)`);
-        this.log(`URL:      ${env.url}`);
-        return { environment: env.name, project: `${project.name} (${env.name}, via --env)`, url: env.url };
-    }
-    reportPinnedProject(projectId) {
         const project = configManager.getProject(projectId);
         if (!project) {
             const note = `loopress.json pins project "${projectId}", but it no longer exists. Run \`lps project config\` to configure it.`;
@@ -71,28 +45,22 @@ export default class Status extends Command {
             this.log(note);
             return { note, project: project.name };
         }
-        if (envNames.length === 1) {
-            const env = project.environments[envNames[0]];
-            this.log(`Project:  ${project.name} (${env.name})`);
-            this.log(`URL:      ${env.url}`);
-            return { environment: env.name, project: `${project.name} (${env.name})`, url: env.url };
+        if (envName) {
+            const env = project.environments[envName];
+            if (!env) {
+                this.error(`Environment "${envName}" not found in project "${project.name}". Available: ${envNames.join(', ')}`);
+            }
+            return this.logTarget(`${project.name} (${env.name}, via --env)`, env);
         }
-        const current = configManager.getCurrentProject();
-        const currentEnv = current?.id === projectId ? configManager.getCurrentEnv() : null;
-        if (!currentEnv) {
-            const note = `"${project.name}" has multiple environments and isn't the globally active project. Run \`lps project switch\` to pick one.`;
-            this.log(`Project:  ${project.name} ${c('yellow', '(ambiguous)')}`);
+        const env = configManager.getDefaultEnvironment(projectId);
+        if (!env) {
+            const note = `"${project.name}" has no "local" environment, pass --env to pick one.`;
+            this.log(`Project:  ${project.name} ${c('yellow', '(no default environment)')}`);
             this.log(`Environments: ${envNames.join(', ')}`);
             this.log('');
-            this.warn(`"${project.name}" has multiple environments and isn't the globally active project.`);
-            this.log('Run `lps project switch` to pick one before running commands here.');
-            if (current) {
-                this.log(`(Globally active project right now: "${current.name}")`);
-            }
+            this.warn(note);
             return { environments: envNames, note, project: project.name };
         }
-        this.log(`Project:  ${project.name} (${currentEnv.name})`);
-        this.log(`URL:      ${currentEnv.url}`);
-        return { environment: currentEnv.name, project: `${project.name} (${currentEnv.name})`, url: currentEnv.url };
+        return this.logTarget(`${project.name} (${env.name})`, env);
     }
 }

package/dist/commands/taxonomy/diff.d.ts

@@ -0,0 +1,2 @@
+declare const _default: typeof import("../../lib/diff-command.js").DiffCommand;
+export default _default;

package/dist/commands/taxonomy/diff.js

@@ -0,0 +1,5 @@
+import { resourceDiffCommand } from '../../lib/diff-command.js';
+export default resourceDiffCommand('taxonomy', {
+    description: 'Show what differs in taxonomies between your local taxonomies/ files and a WordPress environment, or between two environments',
+    pathNoun: 'taxonomies directory',
+});

package/dist/commands/taxonomy/list.d.ts

@@ -0,0 +1,2 @@
+declare const _default: import("../../lib/php-files-command.js").CommandClass<import("../../lib/declared-type-command.js").DeclaredListCommand>;
+export default _default;

package/dist/commands/taxonomy/list.js

@@ -0,0 +1,5 @@
+import { declaredListCommand } from '../../lib/declared-type-command.js';
+import { TAXONOMY_SPEC } from '../../utils/declared-type-format.js';
+import { pluralize } from '../../utils/pluralize.js';
+export default declaredListCommand(TAXONOMY_SPEC, 'List the taxonomies registered on WordPress, where each comes from (Loopress, WordPress, ACF, CPT UI, a theme or plugin) ' +
+    'and the post types it attaches to. Only Loopress ones are pulled and pushed.', (count) => pluralize(count, 'term'));

package/dist/commands/taxonomy/pull.d.ts

@@ -0,0 +1,2 @@
+declare const _default: import("../../lib/php-files-command.js").CommandClass<import("../../lib/declared-type-command.js").DeclaredPullCommand>;
+export default _default;

package/dist/commands/taxonomy/pull.js

@@ -0,0 +1,4 @@
+import { declaredPullCommand } from '../../lib/declared-type-command.js';
+import { TAXONOMY_SPEC } from '../../utils/declared-type-format.js';
+export default declaredPullCommand(TAXONOMY_SPEC, 'Pull the taxonomies Loopress manages on WordPress into taxonomies/<slug>.json files, never their terms. ' +
+    'Taxonomies registered by a theme or another plugin are never pulled, see `lps taxonomy list`.');

package/dist/commands/taxonomy/push.d.ts

@@ -0,0 +1,2 @@
+declare const _default: import("../../lib/php-files-command.js").CommandClass<import("../../lib/declared-type-command.js").DeclaredPushCommand>;
+export default _default;

package/dist/commands/taxonomy/push.js

@@ -0,0 +1,4 @@
+import { declaredPushCommand } from '../../lib/declared-type-command.js';
+import { TAXONOMY_SPEC } from '../../utils/declared-type-format.js';
+export default declaredPushCommand(TAXONOMY_SPEC, 'Push local taxonomies (taxonomies/<slug>.json, register_taxonomy() arguments plus "object_type") to WordPress. ' +
+    'Create or update only, never deletes a taxonomy (use `lps taxonomy rm`), never touches its terms.');

package/dist/commands/taxonomy/rm.d.ts

@@ -0,0 +1,2 @@
+declare const _default: import("../../lib/php-files-command.js").CommandClass<import("../../lib/declared-type-command.js").DeclaredRmCommand>;
+export default _default;

package/dist/commands/taxonomy/rm.js

@@ -0,0 +1,3 @@
+import { declaredRmCommand } from '../../lib/declared-type-command.js';
+import { TAXONOMY_SPEC } from '../../utils/declared-type-format.js';
+export default declaredRmCommand(TAXONOMY_SPEC, 'Stop registering a Loopress taxonomy on WordPress. Its terms are kept in the database, hidden until the taxonomy is pushed again.', "Its terms are kept in the database, hidden until it's pushed again.");

package/dist/commands/taxonomy/rollback.d.ts

@@ -0,0 +1,2 @@
+declare const _default: typeof import("../../lib/base.js").LoopressCommand;
+export default _default;

package/dist/commands/taxonomy/rollback.js

@@ -0,0 +1,5 @@
+import { resourceRollbackCommand } from '../../lib/rollback-command.js';
+export default resourceRollbackCommand('taxonomy', {
+    description: 'Restore taxonomies to a snapshot taken automatically before an earlier `lps taxonomy push`',
+    pathNoun: 'taxonomies directory',
+});

package/dist/commands/theme/template/push.d.ts

@@ -1,11 +1,13 @@
 import { PushCommand } from '../../../lib/push-command.js';
 type PushResult = {
     active?: boolean;
+    blockStyles: string[];
     customized?: string[];
     parts: string[];
     status: 'dry-run' | 'success';
     stylesheet?: string;
     templates: string[];
+    unknownBlocks?: string[];
 };
 export default class Push extends PushCommand {
     static description: string;

package/dist/commands/theme/template/push.js

@@ -4,8 +4,9 @@ import { formatPageProblems } from '../../../utils/page-format.js';
 import { pluralize } from '../../../utils/pluralize.js';
 import { resolveResourceDir } from '../../../utils/resource-dirs.js';
 import { CHILD_THEME_ENDPOINT, readLocalTemplates } from '../../../utils/template-format.js';
+import { bundleThemeCss } from '../../../utils/theme-css.js';
 export default class Push extends PushCommand {
-    static description = 'Push block templates (templates/<slug>.html) and template parts (parts/<slug>.html) as the files of a child theme of the active block theme, <parent>-loopress. The child mirrors the project: files removed locally are removed from it. It is never activated for you.';
+    static description = 'Push block templates (theme/templates/<slug>.html), template parts (theme/parts/<slug>.html) and the theme CSS (theme/css/style.css, theme/blocks/<namespace>/<block>.css, bundled with esbuild) as the files of a child theme of the active block theme, <parent>-loopress. The child mirrors the project: files removed locally are removed from it. It is never activated for you.';
     static enableJsonFlag = true;
     static examples = ['$ lps theme template push', '$ lps theme template push --dry-run'];
     static flags = {
@@ -19,17 +20,24 @@ export default class Push extends PushCommand {
         this.log(`Pushing templates and parts to ${this.siteConfig.url}`);
         this.log(`Templates path: ${templatesPath}, parts path: ${partsPath}`);
         const { parts, problems, templates } = await readLocalTemplates(templatesPath, partsPath);
+        const { css, problems: cssProblems } = await bundleThemeCss(resolveResourceDir('css', this.localConfig), resolveResourceDir('blockCss', this.localConfig));
+        problems.push(...cssProblems);
         if (problems.length > 0) {
             this.error(`Nothing was pushed, fix these files first:\n  ${formatPageProblems(problems)}`);
         }
-        const names = { parts: parts.map((part) => part.slug), templates: templates.map((template) => template.slug) };
-        this.log(`Found ${pluralize(templates.length, 'template')} and ${pluralize(parts.length, 'part')}`);
+        const names = {
+            blockStyles: css.blocks.map((block) => block.name),
+            parts: parts.map((part) => part.slug),
+            templates: templates.map((template) => template.slug),
+        };
+        this.log(`Found ${pluralize(templates.length, 'template')}, ${pluralize(parts.length, 'part')}, ` +
+            `${css.style ? 'a global stylesheet' : 'no global stylesheet'} and ${pluralize(css.blocks.length, 'block stylesheet')}`);
         if (this.dryRun) {
             await this.previewRemote(getResourceStateProvider('template'), templatesPath);
             this.log('[dry-run] Would rewrite the child theme with them.');
             return { ...names, status: 'dry-run' };
         }
-        const child = await this.wp.put(CHILD_THEME_ENDPOINT, { parts, templates });
+        const child = await this.wp.put(CHILD_THEME_ENDPOINT, { ...css, parts, templates });
         await this.recordSuccess();
         this.log(`Child theme ${child.stylesheet} written.`);
         if (!child.active) {
@@ -39,6 +47,19 @@ export default class Push extends PushCommand {
             this.warn(`Edited in the Site Editor, their database copy still wins over the pushed file: ${child.customized.join(', ')}. ` +
                 'Clear their customizations in Appearance > Editor to use the pushed version.');
         }
-        return { ...names, active: child.active, customized: child.customized, status: 'success', stylesheet: child.stylesheet };
+        // A warning, never a refusal: the block may come from a plugin active on another environment.
+        const unknownBlocks = child.unknownBlocks ?? [];
+        if (unknownBlocks.length > 0) {
+            this.warn(`No block registered on this site under these names, their CSS will never load: ${unknownBlocks.join(', ')}. ` +
+                'Check the file name in theme/blocks/, or activate the plugin that provides the block.');
+        }
+        return {
+            ...names,
+            active: child.active,
+            customized: child.customized,
+            status: 'success',
+            stylesheet: child.stylesheet,
+            unknownBlocks,
+        };
     }
 }

package/dist/config/project-config.manager.d.ts

@@ -8,15 +8,15 @@ export declare class ProjectConfigManager {
         id: string;
     });
     getConfigFilePath(): string;
-    getCurrentEnv(): EnvironmentConfig | null;
     getCurrentProject(): null | (ProjectConfig & {
         id: string;
     });
+    getDefaultEnvironment(projectId: string): EnvironmentConfig | null;
     getEnvironment(projectId: string, envName: string): EnvironmentConfig | null;
     getProject(id: string): null | ProjectConfig;
     isTelemetryDisabled(): boolean;
     listEnvironments(projectId: string): Array<EnvironmentConfig & {
-        isCurrent: boolean;
+        isDefault: boolean;
     }>;
     listProjects(): Array<ProjectConfig & {
         id: string;
@@ -26,7 +26,7 @@ export declare class ProjectConfigManager {
     removeEnvironment(projectId: string, envName: string): void;
     removeProject(id: string): void;
     setConfigDir(configDir: string): void;
-    setCurrent(projectId: string, envName: string): void;
+    setCurrent(projectId: string): void;
     setEnvironment(projectId: string, envName: string, env: EnvironmentConfig): void;
     setEnvironmentApiId(projectId: string, envName: string, apiEnvironmentId: string): void;
     setProject(id: string, project: ProjectConfig): void;

package/dist/config/project-config.manager.js

@@ -1,5 +1,6 @@
 import { existsSync, mkdirSync } from 'node:fs';
 import { join } from 'node:path';
+import { LOCAL_ENVIRONMENT_NAME } from '../lib/wp-client.js';
 import { toSlug } from '../utils/to-slug.js';
 import { readJsonFile, writeJsonFileAtomic } from './json-file.js';
 export class ProjectConfigManager {
@@ -39,15 +40,6 @@ export class ProjectConfigManager {
     getConfigFilePath() {
         return join(this.requireConfigDir(), 'config.json');
     }
-    getCurrentEnv() {
-        const config = this.readConfig();
-        if (!config.currentProject)
-            return null;
-        const project = config.projects[config.currentProject.id];
-        if (!project)
-            return null;
-        return project.environments[config.currentProject.env] ?? null;
-    }
     getCurrentProject() {
         const config = this.readConfig();
         if (!config.currentProject)
@@ -57,6 +49,17 @@ export class ProjectConfigManager {
             return null;
         return { ...project, id: config.currentProject.id };
     }
+    // The environment a command targets without --env: "local" when the project has one (where
+    // you work, so it's the safe default), otherwise its only environment. Null when the project
+    // has several remote ones and no local: the user must say which one with --env, since a
+    // remembered choice is exactly the hidden state that sends a push to the wrong site.
+    getDefaultEnvironment(projectId) {
+        const environments = this.getProject(projectId)?.environments ?? {};
+        const names = Object.keys(environments);
+        if (Object.hasOwn(environments, LOCAL_ENVIRONMENT_NAME))
+            return environments[LOCAL_ENVIRONMENT_NAME];
+        return names.length === 1 ? environments[names[0]] : null;
+    }
     getEnvironment(projectId, envName) {
         const project = this.getProject(projectId);
         if (!project)
@@ -71,14 +74,11 @@ export class ProjectConfigManager {
         return this.readConfig().telemetry?.disabled ?? false;
     }
     listEnvironments(projectId) {
-        const config = this.readConfig();
-        const project = config.projects[projectId];
+        const project = this.getProject(projectId);
         if (!project)
             return [];
-        return Object.values(project.environments).map((env) => ({
-            ...env,
-            isCurrent: config.currentProject?.id === projectId && config.currentProject.env === env.name,
-        }));
+        const defaultEnv = this.getDefaultEnvironment(projectId);
+        return Object.values(project.environments).map((env) => ({ ...env, isDefault: env.name === defaultEnv?.name }));
     }
     listProjects() {
         const config = this.readConfig();
@@ -101,10 +101,6 @@ export class ProjectConfigManager {
         if (!project)
             return;
         Reflect.deleteProperty(project.environments, envName);
-        if (config.currentProject?.id === projectId && config.currentProject.env === envName) {
-            const remaining = Object.keys(project.environments);
-            config.currentProject = remaining.length > 0 ? { env: remaining[0], id: projectId } : null;
-        }
         this.writeConfig(config);
     }
     removeProject(id) {
@@ -112,9 +108,7 @@ export class ProjectConfigManager {
         Reflect.deleteProperty(config.projects, id);
         if (config.currentProject?.id === id) {
             const [nextId] = Object.keys(config.projects);
-            const nextProject = nextId ? config.projects[nextId] : undefined;
-            const [nextEnv] = nextProject ? Object.keys(nextProject.environments) : [];
-            config.currentProject = nextId && nextEnv ? { env: nextEnv, id: nextId } : null;
+            config.currentProject = nextId ? { id: nextId } : null;
         }
         this.writeConfig(config);
     }
@@ -124,11 +118,11 @@ export class ProjectConfigManager {
     setConfigDir(configDir) {
         this.configDir = configDir;
     }
-    setCurrent(projectId, envName) {
+    setCurrent(projectId) {
         const config = this.readConfig();
         if (!Object.hasOwn(config.projects, projectId))
             return;
-        config.currentProject = { env: envName, id: projectId };
+        config.currentProject = { id: projectId };
         this.writeConfig(config);
     }
     setEnvironment(projectId, envName, env) {
@@ -137,7 +131,7 @@ export class ProjectConfigManager {
         if (!project)
             return;
         project.environments[envName] = env;
-        config.currentProject ??= { env: envName, id: projectId };
+        config.currentProject ??= { id: projectId };
         this.writeConfig(config);
     }
     setEnvironmentApiId(projectId, envName, apiEnvironmentId) {
@@ -151,11 +145,7 @@ export class ProjectConfigManager {
     setProject(id, project) {
         const config = this.readConfig();
         config.projects[id] = project;
-        if (!config.currentProject) {
-            const [firstEnv] = Object.keys(project.environments);
-            if (firstEnv)
-                config.currentProject = { env: firstEnv, id };
-        }
+        config.currentProject ??= { id };
         this.writeConfig(config);
     }
     setProjectApiId(id, apiProjectId) {
@@ -203,7 +193,8 @@ export class ProjectConfigManager {
         if (value === null || typeof value !== 'object')
             return null;
         const pointer = value;
-        return typeof pointer.id === 'string' && typeof pointer.env === 'string' ? { env: pointer.env, id: pointer.id } : null;
+        // A legacy `env` field, from when switch also picked the environment, is dropped here.
+        return typeof pointer.id === 'string' ? { id: pointer.id } : null;
     }
     sanitizeProjects(value) {
         if (typeof value !== 'object' || value === null)