@loopress/cli 0.20.0 → 0.21.0

package/dist/commands/page/list.js

@@ -1,26 +1,19 @@
-import { Flags } from '@oclif/core';
 import { LoopressCommand } from '../../lib/base.js';
 import { getPageId, getPageTitle, PAGE_ENDPOINT, PAGE_LIST_QUERY } from '../../utils/page-format.js';
 export default class List extends LoopressCommand {
     static description = 'List pages from WordPress';
+    static enableJsonFlag = true;
     static examples = ['$ lps page list'];
-    static flags = {
-        json: Flags.boolean({ char: 'j', description: 'Output in JSON format' }),
-    };
     async run() {
-        const { flags } = await this.parse(List);
         const pages = await this.wp.get(`${PAGE_ENDPOINT}?${PAGE_LIST_QUERY}`);
-        if (flags.json) {
-            this.log(JSON.stringify(pages, null, 2));
-            return;
-        }
         this.log(`Pages (${pages.length}):`);
         if (pages.length === 0) {
             this.log('  (none)');
-            return;
+            return pages;
         }
         for (const page of pages) {
             this.log(`  ${getPageId(page) ?? '(no id)'}. ${getPageTitle(page)}`);
         }
+        return pages;
     }
 }

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

@@ -1,13 +1,25 @@
 import { LoopressCommand } from '../../lib/base.js';
+type PulledPage = {
+    id: number;
+    title: string;
+};
+type PullResult = {
+    orphans: string[];
+    pulled: PulledPage[];
+    skipped: number;
+    status: 'dry-run' | 'success';
+};
 export default class Pull extends LoopressCommand {
     static args: {
         path: import("@oclif/core/interfaces").Arg<string | undefined, Record<string, unknown>>;
     };
     static description: string;
+    static enableJsonFlag: boolean;
     static examples: string[];
     static flags: {
         yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
-    run(): Promise<void>;
+    run(): Promise<PullResult>;
 }
+export {};

package/dist/commands/page/pull.js

@@ -10,6 +10,7 @@ export default class Pull extends LoopressCommand {
         path: Args.string({ description: 'Path to pages directory (overrides project config)' }),
     };
     static description = 'Pull pages from WordPress';
+    static enableJsonFlag = true;
     static examples = ['$ lps page pull'];
     static flags = {
         ...LoopressCommand.dryRunFlag,
@@ -32,12 +33,13 @@ export default class Pull extends LoopressCommand {
             extensions: ['.json', '.html'],
             key: numericPrefixKey,
         });
+        const pulled = withId.map((page) => ({ id: getPageId(page), title: getPageTitle(page) }));
         if (this.dryRun) {
             this.log(`[dry-run] Would pull ${withId.length} page${withId.length === 1 ? '' : 's'} to ${path}`);
             if (orphans.length > 0) {
                 this.log(`[dry-run] Would remove ${orphans.length} local file${orphans.length === 1 ? '' : 's'} in ${path} no longer present on WordPress: ${orphans.join(', ')}`);
             }
-            return;
+            return { orphans, pulled, skipped, status: 'dry-run' };
         }
         if (withId.length > 0)
             await mkdir(path, { recursive: true });
@@ -55,11 +57,12 @@ export default class Pull extends LoopressCommand {
                 },
                 title: `Pull ${title}`,
             };
-        })).run();
+        }), { renderer: this.jsonEnabled() ? 'silent' : 'default' }).run();
         await this.removeOrphanedFiles(path, orphans, `in ${path} no longer present on WordPress`);
         this.log(`Pulled ${withId.length} page${withId.length === 1 ? '' : 's'} to ${path}`);
         if (skipped > 0) {
             this.warn(`${skipped} page${skipped === 1 ? '' : 's'} skipped because they have no id`);
         }
+        return { orphans, pulled, skipped, status: 'success' };
     }
 }

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

@@ -1,16 +1,26 @@
 import { PushCommand } from '../../lib/push-command.js';
+type PushedPage = {
+    id: null | number;
+    title: string;
+};
+type PushResult = {
+    pushed: PushedPage[];
+    status: 'dry-run' | 'success';
+};
 export default class Push extends PushCommand {
     static args: {
         path: import("@oclif/core/interfaces").Arg<string | undefined, Record<string, unknown>>;
     };
     static description: string;
+    static enableJsonFlag: boolean;
     static examples: string[];
     static flags: {
         yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
-    run(): Promise<void>;
+    run(): Promise<PushResult>;
     private ensureCanonicalFilename;
     private loadFiles;
     private pushPage;
 }
+export {};

package/dist/commands/page/push.js

@@ -10,6 +10,7 @@ export default class Push extends PushCommand {
         path: Args.string({ description: 'Path to pages directory (overrides project config)' }),
     };
     static description = 'Push pages to WordPress. Local files created or updated remotely are renamed on disk to the `<id>-<slug>` convention.';
+    static enableJsonFlag = true;
     static examples = ['$ lps page push'];
     static flags = {
         ...PushCommand.dryRunFlag,
@@ -23,17 +24,22 @@ export default class Push extends PushCommand {
         this.log(`Pages path: ${path}`);
         const pages = await this.loadFiles(path);
         this.log(`Found ${pages.length} page${pages.length === 1 ? '' : 's'} to push`);
+        const pushed = [];
         await new Listr(pages.map((page) => ({
-            task: async (_ctx, task) => this.pushPage(page, task),
+            task: async (_ctx, task) => {
+                const id = await this.pushPage(page, task);
+                pushed.push({ id, title: getPageTitle(page.meta) });
+            },
             title: `Push ${getPageTitle(page.meta)}`,
-        })), { concurrent: false, exitOnError: false }).run();
+        })), { concurrent: false, exitOnError: false, renderer: this.jsonEnabled() ? 'silent' : 'default' }).run();
         if (this.failedCount > 0) {
             this.error(`${this.failedCount} page${this.failedCount === 1 ? '' : 's'} failed to push.`);
         }
         if (this.dryRun)
-            return;
+            return { pushed, status: 'dry-run' };
         await this.recordSuccess();
         this.log('All pages pushed.');
+        return { pushed, status: 'success' };
     }
     // Renames the local file pair to the `<id>-<slug>` convention used by `page pull` whenever it
     // doesn't already match, same principle as ensureCanonicalFilename in commands/snippet/push.ts:
@@ -108,16 +114,16 @@ export default class Push extends PushCommand {
         if (this.dryRun) {
             if (task)
                 task.output = `[dry-run] Would push: ${title}`;
-            return;
+            return getPageId(page.meta);
         }
         try {
-            const id = getPageId(page.meta);
+            let id = getPageId(page.meta);
             const payload = { ...page.meta, content: page.content };
             if (id === null) {
                 const created = await this.wp.post(PAGE_ENDPOINT, payload);
-                const newId = getPageId(created);
-                if (newId !== null)
-                    await this.ensureCanonicalFilename(page, newId, title);
+                id = getPageId(created);
+                if (id !== null)
+                    await this.ensureCanonicalFilename(page, id, title);
             }
             else {
                 try {
@@ -126,17 +132,23 @@ export default class Push extends PushCommand {
                 }
                 catch (error) {
                     // The id recorded locally doesn't exist on this site (e.g. a fresh install): create
-                    // it instead of failing, and adopt whatever id the site assigns.
+                    // it instead of failing, and adopt whatever id the site assigns. WordPress core
+                    // rejects any POST that still carries a (now stale) `id` field with a 400 "Cannot
+                    // create existing post", regardless of whether that id actually exists, so it must
+                    // be stripped before falling back to create.
                     if (!isNotFoundError(error))
                         throw error;
-                    const created = await this.wp.post(PAGE_ENDPOINT, payload);
-                    const newId = getPageId(created);
-                    if (newId !== null)
-                        await this.ensureCanonicalFilename(page, newId, title);
+                    const newPayload = { ...payload };
+                    delete newPayload.id;
+                    const created = await this.wp.post(PAGE_ENDPOINT, newPayload);
+                    id = getPageId(created);
+                    if (id !== null)
+                        await this.ensureCanonicalFilename(page, id, title);
                 }
             }
             if (task)
                 task.output = `Pushed: ${title}`;
+            return id;
         }
         catch (error) {
             this.reportTaskFailure(`Failed to push ${title}: ${error.message}`, error, task);

package/dist/commands/plugin/add.js

@@ -18,15 +18,15 @@ export default class Add extends LoopressCommand {
             this.log(`${slug} is already in loopress.json, nothing to do.`);
             return;
         }
-        const updated = existing[slug] !== undefined;
+        const isUpdated = existing[slug] !== undefined;
         if (this.dryRun) {
-            this.log(`[dry-run] Would ${updated ? 'update' : 'add'} ${slug} in loopress.json`);
+            this.log(`[dry-run] Would ${isUpdated ? 'update' : 'add'} ${slug} in loopress.json`);
             return;
         }
         await writeLocalConfig({
             ...this.localConfig,
             plugins: { ...existing, [slug]: 'latest' },
         });
-        this.log(`${updated ? 'Updated' : 'Added'} ${slug}`);
+        this.log(`${isUpdated ? 'Updated' : 'Added'} ${slug}`);
     }
 }

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

@@ -1,9 +1,15 @@
 import { LoopressCommand } from '../../lib/base.js';
+import { type MergeResult } from '../../utils/plugins.js';
+type PullResult = MergeResult & {
+    status: 'dry-run' | 'success';
+};
 export default class Pull extends LoopressCommand {
     static description: string;
+    static enableJsonFlag: boolean;
     static examples: string[];
     static flags: {
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
-    run(): Promise<void>;
+    run(): Promise<PullResult>;
 }
+export {};

package/dist/commands/plugin/pull.js

@@ -4,6 +4,7 @@ import { writeLocalConfig } from '../../utils/loopress-config.js';
 import { mergePluginManifest, parseInstalledPlugins } from '../../utils/plugins.js';
 export default class Pull extends LoopressCommand {
     static description = 'Pull installed plugins from WordPress into loopress.json';
+    static enableJsonFlag = true;
     static examples = ['$ lps plugin pull', '$ lps plugin pull --dry-run'];
     static flags = {
         ...LoopressCommand.dryRunFlag,
@@ -31,7 +32,7 @@ export default class Pull extends LoopressCommand {
                 for (const u of updated)
                     this.log(`  ~ ${u.slug} (${u.from} → ${u.to})`);
             }
-            return;
+            return { added, merged, status: 'dry-run', updated };
         }
         await writeLocalConfig({ ...this.localConfig, plugins: merged });
         this.log(`Wrote ${Object.keys(merged).length} plugins to loopress.json`);
@@ -39,5 +40,6 @@ export default class Pull extends LoopressCommand {
             this.log(`  + Added: ${added.join(', ')}`);
         for (const u of updated)
             this.log(`  ~ Updated: ${u.slug} ${u.from} → ${u.to}`);
+        return { added, merged, status: 'success', updated };
     }
 }

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

@@ -1,13 +1,21 @@
 import { PushCommand } from '../../lib/push-command.js';
+type PushResult = {
+    activated: string[];
+    installed: string[];
+    skippedComposerManaged: string[];
+    status: 'dry-run' | 'in-sync' | 'success';
+};
 export default class Push extends PushCommand {
     static description: string;
+    static enableJsonFlag: boolean;
     static examples: string[];
     static flags: {
         yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
-    run(): Promise<void>;
+    run(): Promise<PushResult>;
     private activatePlugin;
     private installPlugin;
     private performPluginAction;
 }
+export {};

package/dist/commands/plugin/push.js

@@ -4,6 +4,7 @@ import { getComposerManagedSlugs, readComposerJson } from '../../utils/composer.
 import { diffPlugins, parseInstalledPlugins } from '../../utils/plugins.js';
 export default class Push extends PushCommand {
     static description = 'Push plugins to WordPress to match loopress.json';
+    static enableJsonFlag = true;
     static examples = ['$ lps plugin push', '$ lps plugin push --dry-run'];
     static flags = {
         ...PushCommand.dryRunFlag,
@@ -18,9 +19,9 @@ export default class Push extends PushCommand {
         const composerJson = await readComposerJson();
         const composerSlugs = composerJson ? getComposerManagedSlugs(composerJson) : [];
         const filteredManifest = Object.fromEntries(Object.entries(manifest).filter(([slug]) => !composerSlugs.includes(slug)));
-        const skipped = composerSlugs.filter((slug) => slug in manifest);
-        if (skipped.length > 0) {
-            this.log(`Skipping ${skipped.length} Composer-managed ${skipped.length === 1 ? 'plugin' : 'plugins'}: ${skipped.join(', ')}`);
+        const skippedComposerManaged = composerSlugs.filter((slug) => Object.hasOwn(manifest, slug));
+        if (skippedComposerManaged.length > 0) {
+            this.log(`Skipping ${skippedComposerManaged.length} Composer-managed ${skippedComposerManaged.length === 1 ? 'plugin' : 'plugins'}: ${skippedComposerManaged.join(', ')}`);
             this.log('Run `lps composer push` to deploy them.');
         }
         this.log(`Pushing plugins to ${url}`);
@@ -29,7 +30,7 @@ export default class Push extends PushCommand {
         const { toActivate, toInstall } = diffPlugins(filteredManifest, installed);
         if (toInstall.length === 0 && toActivate.length === 0) {
             this.log('Everything is already in sync.');
-            return;
+            return { activated: [], installed: [], skippedComposerManaged, status: 'in-sync' };
         }
         if (toInstall.length > 0) {
             this.log(`\nTo install (${toInstall.length}):`);
@@ -41,32 +42,35 @@ export default class Push extends PushCommand {
             for (const a of toActivate)
                 this.log(`  ↑ ${a.slug}`);
         }
+        const installedSlugs = toInstall.map((a) => a.slug);
+        const activatedSlugs = toActivate.map((a) => a.slug);
         if (this.dryRun)
-            return;
+            return { activated: activatedSlugs, installed: installedSlugs, skippedComposerManaged, status: 'dry-run' };
         // Installing with `status: active` activates in the same call, so installs never need a
         // separate activation step the way the old custom endpoint's two-step flow did.
         if (toInstall.length > 0) {
             await new Listr(toInstall.map((action) => ({
                 task: async (_ctx, task) => this.installPlugin(action.slug, task),
                 title: `Install ${action.slug}`,
-            })), { concurrent: false, exitOnError: false }).run();
+            })), { concurrent: false, exitOnError: false, renderer: this.jsonEnabled() ? 'silent' : 'default' }).run();
         }
         if (toActivate.length > 0) {
             await new Listr(toActivate.map((action) => ({
                 task: async (_ctx, task) => this.activatePlugin(action.file, action.slug, task),
                 title: `Activate ${action.slug}`,
-            })), { concurrent: false, exitOnError: false }).run();
+            })), { concurrent: false, exitOnError: false, renderer: this.jsonEnabled() ? 'silent' : 'default' }).run();
         }
         if (this.failedCount > 0) {
             this.error(`${this.failedCount} plugin${this.failedCount === 1 ? '' : 's'} failed to install or activate.`);
         }
         await this.recordSuccess();
+        return { activated: activatedSlugs, installed: installedSlugs, skippedComposerManaged, status: 'success' };
     }
     async activatePlugin(file, slug, task) {
-        await this.performPluginAction('activate', slug, () => this.wp.put(`wp/v2/plugins/${file}`, { status: 'active' }), task);
+        await this.performPluginAction('activate', slug, async () => this.wp.put(`wp/v2/plugins/${file}`, { status: 'active' }), task);
     }
     async installPlugin(slug, task) {
-        await this.performPluginAction('install', slug, () => this.wp.post('wp/v2/plugins', { slug, status: 'active' }), task);
+        await this.performPluginAction('install', slug, async () => this.wp.post('wp/v2/plugins', { slug, status: 'active' }), task);
     }
     async performPluginAction(verb, slug, request, task) {
         try {

package/dist/commands/project/config.js

@@ -36,11 +36,11 @@ export default class Config extends Command {
             : envChoice;
         const existingEnv = configManager.getEnvironment(projectId, envName);
         if (existingEnv) {
-            const overwrite = await confirm({
+            const isOverwrite = await confirm({
                 default: false,
                 message: `"${projectName}/${envName}" already exists. Overwrite?`,
             });
-            if (!overwrite) {
+            if (!isOverwrite) {
                 this.log('Aborted.');
                 return;
             }
@@ -92,17 +92,17 @@ export default class Config extends Command {
         // 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)
+        const isAlreadyActive = await isLoopressFullActive(wp).catch(() => true);
+        if (isAlreadyActive)
             return;
-        const proceed = await confirm({
+        const isProceed = 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)
+        if (!isProceed)
             return;
         try {
-            await bootstrapLoopressFull(wp, url, (message) => this.log(message));
+            await bootstrapLoopressFull(wp, url, (message) => { this.log(message); });
         }
         catch (error) {
             this.warn(error.message);
@@ -137,7 +137,7 @@ export default class Config extends Command {
             return this.promptManualCredentials();
         }
         try {
-            const { password, userLogin } = await authorizeWithBrowser(url, (message) => this.log(message));
+            const { password, userLogin } = await authorizeWithBrowser(url, (message) => { this.log(message); });
             return { appPassword: password, user: userLogin };
         }
         catch (error) {

package/dist/commands/project/list.js

@@ -18,8 +18,8 @@ export default class List extends Command {
             const currentTag = project.isCurrent ? ` ${c('green', '[current]')}` : '';
             this.log(`${marker} ${name}${currentTag}`);
             for (const env of envs) {
-                const envMarker = env.isCurrent ? c('cyan', '·') : c('dim', '·');
-                const envName = env.isCurrent ? c('cyan', env.name.padEnd(15)) : c('dim', env.name.padEnd(15));
+                const envMarker = c(env.isCurrent ? 'cyan' : 'dim', '·');
+                const envName = c(env.isCurrent ? 'cyan' : 'dim', env.name.padEnd(15));
                 const envUrl = c('dim', env.url);
                 const activeTag = env.isCurrent ? ` ${c('cyan', '←')}` : '';
                 this.log(`  ${envMarker} ${envName} ${envUrl}${activeTag}`);

package/dist/commands/project/push.js

@@ -16,7 +16,7 @@ export default class Push extends Command {
     yes = false;
     async run() {
         const { flags } = await this.parse(Push);
-        this.yes = Boolean(flags.yes);
+        this.yes = flags.yes;
         const token = authManager.getAuth()?.token;
         if (!token) {
             this.error('Not logged in. Run `lps login` first.');
@@ -158,8 +158,8 @@ 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 this.confirmLink(`Environment "${env.name}" already exists on "${apiProject?.name}". Link to it instead of creating a new one?`);
-            if (link) {
+            const isLink = await this.confirmLink(`Environment "${env.name}" already exists on "${apiProject?.name}". Link to it instead of creating a new one?`);
+            if (isLink) {
                 claimedEnvironmentIds.add(match.id);
                 return { action: 'link', apiEnvironmentId: match.id, env, projectId };
             }
@@ -174,8 +174,8 @@ export default class Push extends Command {
         const slug = toSlug(project.name);
         const match = apiProjects.find((candidate) => candidate.slug === slug && !claimedProjectIds.has(candidate.id));
         if (match) {
-            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) {
+            const isLink = await this.confirmLink(`A project named "${project.name}" already exists on your account. Link to it instead of creating a new one?`);
+            if (isLink) {
                 claimedProjectIds.add(match.id);
                 return { action: 'link', apiProjectId: match.id, project };
             }

package/dist/commands/project/switch.js

@@ -28,7 +28,7 @@ export default class Switch extends Command {
             return { envName: env.name, projectId: project.id, projectName: project.name };
         }
         const choices = groups.flatMap(({ envs, project }) => [
-            new Separator(project.isCurrent ? c('green', `─── ${project.name} ───`) : c('dim', `─── ${project.name} ───`)),
+            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}`,
@@ -40,7 +40,7 @@ export default class Switch extends Command {
             default: current && `${current.project.id}::${current.env.name}`,
             message: 'Select project / environment',
         });
-        const [projectId, envName] = chosen.split('::');
+        const [projectId, envName] = chosen.split('::', 2);
         const { project } = groups.find((group) => group.project.id === projectId);
         return { envName, projectId, projectName: project.name };
     }

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

@@ -1,9 +1,8 @@
 import { LoopressCommand } from '../../lib/base.js';
+import { type NormalizedSnippet } from '../../utils/snippet-format.js';
 export default class List extends LoopressCommand {
     static description: string;
+    static enableJsonFlag: boolean;
     static examples: string[];
-    static flags: {
-        json: import("@oclif/core/interfaces").BooleanFlag<boolean>;
-    };
-    run(): Promise<void>;
+    run(): Promise<NormalizedSnippet[]>;
 }

package/dist/commands/snippet/list.js

@@ -1,23 +1,15 @@
-import { Flags } from '@oclif/core';
 import { LoopressCommand } from '../../lib/base.js';
 import { normalizeSnippet, SNIPPETS_ENDPOINT } from '../../utils/snippet-format.js';
 export default class List extends LoopressCommand {
     static description = 'List snippets from WordPress';
+    static enableJsonFlag = true;
     static examples = ['$ lps snippet list'];
-    static flags = {
-        json: Flags.boolean({ char: 'j', description: 'Output in JSON format' }),
-    };
     async run() {
-        const { flags } = await this.parse(List);
         const remoteList = await this.wp.get(SNIPPETS_ENDPOINT);
         const snippets = remoteList.map((r) => normalizeSnippet(r));
-        if (flags.json) {
-            this.log(JSON.stringify(snippets, null, 2));
-            return;
-        }
         if (snippets.length === 0) {
             this.log('No snippets found');
-            return;
+            return snippets;
         }
         this.log(`Found ${snippets.length} snippet${snippets.length === 1 ? '' : 's'}:`);
         this.log('');
@@ -32,5 +24,6 @@ export default class List extends LoopressCommand {
             }
             this.log('');
         }
+        return snippets;
     }
 }

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

@@ -1,13 +1,25 @@
 import { LoopressCommand } from '../../lib/base.js';
+type PulledSnippet = {
+    id: number;
+    name: string;
+};
+type PullResult = {
+    orphans: string[];
+    pulled: PulledSnippet[];
+    skipped: number;
+    status: 'dry-run' | 'success';
+};
 export default class Pull extends LoopressCommand {
     static args: {
         path: import("@oclif/core/interfaces").Arg<string | undefined, Record<string, unknown>>;
     };
     static description: string;
+    static enableJsonFlag: boolean;
     static examples: string[];
     static flags: {
         yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
-    run(): Promise<void>;
+    run(): Promise<PullResult>;
 }
+export {};

package/dist/commands/snippet/pull.js

@@ -18,6 +18,7 @@ export default class Pull extends LoopressCommand {
         path: Args.string({ description: 'Path to snippets directory (overrides project config)' }),
     };
     static description = 'Pull snippets from WordPress';
+    static enableJsonFlag = true;
     static examples = ['$ lps snippet pull', '$ lps snippet pull --path ./snippets'];
     static flags = {
         ...LoopressCommand.dryRunFlag,
@@ -40,12 +41,13 @@ export default class Pull extends LoopressCommand {
             extensions: ['.json', ...Object.values(EXTENSIONS).map((ext) => `.${ext}`)],
             key: numericPrefixKey,
         });
+        const pulled = pullable.map((snippet) => ({ id: snippet.id, name: snippet.name }));
         if (this.dryRun) {
             this.log(`[dry-run] Would pull ${snippets.length} snippet${snippets.length === 1 ? '' : 's'} to ${path}`);
             if (orphans.length > 0) {
                 this.log(`[dry-run] Would remove ${orphans.length} local file${orphans.length === 1 ? '' : 's'} whose snippet no longer exists on WordPress: ${orphans.join(', ')}`);
             }
-            return;
+            return { orphans, pulled, skipped, status: 'dry-run' };
         }
         await mkdir(path, { recursive: true });
         await new Listr(pullable.map((snippet) => ({
@@ -57,11 +59,12 @@ export default class Pull extends LoopressCommand {
                 task.output = `Pulled: ${snippet.name}`;
             },
             title: `Pull ${snippet.name}`,
-        }))).run();
+        })), { renderer: this.jsonEnabled() ? 'silent' : 'default' }).run();
         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`);
         }
+        return { orphans, pulled, skipped, status: 'success' };
     }
 }

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

@@ -1,17 +1,27 @@
 import { PushCommand } from '../../lib/push-command.js';
+type PushedSnippet = {
+    id?: number;
+    name: string;
+};
+type PushResult = {
+    pushed: PushedSnippet[];
+    status: 'dry-run' | 'success';
+};
 export default class Push extends PushCommand {
     static args: {
         path: import("@oclif/core/interfaces").Arg<string | undefined, Record<string, unknown>>;
     };
     static description: string;
+    static enableJsonFlag: boolean;
     static examples: string[];
     static flags: {
         yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
-    run(): Promise<void>;
+    run(): Promise<PushResult>;
     private ensureCanonicalFilename;
     private loadSnippets;
     private pushSnippet;
     private toPayload;
 }
+export {};