@wirecat/cli-core 0.529.0 → 0.531.0

package/dist/files/safe-name.js

@@ -0,0 +1,18 @@
+import { basename } from "node:path";
+/**
+ * A name another person chose cannot climb out of the folder, cannot be a dot file we would then
+ * hide, and carries no control or direction character to rewrite the terminal or disguise its extension.
+ */
+export const safeName = (name, platform = process.platform) => {
+    let plain = name === undefined
+        ? ""
+        : basename(name.replaceAll("\\", "/"))
+            .replace(/[\p{Cc}​-‏‪-‮⁦-⁩]/gu, "")
+            .replace(/^\.+/, "");
+    if (platform === "win32") {
+        plain = plain.replace(/[<>:"|?*]/g, "_").replace(/[. ]+$/, "");
+        if (/^(con|prn|aux|nul|com[1-9]|lpt[1-9])(?:\.|$)/i.test(plain))
+            return undefined;
+    }
+    return plain === "" ? undefined : plain;
+};

package/dist/mail/account-key.d.ts

@@ -0,0 +1,2 @@
+import type { AccountKey } from "../store/index.js";
+export declare const mailAccountKey: (address: string) => AccountKey;

package/dist/mail/account-key.js

@@ -0,0 +1 @@
+export const mailAccountKey = (address) => ({ provider: "email", account: address.toLowerCase() });

package/dist/mail/cli/index.js

@@ -84,9 +84,9 @@ const searchCommand = (mail) => {
         partial: true,
         output: { entity: "message", view: "hit" },
         examples: [
-            `${mail.app.command} ya search messages invoice`,
-            `${mail.app.command} ya search messages 'from:ann@example.com has:attachment date>30d'`,
-            `${mail.app.command} ya search messages 'list:github.com is:unread' --newest`,
+            `${mail.app.command} personal search messages invoice`,
+            `${mail.app.command} personal search messages 'from:ann@example.com has:attachment date>30d'`,
+            `${mail.app.command} personal search messages 'list:github.com is:unread' --newest`,
         ],
     })
         .summary(mail.server ? "find emails by words and fields" : "find stored emails by words and fields")
@@ -103,7 +103,7 @@ const searchCommand = (mail) => {
         .option("--timezone <zone>", "the IANA timezone for calendar date boundaries")
         .addHelpText("after", `
 Fields: from: to: cc: bcc: subject: body: mailbox: (or label:) list: (a List-Id or its domain) is:unread|read|starred
-has:attachment, filename:, content: (the text of attachments), size>, date: — \`${mail.app.command} ya search messages 'content:invoice'\`.
+has:attachment, filename:, content: (the text of attachments), size>, date: — \`${mail.app.command} personal search messages 'content:invoice'\`.
 chat: names a thread by its subject. Mail sent to many is in the mailbox bulk; NOT mailbox:bulk leaves it out.
 Mail from senders with signs of fraud comes after the rest of the page; --plain-order keeps it in place.
 --json: the list is items; each hit's reference is ref (msg:email/…), which messages show reads.${mail.server
@@ -194,8 +194,8 @@ const messagesCommand = (mail) => {
     annotate(messages.command("show"), {
         effects: LOCAL_READ,
         examples: [
-            `${mail.app.command} ya messages show 'msg:email/me%40example.com/thread-1/%3Cid%40example.com%3E'`,
-            `${mail.app.command} ya messages show 'msg:email/me%40example.com/thread-1/%3Cid%40example.com%3E' --before-n 2`,
+            `${mail.app.command} personal messages show 'msg:email/me%40example.com/thread-1/%3Cid%40example.com%3E'`,
+            `${mail.app.command} personal messages show 'msg:email/me%40example.com/thread-1/%3Cid%40example.com%3E' --before-n 2`,
         ],
     })
         .summary("one stored email whole, with the text of its attachments")
@@ -243,8 +243,8 @@ const conversationsCommand = (mail) => {
     const list = withQuery(annotate(conversations.command("list"), {
         effects: LOCAL_READ,
         examples: [
-            `${mail.app.command} ya conversations list`,
-            `${mail.app.command} ya conversations list subject:invoice emails_count>1`,
+            `${mail.app.command} personal conversations list`,
+            `${mail.app.command} personal conversations list subject:invoice emails_count>1`,
         ],
     })
         .summary("stored email threads, newest first")
@@ -291,8 +291,8 @@ const conversationsCommand = (mail) => {
     annotate(conversations.command("show"), {
         effects: LOCAL_READ,
         examples: [
-            `${mail.app.command} ya conversations show 'chat:email/me%40example.com/thread-1'`,
-            `${mail.app.command} ya conversations show 'msg:email/me%40example.com/thread-1/%3Cid%40example.com%3E'`,
+            `${mail.app.command} personal conversations show 'chat:email/me%40example.com/thread-1'`,
+            `${mail.app.command} personal conversations show 'msg:email/me%40example.com/thread-1/%3Cid%40example.com%3E'`,
         ],
     })
         .summary("one email thread, every email oldest first")

package/dist/mail/index.d.ts

@@ -1,3 +1,4 @@
+export { mailAccountKey } from "./account-key.js";
 export { BULK, bulkBox } from "./bulk.js";
 export { factsOf } from "./facts.js";
 export { GMAIL_ALL_MAIL, mailFolders } from "./folders.js";

package/dist/mail/index.js

@@ -1,3 +1,4 @@
+export { mailAccountKey } from "./account-key.js";
 export { BULK, bulkBox } from "./bulk.js";
 export { factsOf } from "./facts.js";
 export { GMAIL_ALL_MAIL, mailFolders } from "./folders.js";

package/dist/mail/retry.js

@@ -1,4 +1,5 @@
 import { CliError } from "@wirecat/cli-base";
+import { mailAccountKey } from "./account-key.js";
 import { addressOf, FETCH_CHUNK, readAttachments } from "./attachments.js";
 import { mailFolders } from "./folders.js";
 import { mailHeadersOf } from "./headers.js";
@@ -12,7 +13,7 @@ export const RETRYABLE_ATTACHMENTS = ["engine-missing"];
  */
 export const retryUnread = async ({ store, source, account, since, max, maxBodyChars, maxAttachmentChars, now, loadEngine, index, }) => {
     const startedAt = now();
-    const key = { provider: "email", account: account.address.toLowerCase() };
+    const key = mailAccountKey(account.address);
     const find = source.find;
     if (find === undefined)
         throw new CliError("validation_error", "--retry-attachments reads mail again by its Gmail id; an mbox file or a plain IMAP account has none");

package/dist/mail/rule-stored.js

@@ -1,4 +1,5 @@
 import { CliError } from "@wirecat/cli-base";
+import { mailAccountKey } from "./account-key.js";
 import { PRESET_NAMES } from "./presets.js";
 const ACTION_OF = { full: "keep", envelope: "envelope", skip: "skip" };
 const KINDS = {
@@ -56,7 +57,7 @@ export const storedMailRule = (stored) => {
 export const storedMailRuleSet = async (store, account, query) => {
     let stored;
     try {
-        const { id } = await store.accounts.storedAccount({ provider: "email", account: account.address.toLowerCase() });
+        const { id } = await store.accounts.storedAccount(mailAccountKey(account.address));
         stored = await store.mail.rules.list(id, query);
     }
     catch (error) {

package/dist/mail/sender-scores.js

@@ -1,4 +1,5 @@
 import { CliError } from "@wirecat/cli-base";
+import { mailAccountKey } from "./account-key.js";
 import { domainsToAge, groupKey, probeMail } from "./probe.js";
 import { decide } from "./rules.js";
 import { FRAUD_SIGNALS, FREE_MAIL, registrableOf, WEIGHTS } from "./score.js";
@@ -18,7 +19,7 @@ const RECENT = 2 * 365 * DAY;
  * looked up first, for senders heard from in the last two years.
  */
 export const scoreSenders = async (store, address, ages, now = Date.now(), rules = []) => {
-    const key = { provider: "email", account: address.toLowerCase() };
+    const key = mailAccountKey(address);
     const last = (await store.sync.syncState(key, SCORED))?.value;
     const done = last === undefined ? undefined : JSON.parse(last);
     if (done !== undefined && done.version === SCORE_VERSION && now - done.at < DAY)

package/dist/mail/sync.js

@@ -2,6 +2,7 @@ import { setTimeout } from "node:timers/promises";
 import { CliError } from "@wirecat/cli-base";
 import { importEngine } from "../attachments/extract.js";
 import { isBusy } from "../store/sqlite/transaction.js";
+import { mailAccountKey } from "./account-key.js";
 import { FETCH_CHUNK, MAX_ATTACHMENT_CHARS, readAttachments } from "./attachments.js";
 import { BULK, bulkBox } from "./bulk.js";
 import { parsedEmail } from "./email-input.js";
@@ -108,7 +109,7 @@ export const syncMail = async (source, store, options) => {
         })),
     ];
     const complete = last.complete && plan.stopped === null && unreadable.length === 0 && remaining === 0;
-    const account = { provider: "email", account: options.account.address.toLowerCase() };
+    const account = mailAccountKey(options.account.address);
     const indexed = plan.stopped === null ? await index?.(store, account) : undefined;
     const deletionsSkipped = rounds.map((round) => round.deletionsSkipped).findLast((text) => text !== undefined);
     const { unreadable: _unreadable, ...rest } = last;
@@ -155,7 +156,7 @@ const syncRound = async (source, store, { account, since, max = 200, maxBodyChar
             index,
         });
     const startedAt = now();
-    const key = { provider: "email", account: account.address.toLowerCase() };
+    const key = mailAccountKey(account.address);
     const accountId = await store.accounts.saveAccount(key, { name: account.address });
     const folders = await mailFolders(account, source);
     if (folders.length < 1 || folders.length > 50)

package/dist/ports/messenger.d.ts

@@ -272,6 +272,7 @@ export interface ForumControl {
     enableForum(chatId: Id): Promise<ForumState>;
     createTopic(chatId: Id, title: string, options: {
         sendId: string;
+        icon?: string;
     }): Promise<Topic>;
 }
 export interface TopicEditing {
@@ -281,6 +282,8 @@ export interface TopicEditing {
     orderPinnedTopics(chatId: Id, topicIds: Id[]): Promise<void>;
     /** Deletes the topic and every message in it, for everyone; it cannot be undone. A topic that is gone is `not_found`. */
     deleteTopic?(chatId: Id, topicId: Id): Promise<void>;
+    /** The emojis any account may give a topic as its icon. */
+    topicIcons?(): Promise<string[]>;
 }
 export interface ThreadAddressing {
     validateThread(chatId: Id, threadId: Id, options: {

package/dist/services/file-download.d.ts

@@ -1,5 +1,6 @@
 import type { RefreshedFile } from "../domain/attachments.js";
 import type { Id } from "../domain/models.js";
+import { safeName } from "../files/safe-name.js";
 import type { RemoteFile } from "../ports/messenger.js";
 import { batchProgress } from "./batch.js";
 import type { MessagesService } from "./messages.js";
@@ -39,11 +40,7 @@ export declare const saveFiles: (messages: Pick<MessagesService, "keepDownloaded
         maxErrorPercent: number;
     };
 }>;
-/**
- * A name another person chose cannot climb out of the folder, cannot be a dot file we would then
- * hide, and carries no control or direction character to rewrite the terminal or disguise its extension.
- */
-export declare const safeName: (name: string | undefined, platform?: NodeJS.Platform) => string | undefined;
+export { safeName };
 /**
  * The bytes go to a temporary name first and are then hard-linked to the real one, which fails if the
  * name is taken — so an existing file survives, and an interrupted download leaves no half file under

package/dist/services/file-download.js

@@ -1,9 +1,10 @@
 import { createWriteStream, mkdirSync } from "node:fs";
 import { link, rm } from "node:fs/promises";
-import { basename, join, resolve } from "node:path";
+import { join, resolve } from "node:path";
 import { Readable, Transform } from "node:stream";
 import { pipeline } from "node:stream/promises";
 import { CliError } from "@wirecat/cli-base";
+import { safeName } from "../files/safe-name.js";
 import { batchProgress } from "./batch.js";
 /** The files the messenger gave a new reference for while their bytes were read, to save without a path. */
 export const refreshedFiles = (files) => files.flatMap(({ kind, name, position, providerRef }) => providerRef === undefined || Object.keys(providerRef).length === 0
@@ -90,23 +91,7 @@ const EXTENSIONS = {
 };
 /** Telegram sends a photo with no name and no type; it is always a JPEG. */
 const BY_KIND = { photo: "jpg" };
-/**
- * A name another person chose cannot climb out of the folder, cannot be a dot file we would then
- * hide, and carries no control or direction character to rewrite the terminal or disguise its extension.
- */
-export const safeName = (name, platform = process.platform) => {
-    let plain = name === undefined
-        ? ""
-        : basename(name.replaceAll("\\", "/"))
-            .replace(/[\p{Cc}​-‏‪-‮⁦-⁩]/gu, "")
-            .replace(/^\.+/, "");
-    if (platform === "win32") {
-        plain = plain.replace(/[<>:"|?*]/g, "_").replace(/[. ]+$/, "");
-        if (/^(con|prn|aux|nul|com[1-9]|lpt[1-9])(?:\.|$)/i.test(plain))
-            return undefined;
-    }
-    return plain === "" ? undefined : plain;
-};
+export { safeName };
 /**
  * The bytes go to a temporary name first and are then hard-linked to the real one, which fails if the
  * name is taken — so an existing file survives, and an interrupted download leaves no half file under

package/dist/services/tasks.d.ts

@@ -2,7 +2,7 @@ import { type QueryAst } from "@wirecat/cli-base/query";
 import type * as v from "valibot";
 import { task as taskEntity } from "../entities/task.js";
 import { type CloseSuggestion, type SimilarTask, type TaskDraft, type TaskPair, type TaskPairSide, type TaskRelation } from "../store/index.js";
-import type { AccountKey, MessageStore, TaskDetails, ThingLink } from "../store/store.js";
+import type { AccountKey, Author, MessageStore, TaskDetails, ThingLink } from "../store/store.js";
 import { type ClosedState, type GroupStats, TASK_KINDS, TASK_STATES, type Task, type TaskKind, type TaskOrigin } from "../tasks/index.js";
 import type { ServiceDeps } from "./deps.js";
 import { type TaskCategory, type UnlistedCategory } from "./task-categories.js";
@@ -126,7 +126,9 @@ export interface TasksService {
      * unless `choice` says to attach the message to one of them or to open the task anyway. A task opened while one
      * closed in the last 14 days is likely or maybe the same job follows it up (`followUpOf`); that one stays closed.
      */
-    add(message: string, type: TaskKind, origin: TaskOrigin, plan?: TaskPlan, choice?: DuplicateChoice): Promise<{
+    add(message: string, type: TaskKind, origin: TaskOrigin, plan?: TaskPlan, choice?: DuplicateChoice, 
+    /** Who adds it, when an agent is known by name; otherwise the origin says. */
+    author?: Author): Promise<{
         task: TaskView;
         created: boolean;
         attached?: boolean;

package/dist/services/tasks.js

@@ -218,7 +218,7 @@ export const tasksService = (deps, categories) => {
                 ...(previews ? { previews: await Promise.all(shown.map(({ task }) => taskView(store, account, task))) } : {}),
             };
         }),
-        add: (message, type, origin, { category: named, ...plan } = {}, choice = {}) => inStore(async (store, account) => {
+        add: (message, type, origin, { category: named, ...plan } = {}, choice = {}, author = undefined) => inStore(async (store, account) => {
             const chosen = named === undefined ? undefined : await categoryOf(store, named);
             const source = message.trim();
             const native = isNoteReference(source);
@@ -253,7 +253,7 @@ export const tasksService = (deps, categories) => {
                 ...(plan.dueAt ? { due: plan.dueAt } : {}),
             };
             const subject = { account: taskAccount(account), draft };
-            const by = origin === "owner" ? "owner" : { bot: origin };
+            const by = author ?? (origin === "owner" ? "owner" : { bot: origin });
             if (choice.attach !== undefined) {
                 const target = await owned(store, account, choice.attach);
                 const similarity = await store.tasks.compare(subject, target.id);

package/dist/services/topics.d.ts

@@ -3,6 +3,8 @@ import { type Operated } from "../sends/guarded.js";
 import type { ServiceDeps } from "./deps.js";
 export interface TopicsService {
     show(chat: string, topic: string): Promise<Topic>;
+    /** The emojis `create` and `edit` take as an icon. */
+    icons(): Promise<string[]>;
     enable(chat: string, options: {
         upgrade: boolean;
     }): Promise<Operated<{
@@ -13,6 +15,7 @@ export interface TopicsService {
     }>>;
     create(chat: string, title: string, options: {
         sendId?: string;
+        icon?: string;
     }): Promise<Operated<{
         chatId: string;
         topic: Topic;

package/dist/services/topics.js

@@ -2,9 +2,9 @@ import { CliError, isCliError } from "@wirecat/cli-base";
 import { capability } from "../ports/messenger.js";
 import { guardedWrite } from "../sends/guarded.js";
 import { newOperationId, newSendId } from "../sends/send-id.js";
-const actionOf = ({ title, closed, pinned, hidden }) => {
+const actionOf = ({ title, closed, pinned, hidden, icon }) => {
     const set = [closed, pinned, hidden].filter((one) => one !== undefined).length;
-    if (title !== undefined || set !== 1)
+    if (title !== undefined || icon !== undefined || set !== 1)
         return "topic-edit";
     if (closed !== undefined)
         return closed ? "topic-close" : "topic-reopen";
@@ -16,6 +16,12 @@ const validTitle = (title) => {
     if (title.trim() === "" || new TextEncoder().encode(title).byteLength > 128)
         throw new CliError("validation_error", "a topic title needs 1–128 UTF-8 bytes");
 };
+const validIcon = (icon) => {
+    const emoji = icon.trim();
+    if (emoji === "")
+        throw new CliError("validation_error", "which icon? give an emoji from `topics icons list`");
+    return emoji;
+};
 export const topicsService = (deps) => {
     const online = async () => {
         if (deps.offline)
@@ -31,6 +37,11 @@ export const topicsService = (deps) => {
                 throw new CliError("validation_error", "which topic? give its id, from `topics list`");
             return capability(await deps.connection(), "topic", "show a forum topic")(chat, id);
         },
+        icons: async () => {
+            if (deps.offline)
+                throw new CliError("validation_error", "`topics icons list` asks the messenger; not with --offline");
+            return capability(await deps.connection(), "topicIcons", "list topic icons")();
+        },
         enable: async (chat, { upgrade }) => {
             const connection = await online();
             const probe = capability(connection, "forumState", "configure forum topics");
@@ -108,8 +119,9 @@ export const topicsService = (deps) => {
                 forum: true,
             };
         },
-        create: async (chat, title, { sendId }) => {
+        create: async (chat, title, { sendId, icon: typedIcon }) => {
             validTitle(title);
+            const icon = typedIcon === undefined ? undefined : validIcon(typedIcon);
             const connection = await online();
             const create = capability(connection, "createTopic", "create forum topics");
             const probe = capability(connection, "forumState", "read forum settings");
@@ -124,7 +136,7 @@ export const topicsService = (deps) => {
                 action: "topic-create",
                 key: "topics.create",
                 length: title.length,
-            }, () => create(target, title, { sendId: id }), (done) => ({ threadId: done.id, resultChatId: target }), async () => {
+            }, () => create(target, title, { sendId: id, ...(icon === undefined ? {} : { icon }) }), (done) => ({ threadId: done.id, resultChatId: target }), async () => {
                 const state = await probe(chatId);
                 if (!state.forum)
                     throw new CliError("validation_error", "enable topics first with `topics enable`; creating a topic never upgrades a group");
@@ -138,8 +150,9 @@ export const topicsService = (deps) => {
         },
         edit: async (chat, typedTopic, change) => {
             const { title, closed, pinned, hidden } = change;
-            if (title === undefined && closed === undefined && pinned === undefined && hidden === undefined)
-                throw new CliError("validation_error", "nothing to change — rename, close, reopen, pin, unpin or hide it");
+            const icon = change.icon == null ? change.icon : validIcon(change.icon);
+            if ([title, closed, pinned, hidden, icon].every((one) => one === undefined))
+                throw new CliError("validation_error", "nothing to change — rename, close, reopen, pin, unpin, hide or give an icon");
             if (title !== undefined)
                 validTitle(title);
             const topicId = typedTopic.trim();
@@ -155,6 +168,7 @@ export const topicsService = (deps) => {
                 ...(closed === undefined ? {} : { closed }),
                 ...(pinned === undefined ? {} : { pinned }),
                 ...(hidden === undefined ? {} : { hidden }),
+                ...(icon === undefined ? {} : { icon }),
             }));
             return { operationId, chatId, topic };
         },

package/dist/store/index.d.ts

@@ -59,6 +59,7 @@ export type { TaskSort } from "./sqlite/task-board.js";
 export { PROJECT_TASK_FIELDS, TASK_SORTS } from "./sqlite/task-board.js";
 export type { SimilarSubject, SimilarTask, TaskDraft } from "./sqlite/task-candidates.js";
 export { type AgentCloseSettings, type CloseSuggestion, certaintyOf } from "./sqlite/task-close.js";
+export type { NamedAuthor, TaskComment } from "./sqlite/task-comments.js";
 export type { TaskPair, TaskPairDecision, TaskPairSide } from "./sqlite/task-pairs.js";
 export { TASK_PAIR_DECISIONS } from "./sqlite/task-pairs.js";
 export type { TaskRelation, TaskRelationKind, TaskRelationsStore } from "./sqlite/task-relations.js";

package/dist/store/sqlite/task-board.js

@@ -27,7 +27,9 @@ export const PROJECT_TASK_FIELDS = [
         relation: "low, medium, high, critical or none; a range of labels or of the 0–1 number: priority:[high TO critical]",
     }),
     field("agent", "kw", "agent:hermes", { relation: "the agent working on it, by name; none or any" }),
-    field("due", "time", "due:[today TO 2026-12-31]", { relation: "the due date; NOT due:* has none" }),
+    field("due", "time", "due:[today TO 2026-12-31]", {
+        relation: "the due date; today, tomorrow and yesterday are your calendar days; NOT due:* has none",
+    }),
     field("overdue", "bool", "overdue:true", { relation: "open work due before today" }),
     field("when", "enum", "when:today", {
         values: ["overdue", "today", "upcoming", "later", "none"],

package/dist/store/sqlite/task-comments.d.ts

@@ -0,0 +1,23 @@
+import type { Actor } from "./actors.js";
+import { type Author } from "./memories.js";
+import type { StoreContext } from "./open.js";
+type Row = Record<string, unknown>;
+/** Who wrote a row with `author_type` and `author_id`, by the name the store holds now. */
+export type NamedAuthor = {
+    type: Actor["type"];
+    id: string;
+    name: string | null;
+    owner: boolean;
+};
+export interface TaskComment {
+    ref: string;
+    text: string;
+    author: NamedAuthor | null;
+    at: string;
+}
+export declare const authorNamed: (database: StoreContext["database"], row: Row) => NamedAuthor | null;
+/** A task's comments are notes about it, oldest first. */
+export declare const commentsOf: (database: StoreContext["database"], task: number) => TaskComment[];
+/** In the author's name, unlike `notes.addNote`, which is always the owner's. */
+export declare const addComment: (context: StoreContext, task: number, body: string, by: Author) => TaskComment;
+export {};

package/dist/store/sqlite/task-comments.js

@@ -0,0 +1,41 @@
+import { authorActor } from "./memories.js";
+import { write } from "./transaction.js";
+export const authorNamed = (database, row) => {
+    if (row.author_type == null)
+        return null;
+    const named = database
+        .prepare(row.author_type === "bot"
+        ? "SELECT name, 0 AS owner FROM bots WHERE id = ?"
+        : "SELECT name, owner FROM persons WHERE id = ?")
+        .get(Number(row.author_id));
+    return {
+        type: row.author_type,
+        id: String(row.author_id),
+        name: named?.name == null ? null : String(named.name),
+        owner: Number(named?.owner) === 1,
+    };
+};
+/** A task's comments are notes about it, oldest first. */
+export const commentsOf = (database, task) => database
+    .prepare("SELECT n.id, n.body, n.created_at, n.author_type, n.author_id FROM notes n " +
+    "WHERE n.notable_type = 'task' AND n.notable_id = ? AND n.deleted_at IS NULL ORDER BY n.created_at, n.id")
+    .all(task)
+    .map((row) => ({
+    ref: `note:${row.id}`,
+    text: String(row.body),
+    author: authorNamed(database, row),
+    at: new Date(Number(row.created_at)).toISOString(),
+}));
+/** In the author's name, unlike `notes.addNote`, which is always the owner's. */
+export const addComment = (context, task, body, by) => {
+    const { database, now } = context;
+    const made = write(database, () => {
+        const { actor } = authorActor(context, by);
+        const at = now();
+        return Number(database
+            .prepare("INSERT INTO notes (notable_type, notable_id, body, author_type, author_id, revision, created_at, updated_at) " +
+            "VALUES ('task', ?, ?, ?, ?, 1, ?, ?) RETURNING id")
+            .get(task, body, actor.type, actor.id, at, at)?.id);
+    });
+    return commentsOf(database, task).find((one) => one.ref === `note:${made}`);
+};

package/dist/store/sqlite/things.js

@@ -223,7 +223,14 @@ export const stateOfThing = (database, thing) => {
 /** The `tasks.id` behind what callers name a task by: the package's id, or the task's key. */
 export const taskRowOf = (database, id) => {
     const row = database.prepare("SELECT id FROM tasks WHERE package_id = ? OR key = ? ORDER BY id LIMIT 1").get(id, id);
-    return row ? Number(row.id) : undefined;
+    if (row)
+        return Number(row.id);
+    // A task moved to another project took a new key there; its old key still names it.
+    const moved = database
+        .prepare("SELECT task_id FROM task_events WHERE kind = 'project' AND json_extract(changes, '$.key[0]') = ? " +
+        "ORDER BY id DESC LIMIT 1")
+        .get(id);
+    return moved ? Number(moved.task_id) : undefined;
 };
 /** The package's id for a task row: the reverse of `taskRowOf`. */
 export const taskIdOf = (database, rowId) => {

package/dist/store/sqlite/work-tasks.d.ts

@@ -8,9 +8,10 @@ import type { StoreContext } from "./open.js";
 import { type Where } from "./query-filter.js";
 import { type DueGroup, type TaskSort } from "./task-board.js";
 import { type CloseSuggestion } from "./task-close.js";
+import { type NamedAuthor, type TaskComment } from "./task-comments.js";
 import { type TaskRelationsStore } from "./task-relations.js";
 export declare const TASK_ROLES: readonly ["assignee", "reviewer", "watcher"];
-export declare const TASK_EVENT_KINDS: readonly ["created", "title", "description", "status", "assigned", "unassigned", "due", "priority", "moved", "linked", "unlinked", "category", "close-suggestion", "agent", "claimed", "released", "next-check", "resolution", "verdict", "close-certainty", "tags"];
+export declare const TASK_EVENT_KINDS: readonly ["created", "title", "description", "status", "assigned", "unassigned", "due", "priority", "moved", "linked", "unlinked", "category", "close-suggestion", "agent", "claimed", "released", "next-check", "resolution", "verdict", "close-certainty", "tags", "project", "type"];
 export type WorkTaskType = (typeof WORK_TASK_TYPES)[number];
 export type TaskType = (typeof TASK_TYPES)[number];
 export type TaskRole = (typeof TASK_ROLES)[number];
@@ -30,8 +31,12 @@ export interface TaskFields {
     reason?: string | null;
     /** A name the caller checked against its category list; every change is a `category` event. */
     category?: string;
+    /** Another owner project, by key or `project:<id>`: the task takes the next key there; its old key still finds it. */
+    project?: string;
+    /** A project task's kind, bug, feature or chore. */
+    type?: WorkTaskType;
 }
-export interface NewWorkTask extends Omit<TaskFields, "reason"> {
+export interface NewWorkTask extends Omit<TaskFields, "reason" | "project" | "type"> {
     /** `project:<id>` or the project's key. */
     project: string;
     title: string;
@@ -51,6 +56,8 @@ export interface TaskDetails {
     /** The project's key and name, as `boardCounts` names it. */
     projectKey: string;
     projectName: string;
+    /** Who added it: the owner, or an agent by its bot name; `null` for a row with no author. */
+    createdBy: NamedAuthor | null;
     title: string;
     description: string | null;
     type: TaskType;
@@ -185,6 +192,10 @@ export interface WorkTasks extends TaskRelationsStore {
         limit?: number;
     }): Promise<TaskDetails[]>;
     events(id: string): Promise<TaskEvent[]>;
+    /** A note about the task in the author's name, as an agent's comment is the agent's; the revision stays. */
+    comment(id: string, text: string, by: Author): Promise<TaskComment>;
+    /** Its comments, oldest first. */
+    comments(id: string): Promise<TaskComment[]>;
     /** The owner's project tasks, every project at once, narrowed by a query over `PROJECT_TASK_FIELDS`. */
     board(query: QueryAst | undefined, options: BoardOptions): Promise<{
         items: TaskDetails[];

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

@@ -10,6 +10,7 @@ import { queryFilter } from "./query-filter.js";
 import { stemmerCache } from "./stems.js";
 import { BOARD_SCOPE, boardOrder, dueGroups, projectTaskColumns, withoutField, } from "./task-board.js";
 import { closeSuggestionOf, SUGGESTION_COLUMNS } from "./task-close.js";
+import { addComment, authorNamed, commentsOf } from "./task-comments.js";
 import { INBOX_KEY_LENGTH, inboxKey } from "./task-keys.js";
 import { taskRelationsStoreOver } from "./task-relations.js";
 import { taskIdOf, taskRowOf, thingOf } from "./things.js";
@@ -37,6 +38,8 @@ export const TASK_EVENT_KINDS = [
     "verdict",
     "close-certainty",
     "tags",
+    "project",
+    "type",
 ];
 const iso = (value) => (value == null ? null : new Date(Number(value)).toISOString());
 export const recordTaskEvent = (database, task, actor, kind, changes, at) => {
@@ -122,6 +125,7 @@ export const workTaskOperationsOver = (context) => {
             key: String(row.key),
             project: `project:${row.project_id}`,
             ...projectNamed(Number(row.project_id)),
+            createdBy: authorNamed(database, row),
             title: String(row.title),
             description: row.description == null ? null : String(row.description),
             type: row.type,
@@ -164,11 +168,43 @@ export const workTaskOperationsOver = (context) => {
     const countOf = (where) => Number(database.prepare(`SELECT count(*) AS n FROM tasks t WHERE ${where.sql}`).get(...where.params)?.n ?? 0);
     const rowById = (id) => database.prepare("SELECT * FROM tasks WHERE id = ?").get(id);
     const details = (id) => detailsOf(rowById(id));
+    const onBoard = (task) => Boolean(database.prepare(`SELECT 1 FROM tasks t WHERE t.id = ? AND ${BOARD_SCOPE}`).get(task));
+    /** The task takes the target's next number and key; the source's counter never goes back, so its old key stays unique. */
+    const moveToProject = (task, before, target, actor, at) => {
+        if (target === Number(before.project_id))
+            return;
+        if (!onBoard(task))
+            throw new CliError("validation_error", "only a project task moves to another project");
+        const targetRow = database.prepare("SELECT key FROM projects WHERE id = ?").get(target);
+        if (String(targetRow.key).length === INBOX_KEY_LENGTH)
+            throw new CliError("validation_error", "a task moves only into one of your projects");
+        const children = database
+            .prepare("SELECT key FROM tasks WHERE parent_id = ? AND deleted_at IS NULL ORDER BY id")
+            .all(task)
+            .map((row) => String(row.key));
+        if (children.length)
+            throw new CliError("validation_error", `${String(before.key)} has subtasks (${children.join(", ")}); move or detach them first`, { reason: "task.has-subtasks", subtasks: children });
+        const counted = database
+            .prepare("UPDATE projects SET tasks_count = tasks_count + 1, updated_at = ? WHERE id = ? RETURNING key, tasks_count")
+            .get(at, target);
+        const number = Number(counted.tasks_count);
+        const key = `${String(counted.key)}-${number}`;
+        database
+            .prepare("UPDATE tasks SET project_id = ?, number = ?, key = ?, parent_id = NULL, updated_at = ? WHERE id = ?")
+            .run(target, number, key, at, task);
+        recordTaskEvent(database, task, actor, "project", {
+            project: [`project:${before.project_id}`, `project:${target}`],
+            key: [String(before.key), key],
+            ...(before.parent_id == null ? {} : { parent: [`task:${taskIdOf(database, Number(before.parent_id))}`, null] }),
+        }, at);
+    };
     const edit = (task, fields, actor, created = false) => {
         const before = rowById(task);
         const at = now();
         const set = [];
         const values = [];
+        // The revision triggers do not watch `type`, so a kind change bumps it here.
+        let bump = false;
         const change = (column, value, event, key = column) => {
             if ((before[column] ?? null) === value)
                 return;
@@ -237,8 +273,21 @@ export const workTaskOperationsOver = (context) => {
         }
         if (fields.category !== undefined)
             change("category", fields.category, "category");
-        if (set.length === 0)
+        if (fields.type !== undefined && fields.type !== before.type) {
+            if (!WORK_TASK_TYPES.includes(fields.type) || !onBoard(task))
+                throw new CliError("validation_error", `a project task's kind is one of ${WORK_TASK_TYPES.join(", ")}`);
+            change("type", fields.type, "type");
+            bump = true;
+        }
+        if (fields.project !== undefined)
+            moveToProject(task, before, projectOf(fields.project), actor, at);
+        if (set.length === 0) {
+            if (bump)
+                database.prepare("UPDATE tasks SET revision = revision + 1 WHERE id = ?").run(task);
             return;
+        }
+        if (bump)
+            set.push("revision = revision + 1");
         database.prepare(`UPDATE tasks SET ${set.join(", ")}, updated_at = ? WHERE id = ?`).run(...values, at, task);
     };
     /** The column's tasks in board order, the moving one left out; no priority reads as 0, the bottom of low. */
@@ -555,6 +604,8 @@ export const workTaskOperationsOver = (context) => {
             owner: Number(row.owner) === 1,
             tasks: Number(row.n),
         })),
+        comment: async (id, text, by) => addComment(context, required(id), textOf(text, "a comment", 20000), by),
+        comments: async (id) => commentsOf(database, required(id)),
         events: async (id) => database
             .prepare("SELECT e.actor_type, e.actor_id, e.kind, e.changes, e.created_at, " +
             "CASE e.actor_type WHEN 'person' THEN p.name WHEN 'bot' THEN b.name END AS actor_name FROM task_events e " +