saeeol 1.2.7 → 1.2.9

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,101 @@
+/**
+ * session-events.ts — SyncEvent/BusEvent 정의 단일 소스
+ *
+ * core/session.ts와 core/session-types.ts가 공유.
+ * SaeeolSession overlay를 import하지 않아 순환 의존 없음.
+ * TurnOpen/TurnClose는 overlay 의존이 필요하므로 각 소비자에서 직접 추가.
+ */
+
+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 { Info, Summary, Revert } from "./session-types"
+import { ArchivedTimestamp } from "./session-types"
+import { Schema } from "effect"
+import { Permission } from "@/permission"
+import { ProjectID } from "../../project/schema"
+import { WorkspaceID } from "../../control-plane/schema"
+import { NonNegativeInt } from "@/util/schema"
+
+// ── 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" })
 
@@ -280,41 +281,8 @@ const UpdatedEventSchema = Schema.Struct({
 })
 
 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),
-      // Reuses MessageV2.Assistant.fields.error (already Schema.optional) so
-      // the derived zod keeps the same discriminated-union shape on the bus.
-      error: MessageV2.Assistant.fields.error,
-    }),
-  ),
+  ...SyncEvents,
+  ...BusEvents,
   TurnOpen: SaeeolSession.Event.TurnOpen,
   TurnClose: SaeeolSession.Event.TurnClose,
 }
@@ -348,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,
     ),
@@ -840,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))
@@ -945,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 =