@gotcos/glasses-server 6.50.0 → 6.50.1

package/CHANGELOG.md

@@ -1,3 +1,13 @@
+## 6.50.1
+
+What the QA pass on 6.50.0 found in the history it serves (6.50.0 was published before the pass finished; nothing it changed regressed, and a client that does not ask for `?turns` sees no difference).
+
+- **Another agent's message is not the user.** A peer-inbox frame ("Another Claude session sent a message: …") is the user's words only when it is a live Continue (`origin.kind: 'peer'`). Teammate reports, idle notifications and other sessions' messages use the same frame on a plain user row with no origin; 20 of them in one real session were served as `role: 'user'` and would have read "YOU" on the lens. They are left out now.
+- **An injected user row never replaces the prompt on the live stream.** An image attachment's `[Image: source: …]`, a skill's "Base directory for this skill" and a compaction summary are written as `isMeta` / `isCompactSummary` user rows right after the real prompt, and each replaced it on the lens's context line (23 of 91 prompt changes in one session). They emit no prompt now; a live Continue (also `isMeta`) still does. This is a fix to the stream's prompt line since 6.48.x, and it lets the glasses match the current turn when they cut the history.
+- **User rows no person typed are left out too:** Claude's `[Request interrupted by user]` marker and the `# AGENTS.md instructions for …` block Codex writes as a user message.
+- **`?turns` above the maximum is clamped to 40**, not refused: a client asking for more gets the most this server serves instead of looking like an older server.
+- Tests: the teammate frame and a plain other-session frame dropped, an interrupted tool row with text dropped, the injected rows emit no prompt while a Continue and a real prompt do, the clamp, and a route test where the history read throws (the page answers 200 without the fields; QA found that promise untested). Mutation gate: four new cases.
+
 ## 6.50.0
 
 Scroll back through a running session's conversation on the glasses.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotcos/glasses-server",
-  "version": "6.50.0",
+  "version": "6.50.1",
   "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/agent-session-turns.ts

@@ -66,8 +66,17 @@ function cursorQuery(text: string): string {
   return start >= 0 && end > start ? text.slice(start + '<user_query>'.length, end) : text
 }
 
+/**
+ * User rows that no person typed (6.50.1, QA on 6.50.0): Claude's interrupt marker
+ * ("[Request interrupted by user]", 7 in one session) and the AGENTS.md block Codex
+ * writes as a user message (a 4,000-character "# AGENTS.md instructions for …" row in
+ * 3 of the last 60 rollouts). Each read as "YOU" on the lens.
+ */
+const NOT_TYPED_BY_THE_USER = /^\s*(?:\[Request interrupted by user|# AGENTS\.md instructions\b)/
+
 function shaped(role: SessionTurn['role'], raw: string, at: string | undefined): SessionTurn | null {
   if (isWrapperPrompt(raw)) return null
+  if (role === 'user' && NOT_TYPED_BY_THE_USER.test(raw)) return null
   const text = latestAssistantReply(raw)
   if (!text) return null
   return at ? { role, text, at } : { role, text }
@@ -88,8 +97,13 @@ export function sessionTurnFromRecord(provider: AgentProvider, obj: Record<strin
     if (!text) return null
     const at = iso(obj.timestamp)
     if (obj.type === 'assistant') return shaped('assistant', text, at)
-    const peer = (obj.origin as { kind?: unknown } | undefined)?.kind === 'peer' ? unwrapPeerMessage(text) : null
-    if (peer !== null) return shaped('user', peer, at)
+    // A peer-inbox message. Only a live Continue (`origin.kind: 'peer'`, 6.49.x) is the
+    // user's own words; every other message in that frame is another agent talking
+    // (a teammate's report, an idle notification, another session's SendMessage),
+    // written as a plain user row with no origin. QA found 20 of those in one session
+    // rendering as "YOU".
+    const peer = unwrapPeerMessage(text)
+    if (peer !== null) return (obj.origin as { kind?: unknown } | undefined)?.kind === 'peer' ? shaped('user', peer, at) : null
     if (obj.isMeta === true || obj.isCompactSummary === true) return null
     return shaped('user', text, at)
   }
@@ -125,7 +139,7 @@ export interface RecentTurnsRead {
   turns: SessionTurn[]
   /** True when the transcript holds messages before the first one returned. */
   more: boolean
-  /** Bytes of the transcript this read covered, for the log. */
+  /** Bytes of the transcript this read covered (tests assert the window growth by it). */
   bytesRead: number
 }
 
@@ -170,9 +184,12 @@ export async function readRecentSessionTurns(
   }
 }
 
-/** `?turns=N`: an integer 1..SESSION_TURNS_MAX, or null (absent or malformed). */
+/** `?turns=N`: an integer, clamped to 1..SESSION_TURNS_MAX; null when absent, zero or
+ *  malformed. Clamped rather than refused above the max, so a client asking for more
+ *  than this server serves gets the most it can have instead of looking like an older
+ *  server with no history at all. */
 export function parseTurnsParam(value: unknown): number | null {
   if (typeof value !== 'string' || !/^\d{1,3}$/.test(value)) return null
   const n = Number(value)
-  return n >= 1 && n <= SESSION_TURNS_MAX ? n : null
+  return n >= 1 ? Math.min(n, SESSION_TURNS_MAX) : null
 }

package/server/lib/session-stream-events.ts

@@ -559,7 +559,16 @@ function draftsFromClaudeRecord(record: Record<string, unknown>): SessionStreamD
     const message = asRecord(record.message)
     if (!message) return []
     const outcomes = outcomeDrafts(record, message)
-    return outcomes.length > 0 ? outcomes : promptDrafts(message)
+    if (outcomes.length > 0) return outcomes
+    // An INJECTED user row is not the query (6.50.0): an image attachment's
+    // "[Image: source: …]", a skill's "Base directory for this skill", a compaction
+    // summary. Each is written as `isMeta` / `isCompactSummary` right after the real
+    // prompt and used to replace it on the lens's context line (23 of 91 prompt changes
+    // in one real session). A live Continue is the exception: it is `isMeta` and IS
+    // the user's words.
+    const liveContinue = (record.origin as { kind?: unknown } | undefined)?.kind === 'peer'
+    if ((record.isMeta === true || record.isCompactSummary === true) && !liveContinue) return []
+    return promptDrafts(message)
   }
 
   const role = typeof record.role === 'string' ? record.role : ''

package/server/routes/agent-sessions.ts

@@ -61,7 +61,7 @@ import {
   type LockHolderSnapshot,
 } from '../lib/occupancy-probes.js'
 import type { OccupancyDirs } from '../lib/thread-occupancy.js'
-import { parseTurnsParam, readRecentSessionTurns } from '../lib/agent-session-turns.js'
+import { parseTurnsParam, readRecentSessionTurns, type RecentTurnsRead } from '../lib/agent-session-turns.js'
 import { cosSpawnedPids } from '../lib/agent-session-ownership-store.js'
 
 export const agentSessionsRouter = Router()
@@ -479,7 +479,7 @@ agentSessionsRouter.get('/agent-sessions/:provider/:sessionId', async (req, res)
     // when asked, so the detail poll and every older client get the same bytes as before.
     // A failure here costs the history, never the detail page.
     const turnsWanted = parseTurnsParam(req.query.turns)
-    let recentTurns: Awaited<ReturnType<typeof readRecentSessionTurns>> | null = null
+    let recentTurns: RecentTurnsRead | null = null
     if (turnsWanted !== null) {
       try {
         recentTurns = await readRecentSessionTurns(provider, found, turnsWanted)