/**
 * Tests for the Windows npm resolution platform gate.
 *
 * Background (issue #3103, PR #3102):
 *   On Windows, `npm` ships as `npm.cmd`. Node's spawn does not apply PATHEXT
 *   resolution and fails with ENOENT. The fix is to spawn through a shell on
 *   Windows (cmd.exe resolves npm.cmd via PATHEXT). On POSIX, `npm` resolves
 *   without a shell, so spawning `/bin/sh -c` is pure overhead and changes
 *   signal / exit-code semantics — undesirable.
 *
 * Relocation (#498): the SessionStart worker no longer spawns npm itself. It
 * delegates the latest-version lookup to check-latest-version's
 * `checkLatestVersion()`, which routes through `execNpm` in the shell-command
 * projection seam. The PR #3102 contract therefore now lives on `execNpm`.
 * This test locks it there, and additionally locks that the worker does NOT
 * re-introduce a direct npm spawn (which would re-open the gate question in a
 * second place).
 *
 * Source-grep policy: these structural assertions read source via readFileSync.
 * The behavior (Windows-only shell resolution) is platform-gated at runtime and
 * cannot be reached on POSIX CI without a Windows lane; a structural assertion
 * is the minimum-cost contract.
 */

'use strict';

const { test, describe } = require('node:test');
const assert = require('node:assert/strict');
const fs = require('fs');
const path = require('path');
const { scanFencedBlocks } = require('../msd-core/bin/lib/markdown-sectionizer.cjs');
const { LOOKUP_TIMEOUT_MS } = require('../msd-core/bin/check-latest-version.cjs');

/**
 * NOT a margin above LOOKUP_TIMEOUT_MS (imported above for disclosure,
 * per this batch's epic issue -- this file and check-latest-version.cjs
 * once independently guessed the same 15000ms value, causing a Windows
 * double-SIGKILL collision fixed in PR #4428). This specific scenario
 * (missing compiled runtime library) degrades BEFORE reaching the wrapped
 * release-tag lookup at all -- see this test's own #3582 comment
 * ("checkLatestVersion -> not ok" on the degrade, never invoked) -- so its
 * bound has no margin relationship to LOOKUP_TIMEOUT_MS to preserve.
 * Kept as its own independent, pre-existing value.
 */
const WORKER_DEGRADED_PATH_TIMEOUT_MS = 8000;

/**
 * msd-update-banner.js only reads a small cache file from disk and prints
 * JSON -- no subprocess/network work -- so this leaves generous headroom
 * over its sub-second worst case even on a contended CI runner. NOT
 * related to LOOKUP_TIMEOUT_MS or WORKER_DEGRADED_PATH_TIMEOUT_MS above
 * (a different hook script entirely).
 */
const UPDATE_BANNER_HOOK_TIMEOUT_MS = 10_000;

// Imported for disclosure only (see WORKER_DEGRADED_PATH_TIMEOUT_MS's own
// doc comment above): confirms this file references the collision-prone
// constant from check-latest-version.cjs without deriving any of this
// file's own timeout values from it.
void LOOKUP_TIMEOUT_MS;

const WORKER_PATH = path.join(__dirname, '..', 'hooks', 'msd-check-update-worker.js');
const PROJECTION_PATH = path.join(
  __dirname, '..', 'msd-core', 'bin', 'lib', 'shell-command-projection.cjs',
);

// allow-test-rule: structural-regression-guard (#3103)
// This helper feeds real source (via readFileSync) into structural
// assertions below. The behavior it guards — Windows-only shell
// resolution — is platform-gated at runtime and cannot be reached on
// POSIX CI without a Windows lane, so a structural assertion on the
// spawn-options shape is the minimum-cost contract.
function codeOnly(file) {
  return fs.readFileSync(file, 'utf8')
    // eslint-disable-next-line local/no-unbounded-quantifier -- parses this repo's own bounded hooks/lib source, not adversarial input
    .replace(/\/\*[\s\S]*?\*\//g, '')
    // eslint-disable-next-line local/no-unbounded-quantifier -- parses this repo's own bounded hooks source, not adversarial input
    .replace(/(^|[^:])\/\/[^\r\n]*/g, '$1');
}

describe('execNpm: Windows npm spawn platform gate (PR #3102, relocated #498)', () => {
  test('projection seam exists', () => {
    assert.ok(fs.existsSync(PROJECTION_PATH), `not found at ${PROJECTION_PATH}`);
  });

  test('execNpm gates shell to process.platform === "win32"', () => {
    assert.match(
      codeOnly(PROJECTION_PATH),
      /shell:\s*process\.platform\s*===\s*['"]win32['"]/,
      [
        'execNpm must gate shell to `process.platform === "win32"`.',
        'A regression to `shell: true` would spawn /bin/sh -c on POSIX',
        '(adds shell overhead, changes signal/exit semantics). See PR #3102.',
      ].join(' '),
    );
  });

  test('no unconditional shell: true on the npm spawn', () => {
    assert.doesNotMatch(
      codeOnly(PROJECTION_PATH),
      /shell\s*:\s*true\s*[,\s}]/,
      'shell: true is forbidden — use the `process.platform === "win32"` gate.',
    );
  });
});

describe('worker delegates the npm spawn (does not re-open the gate, #498)', () => {
  test('worker does NOT spawn npm directly', () => {
    const code = codeOnly(WORKER_PATH);
    assert.doesNotMatch(
      code,
      /(execFileSync|spawnSync|execSync|exec)\s*\(\s*['"]npm['"]/,
      'Worker must delegate to checkLatestVersion(), not spawn npm itself.',
    );
  });

  test('worker requires check-latest-version for the lookup', () => {
    assert.match(codeOnly(WORKER_PATH), /check-latest-version/);
  });
});

// ─── #3582: cold tree (no msd-core/bin/lib/*.cjs) — degrade, not crash ─────
//
// msd-core/bin/lib/semver-compare.cjs, package-identity.cjs, and (via
// check-latest-version.cjs's own transitive requires) msd-core/bin/lib/
// cli-exit.cjs + shell-command-projection.cjs are tsc build artifacts
// (ADR-457), gitignored and absent on a raw plugin-marketplace / git-clone
// install that never ran `npm run build:lib`. This worker is a DETACHED
// SessionStart background process (spawned with stdio: 'ignore' by
// hooks/msd-check-update.js) — before #3582 a missing library crashed the
// worker at module load with no visible signal (stderr discarded by the
// parent) and no cache-file write at all, so the statusline/banner would
// silently never see an update signal. The fix wraps ensureRuntimeBuild()
// and the three compiled-lib requires in one try/catch and degrades to
// no-signal fallbacks (isSemverNewer -> false, checkLatestVersion -> not ok,
// PACKAGE_NAME -> null) on failure — the worker still runs to completion and
// writes a result cache record. Simulated hermetically via a fixture install
// tree that copies hooks/ + the seam module but never msd-core/bin/lib/ or
// tsconfig.build.json (tests/helpers/cold-runtime-lib-fixture.cjs) — the REAL
// msd-core/bin/lib/ is never touched.
{
  const { describe, test } = require('node:test');
  const assert = require('node:assert/strict');
  const fs = require('node:fs');
  const path = require('node:path');
  const { runHook: runHookSeam } = require('./helpers/process-seam.cjs');
  const { buildColdInstallTree } = require('./helpers/cold-runtime-lib-fixture.cjs');
  const { createTempDir, cleanup } = require('./helpers.cjs');

  describe('msd-check-update-worker.js: #3582 cold tree — degrade, not crash', () => {
    test('missing compiled runtime library -> worker still writes a degraded result cache, no crash', (t) => {
      const cold = buildColdInstallTree();
      t.after(cold.cleanup);
      const cacheDir = createTempDir('msd-worker-cold-');
      t.after(() => cleanup(cacheDir));
      const cacheFile = path.join(cacheDir, 'cache.json');

      const env = {
        ...process.env,
        MSD_CACHE_FILE: cacheFile,
        MSD_PROJECT_VERSION_FILE: path.join(cacheDir, 'no-such-project', 'VERSION'),
        MSD_GLOBAL_VERSION_FILE: path.join(cacheDir, 'no-such-global', 'VERSION'),
      };
      const r = runHookSeam(path.join(cold.hooksDir, 'msd-check-update-worker.js'), [], {
        env,
        timeoutMs: WORKER_DEGRADED_PATH_TIMEOUT_MS,
      });
      assert.equal(r.exitCode, 0, `worker must exit 0 on a build failure; stderr: ${r.stderr}`);
      assert.ok(fs.existsSync(cacheFile), 'worker must still reach the end and write a cache record');
      const cache = JSON.parse(fs.readFileSync(cacheFile, 'utf8'));
      assert.equal(cache.package_name, null, 'degraded package_name must be null (no-signal, never a stale/foreign value)');
      assert.ok(!cache.update_available, 'degraded update_available must be falsy');
      assert.equal(cache.installed, '0.0.0', 'installed detection is unaffected by the compiled-lib degrade');
    });
  });
}

// ─── #3582: cold-tree fixture must tolerate a concurrent build-hooks.js
// staging dir, without mutating the live hooks/ tree ─────────────────────
//
// scripts/build-hooks.js writes atomically via a per-PID staging dir
// (hooks/.dist-staging-<pid>) that it creates and removes; up to nine test
// files invoke it concurrently from their `before()` hooks, so the live
// hooks/ dir is never guaranteed stable during a test run. This is proven
// HERMETICALLY, against a fake source tree under a temp dir — planting a
// staging dir inside the REAL repo's hooks/ would itself be the exact
// shared-state race this fixture exists to guard against (other test files
// read hooks/ concurrently), and cleanup() (tests/helpers.cjs) deliberately
// refuses to remove any path outside the known temp roots, so a real-repo
// plant can never be cleaned up through it either.
{
  const { describe, test } = require('node:test');
  const assert = require('node:assert/strict');
  const fs = require('node:fs');
  const os = require('node:os');
  const path = require('node:path');
  const { buildColdInstallTree, REPO_ROOT, shouldCopyHookEntry } = require('./helpers/cold-runtime-lib-fixture.cjs');
  const { cleanup } = require('./helpers.cjs');

  describe('cold-runtime-lib-fixture.cjs: #3582 tolerates a concurrent hooks/.dist-staging-<pid> dir', () => {
    test('a live .dist-staging-test-<random> dir does not break the fixture copy, and is excluded from it', (t) => {
      // Build a hermetic fake source tree — never touch the real repo's hooks/.
      const fakeRepoRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'msd-fake-repo-'));
      t.after(() => cleanup(fakeRepoRoot));

      const fakeHooksDir = path.join(fakeRepoRoot, 'hooks');
      fs.mkdirSync(fakeHooksDir, { recursive: true });
      // Representative real hook entries the fixture must still carry over.
      fs.writeFileSync(path.join(fakeHooksDir, 'hooks.json'), '{}');
      fs.writeFileSync(path.join(fakeHooksDir, 'top-level-hook.js'), '// fake hook\n');
      fs.mkdirSync(path.join(fakeHooksDir, 'lib'), { recursive: true });
      fs.writeFileSync(path.join(fakeHooksDir, 'lib', 'helper.js'), '// fake lib helper\n');
      // Build output dir — must be excluded.
      fs.mkdirSync(path.join(fakeHooksDir, 'dist'), { recursive: true });
      fs.writeFileSync(path.join(fakeHooksDir, 'dist', 'built.js'), '// built output\n');
      // Concurrent build-hooks.js staging dir — must be excluded, and must
      // not break the copy even while "live".
      const stagingName = `.dist-staging-99999`;
      const stagingDir = path.join(fakeHooksDir, stagingName);
      fs.mkdirSync(stagingDir);
      fs.writeFileSync(path.join(stagingDir, 'scratch.txt'), 'transient build output');

      // The fixture also copies msd-core/bin/ensure-runtime-build.cjs — give
      // the fake tree a real copy of it so buildColdInstallTree() succeeds.
      const fakeBinDir = path.join(fakeRepoRoot, 'msd-core', 'bin');
      fs.mkdirSync(fakeBinDir, { recursive: true });
      fs.copyFileSync(
        path.join(REPO_ROOT, 'msd-core', 'bin', 'ensure-runtime-build.cjs'),
        path.join(fakeBinDir, 'ensure-runtime-build.cjs'),
      );

      // Filtered by shouldCopyHookEntry — the same predicate the fixture
      // itself uses to decide what to copy. Up to nine other test files'
      // before() hooks concurrently invoke scripts/build-hooks.js, which
      // creates/removes hooks/.dist-staging-<pid> at unpredictable times, so
      // a RAW (unfiltered) before/after listing comparison of the live
      // hooks/ dir is itself racy — it can observe a sibling build's
      // transient staging dir appear or vanish between the two snapshots and
      // fail with no real defect. Filtering both snapshots the same way the
      // fixture does preserves the actual intent (this test adds/removes no
      // REAL entry in the repo's hooks/) while tolerating scratch that is
      // not this test's doing and is excluded from the fixture anyway.
      const realHooksBefore = fs
        .readdirSync(path.join(REPO_ROOT, 'hooks'))
        .filter(shouldCopyHookEntry)
        .sort();

      const cold = buildColdInstallTree({ repoRoot: fakeRepoRoot });
      t.after(cold.cleanup);

      const entries = fs.readdirSync(cold.hooksDir);
      assert.ok(
        !entries.some((e) => e.startsWith('.dist-staging')),
        `fixture hooks/ must not contain any .dist-staging* entry, got: ${entries.join(', ')}`,
      );
      assert.ok(!entries.includes('dist'), `fixture hooks/ must not contain dist/, got: ${entries.join(', ')}`);
      assert.ok(entries.includes('hooks.json'), 'fixture must still contain the representative hooks.json');
      assert.ok(entries.includes('top-level-hook.js'), 'fixture must still contain the representative top-level hook');
      assert.ok(
        fs.existsSync(path.join(cold.hooksDir, 'lib', 'helper.js')),
        'fixture must still contain the representative hooks/lib/ subdir file',
      );

      const realHooksAfter = fs
        .readdirSync(path.join(REPO_ROOT, 'hooks'))
        .filter(shouldCopyHookEntry)
        .sort();
      assert.deepEqual(
        realHooksAfter,
        realHooksBefore,
        'the real repo hooks/ directory listing, filtered by shouldCopyHookEntry, must be unchanged by ' +
          'this test (transient hooks/.dist-staging-<pid> entries from concurrent build-hooks.js runs are ' +
          'excluded from the comparison since this test does not own them)',
      );
    });
  });

  // Direct pin on shouldCopyHookEntry() itself — the predicate IS the fix
  // (exact 'dist' match plus a '.dist-staging' PREFIX, not a loose
  // startsWith('dist')/includes('dist') substring match). Pinning it only
  // indirectly, via the fixture-shape assertions above, would let a looser
  // implementation (e.g. name.startsWith('dist')) pass every case above
  // while still being wrong — this pins the exact rule.
  describe('cold-runtime-lib-fixture.cjs: shouldCopyHookEntry() name-filter rule', () => {
    const { shouldCopyHookEntry } = require('./helpers/cold-runtime-lib-fixture.cjs');

    test('excludes the build output dir and any .dist-staging* prefix name', () => {
      assert.equal(shouldCopyHookEntry('dist'), false);
      assert.equal(shouldCopyHookEntry('.dist-staging-20836'), false);
      assert.equal(shouldCopyHookEntry('.dist-staging-test-abc'), false);
      assert.equal(shouldCopyHookEntry('.dist-staging'), false);
    });

    test('keeps real hook entries', () => {
      assert.equal(shouldCopyHookEntry('hooks.json'), true);
      assert.equal(shouldCopyHookEntry('lib'), true);
      assert.equal(shouldCopyHookEntry('msd-check-update.js'), true);
    });

    test('does not over-match on a bare "dist" substring/prefix', () => {
      assert.equal(shouldCopyHookEntry('dist-staging-no-dot'), true);
      assert.equal(shouldCopyHookEntry('distant.js'), true);
    });
  });
}


// ────────────────────────────────────────────────────────────────────────
// Folded from tests/bug-2992-check-latest-version.test.cjs — consolidation epic #1969 (B5 #1974)
// ────────────────────────────────────────────────────────────────────────
{
  const { describe: __foldDescribe } = require('node:test');
  __foldDescribe("folded:bug-2992-check-latest-version (consolidation epic #1969 B5 #1974)", () => {
'use strict';
process.env.MSD_TEST_MODE = '1';

const { test, describe } = require('node:test');
const assert = require('node:assert/strict');
const path = require('node:path');

const ROOT = path.join(__dirname, '..');
const { checkLatestVersion, CHECK_REASON, PACKAGE_NAME, REPO_URL } = require(
  path.join(ROOT, 'msd-core', 'bin', 'check-latest-version.cjs'),
);

// checkLatestVersion is a pure-ish function: it runs one fixed
// `git ls-remote --tags` against the baked repository URL, picks the highest
// release tag, and returns { ok, version | reason }. The repository is
// HARDCODED — not a free choice for the caller. Tests use a pluggable spawn
// so no real git process (or network) is involved.

// `git ls-remote --tags --refs` output: one `<sha>\t<ref>` line per tag.
const lsRemote = (...tags) => tags.map((t, i) => `${String(i).padStart(40, '0')}\trefs/tags/${t}`).join('\n') + '\n';
const okSpawn = (...tags) => () => ({ status: 0, stdout: lsRemote(...tags), stderr: '' });

describe('Bug #2992: deterministic latest-version check', () => {
  test('PACKAGE_NAME is the constant @golem15/msd-core (no callers can override)', () => {
    assert.equal(PACKAGE_NAME, '@golem15/msd-core');
  });

  test('REPO_URL is the baked repository the release tags are read from', () => {
    assert.equal(REPO_URL, 'https://git.golem15.com/golem15/msd-core');
  });

  test('CHECK_REASON enum exposes the documented codes', () => {
    assert.deepEqual(
      Object.keys(CHECK_REASON).sort(),
      ['FAIL_LOOKUP_FAILED', 'FAIL_NO_RELEASES', 'OK'].sort(),
    );
  });

  test('returns { ok: true, version } for the highest vX.Y.Z tag', () => {
    const r = checkLatestVersion({ spawn: okSpawn('v1.39.1', 'v1.40.0', 'v1.9.9') });
    assert.deepEqual(r, { ok: true, version: '1.40.0', reason: CHECK_REASON.OK });
  });
});

describe('Bug #2992: error paths', () => {
  const { checkLatestVersion, CHECK_REASON } = require(require('node:path').join(__dirname, '..', 'msd-core', 'bin', 'check-latest-version.cjs'));

  test('FAIL_LOOKUP_FAILED when git exits non-zero (e.g. offline, repo moved)', () => {
    const r = checkLatestVersion({
      spawn: () => ({ status: 128, stdout: '', stderr: 'fatal: unable to access\n' }),
    });
    assert.equal(r.ok, false);
    assert.equal(r.reason, CHECK_REASON.FAIL_LOOKUP_FAILED);
    assert.equal(r.detail, 'fatal: unable to access',
      'detail should be the trimmed stderr when git reports a real error');
  });

  // #2993 CR: distinguish timeout from genuine failure in `detail`.
  // spawnSync sets status=null and signal='SIGTERM' on timeout; stderr is
  // typically empty. Without the signal-first branch, both shape as
  // 'git exited non-zero' and the operator cannot tell timeout from failure.
  test('FAIL_LOOKUP_FAILED detail names the signal when spawn times out', () => {
    const r = checkLatestVersion({
      spawn: () => ({ status: null, signal: 'SIGTERM', stdout: '', stderr: '' }),
    });
    assert.equal(r.ok, false);
    assert.equal(r.reason, CHECK_REASON.FAIL_LOOKUP_FAILED);
    assert.equal(r.detail, 'git timed out (signal: SIGTERM)',
      'detail should explicitly name the signal when status is null and signal is set');
  });

  test('FAIL_LOOKUP_FAILED detail falls back to generic when neither stderr nor signal is present', () => {
    const r = checkLatestVersion({
      spawn: () => ({ status: 1, stdout: '', stderr: '' }),
    });
    assert.equal(r.detail, 'git exited non-zero');
  });

  test('FAIL_NO_RELEASES when the remote has no release tags', () => {
    const r = checkLatestVersion({ spawn: () => ({ status: 0, stdout: '', stderr: '' }) });
    assert.equal(r.ok, false);
    assert.equal(r.reason, CHECK_REASON.FAIL_NO_RELEASES);
  });

  test('FAIL_NO_RELEASES when only non-release refs exist (junk tags are data, not versions)', () => {
    const r = checkLatestVersion({ spawn: okSpawn('nightly', 'v1.2', '1.2.3', 'v1.2.3<script>') });
    assert.equal(r.ok, false);
    assert.equal(r.reason, CHECK_REASON.FAIL_NO_RELEASES);
  });

  test('the latest channel skips prerelease tags', () => {
    const r = checkLatestVersion({ spawn: okSpawn('v1.39.0', 'v1.40.0-rc.1') });
    assert.deepEqual(r, { ok: true, version: '1.39.0', reason: CHECK_REASON.OK });
  });
});

describe('pickLatestVersion / compareVersions: release tag ordering', () => {
  const { pickLatestVersion, compareVersions } = require(
    path.join(ROOT, 'msd-core', 'bin', 'check-latest-version.cjs'),
  );

  test('numeric, not lexical: 1.10.0 beats 1.9.0', () => {
    assert.equal(pickLatestVersion(lsRemote('v1.9.0', 'v1.10.0')), '1.10.0');
  });

  test('only refs/tags/v* refs count (branches named like versions are ignored)', () => {
    const out = `${'a'.repeat(40)}\trefs/heads/v9.9.9\n${'b'.repeat(40)}\trefs/tags/v1.0.0\n`;
    assert.equal(pickLatestVersion(out), '1.0.0');
  });

  test('next channel: rc.10 beats rc.2 and the final release beats its prereleases', () => {
    assert.equal(pickLatestVersion(lsRemote('v2.0.0-rc.2', 'v2.0.0-rc.10'), 'next'), '2.0.0-rc.10');
    assert.equal(pickLatestVersion(lsRemote('v2.0.0-rc.10', 'v2.0.0'), 'next'), '2.0.0');
    assert.equal(pickLatestVersion(lsRemote('v1.9.0', 'v2.0.0-rc.1'), 'next'), '2.0.0-rc.1');
  });

  test('compareVersions orders prereleases below their release', () => {
    assert.equal(compareVersions('2.0.0', '2.0.0-rc.1'), 1);
    assert.equal(compareVersions('2.0.0-rc.1', '2.0.0'), -1);
    assert.equal(compareVersions('2.0.0-rc.1', '2.0.0-rc.1'), 0);
    assert.equal(compareVersions('2.0.0-alpha', '2.0.0-beta'), -1);
  });
});

describe('Issue #815: --next channel support', () => {
  const { buildLsRemoteArgs, resolveTag, ALLOWED_TAGS } = require(
    path.join(ROOT, 'msd-core', 'bin', 'check-latest-version.cjs'),
  );

  test('ALLOWED_TAGS is the sanctioned channel allowlist (latest, next)', () => {
    assert.deepEqual([...ALLOWED_TAGS].sort(), ['latest', 'next']);
  });

  test('buildLsRemoteArgs lists only tag refs of the baked repository', () => {
    assert.deepEqual(buildLsRemoteArgs(), ['ls-remote', '--tags', '--refs', 'https://git.golem15.com/golem15/msd-core.git']);
  });

  test('resolveTag defaults to latest when no --tag flag', () => {
    assert.equal(resolveTag(['--json']), 'latest');
  });

  test('resolveTag reads --tag next', () => {
    assert.equal(resolveTag(['--json', '--tag', 'next']), 'next');
  });

  test('resolveTag rejects an unknown tag (typo guard)', () => {
    assert.throws(() => resolveTag(['--tag', 'nightly']), /invalid --tag 'nightly'/);
  });

  test('resolveTag rejects --tag with no value', () => {
    assert.throws(() => resolveTag(['--tag']), /invalid --tag ''/);
  });

  test('checkLatestVersion accepts an RC under the next channel', () => {
    const r = checkLatestVersion({ tag: 'next', spawn: okSpawn('v1.3.1', 'v1.4.0-rc.1') });
    assert.deepEqual(r, { ok: true, version: '1.4.0-rc.1', reason: CHECK_REASON.OK });
  });

  test('checkLatestVersion rejects an out-of-allowlist channel even with an injected spawn', () => {
    assert.throws(
      () => checkLatestVersion({ tag: 'nightly', spawn: okSpawn('v9.9.9') }),
      /invalid channel 'nightly'/,
    );
  });

  test('resolveTag handles the --tag=next equals form', () => {
    assert.equal(resolveTag(['--json', '--tag=next']), 'next');
  });

  test('resolveTag rejects an unknown --tag=value equals form (no silent fallback)', () => {
    assert.throws(() => resolveTag(['--tag=nightly']), /invalid --tag 'nightly'/);
  });
});
  });
}


// ────────────────────────────────────────────────────────────────────────
// Folded from tests/bug-378-update-check-scoped-name.test.cjs — consolidation epic #1969 (B5 #1974)
// ────────────────────────────────────────────────────────────────────────
{
  const { describe: __foldDescribe } = require('node:test');
  __foldDescribe("folded:bug-378-update-check-scoped-name (consolidation epic #1969 B5 #1974)", () => {
/**
 * Regression test for #378 / #498: the SessionStart update worker must end up
 * querying the SCOPED package name (@golem15/msd-core) when it asks
 * npm for the latest version.
 *
 * Background (#378): the worker once hardcoded the unscoped 'msd-core',
 * which 404s from the registry, leaving update_available permanently false.
 *
 * Original #378 fix derived the name from `require('../package.json').name`.
 * That is broken at runtime (#498): no package.json in the installed tree
 * carries a `.name`. It used to resolve to the synthetic `{"type":"commonjs"}`
 * marker MSD wrote at the config root, so post-install the worker queried
 * `npm view undefined version` → latest stayed null → update_available
 * permanently false; since #2544 MSD writes no marker there at all, so the
 * require would now fail to resolve outright. The old structural test passed
 * only because it grepped the DEV tree, where package.json still has a name.
 *
 * New contract (#498): the worker no longer resolves the package name itself.
 * It delegates the latest-version lookup to check-latest-version.cjs's
 * `checkLatestVersion()`, whose `PACKAGE_NAME` is sourced from the baked Package
 * Identity seam (`msd-core/bin/lib/package-identity.cjs`). The seam's value
 * is a build-time constant, correct in every install layout, so the
 * undefined-at-runtime failure cannot recur. This test locks that contract:
 *
 *   1. Structural: worker must NOT contain the bare unscoped literal.
 *   2. Structural: worker must NOT use `require(...package.json...).name`
 *      (the runtime-broken path).
 *   3. Structural: worker delegates to check-latest-version's
 *      `checkLatestVersion` rather than calling `npm view` itself.
 *   4. Single-source: check-latest-version's PACKAGE_NAME === the seam's
 *      packageName === the scoped '@golem15/msd-core'.
 *
 * Source-grep policy: this test reads hook source via readFileSync. Since
 * #3545 lint-no-source-grep also covers hooks/, not just bin/lib/msd-core.
 * The behavior (correct name → no E404) only manifests at runtime
 * against the live registry; structural assertions are the minimum-cost
 * contract for the worker, the same rationale #378 carried. See the
 * site-scoped `allow-test-rule` marker directly above the readFileSync()
 * call in workerCodeOnly() below.
 */

'use strict';

const { test, describe } = require('node:test');
const assert = require('node:assert/strict');
const fs = require('fs');
const path = require('path');

const WORKER_PATH = path.join(__dirname, '..', 'hooks', 'msd-check-update-worker.js');
const PKG_PATH = path.join(__dirname, '..', 'package.json');
const SEAM = require('../msd-core/bin/lib/package-identity.cjs');
const { PACKAGE_NAME } = require('../msd-core/bin/check-latest-version.cjs');

function workerCodeOnly() {
  // allow-test-rule: structural-regression-guard (see #378) — the behavior
  // being tested (correct scoped package name → no E404) only manifests at
  // runtime against the live npm registry, which CI does not call; the
  // stripped-source text is the minimum-cost contract for this worker
  // (#3545 widening brings hooks/ into the rule's scope)
  const src = fs.readFileSync(WORKER_PATH, 'utf8');
  return src
    // eslint-disable-next-line local/no-unbounded-quantifier -- parses this repo's own bounded hooks source, not adversarial input
    .replace(/\/\*[\s\S]*?\*\//g, '')
    // eslint-disable-next-line local/no-unbounded-quantifier -- parses this repo's own bounded hooks source, not adversarial input
    .replace(/(^|[^:])\/\/[^\r\n]*/g, '$1');
}

describe('bug #378 / #498: update worker queries the scoped name via the seam', () => {
  test('worker file exists', () => {
    assert.ok(fs.existsSync(WORKER_PATH), `worker not found at ${WORKER_PATH}`);
  });

  test('package.json name is the scoped @golem15/msd-core', () => {
    const pkg = JSON.parse(fs.readFileSync(PKG_PATH, 'utf8'));
    assert.equal(pkg.name, '@golem15/msd-core');
  });

  test('worker does NOT hardcode the unscoped msd-core as a string literal', () => {
    assert.doesNotMatch(
      workerCodeOnly(),
      /['"]msd-core['"]/,
      "Worker must not pass the unscoped 'msd-core' to npm — it 404s.",
    );
  });

  test('worker does NOT resolve the name via require(package.json).name (broken at runtime)', () => {
    assert.doesNotMatch(
      workerCodeOnly(),
      /require\s*\(\s*['"][^'"]*package\.json['"]\s*\)\s*\.name/,
      [
        'require(package.json).name never yields a name in the installed tree —',
        'MSD stages only {"type":"commonjs"} markers, and since #2544 none at the',
        'config root. The worker must delegate to checkLatestVersion(), which',
        'sources the name from the baked seam.',
      ].join(' '),
    );
  });

  test('worker delegates the latest-version lookup to checkLatestVersion', () => {
    const code = workerCodeOnly();
    assert.match(
      code,
      /check-latest-version/,
      'Worker must require check-latest-version.cjs and call checkLatestVersion().',
    );
    assert.match(code, /checkLatestVersion\s*\(/);
  });

  test('check-latest-version PACKAGE_NAME is single-sourced from the seam', () => {
    assert.equal(PACKAGE_NAME, SEAM.packageName);
    assert.equal(SEAM.packageName, '@golem15/msd-core');
  });
});
  });
}


// ────────────────────────────────────────────────────────────────────────
// Folded from tests/bug-2784-update-cache-clear-path.test.cjs — consolidation epic #1969 (B5 #1974)
// ────────────────────────────────────────────────────────────────────────
{
  const { describe: __foldDescribe } = require('node:test');
  __foldDescribe("folded:bug-2784-update-cache-clear-path (consolidation epic #1969 B5 #1974)", () => {
// Reads hook .js or bin/install.js source to assert structural invariants
// (search array order, function wiring, path constants) that cannot be
// verified by observing runtime outputs alone. Per CONTRIBUTING.md exception
// matrix. See the site-scoped `allow-test-rule` marker directly above the
// readFileSync() call below (#3545 widening brings hooks/ into scope).

/**
 * Regression test for bug #2784
 *
 * /msd-update cache-clear step only cleared per-runtime cache paths
 * (e.g. ~/.claude/cache/msd-update-check.json) but the SessionStart hook
 * (hooks/msd-check-update.js) writes to the shared tool-agnostic path
 * ~/.cache/msd/msd-update-check.json. After a successful update, the statusline
 * kept showing the stale "⬆ /msd-update" indicator because the actual cache
 * file was never deleted.
 *
 * Fix: add `rm -f "$HOME/.cache/msd/msd-update-check.json"` to the
 * run_update step's cache-clear block in msd-core/workflows/update.md.
 */

'use strict';

const { describe, test } = require('node:test');
const assert = require('node:assert/strict');
const fs = require('node:fs');
const path = require('node:path');

const REPO_ROOT = path.join(__dirname, '..');
const UPDATE_WORKFLOW = path.join(
  REPO_ROOT,
  'msd-core',
  'workflows',
  'update.md'
);
const CHECK_UPDATE_HOOK = path.join(REPO_ROOT, 'hooks', 'msd-check-update.js');

describe('bug-2784: update.md cache-clear covers shared cache path', () => {
  test('msd-check-update.js hook constructs cache dir from .cache and msd path segments', () => {
    // allow-test-rule: structural-regression-guard (see #2784) — asserts
    // the path.join() segment structure that cannot be verified by
    // observing runtime outputs alone (#3545)
    const hookContent = fs.readFileSync(CHECK_UPDATE_HOOK, 'utf-8');
    // Parse the path.join() call structurally rather than text-grepping.
    // eslint-disable-next-line local/no-unbounded-quantifier -- parses this repo's own bounded hooks/msd-check-update.js source, not adversarial input
    const m = hookContent.match(/const cacheDir\s*=\s*path\.join\(([^)]+)\)/);
    assert.ok(
      m !== null,
      'hook must assign cacheDir via path.join() with explicit path segments'
    );
    const segments = m[1].split(',').map((a) => a.trim().replace(/^['"]|['"]$/g, ''));
    assert.ok(
      segments.includes('.cache'),
      `hook cacheDir path.join() must include '.cache' segment; got: ${JSON.stringify(segments)}`
    );
    assert.ok(
      segments.includes('msd'),
      `hook cacheDir path.join() must include 'msd' segment; got: ${JSON.stringify(segments)}`
    );
  });

  test('update.md run_update bash commands include rm for shared msd cache file', () => {
    const workflowContent = fs.readFileSync(UPDATE_WORKFLOW, 'utf-8');
    // Parse the step block structurally, then extract only bash fenced code lines.
    // eslint-disable-next-line local/no-unbounded-quantifier -- parses this repo's own workflow .md content, fixed-size author-controlled content
    const stepMatch = workflowContent.match(/<step name="run_update">[\s\S]*?<\/step>/);
    assert.ok(stepMatch, 'update.md must have a <step name="run_update"> block');
    const stepContent = stepMatch[0];

    const bashLines = [];
    const stepLines = stepContent.split(/\r?\n/);
    for (const block of scanFencedBlocks(stepLines)) {
      if (block.closeLineIdx === -1) continue;
      const info = (block.infoString || '').trim();
      if (info !== 'bash' && info !== 'sh') continue;
      for (const line of stepLines.slice(block.openLineIdx + 1, block.closeLineIdx)) {
        const trimmed = line.trim();
        if (trimmed) bashLines.push(trimmed);
      }
    }

    const sharedCacheClearCmds = bashLines.filter(
      (line) => /^rm\b/.test(line) && line.includes('.cache/msd/msd-update-check') && line.includes('*.json')
    );
    assert.ok(
      sharedCacheClearCmds.length > 0,
      [
        'run_update step bash blocks must include an `rm` command targeting .cache/msd/msd-update-check*.json (glob form clearing legacy + per-package variants).',
        `Bash lines found: ${JSON.stringify(bashLines)}`,
      ].join('\n')
    );
    const hasHomeExpansion = sharedCacheClearCmds.some(
      (line) => line.includes('$HOME') || line.includes('~/')
    );
    assert.ok(
      hasHomeExpansion,
      `shared cache rm command must use $HOME or ~/ expansion; found: ${JSON.stringify(sharedCacheClearCmds)}`
    );
  });
});
  });
}


// ────────────────────────────────────────────────────────────────────────
// Folded from tests/feat-2795-update-banner.test.cjs — consolidation epic #1969 (B5 #1974)
// ────────────────────────────────────────────────────────────────────────
{
  const { describe: __foldDescribe } = require('node:test');
  __foldDescribe("folded:feat-2795-update-banner (consolidation epic #1969 B5 #1974)", () => {
/**
 * Tests for msd-update-banner.js (#2795).
 *
 * The banner hook is an opt-in SessionStart consumer of the update cache that
 * msd-check-update-worker.js writes. When a user declines MSD's statusline,
 * install.js may register this hook so update availability still surfaces in
 * runtimes that use a non-MSD statusline.
 *
 * Tests follow the typed-IR convention (CONTRIBUTING.md "Prohibited: Raw Text
 * Matching on Test Outputs"): assert on parsed JSON envelopes, not on raw
 * stdout substrings.
 */

'use strict';

const { test, describe } = require('node:test');
const assert = require('node:assert/strict');
const fs = require('node:fs');
const os = require('node:os');
const path = require('node:path');
const { runHook: seamRunHook } = require('./helpers/process-seam.cjs');
const { cleanup } = require('./helpers.cjs');

const HOOK_PATH = path.join(__dirname, '..', 'hooks', 'msd-update-banner.js');
const {
  buildBannerOutput,
  shouldSuppressFailureWarning,
  RATE_LIMIT_SECONDS,
} = require('../hooks/msd-update-banner.js');
const { updateCacheFileName } = require('../msd-core/bin/lib/package-identity.cjs');

// ─── Pure function: buildBannerOutput ───────────────────────────────────────

describe('buildBannerOutput', () => {
  test('returns null when cache is missing', () => {
    const out = buildBannerOutput({
      cache: null,
      parseError: false,
      suppressFailureWarning: false,
    });
    assert.equal(out, null);
  });

  test('returns null when update_available is false', () => {
    const out = buildBannerOutput({
      cache: { update_available: false, installed: '1.40.0', latest: '1.40.0' },
      parseError: false,
      suppressFailureWarning: false,
    });
    assert.equal(out, null);
  });

  test('returns banner envelope when update_available is true', () => {
    const out = buildBannerOutput({
      cache: { update_available: true, installed: '1.39.0', latest: '1.40.0', package_name: '@golem15/msd-core' },
      parseError: false,
      suppressFailureWarning: false,
    });
    assert.ok(out, 'expected banner envelope');
    assert.equal(typeof out.systemMessage, 'string');
    assert.ok(
      out.systemMessage.includes('1.39.0'),
      'banner should name installed version'
    );
    assert.ok(
      out.systemMessage.includes('1.40.0'),
      'banner should name latest version'
    );
    assert.ok(
      out.systemMessage.includes('/msd:update'),
      'banner should reference /msd:update command'
    );
  });

  test('returns failure diagnostic on parseError when not suppressed', () => {
    const out = buildBannerOutput({
      cache: null,
      parseError: true,
      suppressFailureWarning: false,
    });
    assert.ok(out, 'expected diagnostic envelope');
    assert.equal(typeof out.systemMessage, 'string');
    assert.ok(
      /check failed/i.test(out.systemMessage),
      'diagnostic should describe a failed check'
    );
  });

  test('returns null on parseError when suppressed by rate limit', () => {
    const out = buildBannerOutput({
      cache: null,
      parseError: true,
      suppressFailureWarning: true,
    });
    assert.equal(out, null);
  });

  test('falls back to "unknown" when installed/latest missing', () => {
    const out = buildBannerOutput({
      cache: { update_available: true, package_name: '@golem15/msd-core' },
      parseError: false,
      suppressFailureWarning: false,
    });
    assert.ok(out);
    assert.ok(
      out.systemMessage.includes('unknown'),
      'banner should degrade gracefully when versions are absent'
    );
  });
});

// ─── Pure function: shouldSuppressFailureWarning ────────────────────────────

describe('shouldSuppressFailureWarning', () => {
  function tmpDir() {
    return fs.mkdtempSync(path.join(os.tmpdir(), 'msd-banner-supp-'));
  }

  test('returns false when sentinel file is missing', () => {
    const dir = tmpDir();
    try {
      const result = shouldSuppressFailureWarning(
        path.join(dir, 'no-such-file'),
        100
      );
      assert.equal(result, false);
    } finally {
      cleanup(dir);
    }
  });

  test('returns true within rate-limit window', () => {
    const dir = tmpDir();
    try {
      const f = path.join(dir, 'sentinel');
      fs.writeFileSync(f, '1000');
      const result = shouldSuppressFailureWarning(f, 1000 + RATE_LIMIT_SECONDS - 1);
      assert.equal(result, true);
    } finally {
      cleanup(dir);
    }
  });

  test('returns false outside rate-limit window', () => {
    const dir = tmpDir();
    try {
      const f = path.join(dir, 'sentinel');
      fs.writeFileSync(f, '1000');
      const result = shouldSuppressFailureWarning(f, 1000 + RATE_LIMIT_SECONDS + 1);
      assert.equal(result, false);
    } finally {
      cleanup(dir);
    }
  });

  test('returns false when sentinel content is non-numeric', () => {
    const dir = tmpDir();
    try {
      const f = path.join(dir, 'sentinel');
      fs.writeFileSync(f, 'garbage-not-a-number');
      const result = shouldSuppressFailureWarning(f, 100);
      assert.equal(result, false);
    } finally {
      cleanup(dir);
    }
  });
});

// ─── End-to-end: spawn the hook against fixture cache states ────────────────

describe('msd-update-banner.js end-to-end', () => {
  function setupHome() {
    const home = fs.mkdtempSync(path.join(os.tmpdir(), 'msd-banner-home-'));
    fs.mkdirSync(path.join(home, '.cache', 'msd'), { recursive: true });
    return home;
  }

  function runHook(home) {
    // 10000ms: previously UNBOUNDED (no `timeout` option passed to
    // spawnSync). msd-update-banner.js only reads a small cache file from
    // disk and prints JSON — no subprocess/network work — so 10s is
    // generous headroom over its sub-second worst case even on a
    // contended CI runner.
    const r = seamRunHook(HOOK_PATH, [], {
      env: { ...process.env, HOME: home, USERPROFILE: home },
      timeoutMs: UPDATE_BANNER_HOOK_TIMEOUT_MS,
    });
    return { status: r.exitCode, stdout: r.stdout, stderr: r.stderr };
  }

  function writeCache(home, contents) {
    fs.writeFileSync(
      path.join(home, '.cache', 'msd', updateCacheFileName),
      typeof contents === 'string' ? contents : JSON.stringify(contents)
    );
  }

  test('exits 0 with empty stdout when cache file missing', () => {
    const home = setupHome();
    try {
      const r = runHook(home);
      assert.equal(r.status, 0, `expected exit 0, got ${r.status} stderr=${r.stderr}`);
      assert.equal(r.stdout.trim(), '');
    } finally {
      cleanup(home);
    }
  });

  test('emits valid SessionStart JSON when update_available=true', () => {
    const home = setupHome();
    try {
      writeCache(home, {
        update_available: true,
        installed: '1.39.0',
        latest: '1.40.0',
        package_name: '@golem15/msd-core',
      });
      const r = runHook(home);
      assert.equal(r.status, 0);
      const parsed = JSON.parse(r.stdout);
      assert.equal(typeof parsed.systemMessage, 'string');
      assert.ok(parsed.systemMessage.includes('1.40.0'));
      assert.ok(parsed.systemMessage.includes('/msd:update'));
    } finally {
      cleanup(home);
    }
  });

  test('exits silent when update_available=false', () => {
    const home = setupHome();
    try {
      writeCache(home, {
        update_available: false,
        installed: '1.40.0',
        latest: '1.40.0',
      });
      const r = runHook(home);
      assert.equal(r.status, 0);
      assert.equal(r.stdout.trim(), '');
    } finally {
      cleanup(home);
    }
  });

  test('emits failure diagnostic when cache JSON is malformed', () => {
    const home = setupHome();
    try {
      writeCache(home, 'not json {{{{');
      const r = runHook(home);
      assert.equal(r.status, 0);
      const parsed = JSON.parse(r.stdout);
      assert.equal(typeof parsed.systemMessage, 'string');
      assert.ok(/check failed/i.test(parsed.systemMessage));
    } finally {
      cleanup(home);
    }
  });

  test('suppresses repeat failure diagnostic within 24h via sentinel', () => {
    const home = setupHome();
    try {
      writeCache(home, 'not json');
      const r1 = runHook(home);
      assert.equal(
        r1.status,
        0,
        `expected exit 0, got ${r1.status} stderr=${r1.stderr}`
      );
      const parsed1 = JSON.parse(r1.stdout);
      assert.ok(/check failed/i.test(parsed1.systemMessage));

      // Sentinel should now exist so the next run is silent
      const sentinel = path.join(home, '.cache', 'msd', 'banner-failure-warned-at');
      assert.ok(fs.existsSync(sentinel), 'first run must record the warning sentinel');

      const r2 = runHook(home);
      assert.equal(r2.status, 0);
      assert.equal(
        r2.stdout.trim(),
        '',
        'subsequent run within rate-limit window must stay silent'
      );
    } finally {
      cleanup(home);
    }
  });

  test('handles cache present but update_available field absent (older cache schema)', () => {
    const home = setupHome();
    try {
      writeCache(home, { installed: '1.40.0', latest: '1.40.0' });
      const r = runHook(home);
      assert.equal(r.status, 0);
      assert.equal(r.stdout.trim(), '');
    } finally {
      cleanup(home);
    }
  });
});

// ─── #3582: cold tree (no msd-core/bin/lib/*.cjs) — degrade, not crash ─────
//
// msd-core/bin/lib/package-identity.cjs is a tsc build artifact (ADR-457),
// gitignored and absent on a raw plugin-marketplace / git-clone install that
// never ran `npm run build:lib`. This is an opt-in SessionStart hook — a
// build failure must degrade (PACKAGE_NAME stays null), not crash session
// start. The DEGRADED VERDICT this locks: buildBannerOutput's own lineage
// guard (`!cache.package_name || cache.package_name !== PACKAGE_NAME`)
// unconditionally distrusts ANY cache once PACKAGE_NAME is null, so even a
// cache written by a healthy worker (real package_name, update_available:
// true) must be suppressed rather than surfaced — the hook stays SILENT
// (exit 0, empty stdout), never a crash and never a stale/wrong banner.
// Simulated hermetically via tests/helpers/cold-runtime-lib-fixture.cjs — the
// REAL msd-core/bin/lib/ is never touched.
describe('msd-update-banner.js: #3582 cold tree — degrades to silent, never crashes', () => {
  const { buildColdInstallTree } = require('./helpers/cold-runtime-lib-fixture.cjs');

  test('missing compiled runtime library -> exits 0 with empty stdout even for an otherwise-valid update-available cache', (t) => {
    const cold = buildColdInstallTree();
    t.after(cold.cleanup);
    const home = fs.mkdtempSync(path.join(os.tmpdir(), 'msd-banner-cold-home-'));
    t.after(() => cleanup(home));
    fs.mkdirSync(path.join(home, '.cache', 'msd'), { recursive: true });
    // The generic fallback filename the hook's own #3582 degrade uses when
    // package-identity.cjs cannot be built (mirrors msd-check-update.js's
    // identical fallback literal) — this is the SAME cache path a degraded
    // worker would also have written to.
    fs.writeFileSync(
      path.join(home, '.cache', 'msd', 'msd-update-check.json'),
      JSON.stringify({
        update_available: true,
        installed: '1.39.0',
        latest: '1.40.0',
        package_name: '@golem15/msd-core',
      }),
    );

    const r = seamRunHook(path.join(cold.hooksDir, 'msd-update-banner.js'), [], {
      env: { ...process.env, HOME: home, USERPROFILE: home },
      timeoutMs: UPDATE_BANNER_HOOK_TIMEOUT_MS,
    });

    assert.equal(r.exitCode, 0, `hook must exit 0 on a build failure; stderr: ${r.stderr}`);
    assert.equal(
      r.stdout.trim(),
      '',
      'a cold tree must silently suppress the banner (PACKAGE_NAME degrades to null, ' +
        'so the lineage guard distrusts every cache) rather than crash or print stale content',
    );
  });
});

// ─── Install.js wiring: prompt + SessionStart entry registration ────────────
//
// These tests load bin/install.js as a module via MSD_TEST_MODE and assert on
// pure exported helpers. The shape mirrors how runtime-prompt-builder /
// statusline tests interact with install.js.

describe('install.js update-banner wiring', () => {
  process.env.MSD_TEST_MODE = '1';
  // Re-require fresh so test-mode exports are populated.
  const installPath = path.join(__dirname, '..', 'bin', 'install.js');
  delete require.cache[installPath];
  const installExports = require(installPath);

  test('exports buildUpdateBannerPromptText for structural prompt assertions', () => {
    assert.equal(
      typeof installExports.buildUpdateBannerPromptText,
      'function',
      'install.js must export buildUpdateBannerPromptText so tests can assert without grepping source'
    );
    const text = installExports.buildUpdateBannerPromptText();
    assert.equal(typeof text, 'string');
    assert.ok(text.length > 0);
    // Strip ANSI color escapes before structural assertions — the choice
    // digits are wrapped in color codes so word-boundary regex against the
    // raw text would miss them.
    // eslint-disable-next-line no-control-regex -- \x1b (ESC) is the required leading byte of ANSI SGR color sequences; matching it is the purpose of stripping ANSI codes from captured CLI/console output
    const stripped = text.replace(/\x1b\[[0-9;]*m/g, '');
    // Prompt must offer at least two choices (default + opt-in).
    assert.match(stripped, /\b1\b/);
    assert.match(stripped, /\b2\b/);
  });

  test('parseUpdateBannerInput defaults to false on empty / "1"', () => {
    assert.equal(typeof installExports.parseUpdateBannerInput, 'function');
    assert.equal(installExports.parseUpdateBannerInput(''), false);
    assert.equal(installExports.parseUpdateBannerInput('  '), false);
    assert.equal(installExports.parseUpdateBannerInput('1'), false);
  });

  test('parseUpdateBannerInput returns true on "2"', () => {
    assert.equal(installExports.parseUpdateBannerInput('2'), true);
    assert.equal(installExports.parseUpdateBannerInput('2 '), true);
  });

  test('parseUpdateBannerInput accepts "y" / "yes" affirmative shortcuts', () => {
    assert.equal(installExports.parseUpdateBannerInput('y'), true);
    assert.equal(installExports.parseUpdateBannerInput('Y'), true);
    assert.equal(installExports.parseUpdateBannerInput('yes'), true);
    assert.equal(installExports.parseUpdateBannerInput('YES'), true);
  });

  test('buildUpdateBannerHookEntry produces a SessionStart hook entry', () => {
    assert.equal(typeof installExports.buildUpdateBannerHookEntry, 'function');
    const entry = installExports.buildUpdateBannerHookEntry(
      '"/usr/local/bin/node" "/home/u/.claude/hooks/msd-update-banner.js"'
    );
    assert.ok(entry, 'expected hook entry object');
    assert.ok(Array.isArray(entry.hooks), 'entry.hooks must be an array');
    assert.equal(entry.hooks.length, 1);
    assert.equal(entry.hooks[0].type, 'command');
    assert.ok(
      entry.hooks[0].command.includes('msd-update-banner.js'),
      'command must reference the banner hook'
    );
  });

  test('buildUpdateBannerHookEntry returns null on null command', () => {
    assert.equal(installExports.buildUpdateBannerHookEntry(null), null);
    assert.equal(installExports.buildUpdateBannerHookEntry(''), null);
  });
});
  });
}