@gotcos/glasses-server 6.44.14 → 6.44.16

package/CHANGELOG.md

@@ -1,3 +1,38 @@
+## 6.44.16
+
+A G2 recording keeps its session id when its sidecar only exists in the server's own recordings folder.
+
+- Chelsie and Queen, 2026-09-08: "Meetings to review is empty, and it says 3
+  recent meetings have no session id." On Miles's Mac the same three-row case
+  was three KC-week recordings whose markdown reached the operations tree
+  through git while the `.g2-chunks.json` sidecar, which is gitignored there,
+  did not; the row lost its session id and Speakers could not open it.
+- `listCosOperationsMeetings` now reads the sidecar from the server's own
+  recordings root by the same stem when the copy beside the markdown is
+  missing (`recordingsRoot` option, default the data dir). The review lookup
+  already resolves a session through the standalone store, so the row opens.
+  A meeting with no sidecar anywhere still omits the field rather than
+  inventing one.
+
+## 6.44.15
+
+Voice profiles named after a spoken sentence are repaired at load.
+
+- Chelsie 2026-09-08: her `voice-profiles.json` held two profiles whose name
+  was the entire enrolment speech ("My Voice My Name Is Chelsie Hodgkiss And I
+  Am The Director…", ~600 characters), written by a client older than the
+  8/25 guards. Speaker ID was "active" and could never match anything, and the
+  file cannot be repaired by hand because the server rewrites it from memory.
+- `normalizeProfileStore` now renames a name that is too long, has more than
+  four words, or is three-plus words with sentence punctuation to
+  `Unnamed voice N`, keeps every embedding and its provenance, records
+  `renamedFrom` (the first 60 characters) and `needsName: true`, and counts it
+  in the load repairs (`profilesRenamed`). Legacy short labels ("MU",
+  "Speaker 2", "Luke H.") are never touched. `GET /api/voice/profiles` carries
+  `needsName` and `renamedFrom` so COS Control's speaker review can offer the
+  rename; `POST /api/voice/merge-profiles` folds the placeholder into the
+  right person as before.
+
 ## 6.44.14
 
 The Manage sheet's merge, with a preview, a worker and a receipt, plus a

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotcos/glasses-server",
-  "version": "6.44.14",
+  "version": "6.44.16",
   "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-operations-meetings.ts

@@ -19,6 +19,7 @@ import { discoveredDomains, domainAbbreviation as deriveAbbr, isSafeDomainName a
 import type { MeetingDetail, MeetingMeta } from './meeting-store.js'
 import { MEETING_SOURCE_MAX_BYTES, meetingDayCountsFromNames, meetingListLimit } from './meeting-store.js'
 import { meetingVoiceReview, parseSidecarListHead } from './meeting-voice-review.js'
+import { dataPath } from './data-dir.js'
 
 /**
  * The four domains of ONE user's COS. Retained as the documented example layout
@@ -528,9 +529,12 @@ export function listCosOperationsMeetings(options: {
   domain?: string
   month?: string
   day?: string
+  /** The server's own recordings root; tests point it at a temp dir. */
+  recordingsRoot?: string
 } = {}): CosOperationsMeetingMeta[] {
   const operationsDir = resolveCosOperationsDir()
   if (!operationsDir) return []
+  const recordingsRoot = options.recordingsRoot ?? dataPath('recordings')
 
   const scoped = Boolean(options.month || options.day)
   const limit = meetingListLimit(options.limit, scoped)
@@ -569,7 +573,17 @@ export function listCosOperationsMeetings(options: {
               meta.recordId = `ops:${domain}:${month}:${file}`
               meta.mutable = true
               meta.canonicalRecord = `operations/${domain}/meetings/${month}/${file}`
-              const hints = sidecarListHints(monthDir, file)
+              // The sidecar is gitignored in the operations tree, so a G2 recording
+              // whose markdown arrived through git or iCloud (three of Miles's KC-week
+              // recordings, 2026-09-08) sits there without one and the row loses its
+              // session id, which reads as "cannot be reviewed". The server that saved
+              // it still holds the sidecar under its own recordings root by the same
+              // stem; read that one when the copy beside the markdown is missing.
+              let hints = sidecarListHints(monthDir, file)
+              if (!hints.sessionId && recordingsRoot) {
+                const fallbackDir = join(recordingsRoot, month)
+                if (existsSync(fallbackDir)) hints = sidecarListHints(fallbackDir, file)
+              }
               if (hints.sessionId) meta.sessionId = hints.sessionId
               if (hints.speakers.length > 0) {
                 meta.voiceReview = meetingVoiceReview(hints.speakers, hints.sessionId)

package/server/lib/voice-profile-store.ts

@@ -11,6 +11,7 @@
 // by execution rather than by asserting on source text. speaker-embeddings.ts
 // delegates to it and keeps the sherpa-onnx manager in sync.
 
+import { checkSpeakerName, type SpeakerNameRejection } from './speaker-name.js'
 import { existsSync, mkdirSync, readdirSync, statSync, unlinkSync } from 'node:fs'
 import { basename, dirname, join, resolve } from 'node:path'
 import { durableAtomicWriteFileSync, loadJsonOrQuarantine } from './atomic-fs.js'
@@ -21,6 +22,12 @@ export interface VoiceProfile {
   embeddings: number[][]
   /** Provenance: 'manual' | 'fireflies' | 'g2-training' | 'auto:<sessionId>' | … */
   sources?: string[]
+  /** The name on disk was a spoken sentence, not a name (an old client's
+   *  "enroll my voice" fall-through); it was renamed at load and the training
+   *  kept. COS Control's speaker review shows it for renaming. */
+  needsName?: boolean
+  /** The first 60 characters of the name it was loaded with. */
+  renamedFrom?: string
 }
 
 export interface ProfileStore {
@@ -42,6 +49,9 @@ export interface StoreRepairs {
   dimensionMismatch: number
   /** Entries with no usable name, or duplicate names collapsed. */
   profilesDropped: number
+  /** Names that were a whole spoken sentence (too long, too many words, or
+   *  sentence punctuation), renamed to `Unnamed voice N` with the training kept. */
+  profilesRenamed: number
 }
 
 export function emptyRepairs(): StoreRepairs {
@@ -51,17 +61,19 @@ export function emptyRepairs(): StoreRepairs {
     embeddingsDropped: 0,
     dimensionMismatch: 0,
     profilesDropped: 0,
+    profilesRenamed: 0,
   }
 }
 
 export function hasRepairs(r: StoreRepairs): boolean {
   return r.sourcesRealigned > 0 || r.sourcesCoerced > 0 || r.embeddingsDropped > 0
-    || r.dimensionMismatch > 0 || r.profilesDropped > 0
+    || r.dimensionMismatch > 0 || r.profilesDropped > 0 || r.profilesRenamed > 0
 }
 
 export function describeRepairs(r: StoreRepairs): string {
   const parts: string[] = []
   if (r.profilesDropped) parts.push(`${r.profilesDropped} unusable profile(s)`)
+  if (r.profilesRenamed) parts.push(`${r.profilesRenamed} profile(s) named after a spoken sentence, renamed for review`)
   if (r.embeddingsDropped) parts.push(`${r.embeddingsDropped} unusable embedding row(s)`)
   if (r.sourcesRealigned) parts.push(`${r.sourcesRealigned} profile(s) with misaligned sources[]`)
   if (r.sourcesCoerced) parts.push(`${r.sourcesCoerced} null/non-string source slot(s)`)
@@ -106,11 +118,27 @@ export function normalizeProfileStore(raw: unknown): { store: ProfileStore; repa
   if (!Array.isArray(rawProfiles)) return { store: { profiles: [] }, repairs }
 
   const byName = new Map<string, VoiceProfile>()
+  let unnamed = 0
   for (const candidate of rawProfiles) {
-    const name = typeof (candidate as VoiceProfile)?.name === 'string'
+    const rawName = typeof (candidate as VoiceProfile)?.name === 'string'
       ? (candidate as VoiceProfile).name.trim()
       : ''
-    if (!name) { repairs.profilesDropped++; continue }
+    if (!rawName) { repairs.profilesDropped++; continue }
+    // Chelsie's store (2026-09-08) held two profiles named with the entire
+    // enrolment speech (~600 characters), written by a client older than the
+    // 6.8.433 / server 8/25 guards. A name like that can never match a speaker
+    // label, and the file cannot be repaired by hand because the server
+    // rewrites it from memory. Keep the training, give it a placeholder name,
+    // and flag it so COS Control offers a rename. Only the sentence shapes are
+    // renamed; legacy short labels ("MU", "Speaker 2") are left exactly as is.
+    const junk = junkNameReason(rawName)
+    let renamedFrom: string | undefined
+    let name = rawName
+    if (junk) {
+      do { name = `Unnamed voice ${++unnamed}` } while (byName.has(name) || rawProfiles.some(p => (p as VoiceProfile)?.name === name))
+      renamedFrom = rawName.slice(0, 60)
+      repairs.profilesRenamed++
+    }
 
     const rawEmbeddings = Array.isArray((candidate as VoiceProfile).embeddings)
       ? (candidate as VoiceProfile).embeddings as unknown[]
@@ -154,12 +182,23 @@ export function normalizeProfileStore(raw: unknown): { store: ProfileStore; repa
       existing.sources!.push(...sources)
       continue
     }
-    byName.set(name, { name, embeddings, sources })
+    byName.set(name, { name, embeddings, sources, ...(renamedFrom ? { needsName: true, renamedFrom } : {}) })
   }
 
   return { store: { profiles: [...byName.values()] }, repairs }
 }
 
+/** The sentence shapes only: too long, too many words, or sentence punctuation. */
+export function junkNameReason(name: string): SpeakerNameRejection | null {
+  const check = checkSpeakerName(name, { ownerLabel: name })
+  if (check.ok) return null
+  if (check.reason === 'too_long' || check.reason === 'too_many_words') return check.reason
+  // Sentence punctuation alone is not enough: "Luke H." is a legitimate legacy
+  // alias. Three or more words with punctuation is speech, not a name.
+  if (check.reason === 'sentence_like' && name.trim().split(/\s+/).length >= 3) return check.reason
+  return null
+}
+
 export type StoreLoad = {
   store: ProfileStore
   repairs: StoreRepairs

package/server/routes/voice.ts

@@ -540,6 +540,10 @@ voiceRouter.get('/voice/profiles', (_req, res) => {
             embeddings: p.embeddings.length,
             isOwner: p.name === owner,
             sources: bySource,
+            // 6.44.15: a profile whose name on disk was a spoken sentence was
+            // renamed at load; the client offers a rename.
+            needsName: p.needsName === true,
+            ...(p.renamedFrom ? { renamedFrom: p.renamedFrom } : {}),
             // Provenance alignment is now an invariant; surfacing it makes a
             // future regression visible instead of silent.
             sourcesAligned: (p.sources?.length ?? 0) === p.embeddings.length,