@loopress/cli 0.18.0 → 0.20.0

package/dist/commands/project/config.js

@@ -1,7 +1,11 @@
 import { confirm, input, password as passwordPrompt, select } from '@inquirer/prompts';
 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 { isLoopressFullActive } from '../../lib/plugin-detection.js';
 import { authorizeWithBrowser } from '../../lib/wp-authorize-flow.js';
+import { WpClient } from '../../lib/wp-client.js';
 import { diagnoseWpSite } from '../../lib/wp-site-diagnostic.js';
 const NEW_PROJECT = '__new__';
 const AUTH_BROWSER = 'browser';
@@ -11,6 +15,9 @@ export default class Config extends Command {
     static examples = ['$ lps project config'];
     async run() {
         await this.parse(Config);
+        if (!isInteractive()) {
+            this.error('lps project config asks its questions interactively and needs a terminal. Configure the project on your machine, then target environments in CI with --env.');
+        }
         const { projectId, projectName } = await this.resolveProject();
         const envChoice = await select({
             choices: [
@@ -74,8 +81,33 @@ export default class Config extends Command {
             configManager.setProject(projectId, project);
         }
         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 { token, url } = env;
+        if (!token)
+            return; // env was just built with one a few lines up; guard only narrows the type
+        const wp = new WpClient(url, token);
+        // A detection failure (network hiccup, plugins REST disabled) is treated as "assume it's
+        // there" rather than surprising the user with an account-creating prompt they didn't ask
+        // for; `lps project config` can simply be re-run if it genuinely isn't installed.
+        const alreadyActive = await isLoopressFullActive(wp).catch(() => true);
+        if (alreadyActive)
+            return;
+        const proceed = await confirm({
+            default: true,
+            message: 'Loopress Full was not detected on this site. Install it now? (creates a temporary admin account, removed automatically afterward)',
+        });
+        if (!proceed)
+            return;
+        try {
+            await bootstrapLoopressFull(wp, url, (message) => this.log(message));
+        }
+        catch (error) {
+            this.warn(error.message);
+        }
+    }
     async promptManualCredentials() {
         const user = await input({
             message: 'Username',

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

@@ -2,9 +2,14 @@ import { Command } from '@oclif/core';
 export default class Push extends Command {
     static description: string;
     static examples: string[];
+    static flags: {
+        yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+    };
+    private yes;
     run(): Promise<void>;
     private applyEnvironment;
     private applyProject;
+    private confirmLink;
     private fetchApiProjects;
     private planEnvironment;
     private planProject;

package/dist/commands/project/push.js

@@ -1,15 +1,22 @@
 import { confirm } from '@inquirer/prompts';
 import { Command } from '@oclif/core';
 import { Listr } from 'listr2';
-import slugify from 'slugify';
 import { authManager } from '../../config/auth.manager.js';
 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 { toSlug } from '../../utils/to-slug.js';
 export default class Push extends Command {
     static description = 'Push locally configured projects, environments and credentials to your Loopress account';
     static examples = ['$ lps project push'];
+    static flags = {
+        ...LoopressCommand.yesFlag,
+    };
+    yes = false;
     async run() {
-        await this.parse(Push);
+        const { flags } = await this.parse(Push);
+        this.yes = Boolean(flags.yes);
         const token = authManager.getAuth()?.token;
         if (!token) {
             this.error('Not logged in. Run `lps login` first.');
@@ -126,6 +133,15 @@ export default class Push extends Command {
             throw error;
         }
     }
+    // Linking to the existing match is the safe default: outside a TTY (or with --yes) it is
+    // taken without prompting, and logged, so CI runs never mint duplicate projects.
+    async confirmLink(message) {
+        if (this.yes || !isInteractive()) {
+            this.log(`${message} Assuming yes (link).`);
+            return true;
+        }
+        return confirm({ default: true, message });
+    }
     async fetchApiProjects(api) {
         try {
             return await api.get('projects');
@@ -142,10 +158,7 @@ export default class Push extends Command {
         }
         const match = apiProject?.environments.find((candidate) => candidate.name === env.name && !claimedEnvironmentIds.has(candidate.id));
         if (match) {
-            const link = await confirm({
-                default: true,
-                message: `Environment "${env.name}" already exists on "${apiProject?.name}". Link to it instead of creating a new one?`,
-            });
+            const link = await this.confirmLink(`Environment "${env.name}" already exists on "${apiProject?.name}". Link to it instead of creating a new one?`);
             if (link) {
                 claimedEnvironmentIds.add(match.id);
                 return { action: 'link', apiEnvironmentId: match.id, env, projectId };
@@ -158,13 +171,10 @@ export default class Push extends Command {
             claimedProjectIds.add(project.apiProjectId);
             return { action: 'synced', apiProjectId: project.apiProjectId, project };
         }
-        const slug = slugify(project.name, { lower: true, strict: true });
+        const slug = toSlug(project.name);
         const match = apiProjects.find((candidate) => candidate.slug === slug && !claimedProjectIds.has(candidate.id));
         if (match) {
-            const link = await confirm({
-                default: true,
-                message: `A project named "${project.name}" already exists on your account. Link to it instead of creating a new one?`,
-            });
+            const link = await this.confirmLink(`A project named "${project.name}" already exists on your account. Link to it instead of creating a new one?`);
             if (link) {
                 claimedProjectIds.add(match.id);
                 return { action: 'link', apiProjectId: match.id, project };

package/dist/commands/project/rotate.d.ts

@@ -0,0 +1,7 @@
+import { LoopressCommand } from '../../lib/base.js';
+export default class Rotate extends LoopressCommand {
+    static description: string;
+    static examples: string[];
+    protected maybeAutoRotate(): Promise<void>;
+    run(): Promise<void>;
+}

package/dist/commands/project/rotate.js

@@ -0,0 +1,20 @@
+import { configManager } from '../../config/project-config.manager.js';
+import { LoopressCommand } from '../../lib/base.js';
+import { rotateAppPassword } from '../../lib/rotate-app-password.js';
+export default class Rotate extends LoopressCommand {
+    static description = 'Rotate the WordPress application password for the current (or --env) environment';
+    static examples = ['$ lps project rotate', '$ lps project rotate --env staging'];
+    // The rotation above always runs, regardless of age; the background age-check in
+    // LoopressCommand would just redo the same work a second time right after.
+    async maybeAutoRotate() { }
+    async run() {
+        const { name, token, url } = this.siteConfig;
+        if (!token) {
+            this.error(`No credentials configured for ${url}. Run \`lps project config\` first.`);
+        }
+        this.log(`Rotating application password for ${url}...`);
+        const rotated = await rotateAppPassword({ ...this.siteConfig, token });
+        configManager.setEnvironment(this.projectId, name, rotated);
+        this.log('✓ New application password created and verified, previous one revoked.');
+    }
+}

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

@@ -1,5 +1,4 @@
 import { LoopressCommand } from '../../lib/base.js';
-import { SeoRedirect } from '../../utils/seo-format.js';
 export default class Pull extends LoopressCommand {
     static args: {
         path: import("@oclif/core/interfaces").Arg<string | undefined, Record<string, unknown>>;
@@ -8,12 +7,11 @@ export default class Pull extends LoopressCommand {
     static examples: string[];
     static flags: {
         'post-type': import("@oclif/core/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
-    private findOrphanedFiles;
     private pullPostMeta;
     private pullRedirects;
     private pullSettings;
 }
-export declare function redirectFileBase(redirect: SeoRedirect): string;

package/dist/commands/seo/pull.js

@@ -1,10 +1,10 @@
 import { Args, Flags } from '@oclif/core';
 import { Listr } from 'listr2';
-import { mkdir, readdir, rm, writeFile } from 'node:fs/promises';
-import { extname, join } from 'node:path';
-import slugify from 'slugify';
+import { mkdir, writeFile } from 'node:fs/promises';
+import { join } from 'node:path';
 import { LoopressCommand } from '../../lib/base.js';
-import { DEFAULT_POST_TYPES, SEO_REDIRECTS_ENDPOINT, SEO_SETTINGS_ENDPOINT, seoPostMetaEndpoint, } from '../../utils/seo-format.js';
+import { basenameKey, findOrphanedFiles, numericPrefixKey } from '../../lib/find-orphaned-files.js';
+import { DEFAULT_POST_TYPES, redirectFileBase, SEO_REDIRECTS_ENDPOINT, SEO_SETTINGS_ENDPOINT, seoPostMetaEndpoint, } from '../../utils/seo-format.js';
 export default class Pull extends LoopressCommand {
     static args = {
         path: Args.string({ description: 'Path to SEO directory (overrides project config)' }),
@@ -13,6 +13,7 @@ export default class Pull extends LoopressCommand {
     static examples = ['$ lps seo pull', '$ lps seo pull --post-type post --post-type page'];
     static flags = {
         ...LoopressCommand.dryRunFlag,
+        ...LoopressCommand.yesFlag,
         'post-type': Flags.string({ description: 'Limit post meta to specific post types', multiple: true }),
     };
     async run() {
@@ -28,33 +29,13 @@ export default class Pull extends LoopressCommand {
         }
         await this.pullRedirects(path);
     }
-    // Shared by post-meta (`<slug>.json`) and redirects (`<id>-<slug>.json`) directories: a local
-    // file whose identity is no longer in the current remote list belongs to something deleted on
-    // WordPress. Left on disk, it would silently come back to life on the next `seo push`.
-    async findOrphanedFiles(dir, keepKeys, numericIdPrefix) {
-        let files;
-        try {
-            files = await readdir(dir);
-        }
-        catch (error) {
-            if (error.code === 'ENOENT')
-                return [];
-            throw error;
-        }
-        return files.filter((file) => {
-            if (extname(file) !== '.json')
-                return false;
-            if (numericIdPrefix) {
-                const match = /^(\d+)-/.exec(file);
-                return match !== null && !keepKeys.has(match[1]);
-            }
-            return !keepKeys.has(file.slice(0, -'.json'.length));
-        });
-    }
     async pullPostMeta(postType, basePath) {
         const dir = join(basePath, 'post-meta', postType);
         const remote = await this.wp.get(seoPostMetaEndpoint(postType));
-        const orphans = await this.findOrphanedFiles(dir, new Set(remote.map((post) => post.slug)), false);
+        const orphans = await findOrphanedFiles(dir, new Set(remote.map((post) => post.slug)), {
+            extensions: ['.json'],
+            key: basenameKey,
+        });
         if (this.dryRun) {
             this.log(`[dry-run] Would pull ${remote.length} ${postType} post-meta file(s) to ${dir}`);
             if (orphans.length > 0) {
@@ -71,11 +52,7 @@ export default class Pull extends LoopressCommand {
             },
             title: `Pull ${post.slug}`,
         }))).run();
-        for (const file of orphans)
-            await rm(join(dir, file), { force: true });
-        if (orphans.length > 0) {
-            this.warn(`Removed ${orphans.length} local file${orphans.length === 1 ? '' : 's'} in ${dir} no longer present on WordPress: ${orphans.join(', ')}`);
-        }
+        await this.removeOrphanedFiles(dir, orphans, `in ${dir} no longer present on WordPress`);
         this.log(`Pulled ${remote.length} ${postType} post-meta file(s) to ${dir}`);
     }
     // Redirects are only supported by some SeoProvider backends (RankMath, not Yoast). Unlike
@@ -92,7 +69,10 @@ export default class Pull extends LoopressCommand {
             this.warn(`Skipping redirects: ${error.message}`);
             return;
         }
-        const orphans = await this.findOrphanedFiles(dir, new Set(remote.map((redirect) => String(redirect.id))), true);
+        const orphans = await findOrphanedFiles(dir, new Set(remote.map((redirect) => String(redirect.id))), {
+            extensions: ['.json'],
+            key: numericPrefixKey,
+        });
         if (this.dryRun) {
             this.log(`[dry-run] Would pull ${remote.length} redirect(s) to ${dir}`);
             if (orphans.length > 0) {
@@ -109,11 +89,7 @@ export default class Pull extends LoopressCommand {
             },
             title: `Pull redirect #${redirect.id}`,
         }))).run();
-        for (const file of orphans)
-            await rm(join(dir, file), { force: true });
-        if (orphans.length > 0) {
-            this.warn(`Removed ${orphans.length} local file${orphans.length === 1 ? '' : 's'} in ${dir} no longer present on WordPress: ${orphans.join(', ')}`);
-        }
+        await this.removeOrphanedFiles(dir, orphans, `in ${dir} no longer present on WordPress`);
         this.log(`Pulled ${remote.length} redirect(s) to ${dir}`);
     }
     async pullSettings(basePath) {
@@ -128,7 +104,3 @@ export default class Pull extends LoopressCommand {
         this.log(`Pulled settings to ${file}`);
     }
 }
-export function redirectFileBase(redirect) {
-    const slug = slugify(redirect.urlTo || 'redirect', { lower: true, strict: true });
-    return `${redirect.id}-${slug || 'redirect'}`;
-}

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

@@ -6,9 +6,9 @@ export default class Push extends PushCommand {
     static description: string;
     static examples: string[];
     static flags: {
+        yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
-    private failedCount;
     run(): Promise<void>;
     private jsonFilesIn;
     private pushPostMeta;

package/dist/commands/seo/push.js

@@ -4,8 +4,7 @@ import { readdir, readFile, rm, writeFile } from 'node:fs/promises';
 import { dirname, extname, join } from 'node:path';
 import { PushCommand } from '../../lib/push-command.js';
 import { isNotFoundError } from '../../lib/wp-client.js';
-import { SEO_REDIRECTS_ENDPOINT, SEO_SETTINGS_ENDPOINT, seoPostMetaEndpoint, seoRedirectEndpoint, } from '../../utils/seo-format.js';
-import { redirectFileBase } from './pull.js';
+import { redirectFileBase, SEO_REDIRECTS_ENDPOINT, SEO_SETTINGS_ENDPOINT, seoPostMetaEndpoint, seoRedirectEndpoint, } from '../../utils/seo-format.js';
 export default class Push extends PushCommand {
     static args = {
         path: Args.string({ description: 'Path to SEO directory (overrides project config)' }),
@@ -14,8 +13,8 @@ export default class Push extends PushCommand {
     static examples = ['$ lps seo push'];
     static flags = {
         ...PushCommand.dryRunFlag,
+        ...PushCommand.yesFlag,
     };
-    failedCount = 0;
     async run() {
         const { args } = await this.parse(Push);
         const { url } = this.siteConfig;
@@ -79,13 +78,7 @@ export default class Push extends PushCommand {
                 task.output = `Pushed: ${post.slug}`;
         }
         catch (error) {
-            const message = `Failed to push ${filePath}: ${error.message}`;
-            if (task)
-                task.output = message;
-            else
-                this.warn(`  ${message}`);
-            this.failedCount++;
-            throw error;
+            this.reportTaskFailure(`Failed to push ${filePath}: ${error.message}`, error, task);
         }
     }
     async pushRedirectFile(filePath, task) {
@@ -117,13 +110,7 @@ export default class Push extends PushCommand {
                 task.output = `Pushed: redirect #${created.id}`;
         }
         catch (error) {
-            const message = `Failed to push ${filePath}: ${error.message}`;
-            if (task)
-                task.output = message;
-            else
-                this.warn(`  ${message}`);
-            this.failedCount++;
-            throw error;
+            this.reportTaskFailure(`Failed to push ${filePath}: ${error.message}`, error, task);
         }
     }
     async pushRedirects(basePath) {

package/dist/commands/snippet/publish.js

@@ -1,11 +1,11 @@
 import { Args, Command } from '@oclif/core';
 import { join } from 'node:path';
-import slugify from 'slugify';
 import { authManager } from '../../config/auth.manager.js';
 import { configManager } from '../../config/project-config.manager.js';
 import { ApiClient } from '../../lib/api-client.js';
 import { loadSnippets } from '../../lib/load-snippets.js';
 import { readLocalConfig } from '../../utils/loopress-config.js';
+import { toSlug } from '../../utils/to-slug.js';
 // Publishes to the Loopress api (not a WordPress site), so this does not extend
 // `LoopressCommand`/`PushCommand`: those force an environment to be resolved, but a project
 // publishing its snippets for sharing doesn't need one, a pure "library" project 
@@ -66,7 +66,7 @@ export default class Publish extends Command {
             // Derived from the snippet's name rather than its on-disk filename: the same
             // slugification `push`/`pull` already use for the canonical `<id>-<slug>` filename
             // convention, so it stays stable even for a not-yet-pushed file with an arbitrary name.
-            slug: slugify(snippet.name, { lower: true, strict: true }),
+            slug: toSlug(snippet.name),
             tags: snippet.tags,
             type: snippet.type,
         };

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

@@ -1,7 +1,4 @@
 import { LoopressCommand } from '../../lib/base.js';
-import { NormalizedSnippet } from '../../utils/snippet-format.js';
-export declare function buildSnippetFile(snippet: NormalizedSnippet): string;
-export declare function buildMetaFile(snippet: NormalizedSnippet): string;
 export default class Pull extends LoopressCommand {
     static args: {
         path: import("@oclif/core/interfaces").Arg<string | undefined, Record<string, unknown>>;
@@ -9,8 +6,8 @@ export default class Pull extends LoopressCommand {
     static description: string;
     static examples: string[];
     static flags: {
+        yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
-    private findOrphanedFiles;
 }

package/dist/commands/snippet/pull.js

@@ -1,10 +1,11 @@
 import { Args } from '@oclif/core';
 import { Listr } from 'listr2';
-import { mkdir, readdir, rm, writeFile } from 'node:fs/promises';
-import { extname, join } from 'node:path';
-import slugify from 'slugify';
+import { mkdir, writeFile } from 'node:fs/promises';
+import { join } from 'node:path';
 import { LoopressCommand } from '../../lib/base.js';
-import { normalizeSnippet, SNIPPETS_ENDPOINT } from '../../utils/snippet-format.js';
+import { findOrphanedFiles, numericPrefixKey } from '../../lib/find-orphaned-files.js';
+import { buildMetaFile, buildSnippetFile, normalizeSnippet, SNIPPETS_ENDPOINT } from '../../utils/snippet-format.js';
+import { toSlug } from '../../utils/to-slug.js';
 const EXTENSIONS = {
     css: 'css',
     html: 'html',
@@ -12,32 +13,6 @@ const EXTENSIONS = {
     php: 'php',
     text: 'txt',
 };
-export function buildSnippetFile(snippet) {
-    if (snippet.type === 'php' && !snippet.code.trimStart().startsWith('<?')) {
-        return `<?php\n\n${snippet.code}`;
-    }
-    return snippet.code;
-}
-export function buildMetaFile(snippet) {
-    const meta = {
-        id: snippet.id,
-        name: snippet.name,
-        type: snippet.type,
-        active: snippet.active,
-        location: snippet.location,
-    };
-    if (snippet.description)
-        meta.description = snippet.description;
-    if (snippet.tags.length > 0)
-        meta.tags = snippet.tags;
-    if (snippet.insertMethod === 'shortcode')
-        meta.insertMethod = snippet.insertMethod;
-    if (snippet.priority !== 10)
-        meta.priority = snippet.priority;
-    if (snippet.shortcodeAttributes.length > 0)
-        meta.shortcodeAttributes = snippet.shortcodeAttributes;
-    return JSON.stringify(meta, null, 2) + '\n';
-}
 export default class Pull extends LoopressCommand {
     static args = {
         path: Args.string({ description: 'Path to snippets directory (overrides project config)' }),
@@ -46,6 +21,7 @@ export default class Pull extends LoopressCommand {
     static examples = ['$ lps snippet pull', '$ lps snippet pull --path ./snippets'];
     static flags = {
         ...LoopressCommand.dryRunFlag,
+        ...LoopressCommand.yesFlag,
     };
     async run() {
         const { args } = await this.parse(Pull);
@@ -60,7 +36,10 @@ export default class Pull extends LoopressCommand {
         // Files following the `<id>-<slug>` convention whose id is no longer in the current
         // remote list belong to a snippet that was deleted on WordPress. Left on disk, they'd
         // silently come back to life the next time `snippet push` runs.
-        const orphans = await this.findOrphanedFiles(path, new Set(pullable.map((snippet) => snippet.id)));
+        const orphans = await findOrphanedFiles(path, new Set(pullable.map((snippet) => String(snippet.id))), {
+            extensions: ['.json', ...Object.values(EXTENSIONS).map((ext) => `.${ext}`)],
+            key: numericPrefixKey,
+        });
         if (this.dryRun) {
             this.log(`[dry-run] Would pull ${snippets.length} snippet${snippets.length === 1 ? '' : 's'} to ${path}`);
             if (orphans.length > 0) {
@@ -72,44 +51,17 @@ export default class Pull extends LoopressCommand {
         await new Listr(pullable.map((snippet) => ({
             async task(_ctx, task) {
                 const ext = EXTENSIONS[snippet.type];
-                const slug = slugify(snippet.name, { lower: true, strict: true });
-                const base = `${snippet.id}-${slug}`;
+                const base = `${snippet.id}-${toSlug(snippet.name)}`;
                 await writeFile(join(path, `${base}.${ext}`), buildSnippetFile(snippet));
                 await writeFile(join(path, `${base}.json`), buildMetaFile(snippet));
                 task.output = `Pulled: ${snippet.name}`;
             },
             title: `Pull ${snippet.name}`,
         }))).run();
-        for (const file of orphans)
-            await rm(join(path, file), { force: true });
-        if (orphans.length > 0) {
-            this.warn(`Removed ${orphans.length} local file${orphans.length === 1 ? '' : 's'} whose snippet no longer exists on WordPress: ${orphans.join(', ')}`);
-        }
+        await this.removeOrphanedFiles(path, orphans, 'whose snippet no longer exists on WordPress');
         this.log(`Pulled ${pullable.length} snippet${pullable.length === 1 ? '' : 's'} to ${path}`);
         if (skipped > 0) {
             this.warn(`${skipped} snippet${skipped === 1 ? '' : 's'} skipped because they have no name`);
         }
     }
-    // Only ever matches files already following the `<id>-<slug>` convention that `snippet
-    // pull`/`push` themselves produce, so a hand-created file without a numeric prefix is
-    // never at risk of being picked up here.
-    async findOrphanedFiles(path, keepIds) {
-        let files;
-        try {
-            files = await readdir(path);
-        }
-        catch (error) {
-            if (error.code === 'ENOENT')
-                return [];
-            throw error;
-        }
-        const knownExtensions = new Set(['json', ...Object.values(EXTENSIONS)]);
-        return files.filter((file) => {
-            const ext = extname(file).slice(1);
-            if (!knownExtensions.has(ext))
-                return false;
-            const match = /^(\d+)-/.exec(file);
-            return match !== null && !keepIds.has(Number(match[1]));
-        });
-    }
 }

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

@@ -6,9 +6,9 @@ export default class Push extends PushCommand {
     static description: string;
     static examples: string[];
     static flags: {
+        yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
-    private failedCount;
     run(): Promise<void>;
     private ensureCanonicalFilename;
     private loadSnippets;

package/dist/commands/snippet/push.js

@@ -2,11 +2,11 @@ import { Args } from '@oclif/core';
 import { Listr } from 'listr2';
 import { readFile, rename, rm, writeFile } from 'node:fs/promises';
 import { basename, dirname, extname, join } from 'node:path';
-import slugify from 'slugify';
 import { loadSnippets as loadSnippetsFromDisk } from '../../lib/load-snippets.js';
 import { PushCommand } from '../../lib/push-command.js';
 import { isNotFoundError } from '../../lib/wp-client.js';
 import { normalizeSnippet, SNIPPETS_ENDPOINT, stripPhpOpeningTag } from '../../utils/snippet-format.js';
+import { toSlug } from '../../utils/to-slug.js';
 export default class Push extends PushCommand {
     static args = {
         path: Args.string({ description: 'Path to snippets directory (overrides project config)' }),
@@ -15,8 +15,8 @@ export default class Push extends PushCommand {
     static examples = ['$ lps snippet push', '$ lps snippet push --path ./snippets'];
     static flags = {
         ...PushCommand.dryRunFlag,
+        ...PushCommand.yesFlag,
     };
-    failedCount = 0;
     async run() {
         const { args } = await this.parse(Push);
         const { url } = this.siteConfig;
@@ -44,7 +44,7 @@ export default class Push extends PushCommand {
         const dir = dirname(snippet.path);
         const ext = extname(snippet.path);
         const currentBase = basename(snippet.path, ext);
-        const canonicalBase = `${id}-${slugify(name, { lower: true, strict: true })}`;
+        const canonicalBase = `${id}-${toSlug(name)}`;
         const oldMetaPath = join(dir, `${currentBase}.json`);
         let meta = {};
         try {
@@ -79,9 +79,6 @@ export default class Push extends PushCommand {
             this.error(error.message);
         }
     }
-    // Throwing on failure (rather than returning a boolean) is what lets Listr mark the task as
-    // failed (red cross) instead of completed; `exitOnError: false` on the task list still lets
-    // sibling snippets push regardless.
     async pushSnippet(snippet, task) {
         if (this.dryRun) {
             if (task)
@@ -114,13 +111,7 @@ export default class Push extends PushCommand {
                 task.output = `Pushed: ${snippet.name}`;
         }
         catch (error) {
-            const message = `Failed to push ${snippet.name}: ${error.message}`;
-            if (task)
-                task.output = message;
-            else
-                this.warn(`  ${message}`);
-            this.failedCount++;
-            throw error;
+            this.reportTaskFailure(`Failed to push ${snippet.name}: ${error.message}`, error, task);
         }
     }
     toPayload(snippet) {

package/dist/commands/status.d.ts

@@ -2,7 +2,11 @@ import { Command } from '@oclif/core';
 export default class Status extends Command {
     static description: string;
     static examples: string[];
+    static flags: {
+        env: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+    };
     run(): Promise<void>;
     private reportActiveProject;
+    private reportEnvOverride;
     private reportPinnedProject;
 }

package/dist/commands/status.js

@@ -1,14 +1,20 @@
-import { Command, ux } from '@oclif/core';
+import { Command, Flags, ux } from '@oclif/core';
 import { configManager } from '../config/project-config.manager.js';
 import { readLocalConfig } from '../utils/loopress-config.js';
 const c = ux.colorize;
 export default class Status extends Command {
     static description = 'Show which WordPress project and environment commands will target';
-    static examples = ['$ lps status'];
+    static examples = ['$ lps status', '$ lps status --env staging'];
+    static flags = {
+        env: Flags.string({ description: 'Show what would be targeted with this environment, as other commands do with --env' }),
+    };
     async run() {
-        await this.parse(Status);
+        const { flags } = await this.parse(Status);
         const localConfig = await readLocalConfig();
-        if (localConfig.projectId) {
+        if (flags.env) {
+            this.reportEnvOverride(localConfig.projectId, flags.env);
+        }
+        else if (localConfig.projectId) {
             this.reportPinnedProject(localConfig.projectId);
         }
         else {
@@ -32,6 +38,21 @@ export default class Status extends Command {
         this.log(`Project:  ${project.name} (${env.name})`);
         this.log(`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) {
+            this.log('No project configured. Run `lps project config` first.');
+            return;
+        }
+        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}`);
+    }
     reportPinnedProject(projectId) {
         const project = configManager.getProject(projectId);
         if (!project) {