@wirecat/cli-core 0.572.0 → 0.573.0

package/CHANGELOG.md

@@ -7,6 +7,15 @@ shape of this file.
 
 ## Unreleased
 
+## 0.573.0 — 05.11.2026
+
+### Added
+
+- **The owner sets a confirmed payment's wrong split right.** `store.finance.unallocate` now takes an allocation back
+  from a confirmed payment too, when `by` is the owner; `allocate` then puts the right amounts on, under the same
+  over-payment, over-invoice and currency rules, and the invoices' statuses follow. A bot is still refused on a
+  confirmed payment (`finance.owner-only`); a rejected or deleted payment is `finance.unallocate`.
+
 ## 0.572.0 — 05.11.2026
 
 ### Changed — may break callers

package/dist/errors/reasons.js

@@ -181,7 +181,7 @@ const OTHERS = [
     validation("finance.ref", "the reference names no invoice or payment"),
     validation("finance.status", "the invoice's status does not allow this"),
     validation("finance.total", "the invoice's total is not the sum of its lines"),
-    validation("finance.unallocate", "only a proposed payment's allocations are taken back"),
+    validation("finance.unallocate", "a rejected or deleted payment's allocations are not taken back"),
     validation("record.entity", "the invoice or payment the record stands for was refused"),
     ["record.map-owner-only", "permission_error", "only the owner says which entity a source record is"],
     validation("record.map-type", "a record maps only to an entity of its own type"),

package/dist/store/sqlite/finance.d.ts

@@ -181,8 +181,8 @@ export interface FinanceStore {
         by: Author;
     }): Promise<Allocation>;
     /**
-     * Takes one allocation back from a payment still proposed, under the same rule as `allocate`. A confirmed payment's
-     * allocations are decided money and stay.
+     * Takes one allocation back, under the same rule as `allocate`: the owner from any payment, confirmed too, to set a
+     * wrong split right; a bot only from one it proposed that is still proposed.
      */
     unallocate(input: {
         payment: string;

package/dist/store/sqlite/finance.js

@@ -501,8 +501,8 @@ export const financeOperationsOver = (context) => {
             const row = rowOf(payment, "payment");
             const ref = `payment:${row.id}`;
             mayAllocate(row, by);
-            if (row.review !== "proposed" || row.deleted_at != null)
-                refuse(`${ref} is ${row.deleted_at != null ? "deleted" : String(row.review)}; only a proposed payment's ` +
+            if (row.review === "rejected" || row.deleted_at != null)
+                refuse(`${ref} is ${row.deleted_at != null ? "deleted" : "rejected"}; only a proposed or confirmed payment's ` +
                     "allocations are taken back", "finance.unallocate");
             const target = rowOf(invoice, "invoice");
             write(database, () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wirecat/cli-core",
-  "version": "0.572.0",
+  "version": "0.573.0",
   "description": "The messenger-neutral half of a messaging CLI: domain model, message locators, rendering, name resolution, the SQLite seam, the command skeleton and shared read commands, the send guard, run records and the message store",
   "license": "Apache-2.0",
   "author": "Viacheslav Ptsarev",