@wirecat/cli-core 0.530.0 → 0.532.0

package/dist/services/search-all.js

@@ -1,5 +1,6 @@
 import { CliError } from "@wirecat/cli-base";
 import { parseLucene } from "@wirecat/cli-base/query";
+import { hitFiles } from "../domain/attachments.js";
 import { parseLocator } from "../domain/locator.js";
 import { senderIsMe } from "../domain/models.js";
 import { INTERNAL_NOTES, kindIn } from "../domain/providers.js";
@@ -102,6 +103,7 @@ export const searchAll = async (store, account, request, messenger = {}, searchM
                     : { name: hit.senderName, ref: null },
                 senderIsMe: senderIsMe(hit, at.account),
                 ...(hit.email ? { email: hit.email } : {}),
+                attachments: hitFiles(hit.attachments),
                 foundBy,
             };
         };

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 " +

package/package.json

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

package/parity.json

@@ -3329,6 +3329,9 @@
         "--all": {
           "in": "all"
         },
+        "--attachment": {
+          "in": "all"
+        },
         "--extract": "all",
         "--output-dir": {
           "in": "all"