@pasko70/pibo 1.4.1 → 1.4.2

package/dist/web-annotations/store.js

@@ -179,11 +179,11 @@ export class WebAnnotationStore {
             createdAt: timestamp,
             updatedAt: timestamp,
         };
-        this.db.prepare(`
-			INSERT OR IGNORE INTO web_annotation_bindings (
-				id, pibo_session_id, pibo_room_id, state, url, title, target_id,
-				created_at, updated_at, last_injected_at, closed_at, error, metadata_json
-			) VALUES (${Array.from({ length: BINDING_COLUMNS.length }, () => "?").join(", ")})
+        this.db.prepare(`
+			INSERT OR IGNORE INTO web_annotation_bindings (
+				id, pibo_session_id, pibo_room_id, state, url, title, target_id,
+				created_at, updated_at, last_injected_at, closed_at, error, metadata_json
+			) VALUES (${Array.from({ length: BINDING_COLUMNS.length }, () => "?").join(", ")})
 		`).run(binding.id, binding.piboSessionId, binding.piboRoomId ?? null, binding.state, binding.url, binding.title ?? null, binding.targetId ?? null, binding.createdAt, binding.updatedAt ?? null, null, null, null, stringifyJson(binding.metadata));
         const existing = this.getBinding(binding.piboSessionId, binding.id);
         if (!existing)
@@ -192,24 +192,24 @@ export class WebAnnotationStore {
     }
     listBindings(input) {
         const limit = normalizeLimit(input.limit, 100, 500);
-        return this.db.prepare(`
-			SELECT * FROM web_annotation_bindings
-			WHERE pibo_session_id = ? AND state != 'removed'
-			ORDER BY created_at DESC, id DESC
-			LIMIT ?
+        return this.db.prepare(`
+			SELECT * FROM web_annotation_bindings
+			WHERE pibo_session_id = ? AND state != 'removed'
+			ORDER BY created_at DESC, id DESC
+			LIMIT ?
 		`).all(input.piboSessionId, limit).map(bindingFromRow);
     }
     getBinding(piboSessionId, id) {
-        const row = this.db.prepare(`
-			SELECT * FROM web_annotation_bindings
-			WHERE id = ? AND pibo_session_id = ?
+        const row = this.db.prepare(`
+			SELECT * FROM web_annotation_bindings
+			WHERE id = ? AND pibo_session_id = ?
 		`).get(id, piboSessionId);
         return row ? bindingFromRow(row) : undefined;
     }
     getBindingById(id) {
-        const row = this.db.prepare(`
-			SELECT * FROM web_annotation_bindings
-			WHERE id = ? AND state != 'removed'
+        const row = this.db.prepare(`
+			SELECT * FROM web_annotation_bindings
+			WHERE id = ? AND state != 'removed'
 		`).get(id);
         return row ? bindingFromRow(row) : undefined;
     }
@@ -221,18 +221,18 @@ export class WebAnnotationStore {
             return undefined;
         const state = normalized.state ?? existing.state;
         const timestamp = nowIso(now);
-        this.db.prepare(`
-			UPDATE web_annotation_bindings
-			SET state = ?, title = ?, target_id = ?, updated_at = ?, last_injected_at = ?, closed_at = ?, error = ?, metadata_json = ?
-			WHERE id = ? AND pibo_session_id = ?
+        this.db.prepare(`
+			UPDATE web_annotation_bindings
+			SET state = ?, title = ?, target_id = ?, updated_at = ?, last_injected_at = ?, closed_at = ?, error = ?, metadata_json = ?
+			WHERE id = ? AND pibo_session_id = ?
 		`).run(state, normalized.title !== undefined ? normalized.title : existing.title ?? null, normalized.targetId !== undefined ? normalized.targetId : existing.targetId ?? null, timestamp, normalized.lastInjectedAt !== undefined ? normalized.lastInjectedAt : existing.lastInjectedAt ?? null, normalized.closedAt !== undefined ? normalized.closedAt : existing.closedAt ?? null, normalized.error !== undefined ? normalized.error : existing.error ?? null, normalized.metadata !== undefined ? stringifyJson(normalized.metadata) : stringifyJson(existing.metadata), id, piboSessionId);
         return this.getBinding(piboSessionId, id);
     }
     removeBinding(piboSessionId, id, now = new Date()) {
-        const result = this.db.prepare(`
-			UPDATE web_annotation_bindings
-			SET state = 'removed', updated_at = ?
-			WHERE id = ? AND pibo_session_id = ?
+        const result = this.db.prepare(`
+			UPDATE web_annotation_bindings
+			SET state = 'removed', updated_at = ?
+			WHERE id = ? AND pibo_session_id = ?
 		`).run(nowIso(now), id, piboSessionId);
         return Number(result.changes ?? 0) > 0;
     }
@@ -258,12 +258,12 @@ export class WebAnnotationStore {
             createdAt: timestamp,
             updatedAt: timestamp,
         };
-        this.db.prepare(`
-			INSERT OR IGNORE INTO web_annotations (
-				id, pibo_session_id, pibo_room_id, binding_id, status, note, url, title,
-				target_id, target_kind, viewport_json, target_json, screenshot_ref_json, thread_json,
-				created_at, updated_at, resolved_at, resolved_by, summary, metadata_json
-			) VALUES (${Array.from({ length: ANNOTATION_COLUMNS.length }, () => "?").join(", ")})
+        this.db.prepare(`
+			INSERT OR IGNORE INTO web_annotations (
+				id, pibo_session_id, pibo_room_id, binding_id, status, note, url, title,
+				target_id, target_kind, viewport_json, target_json, screenshot_ref_json, thread_json,
+				created_at, updated_at, resolved_at, resolved_by, summary, metadata_json
+			) VALUES (${Array.from({ length: ANNOTATION_COLUMNS.length }, () => "?").join(", ")})
 		`).run(annotation.id, annotation.piboSessionId, annotation.piboRoomId ?? null, annotation.bindingId ?? null, annotation.status, annotation.note, annotation.url, annotation.title ?? null, annotation.targetId ?? null, annotation.targetKind, stringifyJson(annotation.viewport), stringifyJson(annotation.target), stringifyJson(annotation.screenshotRef), null, annotation.createdAt, annotation.updatedAt ?? null, null, null, null, stringifyJson(annotation.metadata));
         const existing = this.getAnnotation(annotation.piboSessionId, annotation.id);
         if (!existing)
@@ -285,24 +285,24 @@ export class WebAnnotationStore {
             values.push(input.status);
         }
         const where = clauses.length ? `WHERE ${clauses.join(" AND ")}` : "";
-        return this.db.prepare(`
-			SELECT * FROM web_annotations
-			${where}
-			ORDER BY created_at DESC, id DESC
-			LIMIT ?
+        return this.db.prepare(`
+			SELECT * FROM web_annotations
+			${where}
+			ORDER BY created_at DESC, id DESC
+			LIMIT ?
 		`).all(...values, limit).map(annotationFromRow);
     }
     getAnnotation(piboSessionId, id) {
-        const row = this.db.prepare(`
-			SELECT * FROM web_annotations
-			WHERE id = ? AND pibo_session_id = ?
+        const row = this.db.prepare(`
+			SELECT * FROM web_annotations
+			WHERE id = ? AND pibo_session_id = ?
 		`).get(id, piboSessionId);
         return row ? annotationFromRow(row) : undefined;
     }
     getAnnotationById(id) {
-        const row = this.db.prepare(`
-			SELECT * FROM web_annotations
-			WHERE id = ?
+        const row = this.db.prepare(`
+			SELECT * FROM web_annotations
+			WHERE id = ?
 		`).get(id);
         return row ? annotationFromRow(row) : undefined;
     }
@@ -315,10 +315,10 @@ export class WebAnnotationStore {
         const timestamp = nowIso(now);
         const status = normalized.status ?? existing.status;
         const resolvedAt = status === "resolved" && !existing.resolvedAt ? timestamp : existing.resolvedAt ?? null;
-        this.db.prepare(`
-			UPDATE web_annotations
-			SET status = ?, updated_at = ?, resolved_at = ?, resolved_by = ?, summary = ?, metadata_json = ?
-			WHERE id = ? AND pibo_session_id = ?
+        this.db.prepare(`
+			UPDATE web_annotations
+			SET status = ?, updated_at = ?, resolved_at = ?, resolved_by = ?, summary = ?, metadata_json = ?
+			WHERE id = ? AND pibo_session_id = ?
 		`).run(status, timestamp, resolvedAt, normalized.resolvedBy !== undefined ? normalized.resolvedBy : existing.resolvedBy ?? null, normalized.summary !== undefined ? normalized.summary : existing.summary ?? null, normalized.metadata !== undefined ? stringifyJson(normalized.metadata) : stringifyJson(existing.metadata), id, piboSessionId);
         return this.getAnnotation(piboSessionId, id);
     }
@@ -344,61 +344,61 @@ export class WebAnnotationStore {
             ...thread,
             { id: normalized.id ?? `wat_${randomUUID()}`, role: normalized.role, content: normalized.content, createdAt: timestamp },
         ];
-        this.db.prepare(`
-			UPDATE web_annotations
-			SET thread_json = ?, updated_at = ?
-			WHERE id = ? AND pibo_session_id = ?
+        this.db.prepare(`
+			UPDATE web_annotations
+			SET thread_json = ?, updated_at = ?
+			WHERE id = ? AND pibo_session_id = ?
 		`).run(stringifyJson(nextThread), timestamp, normalized.annotationId, normalized.piboSessionId);
         return this.getAnnotation(normalized.piboSessionId, normalized.annotationId);
     }
     applySchema() {
-        this.db.exec(`
-			CREATE TABLE IF NOT EXISTS web_annotation_bindings (
-				id TEXT PRIMARY KEY,
-				pibo_session_id TEXT NOT NULL,
-				pibo_room_id TEXT,
-				state TEXT NOT NULL,
-				url TEXT NOT NULL,
-				title TEXT,
-				target_id TEXT,
-				created_at TEXT NOT NULL,
-				updated_at TEXT,
-				last_injected_at TEXT,
-				closed_at TEXT,
-				error TEXT,
-				metadata_json TEXT
-			);
-			CREATE INDEX IF NOT EXISTS idx_web_annotation_bindings_session
-				ON web_annotation_bindings(pibo_session_id, created_at DESC);
-			CREATE INDEX IF NOT EXISTS idx_web_annotation_bindings_target
-				ON web_annotation_bindings(target_id, state);
-
-			CREATE TABLE IF NOT EXISTS web_annotations (
-				id TEXT PRIMARY KEY,
-				pibo_session_id TEXT NOT NULL,
-				pibo_room_id TEXT,
-				binding_id TEXT,
-				status TEXT NOT NULL,
-				note TEXT NOT NULL,
-				url TEXT NOT NULL,
-				title TEXT,
-				target_id TEXT,
-				target_kind TEXT NOT NULL,
-				viewport_json TEXT NOT NULL,
-				target_json TEXT,
-				screenshot_ref_json TEXT,
-				thread_json TEXT,
-				created_at TEXT NOT NULL,
-				updated_at TEXT,
-				resolved_at TEXT,
-				resolved_by TEXT,
-				summary TEXT,
-				metadata_json TEXT
-			);
-			CREATE INDEX IF NOT EXISTS idx_web_annotations_session_status_created
-				ON web_annotations(pibo_session_id, status, created_at DESC);
-			CREATE INDEX IF NOT EXISTS idx_web_annotations_session_created
-				ON web_annotations(pibo_session_id, created_at DESC);
+        this.db.exec(`
+			CREATE TABLE IF NOT EXISTS web_annotation_bindings (
+				id TEXT PRIMARY KEY,
+				pibo_session_id TEXT NOT NULL,
+				pibo_room_id TEXT,
+				state TEXT NOT NULL,
+				url TEXT NOT NULL,
+				title TEXT,
+				target_id TEXT,
+				created_at TEXT NOT NULL,
+				updated_at TEXT,
+				last_injected_at TEXT,
+				closed_at TEXT,
+				error TEXT,
+				metadata_json TEXT
+			);
+			CREATE INDEX IF NOT EXISTS idx_web_annotation_bindings_session
+				ON web_annotation_bindings(pibo_session_id, created_at DESC);
+			CREATE INDEX IF NOT EXISTS idx_web_annotation_bindings_target
+				ON web_annotation_bindings(target_id, state);
+
+			CREATE TABLE IF NOT EXISTS web_annotations (
+				id TEXT PRIMARY KEY,
+				pibo_session_id TEXT NOT NULL,
+				pibo_room_id TEXT,
+				binding_id TEXT,
+				status TEXT NOT NULL,
+				note TEXT NOT NULL,
+				url TEXT NOT NULL,
+				title TEXT,
+				target_id TEXT,
+				target_kind TEXT NOT NULL,
+				viewport_json TEXT NOT NULL,
+				target_json TEXT,
+				screenshot_ref_json TEXT,
+				thread_json TEXT,
+				created_at TEXT NOT NULL,
+				updated_at TEXT,
+				resolved_at TEXT,
+				resolved_by TEXT,
+				summary TEXT,
+				metadata_json TEXT
+			);
+			CREATE INDEX IF NOT EXISTS idx_web_annotations_session_status_created
+				ON web_annotations(pibo_session_id, status, created_at DESC);
+			CREATE INDEX IF NOT EXISTS idx_web_annotations_session_created
+				ON web_annotations(pibo_session_id, created_at DESC);
 		`);
     }
 }

package/docs/README.md

@@ -1,23 +1,23 @@
-# Pibo Documentation
-
-This directory is the single home for project documentation, specs, plans, and reports.
-
-## Structure
-
-```text
-docs/
-  project/  Normal/current project docs and other canonical documentation
-  specs/    Product, technical, and implementation specifications
-  plans/    Implementation plans and design plans
-  reports/  Investigation reports, validation reports, and generated report artifacts
-  legacy/   Previous documentation set kept for reference
-```
-
-## Rules
-
-- Put normal/current project docs and other canonical documentation in `docs/project/`.
-- Put specifications in `docs/specs/`.
-- Put implementation plans in `docs/plans/`.
-- Put investigation, validation, and incident reports in `docs/reports/`.
-- Do not create new root-level `plans/`, `reports/`, or `specs/` directories.
-- Do not delete legacy docs unless there is a separate cleanup decision.
+# Pibo Documentation
+
+This directory is the single home for project documentation, specs, plans, and reports.
+
+## Structure
+
+```text
+docs/
+  project/  Normal/current project docs and other canonical documentation
+  specs/    Product, technical, and implementation specifications
+  plans/    Implementation plans and design plans
+  reports/  Investigation reports, validation reports, and generated report artifacts
+  legacy/   Previous documentation set kept for reference
+```
+
+## Rules
+
+- Put normal/current project docs and other canonical documentation in `docs/project/`.
+- Put specifications in `docs/specs/`.
+- Put implementation plans in `docs/plans/`.
+- Put investigation, validation, and incident reports in `docs/reports/`.
+- Do not create new root-level `plans/`, `reports/`, or `specs/` directories.
+- Do not delete legacy docs unless there is a separate cleanup decision.

package/docs/ops/install-developer-host.md

@@ -1,112 +1,112 @@
-# Install Pibo as a Developer Host
-
-Use this path when the host is used to develop Pibo itself.
-
-## Philosophy
-
-Production must stay stable while development moves quickly. The dev gateway can restart, fail, or deploy the `dev` branch without interrupting the main production gateway.
-
-Docker compute workers are part of the developer setup. Each agent should get its own isolated container and gateway so agent restarts do not disturb other agents or the host gateways.
-
-## Target layout
-
-```text
-/root/code/pibo                  main / production
-/root/code/pibo/.worktrees/dev   dev / development
-
-/root/.pibo                      production data
-/root/.pibo-dev                  development data
-
-pibo-web.service                 prod web 4788, gateway 4789
-pibo-web-dev.service             dev web 4808, gateway 4809
-```
-
-## Remotes
-
-Each developer host should use its own fork as `origin` and the canonical repo as `upstream`:
-
-```text
-origin    git@github.com:<server-or-user-fork>/pibo.git
-upstream  git@github.com:Pascapone/pibo.git
-```
-
-## Plan the setup
-
-```bash
-pibo setup developer-host \
-  --origin git@github.com:<server-or-user-fork>/pibo.git \
-  --upstream git@github.com:Pascapone/pibo.git \
-  --prod-domain pibo.example.com \
-  --dev-domain dev.pibo.example.com \
-  --print-files
-```
-
-Review the generated systemd units, dev start wrapper, environment template, and Caddyfile. Stage files before applying when testing a fresh host:
-
-```bash
-pibo setup developer-host \
-  --origin git@github.com:<server-or-user-fork>/pibo.git \
-  --prod-domain pibo.example.com \
-  --dev-domain dev.pibo.example.com \
-  --write-to /tmp/pibo-setup
-```
-
-Apply after review:
-
-```bash
-pibo setup developer-host \
-  --origin git@github.com:<server-or-user-fork>/pibo.git \
-  --prod-domain pibo.example.com \
-  --dev-domain dev.pibo.example.com \
-  --apply --yes
-systemctl daemon-reload
-```
-
-## Important detail: service pinning
-
-Do not globally install the dev worktree over the production install. `npm install -g .` from a local checkout creates a global symlink to that checkout; running it again from the dev worktree can accidentally move `/usr/bin/pibo` to dev.
-
-The generated developer services avoid this by pinning production to:
-
-```text
-/usr/bin/node /root/code/pibo/dist/bin/pibo.js
-```
-
-and dev to the wrapper in `/usr/local/bin/pibo-web-dev-start.mjs`, which imports the dev worktree directly.
-
-Run this in each checkout/worktree instead:
-
-```bash
-npm ci
-npm run build
-```
-
-## Important detail: dev internal gateway port
-
-Do not start dev only with:
-
-```bash
-pibo gateway:web --web-port 4808
-```
-
-That changes the web port but leaves the internal gateway at the default `4789`, which collides with production. The generated developer setup uses a wrapper that calls `runWebGatewayServer` with:
-
-```text
-web port:      4808
-internal port: 4809
-```
-
-## Validate
-
-```bash
-systemctl is-active pibo-web
-systemctl is-active pibo-web-dev
-pibo gateway web status
-PIBO_GATEWAY_DEV_PORT=4808 pibo gateway dev status
-pibo setup doctor --domain pibo.example.com --dev-domain dev.pibo.example.com --expected-ip <server-ip> --require-docker --min-swap-gb 8
-pibo compute spawn --help
-```
-
-DNS must point to the host before Caddy can issue public certificates.
-
-Docker and swap are developer-host prerequisites only. User-host installs can ignore Docker and swap. The setup command does not create swap automatically; provision it at the OS level, then verify it with `pibo setup doctor --min-swap-gb 8`.
+# Install Pibo as a Developer Host
+
+Use this path when the host is used to develop Pibo itself.
+
+## Philosophy
+
+Production must stay stable while development moves quickly. The dev gateway can restart, fail, or deploy the `dev` branch without interrupting the main production gateway.
+
+Docker compute workers are part of the developer setup. Each agent should get its own isolated container and gateway so agent restarts do not disturb other agents or the host gateways.
+
+## Target layout
+
+```text
+/root/code/pibo                  main / production
+/root/code/pibo/.worktrees/dev   dev / development
+
+/root/.pibo                      production data
+/root/.pibo-dev                  development data
+
+pibo-web.service                 prod web 4788, gateway 4789
+pibo-web-dev.service             dev web 4808, gateway 4809
+```
+
+## Remotes
+
+Each developer host should use its own fork as `origin` and the canonical repo as `upstream`:
+
+```text
+origin    git@github.com:<server-or-user-fork>/pibo.git
+upstream  git@github.com:Pascapone/pibo.git
+```
+
+## Plan the setup
+
+```bash
+pibo setup developer-host \
+  --origin git@github.com:<server-or-user-fork>/pibo.git \
+  --upstream git@github.com:Pascapone/pibo.git \
+  --prod-domain pibo.example.com \
+  --dev-domain dev.pibo.example.com \
+  --print-files
+```
+
+Review the generated systemd units, dev start wrapper, environment template, and Caddyfile. Stage files before applying when testing a fresh host:
+
+```bash
+pibo setup developer-host \
+  --origin git@github.com:<server-or-user-fork>/pibo.git \
+  --prod-domain pibo.example.com \
+  --dev-domain dev.pibo.example.com \
+  --write-to /tmp/pibo-setup
+```
+
+Apply after review:
+
+```bash
+pibo setup developer-host \
+  --origin git@github.com:<server-or-user-fork>/pibo.git \
+  --prod-domain pibo.example.com \
+  --dev-domain dev.pibo.example.com \
+  --apply --yes
+systemctl daemon-reload
+```
+
+## Important detail: service pinning
+
+Do not globally install the dev worktree over the production install. `npm install -g .` from a local checkout creates a global symlink to that checkout; running it again from the dev worktree can accidentally move `/usr/bin/pibo` to dev.
+
+The generated developer services avoid this by pinning production to:
+
+```text
+/usr/bin/node /root/code/pibo/dist/bin/pibo.js
+```
+
+and dev to the wrapper in `/usr/local/bin/pibo-web-dev-start.mjs`, which imports the dev worktree directly.
+
+Run this in each checkout/worktree instead:
+
+```bash
+npm ci
+npm run build
+```
+
+## Important detail: dev internal gateway port
+
+Do not start dev only with:
+
+```bash
+pibo gateway:web --web-port 4808
+```
+
+That changes the web port but leaves the internal gateway at the default `4789`, which collides with production. The generated developer setup uses a wrapper that calls `runWebGatewayServer` with:
+
+```text
+web port:      4808
+internal port: 4809
+```
+
+## Validate
+
+```bash
+systemctl is-active pibo-web
+systemctl is-active pibo-web-dev
+pibo gateway web status
+PIBO_GATEWAY_DEV_PORT=4808 pibo gateway dev status
+pibo setup doctor --domain pibo.example.com --dev-domain dev.pibo.example.com --expected-ip <server-ip> --require-docker --min-swap-gb 8
+pibo compute spawn --help
+```
+
+DNS must point to the host before Caddy can issue public certificates.
+
+Docker and swap are developer-host prerequisites only. User-host installs can ignore Docker and swap. The setup command does not create swap automatically; provision it at the OS level, then verify it with `pibo setup doctor --min-swap-gb 8`.