@bergetai/odoo-cli 0.12.4 → 0.12.6

package/README.md

@@ -68,6 +68,9 @@ odoo sales orders update 11515 --set plan_id=2 --dry-run
 odoo sales lines add 11515 --lines extra.json
 odoo sales lines update 11818 --lines fix.json
 odoo sales plans list
+odoo sales plans pricing list --pricelist "SEK price list"   # recurring prices (sale.subscription.pricing)
+odoo sales plans pricing create --pricelist 11 --product 188 --plan Yearly --price 19800 --dry-run
+odoo sales plans pricing update 131 --price 19800 --dry-run
 odoo sales invoices precedent --product 188        # invoicing currency history
 odoo sales pdf 11515 --out ./quotation.pdf
 
@@ -155,7 +158,7 @@ Note: `odoo --version` prints the CLI's own version; `odoo version` queries the
 
 ## Features
 
-- Sales surface (`sales`) — quotations and sales orders (`sale.order`) without the raw XML-RPC plumbing: `orders list` (curated columns incl. pricelist/currency, `--partner`/`--state` filters plus `--filter`/`--domain`), `orders read` (full header, partner with VAT/address, and the order lines with sections/notes), `orders create` (first-class flags for `--partner`/`--pricelist`/`--payment-terms`/`--validity`/`--opportunity`/`--salesperson`/`--plan`/`--note`, order lines as a JSON array via `--lines <file>`/`-` supporting `line_section`/`line_note` lines and product lines with `tax_ids`; missing line name/price default from the product), `orders update` (header-only `--set` with an allowlist — setting `state` is refused by design: confirm/cancel is a human step in the Odoo UI), `lines add`/`lines update` (same JSON line format), `plans list` (`sale.subscription.plan`), `invoices precedent --product` (posted customer-invoice lines for a product, newest first — invoicing currency precedent), and `pdf <id> --out <path>` (portal quotation PDF via the order's `access_token`, verified `%PDF` magic). **Sending quotations is deliberately not implemented** — it is done by a human via 'Send by Email' in the Odoo UI (gives the customer Accept & Sign), CLI policy 2026-09-24
+- Sales surface (`sales`) — quotations and sales orders (`sale.order`) without the raw XML-RPC plumbing: `orders list` (curated columns incl. pricelist/currency, `--partner`/`--state` filters plus `--filter`/`--domain`), `orders read` (full header, partner with VAT/address, and the order lines with sections/notes), `orders create` (first-class flags for `--partner`/`--pricelist`/`--payment-terms`/`--validity`/`--opportunity`/`--salesperson`/`--plan`/`--note`, order lines as a JSON array via `--lines <file>`/`-` supporting `line_section`/`line_note` lines and product lines with `tax_ids`; missing line name/price default from the product), `orders update` (header-only `--set` with an allowlist — setting `state` is refused by design: confirm/cancel is a human step in the Odoo UI), `lines add`/`lines update` (same JSON line format), `plans list` (`sale.subscription.plan`), `plans pricing list`/`create`/`update` (recurring prices, `sale.subscription.pricing` — one rule per pricelist+product+plan; subscription order lines price from these rules, not from `product.pricelist.item`), `invoices precedent --product` (posted customer-invoice lines for a product, newest first — invoicing currency precedent), and `pdf <id> --out <path>` (portal quotation PDF via the order's `access_token`, verified `%PDF` magic). **Sending quotations is deliberately not implemented** — it is done by a human via 'Send by Email' in the Odoo UI (gives the customer Accept & Sign), CLI policy 2026-09-24
 - Signature pipeline (`sign`) — Odoo Sign with safety rails learned from live incidents:
   - `sign templates list/show` and `sign templates create-from-document <id>` — builds a
     `sign.template` from a Documents file, converting non-PDF files (`.docx` etc.) to PDF through

package/dist/cli.js

@@ -13,7 +13,7 @@ import { partnersCreateCommand, partnersDeleteCommand, partnersListCommand, part
 import { usersDeleteCommand, usersShowCommand, } from "./commands/users.js";
 import { attachmentsDeleteCommand, attachmentsDownloadCommand, attachmentsListCommand, attachmentsShowCommand, } from "./commands/attachments.js";
 import { signTemplatesCreateFromDocumentCommand, signTemplatesListCommand, signTemplatesShowCommand, signItemsAddCommand, signRequestsCreateCommand, signRequestsListCommand, signRequestsCancelCommand, } from "./commands/sign.js";
-import { invoicesPrecedentCommand, linesAddCommand, linesUpdateCommand, ordersCreateCommand, ordersListCommand, ordersReadCommand, ordersUpdateCommand, pdfCommand, plansListCommand, } from "./commands/sales.js";
+import { invoicesPrecedentCommand, linesAddCommand, linesUpdateCommand, ordersCreateCommand, ordersListCommand, ordersReadCommand, ordersUpdateCommand, pdfCommand, plansListCommand, plansPricingCreateCommand, plansPricingListCommand, plansPricingUpdateCommand, } from "./commands/sales.js";
 import { messagesDeleteCommand, messagesListCommand, messagesScrubCommand, messagesShowCommand, } from "./commands/messages.js";
 import { contactsDeleteCommand, contactsListCommand, contactsShowCommand, listsListCommand, listsShowCommand, tracesDeleteCommand, tracesListCommand, tracesShowCommand, } from "./commands/marketing.js";
 import { modelsFieldsCommand, modelsReadCommand, } from "./commands/models.js";
@@ -1033,6 +1033,59 @@ Examples:
         .action(async (opts) => {
         await run((g) => plansListCommand(opts, g), program);
     });
+    const salesPlansPricing = salesPlans
+        .command("pricing")
+        .description("recurring prices per pricelist/product/plan (sale.subscription.pricing)");
+    salesPlansPricing
+        .command("list")
+        .description("list recurring pricing rules")
+        .option("--pricelist <name|id>", "filter by pricelist (product.pricelist)")
+        .option("--product <name|id>", "filter by product (product.template)")
+        .option("--plan <name|id>", "filter by plan (sale.subscription.plan)")
+        .option("--limit <n>", "max records (default: 50)", "50")
+        .addHelpText("after", `
+Subscription order lines price from these rules, not from
+product.pricelist.item. If an order's pricelist has no rule for the order's
+plan, the line silently prices at 0 — check here before switching pricelists.
+
+Examples:
+  $ odoo sales plans pricing list --pricelist "SEK price list"
+  $ odoo sales plans pricing list --product "Berget Code Seat" --json`)
+        .action(async (opts) => {
+        await run((g) => plansPricingListCommand(opts, g), program);
+    });
+    salesPlansPricing
+        .command("create")
+        .description("create a recurring pricing rule (preview with --dry-run)")
+        .requiredOption("--pricelist <name|id>", "pricelist (product.pricelist)")
+        .requiredOption("--product <name|id>", "product (product.template)")
+        .requiredOption("--plan <name|id>", "recurring plan (sale.subscription.plan)")
+        .requiredOption("--price <amount>", "recurring price (number)")
+        .addHelpText("after", `
+Only one rule per (pricelist, product, plan) is allowed — duplicates are
+rejected; use 'sales plans pricing update' to change an existing rule.
+
+Examples:
+  $ odoo sales plans pricing create --pricelist "SEK price list" --product 188 --plan Yearly --price 19800 --dry-run
+  $ odoo sales plans pricing create --pricelist 11 --product 188 --plan 2 --price 19800`)
+        .action(async (opts) => {
+        await run((g) => plansPricingCreateCommand(opts, g), program);
+    });
+    salesPlansPricing
+        .command("update")
+        .description("update a recurring pricing rule (preview with --dry-run)")
+        .argument("<id>", "sale.subscription.pricing id")
+        .option("--pricelist <name|id>", "pricelist (product.pricelist)")
+        .option("--product <name|id>", "product (product.template)")
+        .option("--plan <name|id>", "recurring plan (sale.subscription.plan)")
+        .option("--price <amount>", "recurring price (number)")
+        .addHelpText("after", `
+Examples:
+  $ odoo sales plans pricing update 131 --price 19800 --dry-run
+  $ odoo sales plans pricing update 131 --plan Yearly`)
+        .action(async (id, opts) => {
+        await run((g) => plansPricingUpdateCommand(id, opts, g), program);
+    });
     const salesInvoices = sales
         .command("invoices")
         .description("invoicing reports (account.move.line)");

package/dist/commands/sales.js

@@ -15,7 +15,8 @@ import { resolveConfig } from "../config.js";
 import { conflictError, networkError, notFoundError, usageError, } from "../errors.js";
 import { buildFilterClauses, parseDomain, parseFilterScalar } from "../filter.js";
 import { ModelFields, resolveReference } from "../model.js";
-import { parseDate } from "../mutation.js";
+import { parseDate, parseDecimal } from "../mutation.js";
+import { XmlRpcFaultError } from "../xmlrpc.js";
 import { colorEnabled, printJson, verbose as logVerbose } from "../output.js";
 import { authenticatedClient, parseId, } from "../resource.js";
 import { formatKeyValues, formatTable, humanCell } from "../table.js";
@@ -25,6 +26,7 @@ import { formatKeyValues, formatTable, humanCell } from "../table.js";
 const SALE_ORDER = "sale.order";
 const SALE_ORDER_LINE = "sale.order.line";
 const SUBSCRIPTION_PLAN = "sale.subscription.plan";
+const SUBSCRIPTION_PRICING = "sale.subscription.pricing";
 const STATE_LABELS = {
     draft: "Quotation",
     sent: "Quotation Sent",
@@ -724,6 +726,232 @@ export async function plansListCommand(opts, globals) {
     ]);
     process.stdout.write(formatTable(headers, rows, colorEnabled(globals.noColor)) + "\n");
 }
+// ---------------------------------------------------------------------------
+// plans pricing — recurring prices per pricelist/product/plan
+// (sale.subscription.pricing). Subscription order lines price from these
+// rules, NOT from product.pricelist.item — an order whose pricelist lacks a
+// pricing rule for the plan on the order silently prices the line at 0.
+// ---------------------------------------------------------------------------
+const PRICING_FIELDS = [
+    "id",
+    "pricelist_id",
+    "product_template_id",
+    "plan_id",
+    "price",
+];
+async function resolvePricingRefs(client, opts, globals) {
+    const refs = {};
+    if (opts.pricelist !== undefined) {
+        refs.pricelist = await resolveReference(client, "product.pricelist", opts.pricelist, { label: "pricelist" });
+        logVerbose(`resolved --pricelist "${opts.pricelist}" -> [${refs.pricelist.id}, "${refs.pricelist.name}"]`, globals.verbose);
+    }
+    if (opts.product !== undefined) {
+        refs.product = await resolveReference(client, "product.template", opts.product, { label: "product template" });
+        logVerbose(`resolved --product "${opts.product}" -> [${refs.product.id}, "${refs.product.name}"]`, globals.verbose);
+    }
+    if (opts.plan !== undefined) {
+        refs.plan = await resolveReference(client, SUBSCRIPTION_PLAN, opts.plan, {
+            label: "plan",
+        });
+        logVerbose(`resolved --plan "${opts.plan}" -> [${refs.plan.id}, "${refs.plan.name}"]`, globals.verbose);
+    }
+    return refs;
+}
+export async function plansPricingListCommand(opts, globals) {
+    const client = await authenticatedClient(globals);
+    await ModelFields.load(client, SUBSCRIPTION_PRICING);
+    const limit = parseCount(opts.limit, "--limit");
+    const refs = await resolvePricingRefs(client, opts, globals);
+    const domain = [];
+    if (refs.pricelist)
+        domain.push(["pricelist_id", "=", refs.pricelist.id]);
+    if (refs.product)
+        domain.push(["product_template_id", "=", refs.product.id]);
+    if (refs.plan)
+        domain.push(["plan_id", "=", refs.plan.id]);
+    const records = (await client.executeKw(SUBSCRIPTION_PRICING, "search_read", [
+        domain,
+    ], {
+        fields: PRICING_FIELDS,
+        limit,
+        order: "id asc",
+    }));
+    if (globals.json) {
+        printJson({ model: SUBSCRIPTION_PRICING, records, count: records.length });
+        return;
+    }
+    const headers = ["ID", "Pricelist", "Product", "Plan", "Price"];
+    const rows = records.map((r) => [
+        String(r.id),
+        humanCell(r.pricelist_id),
+        humanCell(r.product_template_id),
+        humanCell(r.plan_id),
+        humanCell(r.price),
+    ]);
+    process.stdout.write(formatTable(headers, rows, colorEnabled(globals.noColor)) + "\n");
+}
+export async function plansPricingCreateCommand(opts, globals) {
+    const client = await authenticatedClient(globals);
+    await ModelFields.load(client, SUBSCRIPTION_PRICING);
+    const refs = await resolvePricingRefs(client, opts, globals);
+    const price = parseDecimal(opts.price, "--price");
+    // Guardrail: one rule per (pricelist, product, plan) — a duplicate would
+    // make line pricing ambiguous.
+    const duplicates = (await client.executeKw(SUBSCRIPTION_PRICING, "search_read", [
+        [
+            ["pricelist_id", "=", refs.pricelist.id],
+            ["product_template_id", "=", refs.product.id],
+            ["plan_id", "=", refs.plan.id],
+        ],
+    ], { fields: ["id", "price"], limit: 5 }));
+    if (duplicates.length > 0) {
+        throw conflictError("duplicate_pricing", `a pricing rule already exists for pricelist ${refs.pricelist.id}, product ${refs.product.id}, plan ${refs.plan.id} (id ${duplicates[0].id}, price ${duplicates[0].price})`, `use 'sales plans pricing update ${duplicates[0].id}' to change it`);
+    }
+    const values = {
+        pricelist_id: refs.pricelist.id,
+        product_template_id: refs.product.id,
+        plan_id: refs.plan.id,
+        price,
+    };
+    if (globals.dryRun) {
+        if (globals.json) {
+            printJson({
+                model: SUBSCRIPTION_PRICING,
+                action: "create",
+                dry_run: true,
+                values,
+            });
+            return;
+        }
+        const lines = [
+            `Would create ${SUBSCRIPTION_PRICING}:`,
+            `  pricelist_id:        [${refs.pricelist.id}, ${JSON.stringify(refs.pricelist.name)}]`,
+            `  product_template_id: [${refs.product.id}, ${JSON.stringify(refs.product.name)}]`,
+            `  plan_id:             [${refs.plan.id}, ${JSON.stringify(refs.plan.name)}]`,
+            `  price:               ${price}`,
+        ];
+        process.stdout.write(lines.join("\n") + "\n");
+        return;
+    }
+    let id;
+    try {
+        id = await client.executeKw(SUBSCRIPTION_PRICING, "create", [values]);
+    }
+    catch (error) {
+        if (!(error instanceof XmlRpcFaultError) &&
+            error.exitCode === 6) {
+            throw networkError(error.message, "the request reached the server; the record may have been created — verify with list before retrying");
+        }
+        if (error instanceof XmlRpcFaultError) {
+            throw conflictError("server_fault", `the server rejected the create (fault ${error.faultCode})`, "retry with --verbose to see the full server response");
+        }
+        throw error;
+    }
+    if (typeof id !== "number" || !Number.isInteger(id)) {
+        throw networkError(`unexpected create() response from the server: ${JSON.stringify(id)}`);
+    }
+    if (globals.json) {
+        printJson({
+            model: SUBSCRIPTION_PRICING,
+            action: "create",
+            dry_run: false,
+            id,
+        });
+        return;
+    }
+    const color = colorEnabled(false) && process.stdout.isTTY === true;
+    const line = `Created ${SUBSCRIPTION_PRICING} ${id}`;
+    process.stdout.write((color ? `\x1b[32m${line}\x1b[0m` : line) + "\n");
+}
+export async function plansPricingUpdateCommand(idArg, opts, globals) {
+    const pricingId = parseId(idArg);
+    const client = await authenticatedClient(globals);
+    await ModelFields.load(client, SUBSCRIPTION_PRICING);
+    const refs = await resolvePricingRefs(client, opts, globals);
+    const values = {};
+    if (refs.pricelist)
+        values.pricelist_id = refs.pricelist.id;
+    if (refs.product)
+        values.product_template_id = refs.product.id;
+    if (refs.plan)
+        values.plan_id = refs.plan.id;
+    if (opts.price !== undefined)
+        values.price = parseDecimal(opts.price, "--price");
+    if (Object.keys(values).length === 0) {
+        throw usageError("nothing to update", "pass at least one of --pricelist, --product, --plan, --price");
+    }
+    // Read-first pass: existence check + diff, mirroring orders update.
+    const rows = (await client.executeKw(SUBSCRIPTION_PRICING, "read", [
+        [pricingId],
+    ], { fields: PRICING_FIELDS }));
+    if (rows.length === 0) {
+        throw notFoundError("record_not_found", `no ${SUBSCRIPTION_PRICING} record with id ${pricingId}`);
+    }
+    const rec = rows[0];
+    const changes = {};
+    for (const [field, after] of Object.entries(values)) {
+        const before = rec[field];
+        const beforeId = Array.isArray(before) ? before[0] : before;
+        if (beforeId !== after) {
+            changes[field] = { before, after };
+        }
+    }
+    if (globals.dryRun) {
+        if (globals.json) {
+            printJson({
+                model: SUBSCRIPTION_PRICING,
+                action: "update",
+                dry_run: true,
+                changes: { [String(pricingId)]: changes },
+            });
+            return;
+        }
+        const lines = [];
+        if (Object.keys(changes).length === 0) {
+            lines.push(`Unchanged ${SUBSCRIPTION_PRICING} ${pricingId} — nothing to change`);
+        }
+        else {
+            lines.push(`Would update ${SUBSCRIPTION_PRICING} ${pricingId}:`);
+            for (const [field, change] of Object.entries(changes)) {
+                lines.push(`  ${field}: ${humanCell(change.before)} -> ${humanCell(change.after)}`);
+            }
+        }
+        process.stdout.write(lines.join("\n") + "\n");
+        return;
+    }
+    if (Object.keys(changes).length > 0) {
+        try {
+            await client.executeKw(SUBSCRIPTION_PRICING, "write", [
+                [pricingId],
+                values,
+            ]);
+        }
+        catch (error) {
+            if (error instanceof XmlRpcFaultError) {
+                throw conflictError("server_fault", `the server rejected the update (fault ${error.faultCode})`, "retry with --verbose to see the full server response");
+            }
+            throw error;
+        }
+    }
+    if (globals.json) {
+        printJson({
+            model: SUBSCRIPTION_PRICING,
+            action: "update",
+            dry_run: false,
+            changes: { [String(pricingId)]: changes },
+        });
+        return;
+    }
+    const color = colorEnabled(false) && process.stdout.isTTY === true;
+    const line = Object.keys(changes).length === 0
+        ? `Unchanged ${SUBSCRIPTION_PRICING} ${pricingId}`
+        : `Updated ${SUBSCRIPTION_PRICING} ${pricingId}`;
+    process.stdout.write((Object.keys(changes).length === 0
+        ? line
+        : color
+            ? `\x1b[33m${line}\x1b[0m`
+            : line) + "\n");
+}
 export async function invoicesPrecedentCommand(opts, globals) {
     const client = await authenticatedClient(globals);
     if (!opts.product) {

package/package.json

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