@wirecat/cli-core 0.559.0 → 0.561.0

package/dist/store/sqlite/open-store.js

@@ -4,7 +4,7 @@ import { CliError } from "@wirecat/cli-base";
 import { DEFAULT_DECISION_POLICY } from "../decisions.js";
 import { migrate } from "../migrations.js";
 import { storeCapable } from "../open.js";
-import { storePath } from "../path.js";
+import { refuseOwnerStoreInTests, storePath } from "../path.js";
 import { sharedConfigPath } from "../shared-config.js";
 import { backfillNormalized, pendingNormalization } from "./backfill.js";
 import { fillFiles } from "./files.js";
@@ -18,6 +18,7 @@ import { assertReadCompatible } from "./read-compatibility.js";
 import { stemsState } from "./stems.js";
 import { fillStems, followDefaultStemmers } from "./stems-fill.js";
 import { moveSuggestedLinks } from "./suggested-links.js";
+import { purgeTestAccounts } from "./test-accounts.js";
 import { fillVectorIndexes } from "./vector-index.js";
 export const BACKFILL_ON_OPEN = 5_000;
 const NOT_WRITABLE = new Set(["EACCES", "EPERM", "EROFS"]);
@@ -39,6 +40,7 @@ export const openStoreContext = async ({ path, env, now = Date.now, command, ver
         throw new CliError("validation_error", "readOnly must be a boolean");
     await storeCapable();
     const file = path ?? storePath(env);
+    refuseOwnerStoreInTests(file);
     if (readOnly) {
         if (!statSync(file, { throwIfNoEntry: false })?.isFile())
             throw new CliError("not_found", "The store does not exist as a file; read-only opening does not create it");
@@ -59,6 +61,7 @@ export const openStoreContext = async ({ path, env, now = Date.now, command, ver
             fillJoinKeys(database);
             moveSuggestedLinks(database, now);
             movePaymentFacts({ ...opened, now, decisions });
+            purgeTestAccounts({ ...opened, now, decisions }, warn);
             const pending = pendingNormalization(database);
             if (pending > 0 && pending <= BACKFILL_ON_OPEN)
                 backfillNormalized(database);

package/dist/store/sqlite/open.js

@@ -1,5 +1,5 @@
-// biome-ignore-all lint/plugin/no-sqlite-driver: cli-core owns the store, so it opens the driver itself
 import { PRAGMAS } from "../driver.js";
+import { refuseOwnerStoreInTests } from "../path.js";
 import { quietSqliteWarning } from "../sqlite-warning.js";
 /**
  * One SQLite connection, seen two ways: the `CacheDatabase` seam the hand-written SQL and the
@@ -8,6 +8,7 @@ import { quietSqliteWarning } from "../sqlite-warning.js";
  * `../open.ts` for why a static import breaks the other runtime.
  */
 export const openSqlite = async (path, { readOnly = false } = {}) => {
+    refuseOwnerStoreInTests(path);
     const opened = "Bun" in globalThis ? await openUnderBun(path, readOnly) : await openUnderNode(path, readOnly);
     try {
         opened.database.exec(readOnly ? "PRAGMA busy_timeout=5000; PRAGMA foreign_keys=ON; PRAGMA query_only=ON" : PRAGMAS);

package/dist/store/sqlite/proposed-actions.js

@@ -2,6 +2,7 @@ import { CliError } from "@wirecat/cli-base";
 import { MEETING_FINDINGS_KIND } from "../../meetings/index.js";
 import { assertMayDecide, deciderAuthor } from "../decisions.js";
 import { EVIDENCE_FINDINGS_KIND } from "../evidence-findings.js";
+import { MAIL_FINDINGS_KIND } from "../mail-findings.js";
 import { proposedBySql } from "./actors.js";
 import { authorActor } from "./memories.js";
 import { requiredThing } from "./notes.js";
@@ -15,7 +16,7 @@ const json = (value) => (value == null ? null : JSON.parse(String(value)));
  * A findings batch closed with nothing accepted is stored `approved`, as older builds read it; it reads as
  * `rejected`, the decision the owner made.
  */
-const STATUS = `CASE WHEN p.status = 'approved' AND p.kind IN ('${MEETING_FINDINGS_KIND}', '${EVIDENCE_FINDINGS_KIND}') ` +
+const STATUS = `CASE WHEN p.status = 'approved' AND p.kind IN ('${MEETING_FINDINGS_KIND}', '${EVIDENCE_FINDINGS_KIND}', '${MAIL_FINDINGS_KIND}') ` +
     "AND json_array_length(p.result, '$.accepted') = 0 THEN 'rejected' ELSE p.status END";
 const NAMED = `SELECT p.*, ${STATUS} AS shown_status, b.name AS proposer_name, b.kind AS proposer_kind FROM proposed_actions p ` +
     "LEFT JOIN bots b ON p.proposed_by_type = 'bot' AND b.id = p.proposed_by_id";

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

@@ -36,4 +36,6 @@ export interface StoreTaskStore extends Omit<Required<TaskStore>, "reviewDuplica
         closeSuggestion: CloseSuggestion | null;
     }[]>;
 }
+/** The inbox project of a task account (`telegram:100`), made the first time `create` asks for it. */
+export declare const inboxProject: ({ database, now }: Pick<StoreContext, "database" | "now">, account: string, create: boolean) => number | undefined;
 export declare const taskStoreOver: (context: StoreContext) => StoreTaskStore;

package/dist/store/sqlite/tasks.js

@@ -18,32 +18,34 @@ export { inboxKey, linkInbox, taskAccountOf } from "./task-keys.js";
 export { taskIdOf, taskRowOf } from "./things.js";
 const COLUMNS = "t.id AS row_id, t.package_id, t.source_locator, t.source_kind, t.source_group, t.type, t.status, t.close_reason, t.source, t.created_at, t.due_at, t.closed_at, " +
     "t.closed_by_type, t.closed_by_id, t.key, t.category, t.claimed_by, t.claimed_until, p.name AS account";
+/** The inbox project of a task account (`telegram:100`), made the first time `create` asks for it. */
+export const inboxProject = ({ database, now }, account, create) => {
+    const key = inboxKey(account);
+    const found = database.prepare("SELECT id FROM projects WHERE key = ?").get(key);
+    if (!create)
+        return found ? Number(found.id) : undefined;
+    const [provider, external] = [account.slice(0, account.indexOf(":")), account.slice(account.indexOf(":") + 1)];
+    const owning = database
+        .prepare("SELECT id FROM accounts WHERE provider = ? AND external_id = ?")
+        .get(provider, external);
+    const accountId = owning ? Number(owning.id) : null;
+    // A task can arrive before its messenger saved the account; the project learns it on a later task.
+    if (found) {
+        if (accountId !== null)
+            database
+                .prepare("UPDATE projects SET account_id = ? WHERE id = ? AND account_id IS NULL")
+                .run(accountId, Number(found.id));
+        return Number(found.id);
+    }
+    const at = now();
+    return Number(database
+        .prepare("INSERT INTO projects (key, name, name_folded, description, type, account_id, scope, tasks_count, status, created_at, updated_at) " +
+        "VALUES (?, ?, ?, ?, 'personal', ?, 'personal', 0, 'active', ?, ?) RETURNING id")
+        .get(key, account, folded(account), `What waits on the owner in ${account}`, accountId, at, at)?.id);
+};
 export const taskStoreOver = (context) => {
     const { database, now } = context;
-    const inbox = (account, create) => {
-        const key = inboxKey(account);
-        const found = database.prepare("SELECT id FROM projects WHERE key = ?").get(key);
-        if (!create)
-            return found ? Number(found.id) : undefined;
-        const [provider, external] = [account.slice(0, account.indexOf(":")), account.slice(account.indexOf(":") + 1)];
-        const owning = database
-            .prepare("SELECT id FROM accounts WHERE provider = ? AND external_id = ?")
-            .get(provider, external);
-        const accountId = owning ? Number(owning.id) : null;
-        // A task can arrive before its messenger saved the account; the project learns it on a later task.
-        if (found) {
-            if (accountId !== null)
-                database
-                    .prepare("UPDATE projects SET account_id = ? WHERE id = ? AND account_id IS NULL")
-                    .run(accountId, Number(found.id));
-            return Number(found.id);
-        }
-        const at = now();
-        return Number(database
-            .prepare("INSERT INTO projects (key, name, name_folded, description, type, account_id, scope, tasks_count, status, created_at, updated_at) " +
-            "VALUES (?, ?, ?, ?, 'personal', ?, 'personal', 0, 'active', ?, ?) RETURNING id")
-            .get(key, account, folded(account), `What waits on the owner in ${account}`, accountId, at, at)?.id);
-    };
+    const inbox = (account, create) => inboxProject(context, account, create);
     // A task made here has no package id and may be a type the package does not know; the package never sees it.
     const select = (where) => database.prepare(`SELECT ${COLUMNS} FROM tasks t JOIN projects p ON p.id = t.project_id WHERE t.package_id IS NOT NULL AND ${where}`);
     const toTask = (row) => {

package/dist/store/sqlite/test-accounts.d.ts

@@ -0,0 +1,8 @@
+import type { StoreContext } from "./open.js";
+/** Set by store version 48 and deleted once the purge ran. */
+export declare const TEST_ACCOUNTS_TO_PURGE = "test_accounts_to_purge";
+/**
+ * Removes every account of the test provider, as `cache clear` removes one. A failure keeps the key for the next open
+ * and only warns: a leftover test account must never stop the owner's store from opening.
+ */
+export declare const purgeTestAccounts: (context: StoreContext, warn: (line: string) => void) => number;

package/dist/store/sqlite/test-accounts.js

@@ -0,0 +1,31 @@
+import { purgeAccount } from "./accounts.js";
+import { write } from "./transaction.js";
+/** Set by store version 48 and deleted once the purge ran. */
+export const TEST_ACCOUNTS_TO_PURGE = "test_accounts_to_purge";
+/** The fake messenger of `src/testing/shared-program.ts`; no tool writes it. */
+const TEST_PROVIDER = "chat";
+/**
+ * Removes every account of the test provider, as `cache clear` removes one. A failure keeps the key for the next open
+ * and only warns: a leftover test account must never stop the owner's store from opening.
+ */
+export const purgeTestAccounts = (context, warn) => {
+    const { database } = context;
+    if (!database.prepare("SELECT 1 AS found FROM store_settings WHERE key = ?").get(TEST_ACCOUNTS_TO_PURGE))
+        return 0;
+    try {
+        return write(database, () => {
+            const pks = database
+                .prepare("SELECT id FROM accounts WHERE provider = ? ORDER BY id")
+                .all(TEST_PROVIDER)
+                .map((row) => Number(row.id));
+            for (const pk of pks)
+                purgeAccount(context, pk);
+            database.prepare("DELETE FROM store_settings WHERE key = ?").run(TEST_ACCOUNTS_TO_PURGE);
+            return pks.length;
+        });
+    }
+    catch (error) {
+        warn(`could not remove the test accounts from the store; the next open tries again: ${String(error)}`);
+        return 0;
+    }
+};

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

@@ -232,6 +232,7 @@ export declare const checkRevision: (row: Row, expected: number | undefined) =>
 type WorkTaskOperations = Omit<WorkTasks, "create" | "assign"> & {
     create(input: NewWorkTask, by: Author): TaskDetails;
     assign(id: string, assignment: TaskAssignment, by: Author, check?: RevisionCheck): TaskDetails;
+    editNow(id: string, fields: TaskFields, by: Author): TaskDetails;
 };
 export declare const workTaskOperationsOver: (context: StoreContext) => WorkTaskOperations;
 export declare const workTasksOver: (context: StoreContext) => WorkTasks;

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

@@ -396,6 +396,12 @@ export const workTaskOperationsOver = (context) => {
             });
             return details(task);
         },
+        /** `edit` inside the caller's write, so a decision that edits a task rolls back with it. */
+        editNow: (id, fields, by) => {
+            const task = required(id);
+            write(database, () => edit(task, fields, authorActor(context, by).actor));
+            return details(task);
+        },
         move: async (id, move, by, check = {}) => {
             const task = required(id);
             return write(database, () => {

package/dist/store/store.d.ts

@@ -9,6 +9,7 @@ import { type ConversationsStore } from "./conversations.js";
 import { type CountersStore } from "./counters.js";
 import { type DecisionPolicy } from "./decisions.js";
 import type { EvidenceFindingsStore } from "./evidence-findings.js";
+import type { MailFindingsStore } from "./mail-findings.js";
 import type { MeetingFindingsStore } from "./meeting-findings.js";
 import { type MentionsStore } from "./mentions.js";
 import { type MessagesStore } from "./messages.js";
@@ -94,6 +95,8 @@ export interface MessageStore {
     readonly meetingFindings: MeetingFindingsStore;
     /** Cited facts and follow-ups about one project from stored messages, mail and documents, for review. */
     readonly evidenceFindings: EvidenceFindingsStore;
+    /** What an agent read in new mail, cited and staged per mailbox; only the owner decides it. */
+    readonly mailFindings: MailFindingsStore;
     /** What any ref names, a page of them at a time: read once per page, never once per row. */
     readonly refs: RefsStore;
     readonly facts: FactsStore;

package/dist/store/store.js

@@ -28,6 +28,7 @@ import { financeStoreOver } from "./sqlite/finance.js";
 import { involvementStoreOver } from "./sqlite/involvements.js";
 import { knowledgeStoreOver } from "./sqlite/knowledge.js";
 import { logPruner } from "./sqlite/log-pruning.js";
+import { mailFindingsStoreOver } from "./sqlite/mail-findings.js";
 import { meetingFindingsStoreOver } from "./sqlite/meeting-findings.js";
 import { meetingVectorsOver } from "./sqlite/meeting-vectors.js";
 import { meetingStoreOver } from "./sqlite/meetings.js";
@@ -108,6 +109,7 @@ const storeOver = (context) => {
     const calendar = once(() => ({ ...eventStoreOver(context), ...eventSearchOver(context) }));
     const meetingFindings = once(() => meetingFindingsStoreOver(context));
     const evidenceFindings = once(() => evidenceFindingsStoreOver(context));
+    const mailFindings = once(() => mailFindingsStoreOver(context));
     const refs = {
         summaries: async (texts, options) => refSummariesOf(context.database, texts, options),
     };
@@ -211,6 +213,9 @@ const storeOver = (context) => {
         get evidenceFindings() {
             return evidenceFindings();
         },
+        get mailFindings() {
+            return mailFindings();
+        },
         get facts() {
             return facts();
         },

package/package.json

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