@wirecat/cli-core 0.571.0 → 0.573.0

package/CHANGELOG.md

@@ -7,6 +7,24 @@ 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
+
+- **`facts.unconfirm` is now `facts.revoke`**, its refusal reason `fact.unconfirm-correction` is
+  `fact.revoke-correction`, and the step it writes into a fact's `metadata.history` is `revoked`. memo's command is
+  `memo facts revoke`, a word the shared vocabulary has ("take back what was granted"); `unconfirm` is not in it. Nothing
+  called the old names but memo's unreleased command, which moves with this.
+
 ## 0.571.0 — 05.11.2026
 
 ### Added

package/dist/errors/reasons.js

@@ -164,10 +164,10 @@ const OTHERS = [
     validation("input.required", "the command needs an answer and cannot ask for one"),
     validation("input.too-large", "the input is over its size limit"),
     ["meeting-project.stale", "configuration_error", "the meeting's project changed meanwhile"],
-    validation("fact.not-accepted", "only an accepted fact can be unconfirmed"),
-    validation("fact.owner-written", "the owner wrote this fact; it was never proposed, so it cannot be unconfirmed"),
+    validation("fact.not-accepted", "only an accepted fact can be revoked"),
+    validation("fact.owner-written", "the owner wrote this fact; it was never proposed, so it cannot be revoked"),
     validation("fact.payment-retired", "a payment is a payments row in store.finance, not a fact"),
-    validation("fact.unconfirm-correction", "an accepted correction cannot be unconfirmed; its superseded fact stays"),
+    validation("fact.revoke-correction", "an accepted correction cannot be revoked; its superseded fact stays"),
     validation("finance.allocate", "the invoice or payment cannot take an allocation in its state"),
     validation("finance.currency", "the payment and the invoice are in different currencies; nothing is converted"),
     validation("finance.field", "a field of the invoice or payment is not valid"),
@@ -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/facts.d.ts

@@ -74,7 +74,7 @@ export interface FactsStore {
      * The owner takes back the approval of an accepted fact an agent proposed: it waits as `proposed` again. The step is
      * kept in `metadata.history`, with the decision it took back.
      */
-    unconfirm(reference: string, by: Decider, reason?: string): Promise<Fact>;
+    revoke(reference: string, by: Decider, reason?: string): Promise<Fact>;
     /** The owner, or an agent for the owner, turns down a proposed fact; it stays, marked `rejected`. */
     reject(reference: string, by: Decider, reason?: string): Promise<Fact>;
     /**

package/dist/store/sqlite/facts.js

@@ -154,24 +154,24 @@ export const factOperationsOver = (context) => {
             database.prepare("UPDATE facts SET status = 'reversed', updated_at = ? WHERE id = ?").run(now(), Number(row.id));
             return factOf(rowOf(reference));
         },
-        unconfirm: (reference, by, reason) => {
+        revoke: (reference, by, reason) => {
             const row = rowOf(reference);
             const ref = `fact:${row.id}`;
             assertMayWithdraw({ id: ref, proposedBy: { type: "unknown" }, proposed: false }, by);
             write(database, () => {
                 const current = database.prepare("SELECT * FROM facts WHERE id = ?").get(Number(row.id));
                 if (current.status !== "accepted")
-                    throw new CliError("validation_error", `only an accepted fact can be unconfirmed; ${ref} is ${String(current.status)}`, { reason: "fact.not-accepted" });
+                    throw new CliError("validation_error", `only an accepted fact can be revoked; ${ref} is ${String(current.status)}`, { reason: "fact.not-accepted" });
                 if (current.author_type === "person")
                     throw new CliError("validation_error", `${ref} is the owner's own fact, never proposed; correct it with a fact that supersedes it, or remove it`, { reason: "fact.owner-written" });
                 // Accepting it marked the fact it corrects superseded, and what that one was before is not kept.
                 if (current.supersedes_id != null)
                     throw new CliError("validation_error", `${ref} corrects fact:${current.supersedes_id}, which stopped holding when ${ref} was accepted; ` +
-                        "remove it or correct it with another fact instead", { reason: "fact.unconfirm-correction" });
+                        "remove it or correct it with another fact instead", { reason: "fact.revoke-correction" });
                 const at = now();
                 const { decision, history, ...metadata } = (current.metadata == null ? {} : JSON.parse(String(current.metadata)));
                 const step = {
-                    step: "unconfirmed",
+                    step: "revoked",
                     by: "owner",
                     at: iso(at),
                     confirmedBy: { type: current.confirmed_by_type, id: String(current.confirmed_by_id) },
@@ -236,7 +236,7 @@ export const factsStoreOver = (context) => {
         list: async (options) => operations.list(options),
         accept: async (reference, by, reason) => operations.accept(reference, by, reason),
         reverse: async (reference, by) => operations.reverse(reference, by),
-        unconfirm: async (reference, by, reason) => operations.unconfirm(reference, by, reason),
+        revoke: async (reference, by, reason) => operations.revoke(reference, by, reason),
         reject: async (reference, by, reason) => operations.reject(reference, by, reason),
         remove: async (reference, by) => operations.remove(reference, by),
         sources: async (reference) => operations.sources(reference),

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.571.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",