@wirecat/cli-core 0.514.0 → 0.516.0

package/CHANGELOG.md

@@ -7,6 +7,44 @@ shape of this file.
 
 ## Unreleased
 
+## 0.516.0 — 01.11.2026
+
+### Fixed
+
+- **`syncMail` counts what `max` left unread in `remaining`**, not 0, so a CLI can say how much is left after a run
+  cut short by `--limit`.
+- **`tasks update --priority none` clears the priority** instead of failing with "a priority score is 0 to 1":
+  Commander stores an option parser's `null` as an empty string. `priorityOption` reads `none` as the word `"none"`
+  for a command's own option; a consumer that maps it to `null` (as `--due none`) clears the priority.
+
+### Added
+
+- **Store version 42: a task's revision goes up only for a change a person could overwrite** (`minCompatible` stays
+  34): its title, description, status, priority, agent, due date, project or parent, an assignee added or removed, or
+  a tag added or removed. A claim taken, renewed or released, a follow-up time, a close suggestion or certainty no
+  longer bump it, so a board no longer reports a conflict while an agent simply works. Triggers do it, so writes by
+  older builds follow the same rule.
+
+### Changed — may break callers
+
+- **`tasks.move` beside a task in another column moves the task into that column**: without `status` it takes the
+  neighbour's status. A `status` that differs from the neighbour's is refused with `validation_error`, reason
+  `task.column` (`HRB-2 is in in_progress, not todo: pick a task in the todo column`), with `anchor`, `anchorStatus`
+  and `requestedStatus` in the details. A neighbour off every project board, or the task itself, is refused with a
+  message that says so.
+
+## 0.515.0 — 01.11.2026
+
+### Added
+
+- **`storedMailRules(store, account, query?)`** reads one account's mail rules from the store in the shape
+  `syncMail` takes as `account.rules`, and `storedMailRule(stored)` reads one; a rule with a condition this build does
+  not know is left out. An account the store has not seen has no rules.
+- **`labelName(raw)`, `systemLabel(name)` and `decodeMailboxName(name)`** name Gmail labels as people read them
+  (`\Inbox` and `Inbox` are one label, modified UTF-7 is decoded), moved from memo.
+- **`mailProgress(store, key)`** gives each synced folder's window, done and total from the saved resume points,
+  without asking the mailbox. It treats only `not_found` as "no account yet" and passes other store errors on.
+
 ## 0.514.0 — 01.11.2026
 
 ### Fixed

package/dist/cli/messenger/tasks-command.js

@@ -8,7 +8,7 @@ import { aheadOf, momentOf } from "../../services/moment.js";
 import { unlistedNote } from "../../services/task-categories.js";
 import { ARRIVAL_WINDOW, applyTaskRulesToChats } from "../../services/task-rules.js";
 import { CLAIM_LEASE_MS, closedStateOf, TASK_PAIR_QUERY, TASK_QUERY, taskLinkKindOf, taskTypeOf, } from "../../services/tasks.js";
-import { PRIORITY_LABELS, priorityOf } from "../../tasks/index.js";
+import { PRIORITY_LABELS, priorityOption } from "../../tasks/index.js";
 import { actorOf, holderOf as holderIn } from "../author.js";
 import { environmentOf } from "../context.js";
 import { destroysNothing, LOCAL_READ, LOCAL_WRITE, ownerOnlyOption } from "../effects.js";
@@ -119,7 +119,7 @@ const HOLDER_HELP = "who you are for claims: a session id or a chat id; default
 const planFrom = (options) => {
     const plan = {};
     if (options.priority !== undefined)
-        plan.priority = options.priority;
+        plan.priority = options.priority === "none" ? null : options.priority;
     if (options.parent !== undefined)
         plan.parent = options.parent === "none" ? null : options.parent;
     if (options.category !== undefined)
@@ -208,7 +208,7 @@ export const tasksCommand = (messenger) => {
         .description("update a task's text, schedule, category, assignees, priority or parent; nothing reaches the messenger")
         .argument("<task>", "the task's id or key, as tasks list shows it")
         .option("--category <name>", `${categoryHelp}; a change is kept in the task's history`)
-        .option("--priority <n>", `${PRIORITY_LABELS.join(", ")} or none; 0 (urgent) … 4 (low) still read as before`, priorityOf)
+        .option("--priority <n>", `${PRIORITY_LABELS.join(", ")} or none; 0 (urgent) … 4 (low) still read as before`, priorityOption)
         .option("--parent <id>", "parent task in the same project; none clears it"), "send Alex the API token").action(async function (id) {
         const context = messengerContext(this, messenger);
         refuseLocalWrite(context, messenger.app.command, "tasks.update");

package/dist/errors/reasons.js

@@ -121,6 +121,7 @@ const OTHERS = [
     validation("task.claimed", "someone else holds the task"),
     validation("task.closed", "the task is already closed"),
     ["task.stale", "configuration_error", "the task changed since it was read; read it again"],
+    validation("task.column", "the task beside which it moves is in another column than the status asked for"),
     validation("task.dependency-cycle", "the task would depend on itself"),
     validation("task.duplicate-candidates", "open tasks may be the same job: attach to one, or add it as new"),
     validation("task.certainty-required", "an agent closes a task with its certainty, 0 to 1"),

package/dist/mail/index.d.ts

@@ -3,11 +3,14 @@ export { factsOf } from "./facts.js";
 export { GMAIL_ALL_MAIL, mailFolders } from "./folders.js";
 export { dropMissing, type Gone, tooManyGone } from "./gone.js";
 export { type AuthVerdicts, HEADER_FIELDS, headersOf, type MailHeaders, mailHeadersOf } from "./headers.js";
+export { decodeMailboxName, labelName, systemLabel } from "./labels.js";
 export { type Address, htmlToText, looksLikeHtml, MAX_BODY_CHARS, type ParsedMail, parseMail, } from "./parse.js";
 export { PRESET_NAMES, PRESETS, type PresetName, presetMatches } from "./presets.js";
 export { domainsToAge, groupKey, type Probe, type ProbeGroup, probeMail, SUGGEST_RULE } from "./probe.js";
+export { type MailFolderProgress, mailProgress } from "./progress.js";
 export { type FileStamp, FULL_PASS_MS, MBOX_RESUME, type MboxPoint, mergeRanges, parsePoint, type Ranges, type ResumePoint, rangesOf, resumeKey, rulesKey, within, } from "./resume.js";
 export { ruleInput, STORE_ACTION } from "./rule-input.js";
+export { storedMailRule, storedMailRules } from "./rule-stored.js";
 export { mapProposedTags, ruleTagger, settleRuleTags, type TagOutcome } from "./rule-tags.js";
 export { conditionKeys, decide, describeMatch, ENVELOPE, envelopeBox, envelopeBoxOf, type MailAction, type MailFacts, type MailRule, type MailRuleMatch, matches, sameMatch, tagsOf, } from "./rules.js";
 export { type DomainAge, FRAUD_SIGNALS, FREE_MAIL, registrableOf, SCORE_RULE, type Scored, type Signal, scoreGroup, WEIGHTS, } from "./score.js";

package/dist/mail/index.js

@@ -3,11 +3,14 @@ export { factsOf } from "./facts.js";
 export { GMAIL_ALL_MAIL, mailFolders } from "./folders.js";
 export { dropMissing, tooManyGone } from "./gone.js";
 export { HEADER_FIELDS, headersOf, mailHeadersOf } from "./headers.js";
+export { decodeMailboxName, labelName, systemLabel } from "./labels.js";
 export { htmlToText, looksLikeHtml, MAX_BODY_CHARS, parseMail, } from "./parse.js";
 export { PRESET_NAMES, PRESETS, presetMatches } from "./presets.js";
 export { domainsToAge, groupKey, probeMail, SUGGEST_RULE } from "./probe.js";
+export { mailProgress } from "./progress.js";
 export { FULL_PASS_MS, MBOX_RESUME, mergeRanges, parsePoint, rangesOf, resumeKey, rulesKey, within, } from "./resume.js";
 export { ruleInput, STORE_ACTION } from "./rule-input.js";
+export { storedMailRule, storedMailRules } from "./rule-stored.js";
 export { mapProposedTags, ruleTagger, settleRuleTags } from "./rule-tags.js";
 export { conditionKeys, decide, describeMatch, ENVELOPE, envelopeBox, envelopeBoxOf, matches, sameMatch, tagsOf, } from "./rules.js";
 export { FRAUD_SIGNALS, FREE_MAIL, registrableOf, SCORE_RULE, scoreGroup, WEIGHTS, } from "./score.js";

package/dist/mail/labels.d.ts

@@ -0,0 +1,5 @@
+/** IMAP's modified UTF-7 (RFC 3501 §5.1.3): `&` opens base64 of UTF-16BE, `,` stands for `/`, `-` closes. */
+export declare const decodeMailboxName: (name: string) => string;
+export declare const systemLabel: (name: string) => string;
+/** A label as people read it: `\\Important` and `Important` are one label, and names are decoded. */
+export declare const labelName: (raw: string) => string;

package/dist/mail/labels.js

@@ -0,0 +1,27 @@
+const SYSTEM_LABELS = {
+    inbox: "Inbox",
+    sent: "Sent",
+    important: "Important",
+    starred: "Starred",
+    draft: "Drafts",
+    drafts: "Drafts",
+    spam: "Spam",
+    trash: "Trash",
+    junk: "Spam",
+};
+/** IMAP's modified UTF-7 (RFC 3501 §5.1.3): `&` opens base64 of UTF-16BE, `,` stands for `/`, `-` closes. */
+export const decodeMailboxName = (name) => name.replace(/&([^-]*)-/g, (_, run) => {
+    if (run === "")
+        return "&";
+    const bytes = Buffer.from(run.replace(/,/g, "/"), "base64");
+    let text = "";
+    for (let at = 0; at + 1 < bytes.length; at += 2)
+        text += String.fromCharCode(bytes.readUInt16BE(at));
+    return text;
+});
+export const systemLabel = (name) => SYSTEM_LABELS[name.toLowerCase()] ?? name;
+/** A label as people read it: `\\Important` and `Important` are one label, and names are decoded. */
+export const labelName = (raw) => {
+    const system = raw.startsWith("\\") ? raw.slice(1) : raw;
+    return SYSTEM_LABELS[system.toLowerCase()] ?? decodeMailboxName(system);
+};

package/dist/mail/progress.d.ts

@@ -0,0 +1,10 @@
+import type { AccountKey, MessageStore } from "../store/index.js";
+export interface MailFolderProgress {
+    folder: string;
+    since: string;
+    done: number;
+    total: number;
+    updatedAt: string;
+}
+/** How far the sync of each stored folder is, from the points the runs saved; the mailbox is not asked. */
+export declare const mailProgress: (store: MessageStore, key: AccountKey) => Promise<MailFolderProgress[]>;

package/dist/mail/progress.js

@@ -0,0 +1,29 @@
+import { CliError } from "@wirecat/cli-base";
+import { parsePoint, resumeKey } from "./resume.js";
+/** How far the sync of each stored folder is, from the points the runs saved; the mailbox is not asked. */
+export const mailProgress = async (store, key) => {
+    let accountId;
+    try {
+        accountId = (await store.accounts.storedAccount(key)).id;
+    }
+    catch (error) {
+        if (error instanceof CliError && error.code === "not_found")
+            return [];
+        throw error;
+    }
+    const folders = [];
+    for (const { kind, externalId: folder } of await store.mail.mailboxes(accountId)) {
+        if (kind !== null)
+            continue;
+        const point = parsePoint((await store.sync.syncState(key, resumeKey(folder)))?.value);
+        if (point !== undefined)
+            folders.push({
+                folder,
+                since: point.window.since,
+                done: point.window.done,
+                total: point.window.total,
+                updatedAt: point.updatedAt,
+            });
+    }
+    return folders;
+};

package/dist/mail/quick.d.ts

@@ -33,6 +33,8 @@ export interface QuickRead {
     }[];
     /** Mail of the folder is left for a later round or run. */
     capped: boolean;
+    /** How many of the open mail `max` left for the next run; an unbudgeted run only. */
+    left: number;
 }
 /**
  * One round of a folder whose server gives no cheap message id: UID, flags, date and size of what is not done, then

package/dist/mail/quick.js

@@ -7,7 +7,7 @@ export const keyOf = (folder, uid) => `${folder}\0${uid}`;
  * the raw bytes of the newest of it, whose Message-ID gives the id the full listing would have given.
  */
 export const readQuick = async (source, since, folder, searched, done, max, plan, located) => {
-    const answer = { listing: [], read: [], unreadable: [], capped: false };
+    const answer = { listing: [], read: [], unreadable: [], capped: false, left: 0 };
     let items = plan?.quick.get(folder);
     if (items === undefined) {
         // Mail whose location is known is listed for its flags only; it is never read again.
@@ -40,6 +40,8 @@ export const readQuick = async (source, since, folder, searched, done, max, plan
     if (plan !== undefined)
         plan.remaining += open.length - slice.length;
     answer.capped = open.length > slice.length;
+    if (plan === undefined)
+        answer.left = open.length - slice.length;
     if (slice.length === 0)
         return answer;
     if (plan !== undefined) {

package/dist/mail/rule-stored.d.ts

@@ -0,0 +1,9 @@
+import type { QueryAst } from "@wirecat/cli-base/query";
+import type { MessageStore, MailRule as StoredRule } from "../store/index.js";
+import type { MailRule } from "./rules.js";
+/** A stored rule as the sync reads it; `undefined` for one this build cannot read, so it is left to the build that can. */
+export declare const storedMailRule: (stored: StoredRule) => MailRule | undefined;
+/** One account's rules from the store, for `account.rules` of the sync; none when the store has not seen the account. */
+export declare const storedMailRules: (store: MessageStore, account: {
+    address: string;
+}, query?: QueryAst) => Promise<MailRule[]>;

package/dist/mail/rule-stored.js

@@ -0,0 +1,57 @@
+import { CliError } from "@wirecat/cli-base";
+import { PRESET_NAMES } from "./presets.js";
+const ACTION_OF = { full: "keep", envelope: "envelope", skip: "skip" };
+const KINDS = {
+    from: "string",
+    from_contains: "string",
+    domain: "string",
+    to: "string",
+    list_id: "string",
+    subject: "string",
+    older_than_days: "number",
+    larger_than_bytes: "number",
+    has_attachments: "boolean",
+    label: "string",
+    unread: "boolean",
+    bulk: "boolean",
+    score_at_least: "number",
+    preset: "string",
+};
+/** A stored rule as the sync reads it; `undefined` for one this build cannot read, so it is left to the build that can. */
+export const storedMailRule = (stored) => {
+    const action = ACTION_OF[String(stored.actions.store)];
+    const entries = Object.entries(stored.conditions);
+    const known = entries.every(([key, value]) => key in KINDS &&
+        typeof value === KINDS[key] &&
+        !(key === "preset" && !PRESET_NAMES.includes(value)));
+    if (action === undefined || !known || entries.length === 0)
+        return undefined;
+    const tags = Array.isArray(stored.actions.tags) ? stored.actions.tags.filter((tag) => typeof tag === "string") : [];
+    const skipped = stored.actions.tags_skipped;
+    const tagsSkipped = skipped !== null && typeof skipped === "object" && !Array.isArray(skipped)
+        ? Object.fromEntries(Object.entries(skipped).filter(([, why]) => typeof why === "string"))
+        : {};
+    return {
+        id: stored.id,
+        match: Object.fromEntries(entries),
+        action,
+        ...(tags.length ? { tags: tags } : {}),
+        ...(Object.keys(tagsSkipped).length ? { tags_skipped: tagsSkipped } : {}),
+        ...(stored.reason === null ? {} : { reason: stored.reason }),
+        ...(stored.protected ? { protected: true } : {}),
+        ...(stored.source === "owner" ? {} : { source: stored.source }),
+        decided_at: new Date(stored.decidedAt).toISOString(),
+    };
+};
+/** One account's rules from the store, for `account.rules` of the sync; none when the store has not seen the account. */
+export const storedMailRules = async (store, account, query) => {
+    try {
+        const { id } = await store.accounts.storedAccount({ provider: "email", account: account.address.toLowerCase() });
+        return (await store.mail.rules.list(id, query)).flatMap((stored) => storedMailRule(stored) ?? []);
+    }
+    catch (error) {
+        if (error instanceof CliError && error.code === "not_found")
+            return [];
+        throw error;
+    }
+};

package/dist/mail/sync.d.ts

@@ -36,7 +36,7 @@ export interface MailSyncResult {
     state?: "partial" | "complete";
     /** Why a budgeted run stopped early: its request budget, its deadline, or a cancel. */
     stopped?: BudgetStop;
-    /** Messages of the window this run did not get to read; a budgeted run only. */
+    /** Messages of the window this run did not get to read: left by a budget, or by `max`. */
     remaining?: number;
     /** Requests a budgeted run spent. */
     requests?: number;

package/dist/mail/sync.js

@@ -121,7 +121,7 @@ export const syncMail = async (source, store, options) => {
         deleted: sum((round) => round.deleted),
         complete,
         state: complete ? "complete" : "partial",
-        remaining,
+        remaining: remaining + (last.remaining ?? 0),
         requests: budget.used,
         ...(plan.stopped === null ? {} : { stopped: plan.stopped }),
         ...(indexed === undefined ? {} : { indexed }),
@@ -183,6 +183,7 @@ const syncRound = async (source, store, { account, since, max = 200, maxBodyChar
             mboxPoint = point;
     }
     let capped = false;
+    let overMax = 0;
     const unreadable = [];
     /** Raw mail read to learn its id, kept for the save so it is not read twice. */
     const rawOf = new Map();
@@ -258,6 +259,7 @@ const syncRound = async (source, store, { account, since, max = 200, maxBodyChar
                     refreshed.set(id, { ...refreshed.get(id), ...flags });
                 if (round.capped)
                     capped = true;
+                overMax += round.left;
                 for (const { item, raw } of round.read) {
                     collected.push(item);
                     rawOf.set(item, raw);
@@ -465,8 +467,10 @@ const syncRound = async (source, store, { account, since, max = 200, maxBodyChar
                 upgraded.set(item.messageId, boxes);
                 pending.push(item);
             }
-            else
+            else {
                 full = true;
+                overMax++;
+            }
             continue;
         }
         if (known.has(item.messageId)) {
@@ -491,8 +495,10 @@ const syncRound = async (source, store, { account, since, max = 200, maxBodyChar
         }
         if (pending.length < max)
             pending.push(item);
-        else
+        else {
             full = true;
+            overMax++;
+        }
     }
     const inMailboxes = (item) => [
         ...[...(foundFolders.get(item.messageId) ?? [])].map(mailbox),
@@ -794,6 +800,7 @@ const syncRound = async (source, store, { account, since, max = 200, maxBodyChar
         envelopes,
         deleted,
         complete,
+        remaining: overMax,
         ...(indexed === undefined ? {} : { indexed }),
         coverage,
         ...(attachments.length ? { attachments } : {}),

package/dist/store/index.d.ts

@@ -5,7 +5,7 @@ export type { TagRedirectIndex, TagRedirectResolution, TagRedirectSnapshot } fro
 export { createTagRedirectIndex } from "../domain/tag-redirects.js";
 export { note } from "../entities/note.js";
 export type { FindingEvidence, FindingOutcome, MeetingFactFinding, MeetingFinding, MeetingFindingsInput, MeetingFindingsReceipt, MeetingTaskFinding, } from "../meetings/index.js";
-export { CLOSED_STATES, isClosedState, PRIORITY_LABELS, type PriorityLabel, priorityLabel, priorityOf, TASK_STATES, type TaskState, } from "../tasks/index.js";
+export { CLOSED_STATES, isClosedState, PRIORITY_LABELS, type PriorityLabel, priorityLabel, priorityOf, priorityOption, TASK_STATES, type TaskState, } from "../tasks/index.js";
 export type { AccountMail, AccountMeetings, AccountStore } from "./account-store.js";
 export type { DocumentTransferOptions, DocumentTransferPreview, DocumentTransferReceipt, } from "./document-transfer/index.js";
 export { applyDocumentTransfer, previewDocumentTransfer } from "./document-transfer/index.js";

package/dist/store/index.js

@@ -1,7 +1,7 @@
 export { FACT_STATUSES, FACT_TYPES, MEMORY_KINDS, MEMORY_SCOPES, MEMORY_STATUSES, TASK_TYPES, WORK_TASK_TYPES, } from "../domain/knowledge.js";
 export { createTagRedirectIndex } from "../domain/tag-redirects.js";
 export { note } from "../entities/note.js";
-export { CLOSED_STATES, isClosedState, PRIORITY_LABELS, priorityLabel, priorityOf, TASK_STATES, } from "../tasks/index.js";
+export { CLOSED_STATES, isClosedState, PRIORITY_LABELS, priorityLabel, priorityOf, priorityOption, TASK_STATES, } from "../tasks/index.js";
 export { applyDocumentTransfer, previewDocumentTransfer } from "./document-transfer/index.js";
 export { PRAGMAS } from "./driver.js";
 export { EVIDENCE_FINDINGS_KIND, validateEvidenceFindings, } from "./evidence-findings.js";

package/dist/store/sqlite/manifest.js

@@ -55,6 +55,7 @@ export const MANIFEST = [
     { name: "20261030114008_version-39-email-locations", version: 39, minCompatible: 34 },
     { name: "20261030225520_version-40-note-trigrams", version: 40, minCompatible: 34 },
     { name: "20261031230945_version-41-task-fields", version: 41, minCompatible: 34 },
+    { name: "20261101002523_version-42-task-revision", version: 42, minCompatible: 34 },
 ];
 export const generatedMigrations = (manifest = MANIFEST, generated = GENERATED) => {
     const migrations = [];

package/dist/store/sqlite/migrations.generated.js

@@ -859,5 +859,16 @@ export const GENERATED = [
             "CREATE TRIGGER tasks_priority_score_au AFTER UPDATE OF priority ON tasks WHEN new.priority IS NOT old.priority AND new.priority_score IS old.priority_score BEGIN\n  UPDATE tasks SET priority_score = CASE new.priority WHEN 0 THEN 0.875 WHEN 1 THEN 0.625 WHEN 2 THEN 0.375 WHEN 3 THEN 0.1875 WHEN 4 THEN 0.0625 END WHERE id = new.id;\nEND;",
             "-- Every build's write bumps it, older ones too, so a stale copy is caught whoever wrote last.\nCREATE TRIGGER tasks_revision_au AFTER UPDATE ON tasks WHEN new.revision IS old.revision BEGIN\n  UPDATE tasks SET revision = old.revision + 1 WHERE id = new.id;\nEND;"
         ]
+    },
+    {
+        "name": "20261101002523_version-42-task-revision",
+        "statements": [
+            "-- A claim renewed, a follow-up time or a close suggestion is an agent at work, not a change a person could overwrite;\n-- bumping on those made the board report a conflict while an agent simply worked.\nDROP TRIGGER tasks_revision_au;",
+            "CREATE TRIGGER tasks_revision_au AFTER UPDATE OF title, description, status, priority, priority_score, agent, due_at, project_id, parent_id ON tasks\n  WHEN new.revision IS old.revision AND (new.title IS NOT old.title OR new.description IS NOT old.description\n    OR new.status IS NOT old.status OR new.priority IS NOT old.priority OR new.priority_score IS NOT old.priority_score\n    OR new.agent IS NOT old.agent OR new.due_at IS NOT old.due_at OR new.project_id IS NOT old.project_id\n    OR new.parent_id IS NOT old.parent_id) BEGIN\n  UPDATE tasks SET revision = old.revision + 1 WHERE id = new.id;\nEND;",
+            "CREATE TRIGGER tasks_revision_assignment_ai AFTER INSERT ON task_assignments BEGIN\n  UPDATE tasks SET revision = revision + 1 WHERE id = new.task_id;\nEND;",
+            "CREATE TRIGGER tasks_revision_assignment_ad AFTER DELETE ON task_assignments BEGIN\n  UPDATE tasks SET revision = revision + 1 WHERE id = old.task_id;\nEND;",
+            "CREATE TRIGGER tasks_revision_tagging_ai AFTER INSERT ON taggings WHEN new.taggable_type = 'task' BEGIN\n  UPDATE tasks SET revision = revision + 1 WHERE id = new.taggable_id;\nEND;",
+            "CREATE TRIGGER tasks_revision_tagging_ad AFTER DELETE ON taggings WHEN old.taggable_type = 'task' BEGIN\n  UPDATE tasks SET revision = revision + 1 WHERE id = old.taggable_id;\nEND;"
+        ]
     }
 ];

package/dist/store/sqlite/work-tasks.js

@@ -239,20 +239,27 @@ export const workTaskOperationsOver = (context) => {
             throw new CliError("validation_error", "give --before or --after, not both");
         if (!database.prepare(`SELECT 1 FROM tasks t WHERE t.id = ? AND ${BOARD_SCOPE}`).get(task))
             throw new CliError("validation_error", "only a project task moves on the board");
-        const status = move.status ?? row.status;
-        const column = columnOf(task, status);
         const neighbour = move.before ?? move.after;
         if (neighbour === undefined) {
-            if (column.length > 0 || move.status === undefined)
+            const status = move.status ?? row.status;
+            if (columnOf(task, status).length > 0 || move.status === undefined)
                 throw new CliError("validation_error", "say where it goes: --before <key> or --after <key>");
             const from = priorityLabel(row.priority_score == null ? null : Number(row.priority_score));
             edit(task, { status: move.status }, actor);
             return { label: { from, to: from, changed: false }, renumbered: [] };
         }
         const target = required(neighbour);
+        const anchor = database.prepare(`SELECT t.key, t.status FROM tasks t WHERE t.id = ? AND ${BOARD_SCOPE}`).get(target);
+        if (target === task)
+            throw new CliError("validation_error", "a task moves beside another task, not itself");
+        if (!anchor)
+            throw new CliError("validation_error", `${neighbour} is not on a project's board`);
+        // A card dropped beside one in another column joins that column; a status that names a different one is a mistake.
+        const status = anchor.status;
+        if (move.status !== undefined && move.status !== status)
+            throw new CliError("validation_error", `${String(anchor.key)} is in ${status}, not ${move.status}: pick a task in the ${move.status} column`, { reason: "task.column", anchor: String(anchor.key), anchorStatus: status, requestedStatus: move.status });
+        const column = columnOf(task, status);
         const at = column.findIndex(({ id }) => id === target);
-        if (at === -1)
-            throw new CliError("validation_error", `${move.before ?? move.after} is not a ${status} task of a project; a task moves beside one in its column`);
         const index = move.before === undefined ? at + 1 : at;
         const above = column[index - 1];
         const below = column[index];
@@ -285,7 +292,7 @@ export const workTaskOperationsOver = (context) => {
                 renumbered.push(one.key);
             });
         }
-        edit(task, { priority: given, ...(move.status === undefined ? {} : { status: move.status }) }, actor);
+        edit(task, { priority: given, status }, actor);
         const to = priorityLabel(given);
         return { label: { from, to, changed: from !== to }, renumbered };
     };

package/dist/tasks/index.d.ts

@@ -1,4 +1,4 @@
 export { CLOSED_STATES, type ClosedState, closeTask, isClosedState, TASK_KINDS, TASK_ORIGINS, TASK_STATES, TASK_VERDICTS, type Task, type TaskKind, type TaskOrigin, type TaskState, type TaskVerdict, } from "./model.js";
-export { checkedScore, LEGACY_PRIORITY_SCORES, legacyPriority, PRIORITY_LABELS, type PriorityLabel, priorityBand, priorityLabel, priorityOf, } from "./priority.js";
+export { checkedScore, LEGACY_PRIORITY_SCORES, legacyPriority, PRIORITY_LABELS, type PriorityLabel, priorityBand, priorityLabel, priorityOf, priorityOption, } from "./priority.js";
 export { type CloseOptions, createTaskService, type GroupStats, type NewTask, type TaskService, type TaskServiceOptions, } from "./service.js";
 export { matches, type TaskFilter, type TaskStore } from "./store.js";

package/dist/tasks/index.js

@@ -1,4 +1,4 @@
 export { CLOSED_STATES, closeTask, isClosedState, TASK_KINDS, TASK_ORIGINS, TASK_STATES, TASK_VERDICTS, } from "./model.js";
-export { checkedScore, LEGACY_PRIORITY_SCORES, legacyPriority, PRIORITY_LABELS, priorityBand, priorityLabel, priorityOf, } from "./priority.js";
+export { checkedScore, LEGACY_PRIORITY_SCORES, legacyPriority, PRIORITY_LABELS, priorityBand, priorityLabel, priorityOf, priorityOption, } from "./priority.js";
 export { createTaskService, } from "./service.js";
 export { matches } from "./store.js";

package/dist/tasks/priority.d.ts

@@ -16,3 +16,5 @@ export declare const checkedScore: (score: number) => number;
  * `none` clears it.
  */
 export declare const priorityOf: (value: string) => number | null;
+/** `priorityOf` for a command-line option: Commander stores a parser's `null` as an empty string, so `none` stays a word. */
+export declare const priorityOption: (value: string) => number | "none";

package/dist/tasks/priority.js

@@ -35,3 +35,5 @@ export const priorityOf = (value) => {
         return LEGACY_PRIORITY_SCORES[Number(word)];
     throw new CliError("validation_error", `a priority is ${PRIORITY_LABELS.join(", ")} or none; 0 (urgent) … 4 (low) still work as critical, high, medium, low, low`);
 };
+/** `priorityOf` for a command-line option: Commander stores a parser's `null` as an empty string, so `none` stays a word. */
+export const priorityOption = (value) => priorityOf(value) ?? "none";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wirecat/cli-core",
-  "version": "0.514.0",
+  "version": "0.516.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",