@mailmodo/cli 0.0.20-beta.pr22.37 → 0.0.20-beta.pr23.33

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

@@ -16,9 +16,6 @@ export default class Deploy extends BaseCommand {
     run(): Promise<void>;
     private buildDeployPayload;
     private mapEmailToPayload;
-    private buildBrandSection;
-    private buildProductSection;
-    private buildSenderSection;
     private buildProjectPayload;
     private confirmDeploy;
     private ensureDomainReady;

package/dist/commands/deploy/index.js

@@ -77,39 +77,30 @@ export default class Deploy extends BaseCommand {
             trigger: email.trigger,
         };
     }
-    buildBrandSection(project) {
-        return {
-            colors: [project?.brandColor || DEFAULT_BRAND_COLOR],
-            logoUrl: project?.logoUrl || '',
-        };
-    }
-    buildProductSection(project) {
-        return {
-            businessType: project?.type || '',
-            description: '',
-            pricingModel: '',
-            productName: project?.name || '',
-            saasModel: '',
-            targetUser: '',
-            url: project?.url || '',
-        };
-    }
-    buildSenderSection(project) {
-        return {
-            address: project?.address || '',
-            domain: project?.domain || '',
-            fromEmail: project?.fromEmail || '',
-            fromName: project?.fromName || '',
-            replyTo: project?.replyTo || project?.fromEmail || '',
-        };
-    }
     buildProjectPayload(project) {
         return {
-            brand: this.buildBrandSection(project),
+            brand: {
+                colors: [project?.brandColor || DEFAULT_BRAND_COLOR],
+                logoUrl: project?.logoUrl || '',
+            },
             emailStyle: project?.emailStyle || 'branded',
             monthlyCap: project?.monthlyCap ?? DEFAULT_MONTHLY_CAP,
-            product: this.buildProductSection(project),
-            senderDetails: this.buildSenderSection(project),
+            product: {
+                businessType: project?.type || '',
+                description: '',
+                pricingModel: '',
+                productName: project?.name || '',
+                saasModel: '',
+                targetUser: '',
+                url: project?.url || '',
+            },
+            senderDetails: {
+                address: project?.address || '',
+                domain: project?.domain || '',
+                fromEmail: project?.fromEmail || '',
+                fromName: project?.fromName || '',
+                replyTo: project?.replyTo || project?.fromEmail || '',
+            },
             webhookUrl: project?.webhookUrl || '',
         };
     }

package/dist/commands/domain/index.js

@@ -124,15 +124,15 @@ export default class Domain extends BaseCommand {
         if (!response.ok) {
             this.handleApiError(response);
         }
-        const { dkim, dmarc, returnPath, domainStatus } = response.data;
+        const { dkim, dmarc, spf } = response.data;
         if (jsonOutput) {
-            this.log(JSON.stringify({ dkim, dmarc, returnPath, domainStatus }, null, 2));
+            this.log(JSON.stringify({ dkim, dmarc, spf }, null, 2));
             return;
         }
-        this.log(`  DKIM         ${dkim ? chalk.green('✓') : chalk.red('✗  Not found')}`);
-        this.log(`  DMARC        ${dmarc ? chalk.green('✓') : chalk.red('✗  Not found')}`);
-        this.log(`  Return Path  ${returnPath ? chalk.green('✓') : chalk.red('✗  Not found')}`);
-        const allPassed = domainStatus === 'VERIFIED';
+        this.log(`  SPF    ${spf ? chalk.green('✓') : chalk.red('✗  Not found')}`);
+        this.log(`  DKIM   ${dkim ? chalk.green('✓') : chalk.red('✗  Not found')}`);
+        this.log(`  DMARC  ${dmarc ? chalk.green('✓') : chalk.red('✗  Not found')}`);
+        const allPassed = spf && dkim && dmarc;
         if (allPassed) {
             this.log(`\n  ${chalk.green('✓')} Domain verified.\n`);
         }
@@ -144,11 +144,6 @@ export default class Domain extends BaseCommand {
                 this.log(`  - Including the full domain in the Host field`);
                 this.log(`  - Cloudflare: proxy must be OFF (grey cloud, not orange)`);
             }
-            if (!returnPath) {
-                this.log(`\n  Return Path common mistakes:`);
-                this.log(`  - Missing or incorrect CNAME for mm-bounce subdomain`);
-                this.log(`  - Cloudflare: proxy must be OFF (grey cloud, not orange)`);
-            }
             this.log(`\n  Fix the records and run ${chalk.cyan('mailmodo domain --verify')} again.`);
             this.log(`  Help: ${chalk.cyan(DNS_GUIDE_URL)}\n`);
         }

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

@@ -10,15 +10,5 @@ export default class Edit extends BaseCommand {
         json: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
-    private showFieldDiff;
-    private stripHtml;
-    private truncate;
-    private showHtmlChange;
-    private showUnchangedField;
-    private showUnchangedHtml;
-    private showSuggestedChanges;
-    private showUnchanged;
-    private buildDiffPreview;
-    private showChangeSummary;
     run(): Promise<void>;
 }

package/dist/commands/edit/index.js

@@ -22,123 +22,6 @@ export default class Edit extends BaseCommand {
             description: 'Natural language description of the change',
         }),
     };
-    showFieldDiff(label, oldVal, newVal) {
-        if (!newVal || oldVal === newVal)
-            return false;
-        this.log(`\n  ${label}:`);
-        if (oldVal)
-            this.log(`  ${chalk.red(`- ${oldVal}`)}`);
-        this.log(`  ${chalk.green(`+ ${newVal}`)}`);
-        return true;
-    }
-    stripHtml(html) {
-        return html
-            .replaceAll(/<style[^>]*>[\s\S]*?<\/style>/gi, '')
-            .replaceAll(/<script[^>]*>[\s\S]*?<\/script>/gi, '')
-            .replaceAll(/<[^>]+>/g, ' ')
-            .replaceAll('&nbsp;', ' ')
-            .replaceAll('&amp;', '&')
-            .replaceAll('&lt;', '<')
-            .replaceAll('&gt;', '>')
-            .replaceAll(/\s+/g, ' ')
-            .trim();
-    }
-    truncate(text, max) {
-        return text.length > max ? `${text.slice(0, max)}…` : text;
-    }
-    showHtmlChange(oldHtml, newHtml) {
-        if (!newHtml || oldHtml === newHtml)
-            return false;
-        this.log(`\n  HTML Body:`);
-        const MAX = 500;
-        if (oldHtml) {
-            const oldText = this.truncate(this.stripHtml(oldHtml), MAX);
-            this.log(`  ${chalk.red(`- ${oldText}`)}`);
-        }
-        const newText = this.truncate(this.stripHtml(newHtml), MAX);
-        this.log(`  ${chalk.green(`+ ${newText}`)}`);
-        return true;
-    }
-    showUnchangedField(label, value) {
-        if (!value)
-            return;
-        this.log(`\n  ${label}:`);
-        this.log(`  ${chalk.dim(value)}`);
-    }
-    showUnchangedHtml(templateHtml) {
-        if (!templateHtml)
-            return;
-        this.log(`\n  HTML Body:`);
-        this.log(`  ${chalk.dim(this.truncate(this.stripHtml(templateHtml), 500))}`);
-    }
-    showSuggestedChanges(email, updated, templateHtml, changed) {
-        this.log('\n  Suggested Changes:');
-        if (changed.subject)
-            this.showFieldDiff('Subject', email.subject, updated.subject);
-        if (changed.preview)
-            this.showFieldDiff('Preview Text', email.previewText, updated.previewText);
-        if (changed.html)
-            this.showHtmlChange(templateHtml, updated.html);
-        if (changed.cta)
-            this.showFieldDiff('CTA Text', undefined, updated.ctaText);
-        if (!changed.subject && !changed.preview && !changed.html && !changed.cta) {
-            this.log(`\n  ${chalk.dim('No changes detected.')}`);
-        }
-    }
-    showUnchanged(email, templateHtml, changed) {
-        const hasContent = !changed.subject ||
-            (!changed.preview && Boolean(email.previewText)) ||
-            (!changed.html && Boolean(templateHtml));
-        if (!hasContent)
-            return;
-        this.log('\n  Unchanged:');
-        if (!changed.subject)
-            this.showUnchangedField('Subject', email.subject);
-        if (!changed.preview)
-            this.showUnchangedField('Preview Text', email.previewText);
-        if (!changed.html)
-            this.showUnchangedHtml(templateHtml);
-    }
-    buildDiffPreview(email, updated, templateHtml) {
-        const subjectChanged = Boolean(updated.subject) && updated.subject !== email.subject;
-        const previewChanged = Boolean(updated.previewText) && updated.previewText !== email.previewText;
-        const htmlChanged = Boolean(updated.html) && updated.html !== templateHtml;
-        const diff = {};
-        diff.subject = subjectChanged
-            ? { new: updated.subject, old: email.subject }
-            : { unchanged: true, value: email.subject };
-        if (email.previewText ?? updated.previewText) {
-            diff.previewText = previewChanged
-                ? { new: updated.previewText, old: email.previewText }
-                : { unchanged: true, value: email.previewText };
-        }
-        if (templateHtml ?? updated.html) {
-            const oldText = templateHtml
-                ? this.truncate(this.stripHtml(templateHtml), 500)
-                : null;
-            const newText = updated.html
-                ? this.truncate(this.stripHtml(updated.html), 500)
-                : null;
-            diff.html = htmlChanged
-                ? { new: newText, old: oldText }
-                : { unchanged: true, value: oldText };
-        }
-        if (updated.ctaText) {
-            diff.ctaText = { new: updated.ctaText };
-        }
-        return { diff };
-    }
-    showChangeSummary(email, updated, templateHtml) {
-        const changed = {
-            cta: Boolean(updated.ctaText),
-            html: Boolean(updated.html) && updated.html !== templateHtml,
-            preview: Boolean(updated.previewText) &&
-                updated.previewText !== email.previewText,
-            subject: Boolean(updated.subject) && updated.subject !== email.subject,
-        };
-        this.showSuggestedChanges(email, updated, templateHtml, changed);
-        this.showUnchanged(email, templateHtml, changed);
-    }
     async run() {
         const { args, flags } = await this.parse(Edit);
         await this.ensureAuth();
@@ -149,6 +32,9 @@ export default class Edit extends BaseCommand {
         }
         const email = yamlConfig.emails[emailIndex];
         const templateHtml = await loadTemplate(`${email.id}.html`);
+        if (!flags.json) {
+            this.log(`\n  Current subject: '${chalk.cyan(email.subject)}'`);
+        }
         let changeDescription = flags.change;
         if (!changeDescription) {
             changeDescription = await input({
@@ -172,11 +58,12 @@ export default class Edit extends BaseCommand {
             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);
+        const oldSubject = email.subject;
+        const newSubject = updated.subject || email.subject;
+        if (!flags.json && oldSubject !== newSubject) {
+            this.log(`\n  Suggested subject:`);
+            this.log(`  ${chalk.red(`- ${oldSubject}`)}`);
+            this.log(`  ${chalk.green(`+ ${newSubject}`)}`);
         }
         if (!flags.yes) {
             const accepted = await confirm({
@@ -188,8 +75,6 @@ export default class Edit extends BaseCommand {
                 return;
             }
         }
-        const oldSubject = email.subject;
-        const newSubject = updated.subject || email.subject;
         if (updated.subject)
             email.subject = updated.subject;
         if (updated.previewText)
@@ -206,9 +91,6 @@ export default class Edit extends BaseCommand {
         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 },

package/dist/commands/preview/index.js

@@ -89,11 +89,7 @@ export default class Preview extends BaseCommand {
             const rendered = templateHtml
                 ? renderTemplate(templateHtml, sampleData)
                 : '';
-            await this.sendTestEmail(email, rendered, {
-                domain: yamlConfig.project?.domain,
-                jsonOutput: flags.json,
-                toAddress: flags.send,
-            });
+            await this.sendTestEmail(email, rendered, yamlConfig.project?.domain, flags.send, flags.json);
             return;
         }
         if (flags.text) {
@@ -130,8 +126,7 @@ export default class Preview extends BaseCommand {
      * Calls the API to send a test email to the specified address.
      * Before domain verification, tests send via the mailmodo.com domain.
      */
-    async sendTestEmail(email, html, opts) {
-        const { domain, jsonOutput, toAddress } = opts;
+    async sendTestEmail(email, html, domain, toAddress, jsonOutput) {
         await this.ensureAuth();
         const response = await this.withApiSpinner({ json: jsonOutput, text: '  Sending test email...' }, () => this.apiClient.post(`${API_ENDPOINTS.PREVIEW}/send`, {
             domain,

package/dist/commands/settings/index.js

@@ -52,7 +52,7 @@ export default class Settings extends BaseCommand {
             const key = flags.set.slice(0, eqIndex).trim();
             const propKey = settingKeyToProp(key);
             const value = flags.set.slice(eqIndex + 1).trim();
-            if (!(propKey in project)) {
+            if (!(propKey in project) && key !== 'logo_file') {
                 this.error(`Unknown setting: ${key}`);
             }
             project[propKey] =
@@ -73,7 +73,11 @@ export default class Settings extends BaseCommand {
         const domainVerified = await this.fetchDomainVerified(project.domain);
         this.log(`\n  Current settings for ${chalk.bold(project.name || 'project')}:\n`);
         for (const [group, keys] of Object.entries(SETTINGS_GROUPS)) {
-            const availableKeys = keys.filter((key) => settingKeyToProp(key) in project);
+            const availableKeys = keys.filter((key) => {
+                if (group === 'brand' && key === 'logo_file')
+                    return true;
+                return settingKeyToProp(key) in project;
+            });
             if (availableKeys.length === 0) {
                 const hint = SETUP_HINTS[settingKeyToProp(keys[0])];
                 if (hint) {
@@ -98,7 +102,8 @@ export default class Settings extends BaseCommand {
                 }
                 this.log(`  ${key.padEnd(16)} ${displayValue}`);
             }
-            const missingKeys = keys.filter((key) => !(settingKeyToProp(key) in project));
+            const missingKeys = keys.filter((key) => !(settingKeyToProp(key) in project) &&
+                !(group === 'brand' && key === 'logo_file'));
             for (const key of missingKeys) {
                 const hint = SETUP_HINTS[settingKeyToProp(key)];
                 if (hint) {
@@ -125,6 +130,10 @@ export default class Settings extends BaseCommand {
             return;
         const editPropKey = settingKeyToProp(editKey);
         if (!(editPropKey in project)) {
+            if (editKey === 'logo_file') {
+                await this.handleLogoUpload(yamlConfig);
+                return;
+            }
             const hint = SETUP_HINTS[editPropKey];
             if (hint) {
                 this.log(`\n  ${editKey} is not configured yet. Run ${chalk.cyan(hint)} to set it up.\n`);
@@ -252,8 +261,16 @@ export default class Settings extends BaseCommand {
         }
         await this.ensureAuth();
         const fileBuffer = await readFile(resolvedPath);
+        const ext = resolvedPath.split('.').pop()?.toLowerCase();
+        const mimeTypes = {
+            png: 'image/png',
+            jpg: 'image/jpeg',
+            jpeg: 'image/jpeg',
+            svg: 'image/svg+xml',
+        };
+        const mimeType = mimeTypes[ext ?? ''] ?? 'application/octet-stream';
         const formData = new FormData();
-        formData.append('logo', new Blob([new Uint8Array(fileBuffer)]), logoPath.split(/[/\\]/).pop() || 'logo.png');
+        formData.append('logo', new Blob([new Uint8Array(fileBuffer)], { type: mimeType }), logoPath.split(/[/\\]/).pop() || 'logo.png');
         const response = await this.apiClient.postFormData(API_ENDPOINTS.ASSETS_LOGO, formData);
         if (!response.ok) {
             this.handleApiError(response);

package/dist/lib/constants.d.ts

@@ -11,7 +11,7 @@ export declare const API_ENDPOINTS: Readonly<{
     DOMAIN: "/domain";
     DOMAIN_STATUS: "/domain/status";
     DOMAIN_VERIFY: "/domain/verify";
-    EDIT: "/email/edit";
+    EDIT: "/edit";
     EVENTS: "/events";
     GENERATE: "/email/generate";
     LOGS: "/logs";

package/dist/lib/constants.js

@@ -17,7 +17,7 @@ export const API_ENDPOINTS = Object.freeze({
     DOMAIN: '/domain',
     DOMAIN_STATUS: '/domain/status',
     DOMAIN_VERIFY: '/domain/verify',
-    EDIT: '/email/edit',
+    EDIT: '/edit',
     EVENTS: '/events',
     GENERATE: '/email/generate',
     LOGS: '/logs',

package/oclif.manifest.json

@@ -618,5 +618,5 @@
       ]
     }
   },
-  "version": "0.0.20-beta.pr22.37"
+  "version": "0.0.20-beta.pr23.33"
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@mailmodo/cli",
   "description": "Email lifecycle automation for the AI-native builder generation.",
-  "version": "0.0.20-beta.pr22.37",
+  "version": "0.0.20-beta.pr23.33",
   "author": "provishalk",
   "bin": {
     "mailmodo": "bin/run.js"