@loopress/cli 0.32.2 → 0.34.0

package/dist/commands/option/list.js

@@ -79,11 +79,14 @@ const NON_TTY_FALLBACK_COLUMNS = 300;
 // column. See confidenceColor() above for SOURCE?/PLUGIN's shared sûr/pas sûr color coding.
 function renderTable(rows, { includeCore }) {
     const previousColumns = process.env.COLUMNS;
-    if (!process.stdout.columns)
+    // Undefined on a non-TTY stdout, despite its `number` type.
+    const ttyColumns = process.stdout.columns > 0 ? process.stdout.columns : undefined;
+    if (!previousColumns && ttyColumns === undefined)
         process.env.COLUMNS = String(NON_TTY_FALLBACK_COLUMNS);
     // Mirrors tty-table's own getAvailableWidth(): its column-width percentages are computed against
     // this same number, so fillPercent() above needs it too to reserve the right amount for AUTOLOAD/CORE.
-    const width = (process.stdout.columns || Number(process.env.COLUMNS) || NON_TTY_FALLBACK_COLUMNS) - 2;
+    const envColumns = Number(process.env.COLUMNS);
+    const width = (ttyColumns ?? (envColumns > 0 ? envColumns : NON_TTY_FALLBACK_COLUMNS)) - 2;
     const header = [
         { align: 'left', headerAlign: 'left', value: 'NAME', width: columnWidth(COLUMN_WEIGHT.name, width, includeCore) },
         { align: 'left', headerAlign: 'left', value: 'AUTOLOAD' },

package/dist/commands/option/push.js

@@ -1,5 +1,6 @@
 import { Args } from '@oclif/core';
 import { loadFiles } from '../../lib/load-files.js';
+import { errorMessage } from '../../lib/narrow.js';
 import { PushCommand } from '../../lib/push-command.js';
 import { getResourceStateProvider } from '../../lib/resource-state.js';
 import { isNotFoundError } from '../../lib/wp-client.js';
@@ -39,6 +40,7 @@ export default class Push extends PushCommand {
             this.log(`Skipping ${pluralize(skipped.length, 'readonly option')}: ${skipped.map((option) => option.name).join(', ')}`);
         }
         this.log(`Found ${pluralize(writable.length, 'option')} to push`);
+        this.warnEnvironmentUrls(writable.map((option) => ({ id: `options/${option.name}`, value: option.value })));
         const pushed = [];
         await this.runPushTasks(writable, (option) => option.name, async (option, task) => {
             await this.pushOption(option, task);
@@ -89,7 +91,7 @@ export default class Push extends PushCommand {
                 task.output = `Pushed: ${option.name}`;
         }
         catch (error) {
-            this.reportTaskFailure(`Failed to push ${option.name}: ${error.message}`, error, task);
+            this.reportTaskFailure(`Failed to push ${option.name}: ${errorMessage(error)}`, error, task);
         }
     }
 }

package/dist/commands/option/{remove.d.ts → rm.d.ts}

@@ -5,6 +5,7 @@ type RemoveResult = {
     status: 'aborted' | 'dry-run' | 'success';
 };
 export default class Remove extends LoopressCommand {
+    static aliases: string[];
     static args: {
         name: import("@oclif/core/interfaces").Arg<string, Record<string, unknown>>;
     };

package/dist/commands/option/{remove.js → rm.js}

@@ -8,13 +8,14 @@ import { isInteractive } from '../../lib/interactive.js';
 import { isNotFoundError } from '../../lib/wp-client.js';
 import { optionEndpoint, optionFileName } from '../../utils/option-format.js';
 export default class Remove extends LoopressCommand {
+    static aliases = ['option:remove'];
     static args = {
         name: Args.string({ description: 'Option name to stop tracking', required: true }),
     };
     static description = 'Stop tracking an option locally and delete it from WordPress. Local untracking always happens (git-reversible); ' +
         'pass --local-only to skip the WordPress delete and only stop tracking.';
     static enableJsonFlag = true;
-    static examples = ['$ lps option remove wpseo_titles', '$ lps option remove wpseo_titles --local-only'];
+    static examples = ['$ lps option rm wpseo_titles', '$ lps option rm wpseo_titles --local-only'];
     static flags = {
         ...LoopressCommand.dryRunFlag,
         ...LoopressCommand.yesFlag,

package/dist/commands/page/push.js

@@ -1,6 +1,7 @@
 import { confirm } from '@inquirer/prompts';
 import { Args } from '@oclif/core';
 import { isInteractive } from '../../lib/interactive.js';
+import { errorMessage } from '../../lib/narrow.js';
 import { PushCommand } from '../../lib/push-command.js';
 import { getResourceStateProvider } from '../../lib/resource-state.js';
 import { isConflictCode } from '../../lib/wp-client.js';
@@ -34,6 +35,7 @@ export default class Push extends PushCommand {
             this.error(`No page "${args.slug}" in ${path} (expected ${args.slug}.html).`);
         }
         this.log(`Found ${pluralize(selected.length, 'page')} to push`);
+        this.warnEnvironmentUrls(selected.map((page) => ({ id: `pages/${page.slug}`, value: page.html })));
         const provider = getResourceStateProvider('page');
         const beforeState = await this.captureBeforePushState(provider, path, args.slug === undefined ? undefined : (id) => id === args.slug);
         const pushed = [];
@@ -90,7 +92,7 @@ export default class Push extends PushCommand {
                 task?.skip?.(`${page.slug} already exists on the site, not created by Loopress`);
                 return false;
             }
-            this.reportTaskFailure(`Failed to push ${page.slug}: ${error.message}`, error, task);
+            this.reportTaskFailure(`Failed to push ${page.slug}: ${errorMessage(error)}`, error, task);
         }
     }
 }

package/dist/commands/plugin/add.d.ts

@@ -1,5 +1,6 @@
 import { LoopressCommand } from '../../lib/base.js';
 type AddResult = {
+    dependencies: string[];
     slug: string;
     status: 'added' | 'dry-run' | 'unchanged' | 'updated';
     version: string;

package/dist/commands/plugin/add.js

@@ -1,6 +1,7 @@
 import { Args, Flags } from '@oclif/core';
+import got from 'got';
 import { LoopressCommand } from '../../lib/base.js';
-import { pinVersion } from '../../types/plugin.js';
+import { pinActive, pinVersion } from '../../types/plugin.js';
 import { writeLocalConfig } from '../../utils/loopress-config.js';
 import { isExactVersion } from '../../utils/version.js';
 export default class Add extends LoopressCommand {
@@ -26,22 +27,62 @@ export default class Add extends LoopressCommand {
             this.error(`--version must be an exact version like 9.4.2, not a Composer constraint. Got "${version}".`);
         }
         const existing = this.localConfig.plugins ?? {};
+        const required = await requiredPlugins(slug);
+        const dependencies = required.filter((dep) => !Object.hasOwn(existing, dep));
+        // An inactive pin keeps its dependency off, so push could never activate `slug`.
+        const inactive = required.filter((dep) => Object.hasOwn(existing, dep) && !pinActive(existing[dep]));
+        if (inactive.length > 0) {
+            this.warn(`${slug} requires ${inactive.join(', ')}, pinned inactive in loopress.json. Remove \`"active": false\` there, or push cannot activate ${slug}.`);
+        }
         const current = existing[slug];
-        if (current !== undefined && pinVersion(current) === version) {
+        if (current !== undefined && pinVersion(current) === version && dependencies.length === 0) {
             this.log(`${slug} is already pinned to ${version} in loopress.json, nothing to do.`);
-            return { slug, status: 'unchanged', version };
+            return { dependencies, slug, status: 'unchanged', version };
         }
         const isUpdated = current !== undefined;
+        const requires = dependencies.length > 0 ? `, with required ${dependencies.join(', ')} (latest)` : '';
         if (this.dryRun) {
-            this.log(`[dry-run] Would ${isUpdated ? 'update' : 'add'} ${slug} (${version}) in loopress.json`);
-            return { slug, status: 'dry-run', version };
+            this.log(`[dry-run] Would ${isUpdated ? 'update' : 'add'} ${slug} (${version})${requires} in loopress.json`);
+            return { dependencies, slug, status: 'dry-run', version };
         }
         await writeLocalConfig({
             ...this.localConfig,
-            // Re-pinning a plugin recorded as inactive keeps it inactive.
-            plugins: { ...existing, [slug]: typeof current === 'object' ? { ...current, version } : version },
+            plugins: {
+                ...existing,
+                ...Object.fromEntries(dependencies.map((dep) => [dep, 'latest'])),
+                // Re-pinning a plugin recorded as inactive keeps it inactive.
+                [slug]: typeof current === 'object' ? { ...current, version } : version,
+            },
         });
-        this.log(`${isUpdated ? 'Updated' : 'Added'} ${slug} (${version}). Run \`lps plugin push\` to apply.`);
-        return { slug, status: isUpdated ? 'updated' : 'added', version };
+        this.log(`${isUpdated ? 'Updated' : 'Added'} ${slug} (${version})${requires}. Run \`lps plugin push\` to apply.`);
+        return { dependencies, slug, status: isUpdated ? 'updated' : 'added', version };
+    }
+}
+// Every plugin `slug` needs active first, transitively, from the `Requires Plugins` header
+// WordPress.org exposes as `requires_plugins`. Not on WordPress.org or unreachable: none known,
+// and `lps plugin push` still names a missing one when WordPress refuses the activation.
+async function requiredPlugins(slug) {
+    const found = [];
+    const queue = [slug];
+    for (let next = queue.shift(); next !== undefined; next = queue.shift()) {
+        let info;
+        try {
+            info = await got('https://api.wordpress.org/plugins/info/1.2/', {
+                searchParams: { action: 'plugin_information', 'request[slug]': next },
+                timeout: { request: 10_000 },
+            }).json();
+        }
+        catch {
+            continue;
+        }
+        const requires = Array.isArray(info.requires_plugins) ? info.requires_plugins : [];
+        for (const dep of requires) {
+            if (typeof dep !== 'string' || dep === slug || found.includes(dep)) {
+                continue;
+            }
+            found.push(dep);
+            queue.push(dep);
+        }
     }
+    return found;
 }

package/dist/commands/plugin/audit.d.ts

@@ -1,9 +1,9 @@
 import { LoopressCommand } from '../../lib/base.js';
 type Advisory = {
-    cve?: string;
-    fixedIn?: string;
-    score?: number;
-    severity?: string;
+    cve?: string | undefined;
+    fixedIn?: string | undefined;
+    score?: number | undefined;
+    severity?: string | undefined;
     slug: string;
     title: string;
 };

package/dist/commands/plugin/audit.js

@@ -1,5 +1,6 @@
 import got from 'got';
 import { LoopressCommand } from '../../lib/base.js';
+import { errorMessage, isRecord } from '../../lib/narrow.js';
 import { withSpinner } from '../../lib/spinner.js';
 import { pinVersion } from '../../types/plugin.js';
 import { compareVersions } from '../../utils/version.js';
@@ -19,8 +20,8 @@ export default class Audit extends LoopressCommand {
         }
         const advisories = [];
         const health = [];
-        for (const slug of slugs) {
-            const found = await withSpinner(`Auditing ${slug}`, async () => this.auditOne(slug, pinVersion(manifest[slug])));
+        for (const [slug, pin] of Object.entries(manifest)) {
+            const found = await withSpinner(`Auditing ${slug}`, async () => this.auditOne(slug, pinVersion(pin)));
             advisories.push(...found.advisories);
             health.push(...found.health);
         }
@@ -59,11 +60,17 @@ export default class Audit extends LoopressCommand {
         catch (error) {
             // Never swallow this into an empty list: a clean audit must mean "no advisories found",
             // not "the advisory service was unreachable". Fail the command instead.
-            throw new Error(`Could not reach the vulnerability database (wpvulnerability.net) for "${slug}": ${error.message}. ` +
+            throw new Error(`Could not reach the vulnerability database (wpvulnerability.net) for "${slug}": ${errorMessage(error)}. ` +
                 'Audit aborted so a service outage is not reported as a clean result.', { cause: error });
         }
-        const list = Array.isArray(body?.data?.vulnerability) ? body.data.vulnerability : [];
-        return list.map((raw) => normalizeAdvisory(raw));
+        // A 200 with the wrong shape (degraded service) must not read as "no advisories" either.
+        const list = body?.data?.vulnerability;
+        if (!Array.isArray(list) || list.some((raw) => !isRecord(raw))) {
+            throw new TypeError(`Unexpected answer from the vulnerability database (wpvulnerability.net) for "${slug}". ` +
+                'Audit aborted so a malformed response is not reported as a clean result.');
+        }
+        // Already all objects (checked above), the filter only carries that to the type.
+        return list.filter((raw) => isRecord(raw)).map((raw) => normalizeAdvisory(raw));
     }
     report(advisories, health, count) {
         if (advisories.length === 0 && health.length === 0) {
@@ -93,26 +100,28 @@ function advisoryMeta(a) {
     return parts.length > 0 ? ` (${parts.join(', ')})` : '';
 }
 function normalizeAdvisory(raw) {
-    const cvss = (raw.cvss ?? {});
-    const source = Array.isArray(raw.source) ? raw.source : [];
-    const cve = typeof raw.cve === 'string' ? raw.cve : source.find((s) => s.id?.startsWith('CVE-'))?.id;
+    const cvss = isRecord(raw.cvss) ? raw.cvss : {};
+    const sourceIds = Array.isArray(raw.source) ? raw.source.map((s) => (isRecord(s) && typeof s.id === 'string' ? s.id : '')) : [];
+    const cve = typeof raw.cve === 'string' ? raw.cve : sourceIds.find((id) => id.startsWith('CVE-'));
     const range = extractRange(raw);
     return {
         cve,
         fixedIn: range.max?.op === 'lt' ? range.max.version : undefined,
         range,
-        score: cvss.score,
-        severity: cvss.severity,
+        score: typeof cvss.score === 'number' ? cvss.score : undefined,
+        severity: typeof cvss.severity === 'string' ? cvss.severity : undefined,
         title: typeof raw.name === 'string' ? decodeEntities(raw.name) : 'Unnamed advisory',
     };
 }
 // wpvulnerability's `operator`: `max_version` with `max_operator` "lt" (fixed in that version),
 // "le" (affected up to it, no fix yet) or "eq" (that version only), and an optional lower bound `min_version` with
 // `min_operator` "gt" or "ge". No upper bound means every version from the lower one on.
+function toBound(version, operator) {
+    return typeof version === 'string' && version !== '' && typeof operator === 'string' ? { op: operator, version } : undefined;
+}
 function extractRange(raw) {
-    const op = (raw.operator ?? {});
-    const bound = (version, operator) => typeof version === 'string' && version !== '' && typeof operator === 'string' ? { op: operator, version } : undefined;
-    return { max: bound(op.max_version, op.max_operator), min: bound(op.min_version, op.min_operator) };
+    const op = isRecord(raw.operator) ? raw.operator : {};
+    return { max: toBound(op.max_version, op.max_operator), min: toBound(op.min_version, op.min_operator) };
 }
 // A bound we can't evaluate (odd version strings, unknown operator) counts as affected: a false
 // positive is safer than hiding an advisory.
@@ -121,7 +130,8 @@ function affects(version, range) {
         if (!bound)
             return true;
         const order = compareVersions(version, bound.version);
-        return order === null || !Object.hasOwn(ok, bound.op) || ok[bound.op](order);
+        const check = Object.hasOwn(ok, bound.op) ? ok[bound.op] : undefined;
+        return order === null || !check || check(order);
     };
     return (within(range.min, { ge: (o) => o >= 0, gt: (o) => o > 0 }) && within(range.max, { eq: (o) => o === 0, le: (o) => o <= 0, lt: (o) => o < 0 }));
 }
@@ -147,9 +157,19 @@ function describeHealth(slug, pinned, info) {
     }
     const lastUpdated = typeof info.last_updated === 'string' ? info.last_updated : null;
     if (lastUpdated) {
-        const years = (Date.now() - Date.parse(lastUpdated)) / (365 * 24 * 3600 * 1000);
+        const years = (Date.now() - parseWordPressDate(lastUpdated)) / (365 * 24 * 3600 * 1000);
         if (years > 2)
             notes.push({ message: `last updated ${lastUpdated.slice(0, 10)} (possibly abandoned)`, slug });
     }
     return notes;
 }
+// WordPress.org's plugins API returns `last_updated` as "2015-03-04 10:00am GMT", which
+// Date.parse() reads as NaN. Rewrite that form to ISO, and leave anything else to Date.parse().
+function parseWordPressDate(value) {
+    const match = /^(\d{4}-\d{2}-\d{2}) (\d{1,2}):(\d{2})(am|pm) GMT$/i.exec(value);
+    if (!match)
+        return Date.parse(value);
+    const [, day, hour, minute, half] = match;
+    const hours = (Number(hour) % 12) + (half?.toLowerCase() === 'pm' ? 12 : 0);
+    return Date.parse(`${day}T${String(hours).padStart(2, '0')}:${minute}:00Z`);
+}

package/dist/commands/plugin/pull.js

@@ -5,7 +5,8 @@ import { readFile } from 'node:fs/promises';
 import { join } from 'node:path';
 import { LoopressCommand } from '../../lib/base.js';
 import { isInteractive } from '../../lib/interactive.js';
-import { getComposerManagedSlugs, NOT_ON_WORDPRESS_ORG, pullIntoComposerJson, } from '../../utils/composer.js';
+import {} from '../../types/plugin.js';
+import { getComposerManagedSlugs, NOT_ON_WORDPRESS_ORG, parseComposerJson, pullIntoComposerJson, } from '../../utils/composer.js';
 import { writeLocalConfig } from '../../utils/loopress-config.js';
 import { mergePluginManifest, parseInstalledPlugins } from '../../utils/plugins.js';
 export default class Pull extends LoopressCommand {
@@ -29,7 +30,7 @@ export default class Pull extends LoopressCommand {
         const raw = await this.wp.get('wp/v2/plugins');
         const composerJsonPath = join(process.cwd(), this.rootDir, 'composer.json');
         const composerJson = existsSync(composerJsonPath)
-            ? JSON.parse(await readFile(composerJsonPath, 'utf8'))
+            ? parseComposerJson(await readFile(composerJsonPath, 'utf8'))
             : undefined;
         const declared = new Set(composerJson ? getComposerManagedSlugs(composerJson) : Object.keys(this.localConfig.plugins ?? {}));
         const all = parseInstalledPlugins(raw);

package/dist/commands/plugin/push.js

@@ -3,10 +3,11 @@ import { existsSync } from 'node:fs';
 import { readFile } from 'node:fs/promises';
 import { join } from 'node:path';
 import { confirmUninstall } from '../../lib/interactive.js';
+import { causeOf, isRecord } from '../../lib/narrow.js';
 import { PushCommand } from '../../lib/push-command.js';
-import { isLocalEnvironment, isNotFoundError } from '../../lib/wp-client.js';
+import { isLocalEnvironment, isNotFoundError, parseBody, responseOf } from '../../lib/wp-client.js';
 import { pinActive, pinVersion, pinVersions, } from '../../types/plugin.js';
-import { requireToInstall, toIntent, wpackagistRequire } from '../../utils/composer.js';
+import { parseComposerJson, requireToInstall, toIntent, wpackagistRequire } from '../../utils/composer.js';
 import { isDowngrade, parseCollisions, SYNC_TIMEOUT_MS, } from '../../utils/plugin-sync.js';
 import { diffPlugins, lockedWpackagistSlugs, parseInstalledPlugins } from '../../utils/plugins.js';
 import { isExactVersion } from '../../utils/version.js';
@@ -105,10 +106,33 @@ export default class Push extends PushCommand {
             status: 'success',
         });
     }
+    // WordPress refuses to activate a plugin whose `Requires Plugins` are inactive, and loopress.json
+    // has no dependency order (dokan-lite sorts before woocommerce). A refused plugin is retried
+    // after the others, until a round activates nothing.
     async activate(plugins) {
-        for (const plugin of plugins) {
-            this.log(`  ⊙ activating ${plugin.slug}`);
-            await this.wp.put(`wp/v2/plugins/${plugin.file}`, { status: 'active' });
+        let pending = plugins;
+        while (pending.length > 0) {
+            const blocked = [];
+            let missing = [];
+            for (const plugin of pending) {
+                this.log(`  ⊙ activating ${plugin.slug}`);
+                try {
+                    await this.wp.put(`wp/v2/plugins/${plugin.file}`, { status: 'active' });
+                }
+                catch (error) {
+                    const notInstalled = missingDependencies(error);
+                    if (!notInstalled)
+                        throw error;
+                    blocked.push(plugin);
+                    missing = [...missing, ...notInstalled];
+                }
+            }
+            if (blocked.length === pending.length) {
+                const hint = missing.length > 0 ? ` Not installed: ${[...new Set(missing)].join(', ')}.` : '';
+                this.error(`Could not activate ${blocked.map((p) => p.slug).join(', ')}: WordPress reports required plugins missing or inactive.${hint} ` +
+                    'Add them with `lps plugin add <slug>`, then push again.');
+            }
+            pending = blocked;
         }
     }
     async deactivate(plugins) {
@@ -177,7 +201,7 @@ export default class Push extends PushCommand {
     async loadSource(activate) {
         const composerJsonPath = join(process.cwd(), this.rootDir, 'composer.json');
         if (existsSync(composerJsonPath)) {
-            const composerJson = JSON.parse(await readFile(composerJsonPath, 'utf8'));
+            const composerJson = parseComposerJson(await readFile(composerJsonPath, 'utf8'));
             const includeDev = isLocalEnvironment(this.siteConfig.name);
             const manifest = wpackagistRequire(composerJson, 'plugin', includeDev);
             if (Object.keys(manifest).length === 0) {
@@ -232,6 +256,17 @@ export default class Push extends PushCommand {
         }
     }
 }
+// The dependency slugs not installed at all when WordPress refused an activation over unmet
+// `Requires Plugins` (empty when they are only inactive), or null for any other error. The REST
+// controller adds `{status: 500}` as the error data, which pushes activate_plugin()'s own data
+// (`{not_installed, inactive}`) into `additional_data`.
+function missingDependencies(error) {
+    const body = parseBody(responseOf(causeOf(error)).body);
+    if (body.code !== 'plugin_missing_dependencies')
+        return null;
+    const data = [body.data, ...(Array.isArray(body.additional_data) ? body.additional_data : [])];
+    return data.flatMap((d) => (isRecord(d) && isRecord(d.not_installed) ? Object.keys(d.not_installed) : []));
+}
 function isNoop(diff, toPrune) {
     return (diff.toInstall.length === 0 &&
         diff.toPin.length === 0 &&

package/dist/commands/plugin/status.js

@@ -2,6 +2,7 @@ import { existsSync } from 'node:fs';
 import { join } from 'node:path';
 import { LoopressCommand } from '../../lib/base.js';
 import { isNotFoundError } from '../../lib/wp-client.js';
+import {} from '../../types/plugin.js';
 import { diffPlugins, lockedWpackagistSlugs, parseInstalledPlugins } from '../../utils/plugins.js';
 export default class Status extends LoopressCommand {
     static description = 'Compare the plugins on WordPress against loopress.json and report drift (missing, wrong version, inactive, untracked)';
@@ -64,7 +65,8 @@ export default class Status extends LoopressCommand {
             drift,
             inactive,
             missing,
-            status: drift ? 'drift' : 'in-sync',
+            // Drift exits with code 1 above, so only the untracked-active report gets here.
+            status: 'in-sync',
             unexpectedlyActive,
             untrackedActive,
             wrongVersion,

package/dist/commands/project/config.js

@@ -3,12 +3,14 @@ import { Command } from '@oclif/core';
 import { configManager } from '../../config/project-config.manager.js';
 import { bootstrapLoopressFull } from '../../lib/bootstrap-full-install.js';
 import { isInteractive } from '../../lib/interactive.js';
+import { errorMessage } from '../../lib/narrow.js';
 import { isLoopressFullActive } from '../../lib/plugin-detection.js';
 import { withSpinner } from '../../lib/spinner.js';
 import { sweepTempAdmins } from '../../lib/temp-admin.js';
 import { authorizeWithBrowser } from '../../lib/wp-authorize-flow.js';
 import { isLocalEnvironment, WpClient } from '../../lib/wp-client.js';
 import { diagnoseWpSite } from '../../lib/wp-site-diagnostic.js';
+import {} from '../../types/config.js';
 import { pluralize } from '../../utils/pluralize.js';
 const NEW_PROJECT = '__new__';
 const AUTH_BROWSER = 'browser';
@@ -92,7 +94,6 @@ export default class Config extends Command {
         }
         this.log(`✓ "${projectName}/${envName}" configured`);
         await this.maybeBootstrapLoopressFull(env);
-        this.log('→ Run `lps project switch` to change the active project or environment');
     }
     async maybeBootstrapLoopressFull(env) {
         const { name, token, url } = env;
@@ -121,7 +122,7 @@ export default class Config extends Command {
             }));
         }
         catch (error) {
-            this.warn(error.message);
+            this.warn(errorMessage(error));
         }
     }
     async promptManualCredentials() {
@@ -157,7 +158,7 @@ export default class Config extends Command {
             return { appPassword: password, user: userLogin };
         }
         catch (error) {
-            this.warn(`${error.message}\nFalling back to manual credential entry.`);
+            this.warn(`${errorMessage(error)}\nFalling back to manual credential entry.`);
             return this.promptManualCredentials();
         }
     }
@@ -171,10 +172,9 @@ export default class Config extends Command {
                 ],
                 message: 'Project',
             });
-            if (choice !== NEW_PROJECT) {
-                const project = projects.find((p) => p.id === choice);
+            const project = projects.find((p) => p.id === choice);
+            if (project)
                 return { projectId: project.id, projectName: project.name };
-            }
         }
         const existingNames = new Set(projects.map((project) => project.name.trim().toLowerCase()));
         const projectName = await input({
@@ -197,7 +197,7 @@ export default class Config extends Command {
                 this.log(`✓ Removed ${pluralize(removed.length, 'leftover temporary admin account')}: ${removed.join(', ')}`);
         }
         catch (error) {
-            this.warn(error.message);
+            this.warn(errorMessage(error));
         }
     }
 }

package/dist/commands/project/pull.js

@@ -3,7 +3,9 @@ import { Listr } from 'listr2';
 import { authManager } from '../../config/auth.manager.js';
 import { configManager } from '../../config/project-config.manager.js';
 import { ApiClient } from '../../lib/api-client.js';
+import { errorMessage } from '../../lib/narrow.js';
 import { withoutSpinner } from '../../lib/spinner.js';
+import {} from '../../types/config.js';
 import { pluralize } from '../../utils/pluralize.js';
 export default class Pull extends Command {
     static description = 'Pull projects and environments from your Loopress account that are not configured locally yet';
@@ -42,7 +44,7 @@ export default class Pull extends Command {
             return await api.get('projects');
         }
         catch (error) {
-            this.error(`Could not fetch projects from the API: ${error.message}`);
+            this.error(`Could not fetch projects from the API: ${errorMessage(error)}`);
         }
     }
     // Reuse a local project already linked to this API project instead of always minting a new

package/dist/commands/project/push.js

@@ -6,7 +6,9 @@ import { configManager } from '../../config/project-config.manager.js';
 import { ApiClient } from '../../lib/api-client.js';
 import { LoopressCommand } from '../../lib/base.js';
 import { isInteractive } from '../../lib/interactive.js';
+import { errorMessage } from '../../lib/narrow.js';
 import { withoutSpinner } from '../../lib/spinner.js';
+import {} from '../../types/config.js';
 import { pluralize } from '../../utils/pluralize.js';
 import { toSlug } from '../../utils/to-slug.js';
 export default class Push extends Command {
@@ -57,7 +59,7 @@ export default class Push extends Command {
                 task.output = envPlan.action === 'create' ? 'Created on the API' : 'Linked to the API';
         }
         catch (error) {
-            const message = `Failed to push "${envPlan.env.name}": ${error.message}`;
+            const message = `Failed to push "${envPlan.env.name}": ${errorMessage(error)}`;
             if (task)
                 task.output = message;
             throw error;
@@ -94,7 +96,7 @@ export default class Push extends Command {
                 task.output = plan.action === 'create' ? 'Created on the API' : 'Linked to the API';
         }
         catch (error) {
-            const message = `Failed to push project "${plan.project.name}": ${error.message}`;
+            const message = `Failed to push project "${plan.project.name}": ${errorMessage(error)}`;
             if (task)
                 task.output = message;
             throw error;
@@ -138,7 +140,7 @@ export default class Push extends Command {
             return await api.get('projects');
         }
         catch (error) {
-            this.warn(`Could not fetch existing projects from the API, will create everything as new: ${error.message}`);
+            this.warn(`Could not fetch existing projects from the API, will create everything as new: ${errorMessage(error)}`);
             return [];
         }
     }
@@ -203,7 +205,7 @@ export default class Push extends Command {
                     await this.pushCredentials(api, plan.apiProjectId, envPlan.apiEnvironmentId, envPlan.env);
                 }
                 catch (error) {
-                    this.warn(`Failed to push "${plan.project.name}/${envPlan.env.name}": ${error.message}`);
+                    this.warn(`Failed to push "${plan.project.name}/${envPlan.env.name}": ${errorMessage(error)}`);
                 }
             }
         }

package/dist/commands/project/{remove.d.ts → rm.d.ts}

@@ -1,5 +1,6 @@
 import { Command } from '@oclif/core';
 export default class Remove extends Command {
+    static aliases: string[];
     static description: string;
     static examples: string[];
     run(): Promise<void>;

package/dist/commands/project/{remove.js → rm.js}

@@ -2,8 +2,9 @@ import { checkbox } from '@inquirer/prompts';
 import { Command } from '@oclif/core';
 import { configManager } from '../../config/project-config.manager.js';
 export default class Remove extends Command {
+    static aliases = ['project:remove'];
     static description = 'Remove one or more WordPress projects or environments';
-    static examples = ['$ lps project remove'];
+    static examples = ['$ lps project rm'];
     async run() {
         await this.parse(Remove);
         const projects = configManager.listProjects();
@@ -38,7 +39,7 @@ export default class Remove extends Command {
             this.log('Nothing removed.');
             return;
         }
-        const selected = chosen.map((index) => targets[Number(index)]);
+        const selected = chosen.flatMap((index) => targets[Number(index)] ?? []);
         const projectsToRemove = new Map(selected.filter((target) => target.kind === 'project').map((target) => [target.projectId, target.projectName]));
         const envsToRemove = selected.filter((target) => target.kind === 'env' && !projectsToRemove.has(target.projectId));
         for (const projectId of projectsToRemove.keys())

package/dist/commands/project/switch.js

@@ -10,18 +10,20 @@ export default class Switch extends Command {
         if (projects.length === 0) {
             this.error('No projects configured. Run `lps project config` first.');
         }
-        const projectId = projects.length === 1
-            ? projects[0].id
+        const [only] = projects;
+        const currentId = projects.find((project) => project.isCurrent)?.id;
+        const projectId = only && projects.length === 1
+            ? only.id
             : await select({
                 choices: projects.map((project) => ({
                     name: `${project.name}${project.isCurrent ? ' [current]' : ''}`,
                     value: project.id,
                 })),
-                default: projects.find((project) => project.isCurrent)?.id,
+                ...(currentId !== undefined && { default: currentId }),
                 message: 'Select project',
             });
         configManager.setCurrent(projectId);
         const project = projects.find((candidate) => candidate.id === projectId);
-        this.log(`✓ Switched to "${project.name}"`);
+        this.log(`✓ Switched to "${project?.name ?? projectId}"`);
     }
 }

package/dist/commands/promote.d.ts

@@ -14,6 +14,7 @@ export default class Promote extends Command {
     static examples: string[];
     static flags: {
         overwrite: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+        'skip-unsupported': import("@oclif/core/interfaces").BooleanFlag<boolean>;
         yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };