@polderlabs/bizar 6.2.4 → 6.3.0

package/plugins/bizar/src/hooks/memory-write-on-end.ts

@@ -1,188 +0,0 @@
-/**
- * plugins/bizar/src/hooks/memory-write-on-end.ts
- *
- * Session-end memory write hook.
- *
- * Fires on `session.idle` (natural completion) or `session.error` events.
- * Writes an automated session summary to the memory vault at:
- *   sessions/<date>-<session-id>.md
- *
- * Key design decisions:
- *   - Configurable: `settings.memory.writeOnSessionEnd` (default true)
- *   - Non-blocking: write is fire-and-forget (success logged, failure warned)
- *   - Idempotent: if the same session fires multiple terminal events,
- *     only the first write succeeds (check if note already exists via API)
- *   - Uses the dashboard API (POST /api/memory/notes) for consistency
- */
-
-import { existsSync, readFileSync } from "node:fs";
-import { join } from "node:path";
-import { homedir } from "node:os";
-
-import type { Logger } from "../logger.js";
-
-const SESSION_NOTES_PREFIX = "sessions/";
-
-export interface MemoryWriteOnEndDeps {
-  worktree: string;
-  logger: Logger;
-  enabled: boolean;
-}
-
-/**
- * Resolve the dashboard server port. Returns null if not available.
- */
-export async function resolveDashboardPort(): Promise<number | null> {
-  const portFile = join(homedir(), ".config", "bizar", "dashboard.port");
-  if (existsSync(portFile)) {
-    try {
-      const raw = readFileSync(portFile, "utf8").trim();
-      const n = parseInt(raw, 10);
-      if (Number.isFinite(n) && n > 0 && n <= 65535) return n;
-    } catch {
-      // fall through
-    }
-  }
-  const envPort = process.env.BIZAR_DASHBOARD_PORT;
-  if (envPort) {
-    const n = parseInt(envPort, 10);
-    if (Number.isFinite(n) && n > 0 && n <= 65535) return n;
-  }
-  return null;
-}
-
-interface SessionEndInfo {
-  sessionID: string;
-  agent: string;
-  startedAt: number;
-  endedAt: number;
-  status: "idle" | "error" | "killed";
-  error?: string;
-}
-
-/**
- * Build a session summary note body.
- */
-function buildSessionSummary(
-  info: SessionEndInfo,
-  conversationPreview: string,
-): string {
-  const durationMin = Math.round((info.endedAt - info.startedAt) / 60_000);
-  const started = new Date(info.startedAt).toISOString();
-  const ended = new Date(info.endedAt).toISOString();
-
-  const lines: string[] = [
-    `# Session Summary: ${info.sessionID.slice(0, 8)}…`,
-    "",
-    `**Agent:** ${info.agent}`,
-    `**Started:** ${started}`,
-    `**Ended:** ${ended}`,
-    `**Duration:** ~${durationMin} min`,
-    `**Status:** ${info.status}${info.error ? ` — ${info.error}` : ""}`,
-    "",
-    "## Conversation Preview",
-    "",
-    conversationPreview.slice(0, 2000),
-    "",
-    "## Key Decisions",
-    "",
-    "- *(add your key decisions here)*",
-    "",
-    "## Tools Used",
-    "",
-    "- *(tools used during this session)*",
-    "",
-    "## Errors / Issues",
-    info.error ? `\n${info.error}\n` : "- *(no errors)*",
-  ];
-
-  return lines.join("\n");
-}
-
-/**
- * Write a session summary note to the memory vault.
- */
-async function writeSessionSummary(
-  port: number,
-  info: SessionEndInfo,
-  conversationPreview: string,
-  worktree: string,
-  logger: Logger,
-): Promise<void> {
-  const date = new Date(info.startedAt).toISOString().slice(0, 10); // YYYY-MM-DD
-  const slug = `${date}-${info.sessionID.slice(0, 8)}.md`;
-  const relPath = `${SESSION_NOTES_PREFIX}${slug}`;
-
-  const body = buildSessionSummary(info, conversationPreview);
-
-  const payload = {
-    path: relPath,
-    frontmatter: {
-      type: "session-summary",
-      session_id: info.sessionID,
-      started_at: new Date(info.startedAt).toISOString(),
-      ended_at: new Date(info.endedAt).toISOString(),
-      duration_minutes: Math.round((info.endedAt - info.startedAt) / 60_000),
-      agent: info.agent,
-      project: worktree.split("/").pop() ?? "unknown",
-      tags: ["auto-generated"],
-      status: info.status,
-    },
-    body,
-  };
-
-  try {
-    const res = await fetch(`http://127.0.0.1:${port}/api/memory/notes`, {
-      method: "POST",
-      headers: { "Content-Type": "application/json" },
-      body: JSON.stringify(payload),
-    });
-
-    if (res.ok || res.status === 409) {
-      // 409 = already exists (idempotent)
-      logger.debug(`bizar: session summary written for ${info.sessionID.slice(0, 8)}`);
-    } else {
-      const text = await res.text().catch(() => "");
-      logger.warn(`bizar: session summary write failed for ${info.sessionID.slice(0, 8)}: HTTP ${res.status} — ${text.slice(0, 200)}`);
-    }
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    logger.warn(`bizar: session summary write failed for ${info.sessionID.slice(0, 8)}: ${msg}`);
-  }
-}
-
-/**
- * Create the session-end memory write hook.
- *
- * Usage in index.ts `event` hook:
- *   const { memoryWriteOnEnd } = createMemoryWriteOnEnd(deps);
- *   if (type === "session.idle" || type === "session.error") {
- *     memoryWriteOnEnd(sessionID, eventPayload).catch(() => {});
- *   }
- */
-export function createMemoryWriteOnEnd(deps: MemoryWriteOnEndDeps) {
-  const { logger } = deps;
-
-  /** Sessions that have already been written (idempotency). */
-  const _writtenSessions = new Set<string>();
-
-  async function memoryWriteOnEnd(
-    sessionID: string,
-    info: SessionEndInfo,
-    conversationPreview: string,
-  ): Promise<void> {
-    if (!deps.enabled) return;
-    if (_writtenSessions.has(sessionID)) return;
-    _writtenSessions.add(sessionID);
-
-    const port = await resolveDashboardPort();
-    if (!port) {
-      logger.debug(`bizar: session-end memory write skipped — dashboard not running`);
-      return;
-    }
-
-    await writeSessionSummary(port, info, conversationPreview, deps.worktree, logger);
-  }
-
-  return { memoryWriteOnEnd };
-}

package/plugins/bizar/src/hooks/skill-curator.ts

@@ -1,180 +0,0 @@
-/**
- * skill-curator.ts — Skill Curator hook (v6.0.0)
- *
- * Pattern: Hermes Agent `agent/curator.py` (the "closed learning loop"
- * differentiator). Of 106 cataloged projects, exactly one has this.
- *
- * The Curator watches for signals (skill failures, repeated errors,
- * new lessons entries) and:
- *   1. Increments per-skill counters
- *   2. Flags stale skills (no usage in N days)
- *   3. Proposes skill revisions when 5+ failures accumulate
- *   4. Surfaces new patterns from `AGENTS_SELF_IMPROVEMENT.md`
- *
- * v6.0.0 — initial implementation. Tracks in-memory counters; future
- * versions will persist to `.bizar/skills/usage.jsonl` and run a
- * scheduled review pass.
- *
- * See: research/agent-harness-survey/final-reports/06-bizar-improvement-plan.md
- *   § Improvement 1 — SKILL.md + Curator
- */
-
-import { existsSync, readFileSync, writeFileSync, mkdirSync } from "node:fs";
-import { join } from "node:path";
-import { homedir } from "node:os";
-
-import type { Logger } from "../logger.js";
-
-export interface SkillCuratorDeps {
-  worktree: string;
-  logger: Logger;
-  /** Number of failures before proposing a revision. Default: 5. */
-  revisionThreshold?: number;
-  /** Number of days without use before flagging a skill stale. Default: 30. */
-  staleDays?: number;
-}
-
-export interface SkillUsageCounter {
-  name: string;
-  useCount: number;
-  failureCount: number;
-  lastUsed: string | null;
-  status: "ok" | "stale" | "needs-revision" | "new";
-}
-
-/**
- * In-memory + on-disk usage tracker. Keyed by skill name. Persisted to
- * ~/.bizar/skills/usage.jsonl on every update.
- */
-function getUsageFile(): string {
-  // Allow override via env var (mainly for tests)
-  const override = process.env.BIZAR_SKILL_USAGE_FILE;
-  if (override) return override;
-  return join(homedir(), ".bizar", "skills", "usage.jsonl");
-}
-
-function loadUsage(): Record<string, SkillUsageCounter> {
-  if (!existsSync(getUsageFile())) return {};
-  try {
-    const text = readFileSync(getUsageFile(), "utf8");
-    const map: Record<string, SkillUsageCounter> = {};
-    for (const line of text.split("\n")) {
-      if (!line.trim()) continue;
-      try {
-        const entry = JSON.parse(line) as SkillUsageCounter;
-        map[entry.name] = entry;
-      } catch {
-        // skip malformed line
-      }
-    }
-    return map;
-  } catch {
-    return {};
-  }
-}
-
-function saveUsage(map: Record<string, SkillUsageCounter>): void {
-  try {
-    mkdirSync(join(process.env.BIZAR_SKILL_USAGE_DIR ?? join(homedir(), ".bizar", "skills")), { recursive: true });
-    const text = Object.values(map)
-      .map((e) => JSON.stringify(e))
-      .join("\n");
-    writeFileSync(getUsageFile(), text + "\n", "utf8");
-  } catch (err) {
-    // best-effort; not critical
-  }
-}
-
-export function getSkillUsage(name: string): SkillUsageCounter | null {
-  const map = loadUsage();
-  return map[name] ?? null;
-}
-
-export function recordSkillUse(name: string, outcome: "success" | "failure"): void {
-  const map = loadUsage();
-  const entry = map[name] ?? {
-    name,
-    useCount: 0,
-    failureCount: 0,
-    lastUsed: null,
-    status: "new" as const,
-  };
-  if (outcome === "success") entry.useCount += 1;
-  else entry.failureCount += 1;
-  entry.lastUsed = new Date().toISOString();
-  entry.status = entry.failureCount >= 5 ? "needs-revision" : "ok";
-  map[name] = entry;
-  saveUsage(map);
-}
-
-export interface CuratorReport {
-  totalSkills: number;
-  ok: number;
-  needsRevision: number;
-  stale: number;
-  new: number;
-  proposals: Array<{ name: string; reason: string; failureCount: number }>;
-}
-
-export function generateCuratorReport(deps: SkillCuratorDeps): CuratorReport {
-  const map = loadUsage();
-  const threshold = deps.revisionThreshold ?? 5;
-  const staleDays = deps.staleDays ?? 30;
-  const now = Date.now();
-  const staleCutoff = now - staleDays * 24 * 60 * 60 * 1000;
-
-  const entries = Object.values(map);
-  const proposals: CuratorReport["proposals"] = [];
-  for (const e of entries) {
-    if (e.failureCount >= threshold) {
-      proposals.push({
-        name: e.name,
-        reason: `${e.failureCount} failures recorded; consider revision.`,
-        failureCount: e.failureCount,
-      });
-    }
-  }
-
-  return {
-    totalSkills: entries.length,
-    ok: entries.filter((e) => e.status === "ok").length,
-    needsRevision: entries.filter((e) => e.status === "needs-revision").length,
-    stale: entries.filter(
-      (e) =>
-        e.lastUsed !== null &&
-        new Date(e.lastUsed).getTime() < staleCutoff &&
-        e.useCount > 0,
-    ).length,
-    new: entries.filter((e) => e.status === "new").length,
-    proposals,
-  };
-}
-
-export interface SkillCuratorHook {
-  /** Manually trigger a curator report. */
-  report: () => CuratorReport;
-  /** Record a skill use (success or failure). */
-  record: (name: string, outcome: "success" | "failure") => void;
-}
-
-export function createSkillCurator(deps: SkillCuratorDeps): SkillCuratorHook {
-  const logger = deps.logger;
-  return {
-    report: () => {
-      const r = generateCuratorReport(deps);
-      logger.info(
-        `bizar: skill curator — ${r.totalSkills} tracked, ${r.needsRevision} need revision, ${r.stale} stale, ${r.proposals.length} proposals`,
-      );
-      return r;
-    },
-    record: (name, outcome) => {
-      recordSkillUse(name, outcome);
-      const usage = getSkillUsage(name);
-      if (usage && usage.failureCount >= (deps.revisionThreshold ?? 5)) {
-        logger.warn(
-          `bizar: skill '${name}' has ${usage.failureCount} failures — propose revision.`,
-        );
-      }
-    },
-  };
-}