forge-workflow 0.1.0-beta.3 → 0.1.0-beta.5

package/skills/worktree/evals/scorecard.json

@@ -0,0 +1,41 @@
+{
+  "skill": "worktree",
+  "fixtures": "present",
+  "static": {
+    "token_cost": {
+      "desc_chars": 1013,
+      "body_lines": 75,
+      "score": 43
+    },
+    "caps": {
+      "desc_within": true,
+      "body_within": true,
+      "score": 100
+    },
+    "description_quality": {
+      "has_trigger_cues": true,
+      "has_disambiguation_cues": true,
+      "adequate_length": true,
+      "score": 100
+    }
+  },
+  "router_reachability": {
+    "has_curated_rule": true,
+    "router_exempt": false,
+    "fixtures": "present",
+    "fixtures_total": 6,
+    "fixtures_best_hit": 6,
+    "reachable": true,
+    "keyword_alignment": 1
+  },
+  "behavioral": {
+    "trigger_recall": null,
+    "trigger_precision": null,
+    "disambiguation": null,
+    "chain_correctness": null,
+    "outcome_quality": null,
+    "variance": null,
+    "note": "behavioral — W5 (LLM judge): semantic recall/precision/chain/outcome/variance, not deterministic"
+  },
+  "composite": 83
+}

package/lib/adapters/beads-issue-adapter.js

@@ -1,127 +0,0 @@
-'use strict';
-
-const {
-  IssueAdapter,
-  decideIssueAuthority,
-  normalizeIssueStatus,
-} = require('../issue-adapter.js');
-
-class BeadsIssueAdapter extends IssueAdapter {
-  constructor(options = {}) {
-    super({
-      id: options.id || 'beads',
-      kind: 'issue',
-      name: options.name || 'Beads Issue Adapter',
-      version: options.version || '0.1.0',
-    });
-    this.runBeadsOperation = options.runBeadsOperation;
-  }
-
-  run(operation, args = [], context = {}) {
-    if (typeof this.runBeadsOperation !== 'function') {
-      throw new TypeError('runBeadsOperation is not configured on this adapter');
-    }
-
-    return this.runBeadsOperation(operation, args, context, context.deps || {});
-  }
-
-  list(args = [], context = {}) {
-    return this.run('list', args, context);
-  }
-
-  ready(args = [], context = {}) {
-    return this.run('ready', args, context);
-  }
-
-  read(args = [], context = {}) {
-    return this.run('show', args, context);
-  }
-
-  show(args = [], context = {}) {
-    return this.read(args, context);
-  }
-
-  create(args = [], context = {}) {
-    return this.run('create', args, context);
-  }
-
-  search(args = [], context = {}) {
-    return this.run('search', args, context);
-  }
-
-  stats(args = [], context = {}) {
-    return this.run('stats', args, context);
-  }
-
-  update(args = [], context = {}) {
-    return this.run('update', args, context);
-  }
-
-  close(args = [], context = {}) {
-    return this.run('close', args, context);
-  }
-
-  comment(args = [], context = {}) {
-    return this.run('comment', args, context);
-  }
-
-  depAdd(args = [], context = {}) {
-    return this.run('dep.add', args, context);
-  }
-
-  depRemove(args = [], context = {}) {
-    return this.run('dep.remove', args, context);
-  }
-
-  // De-bead parity: claim/release and the KAP-7/KAP-12 derived reads now route
-  // through the backend abstraction (the translations moved out of _issue.js). The
-  // beads layer maps claim -> `update <id> --claim`; release has no bd equivalent
-  // and returns the Kernel-only contract error (handled in runBeadsOperation).
-  claim(args = [], context = {}) {
-    return this.run('claim', args, context);
-  }
-
-  release(args = [], context = {}) {
-    return this.run('release', args, context);
-  }
-
-  blocked(args = [], context = {}) {
-    return this.run('blocked', args, context);
-  }
-
-  stale(args = [], context = {}) {
-    return this.run('stale', args, context);
-  }
-
-  orphans(args = [], context = {}) {
-    return this.run('orphans', args, context);
-  }
-
-  lint(args = [], context = {}) {
-    return this.run('lint', args, context);
-  }
-
-  // Lease ownership is Kernel-only; the beads layer returns the Kernel-only contract
-  // error (handled in runBeadsOperation), matching the release passthrough.
-  owns(args = [], context = {}) {
-    return this.run('owns', args, context);
-  }
-
-  // Active-lease listing is Kernel-only (kernel_claims); the beads layer returns the
-  // Kernel-only contract error (handled in runBeadsOperation), matching owns/release.
-  claims(args = [], context = {}) {
-    return this.run('claims', args, context);
-  }
-
-  mapStatus(status, context = {}) {
-    return normalizeIssueStatus(status, context);
-  }
-
-  decideAuthority(change = {}, context = {}) {
-    return decideIssueAuthority(change, context);
-  }
-}
-
-module.exports = {
-  BeadsIssueAdapter,
-};

package/lib/beads-nudge.js

@@ -1,91 +0,0 @@
-'use strict';
-
-const { resolveIssueBackend } = require('./issue-backend');
-const { detectBeadsJsonlSource } = require('./beads-detect');
-
-// Unmigrated legacy-store nudge (kernel issue a5399f3d — upgrade-safety
-// 0.0.10 -> current).
-//
-// The 0.0.10 -> current upgrade flipped the DEFAULT issue backend to the Kernel,
-// but the legacy -> kernel migration fires ONLY from `forge setup`/`init`, never
-// lazily on the issue path. So a user who runs `bun update` then `forge list` /
-// `forge ready` reads an EMPTY kernel and their 0.0.10 issues APPEAR GONE — the
-// data is safe on disk, just invisible, with no hint. This helper closes that
-// footgun: when the resolved backend is the (default) Kernel, a read comes back
-// empty, AND a legacy jsonl store still exists, it prints a one-time guided hint.
-//
-// It lives in this NEUTRAL module (not lib/commands/_issue.js) on purpose: the
-// message text names the retired backend + its `.beads` store, and _issue.js is a
-// D20 release-readiness hot-path surface that must stay free of those tokens (the
-// bd-call-site audit + kernel-backed checks scan it). Keeping the strings here
-// lets the hot path call a token-free helper.
-//
-// Best-effort and non-blocking: it NEVER throws and never alters the read result.
-// Fires at most once per project root per process to avoid spam.
-const NUDGE_READS = new Set(['list', 'ready']);
-const warnedRoots = new Set();
-
-// A kernel read looks empty when its contract data carries no issues. Handles the
-// issue.list/issue.ready shape ({ issues, count }) plus a bare array / null, and
-// is deliberately conservative: anything it cannot confirm as empty is treated as
-// NON-empty so a user with real kernel issues is never nagged.
-function kernelReadLooksEmpty(result) {
-  if (!result || typeof result !== 'object' || result.ok !== true) {
-    return false;
-  }
-  const data = result.data;
-  if (data === null || data === undefined) {
-    return true;
-  }
-  if (Array.isArray(data)) {
-    return data.length === 0;
-  }
-  if (Array.isArray(data.issues)) {
-    return data.issues.length === 0;
-  }
-  if (Array.isArray(data.items)) {
-    return data.items.length === 0;
-  }
-  if (typeof data.count === 'number') {
-    return data.count === 0;
-  }
-  return false;
-}
-
-function maybeWarnUnmigratedBeads(subcommand, result, projectRoot, rawOpts = {}) {
-  try {
-    if (!NUDGE_READS.has(subcommand) || !projectRoot) {
-      return;
-    }
-    if (warnedRoots.has(projectRoot)) {
-      return;
-    }
-    const env = rawOpts.env || process.env;
-    // Resolve the EFFECTIVE backend (default kernel). An explicit opt-in to the
-    // retired backend means the user chose it deliberately — nothing to nudge.
-    const backend = resolveIssueBackend({ deps: rawOpts, env, projectRoot, warn: () => {} });
-    if (backend !== 'kernel' || !kernelReadLooksEmpty(result)) {
-      return;
-    }
-    if (!detectBeadsJsonlSource(projectRoot)) {
-      return;
-    }
-    warnedRoots.add(projectRoot);
-    console.error(
-      '\n[forge] Legacy issue data detected (.beads/*.jsonl) but the Kernel issue store is empty.\n'
-      + 'Forge now defaults to the Kernel backend (breaking change since 0.0.10). Your legacy\n'
-      + 'issues are safe on disk but will not appear until migrated. To migrate:\n'
-      + '  forge migrate --from beads   # import your legacy issues into the Kernel\n'
-      + '  forge setup                  # (re)wire hooks + provision the Kernel store\n'
-      + 'Prefer to stay on the legacy backend? Set `issueBackend: beads` in .forge/config.yaml '
-      + '(or FORGE_ISSUE_BACKEND=beads).\n',
-    );
-  } catch (_err) { // NOSONAR S2486 — line-scoped; must sit on the catch line
-    /* the nudge is a best-effort hint; it must never break a read */
-  }
-}
-
-module.exports = {
-  maybeWarnUnmigratedBeads,
-  kernelReadLooksEmpty,
-};