@mailmodo/cli 0.0.29-beta.pr31.49 → 0.0.30-beta.pr32.50

package/dist/commands/edit/index.d.ts

@@ -10,6 +10,17 @@ export default class Edit extends BaseCommand {
         json: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
+    run(): Promise<void>;
+    private runEditStep;
+    private handleUserAction;
+    private callEditApi;
+    private finalizeEdit;
+    private applyEmailChanges;
+    private persistChanges;
+    private logJsonResult;
+    private handleAcceptOutput;
+    private promptEditAction;
+    private askChangeDescription;
     private showFieldDiff;
     private stripHtml;
     private truncate;
@@ -20,5 +31,4 @@ export default class Edit extends BaseCommand {
     private showUnchanged;
     private buildDiffPreview;
     private showChangeSummary;
-    run(): Promise<void>;
 }

package/dist/commands/edit/index.js

@@ -1,5 +1,5 @@
 import { Args, Flags } from '@oclif/core';
-import { confirm, input } from '@inquirer/prompts';
+import { confirm, input, select } from '@inquirer/prompts';
 import chalk from 'chalk';
 import { BaseCommand } from '../../lib/base-command.js';
 import { API_ENDPOINTS } from '../../lib/constants.js';
@@ -22,6 +22,149 @@ export default class Edit extends BaseCommand {
             description: 'Natural language description of the change',
         }),
     };
+    async run() {
+        const { args, flags } = await this.parse(Edit);
+        await this.ensureAuth();
+        const yamlConfig = await this.ensureYaml();
+        const emailIndex = yamlConfig.emails.findIndex((e) => e.id === args.id);
+        if (emailIndex === -1) {
+            this.error(`Email '${args.id}' not found in mailmodo.yaml.`);
+        }
+        const ctx = {
+            email: yamlConfig.emails[emailIndex],
+            emailIndex,
+            templateHtml: await loadTemplate(`${yamlConfig.emails[emailIndex].id}.html`),
+            yamlConfig,
+        };
+        const editFlags = {
+            json: flags.json ?? false,
+            yes: flags.yes ?? false,
+        };
+        const initialChange = flags.change ?? (await this.askChangeDescription());
+        await this.runEditStep(ctx, initialChange, editFlags);
+    }
+    async runEditStep(ctx, changeDescription, flags) {
+        const response = await this.withApiSpinner({ json: flags.json, text: '  Applying AI edits...' }, () => this.callEditApi(changeDescription, ctx.email, ctx.templateHtml));
+        if (!response.ok) {
+            this.handleApiError(response);
+        }
+        const updated = response.data;
+        if (flags.json) {
+            this.log(JSON.stringify(this.buildDiffPreview(ctx.email, updated, ctx.templateHtml), null, 2));
+        }
+        else {
+            this.showChangeSummary(ctx.email, updated, ctx.templateHtml);
+        }
+        if (flags.yes) {
+            await this.finalizeEdit(ctx, updated, flags);
+            return;
+        }
+        await this.handleUserAction(ctx, updated, changeDescription, flags);
+    }
+    async handleUserAction(ctx, updated, changeDescription, flags) {
+        this.log('');
+        const action = await this.promptEditAction();
+        if (action === 'skip') {
+            this.log('\n  Changes discarded.\n');
+            return;
+        }
+        if (action === 'retry') {
+            const newChange = await this.askChangeDescription();
+            await this.runEditStep(ctx, newChange, flags);
+            return;
+        }
+        await this.finalizeEdit(ctx, updated, flags);
+    }
+    callEditApi(changeDescription, email, templateHtml) {
+        return this.apiClient.post(API_ENDPOINTS.EDIT, {
+            changeRequest: changeDescription,
+            currentEmail: {
+                condition: email.condition,
+                goal: email.goal,
+                html: templateHtml,
+                id: email.id,
+                previewText: email.previewText,
+                subject: email.subject,
+                trigger: email.trigger,
+            },
+        });
+    }
+    async finalizeEdit(ctx, updated, flags) {
+        const oldSubject = ctx.email.subject;
+        this.applyEmailChanges(ctx.email, updated);
+        await this.persistChanges(ctx, updated);
+        if (flags.json) {
+            this.logJsonResult(ctx.email, updated, oldSubject);
+        }
+        else if (flags.yes) {
+            this.log(`\n  Updated ${chalk.green('mailmodo.yaml')}\n`);
+        }
+        else {
+            await this.handleAcceptOutput(ctx.email);
+        }
+    }
+    applyEmailChanges(email, updated) {
+        if (updated.subject)
+            email.subject = updated.subject;
+        if (updated.previewText)
+            email.previewText = updated.previewText;
+        if (updated.ctaText)
+            email.ctaText = updated.ctaText;
+    }
+    async persistChanges(ctx, updated) {
+        const updatedYaml = {
+            ...ctx.yamlConfig,
+            emails: [...ctx.yamlConfig.emails],
+        };
+        updatedYaml.emails[ctx.emailIndex] = ctx.email;
+        await saveYaml(updatedYaml);
+        if (updated.html) {
+            await saveTemplate(`${ctx.email.id}.html`, updated.html);
+        }
+    }
+    logJsonResult(email, updated, oldSubject) {
+        this.log(JSON.stringify({
+            diff: {
+                previewText: updated.previewText && updated.previewText !== email.previewText
+                    ? { new: updated.previewText, old: email.previewText }
+                    : undefined,
+                subject: oldSubject === email.subject
+                    ? undefined
+                    : { new: email.subject, old: oldSubject },
+            },
+            email,
+            status: 'updated',
+        }, null, 2));
+    }
+    async handleAcceptOutput(email) {
+        this.log(`\n  Updated ${chalk.green('mailmodo.yaml')}`);
+        const shouldPreview = await confirm({
+            default: true,
+            message: 'Preview the change?',
+        });
+        if (shouldPreview) {
+            await this.config.runCommand('preview', [email.id]);
+        }
+        else {
+            this.log(`  Run: ${chalk.cyan(`mailmodo preview ${email.id}`)}\n`);
+        }
+    }
+    async promptEditAction() {
+        return select({
+            message: 'Accept, try again, or skip?',
+            choices: [
+                { name: 'Accept', value: 'accept' },
+                { name: 'Try again', value: 'retry' },
+                { name: 'Skip', value: 'skip' },
+            ],
+        });
+    }
+    async askChangeDescription() {
+        return input({
+            message: 'What do you want to change?',
+            validate: (value) => value?.trim() ? true : 'Please describe the change',
+        });
+    }
     showFieldDiff(label, oldVal, newVal) {
         if (!newVal || oldVal === newVal)
             return false;
@@ -139,88 +282,4 @@ export default class Edit extends BaseCommand {
         this.showSuggestedChanges(email, updated, templateHtml, changed);
         this.showUnchanged(email, templateHtml, changed);
     }
-    async run() {
-        const { args, flags } = await this.parse(Edit);
-        await this.ensureAuth();
-        const yamlConfig = await this.ensureYaml();
-        const emailIndex = yamlConfig.emails.findIndex((e) => e.id === args.id);
-        if (emailIndex === -1) {
-            this.error(`Email '${args.id}' not found in mailmodo.yaml.`);
-        }
-        const email = yamlConfig.emails[emailIndex];
-        const templateHtml = await loadTemplate(`${email.id}.html`);
-        let changeDescription = flags.change;
-        if (!changeDescription) {
-            changeDescription = await input({
-                message: 'What do you want to change?',
-                validate: (value) => value?.trim() ? true : 'Please describe the change',
-            });
-        }
-        const response = await this.withApiSpinner({ json: flags.json, text: '  Applying AI edits...' }, () => this.apiClient.post(API_ENDPOINTS.EDIT, {
-            changeRequest: changeDescription,
-            currentEmail: {
-                condition: email.condition,
-                goal: email.goal,
-                html: templateHtml,
-                id: email.id,
-                previewText: email.previewText,
-                subject: email.subject,
-                trigger: email.trigger,
-            },
-        }));
-        if (!response.ok) {
-            this.handleApiError(response);
-        }
-        const updated = response.data;
-        if (flags.json) {
-            this.log(JSON.stringify(this.buildDiffPreview(email, updated, templateHtml), null, 2));
-        }
-        else {
-            this.showChangeSummary(email, updated, templateHtml);
-        }
-        if (!flags.yes) {
-            const accepted = await confirm({
-                default: true,
-                message: 'Accept changes?',
-            });
-            if (!accepted) {
-                this.log('\n  Changes discarded.\n');
-                return;
-            }
-        }
-        const oldSubject = email.subject;
-        const newSubject = updated.subject || email.subject;
-        if (updated.subject)
-            email.subject = updated.subject;
-        if (updated.previewText)
-            email.previewText = updated.previewText;
-        if (updated.ctaText)
-            email.ctaText = updated.ctaText;
-        const updatedYaml = {
-            ...yamlConfig,
-            emails: [...yamlConfig.emails],
-        };
-        updatedYaml.emails[emailIndex] = email;
-        await saveYaml(updatedYaml);
-        if (updated.html) {
-            await saveTemplate(`${email.id}.html`, updated.html);
-        }
-        if (flags.json) {
-            this.log(JSON.stringify({
-                diff: {
-                    previewText: updated.previewText && updated.previewText !== email.previewText
-                        ? { new: updated.previewText, old: email.previewText }
-                        : undefined,
-                    subject: oldSubject === newSubject
-                        ? undefined
-                        : { new: newSubject, old: oldSubject },
-                },
-                email,
-                status: 'updated',
-            }, null, 2));
-            return;
-        }
-        this.log(`\n  Updated ${chalk.green('mailmodo.yaml')}`);
-        this.log(`  Preview the change: ${chalk.cyan(`mailmodo preview ${email.id}`)}\n`);
-    }
 }

package/dist/commands/emails/index.d.ts

@@ -7,4 +7,6 @@ export default class Emails extends BaseCommand {
         yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
+    private openTemplateInEditor;
+    private trySpawnEditor;
 }

package/dist/commands/emails/index.js

@@ -1,5 +1,8 @@
-import { input } from '@inquirer/prompts';
+import { spawn } from 'node:child_process';
+import { join } from 'node:path';
+import { confirm, input } from '@inquirer/prompts';
 import chalk from 'chalk';
+import open from 'open';
 import { BaseCommand } from '../../lib/base-command.js';
 export default class Emails extends BaseCommand {
     static description = 'List and view configured email sequences';
@@ -61,7 +64,48 @@ export default class Emails extends BaseCommand {
                     this.log(`  ${chalk.bold('Goal:')}     ${email.goal}`);
                 }
                 this.log('');
+                const openIt = await confirm({
+                    default: true,
+                    message: 'Open template in editor?',
+                });
+                if (openIt) {
+                    await this.openTemplateInEditor(email.template);
+                }
             }
         }
     }
+    async openTemplateInEditor(template) {
+        const templatePath = join(process.cwd(), template);
+        this.log(`\n  Opening ${template}...\n`);
+        const editor = process.env.VISUAL || process.env.EDITOR;
+        if (editor) {
+            const child = spawn(editor, [templatePath], { stdio: 'inherit' });
+            await new Promise((resolve) => {
+                child.on('close', resolve);
+            });
+            return;
+        }
+        if (await this.trySpawnEditor('code', templatePath))
+            return;
+        try {
+            await open(templatePath);
+        }
+        catch {
+            this.log(`  ${chalk.dim(`Could not open editor. Open the file manually: ${templatePath}`)}`);
+        }
+    }
+    trySpawnEditor(editor, filePath) {
+        const [cmd, args] = process.platform === 'win32'
+            ? ['cmd.exe', ['/c', editor, filePath]]
+            : [editor, [filePath]];
+        return new Promise((resolve) => {
+            const child = spawn(cmd, [...args], { stdio: 'ignore' });
+            child.on('error', () => {
+                resolve(false);
+            });
+            child.on('close', (code) => {
+                resolve(code === 0);
+            });
+        });
+    }
 }

package/oclif.manifest.json

@@ -228,19 +228,13 @@
         "index.js"
       ]
     },
-    "edit": {
+    "emails": {
       "aliases": [],
-      "args": {
-        "id": {
-          "description": "Email template ID to edit",
-          "name": "id",
-          "required": true
-        }
-      },
-      "description": "Edit an email using AI-assisted natural language changes",
+      "args": {},
+      "description": "List and view configured email sequences",
       "examples": [
-        "<%= config.bin %> edit welcome",
-        "<%= config.bin %> edit welcome --change \"make subject more urgent\" --yes"
+        "<%= config.bin %> emails",
+        "<%= config.bin %> emails --json"
       ],
       "flags": {
         "json": {
@@ -255,18 +249,11 @@
           "name": "yes",
           "allowNo": false,
           "type": "boolean"
-        },
-        "change": {
-          "description": "Natural language description of the change",
-          "name": "change",
-          "hasDynamicHelp": false,
-          "multiple": false,
-          "type": "option"
         }
       },
       "hasDynamicHelp": false,
       "hiddenAliases": [],
-      "id": "edit",
+      "id": "emails",
       "pluginAlias": "@mailmodo/cli",
       "pluginName": "@mailmodo/cli",
       "pluginType": "core",
@@ -276,17 +263,23 @@
       "relativePath": [
         "dist",
         "commands",
-        "edit",
+        "emails",
         "index.js"
       ]
     },
-    "emails": {
+    "edit": {
       "aliases": [],
-      "args": {},
-      "description": "List and view configured email sequences",
+      "args": {
+        "id": {
+          "description": "Email template ID to edit",
+          "name": "id",
+          "required": true
+        }
+      },
+      "description": "Edit an email using AI-assisted natural language changes",
       "examples": [
-        "<%= config.bin %> emails",
-        "<%= config.bin %> emails --json"
+        "<%= config.bin %> edit welcome",
+        "<%= config.bin %> edit welcome --change \"make subject more urgent\" --yes"
       ],
       "flags": {
         "json": {
@@ -301,11 +294,18 @@
           "name": "yes",
           "allowNo": false,
           "type": "boolean"
+        },
+        "change": {
+          "description": "Natural language description of the change",
+          "name": "change",
+          "hasDynamicHelp": false,
+          "multiple": false,
+          "type": "option"
         }
       },
       "hasDynamicHelp": false,
       "hiddenAliases": [],
-      "id": "emails",
+      "id": "edit",
       "pluginAlias": "@mailmodo/cli",
       "pluginName": "@mailmodo/cli",
       "pluginType": "core",
@@ -315,7 +315,7 @@
       "relativePath": [
         "dist",
         "commands",
-        "emails",
+        "edit",
         "index.js"
       ]
     },
@@ -657,5 +657,5 @@
       ]
     }
   },
-  "version": "0.0.29-beta.pr31.49"
+  "version": "0.0.30-beta.pr32.50"
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@mailmodo/cli",
   "description": "Email lifecycle automation for the AI-native builder generation.",
-  "version": "0.0.29-beta.pr31.49",
+  "version": "0.0.30-beta.pr32.50",
   "author": "provishalk",
   "bin": {
     "mailmodo": "bin/run.js"