@wirecat/cli-core 0.362.0 → 0.363.0

package/dist/store/sqlite/meeting-findings.js

@@ -1,12 +1,9 @@
 import { CliError } from "@wirecat/cli-base";
 import { findingHash, MEETING_FINDINGS_KIND, meetingCueFingerprint, meetingFindingFingerprint, parseMeetingReference, validateMeetingFindings, } from "../../meetings/index.js";
-import { factOperationsOver } from "./facts.js";
-import { authorActor, linkEvidence } from "./memories.js";
-import { requiredThing } from "./notes.js";
+import { findingsCoreOver } from "./findings-core.js";
+import { authorActor } from "./memories.js";
 import { proposalOperationsOver } from "./proposed-actions.js";
-import { taskRowOf } from "./things.js";
 import { write } from "./transaction.js";
-import { workTaskOperationsOver } from "./work-tasks.js";
 const stale = () => {
     throw new CliError("validation_error", "meeting findings changed since they were read; stage a new batch", {
         reason: "meeting-findings.stale",
@@ -18,8 +15,6 @@ const invalid = (message) => {
 export const meetingFindingsStoreOver = (context) => {
     const { database } = context;
     const proposals = proposalOperationsOver(context);
-    const facts = factOperationsOver(context);
-    const tasks = workTaskOperationsOver(context);
     const accountId = (key) => {
         if (!key ||
             typeof key.provider !== "string" ||
@@ -68,109 +63,56 @@ export const meetingFindingsStoreOver = (context) => {
             receipt: proposal.result,
         };
     };
+    const core = findingsCoreOver(context, MEETING_FINDINGS_KIND);
     const snapshot = (account, input) => {
         let readBytes = 0;
-        const seen = new Map();
-        for (const finding of input.findings) {
-            const project = requiredThing(database, finding.project, "the project");
-            if (project.type !== "project")
-                return invalid("a finding needs an owner project");
-            const projectRow = database
-                .prepare("SELECT id,key,name,description,type,scope,updated_at FROM projects WHERE id=? AND account_id IS NULL AND deleted_at IS NULL")
-                .get(project.id);
-            if (!projectRow)
-                throw new CliError("not_found", "the owner project is unavailable");
-            seen.set(finding.project, projectRow);
-            if (finding.kind === "task" && finding.assignee) {
-                const person = requiredThing(database, finding.assignee, "the assignee");
-                if (person.type !== "person")
-                    return invalid("the assignee must be a stored person");
-                const personRow = database.prepare("SELECT id,name,updated_at FROM persons WHERE id=?").get(person.id);
-                if (!personRow)
-                    throw new CliError("not_found", "the assignee is unavailable");
-                seen.set(finding.assignee, personRow);
-            }
-            for (const evidence of finding.evidence) {
-                const parsed = parseMeetingReference(evidence.reference);
-                if (parsed.accountId !== account)
-                    throw new CliError("not_found", "the cited cue belongs to another account");
-                const membership = database
-                    .prepare("SELECT id,updated_at,confirmed,source FROM links WHERE from_type='meeting' AND from_id=? AND to_type='project' AND to_id=? AND kind='member-of' AND anchor IS NULL AND confirmed=1")
-                    .get(parsed.meetingId, project.id);
-                if (!membership)
-                    return invalid("each cited meeting needs confirmed membership in the finding's project");
-                seen.set(`${finding.project}/${parsed.meetingId}`, membership);
-                if (seen.has(evidence.reference)) {
-                    if (seen.get(evidence.reference) !== evidence.fingerprint)
-                        return stale();
-                    continue;
-                }
-                const query = "FROM meeting_transcript_rows r JOIN meeting_transcripts t ON t.id=r.meeting_transcript_id JOIN meetings m ON m.id=t.meeting_id WHERE m.id=? AND m.account_id=? AND m.deleted_at IS NULL AND t.id=? AND t.deleted_at IS NULL AND t.superseded_at IS NULL AND r.position=?";
-                const params = [parsed.meetingId, account, parsed.transcriptId, parsed.cuePosition];
-                const budget = database
-                    .prepare(`SELECT length(CAST(r.text AS BLOB))+coalesce(length(CAST(r.speaker_name AS BLOB)),0)+coalesce(length(CAST(t.content_hash AS BLOB)),0) AS bytes ${query}`)
-                    .get(...params);
-                if (!budget)
-                    return stale();
-                readBytes += Number(budget.bytes);
-                if (!Number.isSafeInteger(readBytes) || readBytes > 65536)
-                    return invalid("finding evidence exceeds the 65536-byte stored text budget");
-                const row = database
-                    .prepare(`SELECT t.content_hash,r.start_ms,r.end_ms,r.speaker_participant_id,r.speaker_name,r.text ${query}`)
-                    .get(...params);
-                if (!row)
+        return core.snapshot(input, (evidence, finding, projectId, seen) => {
+            const parsed = parseMeetingReference(evidence.reference);
+            if (parsed.accountId !== account)
+                throw new CliError("not_found", "the cited cue belongs to another account");
+            const membership = database
+                .prepare("SELECT id,updated_at,confirmed,source FROM links WHERE from_type='meeting' AND from_id=? AND to_type='project' AND to_id=? AND kind='member-of' AND anchor IS NULL AND confirmed=1")
+                .get(parsed.meetingId, projectId);
+            if (!membership)
+                return invalid("each cited meeting needs confirmed membership in the finding's project");
+            seen.set(`${finding.project}/${parsed.meetingId}`, membership);
+            if (seen.has(evidence.reference)) {
+                if (seen.get(evidence.reference) !== evidence.fingerprint)
                     return stale();
-                const fingerprint = meetingCueFingerprint({
-                    reference: evidence.reference,
-                    transcriptId: parsed.transcriptId,
-                    contentHash: row.content_hash == null ? null : String(row.content_hash),
-                    position: parsed.cuePosition,
-                    startMs: Number(row.start_ms),
-                    endMs: Number(row.end_ms),
-                    speakerParticipantId: row.speaker_participant_id == null ? null : Number(row.speaker_participant_id),
-                    speakerName: row.speaker_name == null ? null : String(row.speaker_name),
-                    text: String(row.text),
-                });
-                if (fingerprint !== evidence.fingerprint)
-                    return stale();
-                seen.set(evidence.reference, fingerprint);
+                return;
             }
-        }
-        return findingHash([...seen].sort(([a], [b]) => a.localeCompare(b)));
-    };
-    const previous = (account, fingerprint) => {
-        const row = database
-            .prepare("SELECT substr(j.value,1,4096) AS item FROM proposed_actions p,json_each(p.result,'$.items') j WHERE p.account_id=? AND p.kind=? AND p.status='approved' AND json_extract(j.value,'$.fingerprint')=? ORDER BY p.id LIMIT 1")
-            .get(account, MEETING_FINDINGS_KIND, fingerprint);
-        return row ? JSON.parse(String(row.item)) : undefined;
-    };
-    const create = (finding, agent) => {
-        const evidence = finding.evidence.map((item) => item.reference);
-        if (finding.kind === "fact") {
-            const fact = facts.add({
-                type: finding.type,
-                statement: finding.statement,
-                project: finding.project,
-                evidence,
-                by: agent,
+            const query = "FROM meeting_transcript_rows r JOIN meeting_transcripts t ON t.id=r.meeting_transcript_id JOIN meetings m ON m.id=t.meeting_id WHERE m.id=? AND m.account_id=? AND m.deleted_at IS NULL AND t.id=? AND t.deleted_at IS NULL AND t.superseded_at IS NULL AND r.position=?";
+            const params = [parsed.meetingId, account, parsed.transcriptId, parsed.cuePosition];
+            const budget = database
+                .prepare(`SELECT length(CAST(r.text AS BLOB))+coalesce(length(CAST(r.speaker_name AS BLOB)),0)+coalesce(length(CAST(t.content_hash AS BLOB)),0) AS bytes ${query}`)
+                .get(...params);
+            if (!budget)
+                return stale();
+            readBytes += Number(budget.bytes);
+            if (!Number.isSafeInteger(readBytes) || readBytes > 65536)
+                return invalid("finding evidence exceeds the 65536-byte stored text budget");
+            const row = database
+                .prepare(`SELECT t.content_hash,r.start_ms,r.end_ms,r.speaker_participant_id,r.speaker_name,r.text ${query}`)
+                .get(...params);
+            if (!row)
+                return stale();
+            const fingerprint = meetingCueFingerprint({
+                reference: evidence.reference,
+                transcriptId: parsed.transcriptId,
+                contentHash: row.content_hash == null ? null : String(row.content_hash),
+                position: parsed.cuePosition,
+                startMs: Number(row.start_ms),
+                endMs: Number(row.end_ms),
+                speakerParticipantId: row.speaker_participant_id == null ? null : Number(row.speaker_participant_id),
+                speakerName: row.speaker_name == null ? null : String(row.speaker_name),
+                text: String(row.text),
             });
-            return facts.accept(fact.ref).ref;
-        }
-        const task = tasks.create({
-            project: finding.project,
-            title: finding.title,
-            type: finding.type,
-            description: `${finding.commitment}: ${finding.reason}`,
-            ...(finding.dueAt ? { dueAt: new Date(finding.dueAt) } : {}),
-        }, agent);
-        const id = taskRowOf(database, task.id);
-        if (id === undefined)
-            throw new CliError("configuration_error", "created finding task is unavailable");
-        linkEvidence(context, { type: finding.type, id }, evidence, "agent");
-        if (finding.assignee)
-            tasks.assign(task.id, { to: finding.assignee, role: "assignee" }, agent);
-        return task.ref;
+            if (fingerprint !== evidence.fingerprint)
+                return stale();
+            seen.set(evidence.reference, fingerprint);
+        });
     };
+    const previous = core.previous;
     return {
         stage: async (key, raw, agent) => {
             const input = validateMeetingFindings(raw);
@@ -248,23 +190,7 @@ export const meetingFindingsStoreOver = (context) => {
                 bot: proposal.proposedBy.name,
                 ...(input.agentModel === null ? {} : { model: input.agentModel }),
             };
-            const items = batch.input.findings.map((finding) => {
-                const fingerprint = meetingFindingFingerprint(finding);
-                const prior = previous(account, fingerprint);
-                if (prior) {
-                    if (accepted.includes(finding.id) && prior.status === "rejected")
-                        return invalid("this unchanged finding was already rejected; changed evidence needs a new candidate");
-                    return { ...prior, id: finding.id, reused: true };
-                }
-                const status = accepted.includes(finding.id) ? "accepted" : "rejected";
-                return {
-                    id: finding.id,
-                    fingerprint,
-                    status,
-                    reference: status === "accepted" ? create(finding, agent) : null,
-                    reused: false,
-                };
-            });
+            const items = core.outcomes(account, batch.input, accepted, agent);
             return view(proposals.approve(id, {
                 previewFingerprint: batch.fingerprint,
                 accepted,

package/dist/store/sqlite/proposed-actions.d.ts

@@ -63,8 +63,11 @@ export interface ProposedActionsStore {
     failed(id: string, error: string): Promise<ProposedAction>;
     judge(id: string, verdict: ProposedAction["verdict"]): Promise<ProposedAction>;
 }
-type ProposalOperations = {
+type ProposalOperations = Omit<{
     [K in keyof ProposedActionsStore]: ProposedActionsStore[K] extends (...args: infer A) => Promise<infer R> ? (...args: A) => R : never;
+}, "approve"> & {
+    /** `by` records an agent deciding for the owner; the public store always records the owner. */
+    approve(id: string, result?: unknown, by?: Author): ProposedAction;
 };
 export declare const proposalOperationsOver: (context: StoreContext) => ProposalOperations;
 export declare const proposedActionsStoreOver: (context: StoreContext) => ProposedActionsStore;

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

@@ -1,5 +1,6 @@
 import { CliError } from "@wirecat/cli-base";
 import { MEETING_FINDINGS_KIND } from "../../meetings/index.js";
+import { EVIDENCE_FINDINGS_KIND } from "../evidence-findings.js";
 import { authorActor } from "./memories.js";
 import { requiredThing } from "./notes.js";
 import { referenceOfThing, storedThing } from "./things.js";
@@ -60,8 +61,8 @@ export const proposalOperationsOver = (context) => {
             .run(to, now(), ...values, Number(id));
         return actionOf(rowOf(id));
     });
-    const decide = (id, to, result) => {
-        const { actor } = authorActor(context, "owner");
+    const decide = (id, to, result, by = "owner") => {
+        const { actor } = authorActor(context, by);
         return move(id, ["proposed"], to, ", decided_by_type = ?, decided_by_id = ?, decided_at = ?, result = ?", [
             actor.type,
             actor.id,
@@ -109,7 +110,7 @@ export const proposalOperationsOver = (context) => {
                 .all(status ?? null, status ?? null, Math.min(Math.max(limit, 1), 500))
                 .map(actionOf);
         },
-        approve: (id, result) => decide(id, "approved", result),
+        approve: (id, result, by) => decide(id, "approved", result, by),
         reject: (id) => decide(id, "rejected"),
         executed: (id, result) => move(id, ["approved"], "executed", ", executed_at = ?, result = ?", [
             now(),
@@ -133,15 +134,22 @@ export const proposalOperationsOver = (context) => {
 export const proposedActionsStoreOver = (context) => {
     const operations = proposalOperationsOver(context);
     const generic = (id) => {
-        if (operations.get(id).kind === MEETING_FINDINGS_KIND)
+        const kind = operations.get(id).kind;
+        if (kind === MEETING_FINDINGS_KIND)
             throw new CliError("validation_error", "review meeting findings through the dedicated findings command", {
                 reason: "meeting-findings.dedicated-review",
             });
+        if (kind === EVIDENCE_FINDINGS_KIND)
+            throw new CliError("validation_error", "review evidence findings through the dedicated findings command", {
+                reason: "evidence-findings.dedicated-review",
+            });
     };
     return {
         propose: async (input) => {
             if (input.kind === MEETING_FINDINGS_KIND)
                 throw new CliError("validation_error", "stage meeting findings through the dedicated findings command");
+            if (input.kind === EVIDENCE_FINDINGS_KIND)
+                throw new CliError("validation_error", "stage evidence findings through the dedicated findings command");
             return operations.propose(input);
         },
         get: async (id) => operations.get(id),

package/dist/store/store.d.ts

@@ -1,10 +1,11 @@
-import type { ContextEvidenceReader, ContextIndex } from "../context/evidence.js";
+import type { ContextEvidenceReader, ContextIndex, ContextRecordReader } from "../context/evidence.js";
 import { type AccountStore } from "./account-store.js";
 import { type AccountsStore } from "./accounts.js";
 import { type AttachmentsStore } from "./attachments.js";
 import { type ChatsStore } from "./chats.js";
 import { type ConversationsStore } from "./conversations.js";
 import { type CountersStore } from "./counters.js";
+import type { EvidenceFindingsStore } from "./evidence-findings.js";
 import type { MeetingFindingsStore } from "./meeting-findings.js";
 import { type MentionsStore } from "./mentions.js";
 import { type MessagesStore } from "./messages.js";
@@ -35,6 +36,7 @@ import { type VectorsStore } from "./vectors.js";
 export interface MessageStore {
     readonly contextEvidence?: ContextEvidenceReader;
     readonly contextIndex?: ContextIndex;
+    readonly contextRecords?: ContextRecordReader;
     /** Accounts, account groups and which account each app profile is. */
     readonly accounts: AccountsStore;
     /** One account's chats: the list, what left, what is complete, their metadata. */
@@ -77,6 +79,8 @@ export interface MessageStore {
     readonly catalog: CatalogStore;
     /** Choices that hold until replaced, each with its evidence. */
     readonly meetingFindings: MeetingFindingsStore;
+    /** Cited facts and follow-ups about one project from stored messages, mail and documents, for review. */
+    readonly evidenceFindings: EvidenceFindingsStore;
     readonly facts: FactsStore;
     /** What agents concluded: summaries, digests, facts, preferences, each with evidence and a scope. */
     readonly memories: MemoriesStore;

package/dist/store/store.js

@@ -16,8 +16,9 @@ import { agentActionsStoreOver } from "./sqlite/agent-actions.js";
 import * as attachmentTexts from "./sqlite/attachment-texts.js";
 import { botUpdateStoreOver } from "./sqlite/bot-updates.js";
 import { catalogStoreOver } from "./sqlite/catalog.js";
-import { contextEvidenceOver, contextIndexOver } from "./sqlite/context-evidence.js";
+import { contextEvidenceOver, contextIndexOver, contextRecordsOver } from "./sqlite/context-evidence.js";
 import { mailStoreOver } from "./sqlite/emails.js";
+import { evidenceFindingsStoreOver } from "./sqlite/evidence-findings.js";
 import { factsStoreOver } from "./sqlite/facts.js";
 import { involvementStoreOver } from "./sqlite/involvements.js";
 import { knowledgeStoreOver } from "./sqlite/knowledge.js";
@@ -82,6 +83,7 @@ const storeOver = (context) => {
     };
     const contextEvidence = once(() => contextEvidenceOver(context));
     const contextIndex = once(() => contextIndexOver(context, stemmerFor));
+    const contextRecords = once(() => contextRecordsOver(context));
     const searches = once(() => searchesStoreOver(context));
     const queries = once(() => queriesStoreOver(context, stemmerFor));
     const searchIndex = once(() => searchIndexStoreOver(context));
@@ -93,6 +95,7 @@ const storeOver = (context) => {
     const notes = once(() => ({ ...notesStoreOver(context), ...noteSearchOver(context) }));
     const catalog = once(() => catalogStoreOver(context));
     const meetingFindings = once(() => meetingFindingsStoreOver(context));
+    const evidenceFindings = once(() => evidenceFindingsStoreOver(context));
     const facts = once(() => factsStoreOver(context));
     const memories = once(() => memoriesStoreOver(context));
     const proposedActions = once(() => proposedActionsStoreOver(context));
@@ -105,6 +108,9 @@ const storeOver = (context) => {
         get contextIndex() {
             return contextIndex();
         },
+        get contextRecords() {
+            return contextRecords();
+        },
         meetingVectors: meetingVectorsOver(context),
         get accounts() {
             return areas.accounts().whole;
@@ -175,6 +181,9 @@ const storeOver = (context) => {
         get meetingFindings() {
             return meetingFindings();
         },
+        get evidenceFindings() {
+            return evidenceFindings();
+        },
         get facts() {
             return facts();
         },

package/package.json

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