@gotcos/glasses-server 6.44.6 → 6.44.7

package/CHANGELOG.md

@@ -1,3 +1,13 @@
+## 6.44.7
+
+One learning write: a review decision.
+
+- `POST /api/context/learning/:id/review` with `{ "decision": "dismissed" | "reopened" }`
+  appends a review-ledger row through the bridge's `learning-decide`. A
+  dismissed proposal leaves the To review set on the next read; reopened puts
+  it back. Nothing else is written: no skill, no memory, no graph. COS Control
+  0.5.191's Memories tab uses it for Dismiss and Restore proposal.
+
 ## 6.44.6
 
 What /qa found in 6.44.5 before anyone installed it.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotcos/glasses-server",
-  "version": "6.44.6",
+  "version": "6.44.7",
   "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

@@ -776,6 +776,23 @@ export function normalizeGraphPassages(value: unknown): Record<string, unknown>
   }
 }
 
+/** The review decision the bridge wrote back: lesson, decision, stamp, id, who. */
+export function normalizeReviewDecision(value: unknown): Record<string, unknown> | null {
+  const source = asRecord(value)
+  const row = asRecord(source?.decision)
+  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
+  return {
+    lesson_id: lessonId,
+    decision,
+    ts: isoOrAbsent(row.ts) ?? null,
+    event_id: LEARNING_EVENT_ID_PATTERN.test(String(row.event_id ?? '')) ? String(row.event_id) : null,
+    by: stringOrAbsent(row.by, 32) ?? null,
+  }
+}
+
 export function normalizeIndexBuildKickoff(value: unknown): { started: boolean; already_running: boolean; pid: number | null; receipt: Record<string, unknown> | null } {
   const source = asRecord(value) ?? {}
   return {

package/server/lib/python-bridge.ts

@@ -56,6 +56,7 @@ export const LEARNING_COMMANDS = [
   'graph-passages',
   'graph-index-build',
   'learning-to-review',
+  'learning-decide',
 ] as const
 
 // The optional Python bridge is available only when the user points us at a real
@@ -228,6 +229,7 @@ function standaloneNoop(args: string[]): unknown {
     case 'graph-passages':
     case 'graph-index-build':
     case 'learning-to-review':
+    case 'learning-decide':
       return { error: 'cos_pipeline_not_configured' }
     case 'task-rows':
     case 'task-capture':

package/server/routes/memory.ts

@@ -13,7 +13,7 @@ import { searchMemories } from '../lib/context-library-search.js'
 function contextConfigured(): boolean {
   return contextSourceAvailable() !== null
 }
-import { LEARNING_REVIEW_LIMIT,
+import { normalizeReviewDecision, LEARNING_REVIEW_LIMIT,
   GRAPH_ENTITY_ID_LIMIT,
   LEARNING_EVENT_ID_PATTERN,
   MEMORY_ID_PATTERN,
@@ -167,6 +167,26 @@ memoryRouter.get('/context/learning/review', async (req, res) => {
   }
 })
 
+memoryRouter.post('/context/learning/:id/review', async (req, res) => {
+  noStore(res)
+  // The one learning write (6.44.7): a review decision on a lesson, appended to
+  // the review ledger by the bridge. Dismissed leaves To review, reopened
+  // returns; nothing else is touched. The lesson id is a store id, not an event id.
+  const lessonId = String(req.params.id)
+  const decision = typeof req.body?.decision === 'string' ? req.body.decision : ''
+  if (!lessonId || lessonId.length > 200 || CONTROL_CHARACTER.test(lessonId)) { res.status(400).json({ error: 'invalid_lesson_id' }); return }
+  if (decision !== 'dismissed' && decision !== 'reopened') { res.status(400).json({ error: 'invalid_decision' }); 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(['learning-decide', `--id=${lessonId}`, `--decision=${decision}`, ...(note ? [`--note=${note}`] : [])], 8_000)
+    sendBridgeAnswer(res, answer, value => normalizeReviewDecision(value))
+  } catch (error) {
+    console.warn('[context] learning decide bridge failure:', (error as Error).message)
+    res.status(503).json({ error: 'learning_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 }