@bergetai/odoo-cli 0.5.3 → 0.6.1

package/README.md

@@ -57,6 +57,12 @@ odoo crm teams delete 5 --dry-run
 # Internal notes (NEVER sends email to customer)
 odoo helpdesk tickets add-note 398 --body "Internal note text"
 odoo helpdesk tickets add-note 398 --body-file note.md
+
+# Log note correction (message-update / message-delete; comment-type only)
+odoo crm records message-update 270799 --body-file fixed.html --dry-run
+odoo crm records message-update 270799 --body-file fixed.html
+odoo crm records message-delete 253924 258332 --dry-run
+odoo crm records message-delete 253924 258332
 ```
 
 Configuration precedence: flags > environment (`ODOO_URL`, `ODOO_DB`, `ODOO_USERNAME`, `ODOO_API_KEY`) > `$XDG_CONFIG_HOME/odoo/config.json`. The API key is never accepted as a flag value.
@@ -75,6 +81,7 @@ Note: `odoo --version` prints the CLI's own version; `odoo version` queries the
 - Writes (`create`, `update`, `delete`) — single-call atomic batch `write`, client-side no-op diffing, delete convergence with `skipped` ids, post-failure "record may have been created" hint on `create`
 - `--dry-run` for all mutations — full validation, resolution, and before/after diffs without mutating
 - Internal notes (`add-note`) — creates internal Log Notes only (subtype_id=2, message_type=comment), **NEVER sends email to customer**, by design prevents accidental customer communication
+- Log note correction (`message-update`, `message-delete`) — repair formatting or remove duplicate log notes; strictly limited to comment-type messages (emails/notifications refused by design), full `--dry-run` support, unknown delete ids skipped
 - Config sources and precedence with atomic `0600` config writes, XML-RPC transport with timeouts and HTTPS enforcement, and an auth failure taxonomy (db missing / bad credentials / unreachable)
 
 ## Compatibility

package/dist/cli.js

@@ -4,7 +4,7 @@ import { authLoginCommand, authStatusCommand } from "./commands/auth.js";
 import { leadsCreateCommand, leadsDeleteCommand, leadsListCommand, leadsShowCommand, leadsUpdateCommand, crmLeads, } from "./commands/crm.js";
 import { teamsCreateCommand, teamsDeleteCommand, teamsListCommand, teamsShowCommand, teamsUpdateCommand, crmTeams, } from "./commands/teams.js";
 import { ticketsCreateCommand, ticketsDeleteCommand, ticketsListCommand, ticketsShowCommand, ticketsUpdateCommand, helpdeskTickets, } from "./commands/helpdesk.js";
-import { addNoteCommand, listMessagesCommand, } from "./commands/notes.js";
+import { addNoteCommand, deleteMessageCommand, listMessagesCommand, updateMessageCommand, } from "./commands/notes.js";
 import { activitiesListCommand, activityDoneCommand, activityUpdateCommand, scheduleCommand, } from "./commands/activities.js";
 import { createTaskCommand, projectListCommand, projectShowCommand, tasksListCommand, tasksShowCommand, } from "./commands/project.js";
 import { documentsDocuments, documentsListCommand, foldersListCommand, documentsShowCommand, documentsCreateCommand, documentsUpdateCommand, documentsDeleteCommand, treeCommand, uploadCommand, } from "./commands/documents.js";
@@ -177,6 +177,41 @@ Examples:
         .action(async (id, opts) => {
         await run((g) => listMessagesCommand(id, opts, g, "crm.lead"), program);
     });
+    leads
+        .command("message-update")
+        .description("update the body of an existing log note (preview with --dry-run)")
+        .argument("<id>", "message id (see: records messages <id>)")
+        .option("--body <text>", "new body (HTML allowed)")
+        .option("--body-file <path>", "read new body from file ('-' for stdin)")
+        .addHelpText("after", `
+Log notes (message_type "comment") and "Activities" traces (Meeting/To-Do
+done — for correcting minutes after the fact) can be updated. Emails and other
+notifications are refused by design. The message id and date are preserved.
+
+Examples:
+  # Preview a formatting repair
+  $ odoo crm records message-update 270799 --body-file fixed.html --dry-run
+
+  # Apply it
+  $ odoo crm records message-update 270799 --body-file fixed.html`)
+        .action(async (id, opts) => {
+        await run((g) => updateMessageCommand(id, opts, g, "crm.lead"), program);
+    });
+    leads
+        .command("message-delete")
+        .description("delete log notes, e.g. duplicate logs (preview with --dry-run; unknown ids are skipped)")
+        .argument("<id...>", "message ids (see: records messages <id>)")
+        .addHelpText("after", `
+Only internal log notes (message_type "comment") can be deleted — emails and
+notifications are refused by design. Deletion is irreversible: run --dry-run
+first and consider consolidating content into the note you keep.
+
+Examples:
+  $ odoo crm records message-delete 253924 258332 --dry-run
+  $ odoo crm records message-delete 253924 258332 --json`)
+        .action(async (ids) => {
+        await run((g) => deleteMessageCommand(ids, g, "crm.lead"), program);
+    });
     leads
         .command("schedule")
         .description("schedule a new activity on a record (preview with --dry-run)")
@@ -400,6 +435,38 @@ Examples:
         .action(async (id, opts) => {
         await run((g) => listMessagesCommand(id, opts, g, "helpdesk.ticket"), program);
     });
+    tickets
+        .command("message-update")
+        .description("update the body of an existing log note (preview with --dry-run)")
+        .argument("<id>", "message id (see: tickets messages <id>)")
+        .option("--body <text>", "new body (HTML allowed)")
+        .option("--body-file <path>", "read new body from file ('-' for stdin)")
+        .addHelpText("after", `
+Log notes (message_type "comment") and "Activities" traces (Meeting/To-Do
+done — for correcting minutes after the fact) can be updated. Emails and other
+notifications are refused by design. The message id and date are preserved.
+
+Examples:
+  $ odoo helpdesk tickets message-update 270799 --body-file fixed.html --dry-run
+  $ odoo helpdesk tickets message-update 270799 --body-file fixed.html`)
+        .action(async (id, opts) => {
+        await run((g) => updateMessageCommand(id, opts, g, "helpdesk.ticket"), program);
+    });
+    tickets
+        .command("message-delete")
+        .description("delete log notes, e.g. duplicate logs (preview with --dry-run; unknown ids are skipped)")
+        .argument("<id...>", "message ids (see: tickets messages <id>)")
+        .addHelpText("after", `
+Only internal log notes (message_type "comment") can be deleted — emails and
+notifications are refused by design. Deletion is irreversible: run --dry-run
+first and consider consolidating content into the note you keep.
+
+Examples:
+  $ odoo helpdesk tickets message-delete 253924 258332 --dry-run
+  $ odoo helpdesk tickets message-delete 253924 258332 --json`)
+        .action(async (ids) => {
+        await run((g) => deleteMessageCommand(ids, g, "helpdesk.ticket"), program);
+    });
     const project = program
         .command("project")
         .description("Project app (project.project, project.task)");

package/dist/commands/notes.js

@@ -113,6 +113,164 @@ export async function addNoteCommand(recordId, opts, globals, model) {
         console.log(`  Type: Internal note (not sent to customer)`);
     }
 }
+/**
+ * Read a mail.message and enforce the safety rules shared by
+ * updateMessageCommand and deleteMessageCommand:
+ * - the message must exist
+ * - it must belong to the command's thread model (crm.lead / helpdesk.ticket)
+ * - for updates: it must be a comment (log note) OR an "Activities"
+ *   notification (the "Meeting done"/"To-Do done" trace, whose embedded
+ *   note may need correction after the activity is marked done)
+ * - for deletes: strictly comments — deleting activity traces would
+ *   corrupt reporting (the done-message is all that remains of an activity)
+ */
+async function readLogNote(client, messageId, spec, opts = {}) {
+    const rows = (await client.executeKw("mail.message", "read", [[messageId]], {
+        fields: ["id", "model", "res_id", "body", "message_type", "subtype_id", "date"],
+    }));
+    if (!rows || rows.length === 0) {
+        throw notFoundError("message_not_found", `mail.message #${messageId} does not exist`, "check the message id (see: records messages <id>)");
+    }
+    const msg = rows[0];
+    if (msg.model !== spec.model) {
+        throw usageError(`mail.message #${messageId} belongs to ${msg.model}, not ${spec.model}`, `use the ${msg.model === "helpdesk.ticket" ? "helpdesk tickets" : "crm records"} command for this message`);
+    }
+    const subtypeName = Array.isArray(msg.subtype_id) ? msg.subtype_id[1] : undefined;
+    const isComment = msg.message_type === "comment";
+    const isActivityTrace = msg.message_type === "notification" && subtypeName === "Activities";
+    if (opts.forDelete ? !isComment : !(isComment || isActivityTrace)) {
+        const allowed = opts.forDelete
+            ? 'only "comment" (log notes) can be deleted'
+            : 'only "comment" (log notes) and "Activities" traces can be modified';
+        throw usageError(`mail.message #${messageId} has type "${msg.message_type}"${subtypeName ? ` / subtype "${subtypeName}"` : ""} — ${allowed}`, "emails and system notifications are left untouched by design");
+    }
+    return msg;
+}
+function bodyPreview(body, max = 80) {
+    const text = body
+        .replace(/<[^>]*>/g, "")
+        .replace(/\s+/g, " ")
+        .trim();
+    return text.length > max ? `${text.substring(0, max)}...` : text;
+}
+/**
+ * Update the body of an existing internal log note (mail.message).
+ *
+ * Use case: repair formatting (escaped HTML, markdown, platform noise) or
+ * correct a note posted with wrong content. The message id and date are
+ * preserved — only the body changes.
+ */
+export async function updateMessageCommand(messageId, opts, globals, model) {
+    const spec = threadModel(model);
+    const id = Number.parseInt(messageId, 10);
+    if (Number.isNaN(id) || id <= 0) {
+        throw usageError(`invalid message id: ${messageId}`, "provide a positive integer");
+    }
+    const body = await readBody(opts);
+    const client = await buildClient(globals);
+    const msg = await readLogNote(client, id, spec);
+    if (globals.dryRun) {
+        const result = {
+            dry_run: true,
+            message_id: id,
+            model: msg.model,
+            res_id: msg.res_id,
+            old_body_preview: bodyPreview(msg.body),
+            new_body_preview: bodyPreview(body),
+            new_body_length: body.length,
+        };
+        if (globals.json) {
+            printJson(result);
+        }
+        else {
+            console.log(`[DRY RUN] Would update log note #${id} on ${msg.model} #${msg.res_id}:`);
+            console.log(`  Old: ${result.old_body_preview}`);
+            console.log(`  New: ${result.new_body_preview}`);
+        }
+        return;
+    }
+    await client.executeKw("mail.message", "write", [[id], { body }]);
+    const result = {
+        message_id: id,
+        model: msg.model,
+        res_id: msg.res_id,
+        updated: true,
+    };
+    if (globals.json) {
+        printJson(result);
+    }
+    else {
+        console.log(`✓ Log note #${id} updated on ${msg.model} #${msg.res_id}`);
+    }
+}
+/**
+ * Delete internal log notes (mail.message) — e.g. duplicate logs of the
+ * same dialog. Unknown ids are skipped with a warning. Only comment-type
+ * messages can be deleted.
+ */
+export async function deleteMessageCommand(messageIds, globals, model) {
+    const spec = threadModel(model);
+    const client = await buildClient(globals);
+    const toDelete = [];
+    const skipped = [];
+    for (const raw of messageIds) {
+        const id = Number.parseInt(raw, 10);
+        if (Number.isNaN(id) || id <= 0) {
+            skipped.push({ id: raw, reason: "invalid id" });
+            continue;
+        }
+        try {
+            toDelete.push(await readLogNote(client, id, spec, { forDelete: true }));
+        }
+        catch (error) {
+            if (error instanceof Error && "exitCode" in error && error.exitCode === 4) {
+                skipped.push({ id: raw, reason: "not found" });
+                continue;
+            }
+            throw error;
+        }
+    }
+    if (globals.dryRun) {
+        const result = {
+            dry_run: true,
+            would_delete: toDelete.map((m) => ({
+                message_id: m.id,
+                res_id: m.res_id,
+                date: m.date,
+                body_preview: bodyPreview(m.body),
+            })),
+            skipped,
+        };
+        if (globals.json) {
+            printJson(result);
+        }
+        else {
+            console.log(`[DRY RUN] Would delete ${toDelete.length} log note(s):`);
+            for (const m of toDelete) {
+                console.log(`  #${m.id} (${m.date}) ${bodyPreview(m.body)}`);
+            }
+            for (const s of skipped) {
+                console.log(`  skipped #${s.id}: ${s.reason}`);
+            }
+        }
+        return;
+    }
+    const deleted = [];
+    for (const m of toDelete) {
+        await client.executeKw("mail.message", "unlink", [[m.id]]);
+        deleted.push(m.id);
+    }
+    const result = { deleted, skipped };
+    if (globals.json) {
+        printJson(result);
+    }
+    else {
+        console.log(`✓ Deleted ${deleted.length} log note(s): ${deleted.join(", ") || "none"}`);
+        for (const s of skipped) {
+            console.log(`  skipped #${s.id}: ${s.reason}`);
+        }
+    }
+}
 /**
  * List messages for a mail-thread record (emails, comments, notes).
  *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bergetai/odoo-cli",
-  "version": "0.5.3",
+  "version": "0.6.1",
   "description": "Agent-first CLI for the Odoo External API",
   "publishConfig": {
     "access": "public"