saeeol 1.2.8 → 1.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
-  "version": "1.2.8",
+  "version": "1.3.0",
   "name": "saeeol",
   "type": "module",
   "license": "Apache-2.0",
@@ -50,8 +50,8 @@
     "@babel/core": "7.28.4",
     "@effect/language-service": "0.84.2",
     "@octokit/webhooks-types": "7.6.1",
-    "@saeeol/script": "7.3.4",
-    "@saeeol/core": "7.3.4",
+    "@saeeol/script": "7.3.6",
+    "@saeeol/core": "7.3.6",
     "@parcel/watcher-darwin-arm64": "2.5.1",
     "@parcel/watcher-darwin-x64": "2.5.1",
     "@parcel/watcher-linux-arm64-glibc": "2.5.1",
@@ -133,14 +133,14 @@
     "@parcel/watcher": "2.5.1",
     "@pierre/diffs": "1.1.0-beta.18",
     "@saeeol/boxes": "0.2.0",
-    "@saeeol/core": "7.3.4",
-    "@saeeol/gateway": "7.3.3",
-    "@saeeol/i18n": "7.3.4",
-    "@saeeol/indexing": "7.3.3",
-    "@saeeol/plugin": "7.3.4",
-    "@saeeol/script": "7.3.4",
-    "@saeeol/sdk": "7.3.4",
-    "@saeeol/telemetry": "7.3.3",
+    "@saeeol/core": "7.3.6",
+    "@saeeol/gateway": "7.3.6",
+    "@saeeol/i18n": "7.3.6",
+    "@saeeol/indexing": "7.3.6",
+    "@saeeol/plugin": "7.3.6",
+    "@saeeol/script": "7.3.6",
+    "@saeeol/sdk": "7.3.6",
+    "@saeeol/telemetry": "7.3.6",
     "@solid-primitives/event-bus": "1.1.2",
     "@solid-primitives/scheduled": "1.5.2",
     "@standard-schema/spec": "1.0.0",

package/src/cli/cmd/tui/component/dialog/dialog-command.tsx

@@ -1,4 +1,4 @@
-import { useDialog } from "@tui/ui/dialog"
+import { useDialog } from "@tui/ui/dialog"
 import { DialogSelect, type DialogSelectOption, type DialogSelectRef } from "@tui/ui/dialog-select"
 import {
   createContext,
@@ -74,7 +74,6 @@ function init() {
     for (const option of entries()) {
       if (!isEnabled(option)) continue
       if (option.keybind && keybind.match(option.keybind, evt)) {
-        // Require double-tap for agent cycle keybinds
         if (DOUBLE_TAB_KEYS.has(option.keybind)) {
           const now = Date.now()
           const match = option.keybind === lastTabKey && now - lastTabTime < DOUBLE_TAB_WINDOW
@@ -187,4 +186,4 @@ function DialogCommand(props: { options: CommandOption[]; suggestedOptions: Comm
     return [...props.suggestedOptions, ...props.options]
   }
   return <DialogSelect ref={(r) => (ref = r)} title={t("cmd.title")} options={list()} />
-}
+}

package/src/cli/cmd/tui/component/dialog/dialog-mcp.tsx

@@ -27,7 +27,6 @@ export function DialogMcp() {
   const [loading, setLoading] = createSignal<string | null>(null)
 
   const options = createMemo(() => {
-    // Track sync data and loading state to trigger re-render when they change
     const mcpData = sync.data.mcp
     const loadingMcp = loading()
 
@@ -56,7 +55,6 @@ export function DialogMcp() {
         setLoading(option.value)
         try {
           await local.mcp.toggle(option.value)
-          // Refresh MCP status from server
           const status = await sdk.client.mcp.status()
           if (status.data) {
             sync.set("mcp", status.data)

package/src/cli/cmd/tui/component/dialog/dialog-stash.tsx

@@ -36,7 +36,6 @@ export function DialogStash(props: { onSelect: (entry: StashEntry) => void }) {
 
   const options = createMemo(() => {
     const entries = stash.list()
-    // Show most recent first
     return entries
       .map((entry, index) => {
         const isDeleting = toDelete() === index

package/src/cli/cmd/tui/context/app/helper.tsx

@@ -1,4 +1,4 @@
-import { createContext, Show, useContext, type ParentProps } from "solid-js"
+import { createContext, Show, useContext, type ParentProps } from "solid-js"
 
 export function createSimpleContext<T, Props extends Record<string, any>>(input: {
   name: string
@@ -22,4 +22,4 @@ export function createSimpleContext<T, Props extends Record<string, any>>(input:
       return value
     },
   }
-}
+}

package/src/cli/cmd/tui/context/app/sdk.tsx

@@ -1,4 +1,4 @@
-import { createSaeeolClient } from "@saeeol/sdk/v2"
+import { createSaeeolClient } from "@saeeol/sdk/v2"
 import type { GlobalEvent } from "@saeeol/sdk/v2"
 import { createSimpleContext } from "./helper"
 import { createGlobalEmitter } from "@solid-primitives/event-bus"
@@ -100,8 +100,6 @@ export const { use: useSDK, provider: SDKProvider } = createSimpleContext({
           if (queue.length > 0) flush()
           attempt += 1
           if (abort.signal.aborted || ctrl.signal.aborted) break
-
-          // Exponential backoff
           const backoff = Math.min(retryDelay * 2 ** (attempt - 1), maxRetryDelay)
           await new Promise((resolve) => setTimeout(resolve, backoff))
         }
@@ -139,4 +137,4 @@ export const { use: useSDK, provider: SDKProvider } = createSimpleContext({
       url: props.url,
     }
   },
-})
+})

package/src/cli/cmd/tui/context/app/sync.tsx

@@ -565,7 +565,6 @@ export const { use: useSync, provider: SyncProvider } = createSimpleContext({
         })
         .then(() => {
           if (store.status !== "complete") setStore("status", "partial")
-          // non-blocking
           void Promise.all([
             ...(args.continue ? [] : [sessionListPromise.then((sessions) => setStore("session", reconcile(sessions)))]),
             consoleStatePromise.then((consoleState) => setStore("console_state", reconcile(consoleState))),

package/src/cli/cmd/tui/context/app/theme.tsx

@@ -233,8 +233,8 @@ export const { use: useTheme, provider: ThemeProvider } = createSimpleContext({
     const subtleSyntax = createMemo(() => generateSubtleSyntax(values()))
     return {
       theme: new Proxy({} as Theme, {
+        // @ts-expect-error
         get(_target, prop) {
-          // @ts-expect-error
           return values()[prop]
         },
       }),

package/src/cli/cmd/tui/context/runtime/local.tsx

@@ -99,7 +99,6 @@ export const { use: useLocal, provider: LocalProvider } = createSimpleContext({
           if (agent?.color) {
             const color = agent.color
             if (color.startsWith("#")) return RGBA.fromHex(color)
-            // already validated by config, just satisfying TS here
             return theme[color as keyof typeof theme] as RGBA
           }
           return colors()[index % colors().length]
@@ -440,10 +439,8 @@ export const { use: useLocal, provider: LocalProvider } = createSimpleContext({
       async toggle(name: string) {
         const status = sync.data.mcp[name]
         if (status?.status === "connected") {
-          // Disable: disconnect the MCP
           await sdk.client.mcp.disconnect({ name })
         } else {
-          // Enable/Retry: connect the MCP (handles disabled, failed, and other states)
           await sdk.client.mcp.connect({ name })
         }
       },

package/src/ltm/config.ts

@@ -1,4 +1,4 @@
-/** LTM — hardware auto-detection + deterministic LLM parameter calculation */
+/** LTM — hardware auto-detection + deterministic LLM parameter calculation */
 
 import os from "os"
 import { Effect } from "effect"
@@ -10,8 +10,6 @@ import type { HardwareProfile, LLMBakeParams, LTMConfig } from "./types"
 
 const log = Log.create({ service: "ltm/config" })
 
-// ── Default LTM config ──
-
 export const DEFAULT_LTM_CONFIG: LTMConfig = {
   enabled: false,
   embeddingModel: "auto",
@@ -23,8 +21,6 @@ export const DEFAULT_LTM_CONFIG: LTMConfig = {
   retrieval: { topK: 5, minScore: 0.7, maxTokens: 2000 },
 }
 
-// ── Hardware profiling ──
-
 /** Collect system hardware information */
 export async function profileHardware(): Promise<HardwareProfile> {
   const gpu = await Effect.runPromise(GPU.profile)
@@ -51,8 +47,6 @@ export function hardwareHash(hw: HardwareProfile): string {
   return h.toString(36)
 }
 
-// ── Embedding model selection ──
-
 /** Auto-select embedding model based on available VRAM */
 export function selectEmbeddingModel(hw: HardwareProfile) {
   const vram = hw.availableVRAMMB
@@ -63,8 +57,6 @@ export function selectEmbeddingModel(hw: HardwareProfile) {
   return RAG.EMBEDDING_MODELS[5] // all-minilm-l6, 384d, 80MB
 }
 
-// ── Bake (deterministic parameter calculation) ──
-
  /**
   * Compute hardware-based LLM parameters and persist to disk.
   * Once computed, the same values are reused as long as hardware doesn't change.
@@ -96,8 +88,6 @@ export async function bake(hw: HardwareProfile): Promise<LLMBakeParams> {
   return params
 }
 
-// ── Persistence ──
-
 import path from "path"
 import { mkdir, readFile, writeFile } from "fs/promises"
 
@@ -121,4 +111,4 @@ async function readBake(): Promise<LLMBakeParams | undefined> {
 async function writeBake(params: LLMBakeParams): Promise<void> {
   await mkdir(bakeDir(), { recursive: true })
   await writeFile(bakePath(), JSON.stringify(params, null, 2))
-}
+}

package/src/ltm/memory/procedural.ts

@@ -1,4 +1,4 @@
-/** LTM procedural memory — user coding preferences and patterns (English, LLM-to-LLM) */
+/** LTM procedural memory — user coding preferences and patterns (English, LLM-to-LLM) */
 
 import * as Log from "@saeeol/core/util/log"
 import * as Embedder from "@/provider/local/embedder"
@@ -20,8 +20,6 @@ export function extractStyleSignals(
 ): StyleSignal[] {
   const signals: StyleSignal[] = []
   const ext = filePath.split(".").pop() ?? ""
-
-  // Comment language
   if (content.includes("//") && (ext === "ts" || ext === "js")) {
     const hasKoreanComment = /[ㄱ-ㅎㅏ-ㅣ가-힣]/.test(content)
     signals.push({
@@ -30,15 +28,11 @@ export function extractStyleSignals(
       evidence: hasKoreanComment ? "comments in Korean" : "comments in English",
     })
   }
-
-  // Indentation
   if (content.includes("  ") && !content.includes("\t")) {
     signals.push({ language: ext, pattern: "indent", evidence: "2-space indentation" })
   } else if (content.includes("\t")) {
     signals.push({ language: ext, pattern: "indent", evidence: "tab indentation" })
   }
-
-  // Semicolons
   if (ext === "ts" || ext === "js") {
     const hasSemicolons = /;\s*\n/.test(content)
     signals.push({
@@ -47,8 +41,6 @@ export function extractStyleSignals(
       evidence: hasSemicolons ? "uses semicolons" : "no semicolons",
     })
   }
-
-  // Quote style
   const singleQuotes = (content.match(/'/g) ?? []).length
   const doubleQuotes = (content.match(/"/g) ?? []).length
   if (singleQuotes > doubleQuotes * 2) {
@@ -56,8 +48,6 @@ export function extractStyleSignals(
   } else if (doubleQuotes > singleQuotes * 2) {
     signals.push({ language: ext, pattern: "quotes", evidence: "prefers double quotes" })
   }
-
-  // Naming: camelCase vs snake_case
   const camelCase = (content.match(/[a-z][A-Z]/g) ?? []).length
   const snakeCase = (content.match(/_[a-z]/g) ?? []).length
   if (camelCase > snakeCase * 3) {
@@ -99,4 +89,4 @@ export async function fromStyleSignals(
   }
 
   return memories
-}
+}

package/src/ltm/pipeline.ts

@@ -1,4 +1,4 @@
-/** LTM — background collection pipeline */
+/** LTM — background collection pipeline */
 
 import { Effect } from "effect"
 import * as Log from "@saeeol/core/util/log"
@@ -19,8 +19,6 @@ let running = false
 let config: LTMConfig | undefined
 let unsubscribers: Array<() => void> = []
 
-// ── Pipeline lifecycle ──
-
 /** Start the pipeline */
 export async function start(cfg: LTMConfig, bake: LLMBakeParams): Promise<void> {
   if (running) return
@@ -31,24 +29,18 @@ export async function start(cfg: LTMConfig, bake: LLMBakeParams): Promise<void>
 
   log.info("pipeline starting", { model: bake.embeddingModel })
 
-  // Start embedding server
   const server = await Embedder.start(bake)
   if (server.status !== "running") {
     log.error("embedding server failed to start, pipeline disabled")
     running = false
     return
   }
-
-  // Prune old memories
-  if (cfg.episodic.enabled) {
-    const pruned = await Store.prune(cfg.episodic.retainDays * 24 * 60 * 60 * 1000)
+  if (cfg.episodic.enabled) {    const pruned = await Store.prune(cfg.episodic.retainDays * 24 * 60 * 60 * 1000)
     if (pruned > 0) {
       log.info("pruned old episodic memories", { count: pruned })
       void Bus.publish(LTMEvent.MemoryPruned, { count: pruned, type: "episodic" })
     }
   }
-
-  // Enforce memory limit
   const count = await Store.count()
   if (count > cfg.maxMemories) {
     const excess = count - cfg.maxMemories
@@ -58,8 +50,6 @@ export async function start(cfg: LTMConfig, bake: LLMBakeParams): Promise<void>
     await Store.remove(toRemove)
     log.info("trimmed memories to max", { removed: toRemove.length })
   }
-
-  // Bind BusEvent subscriptions
   bindSubscriptions()
 
   log.info("pipeline started", { memoryCount: await Store.count() })
@@ -69,8 +59,6 @@ export async function start(cfg: LTMConfig, bake: LLMBakeParams): Promise<void>
 export async function stop(): Promise<void> {
   if (!running) return
   running = false
-
-  // Unsubscribe
   for (const unsub of unsubscribers) {
     try { unsub() } catch { /* ignore */ }
   }
@@ -86,8 +74,6 @@ export function isActive(): boolean {
   return running
 }
 
-// ── BusEvent subscription binding ──
-
 function bindSubscriptions() {
   if (!config) return
 
@@ -169,8 +155,6 @@ function bindSubscriptions() {
   log.info("bus subscriptions bound", { count: unsubscribers.length })
 }
 
-// ── Direct-call handlers (called explicitly from external modules) ──
-
 /** Conversation message → episodic memory */
 export async function onMessageCompleted(
   sessionID: string,
@@ -254,4 +238,4 @@ export async function onCodeEdit(
   for (const memory of memories) {
     await Store.upsert(memory)
   }
-}
+}

package/src/ltm/scheduler.ts

@@ -1,4 +1,4 @@
-/** LTM — VRAM/task scheduler */
+/** LTM — VRAM/task scheduler */
 
 import * as Log from "@saeeol/core/util/log"
 import type { HardwareProfile } from "./types"
@@ -18,8 +18,6 @@ export interface Allocation {
 /** Scheduling strategy */
 export type Strategy = "concurrent" | "alternating" | "cpu-fallback" | "no-gpu"
 
-// ── VRAM gauge ──
-
 /** Query current VRAM allocation state */
 export async function allocation(): Promise<Allocation> {
   const gpu = await Effect.runPromise(GPU.profile)
@@ -40,14 +38,11 @@ export async function strategy(): Promise<Strategy> {
   return "cpu-fallback"
 }
 
-// ── Execution decisions ──
-
 /** Whether embedding background work can run */
 export async function canRunEmbedding(): Promise<boolean> {
   const strat = await strategy()
   // no-gpu or cpu-fallback: always allowed (CPU processing)
   if (strat === "no-gpu" || strat === "cpu-fallback") return true
-  // concurrent: always allowed
   if (strat === "concurrent") return true
   // alternating: need at least 2GB VRAM
   const alloc = await allocation()
@@ -61,20 +56,13 @@ export async function canRunConcurrent(hw: HardwareProfile): Promise<boolean> {
   return false
 }
 
-// ── LLM ↔ embedding time-sharing ──
-
 /** Called when LLM requests VRAM — pauses embedding if needed */
 export async function requestLLM(vramNeededMB: number): Promise<boolean> {
   const strat = await strategy()
-
-  // concurrent strategy: both can run
   if (strat === "concurrent") return true
-  // no-gpu: GPU not needed
   if (strat === "no-gpu") return true
 
   const alloc = await allocation()
-
-  // Sufficient VRAM — proceed as-is
   if (alloc.available >= vramNeededMB) {
     log.info("LLM request: enough VRAM", { available: alloc.available, needed: vramNeededMB })
     return true
@@ -126,4 +114,4 @@ export async function resumeEmbedding(bake: {
   } catch (e) {
     log.error("failed to resume embedding server", { error: e })
   }
-}
+}

package/src/ltm/store.ts

@@ -1,4 +1,4 @@
-/** LTM — filesystem-based vector store */
+/** LTM — filesystem-based vector store */
 
 import path from "path"
 import { mkdir, readFile, writeFile, readdir, rm, stat } from "fs/promises"
@@ -8,8 +8,6 @@ import type { Memory, MemoryType } from "./types"
 
 const log = Log.create({ service: "ltm/store" })
 
-// ── Cosine similarity ──
-
 function cosine(a: number[], b: number[]): number {
   let dot = 0
   let na = 0
@@ -24,14 +22,11 @@ function cosine(a: number[], b: number[]): number {
   return denom === 0 ? 0 : dot / denom
 }
 
-// ── File paths ──
-
 function storeDir(): string {
   return path.join(Global.Path.data, "ltm", "memories")
 }
 
 function memoryPath(id: string): string {
-  // Sanitize special characters for safe filenames
   const safe = id.replace(/[:<>\"|?*]/g, "_")
   return path.join(storeDir(), `${safe}.json`)
 }
@@ -44,8 +39,6 @@ async function ensure(): Promise<void> {
   await mkdir(storeDir(), { recursive: true })
 }
 
-// ── Index ──
-
 interface Index {
   memories: Array<{ id: string; type: MemoryType; timestamp: number; source: string }>
 }
@@ -64,8 +57,6 @@ async function writeIndex(idx: Index): Promise<void> {
   await writeFile(indexPath(), JSON.stringify(idx, null, 2))
 }
 
-// ── Public API ──
-
 export async function upsert(memory: Memory): Promise<void> {
   await ensure()
   await writeFile(memoryPath(memory.id), JSON.stringify(memory, null, 2))
@@ -149,4 +140,4 @@ export async function prune(olderThanMs: number): Promise<number> {
 export async function count(): Promise<number> {
   const idx = await readIndex()
   return idx.memories.length
-}
+}

package/src/ltm/types.ts

@@ -1,14 +1,12 @@
-/** LTM — long-term memory type definitions */
+/** LTM — long-term memory type definitions */
 
 import { Schema } from "effect"
 import { optionalOmitUndefined } from "@/util/schema"
 
-// ── Memory types ──
 
 export const MemoryType = Schema.Literals(["episodic", "semantic", "procedural"])
 export type MemoryType = Schema.Schema.Type<typeof MemoryType>
 
-// ── Memory entry ──
 
 export const MemoryMetadata = Schema.Struct({
   source: Schema.String,
@@ -30,7 +28,6 @@ export const Memory = Schema.Struct({
 })
 export type Memory = Schema.Schema.Type<typeof Memory>
 
-// ── Embedding server ──
 
 export const EmbedderStatus = Schema.Literals(["stopped", "starting", "running", "error"])
 export type EmbedderStatus = Schema.Schema.Type<typeof EmbedderStatus>
@@ -45,7 +42,6 @@ export const EmbeddingServer = Schema.Struct({
 })
 export type EmbeddingServer = Schema.Schema.Type<typeof EmbeddingServer>
 
-// ── Hardware profile ──
 
 export const HardwareProfile = Schema.Struct({
   gpuCount: Schema.Number,
@@ -57,7 +53,6 @@ export const HardwareProfile = Schema.Struct({
 })
 export type HardwareProfile = Schema.Schema.Type<typeof HardwareProfile>
 
-// ── LLM parameters (deterministic) ──
 
 export const LLMBakeParams = Schema.Struct({
   /** Embedding model ID */
@@ -79,7 +74,6 @@ export const LLMBakeParams = Schema.Struct({
 })
 export type LLMBakeParams = Schema.Schema.Type<typeof LLMBakeParams>
 
-// ── LTM configuration ──
 
 export const LTMConfig = Schema.Struct({
   enabled: Schema.Boolean,
@@ -105,4 +99,4 @@ export const LTMConfig = Schema.Struct({
     maxTokens: Schema.Number,
   }),
 })
-export type LTMConfig = Schema.Schema.Type<typeof LTMConfig>
+export type LTMConfig = Schema.Schema.Type<typeof LTMConfig>

package/src/provider/local/embedder.ts

@@ -1,4 +1,4 @@
-/** Local embedding server — Ollama-based embedding model lifecycle management */
+/** Local embedding server — Ollama-based embedding model lifecycle management */
 
 import { Effect } from "effect"
 import * as Log from "@saeeol/core/util/log"
@@ -17,8 +17,6 @@ const log = Log.create({ service: "local/embedder" })
 
 let server: EmbeddingServer | undefined
 
-// ── Ollama status checks ──
-
 async function isOllamaRunning(endpoint: string): Promise<boolean> {
   try {
     const res = await fetch(`${endpoint}/api/tags`, { signal: AbortSignal.timeout(3000) })
@@ -53,8 +51,6 @@ async function isModelInstalled(endpoint: string, model: string): Promise<boolea
   }
 }
 
-// ── Pull model from Ollama ──
-
 async function pullModel(endpoint: string, model: string): Promise<void> {
   log.info("pulling embedding model", { model })
   const ollamaModel = getOllamaModelName(model)
@@ -91,8 +87,6 @@ function getOllamaModelName(modelId: string): string {
   return map[modelId] ?? modelId
 }
 
-// ── Embedding API call ──
-
 async function embedViaOllama(endpoint: string, model: string, texts: string[]): Promise<number[][]> {
   const ollamaModel = getOllamaModelName(model)
   const res = await fetch(`${endpoint}/api/embed`, {
@@ -114,16 +108,12 @@ async function embedViaOllama(endpoint: string, model: string, texts: string[]):
   return data.embeddings
 }
 
-// ── VRAM estimation ──
-
 function estimateEmbeddingVRAM(modelId: string): number {
   const model = RAG.EMBEDDING_MODELS.find((m) => m.id === modelId)
   if (!model) return 200
   return Math.ceil(model.sizeBytes * 1.2 / (1024 * 1024))
 }
 
-// ── Public API ──
-
 /** Start the embedding server */
 export async function start(bake: LLMBakeParams): Promise<EmbeddingServer> {
   if (server && server.status === "running") {
@@ -146,8 +136,6 @@ export async function start(bake: LLMBakeParams): Promise<EmbeddingServer> {
   }
 
   void Bus.publish(LTMEvent.EmbedderStatusChanged, { status: "starting", model: bake.embeddingModel })
-
-  // Check if Ollama is running
   const running = await isOllamaRunning(endpoint)
   if (!running) {
     log.warn("Ollama not running, embedding server unavailable", { endpoint })
@@ -155,16 +143,12 @@ export async function start(bake: LLMBakeParams): Promise<EmbeddingServer> {
     void Bus.publish(LTMEvent.EmbedderStatusChanged, { status: "error", model: bake.embeddingModel })
     return server
   }
-
-  // Check if model is installed
   const ollamaModel = getOllamaModelName(bake.embeddingModel)
   const installed = await isModelInstalled(endpoint, ollamaModel)
   if (!installed) {
     log.info("model not installed, pulling", { model: ollamaModel })
     await pullModel(endpoint, ollamaModel)
   }
-
-  // Check if model is loaded (auto-loaded on first call)
   const loaded = await isModelLoaded(endpoint, ollamaModel)
   if (!loaded) {
     // Warmup call — load model into memory
@@ -217,4 +201,4 @@ export async function embedOne(text: string): Promise<number[]> {
 /** VRAM usage in MB */
 export function vramUsage(): number {
   return server?.vramMB ?? 0
-}
+}

package/src/session/core/llm.ts

@@ -1,4 +1,4 @@
-import { Provider } from "@/provider/provider"
+import { Provider } from "@/provider/provider"
 import * as Log from "@saeeol/core/util/log"
 import { Context, Effect, Layer, Record } from "effect"
 import * as Stream from "effect/Stream"
@@ -119,11 +119,8 @@ const live: Layer.Layer<
       system.push(
         [
           ...(isOpenaiOauth ? [] : [SystemPrompt.soul()]),
-          // use agent prompt otherwise provider prompt
           ...(input.agent.prompt ? [input.agent.prompt] : SystemPrompt.provider(input.model)),
-          // any custom prompt passed into this call
           ...input.system,
-          // any custom prompt from last user message
           ...(input.user.system ? [input.user.system] : []),
         ]
           .filter((x) => x)
@@ -435,7 +432,6 @@ const live: Layer.Layer<
               specificationVersion: "v3" as const,
               async transformParams(args) {
                 if (args.type === "stream") {
-                  // @ts-expect-error
                   args.params.prompt = ProviderTransform.message(args.params.prompt, input.model, options)
                 }
                 return args.params
@@ -501,4 +497,4 @@ export function hasToolCalls(messages: ModelMessage[]): boolean {
   return false
 }
 
-export * as LLM from "./llm"
+export * as LLM from "./llm"

package/src/session/core/retry.ts

@@ -1,4 +1,4 @@
-import type { NamedError } from "@saeeol/core/util/error"
+import type { NamedError } from "@saeeol/core/util/error"
 import { Cause, Clock, Duration, Effect, Schedule } from "effect"
 import { MessageV2 } from "../message/message-v2"
 import { isSaeeolError } from "@/saeeol/errors" 
@@ -36,10 +36,8 @@ export function delay(attempt: number, error?: MessageV2.APIError) {
       if (retryAfter) {
         const parsedSeconds = Number.parseFloat(retryAfter)
         if (!Number.isNaN(parsedSeconds)) {
-          // convert seconds to milliseconds
           return cap(Math.ceil(parsedSeconds * 1000))
         }
-        // Try parsing as HTTP date format
         const parsed = Date.parse(retryAfter) - Date.now()
         if (!Number.isNaN(parsed) && parsed > 0) {
           return cap(Math.ceil(parsed))
@@ -68,8 +66,6 @@ export function retryable(error: Err) {
     if (error.data.responseBody?.includes("FreeUsageLimitError")) return undefined
     return error.data.message.includes("Overloaded") ? "Provider is overloaded" : error.data.message
   }
-
-  // Check for rate limit patterns in plain text error messages
   const msg = error.data?.message
   if (typeof msg === "string") {
     const lower = msg.toLowerCase()
@@ -146,4 +142,4 @@ export function policy(opts: {
   )
 }
 
-export * as SessionRetry from "./retry"
+export * as SessionRetry from "./retry"

package/src/session/core/session-events.ts

@@ -0,0 +1,143 @@
+/**
+ * session-events.ts — single source for SyncEvent/BusEvent definitions
+ *
+ * Shared by core/session.ts and core/session-types.ts.
+ * Does not import other core/session-* files, so no circular deps.
+ * TurnOpen/TurnClose require overlay deps, so each consumer adds them directly.
+ */
+
+import { BusEvent } from "@/bus/bus-event"
+import { SyncEvent } from "../../sync"
+import { SessionID } from "./schema"
+import { Snapshot } from "@/snapshot"
+import { MessageV2 } from "../message/message-v2"
+import { Schema } from "effect"
+import { Permission } from "@/permission"
+import { ProjectID } from "../../project/schema"
+import { WorkspaceID } from "../../control-plane/schema"
+import { NonNegativeInt, optionalOmitUndefined } from "@/util/schema"
+
+// ── Info schema (self-contained, mirrors session-types for event registration) ──
+
+const Summary = Schema.Struct({
+  additions: NonNegativeInt,
+  deletions: NonNegativeInt,
+  files: NonNegativeInt,
+  diffs: optionalOmitUndefined(Schema.Array(Snapshot.SummaryFileDiff)),
+})
+
+const Share = Schema.Struct({ url: Schema.String })
+
+const ArchivedTimestamp = Schema.Finite
+
+const Revert = Schema.Struct({
+  messageID: SessionID,
+  partID: optionalOmitUndefined(SessionID),
+  snapshot: optionalOmitUndefined(Schema.String),
+  diff: optionalOmitUndefined(Schema.String),
+})
+
+const Time = Schema.Struct({
+  created: NonNegativeInt,
+  updated: NonNegativeInt,
+  compacting: optionalOmitUndefined(NonNegativeInt),
+  archived: optionalOmitUndefined(ArchivedTimestamp),
+})
+
+const Info = Schema.Struct({
+  id: SessionID,
+  slug: Schema.String,
+  projectID: ProjectID,
+  workspaceID: optionalOmitUndefined(WorkspaceID),
+  directory: Schema.String,
+  path: optionalOmitUndefined(Schema.String),
+  parentID: optionalOmitUndefined(SessionID),
+  summary: optionalOmitUndefined(Summary),
+  share: optionalOmitUndefined(Share),
+  title: Schema.String,
+  version: Schema.String,
+  time: Time,
+  permission: optionalOmitUndefined(Permission.Ruleset),
+  revert: optionalOmitUndefined(Revert),
+})
+
+// ── Update event schemas ──
+
+const UpdatedShare = Schema.Struct({
+  url: Schema.optional(Schema.NullOr(Schema.String)),
+})
+
+const UpdatedTime = Schema.Struct({
+  created: Schema.optional(Schema.NullOr(NonNegativeInt)),
+  updated: Schema.optional(Schema.NullOr(NonNegativeInt)),
+  compacting: Schema.optional(Schema.NullOr(NonNegativeInt)),
+  archived: Schema.optional(Schema.NullOr(ArchivedTimestamp)),
+})
+
+const UpdatedInfo = Schema.Struct({
+  id: Schema.optional(Schema.NullOr(SessionID)),
+  slug: Schema.optional(Schema.NullOr(Schema.String)),
+  projectID: Schema.optional(Schema.NullOr(ProjectID)),
+  workspaceID: Schema.optional(Schema.NullOr(WorkspaceID)),
+  directory: Schema.optional(Schema.NullOr(Schema.String)),
+  path: Schema.optional(Schema.NullOr(Schema.String)),
+  parentID: Schema.optional(Schema.NullOr(SessionID)),
+  summary: Schema.optional(Schema.NullOr(Summary)),
+  share: Schema.optional(UpdatedShare),
+  title: Schema.optional(Schema.NullOr(Schema.String)),
+  version: Schema.optional(Schema.NullOr(Schema.String)),
+  time: Schema.optional(UpdatedTime),
+  permission: Schema.optional(Schema.NullOr(Permission.Ruleset)),
+  revert: Schema.optional(Schema.NullOr(Revert)),
+})
+
+export const CreatedEventSchema = Schema.Struct({
+  sessionID: SessionID,
+  info: Info,
+})
+
+export const UpdatedEventSchema = Schema.Struct({
+  sessionID: SessionID,
+  info: UpdatedInfo,
+})
+
+// ── Event definitions (singleton registration) ──
+
+export const SyncEvents = {
+  Created: SyncEvent.define({
+    type: "session.created",
+    version: 1,
+    aggregate: "sessionID",
+    schema: CreatedEventSchema,
+  }),
+  Updated: SyncEvent.define({
+    type: "session.updated",
+    version: 1,
+    aggregate: "sessionID",
+    schema: UpdatedEventSchema,
+    busSchema: CreatedEventSchema,
+  }),
+  Deleted: SyncEvent.define({
+    type: "session.deleted",
+    version: 1,
+    aggregate: "sessionID",
+    schema: CreatedEventSchema,
+  }),
+}
+
+export const BusEvents = {
+  Diff: BusEvent.define(
+    "session.diff",
+    Schema.Struct({
+      sessionID: SessionID,
+      diff: Schema.Array(Snapshot.FileDiff),
+    }),
+  ),
+  Error: BusEvent.define(
+    "session.error",
+    Schema.Struct({
+      sessionID: Schema.optional(SessionID),
+      error: MessageV2.Assistant.fields.error,
+    }),
+  ),
+}

package/src/session/core/session-types.ts

@@ -19,6 +19,7 @@ import { SaeeolSession } from "@/saeeol/session"
 import { Effect, Schema, Types } from "effect"
 import { zod } from "@/util/effect-zod"
 import { NonNegativeInt, optionalOmitUndefined, withStatics } from "@/util/schema"
+import { SyncEvents, BusEvents } from "./session-events"
 
 const log = Log.create({ service: "session" })
 
@@ -70,11 +71,11 @@ export function sessionPath(worktree: string, cwd: string) {
   return path.relative(path.resolve(worktree), cwd).replaceAll("\\", "/")
 }
 
-const Summary = Schema.Struct({ additions: NonNegativeInt, deletions: NonNegativeInt, files: NonNegativeInt, diffs: optionalOmitUndefined(Schema.Array(Snapshot.SummaryFileDiff)) })
+export const Summary = Schema.Struct({ additions: NonNegativeInt, deletions: NonNegativeInt, files: NonNegativeInt, diffs: optionalOmitUndefined(Schema.Array(Snapshot.SummaryFileDiff)) })
 const Share = Schema.Struct({ url: Schema.String })
 export const ArchivedTimestamp = Schema.Finite
 const Time = Schema.Struct({ created: NonNegativeInt, updated: NonNegativeInt, compacting: optionalOmitUndefined(NonNegativeInt), archived: optionalOmitUndefined(ArchivedTimestamp) })
-const Revert = Schema.Struct({ messageID: MessageID, partID: optionalOmitUndefined(PartID), snapshot: optionalOmitUndefined(Schema.String), diff: optionalOmitUndefined(Schema.String) })
+export const Revert = Schema.Struct({ messageID: MessageID, partID: optionalOmitUndefined(PartID), snapshot: optionalOmitUndefined(Schema.String), diff: optionalOmitUndefined(Schema.String) })
 
 export const Info = Schema.Struct({
   id: SessionID, slug: Schema.String, projectID: ProjectID, workspaceID: optionalOmitUndefined(WorkspaceID),
@@ -125,11 +126,8 @@ const UpdatedInfo = Schema.Struct({
 const UpdatedEventSchema = Schema.Struct({ sessionID: SessionID, info: UpdatedInfo })
 
 export const Event = {
-  Created: SyncEvent.define({ type: "session.created", version: 1, aggregate: "sessionID", schema: CreatedEventSchema }),
-  Updated: SyncEvent.define({ type: "session.updated", version: 1, aggregate: "sessionID", schema: UpdatedEventSchema, busSchema: CreatedEventSchema }),
-  Deleted: SyncEvent.define({ type: "session.deleted", version: 1, aggregate: "sessionID", schema: CreatedEventSchema }),
-  Diff: BusEvent.define("session.diff", Schema.Struct({ sessionID: SessionID, diff: Schema.Array(Snapshot.FileDiff) })),
-  Error: BusEvent.define("session.error", Schema.Struct({ sessionID: Schema.optional(SessionID), error: MessageV2.Assistant.fields.error })),
+  ...SyncEvents,
+  ...BusEvents,
   TurnOpen: SaeeolSession.Event.TurnOpen,
   TurnClose: SaeeolSession.Event.TurnClose,
 }

package/src/session/core/session.ts

@@ -1,4 +1,4 @@
-import { Slug } from "@saeeol/core/util/slug"
+import { Slug } from "@saeeol/core/util/slug"
 import path from "path"
 import { BusEvent } from "@/bus/bus-event"
 import { Bus } from "@/bus"
@@ -33,6 +33,7 @@ import { fn } from "@/util/fn"
 import { Effect, Layer, Option, Context, Schema, Types } from "effect"
 import { zod } from "@/util/effect-zod"
 import { NonNegativeInt, optionalOmitUndefined, withStatics } from "@/util/schema"
+import { SyncEvents, BusEvents } from "./session-events"
 
 const log = Log.create({ service: "session" })
 
@@ -279,7 +280,12 @@ const UpdatedEventSchema = Schema.Struct({
   info: UpdatedInfo,
 })
 
-export { Event } from "./session-types"
+export const Event = {
+  ...SyncEvents,
+  ...BusEvents,
+  TurnOpen: SaeeolSession.Event.TurnOpen,
+  TurnClose: SaeeolSession.Event.TurnClose,
+}
 
 export function plan(input: { slug: string; time: { created: number } }, instance: InstanceContext) {
   const base = instance.project.vcs
@@ -310,11 +316,8 @@ export const getUsage = (input: {
       input.usage.inputTokenDetails?.cacheWriteTokens ??
         input.metadata?.["anthropic"]?.["cacheCreationInputTokens"] ??
         // google-vertex-anthropic returns metadata under "vertex" key
-        // (AnthropicMessagesLanguageModel custom provider key from 'vertex.anthropic.messages')
         input.metadata?.["vertex"]?.["cacheCreationInputTokens"] ??
-        // @ts-expect-error
         input.metadata?.["bedrock"]?.["usage"]?.["cacheWriteInputTokens"] ??
-        // @ts-expect-error
         input.metadata?.["venice"]?.["usage"]?.["cacheCreationInputTokens"] ??
         0,
     ),
@@ -802,9 +805,6 @@ function* listByProject(
       )
     }
   } else if (input.scope !== "project" && !Flag.SAEEOL_EXPERIMENTAL_WORKSPACES) {
-    // if (input.directory) {
-    //   conditions.push(eq(SessionTable.directory, input.directory))
-    // }
   }
   if (input.roots) {
     conditions.push(isNull(SessionTable.parent_id))
@@ -907,4 +907,4 @@ export const updatePartDelta = fn(
   (input) => runPromise((svc) => svc.updatePartDelta(input)),
 )
 
-export * as Session from "./session"
+export * as Session from "./session"

package/src/session/message/message-errors.ts

@@ -1,4 +1,4 @@
-/** Error types + fromError + OutputFormat — extracted from message-v2.ts */
+/** Error types + fromError + OutputFormat — extracted from message-v2.ts */
 
 import { APICallError, LoadAPIKeyError } from "ai"
 import { NamedError } from "@saeeol/core/util/error"
@@ -37,8 +37,6 @@ const _Format = Schema.Union([OutputFormatText, OutputFormatJsonSchema]).annotat
 export { _Format }
 export const Format = Object.assign(_Format, { zod: zod(_Format) })
 export type OutputFormat = Schema.Schema.Type<typeof _Format>
-
-// Assistant error union (Zod)
 import z from "zod"
 const AssistantErrorZod = z.discriminatedUnion("name", [
   AuthError.Schema, NamedError.Unknown.Schema, OutputLengthError.Schema, AbortedError.Schema,
@@ -46,8 +44,6 @@ const AssistantErrorZod = z.discriminatedUnion("name", [
 ])
 export type AssistantError = z.infer<typeof AssistantErrorZod>
 export { AssistantErrorZod }
-
-// Assistant error union (Effect Schema)
 export const AssistantErrorSchema = Schema.Union([
   AuthError.EffectSchema,
   Schema.Struct({ name: Schema.Literal("UnknownError"), data: Schema.Struct({ message: Schema.String }) }).annotate({ identifier: "UnknownError" }),
@@ -80,4 +76,4 @@ export function fromError(e: unknown, ctx: { providerID: ProviderID; aborted?: b
       } catch {}
       return new NamedError.Unknown({ message: JSON.stringify(e) }, { cause: e }).toObject()
   }
-}
+}

package/src/session/message/message-parts.ts

@@ -1,4 +1,4 @@
-/** Part schemas + ToolState + Input schemas — extracted from message-v2.ts */
+/** Part schemas + ToolState + Input schemas — extracted from message-v2.ts */
 
 import { SessionID, MessageID, PartID } from "../core/schema"
 import { LSP } from "@/lsp/lsp"
@@ -81,9 +81,7 @@ export const AgentPartInput = Schema.Struct({ id: Schema.optional(PartID), type:
 export type AgentPartInput = Types.DeepMutable<Schema.Schema.Type<typeof AgentPartInput>>
 export const SubtaskPartInput = Schema.Struct({ id: Schema.optional(PartID), type: Schema.Literal("subtask"), prompt: Schema.String, description: Schema.String, agent: Schema.String, model: Schema.optional(Schema.Struct({ providerID: ProviderID, modelID: ModelID })), command: Schema.optional(Schema.String) }).annotate({ identifier: "SubtaskPartInput" }).pipe(withStatics((s) => ({ zod: zod(s) })))
 export type SubtaskPartInput = Types.DeepMutable<Schema.Schema.Type<typeof SubtaskPartInput>>
-
-// Part union
 const _Part = Schema.Union([TextPart, SubtaskPart, ReasoningPart, FilePart, ToolPart, StepStartPart, StepFinishPart, SnapshotPart, PatchPart, AgentPart, RetryPart, CompactionPart]).annotate({ discriminator: "type", identifier: "Part" })
 export const Part = Object.assign(_Part, { zod: zod(_Part) as unknown as z.ZodType<TextPart | SubtaskPart | ReasoningPart | FilePart | ToolPart | StepStartPart | StepFinishPart | SnapshotPart | PatchPart | AgentPart | RetryPart | CompactionPart> })
 export type Part = TextPart | SubtaskPart | ReasoningPart | FilePart | ToolPart | StepStartPart | StepFinishPart | SnapshotPart | PatchPart | AgentPart | RetryPart | CompactionPart
-export { _Part }
+export { _Part }

package/src/session/prompt/prompt.ts

@@ -1,5 +1,4 @@
-// @ts-ignore
-globalThis.AI_SDK_LOG_WARNINGS = false
+globalThis.AI_SDK_LOG_WARNINGS = false
 
 import { Effect, Layer, Scope, Latch, Context } from "effect"
 import { ChildProcessSpawner } from "effect/unstable/process"
@@ -208,4 +207,4 @@ export const prompt = (input: PromptInput) => runPromise((svc) => svc.prompt(inp
 export const loopExport = (input: LoopInput) => runPromise((svc) => svc.loop(input))
 export const cancel = (sessionID: SessionID) => runPromise((svc) => svc.cancel(sessionID))
 
-export * as SessionPrompt from "./prompt"
+export * as SessionPrompt from "./prompt"

package/src/tool/file/apply_patch.ts

@@ -37,8 +37,6 @@ export const ApplyPatchTool = Tool.define(
       if (!params.patchText) {
         return yield* Effect.fail(new Error("patchText is required"))
       }
-
-      // Parse the patch to get hunks
       let hunks: Patch.Hunk[]
       try {
         const parseResult = Patch.parsePatch(params.patchText)
@@ -56,8 +54,6 @@ export const ApplyPatchTool = Tool.define(
       }
 
       const instance = yield* InstanceState.context
-
-      // Validate file paths and check permissions
       const fileChanges: Array<{
         filePath: string
         oldContent: string
@@ -109,7 +105,6 @@ export const ApplyPatchTool = Tool.define(
           }
 
           case "update": {
-            // Check if file exists for update
             const stats = yield* afs.stat(filePath).pipe(Effect.catch(() => Effect.succeed(undefined)))
             if (!stats || stats.type === "Directory") {
               return yield* Effect.fail(
@@ -132,8 +127,6 @@ export const ApplyPatchTool = Tool.define(
             let newContent = oldContent
             let bom = source.bom
             let encoding = read.encoding 
-
-            // Apply the update chunks to get new content
             try {
               const fileUpdate = Patch.deriveNewContentsFromChunks(filePath, hunk.chunks)
               newContent = fileUpdate.content
@@ -205,8 +198,6 @@ export const ApplyPatchTool = Tool.define(
           }
         }
       }
-
-      // Build per-file metadata for UI rendering (used for both permission and result)
       const files = fileChanges.map((change) => ({
         filePath: change.filePath,
         relativePath: path.relative(instance.worktree, change.movePath ?? change.filePath).replaceAll("\\", "/"),
@@ -216,8 +207,6 @@ export const ApplyPatchTool = Tool.define(
         deletions: change.deletions,
         movePath: change.movePath,
       }))
-
-      // Check permissions if needed
       const relativePaths = fileChanges.map((c) => path.relative(instance.worktree, c.filePath).replaceAll("\\", "/"))
       yield* ctx.ask({
         permission: "edit",
@@ -229,15 +218,12 @@ export const ApplyPatchTool = Tool.define(
           files,
         },
       })
-
-      // Apply the changes
       const updates: Array<{ file: string; event: "add" | "change" | "unlink" }> = []
 
       for (const change of fileChanges) {
         const edited = change.type === "delete" ? undefined : (change.movePath ?? change.filePath)
         switch (change.type) {
           case "add":
-            // Create parent directories (recursive: true is safe on existing/root dirs)
             yield* EncodedIO.write(change.filePath, Bom.join(change.newContent, change.bom), change.encoding) 
             updates.push({ file: change.filePath, event: "add" })
             break
@@ -249,7 +235,6 @@ export const ApplyPatchTool = Tool.define(
 
           case "move":
             if (change.movePath) {
-              // Create parent directories (recursive: true is safe on existing/root dirs)
               yield* EncodedIO.write(change.movePath!, Bom.join(change.newContent, change.bom), change.encoding) 
               yield* afs.remove(change.filePath)
               updates.push({ file: change.filePath, event: "unlink" })
@@ -270,21 +255,15 @@ export const ApplyPatchTool = Tool.define(
           yield* bus.publish(File.Event.Edited, { file: edited })
         }
       }
-
-      // Publish file change events
       for (const update of updates) {
         yield* bus.publish(FileWatcher.Event.Updated, update)
       }
-
-      // Notify LSP of file changes and collect diagnostics
       for (const change of fileChanges) {
         if (change.type === "delete") continue
         const target = change.movePath ?? change.filePath
         yield* lsp.touchFile(target, "document")
       }
       const diagnostics = yield* lsp.diagnostics()
-
-      // Generate output summary
       const summaryLines = fileChanges.map((change) => {
         if (change.type === "add") {
           return `A ${path.relative(instance.worktree, change.filePath).replaceAll("\\", "/")}`

package/src/tool/file/edit-replacers.ts

@@ -1,4 +1,4 @@
-import { levenshtein, SINGLE_CANDIDATE_SIMILARITY_THRESHOLD, MULTIPLE_CANDIDATES_SIMILARITY_THRESHOLD } from "./edit-utils"
+import { levenshtein, SINGLE_CANDIDATE_SIMILARITY_THRESHOLD, MULTIPLE_CANDIDATES_SIMILARITY_THRESHOLD } from "./edit-utils"
 
 export type Replacer = (content: string, find: string) => Generator<string, void, unknown>
 
@@ -151,7 +151,6 @@ export const WhitespaceNormalizedReplacer: Replacer = function* (content, find)
             const match = line.match(regex)
             if (match) yield match[0]
           } catch {
-            // Invalid regex pattern, skip
           }
         }
       }
@@ -285,4 +284,4 @@ export const ContextAwareReplacer: Replacer = function* (content, find) {
       }
     }
   }
-}
+}

package/src/tool/integration/package.ts

@@ -1,4 +1,4 @@
-import { Effect, Schema } from "effect"
+import { Effect, Schema } from "effect"
 import { Npm } from "@saeeol/core/npm"
 import * as Bus from "@/bus"
 import * as Tool from "../core/tool"
@@ -98,8 +98,6 @@ export const PackageTool = Tool.define(
             ]
             return { title: "Available Providers", output: lines.join("\n"), metadata: {} }
           }
-
-          // action === "install"
           const packages = params.packages
           if (!packages || packages.length === 0) {
             return {
@@ -165,4 +163,4 @@ export const PackageTool = Tool.define(
         }),
     }
   }),
-)
+)

package/src/tool/search/warpgrep.ts

@@ -7,9 +7,6 @@ import { Bus } from "../../bus"
 import { TuiEvent } from "../../cli/cmd/tui/event"
 import DESCRIPTION from "./warpgrep.txt"
 
-// FREE_PERIOD_TODO: Remove SAEEOL_WARPGREP_PROXY_URL constant and the proxy
-// fallback below. After the free period ends, require MORPH_API_KEY and
-// return an error when it is missing.
 const SAEEOL_WARPGREP_PROXY_URL = "https://api.saeeol.ai/api/gateway"
 
 const Parameters = Schema.Struct({
@@ -36,8 +33,6 @@ export const CodebaseSearchTool = Tool.define(
 
           const apiKey = process.env["MORPH_API_KEY"]
 
-          // FREE_PERIOD_TODO: Remove proxy fallback — require apiKey, error if missing:
-          //   if (!apiKey) return { title: ..., output: "Set MORPH_API_KEY to use codebase search.", metadata: {} }
           const client = new WarpGrepClient({
             morphApiKey: apiKey ?? "saeeol-free",
             ...(apiKey ? {} : { morphApiUrl: SAEEOL_WARPGREP_PROXY_URL }),
@@ -52,9 +47,6 @@ export const CodebaseSearchTool = Tool.define(
           )
 
           if (!result.success || !result.contexts?.length) {
-            // FREE_PERIOD_TODO: When the proxy stops serving free requests, errors
-            // from the proxy (401/402/429) will surface here. The message below
-            // tells the user exactly what to do.
             const isAuthOrRateLimit =
               result.error && /401|402|429|rate.limit|free.period|unauthorized/i.test(result.error) 
             const apiKeyMsg =

package/src/tool/search/webfetch.ts

@@ -1,4 +1,4 @@
-import { Effect, Schema } from "effect"
+import { Effect, Schema } from "effect"
 import { HttpClient, HttpClientRequest } from "effect/unstable/http"
 import * as Tool from "../core/tool"
 import TurndownService from "turndown"
@@ -49,8 +49,6 @@ export const WebFetchTool = Tool.define(
           })
 
           const timeout = Math.min((params.timeout ?? DEFAULT_TIMEOUT / 1000) * 1000, MAX_TIMEOUT)
-
-          // Build Accept header based on requested format with q parameters for fallbacks
           let acceptHeader = "*/*"
           switch (params.format) {
             case "markdown":
@@ -92,8 +90,6 @@ export const WebFetchTool = Tool.define(
             ),
             Effect.timeoutOrElse({ duration: timeout, orElse: () => Effect.die(new Error("Request timed out")) }),
           )
-
-          // Check content length
           const contentLength = response.headers["content-length"]
           if (contentLength && parseInt(contentLength) > MAX_RESPONSE_SIZE) {
             throw new Error("Response too large (exceeds 5MB limit)")
@@ -125,8 +121,6 @@ export const WebFetchTool = Tool.define(
           }
 
           const content = new TextDecoder().decode(arrayBuffer)
-
-          // Handle content based on requested format and actual content type
           switch (params.format) {
             case "markdown":
               if (contentType.includes("text/html")) {
@@ -167,12 +161,10 @@ async function extractTextFromHTML(html: string) {
         skipContent = true
       },
       text() {
-        // Skip text content inside these elements
       },
     })
     .on("*", {
       element(element) {
-        // Reset skip flag when entering other elements
         if (!["script", "style", "noscript", "iframe", "object", "embed"].includes(element.tagName)) {
           skipContent = false
         }
@@ -199,4 +191,4 @@ function convertHTMLToMarkdown(html: string): string {
   })
   turndownService.remove(["script", "style", "meta", "link"])
   return turndownService.turndown(html)
-}
+}