kenaz 0.3.3 → 0.3.5

package/core/.formation/tools/post-complete/task2345.test.js

@@ -31,8 +31,3 @@ test('post-complete sources contain no observe / huginn-think spawn path', () =>
     assert.doesNotMatch(src, /triggerMuninnObserver|triggerHuginn|incrementMilestoneCounter|writeMuninnSignal|milestone_counter/, `${f} still references a removed trigger`);
   }
 });
-
-test('headless.js has no observe-once / watch-mode / loop-heartbeat entry points', () => {
-  const src = read('headless.js');
-  assert.doesNotMatch(src, /--observe-once|--watch-mode|--loop-heartbeat|muninn-watcher/);
-});

package/core/.formation/tools/pre-dispatch-snapshot.test.js

@@ -30,7 +30,6 @@ const { execSync } = require('child_process');
 // Requiring post-complete wires initGitMerge(deps) as a load-time side effect.
 require('./post-complete');
 const { _gitMergeFromWorktree } = require('./post-complete/git-merge');
-const { detectWorktreeLeak } = require('./runner');
 const {
   captureDispatchDirtySnapshot,
   readDispatchDirtySnapshot,
@@ -93,105 +92,6 @@ function teardown(tmpRoot, repo) {
   fs.rmSync(tmpRoot, { recursive: true, force: true });
 }
 
-function listBranches(repo) {
-  return (gitSafe('git branch --format=%(refname:short)', repo) || '')
-    .split('\n').map(s => s.trim()).filter(Boolean);
-}
-
-// ─────────────────────────────────────────────────────────────────────────────
-// DEFECT-D repro (TASK_1779 run 1): pre-existing dirty files + zero-commit agent
-// → NO leak false positive, NO salvage branch, NO marker, NO exit-code flip.
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('TASK_1785 DEFECT-D: zero-commit + only pre-existing dirt is NOT a leak (no salvage branch)', () => {
-  const { tmpRoot, repo } = setup();
-  try {
-    const sessionId = 'sess1785-neg';
-
-    // Committed baseline the user then dirties BEFORE dispatch.
-    fs.writeFileSync(path.join(repo, 'usercfg.js'), 'module.exports = { v: 1 };\n');
-    git('git add usercfg.js', repo);
-    git('git commit --no-verify -m "seed usercfg"', repo);
-
-    // PRE-EXISTING user WIP: tracked modification + untracked file.
-    const userWip = 'module.exports = { v: 1, userEdit: true };\n';
-    fs.writeFileSync(path.join(repo, 'usercfg.js'), userWip);
-    fs.writeFileSync(path.join(repo, 'user-notes.md'), 'draft\n');
-
-    // Dispatch start: snapshot captured (what setupUnitWorktree does).
-    const snap = captureDispatchDirtySnapshot(repo, { sessionId, taskId: 'TASK_1785N' });
-    assert.ok(snap && snap.dirtyPaths.includes('usercfg.js') && snap.dirtyPaths.includes('user-notes.md'),
-      `snapshot must record pre-existing dirt, got ${JSON.stringify(snap && snap.dirtyPaths)}`);
-
-    // Agent worktree with ZERO commits (default bifrost state under DEFECT-A).
-    const worktree = path.join(tmpRoot, 'wt');
-    git(`git worktree add -b task/task_1785n "${worktree}" HEAD`, repo);
-    const originalWorktreeHead = git('git rev-parse HEAD', worktree);
-
-    const child = { _formation: { sessionId, skill: 'feature_agent' } };
-    const res = detectWorktreeLeak(worktree, originalWorktreeHead, child, repo);
-
-    // NOT a leak.
-    assert.equal(res, null, `pre-existing dirt must not be a leak, got ${JSON.stringify(res)}`);
-    assert.equal(child._formationExitCode, undefined, 'exit code must remain unset');
-    assert.ok(!child._formationLeak, 'no leak record');
-    // No salvage branch, no marker.
-    assert.ok(!listBranches(repo).some(b => b.startsWith('tn1529-salvage/')),
-      'no tn1529-salvage branch may be created for pre-existing dirt');
-    assert.ok(!fs.existsSync(path.join(worktree, 'LEAK_DETECTED')), 'no LEAK_DETECTED marker');
-    // User WIP untouched in the working tree.
-    assert.equal(fs.readFileSync(path.join(repo, 'usercfg.js'), 'utf8'), userWip, 'user WIP must stay untouched');
-  } finally {
-    teardown(tmpRoot, repo);
-  }
-});
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Positive control: a file NEWLY dirtied during the dispatch window IS still
-// detected and salvaged — and the pre-existing file is excluded from the set.
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('TASK_1785 positive control: newly-dirty file during dispatch window is still detected + salvaged', () => {
-  const { tmpRoot, repo } = setup();
-  try {
-    const sessionId = 'sess1785-pos';
-
-    fs.writeFileSync(path.join(repo, 'usercfg.js'), 'user v1\n');
-    fs.writeFileSync(path.join(repo, 'agentfile.js'), 'agent v1\n');
-    git('git add usercfg.js agentfile.js', repo);
-    git('git commit --no-verify -m "seed"', repo);
-
-    // Pre-existing user WIP before dispatch.
-    fs.writeFileSync(path.join(repo, 'usercfg.js'), 'user v1 + WIP\n');
-
-    captureDispatchDirtySnapshot(repo, { sessionId, taskId: 'TASK_1785P' });
-
-    const worktree = path.join(tmpRoot, 'wt');
-    git(`git worktree add -b task/task_1785p "${worktree}" HEAD`, repo);
-    const originalWorktreeHead = git('git rev-parse HEAD', worktree);
-
-    // DURING the dispatch window: agent write leaks to MAIN repo (zero commits).
-    const leakedContent = 'agent v2 LEAKED\n';
-    fs.writeFileSync(path.join(repo, 'agentfile.js'), leakedContent);
-
-    const child = { _formation: { sessionId, skill: 'feature_agent' } };
-    const res = detectWorktreeLeak(worktree, originalWorktreeHead, child, repo);
-
-    assert.ok(res && res.leaked === true, `true leak must still be detected, got ${JSON.stringify(res)}`);
-    assert.deepEqual(res.leakedPaths, ['agentfile.js'],
-      `only the newly-dirty path counts as leak, got ${JSON.stringify(res.leakedPaths)}`);
-    assert.equal(child._formationExitCode, 1, 'true leak must still block the dispatch');
-    assert.ok(res.salvageBranch, 'true leak must still be salvaged');
-    assert.equal(gitSafe(`git show ${res.salvageBranch}:agentfile.js`, repo), leakedContent.trim(),
-      'salvage branch must contain the leaked content');
-    // Salvage tree is HEAD + leaked paths → the pre-existing WIP is NOT on it.
-    assert.equal(gitSafe(`git show ${res.salvageBranch}:usercfg.js`, repo), 'user v1',
-      'pre-existing user WIP must NOT be included in the leak salvage');
-  } finally {
-    teardown(tmpRoot, repo);
-  }
-});
-
 // ─────────────────────────────────────────────────────────────────────────────
 // DEFECT-C repro: scopePaths=[] rescue scan must NOT sweep pre-existing user WIP
 // (tracked modification AND untracked file) into the worktree/merge.