@bongos/core 1.21.176 → 1.21.178

package/scripts/bongos/classify-discipline.js

@@ -1,153 +0,0 @@
-#!/usr/bin/env node
-// classify-discipline — bulk-classify tasks.discipline via the heuristic
-// classifier in src/bongos/task-classifier.js.
-//
-// Usage:
-//   node scripts/bongos/classify-discipline.js [--commit] [--all] [--limit N]
-//
-// Default mode: dry-run. Walks every task whose discipline is currently
-// 'unclassified', runs classifyTaskDiscipline() over title + description +
-// touches, and prints a table of proposed classifications. Nothing is
-// written to the DB.
-//
-// With --commit: same walk, but PATCH /tasks/:id for every proposal that
-// came back as engineer, artist, or ideator. Tasks the classifier leaves as
-// 'unclassified' are skipped (left for hand-triage).
-//
-// With --all: include tasks that already have a non-unclassified
-// discipline. Useful for re-running after tweaking the classifier rules;
-// will overwrite existing classifications. NOT the default.
-//
-// Idempotent without --all: re-running after a successful --commit is a
-// no-op because every previously-classified row falls outside the filter.
-
-const { cliClient, requireSession } = require('./cli-lib');
-// task-classifier carved to modules/lifecycle/ (BV1.R86); a CLI may require a
-// module directly (outside the kernel boundary — the grading/economy precedent).
-const { classifyTaskDiscipline } = require('../../modules/lifecycle/task-classifier');
-
-function pad(s, n) {
-  s = String(s);
-  return s.length >= n ? s.slice(0, n) : s + ' '.repeat(n - s.length);
-}
-
-function emoji(d) {
-  if (d === 'artist') return '🎨';
-  if (d === 'ideator') return '💡';
-  if (d === 'engineer') return '🔧';
-  return '· ';
-}
-
-(async () => {
-  await requireSession();
-  const args = process.argv.slice(2);
-  const commit = args.includes('--commit');
-  const all = args.includes('--all');
-  const limitIdx = args.indexOf('--limit');
-  const limit = limitIdx >= 0 ? Number(args[limitIdx + 1]) : Infinity;
-
-  // Fetch tasks. Without --all we ask the API for only unclassified rows
-  // so we don't have to pull every shipped task to triage them later.
-  const api = await cliClient();
-  const r = await api.tasks.getTasks(all ? {} : { query: { discipline: 'unclassified' } });
-  if (!r.ok) {
-    console.error(`failed (${r.status}):`, r.data);
-    process.exit(1);
-  }
-  const rawTasks = (r.data?.tasks || []).slice(0, limit);
-  // Skip __smoke__ test debris — these aren't real work, classifying them
-  // would just pollute the discipline filter. Mig 005 already filters them
-  // from claimable_tasks / version_progress on the public side.
-  const smokeFiltered = rawTasks.filter((t) => /^__smoke__/.test(t.title || ''));
-  const tasks = rawTasks.filter((t) => !/^__smoke__/.test(t.title || ''));
-  if (tasks.length === 0) {
-    console.log('nothing to classify — every task already has a discipline.');
-    if (smokeFiltered.length) {
-      console.log(`(skipped ${smokeFiltered.length} __smoke__ debris rows)`);
-    }
-    return;
-  }
-
-  // Run classifier locally — no API round-trip per task.
-  const proposals = tasks.map((t) => {
-    const result = classifyTaskDiscipline(t.title, t.description, t.touches);
-    return { task: t, ...result };
-  });
-
-  // Print summary table grouped by discipline.
-  const buckets = {
-    engineer: proposals.filter((p) => p.discipline === 'engineer'),
-    artist: proposals.filter((p) => p.discipline === 'artist'),
-    ideator: proposals.filter((p) => p.discipline === 'ideator'),
-    unclassified: proposals.filter((p) => p.discipline === 'unclassified'),
-  };
-
-  console.log('');
-  console.log(`Proposed classifications across ${proposals.length} tasks:`);
-  console.log(`  ${emoji('engineer')} engineer      ${buckets.engineer.length}`);
-  console.log(`  ${emoji('artist')} artist        ${buckets.artist.length}`);
-  console.log(`  ${emoji('ideator')} ideator       ${buckets.ideator.length}`);
-  console.log(`  ${emoji('unclassified')} unclassified  ${buckets.unclassified.length} (will be skipped)`);
-  if (smokeFiltered.length) {
-    console.log(`  🧪 __smoke__     ${smokeFiltered.length} (filtered as test debris, not classified)`);
-  }
-  console.log('');
-
-  // Detail by bucket. Within each bucket: highest-confidence first so
-  // the strong signals are at the top — easy to spot the weakest.
-  for (const bucket of ['engineer', 'artist', 'ideator']) {
-    if (!buckets[bucket].length) continue;
-    console.log(`--- ${bucket} ---`);
-    const sorted = buckets[bucket].slice().sort((a, b) => b.confidence - a.confidence);
-    for (const p of sorted) {
-      const { task: t } = p;
-      const version = pad(`[${t.version_id || '?'}]`, 10);
-      const conf = `(${p.confidence.toFixed(2)})`;
-      const title = (t.title || '').slice(0, 80);
-      console.log(`  #${pad(t.id, 4)} ${conf}  ${version} ${title}`);
-    }
-    console.log('');
-  }
-
-  if (buckets.unclassified.length) {
-    console.log('--- unclassified (signals too weak / balanced) ---');
-    for (const p of buckets.unclassified.slice(0, 20)) {
-      const { task: t } = p;
-      const version = pad(`[${t.version_id || '?'}]`, 10);
-      const title = (t.title || '').slice(0, 80);
-      console.log(`  #${pad(t.id, 4)}         ${version} ${title}`);
-    }
-    if (buckets.unclassified.length > 20) {
-      console.log(`  ... ${buckets.unclassified.length - 20} more`);
-    }
-    console.log('');
-  }
-
-  if (!commit) {
-    console.log('Dry-run only. Re-run with --commit to write classifications via PATCH.');
-    return;
-  }
-
-  // Commit: PATCH each non-unclassified proposal.
-  const toWrite = [...buckets.engineer, ...buckets.artist, ...buckets.ideator];
-  console.log(`Committing ${toWrite.length} classifications...`);
-  let ok = 0;
-  let failed = 0;
-  for (const p of toWrite) {
-    const res = await api.tasks.patchTasksId({
-      id: p.task.id,
-      body: { discipline: p.discipline },
-    });
-    if (res.ok) {
-      ok++;
-    } else {
-      failed++;
-      console.error(`  PATCH #${p.task.id} failed (${res.status}):`, res.data?.error || res.data);
-    }
-  }
-  console.log('');
-  console.log(`Done. ${ok} updated, ${failed} failed.`);
-})().catch((err) => {
-  console.error('fatal:', err);
-  process.exit(1);
-});

package/scripts/bongos/codemod-rename-src-gds.js

@@ -1,145 +0,0 @@
-#!/usr/bin/env node
-// scripts/gds/codemod-rename-src-gds.js
-//
-// One-shot, auditable codemod for task 2004 (BV1.R124): rename the core source
-// directory src/gds/ -> src/bongos/ and rewrite EVERY in-repo reference to it.
-// Committed so the approach is reproducible and reviewable (task done-when).
-//
-// Deterministic by construction:
-//   1. `git mv src/gds src/bongos`  (idempotent -- skipped if already moved).
-//   2. Over every TRACKED text file, rewrite the literal path token `src/gds`
-//      -> `src/bongos`, guarded by a TRAILING boundary so look-alike tokens are
-//      never clobbered:
-//         src/gds/...     -> src/bongos/...     (directory refs)
-//         "src/gds"       -> "src/bongos"       (bare, at a boundary)
-//         src/gds-client  -> src/gds-client     (UNCHANGED: a look-alike file name)
-//         src/gdsx        -> src/gdsx           (UNCHANGED here: prefix-edge fixture)
-//
-//   Regex: /src\/gds(?![A-Za-z0-9_-])/g  -- trailing guard only. No LEADING guard:
-//   `scripts/gds` contains no `src/gds` substring, and the two real leading-glue
-//   sites (`keyed-to-src/gds` in an audit, `\tsrc/gds/auth.js` in a mirror-export
-//   fixture) are genuine directory refs we DO want rewritten.
-//
-// Excluded from the sweep (see EXCLUDE):
-//   - this script (its own source contains the search literal),
-//   - tests/path_match.mjs + tests/touches_overlap.mjs -- they assert the
-//     "src/gdsx must NOT match src/gds/" boundary edge; a mechanical half-rewrite
-//     (which leaves `src/gdsx` but changes `src/gds/`) would silently gut the
-//     test. Those two are hand-edited to src/bongos / src/bongosx so the edge
-//     stays meaningful.
-//
-// git ls-files is the file source, so gitignored generated docs
-// (docs/repo-map.md, docs/session-log-index.md) are excluded -- they regenerate.
-// Binary files are skipped via a NUL-byte sniff.
-//
-// Idempotent: a second run moves nothing and rewrites nothing (0 replacements).
-
-const { execSync } = require('node:child_process');
-const fs = require('node:fs');
-const path = require('node:path');
-
-const REPO_ROOT = execSync('git rev-parse --show-toplevel').toString().trim();
-
-// (a) The slash form: `src/gds`, `src/gds/routes`, `"src/gds"` ... Trailing guard
-// only (see header) so `src/gds-client` and `src/gdsx` are left alone.
-const NEEDLE = /src\/gds(?![A-Za-z0-9_-])/g;
-const REPLACEMENT = 'src/bongos';
-
-// (b) The SPLIT-SEGMENT form: `path.join(root, 'src', 'gds', ...)` -- the segments
-// are separate string args, so there is no literal `src/gds` substring for (a) to
-// catch. Anchored on a PRECEDING quoted `'src'` segment so it rewrites only the
-// `'gds'` that follows `'src'` -- NEVER `'scripts', 'gds'` (scripts/gds is not
-// renamed). \s between segments tolerates a newline-split path.join. The `gds`
-// quote must close immediately, so `'gdsx'` / `'gds-client'` never match.
-const SEG = /(['"])src\1(\s*,\s*)(['"])gds\3/g;
-const segReplace = (s) => s.replace(SEG, (_m, q1, sep, q2) => `${q1}src${q1}${sep}${q2}bongos${q2}`);
-
-// (c) RELATIVE requires into the moved dir. A file DIRECTLY under src/ addresses
-// its former sibling as `require('./gds/x')` -- no `src/` prefix, so (a) misses it,
-// and after the move it dangles. `./gds` resolves to src/gds ONLY from a src-top-
-// level file; a `./gds` in scripts/* means scripts/gds (NOT renamed) and a deeper
-// src/**/* file resolves elsewhere -- so this pass is gated to `src/<file>` and
-// never fires elsewhere. (Verified: the only matches are src/module-api.js +
-// src/platform-server.js; no `../gds` relative forms exist.) `(?=\/|['"])` keeps
-// `./gds-client` / `./gdsx` safe.
-const isSrcTopLevel = (rel) => /^src\/[^/]+\.(?:js|mjs|cjs)$/.test(rel);
-const REL = /(\.\/)gds(?=\/|['"])/g;
-
-// (d) REGEX-ESCAPED slash form: `src\/gds` (backslash-slash) as written inside a
-// RegExp literal -- e.g. route-rank-check.js's ROUTE_FILE_RE and the tests that
-// match against it. `\/` is not the plain `/` (a) scans for, so (a) misses it.
-const ESC = /src\\\/gds(?![A-Za-z0-9_-])/g;
-
-// (e) WINDOWS double-backslash form: `src\\gds` in JS source (one backslash on
-// disk-per-escape, two chars) -- Windows-path fixtures the matchers normalize
-// (permission_path / publish_manifest). Trailing guard keeps `src\\gdsx` safe.
-const WIN = /src\\\\gds(?![A-Za-z0-9_-])/g;
-
-const EXCLUDE = new Set([
-  'scripts/gds/codemod-rename-src-gds.js',
-  'tests/path_match.mjs',
-  'tests/touches_overlap.mjs',
-]);
-
-function gitMv() {
-  const oldDir = path.join(REPO_ROOT, 'src', 'gds');
-  const newDir = path.join(REPO_ROOT, 'src', 'bongos');
-  if (fs.existsSync(newDir)) {
-    console.log('git mv: src/bongos already exists -- skipping move (idempotent).');
-    return;
-  }
-  if (!fs.existsSync(oldDir)) {
-    console.log('git mv: neither src/gds nor src/bongos present -- nothing to move.');
-    return;
-  }
-  execSync('git mv src/gds src/bongos', { cwd: REPO_ROOT, stdio: 'inherit' });
-  console.log('git mv: src/gds -> src/bongos');
-}
-
-function isBinary(buf) {
-  const n = Math.min(buf.length, 8192);
-  for (let i = 0; i < n; i++) if (buf[i] === 0) return true;
-  return false;
-}
-
-function main() {
-  gitMv();
-
-  const tracked = execSync('git ls-files', { cwd: REPO_ROOT, maxBuffer: 128 * 1024 * 1024 })
-    .toString().split('\n').filter(Boolean);
-
-  let filesChanged = 0;
-  let totalReplacements = 0;
-  const changed = [];
-
-  for (const rel of tracked) {
-    if (EXCLUDE.has(rel)) continue;
-    const abs = path.join(REPO_ROOT, rel);
-    let buf;
-    try { buf = fs.readFileSync(abs); } catch { continue; } // e.g. deleted-in-index
-    if (isBinary(buf)) continue;
-    const before = buf.toString('utf8');
-    const slashN = (before.match(NEEDLE) || []).length;
-    const segN = (before.match(SEG) || []).length;
-    const escN = (before.match(ESC) || []).length;
-    const winN = (before.match(WIN) || []).length;
-    const relN = isSrcTopLevel(rel) ? (before.match(REL) || []).length : 0;
-    if (slashN === 0 && segN === 0 && escN === 0 && winN === 0 && relN === 0) continue;
-    let after = segReplace(before.replace(NEEDLE, REPLACEMENT));
-    after = after.replace(ESC, 'src\\/bongos').replace(WIN, 'src\\\\bongos');
-    if (relN > 0) after = after.replace(REL, '$1bongos');
-    if (after === before) continue;
-    fs.writeFileSync(abs, after);
-    filesChanged++;
-    const n = slashN + segN + escN + winN + relN;
-    totalReplacements += n;
-    changed.push({ rel, n });
-  }
-
-  changed.sort((a, b) => b.n - a.n);
-  console.log(`\ncodemod: rewrote ${totalReplacements} reference(s) across ${filesChanged} file(s).`);
-  for (const c of changed) console.log(`  ${String(c.n).padStart(4)}  ${c.rel}`);
-  console.log('\nExcluded (hand-edited or self): ' + [...EXCLUDE].join(', '));
-}
-
-main();