@lota-sdk/core 0.1.14 → 0.1.15

package/src/runtime/memory-block.ts

@@ -1,4 +1,5 @@
 import { agentDisplayNames, agentShortDisplayNames, resolveAgentNameAlias } from '../config/agent-defaults'
+import { compactWhitespace } from '../utils/string'
 
 function escapeRegex(value: string): string {
   return value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
@@ -173,8 +174,7 @@ export function createMemoryBlockRuntime(options: CreateMemoryBlockRuntimeOption
       )
       .filter(Boolean)
 
-    const candidate = normalizedLines.join(' ').trim()
-    const collapsed = candidate.replace(/\s+/g, ' ').trim()
+    const collapsed = compactWhitespace(normalizedLines.join(' '))
     if (!collapsed) return ''
     return collapsed
   }
@@ -186,6 +186,7 @@ export function createMemoryBlockRuntime(options: CreateMemoryBlockRuntimeOption
     try {
       const parsed: unknown = JSON.parse(trimmed)
       if (!Array.isArray(parsed)) return []
+      if (!parsed.every((item: unknown) => typeof item === 'object' && item !== null)) return []
       return parsed as MemoryBlockEntry[]
     } catch {
       return []

package/src/runtime/memory-pipeline.ts

@@ -1,3 +1,11 @@
+import { compactWhitespace } from '../utils/string'
+
+const SCORE_WEIGHTS = {
+  durability: { core: 0.35, standard: 0.2, weak: 0.05 },
+  type: { decision: 0.25, preference: 0.18, default: 0.1 },
+  maxContentLength: 120,
+} as const
+
 interface MemoryFactInput {
   content: string
   confidence: number
@@ -117,9 +125,20 @@ function scoreFact<T extends MemoryFactInput>(fact: T): number {
   const durability = fact.durability ?? 'standard'
   const type = fact.type ?? 'fact'
 
-  const durabilityWeight = durability === 'core' ? 0.35 : durability === 'standard' ? 0.2 : 0.05
-  const typeWeight = type === 'decision' ? 0.25 : type === 'fact' ? 0.18 : 0.1
-  const lengthWeight = Math.min(fact.content.length, 120) / 120 / 10
+  const durabilityWeight =
+    durability === 'core'
+      ? SCORE_WEIGHTS.durability.core
+      : durability === 'standard'
+        ? SCORE_WEIGHTS.durability.standard
+        : SCORE_WEIGHTS.durability.weak
+  const typeWeight =
+    type === 'decision'
+      ? SCORE_WEIGHTS.type.decision
+      : type === 'fact'
+        ? SCORE_WEIGHTS.type.preference
+        : SCORE_WEIGHTS.type.default
+  const lengthWeight =
+    Math.min(fact.content.length, SCORE_WEIGHTS.maxContentLength) / SCORE_WEIGHTS.maxContentLength / 10
 
   return confidence + durabilityWeight + typeWeight + lengthWeight
 }
@@ -160,7 +179,7 @@ export function postProcessMemoryFacts<T extends MemoryFactInput>(
   const deduped = new Map<string, T>()
 
   for (const fact of rawFacts) {
-    const content = typeof fact.content === 'string' ? fact.content.replace(/\s+/g, ' ').trim() : ''
+    const content = typeof fact.content === 'string' ? compactWhitespace(fact.content) : ''
     if (!content || content.length < minChars || content.length > maxChars) continue
     const normalizedFact = { ...fact, content }
     const key = normalizeFactForDedupe(content)
@@ -464,7 +483,7 @@ export function createMemoryActionPlan<TRelation extends string = string>(params
   }
 
   for (const [index, item] of params.updates.memory.entries()) {
-    const text = typeof item.text === 'string' ? item.text.replace(/\s+/g, ' ').trim() : ''
+    const text = typeof item.text === 'string' ? compactWhitespace(item.text) : ''
     const itemId = typeof item.id === 'string' ? item.id.trim() : ''
 
     switch (item.event) {

package/src/runtime/plugin-types.ts

@@ -3,7 +3,7 @@ export interface LotaPluginContributions {
   schemaFiles: readonly (string | URL)[]
 }
 
-export interface LotaPlugin<TServices = unknown, TTools = unknown> {
+export interface LotaPlugin<TServices = Record<string, unknown>, TTools = Record<string, unknown>> {
   services: TServices
   tools?: TTools
   contributions: LotaPluginContributions

package/src/runtime/runtime-extensions.ts

@@ -1,7 +1,7 @@
 import type { ToolSet } from 'ai'
 
 import type { RecordIdRef } from '../db/record-id'
-import type { ReadableUploadMetadata } from '../services/attachment.service'
+import type { ReadableUploadMetadata } from '../storage/attachment-types'
 import type { LotaRuntimeWorkerExtensions } from './runtime-worker-registry'
 
 export interface LotaRuntimeBackgroundCursor {
@@ -34,7 +34,7 @@ export interface LotaRuntimeWorkspaceProvider {
   ): Promise<LotaRuntimeWorkspaceLifecycleState> | LotaRuntimeWorkspaceLifecycleState
   readProfileProjectionState?(
     workspace: Record<string, unknown>,
-  ): Promise<LotaRuntimeWorkspaceProjectionState | void> | LotaRuntimeWorkspaceProjectionState | void
+  ): Promise<LotaRuntimeWorkspaceProjectionState | undefined> | LotaRuntimeWorkspaceProjectionState | undefined
   buildPromptSummary?(workspaceId: RecordIdRef): Promise<string | undefined>
   listRecentDomainEvents?(workspaceId: RecordIdRef, limit?: number): Promise<Array<Record<string, unknown>>>
   hasActiveKnowledgeSources?(workspaceId: string): Promise<boolean>
@@ -76,11 +76,87 @@ export interface LotaRuntimeTeamThinkToolsParams {
   toolProviders?: ToolSet
 }
 
+export interface BuildContextParams {
+  workstream: unknown
+  workstreamRef: RecordIdRef
+  orgRef: RecordIdRef
+  userRef: RecordIdRef
+  userName?: string | null
+  workspace: Record<string, unknown>
+  onboardingActive: boolean
+  messageText: string
+  linearInstalled: boolean
+  githubInstalled: boolean
+  indexedRepoContext: unknown
+  promptContext: unknown
+  workspaceLifecycleState: unknown
+  workspaceProfileState: unknown
+  promptSummary: string | undefined
+  recentDomainEvents: Array<Record<string, unknown>>
+  retrievedKnowledgeSection: string | undefined
+  [key: string]: unknown
+}
+
+export interface AfterTurnParams {
+  workstream: unknown
+  workstreamRef: RecordIdRef
+  orgRef: RecordIdRef
+  userRef: RecordIdRef
+  userName?: string | null
+  onboardingActive: boolean
+  referenceUserMessage: unknown
+  assistantMessages: unknown[]
+  latestWorkstreamRecord: unknown
+  latestPersistedState: unknown
+  context: Record<string, unknown> | null
+  [key: string]: unknown
+}
+
+export interface ResolveAgentParams {
+  agentId: string
+  mode: string
+  workstream: unknown
+  workstreamRef: RecordIdRef
+  orgRef: RecordIdRef
+  userRef: RecordIdRef
+  userName?: string | null
+  onboardingActive: boolean
+  linearInstalled: boolean
+  githubInstalled: boolean
+  reasoningProfile: string
+  skills?: string[]
+  additionalInstructionSections?: string[]
+  context: Record<string, unknown> | null
+  [key: string]: unknown
+}
+
+export interface BuildExtraInstructionSectionsParams {
+  workstream: unknown
+  workstreamRef: RecordIdRef
+  orgRef: RecordIdRef
+  userRef: RecordIdRef
+  userName?: string | null
+  workspace: Record<string, unknown>
+  onboardingActive: boolean
+  messageText: string
+  linearInstalled: boolean
+  githubInstalled: boolean
+  indexedRepoContext: unknown
+  promptContext: unknown
+  workspaceLifecycleState: unknown
+  workspaceProfileState: unknown
+  promptSummary: string | undefined
+  recentDomainEvents: Array<Record<string, unknown>>
+  retrievedKnowledgeSection: string | undefined
+  context: Record<string, unknown> | null
+  [key: string]: unknown
+}
+
 export interface LotaRuntimeTurnHooks {
-  buildContext?: (params: Record<string, unknown>) => Promise<Record<string, unknown> | void>
-  afterTurn?: (params: Record<string, unknown>) => Promise<void>
-  resolveAgent?: (params: Record<string, unknown>) => Promise<Record<string, unknown> | void>
-  buildExtraInstructionSections?: (params: Record<string, unknown>) => Promise<string[] | void>
+  buildContext?: (params: BuildContextParams) => Promise<Record<string, unknown> | void>
+  afterTurn?: (params: AfterTurnParams) => Promise<void>
+  resolveAgent?: (params: ResolveAgentParams) => Promise<Record<string, unknown> | void>
+  buildExtraInstructionSections?: (params: BuildExtraInstructionSectionsParams) => Promise<string[] | void>
 }
 
 export interface LotaRuntimeAdapters {
@@ -90,8 +166,8 @@ export interface LotaRuntimeAdapters {
     buildTeamThinkAgentTools?: (params: LotaRuntimeTeamThinkToolsParams) => Promise<{ tools: ToolSet }>
   }
   queues?: {
-    enqueuePostChatOrgAction?: (job: Record<string, unknown>) => Promise<void> | void
-    enqueueOnboardingRepoIndexFollowUp?: (job: Record<string, unknown>) => Promise<void> | void
+    enqueuePostChatOrgAction?: (job: Record<string, unknown>) => Promise<void>
+    enqueueOnboardingRepoIndexFollowUp?: (job: Record<string, unknown>) => Promise<void>
   }
   workers?: {
     connectPluginDatabases?: () => Promise<void>
@@ -115,17 +191,17 @@ let runtimeExtensionsState: RuntimeExtensionsState = {
   extraWorkers: {},
 }
 
-export function configureRuntimeExtensions(params?: {
+export function configureRuntimeExtensions(params: {
   adapters?: LotaRuntimeAdapters
   turnHooks?: LotaRuntimeTurnHooks
   toolProviders?: ToolSet
   extraWorkers?: LotaRuntimeWorkerExtensions
 }): void {
   runtimeExtensionsState = {
-    adapters: params?.adapters ?? {},
-    turnHooks: params?.turnHooks ?? {},
-    toolProviders: params?.toolProviders ?? EMPTY_TOOLS,
-    extraWorkers: params?.extraWorkers ?? {},
+    adapters: params.adapters ?? {},
+    turnHooks: params.turnHooks ?? {},
+    toolProviders: params.toolProviders ?? EMPTY_TOOLS,
+    extraWorkers: params.extraWorkers ?? {},
   }
 }
 

package/src/runtime/title-helpers.ts

@@ -1,12 +1,21 @@
+import { compactWhitespace } from '../utils/string'
+
 const TITLE_WORD_LIMIT = 5
 
 export function limitTitleWords(text: string): string {
-  const words = text.replace(/\s+/g, ' ').trim().split(' ').filter(Boolean)
+  const words = compactWhitespace(text).split(' ').filter(Boolean)
   return words.slice(0, TITLE_WORD_LIMIT).join(' ')
 }
 
 export function deriveTitle(text: string): string {
-  const trimmed = text.replace(/\s+/g, ' ').trim()
+  const trimmed = compactWhitespace(text)
   if (trimmed.length <= 60) return trimmed
   return `${trimmed.slice(0, 57)}...`
 }
+
+export function normalizeTitle(value: string): string {
+  const normalized = compactWhitespace(value)
+    .replace(/^["'`]+|["'`]+$/g, '')
+    .replace(/[.!?,;:]+$/g, '')
+  return normalized.length <= 80 ? normalized : normalized.slice(0, 80).trim()
+}

package/src/runtime/workstream-chat-helpers.ts

@@ -114,14 +114,13 @@ export function collectToolOutputErrors(params: {
     if (typeof part !== 'object') continue
     if (part.type !== undefined && typeof part.type !== 'string') continue
     if (!part.type?.startsWith('tool-')) continue
-    if ((part as Record<string, unknown>).state !== 'output-error') continue
+
+    const p = part as Record<string, unknown>
+    if (p.state !== 'output-error') continue
 
     const toolName = part.type.slice('tool-'.length) || 'unknown'
-    const toolCallId =
-      typeof (part as Record<string, unknown>).toolCallId === 'string' && (part as Record<string, unknown>).toolCallId
-        ? ((part as Record<string, unknown>).toolCallId as string)
-        : 'unknown'
-    const errorTextRaw = (part as Record<string, unknown>).errorText
+    const toolCallId = typeof p.toolCallId === 'string' && p.toolCallId ? p.toolCallId : 'unknown'
+    const errorTextRaw = p.errorText
     const errorText =
       typeof errorTextRaw === 'string' && errorTextRaw.trim()
         ? errorTextRaw.trim()

package/src/runtime/workstream-routing-policy.ts

@@ -36,36 +36,6 @@ export function uniqueMentionOrder(message: string): string[] {
   return ordered
 }
 
-const GTM_STRONG_INTENT_PATTERNS: RegExp[] = [
-  /\bgo[-\s]?to[-\s]?market\b/i,
-  /\bgtm\b/i,
-  /\bcontent\s+marketing\b/i,
-  /\bcontent\s+strategy\b/i,
-  /\bdemand\s+generation\b/i,
-  /\bdistribution\s+strategy\b/i,
-  /\bpositioning\s+strategy\b/i,
-  /\blaunch\s+strategy\b/i,
-  /\blaunch\s+plan\b/i,
-]
-
-const GTM_WEAK_INTENT_PATTERNS: RegExp[] = [/\blaunch\b/i, /\bcommunity\b/i, /\bdistribution\b/i, /\bpositioning\b/i]
-
-export function isGtmIntentMessage(message: string): boolean {
-  const text = message.trim()
-  if (!text) return false
-
-  if (GTM_STRONG_INTENT_PATTERNS.some((pattern) => pattern.test(text))) {
-    return true
-  }
-
-  let weakMatches = 0
-  for (const pattern of GTM_WEAK_INTENT_PATTERNS) {
-    if (pattern.test(text)) weakMatches += 1
-    if (weakMatches >= 2) return true
-  }
-  return false
-}
-
 const HIGH_IMPACT_CLASS_PATTERNS: Array<{ className: HighImpactResponseClass; patterns: RegExp[] }> = [
   {
     className: 'architecture-recommendation',

package/src/runtime/workstream-state.ts

@@ -238,13 +238,23 @@ export interface CompactionOutput {
   stateDelta: WorkstreamStateDelta
 }
 
-export const WORKSTREAM_STATE_MAX_KEY_DECISIONS = 8
-export const WORKSTREAM_STATE_MAX_ACTIVE_CONSTRAINTS = 6
-export const WORKSTREAM_STATE_MAX_TASKS = 10
-export const WORKSTREAM_STATE_MAX_OPEN_QUESTIONS = 5
-export const WORKSTREAM_STATE_MAX_RISKS = 5
-export const WORKSTREAM_STATE_MAX_ARTIFACTS = 10
-export const WORKSTREAM_STATE_MAX_AGENT_CONTRIBUTIONS = 6
+const WORKSTREAM_STATE_MAX_KEY_DECISIONS = 8
+const WORKSTREAM_STATE_MAX_ACTIVE_CONSTRAINTS = 6
+const WORKSTREAM_STATE_MAX_TASKS = 10
+const WORKSTREAM_STATE_MAX_OPEN_QUESTIONS = 5
+const WORKSTREAM_STATE_MAX_RISKS = 5
+const WORKSTREAM_STATE_MAX_ARTIFACTS = 10
+const WORKSTREAM_STATE_MAX_AGENT_CONTRIBUTIONS = 6
+
+export function applyWorkstreamStateCaps(state: WorkstreamState): void {
+  state.keyDecisions = state.keyDecisions.slice(-WORKSTREAM_STATE_MAX_KEY_DECISIONS)
+  state.activeConstraints = state.activeConstraints.slice(-WORKSTREAM_STATE_MAX_ACTIVE_CONSTRAINTS)
+  state.tasks = state.tasks.slice(-WORKSTREAM_STATE_MAX_TASKS)
+  state.openQuestions = state.openQuestions.slice(-WORKSTREAM_STATE_MAX_OPEN_QUESTIONS)
+  state.risks = state.risks.slice(-WORKSTREAM_STATE_MAX_RISKS)
+  state.artifacts = state.artifacts.slice(-WORKSTREAM_STATE_MAX_ARTIFACTS)
+  state.agentContributions = state.agentContributions.slice(-WORKSTREAM_STATE_MAX_AGENT_CONTRIBUTIONS)
+}
 
 export function createEmptyWorkstreamState(now = Date.now()): WorkstreamState {
   return {

package/src/services/attachment.service.ts

@@ -3,7 +3,7 @@ import { recordIdToString } from '../db/record-id'
 import { TABLES } from '../db/tables'
 import { attachmentStorageService } from '../storage/attachment-storage.service'
 import type { UploadedWorkstreamAttachment as SdkUploadedWorkstreamAttachment } from '../storage/attachment-storage.service'
-import type { MessagePartLike, ReadableUploadMetadata as SdkReadableUploadMetadata } from '../storage/attachments.types'
+import type { MessagePartLike, ReadableUploadMetadata as SdkReadableUploadMetadata } from '../storage/attachment-types'
 
 export type ReadableUploadMetadata = SdkReadableUploadMetadata
 

package/src/services/context-compaction.service.ts

@@ -6,9 +6,9 @@ import { recordIdToString } from '../db/record-id'
 import { databaseService } from '../db/service'
 import { TABLES } from '../db/tables'
 import { parseWorkstreamState, toStateFieldsUpdated } from '../runtime/context-compaction'
-import { CONTEXT_SIZE, WORKSTREAM_RAW_TAIL_MESSAGES } from '../runtime/context-compaction-constants'
+import { CONTEXT_WINDOW_TOKENS, WORKSTREAM_RAW_TAIL_MESSAGES } from '../runtime/context-compaction-constants'
 import type { WorkstreamState } from '../runtime/workstream-state'
-import { compactMemoryBlockSummary, contextCompactionRuntime } from './context-compaction-runtime'
+import { compactMemoryBlockSummary, contextCompactionRuntime } from './context-compaction-runtime.singleton'
 import { workstreamMessageService } from './workstream-message.service'
 import { WorkstreamSchema } from './workstream.types'
 
@@ -35,7 +35,7 @@ class ContextCompactionService {
     return contextCompactionRuntime.formatWorkstreamStateForPrompt(state)
   }
 
-  estimateThreshold(contextSize = CONTEXT_SIZE): number {
+  estimateThreshold(contextSize = CONTEXT_WINDOW_TOKENS): number {
     return contextCompactionRuntime.estimateThreshold(contextSize)
   }
 

package/src/services/document-chunk.service.ts

@@ -3,6 +3,7 @@ import { createHash } from 'node:crypto'
 import { chunkMarkdownDocument, chunkPagedDocument, chunkPlainTextDocument } from '../document/org-document-chunking'
 import type { ParsedDocumentChunk } from '../document/org-document-chunking'
 import { getDefaultEmbeddings } from '../embeddings/provider'
+import { CHARS_PER_TOKEN_ESTIMATE } from '../utils/string'
 
 type DocumentChunkEmbeddings = {
   embedDocuments(documents: string[]): Promise<number[][]>
@@ -58,8 +59,10 @@ export class DocumentChunkService {
     return createHash('sha256').update(content).digest('hex')
   }
 
+  // Uses 4 chars/token (conservative estimate for document content which tends
+  // to have longer words than conversational text where 3 chars/token is used).
   estimateTokenCount(content: string): number {
-    return Math.max(1, Math.ceil(content.length / 4))
+    return Math.max(1, Math.ceil(content.length / (CHARS_PER_TOKEN_ESTIMATE + 1)))
   }
 
   async embedQuery(query: string): Promise<number[]> {
@@ -95,37 +98,39 @@ export class DocumentChunkService {
 
     await params.archive(staleVersionRows)
 
-    for (const [index, chunk] of params.chunks.entries()) {
-      const contentHash = this.hashContent(chunk.content)
-      const existingRow = existingByChunkKey.get(chunk.chunkKey)
-      const payload = params.buildPayload({
-        chunk,
-        embedding: embeddings[index] ?? [],
-        contentHash,
-        tokenEstimate: this.estimateTokenCount(chunk.content),
-      })
-
-      seenChunkKeys.add(chunk.chunkKey)
-
-      if (!existingRow) {
-        await params.create(payload)
-        continue
-      }
-
-      const current = params.selectShape(existingRow)
-      const hasChanged =
-        current.contentHash !== contentHash ||
-        current.chunkIndex !== chunk.chunkIndex ||
-        (current.sectionPath ?? null) !== (chunk.sectionPath ?? null) ||
-        (current.pageStart ?? null) !== (chunk.pageStart ?? null) ||
-        (current.pageEnd ?? null) !== (chunk.pageEnd ?? null)
-
-      if (!hasChanged) {
-        continue
-      }
-
-      await params.update(existingRow, payload)
-    }
+    await Promise.all(
+      params.chunks.map(async (chunk, index) => {
+        const contentHash = this.hashContent(chunk.content)
+        const existingRow = existingByChunkKey.get(chunk.chunkKey)
+        const payload = params.buildPayload({
+          chunk,
+          embedding: embeddings[index] ?? [],
+          contentHash,
+          tokenEstimate: this.estimateTokenCount(chunk.content),
+        })
+
+        seenChunkKeys.add(chunk.chunkKey)
+
+        if (!existingRow) {
+          await params.create(payload)
+          return
+        }
+
+        const current = params.selectShape(existingRow)
+        const hasChanged =
+          current.contentHash !== contentHash ||
+          current.chunkIndex !== chunk.chunkIndex ||
+          (current.sectionPath ?? null) !== (chunk.sectionPath ?? null) ||
+          (current.pageStart ?? null) !== (chunk.pageStart ?? null) ||
+          (current.pageEnd ?? null) !== (chunk.pageEnd ?? null)
+
+        if (!hasChanged) {
+          return
+        }
+
+        await params.update(existingRow, payload)
+      }),
+    )
 
     const removedCurrentVersionRows = existingRows.filter((row) => {
       const current = params.selectShape(row)

package/src/services/execution-plan.service.ts

@@ -612,6 +612,7 @@ class ExecutionPlanService {
   ): Promise<PlanNodeSpecRecord[]> {
     const createdRecords: PlanNodeSpecRecord[] = []
 
+    // Sequential: SurrealDB transactions require ordered operations
     for (const compiledNode of nodes) {
       const nodeSpecId = new RecordId(TABLES.PLAN_NODE_SPEC, Bun.randomUUIDv7())
       const created = await tx
@@ -657,6 +658,7 @@ class ExecutionPlanService {
     nodeSpecs: PlanNodeSpecRecord[],
   ): Promise<PlanNodeRunRecord[]> {
     const createdNodeRuns: PlanNodeRunRecord[] = []
+    // Sequential: SurrealDB transactions require ordered operations
     for (const nodeSpec of nodeSpecs) {
       const nodeRunId = new RecordId(TABLES.PLAN_NODE_RUN, Bun.randomUUIDv7())
       const created = await tx

package/src/services/learned-skill.service.ts

@@ -16,6 +16,7 @@ const embeddings = getDefaultEmbeddings()
 const PROMOTION_MIN_USES = 5
 const PROMOTION_MIN_SUCCESS_RATE = 0.6
 
+const ACTIVE_SKILL_FILTER = "AND status IN ['learned', 'verified', 'promoted'] AND archivedAt IS NONE"
 const SKILL_EXISTS_TTL_SECONDS = 120
 const SKILL_EXISTS_KEY_PREFIX = 'skill-exists'
 
@@ -176,8 +177,7 @@ class LearnedSkillService {
       new BoundQuery(
         `SELECT id FROM ${TABLES.LEARNED_SKILL}
          WHERE organizationId = $orgRef
-           AND status IN ['learned', 'verified', 'promoted']
-           AND archivedAt IS NONE
+           ${ACTIVE_SKILL_FILTER}
            AND (agentId IS NONE OR agentId = $agentId)
          LIMIT 1`,
         { orgRef, agentId },
@@ -227,8 +227,7 @@ class LearnedSkillService {
         vector::similarity::cosine(embedding, $embedding) AS similarity
       FROM ${TABLES.LEARNED_SKILL}
       WHERE organizationId = $organizationId
-        AND status IN ['learned', 'verified', 'promoted']
-        AND archivedAt IS NONE
+        ${ACTIVE_SKILL_FILTER}
         AND confidence >= $minConfidence
         AND (agentId IS NONE OR agentId = $agentId)
         AND embedding <|${params.limit}|> $embedding
@@ -311,8 +310,7 @@ class LearnedSkillService {
         vector::similarity::cosine(embedding, $embedding) AS similarity
       FROM ${TABLES.LEARNED_SKILL}
       WHERE organizationId = $organizationId
-        AND status IN ['learned', 'verified', 'promoted']
-        AND archivedAt IS NONE
+        ${ACTIVE_SKILL_FILTER}
         AND embedding <|3|> $embedding
       ORDER BY similarity DESC
       LIMIT 1
@@ -333,8 +331,7 @@ class LearnedSkillService {
         `SELECT *, type::string(id) AS id, type::string(organizationId) AS organizationId
          FROM ${TABLES.LEARNED_SKILL}
          WHERE organizationId = $organizationId
-           AND status IN ['learned', 'verified', 'promoted']
-           AND archivedAt IS NONE
+           ${ACTIVE_SKILL_FILTER}
          ORDER BY createdAt DESC`,
         { organizationId: orgRef },
       ),
@@ -357,8 +354,7 @@ class LearnedSkillService {
          FROM ${TABLES.LEARNED_SKILL}
          WHERE organizationId = $organizationId
            AND hash = $hash
-           AND status IN ['learned', 'verified', 'promoted']
-           AND archivedAt IS NONE
+           ${ACTIVE_SKILL_FILTER}
          LIMIT 1`,
         { organizationId: orgRef, hash },
       ),

package/src/services/{memory.utils.ts → memory-utils.ts}

@@ -1,22 +1,18 @@
 import { MEMORY } from '../config/constants'
 import { VECTOR_SEARCH_OVERFETCH_MULTIPLIER } from '../config/search'
 import type { MemorySearchResult } from '../db/memory-types'
-import { resolveCandidateLimit } from '../runtime/retrieval-pipeline'
+import { compactWhitespace } from '../utils/string'
 import type { MemoryRerankOutput } from './memory.service'
 
 export function getCandidateLimit(limit: number): number {
-  return resolveCandidateLimit({
-    limit,
-    multiplier: VECTOR_SEARCH_OVERFETCH_MULTIPLIER,
-    minimum: MEMORY.DEFAULT_CANDIDATE_LIMIT,
-  })
+  return Math.max(limit * VECTOR_SEARCH_OVERFETCH_MULTIPLIER, MEMORY.DEFAULT_CANDIDATE_LIMIT)
 }
 
 export function formatMemoryResults(results: MemorySearchResult[]): string {
   if (results.length === 0) return 'No stored memories.'
 
   const normalize = (value: string) => {
-    const trimmed = value.replace(/\s+/g, ' ').trim()
+    const trimmed = compactWhitespace(value)
     if (trimmed.length <= 400) return trimmed
     return `${trimmed.slice(0, 400)}...`
   }
@@ -65,7 +61,7 @@ export function formatRerankedResults(
       used.add(candidate.id)
       total += 1
       const reason = item.relevance ? ` — ${item.relevance}` : ''
-      const trimmed = candidate.content.replace(/\s+/g, ' ').trim()
+      const trimmed = compactWhitespace(candidate.content)
       const content = trimmed.length <= 400 ? trimmed : `${trimmed.slice(0, 400)}...`
       lines.push(`- ${content}${reason}`)
       if (total >= limit) break