fapony 0.2.0 → 0.2.1

package/src/install/opencode.ts

@@ -113,6 +113,7 @@ export function cmdInstallOpencode(
     reportSkills(linkSkills(skillsDir, dryRun), skillsDir, dryRun);
     installReadHintPlugin(dryRun, getHome);
     installCommitHintPlugin(dryRun, getHome);
+    installEditHintPlugin(dryRun, getHome);
     return;
   }
 
@@ -136,6 +137,7 @@ export function cmdInstallOpencode(
     console.log(computeDiff(before, after));
     installReadHintPlugin(dryRun, getHome);
     installCommitHintPlugin(dryRun, getHome);
+    installEditHintPlugin(dryRun, getHome);
     return;
   }
 
@@ -157,6 +159,7 @@ export function cmdInstallOpencode(
   reportSkills(linkSkills(skillsDir, dryRun), skillsDir, dryRun);
   installReadHintPlugin(dryRun, getHome);
   installCommitHintPlugin(dryRun, getHome);
+  installEditHintPlugin(dryRun, getHome);
 }
 
 /**
@@ -228,7 +231,7 @@ export const FaponyReadHint = async ({ directory }) => {
               const { relative } = require("node:path");
               const { join: pjoin } = require("node:path");
               const wt = ctx.worktree;
-              const abs = realpathSync(filePath.startsWith("/") ? filePath : pjoin(wt, filePath));
+              const abs = realpathSync(filePath.startsWith("/") ? filePath : pjoin(directory, filePath));
               const r = relative(wt, abs).split("\\\\").join("/");
               return r.startsWith("..") ? null : r;
             } catch { return null; }
@@ -308,6 +311,80 @@ export const FaponyCommitHint = async ({ directory }) => {
 `;
 }
 
+/**
+ * OpenCode plugin — the edit hint's in-process shape. `tool.execute.after`
+ * on the edit and write tools: when the touched file has importers, appends
+ * the importer count plus the review-seed command that lists them. Annotate
+ * only (no block, no dedupe beyond what editHintFor does itself).
+ *
+ * This is the OpenCode parity of Claude's Edit PreToolUse hook
+ * (PLAN-edit-importer-hint): same shared logic (editHintFor in src/hook.ts),
+ * same once-per-(session, file) dedupe, same `"edit"` fire surface. One
+ * known difference: OpenCode's only annotate channel is `after`, so the hint
+ * lands one step after the edit instead of before it — the `before` hook can
+ * only mutate args or throw, and a hint must never block a tool call.
+ *
+ * `write` is guarded alongside `edit` for free: a write to a new file
+ * resolves to null inside editHintFor (nothing imports it yet), so only a
+ * write that overwrites an imported file fires. `apply_patch` is out of
+ * scope — its paths live inside patchText, not filePath (separate plan).
+ */
+export function editHintPluginSource(root: string): string {
+  const hookModule = JSON.stringify(join(root, "src", "hook.ts"));
+  return `// fapony edit hint — annotates edits to source files that have importers
+// with the importer count plus the review-seed command that lists them.
+// Annotate only: never blocks, dedupe per (session, file) inside editHintFor.
+// Generated by \`fapony install\` — edit src/hook.ts in the fapony checkout.
+import { editHintFor, recordHintFire } from ${hookModule};
+
+export const FaponyEditHint = async ({ directory }) => {
+  return {
+    "tool.execute.after": async (input, output) => {
+      try {
+        if (input.tool !== "edit" && input.tool !== "write") return;
+        // The only channel that reaches the agent is output.output; if it is
+        // not a string the hint cannot surface, so bail before editHintFor
+        // spends the per-session dedupe row or logs a fire that never showed.
+        if (typeof output.output !== "string") return;
+        const filePath = input.args?.filePath;
+        const hint = editHintFor({
+          filePath,
+          cwd: directory,
+          session: input.sessionID,
+        });
+
+        // --- hint-fire log (PLAN-edit-importer-hint, opencode parity) ---
+        // After output — best-effort, never block the hint.
+        if (hint) {
+          output.output = output.output + "\\n" + hint;
+          try {
+            const { spawnSync } = require("node:child_process");
+            const git = spawnSync("git", ["rev-parse", "--show-toplevel"], { cwd: directory, encoding: "utf-8" });
+            if (git.status === 0) {
+              const worktree = require("node:fs").realpathSync(git.stdout.trim());
+              const rel = typeof filePath === "string" ? (() => {
+                try {
+                  const { relative, join: pjoin } = require("node:path");
+                  const abs = require("node:fs").realpathSync(filePath.startsWith("/") ? filePath : pjoin(directory, filePath));
+                  const r = relative(worktree, abs).split("\\\\").join("/");
+                  return r.startsWith("..") ? null : r;
+                } catch { return null; }
+              })() : null;
+              recordHintFire({ ts: new Date().toISOString(), worktree, surface: "edit", file: rel, count: 1 });
+            }
+          } catch {
+            // best-effort
+          }
+        }
+      } catch {
+        // a hint must never break an edit
+      }
+    },
+  };
+};
+`;
+}
+
 function installReadHintPlugin(dryRun: boolean, getHome: () => string): void {
   const pluginsDir = join(getHome(), ".config", "opencode", "plugins");
   const pluginPath = join(pluginsDir, "fapony-read-hint.ts");
@@ -373,3 +450,36 @@ function installCommitHintPlugin(dryRun: boolean, getHome: () => string): void {
     `  commit hint: ${dryRun ? "would write" : "wrote"} ${pluginPath}`,
   );
 }
+
+function installEditHintPlugin(dryRun: boolean, getHome: () => string): void {
+  const pluginsDir = join(getHome(), ".config", "opencode", "plugins");
+  const pluginPath = join(pluginsDir, "fapony-edit-hint.ts");
+  if (existsSync(pluginPath)) {
+    let current = "";
+    try {
+      current = readFileSync(pluginPath, "utf-8");
+    } catch {
+      current = "";
+    }
+    if (current.includes("editHintFor")) {
+      console.error(`  edit hint: already installed — no change`);
+      return;
+    }
+    console.error(
+      `  edit hint: ${pluginPath} exists but isn't fapony's — not overwriting.`,
+    );
+    return;
+  }
+  if (!dryRun) {
+    try {
+      mkdirSync(pluginsDir, { recursive: true });
+      writeFileSync(pluginPath, editHintPluginSource(INSTALL_ROOT), "utf-8");
+    } catch (e) {
+      console.error(`  edit hint: failed to write — ${(e as Error).message}`);
+      return;
+    }
+  }
+  console.error(
+    `  edit hint: ${dryRun ? "would write" : "wrote"} ${pluginPath}`,
+  );
+}

package/src/install.ts

@@ -28,12 +28,13 @@ export {
   claudeGetPointsToFapony,
   cmdInstallClaude,
 } from "./install/claude.js";
-export { cmdInstallCodex } from "./install/codex.js";
+export { cmdInstallCodex, findCodexHooksJson } from "./install/codex.js";
 export { cmdInstallCursor } from "./install/cursor.js";
 export { detectClients } from "./install/detect.js";
 export {
   cmdInstallOpencode,
   commitHintPluginSource,
+  editHintPluginSource,
   readHintPluginSource,
 } from "./install/opencode.js";
 export {

package/src/lint-baseline.ts

@@ -14,6 +14,7 @@
 import { execSync } from "node:child_process";
 import { mkdirSync, readFileSync, rmSync, writeFileSync } from "node:fs";
 import { join, relative } from "node:path";
+import { DEFAULT_EVIDENCE_FILE } from "./db/index.js";
 import { faponyDir } from "./db/load.js";
 import { assertSafe } from "./safety.js";
 
@@ -57,7 +58,7 @@ function baselinePath(worktree: string): string {
 }
 
 function readEvidenceLintCmd(worktree: string): string | null {
-  const evidence = join(worktree, ".fapony", "evidence.json");
+  const evidence = join(worktree, DEFAULT_EVIDENCE_FILE);
   try {
     const parsed = JSON.parse(readFileSync(evidence, "utf-8")) as {
       commands?: { name?: string; cmd?: string }[];

package/src/mcp/evidence.ts

@@ -17,7 +17,13 @@
 import { execSync } from "node:child_process";
 import { existsSync, readFileSync } from "node:fs";
 import { join, relative } from "node:path";
-import { type Config, evidenceFile, safetyDeny } from "../db/index.js";
+import {
+  type Config,
+  EVIDENCE_FILENAME,
+  evidenceFile,
+  FAPONY_DIR,
+  safetyDeny,
+} from "../db/index.js";
 import { assertSafe } from "../safety.js";
 import type { EvidenceItem, EvidenceStatus } from "./primitives.js";
 
@@ -80,7 +86,13 @@ export function resolveEvidencePath(
       APP_GROUPS.includes(group) &&
       heads.every(([g, a]) => g === group && a === app)
     ) {
-      const candidate = join(worktree, group, app, ".fapony", "evidence.json");
+      const candidate = join(
+        worktree,
+        group,
+        app,
+        FAPONY_DIR,
+        EVIDENCE_FILENAME,
+      );
       if (existsSync(candidate)) return candidate;
     }
   }

package/src/mcp/transport.ts

@@ -110,7 +110,7 @@ function writeStatuslineCache(toolResult: ToolResult): void {
 
 const MCP_PROTOCOL_VERSION = "2025-03-26";
 const SERVER_NAME = "fapony-handcheck";
-const SERVER_VERSION = "0.2.0";
+const SERVER_VERSION = "0.2.1";
 
 // --- JSON-RPC dispatch ---
 

package/src/mem/store.ts

@@ -15,6 +15,7 @@ import {
 } from "node:fs";
 import { hostname, networkInterfaces } from "node:os";
 import { basename, dirname, join, relative } from "node:path";
+import { CONFIG_FILENAME, DEFAULT_MEM_DIR, FAPONY_DIR } from "../db/index.js";
 import { whereMemDir } from "../memory.js";
 
 // --- types ---
@@ -174,7 +175,7 @@ export function initStore(worktree: string, overrideMemDir?: string): void {
       `two or more .fapony/.memory dirs under ${root} — refusing to guess:\n` +
       `${list}\ncd into one, set paths.memDir, or pass --mem-dir`;
   }
-  dir = resolved.dir ?? join(root, ".fapony", ".memory");
+  dir = resolved.dir ?? join(root, DEFAULT_MEM_DIR);
 
   // planBase: the .fapony dir — plan/done/conventions live here.
   // Derive from dir by going up from .fapony/.memory → .fapony
@@ -184,10 +185,11 @@ export function initStore(worktree: string, overrideMemDir?: string): void {
   app = dirname(planBase);
 
   // Read fapony.config.json
-  const configDir = planBase === `${root}/.fapony` ? root : dirname(planBase);
+  const configDir =
+    planBase === `${root}/${FAPONY_DIR}` ? root : dirname(planBase);
   const configPaths = ((): Record<string, string> => {
     try {
-      const raw = readFileSync(`${configDir}/fapony.config.json`, "utf8");
+      const raw = readFileSync(`${configDir}/${CONFIG_FILENAME}`, "utf8");
       return (JSON.parse(raw)?.paths ?? {}) as Record<string, string>;
     } catch {
       return {};
@@ -198,7 +200,7 @@ export function initStore(worktree: string, overrideMemDir?: string): void {
       ? join(configDir, configPaths[key])
       : null;
 
-  planDir = fromConfig("planDir") ?? `${planBase}/plan`;
+  planDir = `${planBase}/plan`;
 
   // done/ sits beside plan/ (same depth, relative links survive)
   doneDir =

package/src/memory.ts

@@ -5,6 +5,7 @@
 import { execSync } from "node:child_process";
 import { existsSync, readdirSync, readFileSync, realpathSync } from "node:fs";
 import { dirname, isAbsolute, join, resolve } from "node:path";
+import { CONFIG_FILENAME, DEFAULT_MEM_DIR, FAPONY_DIR } from "./db/defaults.js";
 import { type Config, loadConfig, safetyDeny } from "./db/index.js";
 import { assertSafe } from "./safety.js";
 import { templateArgs } from "./util.js";
@@ -202,7 +203,7 @@ function walkUpForMemDir(fromDir: string, acceptEmpty: boolean): string | null {
   const boundary = repoRootOf(start) ?? "/";
   let dir = start;
   while (true) {
-    const candidate = join(dir, ".fapony", ".memory");
+    const candidate = join(dir, DEFAULT_MEM_DIR);
     if (existsSync(candidate) && (acceptEmpty || hasMemLogs(candidate))) {
       return candidate;
     }
@@ -254,8 +255,8 @@ function findMemDirsUnder(root: string): string[] {
     }
     for (const e of entries) {
       if (!e.isDirectory()) continue;
-      if (e.name === ".fapony") {
-        const candidate = join(dir, ".fapony", ".memory");
+      if (e.name === FAPONY_DIR) {
+        const candidate = join(dir, DEFAULT_MEM_DIR);
         if (hasMemLogs(candidate)) found.push(candidate);
         continue;
       }
@@ -298,7 +299,7 @@ function resolveMemDirFrom(
   // path source here — spec §1 pins this to fapony.config.json)
   const configDir = root ?? cwd;
   try {
-    const configPath = join(configDir, "fapony.config.json");
+    const configPath = join(configDir, CONFIG_FILENAME);
     if (existsSync(configPath)) {
       const config = loadConfig(configPath);
       if (config?.paths?.memDir) {
@@ -328,7 +329,7 @@ function resolveMemDirFrom(
 
   // Step 4: <repo root>/.fapony/.memory/ — where a new log is created
   if (root) {
-    const rootDir = join(root, ".fapony", ".memory");
+    const rootDir = join(root, DEFAULT_MEM_DIR);
     if (existsSync(rootDir)) return { dir: rootDir, step: "repo-root" };
   }
 

package/src/plan-seed.ts

@@ -38,6 +38,7 @@ import { basename, join, relative, resolve, sep } from "node:path";
 import { collectSourceFiles, isSkippedDir, SCAN_EXTS } from "./analyze.js";
 import { computeModelFit } from "./context/projectHealth.js";
 import { doneDir, planDir, specDir } from "./db/getters.js";
+import { CONFIG_FILENAME } from "./db/index.js";
 import { loadConfig } from "./db/load.js";
 import type { Config } from "./db/types.js";
 import { extractExports } from "./map.js";
@@ -97,7 +98,7 @@ function renderPriorArt(cwd: string, config: Config, roots: string[]): string {
   if (keys.length === 0) return placeholder;
 
   const hits: { shipped: string; line: string }[] = [];
-  for (const dir of [doneDir(config), specDir(config)]) {
+  for (const dir of [doneDir(config), specDir()]) {
     const abs = join(cwd, dir);
     let names: string[];
     try {
@@ -524,7 +525,7 @@ export function cmdPlanSeed(args: string[]): void {
     process.exit(1);
   }
   const cwd = process.cwd();
-  const config = loadConfig(join(cwd, "fapony.config.json"));
+  const config = loadConfig(join(cwd, CONFIG_FILENAME));
   // Resolve the git worktree root so mem/ledger queries hit the same key
   // state.db uses (git rev-parse --show-toplevel). Running from a subdir
   // would otherwise mismatch: stats return empty, Context (fapony) always
@@ -568,7 +569,7 @@ export function cmdPlanSeed(args: string[]): void {
     );
   }
 
-  const planDirAbs = join(cwd, planDir(config));
+  const planDirAbs = join(cwd, planDir());
   const planPath = join(planDirAbs, `PLAN-${name}.md`);
   if (existsSync(planPath)) {
     console.error(
@@ -582,7 +583,7 @@ export function cmdPlanSeed(args: string[]): void {
 
   let specLink: string | null = null;
   if (withSpec) {
-    const specDirAbs = join(cwd, specDir(config));
+    const specDirAbs = join(cwd, specDir());
     const specPath = join(specDirAbs, `SPEC-${name}.md`);
     if (existsSync(specPath)) {
       console.error(
@@ -602,7 +603,7 @@ export function cmdPlanSeed(args: string[]): void {
           : null,
       ),
     );
-    specLink = `../${specDir(config).split("/").pop()}/SPEC-${name}.md`;
+    specLink = `../${specDir().split("/").pop()}/SPEC-${name}.md`;
   }
 
   mkdirSync(planDirAbs, { recursive: true });
@@ -613,7 +614,7 @@ export function cmdPlanSeed(args: string[]): void {
       priorArt,
       contextFapony,
       specLink,
-      `${planDir(config)}/PLAN-${name}.md`,
+      `${planDir()}/PLAN-${name}.md`,
     ),
   );
   console.log(`wrote ${planPath}${specLink ? ` + SPEC-${name}.md` : ""}`);

package/src/setup.ts

@@ -6,6 +6,7 @@ import { existsSync, statSync, writeFileSync } from "node:fs";
 import { join, resolve } from "node:path";
 import { createInterface } from "node:readline";
 import { seedConventionsFile } from "./conventions-seed.js";
+import { CONFIG_FILENAME, FAPONY_DIR } from "./db/index.js";
 import { initProject } from "./init.js";
 import { isAffirmative } from "./util.js";
 
@@ -181,7 +182,7 @@ export async function cmdSetup(deps: SetupDeps = {}): Promise<void> {
       enableMemory,
     });
 
-    const configPath = join(process.cwd(), "fapony.config.json");
+    const configPath = join(process.cwd(), CONFIG_FILENAME);
     if (existsSync(configPath)) {
       const overwrite = await askFn(
         "⚠  fapony.config.json already exists. Overwrite? (y/n)",
@@ -197,7 +198,7 @@ export async function cmdSetup(deps: SetupDeps = {}): Promise<void> {
     console.log(`\n  ✓  Wrote ${configPath}`);
 
     // --- scaffold worktree ---
-    const faponyDir = join(worktreePath, ".fapony");
+    const faponyDir = join(worktreePath, FAPONY_DIR);
     if (!existsSync(faponyDir)) {
       try {
         initProject(worktreePath);

package/src/stats/data.ts

@@ -1,9 +1,8 @@
 // src/stats/data.ts — StatsData shape + getStatsData()
 
-import { readdirSync, readFileSync } from "node:fs";
+import { readdirSync } from "node:fs";
 import { join } from "node:path";
-
-import { type Event, openDb, type Run } from "../db/index.js";
+import { type Event, openDb, PLAN_DIR, type Run } from "../db/index.js";
 import { loadConfig } from "../db/load.js";
 import { enrichGateWindows } from "../gates.js";
 import { avg, minutesBetween } from "../math.js";
@@ -89,9 +88,7 @@ function enrichGates(
 /**
  * Count un-shipped plan files in a worktree's planDir.
  *
- * Reads the *target repo's own* fapony.config.json for `paths.planDir` — the
- * central config's worktrees map is optional and usually absent, and each repo
- * picks its own plan dir (vela uses apps/vela/plan, not .fapony/plan).
+ * planDir is hardcoded to .fapony/plan — not configurable (gitignored = private).
  *
  * Returns null — never 0 — when the path isn't a readable directory, so a
  * sentinel row like "mcp-external" renders as "—" instead of claiming
@@ -99,19 +96,10 @@ function enrichGates(
  */
 export function countPendingPlans(worktree: string): number | null {
   if (!worktree.startsWith("/")) return null;
-  let planDir = ".fapony/plan";
-  try {
-    const cfg = JSON.parse(
-      readFileSync(join(worktree, "fapony.config.json"), "utf8"),
-    ) as { paths?: { planDir?: unknown } };
-    if (typeof cfg.paths?.planDir === "string" && cfg.paths.planDir)
-      planDir = cfg.paths.planDir;
-  } catch {
-    // no config (or unreadable/malformed) — fall back to the scaffold default
-  }
   try {
-    return readdirSync(join(worktree, planDir)).filter((f) => f.endsWith(".md"))
-      .length;
+    return readdirSync(join(worktree, PLAN_DIR)).filter((f) =>
+      f.endsWith(".md"),
+    ).length;
   } catch {
     return null;
   }