@gotcos/glasses-server 6.44.12 → 6.44.13

package/CHANGELOG.md

@@ -1,3 +1,26 @@
+## 6.44.13
+
+Prune or accept a captured memory, and guardrails that prune nonsense for you.
+
+- Miles 2026-09-07, looking at a captured decision that read "AAAA...":
+  "we need the ability to either prune or accept the memory", and "an
+  automated review ... another person could set their own guardrails that
+  automatically just prune out stuff like this."
+- `POST /api/context/memory/:id/review` `{ decision: accept | prune, note? }`
+  stamps a captured memory as reviewed or deletes it; either way a review
+  ledger row (`accepted`, `pruned`) the learning timeline shows. Both are new
+  event types.
+- `GET` and `PUT /api/context/memory-guardrails`: the user's own rules
+  (minimum words, distinct characters, a repeat-ratio ceiling, banned
+  patterns, and an optional model pass: on or off, the tier, at most N per
+  run). The same rules refuse nonsense at capture time in the bridge's COS.
+- `POST /api/context/memory-guardrails/run` `{ days?, apply?, llm? }` scans
+  the captured memories, judges them (rules first; the model pass against
+  the COS philosophy principles only on what survives, only when enabled),
+  and returns every verdict with its reasons. Nothing is deleted without
+  `apply`; a record a person accepted is never flagged.
+- Includes 6.44.12, never published.
+
 ## 6.44.12
 
 The Knowledge down-select, and a fail-closed embedding gate.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotcos/glasses-server",
-  "version": "6.44.12",
+  "version": "6.44.13",
   "description": "COS Glasses \u2014 self-hosted AI heads-up-display server for Even G2 smart glasses, powered by Claude Code, Codex, Cursor Agent CLI, or local Ollama",
   "type": "module",
   "bin": {

package/server/lib/cos-context-browser.ts

@@ -470,7 +470,8 @@ export function normalizeGraphBlock(value: unknown): GraphBlock | null {
 // ── Recent learning payloads (GET /context/learning, /context/learning/:id) ──
 
 export const LEARNING_EVENT_ID_PATTERN = /^evt_[a-f0-9]{16}$/
-export const LEARNING_EVENT_TYPES = new Set(['captured', 'proposed', 'promotable', 'saved', 'retrieved', 'used', 'checked', 'dismissed', 'reverted', 'reopened', 'consolidated', 'previewed'])
+export const LEARNING_EVENT_TYPES = new Set(['captured', 'proposed', 'promotable', 'saved', 'retrieved', 'used', 'checked', 'dismissed', 'reverted', 'reopened', 'consolidated', 'previewed', 'accepted', 'pruned'])
+export const REVIEW_DECISIONS = new Set(['dismissed', 'reopened', 'accepted', 'pruned'])
 const LEARNING_LIST_LIMIT = 50
 /** The strict To review set is small (121 today); one page shows it whole. */
 export const LEARNING_REVIEW_LIMIT = 200
@@ -783,7 +784,7 @@ export function normalizeReviewDecision(value: unknown): Record<string, unknown>
   if (!row) return null
   const decision = stringOrAbsent(row.decision, 16)
   const lessonId = stringOrAbsent(row.lesson_id, 200)
-  if (!lessonId || (decision !== 'dismissed' && decision !== 'reopened')) return null
+  if (!lessonId || !decision || !REVIEW_DECISIONS.has(decision)) return null
   return {
     lesson_id: lessonId,
     decision,
@@ -1018,6 +1019,47 @@ export function normalizeIngestProgress(value: unknown): Record<string, unknown>
   }
 }
 
+export const GUARDRAIL_LLM_TIERS = ['haiku', 'sonnet', 'opus'] as const
+
+/** The user's memory guardrails (6.44.13), every field bounded. */
+export function normalizeGuardrails(value: unknown): Record<string, unknown> | null {
+  const g = asRecord(value)
+  if (!g) return null
+  const llm = asRecord(g.llm_review) ?? {}
+  return {
+    min_words: integerOrAbsent(g.min_words) ?? 6,
+    min_distinct_chars: integerOrAbsent(g.min_distinct_chars) ?? 8,
+    max_repeat_ratio: typeof g.max_repeat_ratio === 'number' && Number.isFinite(g.max_repeat_ratio) ? g.max_repeat_ratio : 0.5,
+    banned_patterns: (Array.isArray(g.banned_patterns) ? g.banned_patterns : []).filter((p): p is string => typeof p === 'string').slice(0, 50).map(p => p.slice(0, 200)),
+    llm_review: {
+      enabled: llm.enabled === true,
+      model: (GUARDRAIL_LLM_TIERS as readonly string[]).includes(String(llm.model)) ? String(llm.model) : 'haiku',
+      max_per_run: integerOrAbsent(llm.max_per_run) ?? 20,
+    },
+    updated_at: isoOrAbsent(g.updated_at) ?? null,
+    by: stringOrAbsent(g.by, 32) ?? null,
+  }
+}
+
+/** One guardrails run: the receipt and the verdicts with their reasons. */
+export function normalizeGuardrailsRun(value: unknown): Record<string, unknown> {
+  const s = asRecord(value) ?? {}
+  const r = asRecord(s.run) ?? {}
+  const verdicts = (Array.isArray(s.verdicts) ? s.verdicts : []).flatMap((row) => {
+    const v = asRecord(row); const id = v ? stringOrAbsent(v.id, 200) : undefined
+    if (!id) return []
+    const verdict = v!.verdict === 'prune' || v!.verdict === 'keep' || v!.verdict === 'review' ? v!.verdict : 'review'
+    return [{ id, type: stringOrAbsent(v!.type, 32) ?? '', created_at: isoOrAbsent(v!.created_at) ?? null, excerpt: typeof v!.excerpt === 'string' ? v!.excerpt.slice(0, 200) : '', verdict,
+      reasons: (Array.isArray(v!.reasons) ? v!.reasons : []).filter((x): x is string => typeof x === 'string').slice(0, 8).map(x => x.slice(0, 300)), by: stringOrAbsent(v!.by, 16) ?? 'rules', applied: v!.applied === true }]
+  }).slice(0, 500)
+  return {
+    run: { id: stringOrAbsent(r.id, 64) ?? null, started_at: isoOrAbsent(r.started_at) ?? null, ended_at: isoOrAbsent(r.ended_at) ?? null, days: integerOrAbsent(r.days) ?? null,
+      scanned: integerOrAbsent(r.scanned) ?? 0, flagged: integerOrAbsent(r.flagged) ?? 0, review: integerOrAbsent(r.review) ?? 0, kept: integerOrAbsent(r.kept) ?? 0,
+      llm_reviewed: integerOrAbsent(r.llm_reviewed) ?? 0, llm_enabled: r.llm_enabled === true, applied: r.applied === true, pruned: integerOrAbsent(r.pruned) ?? 0 },
+    verdicts,
+  }
+}
+
 export function normalizeContextBrowserStatus(value: unknown): ContextBrowserStatus {
   const source = value && typeof value === 'object' && !Array.isArray(value)
     ? value as Record<string, unknown> : {}

package/server/lib/python-bridge.ts

@@ -67,6 +67,9 @@ export const LEARNING_COMMANDS = [
   'graph-ingest-progress',
   'graph-setup-embedding',
   'graph-setup-extraction',
+  'memory-review',
+  'memory-guardrails',
+  'memory-guardrails-run',
 ] as const
 
 // The optional Python bridge is available only when the user points us at a real
@@ -250,6 +253,9 @@ function standaloneNoop(args: string[]): unknown {
     case 'graph-ingest-progress':
     case 'graph-setup-embedding':
     case 'graph-setup-extraction':
+    case 'memory-review':
+    case 'memory-guardrails':
+    case 'memory-guardrails-run':
       return { error: 'cos_pipeline_not_configured' }
     case 'task-rows':
     case 'task-capture':

package/server/routes/memory.ts

@@ -32,6 +32,8 @@ import { normalizeReviewDecision, LEARNING_REVIEW_LIMIT,
   normalizeIngestProgress,
   normalizeEmbeddingBlock,
   normalizeExtractionBlock,
+  normalizeGuardrails,
+  normalizeGuardrailsRun,
   EMBEDDING_PROVIDERS,
   EXTRACTION_TIERS,
   KNOWLEDGE_SETUP_SAMPLE_MAX,
@@ -201,6 +203,92 @@ memoryRouter.post('/context/learning/:id/review', async (req, res) => {
   }
 })
 
+// ── Memory review and guardrails (6.44.13) ──────────────────────────
+//
+// Accept stamps a captured memory as reviewed; prune deletes it. Both are
+// review-ledger rows the timeline shows. The guardrails are the user's own
+// rules; a run scans, judges (rules, then an optional bounded model pass),
+// and prunes only with apply.
+
+memoryRouter.post('/context/memory/:id/review', async (req, res) => {
+  noStore(res)
+  const memoryId = String(req.params.id)
+  const decision = typeof req.body?.decision === 'string' ? req.body.decision : ''
+  if (!memoryId || memoryId.length > 200 || CONTROL_CHARACTER.test(memoryId)) { res.status(400).json({ error: 'invalid_memory_id' }); return }
+  if (decision !== 'accept' && decision !== 'prune') { res.status(400).json({ error: 'invalid_decision', message: 'decision must be accept or prune' }); return }
+  if (!contextConfigured()) { res.status(503).json({ error: pythonBridgeState() }); return }
+  try {
+    const note = typeof req.body?.note === 'string' ? req.body.note.slice(0, 400) : ''
+    const answer = await callPython(['memory-review', `--id=${memoryId}`, `--decision=${decision}`, ...(note ? [`--note=${note}`] : [])], 20_000)
+    const code = bridgeErrorCode(answer)
+    if (code) { sendSetupError(res, code, answer); return }
+    const a = answer as { decision?: unknown; deleted?: unknown }
+    const row = normalizeReviewDecision({ decision: a.decision })
+    if (!row) { res.status(503).json({ error: 'memory_review_unavailable' }); return }
+    res.json({ decision: row, deleted: a.deleted === true })
+  } catch (error) {
+    console.warn('[context] memory review bridge failure:', (error as Error).message)
+    res.status(503).json({ error: 'memory_unavailable' })
+  }
+})
+
+memoryRouter.get('/context/memory-guardrails', async (_req, res) => {
+  noStore(res)
+  if (!contextConfigured()) { res.status(503).json({ error: pythonBridgeState() }); return }
+  try {
+    const answer = await callPython(['memory-guardrails'], 15_000)
+    const code = bridgeErrorCode(answer)
+    if (code) { sendSetupError(res, code, answer); return }
+    const a = answer as { guardrails?: unknown; philosophy_rubric_lines?: unknown }
+    res.json({ guardrails: normalizeGuardrails(a.guardrails), philosophy_rubric_lines: Number.isInteger(a.philosophy_rubric_lines) ? a.philosophy_rubric_lines : 0 })
+  } catch (error) {
+    console.warn('[context] guardrails bridge failure:', (error as Error).message)
+    res.status(503).json({ error: 'memory_unavailable' })
+  }
+})
+
+memoryRouter.put('/context/memory-guardrails', async (req, res) => {
+  noStore(res)
+  if (!contextConfigured()) { res.status(503).json({ error: pythonBridgeState() }); return }
+  const body = req.body
+  if (!body || typeof body !== 'object' || Array.isArray(body)) { res.status(400).json({ error: 'invalid_guardrails', message: 'the body must be a JSON object' }); return }
+  const text = JSON.stringify(body)
+  if (text.length > 16_000) { res.status(400).json({ error: 'invalid_guardrails', message: 'the guardrails patch is too large' }); return }
+  try {
+    const answer = await callPython(['memory-guardrails', '--stdin'], 15_000, text)
+    const code = bridgeErrorCode(answer)
+    if (code) { sendSetupError(res, code, answer); return }
+    const a = answer as { guardrails?: unknown; philosophy_rubric_lines?: unknown }
+    res.json({ guardrails: normalizeGuardrails(a.guardrails), philosophy_rubric_lines: Number.isInteger(a.philosophy_rubric_lines) ? a.philosophy_rubric_lines : 0 })
+  } catch (error) {
+    console.warn('[context] guardrails bridge failure:', (error as Error).message)
+    res.status(503).json({ error: 'memory_unavailable' })
+  }
+})
+
+/** `{ days?, apply?, llm? }` → scan the captured memories; prune the flagged ones only with apply. Bounded: 30 days, one model pass of max_per_run. */
+memoryRouter.post('/context/memory-guardrails/run', async (req, res) => {
+  noStore(res)
+  if (!contextConfigured()) { res.status(503).json({ error: pythonBridgeState() }); return }
+  const body = (req.body ?? {}) as { days?: unknown; apply?: unknown; llm?: unknown }
+  const days = body.days === undefined || body.days === null ? 30 : Number(body.days)
+  if (!Number.isInteger(days) || days < 1 || days > 3650) { res.status(400).json({ error: 'invalid_days', message: 'days must be an integer from 1 to 3650' }); return }
+  if (body.apply !== undefined && typeof body.apply !== 'boolean') { res.status(400).json({ error: 'invalid_apply', message: 'apply must be true or false' }); return }
+  if (body.llm !== undefined && body.llm !== null && typeof body.llm !== 'boolean') { res.status(400).json({ error: 'invalid_llm', message: 'llm must be true or false' }); return }
+  const argv = ['memory-guardrails-run', `--days=${days}`]
+  if (body.apply === true) argv.push('--apply')
+  if (typeof body.llm === 'boolean') argv.push(`--llm=${body.llm}`)
+  try {
+    const answer = await callPython(argv, body.llm === false ? 60_000 : 400_000)
+    const code = bridgeErrorCode(answer)
+    if (code) { sendSetupError(res, code, answer); return }
+    res.json(normalizeGuardrailsRun(answer))
+  } catch (error) {
+    console.warn('[context] guardrails run bridge failure:', (error as Error).message)
+    res.status(503).json({ error: 'memory_unavailable' })
+  }
+})
+
 memoryRouter.get('/context/learning/:id', async (req, res) => {
   noStore(res)
   if (!LEARNING_EVENT_ID_PATTERN.test(req.params.id)) { res.status(400).json({ error: 'invalid_event_id' }); return }