@broberg/lens-daemon 0.1.29 → 0.1.31

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@broberg/lens-daemon",
-  "version": "0.1.29",
+  "version": "0.1.31",
   "description": "The Lens module of the cardmem daemon (flows, critic, coverage, capture sessions, the /lens/* routes and the cardmem-lens MCP) as a package. The daemon mounts createLensRouter(deps) + createLensMcpRouter(deps). Bun runtime.",
   "type": "module",
   "license": "MIT",
@@ -15,8 +15,8 @@
     "sharp": "^0.33.5",
     "turndown": "^7.2.4",
     "zod": "^3.23.8",
-    "@broberg/lens-engine": "0.14.0",
-    "@broberg/lens-svg": "0.1.0"
+    "@broberg/lens-engine": "0.14.1",
+    "@broberg/lens-svg": "0.1.1"
   },
   "devDependencies": {
     "@happy-dom/global-registrator": "^20.10.6",

package/src/artifact-uploads.ts

@@ -121,6 +121,40 @@ export function enqueueArtifact(db: Database, projectSlug: string, key: string |
   ).run(key, projectSlug, skipReason ? 'skipped' : 'pending', Date.now(), skipReason ?? null);
 }
 
+/** lens F006.1 — where one stored artifact stands on its way to the cloud, as a run
+ *  report shows it. A still with no queue row is said to be NOT QUEUED — never
+ *  rounded to "pending", which would promise an upload nothing is going to make. */
+export type CloudCopy =
+  | { status: 'done'; url: string }
+  | { status: 'pending' }
+  /** Failed at least once and will be tried again (the row is back in the queue). */
+  | { status: 'retrying'; error: string; attempts: number }
+  /** Given up after MAX_ATTEMPTS, or the bytes are gone. */
+  | { status: 'failed'; error: string; attempts: number }
+  | { status: 'skipped'; reason: string }
+  | { status: 'not_queued' };
+
+export function cloudCopiesOf(db: Database, keys: string[]): Map<string, CloudCopy> {
+  const out = new Map<string, CloudCopy>();
+  if (keys.length === 0) return out;
+  const rows = db
+    .query(`SELECT * FROM lens_artifact_uploads WHERE artifact_key IN (${keys.map(() => '?').join(',')})`)
+    .all(...keys) as UploadRow[];
+  const byKey = new Map(rows.map((r) => [r.artifact_key, r]));
+  for (const k of keys) {
+    const r = byKey.get(k);
+    if (!r) out.set(k, { status: 'not_queued' });
+    else if (r.status === 'done' && r.cloud_artifact_id) out.set(k, { status: 'done', url: `/api/artifacts/${r.cloud_artifact_id}/raw` });
+    else if (r.status === 'failed') out.set(k, { status: 'failed', error: r.last_error ?? 'no error recorded', attempts: r.attempts });
+    else if (r.status === 'skipped') out.set(k, { status: 'skipped', reason: r.last_error ?? 'held back' });
+    // A retryable failure goes back to 'pending' WITH its error kept (markFailed):
+    // that is a failed upload waiting for another try, not an untried one.
+    else if (r.last_error) out.set(k, { status: 'retrying', error: r.last_error, attempts: r.attempts });
+    else out.set(k, { status: 'pending' });
+  }
+  return out;
+}
+
 /** Queue several at once — the flow path has one per step. */
 export function enqueueArtifacts(db: Database, projectSlug: string, keys: (string | null)[]): void {
   for (const k of keys) enqueueArtifact(db, projectSlug, k);

package/src/flow.ts

@@ -2856,7 +2856,8 @@ export async function runFlow(flow: Flow, env: AuthEnv, opts: RunFlowOptions = {
       // ── leaf step ──
       const i = idx;
       idx += 1;
-      const label = step.label ?? describe(bindSelectorVars(step, vars));
+      // F009.134 — a loop row reaches the label too («mockup-{{s}}» stayed literal).
+      const label = step.label != null ? subst(step.label, vars) : describe(bindSelectorVars(step, vars));
       // F009.122 — which browser this step runs in. `sp` is the page every line
       // below acts on; the loop's own `page` is the default browser.
       const browserName = (step as { browser?: string }).browser ?? 'main';

package/src/service.ts

@@ -13,6 +13,7 @@ import { randomUUID } from 'node:crypto';
 import sharp from 'sharp';
 import { durabilityOf } from './durability.ts';
 import type { Database } from 'bun:sqlite';
+import { cloudCopiesOf } from './artifact-uploads.ts';
 import type { CaptureAction, CaptureMode } from './capture.ts';
 import { coerceStorageSeed } from './seed-storage.ts';
 import type { LensEngine } from './browser.ts';
@@ -686,8 +687,12 @@ function parseFindingsJson(raw: string | null | undefined, port?: number): Recor
   }
 }
 
-export function flowRunView(row: LensFlowRunRow, port: number) {
-  const steps = (JSON.parse(row.steps_json) as StoredFlowStep[]).map((s) => ({
+export function flowRunView(row: LensFlowRunRow, port: number, db?: Database) {
+  const stored = JSON.parse(row.steps_json) as StoredFlowStep[];
+  // lens F006.1 — each still's cloud copy, read from the upload queue, so the report
+  // says where the evidence can be opened from — or why it cannot — per step.
+  const clouds = db ? cloudCopiesOf(db, stored.map((s) => s.artifact_key).filter((k): k is string => !!k)) : null;
+  const steps = stored.map((s) => ({
     index: s.index,
     type: s.type,
     label: s.label,
@@ -712,7 +717,11 @@ export function flowRunView(row: LensFlowRunRow, port: number) {
     // with internal waiting (a click retrying actionability) answerable at all.
     ...(typeof s.ms_step_duration === 'number' ? { ms_step_duration: s.ms_step_duration } : {}),
     screenshot_url: artifactUrl(port, s.artifact_key),
+    ...(clouds && s.artifact_key ? { cloud: clouds.get(s.artifact_key)! } : {}),
   }));
+  // F006.1 — a run whose every still is in the cloud is durable, whatever disk the
+  // daemon wrote it to first: the citation no longer depends on this machine.
+  const allInCloud = clouds !== null && clouds.size > 0 && [...clouds.values()].every((c) => c.status === 'done');
   return {
     flow_run_id: row.id,
     project: row.project_slug,
@@ -746,7 +755,7 @@ export function flowRunView(row: LensFlowRunRow, port: number) {
     artifact_backend: (row.artifact_backend as 'local' | 'tigris' | null) ?? null,
     // F272.2 — the derived one-word answer to the only question a citation asks:
     // can the owner still open this after this machine is gone?
-    evidence_durability: durabilityOf(row.artifact_backend),
+    evidence_durability: allInCloud ? 'durable' : durabilityOf(row.artifact_backend),
     // F074.99 — the run-level findings a reader can ACT on: a password that came
     // back in the URL, a page that was never listening while we wrote to it.
     // `{}` means the run looked and found nothing; NULL means the row predates
@@ -976,7 +985,7 @@ export async function lensRunFlow(deps: LensDeps, input: RunFlowInput) {
     storage: coerceStorageSeed(input.storage), // F074.97 — seed saved client state
   });
   const row = await recordFlowRun(deps.db, deps.storage, project, result, provenance);
-  return flowRunView(row, deps.port);
+  return flowRunView(row, deps.port, deps.db);
 }
 
 export interface RunManuscriptInput {
@@ -1039,7 +1048,7 @@ export async function lensRunManuscript(deps: LensDeps, input: RunManuscriptInpu
     storage: coerceStorageSeed(input.storage), // F074.97 — seed saved client state
   });
   const row = await recordFlowRun(deps.db, deps.storage, project, result, provenance);
-  return flowRunView(row, deps.port);
+  return flowRunView(row, deps.port, deps.db);
 }
 
 export interface ListFlowRunsInput {
@@ -1056,7 +1065,7 @@ export function lensListFlowRuns(deps: LensDeps, input: ListFlowRunsInput) {
     status: input.status,
     limit: input.limit,
   });
-  return { project, count: runs.length, flow_runs: runs.map((r) => flowRunView(r, deps.port)) };
+  return { project, count: runs.length, flow_runs: runs.map((r) => flowRunView(r, deps.port, deps.db)) };
 }
 
 export function lensListBaselines(deps: LensDeps, project: string) {