@bergetai/odoo-cli 0.5.3 → 0.7.1

package/README.md

@@ -57,6 +57,13 @@ 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
+odoo crm records add-note 338 --body "Customer replied: ..." --author 40
+
+# 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.
@@ -74,7 +81,8 @@ Note: `odoo --version` prints the CLI's own version; `odoo version` queries the
 - Documents surface (`documents`) — `tree` (hierarchical folder listing with `--count`/`--depth`/`--folder`), `records list/show` (folders excluded by default, `--type folder|binary|url|all`, `--folder`/`--owner`/`--partner`/`--tag`), `folders list`, `records create` (folder via `--folder-type folder`, link via `--link <url>`), `records update` (move via `--folder`), `records delete`, and `upload` (local file → base64 `datas`, MIME guessed from extension, `--dry-run` previews without sending bytes). Folders are `documents.document` rows with `type='folder'`; the hierarchy is `folder_id`. Module-not-installed error (exit 4)
 - 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
+- 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. Optional `--author <email|name|id>` attributes the note to the partner who actually sent the logged message (resolved via `res.partner`; ambiguous matches list candidates and never guess)
+- 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";
@@ -139,6 +139,7 @@ Examples:
         .argument("<id>", "record id")
         .option("--body <text>", "note body (HTML allowed)")
         .option("--body-file <path>", "read note body from file ('-' for stdin)")
+        .option("--author <email|name|id>", "note author (res.partner; default: your own partner)")
         .addHelpText("after", `
 IMPORTANT: This command creates INTERNAL notes only. These notes:
   • Are NEVER sent to the customer via email
@@ -151,7 +152,8 @@ This is by design to prevent accidental customer emails.
 Examples:
   $ odoo crm records add-note 338 --body "Internal note text"
   $ odoo crm records add-note 338 --body-file note.md
-  $ cat note.md | odoo crm records add-note 338 --body-file -`)
+  $ cat note.md | odoo crm records add-note 338 --body-file -
+  $ odoo crm records add-note 338 --body "Kundens svar: ..." --author andreas@berget.ai`)
         .action(async (id, opts) => {
         await run((g) => addNoteCommand(id, opts, g, "crm.lead"), program);
     });
@@ -177,6 +179,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)")
@@ -356,6 +393,7 @@ the raw form.`)
         .argument("<id>", "ticket id")
         .option("--body <text>", "note body (HTML allowed)")
         .option("--body-file <path>", "read note body from file ('-' for stdin)")
+        .option("--author <email|name|id>", "note author (res.partner; default: your own partner)")
         .addHelpText("after", `
 IMPORTANT: This command creates INTERNAL notes only. These notes:
   • Are NEVER sent to the customer via email
@@ -368,7 +406,8 @@ This is by design to prevent accidental customer emails.
 Examples:
   $ odoo helpdesk tickets add-note 398 --body "Internal note text"
   $ odoo helpdesk tickets add-note 398 --body-file note.md
-  $ cat note.md | odoo helpdesk tickets add-note 398 --body-file -`)
+  $ cat note.md | odoo helpdesk tickets add-note 398 --body-file -
+  $ odoo helpdesk tickets add-note 398 --body "Kundens svar: ..." --author kund@example.com`)
         .action(async (id, opts) => {
         await run((g) => addNoteCommand(id, opts, g, "helpdesk.ticket"), program);
     });
@@ -400,6 +439,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

@@ -1,6 +1,7 @@
 import { OdooClient } from "../client.js";
 import { resolveConfig } from "../config.js";
 import { notFoundError, usageError } from "../errors.js";
+import { resolveReference } from "../model.js";
 import { printJson, verbose as logVerbose } from "../output.js";
 import { formatTable } from "../table.js";
 const THREAD_MODELS = {
@@ -55,6 +56,21 @@ export async function addNoteCommand(recordId, opts, globals, model) {
     if (!rows || rows.length === 0) {
         throw notFoundError("record_not_found", `${spec.model} #${id} does not exist`, `check the ${spec.label} id`);
     }
+    // Optional: attribute the note to the person who actually sent the
+    // logged message (e.g. a forwarded customer email). author_id is a
+    // res.partner — resolve by exact email, name, or numeric id. Without
+    // --author Odoo defaults to the authenticated user's partner.
+    let author;
+    if (opts.author) {
+        author = await resolveReference(client, "res.partner", opts.author, {
+            label: "author",
+            searchFields: ["name"],
+            exactFields: ["email"],
+        });
+        if (globals.verbose) {
+            logVerbose(`resolved --author "${opts.author}" -> [${author.id}, "${author.name}"]`, globals.verbose);
+        }
+    }
     // Create internal Log Note
     // CRITICAL: These parameters ensure the note is INTERNAL ONLY
     // - subtype_id=2: "Note" subtype (internal, not sent to customer)
@@ -66,6 +82,9 @@ export async function addNoteCommand(recordId, opts, globals, model) {
         message_type: "comment",
         subtype_id: 2, // Note subtype = internal only, NEVER sends email
     };
+    if (author) {
+        noteParams.author_id = author.id;
+    }
     if (globals.verbose) {
         logVerbose(`Creating internal note for ${spec.model} #${id} (${rows[0].name})`, globals.verbose);
         logVerbose(`Note parameters: ${JSON.stringify(noteParams, null, 2)}`, globals.verbose);
@@ -78,6 +97,7 @@ export async function addNoteCommand(recordId, opts, globals, model) {
             record_name: rows[0].name,
             note_type: "internal_log_note",
             will_send_email: false,
+            author: author ? { id: author.id, name: author.name } : null,
             body_preview: body.substring(0, 100) + (body.length > 100 ? "..." : ""),
             body_length: body.length,
         };
@@ -88,6 +108,9 @@ export async function addNoteCommand(recordId, opts, globals, model) {
             console.log(`[DRY RUN] Would create internal Log Note:`);
             console.log(`  Record: ${spec.model} #${id} — ${rows[0].name}`);
             console.log(`  Type: Internal note (never sent to customer)`);
+            if (author) {
+                console.log(`  Author: ${author.name} (${author.id})`);
+            }
             console.log(`  Body length: ${body.length} characters`);
             console.log(`  Body preview: ${result.body_preview}`);
         }
@@ -103,6 +126,7 @@ export async function addNoteCommand(recordId, opts, globals, model) {
         record_name: rows[0].name,
         note_type: "internal_log_note",
         email_sent: false,
+        author: author ? { id: author.id, name: author.name } : null,
     };
     if (globals.json) {
         printJson(result);
@@ -111,6 +135,167 @@ export async function addNoteCommand(recordId, opts, globals, model) {
         console.log(`✓ Internal Log Note created (ID: ${noteId})`);
         console.log(`  Record: ${spec.model} #${id} — ${rows[0].name}`);
         console.log(`  Type: Internal note (not sent to customer)`);
+        if (author) {
+            console.log(`  Author: ${author.name} (${author.id})`);
+        }
+    }
+}
+/**
+ * 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}`);
+        }
     }
 }
 /**

package/package.json

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