mulmoclaude 0.3.0 → 0.4.0

package/server/workspace/sources/rateLimiter.ts

@@ -41,7 +41,7 @@ export interface RateLimiterDeps {
 export function defaultRateLimiterDeps(): RateLimiterDeps {
   return {
     now: () => Date.now(),
-    sleep: (ms: number) => new Promise<void>((resolve) => setTimeout(resolve, ms)),
+    sleep: (millis: number) => new Promise<void>((resolve) => setTimeout(resolve, millis)),
   };
 }
 

package/server/workspace/sources/sourceState.ts

@@ -50,10 +50,9 @@ export function validateSourceState(raw: unknown, slug: string): SourceState {
   const record = raw as Record<string, unknown>;
   const lastFetchedAt = typeof record.lastFetchedAt === "string" ? record.lastFetchedAt : null;
   const nextAttemptAt = typeof record.nextAttemptAt === "string" ? record.nextAttemptAt : null;
-  const consecutiveFailures =
-    typeof record.consecutiveFailures === "number" && Number.isFinite(record.consecutiveFailures) && record.consecutiveFailures >= 0
-      ? Math.floor(record.consecutiveFailures)
-      : 0;
+  const emptyBackoffUntil = typeof record.emptyBackoffUntil === "string" ? record.emptyBackoffUntil : null;
+  const consecutiveFailures = toNonNegativeInt(record.consecutiveFailures);
+  const consecutiveEmptyFetches = toNonNegativeInt(record.consecutiveEmptyFetches);
   const cursor = validateCursor(record.cursor);
   return {
     slug,
@@ -61,9 +60,15 @@ export function validateSourceState(raw: unknown, slug: string): SourceState {
     cursor,
     consecutiveFailures,
     nextAttemptAt,
+    consecutiveEmptyFetches,
+    emptyBackoffUntil,
   };
 }
 
+function toNonNegativeInt(value: unknown): number {
+  return typeof value === "number" && Number.isFinite(value) && value >= 0 ? Math.floor(value) : 0;
+}
+
 function validateCursor(raw: unknown): Record<string, string> {
   if (!isRecord(raw)) {
     return {};

package/server/workspace/sources/types.ts

@@ -131,6 +131,13 @@ export interface SourceState {
   // Timestamp after which the next attempt is allowed, so backoff
   // survives server restarts.
   nextAttemptAt: string | null;
+  // Consecutive empty-success count (fetcher returned 0 items).
+  // Reset to 0 when items are found. Drives adaptive empty backoff.
+  consecutiveEmptyFetches: number;
+  // Timestamp after which the next attempt is allowed following
+  // repeated empty fetches. Separate from nextAttemptAt (error
+  // backoff) so the two policies don't interfere.
+  emptyBackoffUntil: string | null;
 }
 
 export function defaultSourceState(slug: string): SourceState {
@@ -140,5 +147,7 @@ export function defaultSourceState(slug: string): SourceState {
     cursor: {},
     consecutiveFailures: 0,
     nextAttemptAt: null,
+    consecutiveEmptyFetches: 0,
+    emptyBackoffUntil: null,
   };
 }

package/server/workspace/sources/urls.ts

@@ -80,7 +80,7 @@ export function normalizeUrl(raw: string): string | null {
   // Sort remaining params for deterministic ordering. Preserve
   // multi-value params by iterating all entries.
   const entries = Array.from(url.searchParams.entries());
-  entries.sort(([a], [b]) => (a < b ? -1 : a > b ? 1 : 0));
+  entries.sort(([leftKey], [rightKey]) => (leftKey < rightKey ? -1 : leftKey > rightKey ? 1 : 0));
   // Clear and reinsert.
   for (const name of Array.from(url.searchParams.keys())) {
     url.searchParams.delete(name);

package/server/workspace/tool-trace/classify.ts

@@ -96,10 +96,10 @@ function imagePointerFromContent(content: string): string | null {
 // regardless of how the tool happened to quote it. Leave "../"
 // prefixes alone — a relative escape is a bug and we want it visible
 // rather than silently fixed up.
-function normalizeWorkspacePath(p: string): string {
-  if (p.startsWith("./")) return p.slice(2);
-  if (p.startsWith("/")) return p.slice(1);
-  return p;
+function normalizeWorkspacePath(candidatePath: string): string {
+  if (candidatePath.startsWith("./")) return candidatePath.slice(2);
+  if (candidatePath.startsWith("/")) return candidatePath.slice(1);
+  return candidatePath;
 }
 
 function inlineWithTruncation(content: string): Classification {

package/server/workspace/workspace.ts

@@ -1,5 +1,5 @@
 import { execSync } from "child_process";
-import fs from "fs";
+import { copyFileSync, existsSync, mkdirSync, readdirSync } from "fs";
 import path from "path";
 import { fileURLToPath } from "url";
 import { log } from "../system/logger/index.js";
@@ -16,12 +16,12 @@ const TEMPLATES_DIR = path.join(__dirname, "helps");
 export { workspacePath };
 
 // Must exist before downstream modules call realpathSync(workspacePath) at their own module-load time.
-fs.mkdirSync(workspacePath, { recursive: true });
+mkdirSync(workspacePath, { recursive: true });
 
 export function initWorkspace(): string {
   // Create directory structure if needed
   for (const key of EAGER_WORKSPACE_DIRS) {
-    fs.mkdirSync(WORKSPACE_PATHS[key], { recursive: true });
+    mkdirSync(WORKSPACE_PATHS[key], { recursive: true });
   }
 
   // Create memory.md if it doesn't exist
@@ -30,9 +30,9 @@ export function initWorkspace(): string {
   }
 
   // Always sync all files from server/helps/ into workspace/helps/
-  fs.mkdirSync(WORKSPACE_PATHS.helps, { recursive: true });
-  for (const file of fs.readdirSync(TEMPLATES_DIR)) {
-    fs.copyFileSync(path.join(TEMPLATES_DIR, file), path.join(WORKSPACE_PATHS.helps, file));
+  mkdirSync(WORKSPACE_PATHS.helps, { recursive: true });
+  for (const file of readdirSync(TEMPLATES_DIR)) {
+    copyFileSync(path.join(TEMPLATES_DIR, file), path.join(WORKSPACE_PATHS.helps, file));
   }
 
   // Create .gitignore if missing. The workspace is a git repo for
@@ -56,7 +56,7 @@ export function initWorkspace(): string {
 
   // Git init if not already a repo
   const gitDir = path.join(workspacePath, ".git");
-  if (!fs.existsSync(gitDir)) {
+  if (!existsSync(gitDir)) {
     execSync("git init", { cwd: workspacePath });
     log.info("workspace", "initialized git repository", { workspacePath });
   }