@1aboveio/skills 1.0.8 → 1.0.10

package/skills/core/polish-cn/scripts/polish-cn.test.mjs

@@ -1,244 +0,0 @@
-import assert from 'node:assert/strict';
-import { spawnSync } from 'node:child_process';
-import { mkdtempSync, readFileSync, writeFileSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { dirname, join } from 'node:path';
-import { fileURLToPath } from 'node:url';
-import test from 'node:test';
-
-const script = join(dirname(fileURLToPath(import.meta.url)), 'polish_blocks.py');
-
-const REPORT = `---
-title: 月报
----
-# 2026 年 10 月报告
-
-本期授权尝试 4,946 笔，授权成功率 45.79%。0 笔销售未能匹配授权，覆盖六个月。
-详见 [方法说明](method.zh.md)。
-
-| 指标 | 值 |
-|---|---|
-| 授权尝试 | 4,946 |
-
-- 授权拒绝 1,111 笔，\`C1\` 为高风险。
-- 疑似欺诈响应不是已确认损失。
-
-\`\`\`python
-print("本期授权尝试 4,946 笔")
-\`\`\`
-`;
-
-function polish(...args) {
-  return spawnSync('python3', [script, ...args], { encoding: 'utf8' });
-}
-
-// Writes a document, extracts it, and returns helpers. `propose` keeps every block it does not
-// rewrite, so the proposal covers all blocks unless `partial` is set.
-function setup(content = REPORT) {
-  const dir = mkdtempSync(join(tmpdir(), 'polish-cn-test-'));
-  const doc = join(dir, 'report.zh.md');
-  writeFileSync(doc, content);
-  const blocksPath = join(dir, 'blocks.json');
-  const result = polish('extract', doc, '--out', blocksPath);
-  assert.equal(result.status, 0, result.stderr);
-  const blocks = JSON.parse(readFileSync(blocksPath, 'utf8')).blocks;
-  const byText = (needle) => blocks.find((block) => block.text.includes(needle));
-  const propose = (entries, { partial = false } = {}) => {
-    const path = join(dir, `proposals-${Math.random().toString(36).slice(2)}.json`);
-    const rewritten = {};
-    for (const [needle, text] of entries) {
-      const block = byText(needle);
-      rewritten[block.id] = { sha: block.sha, text };
-    }
-    const kept = partial ? {} : Object.fromEntries(
-      blocks.filter((block) => !rewritten[block.id]).map((block) => [block.id, 'already natural']));
-    writeFileSync(path, JSON.stringify({ blocks: rewritten, kept, notes: ['note kept'] }));
-    return path;
-  };
-  return { dir, doc, blocksPath, blocks, byText, propose };
-}
-
-test('extract returns prose only: front matter, tables and code fences are never blocks', () => {
-  const { blocks } = setup();
-  assert.deepEqual(blocks.map((block) => block.kind), ['heading', 'paragraph', 'list-item', 'list-item']);
-  assert.ok(blocks.every((block) => !block.text.includes('|') && !block.text.includes('print(')));
-  assert.equal(blocks[0].prefix, '# ');
-  assert.deepEqual(blocks[1].lines, [6, 7]);
-});
-
-test('extract skips HTML comments and blocks, indented code, pipe tables without edge pipes, and link definitions', () => {
-  const { blocks } = setup([
-    '正文一。', '', '<!--', '注释 99', '-->', '', '<div>', '内嵌 HTML 7', '</div>', '',
-    '    indented_code(3)', '', '指标 | 值', '--- | ---', '尝试 | 12', '', '[ref]: https://example.com/x', '', '正文二。', '',
-  ].join('\n'));
-  assert.deepEqual(blocks.map((block) => block.text), ['正文一。', '正文二。']);
-});
-
-test('check accepts a rewrite that keeps every protected token, even in a new order', () => {
-  const { blocksPath, propose } = setup();
-  const proposals = propose([[
-    '本期授权尝试',
-    '本期授权成功率为 45.79%，授权尝试共 4,946 笔。0 笔销售未能匹配授权，覆盖六个月。详见 [方法](method.zh.md)。',
-  ]]);
-  const result = polish('check', blocksPath, proposals);
-  assert.equal(result.status, 0, result.stdout + result.stderr);
-  assert.equal(JSON.parse(result.stdout).accepted.length, 1);
-});
-
-test('check fails when a block is neither rewritten nor kept, or is both, unless --partial', () => {
-  const { dir, blocksPath, byText, propose } = setup();
-  const partial = propose([['疑似欺诈', '疑似欺诈响应并非已确认的损失。']], { partial: true });
-  const strict = polish('check', blocksPath, partial);
-  assert.equal(strict.status, 1);
-  assert.equal(JSON.parse(strict.stdout).unreviewed.length, 3);
-  assert.equal(polish('check', blocksPath, partial, '--partial').status, 0);
-
-  const block = byText('疑似欺诈');
-  const both = JSON.parse(readFileSync(propose([['疑似欺诈', '疑似欺诈响应并非已确认的损失。']]), 'utf8'));
-  both.kept[block.id] = 'also kept';
-  const bothPath = join(dir, 'both.json');
-  writeFileSync(bothPath, JSON.stringify(both));
-  const result = polish('check', blocksPath, bothPath);
-  assert.equal(result.status, 1);
-  assert.deepEqual(JSON.parse(result.stdout).partitionProblems, [block.id]);
-});
-
-test('check rejects changed, added or dropped numbers, signs, currency, links, images, code and protected terms', () => {
-  const { blocksPath, propose } = setup();
-  const cases = [
-    ['本期授权尝试', '本期授权尝试 4,964 笔，授权成功率 45.79%。0 笔销售未能匹配授权，覆盖六个月。详见 [方法说明](method.zh.md)。', /number '4,946'/],
-    ['本期授权尝试', '本期授权尝试 4,946 笔，授权成功率 45.79%。0 笔销售未能匹配授权，覆盖 6 个月。详见 [方法说明](method.zh.md)。', /added number '6'/],
-    ['本期授权尝试', '本期授权尝试 4,946 笔，授权成功率 45.79%。0 笔销售未能匹配授权，覆盖六个月。', /link 'method\.zh\.md'/],
-    ['本期授权尝试', '本期授权尝试 4,946 笔，授权成功率 45.79%。0 笔销售未能匹配授权，覆盖六个月。详见 ![方法说明](method.zh.md)。', /image 'method\.zh\.md'/],
-    ['本期授权尝试', '本期授权尝试 4,946 笔，授权成功率 +45.79%。0 笔销售未能匹配授权，覆盖六个月。详见 [方法说明](method.zh.md)。', /symbol '\+'/],
-    ['本期授权尝试', '本期授权尝试 4,946 笔，授权成功率 45.79%，约 ¥0。0 笔销售未能匹配授权，覆盖六个月。详见 [方法说明](method.zh.md)。', /symbol '¥'/],
-    ['授权拒绝', '拒付 1,111 笔，`C1` 为高风险。', /term '授权拒绝'/],
-    ['授权拒绝', '授权拒绝 1,111 笔，C2 为高风险。', /code '`C1`'/],
-  ];
-  for (const [needle, text, reason] of cases) {
-    const result = polish('check', blocksPath, propose([[needle, text]]), '--protect', '授权拒绝', '--protect', '拒付');
-    assert.equal(result.status, 1, `expected rejection for: ${text}`);
-    const rejected = Object.values(JSON.parse(result.stdout).rejected);
-    assert.match(rejected[0], reason);
-  }
-});
-
-test('check reads TERMINOLOGY.md as concepts: mentions are kept, Avoid terms may become preferred but never appear', () => {
-  const { dir, blocks, blocksPath, propose } = setup('本月拒绝交易 1 笔，授权拒绝 2 笔。\n\n疑似欺诈响应不是已确认损失。\n');
-  const terms = join(dir, 'TERMINOLOGY.md');
-  writeFileSync(terms, [
-    '# Terminology: test', '', '## Disputes', '',
-    '| Concept | English | 简体 | 繁體（港） | Avoid | Meaning | Status | Source |',
-    '| --- | --- | --- | --- | --- | --- | --- | --- |',
-    '| Decline | decline | 授权拒绝 | 授權拒絕 | 拒绝交易、拒付（指授权拒绝） | Issuer refusal | user, 2026-10-20 | user |',
-    '| Chargeback | chargeback | 拒付 | — | 退单（指 Visa 拒付） | Dispute | proposed | a.md |',
-    '| Piped | pipe | 甲\\|乙 | — |  | Escaped pipe | proposed | a.md |', '',
-    '## Sources', '', '| id | Source |', '| --- | --- |', '| a.md | 授权拒绝 |', '',
-  ].join('\n'));
-  const run = (needle, text) => polish('check', blocksPath, propose([[needle, text]]), '--terminology', terms);
-
-  const normalized = run('拒绝交易', '本月授权拒绝 1 笔，授权拒绝 2 笔。');
-  assert.equal(normalized.status, 0, normalized.stdout);
-  assert.deepEqual(Object.keys(JSON.parse(normalized.stdout).avoidedTermsInSource), [blocks[0].id]);
-
-  const added = run('拒绝交易', '本月拒绝交易 1 笔，拒绝交易 2 笔。');
-  assert.equal(added.status, 1);
-  assert.match(Object.values(JSON.parse(added.stdout).rejected)[0], /avoids: 拒绝交易/);
-
-  const dropped = run('拒绝交易', '本月有 1 笔，授权拒绝 2 笔。');
-  assert.equal(dropped.status, 1);
-  assert.match(Object.values(JSON.parse(dropped.stdout).rejected)[0], /concept 'c1'.*c1 = 授权拒绝/);
-
-  const scopedNote = run('疑似欺诈', '疑似欺诈响应不是已确认损失，与拒付无关。');
-  assert.equal(scopedNote.status, 1, 'adding 拒付 adds a mention of the chargeback concept');
-  assert.match(Object.values(JSON.parse(scopedNote.stdout).rejected)[0], /c2 = 拒付/);
-
-  const emDash = run('疑似欺诈', '疑似欺诈响应——不是已确认损失。');
-  assert.equal(emDash.status, 0, 'an em dash is not a protected term: — means an empty cell');
-});
-
-test('check flags ids in kept that are not blocks', () => {
-  const { dir, blocksPath, propose } = setup();
-  const proposals = JSON.parse(readFileSync(propose([]), 'utf8'));
-  proposals.kept.b999_note = 'invented';
-  const path = join(dir, 'unknown.json');
-  writeFileSync(path, JSON.stringify(proposals));
-  const result = polish('check', blocksPath, path);
-  assert.equal(result.status, 1);
-  assert.deepEqual(JSON.parse(result.stdout).partitionProblems, ['b999_note (unknown id in kept)']);
-});
-
-test('check refuses line breaks and new Markdown structure inside a proposal', () => {
-  const { blocksPath, propose } = setup();
-  for (const text of ['疑似欺诈响应\n# 新标题', '疑似欺诈响应\r# 注入', '# 疑似欺诈响应不是已确认损失。', '- 疑似欺诈响应不是已确认损失。']) {
-    const result = polish('check', blocksPath, propose([['疑似欺诈', text]]));
-    assert.equal(result.status, 1, `expected rejection for: ${JSON.stringify(text)}`);
-    assert.match(Object.values(JSON.parse(result.stdout).rejected)[0], /one line|block syntax/);
-  }
-});
-
-test('check refuses deletions, stale blocks and malformed proposal files', () => {
-  const { dir, blocksPath, byText, propose } = setup();
-  const empty = polish('check', blocksPath, propose([['疑似欺诈', '  ']]));
-  assert.equal(empty.status, 1);
-  assert.match(empty.stdout, /never deletes a block/);
-
-  const block = byText('疑似欺诈');
-  const stale = JSON.parse(readFileSync(propose([['疑似欺诈', '疑似欺诈响应并非已确认的损失。']]), 'utf8'));
-  stale.blocks[block.id].sha = 'old';
-  const stalePath = join(dir, 'stale.json');
-  writeFileSync(stalePath, JSON.stringify(stale));
-  const result = polish('check', blocksPath, stalePath);
-  assert.equal(result.status, 1);
-  assert.match(result.stdout, /does not match block sha/);
-
-  const nullPath = join(dir, 'null.json');
-  writeFileSync(nullPath, 'null');
-  const malformed = polish('check', blocksPath, nullPath);
-  assert.equal(malformed.status, 2);
-  assert.match(malformed.stderr, /needs an object/);
-});
-
-test('apply rewrites accepted prose, keeps tables and code byte-identical, and writes the record', () => {
-  const { dir, doc, blocksPath, byText, propose } = setup();
-  const proposals = propose([
-    ['疑似欺诈', '疑似欺诈响应并非已确认的损失。'],
-    ['授权拒绝', '授权拒绝共 1,111 笔，`C1` 列为高风险。'],
-  ]);
-  const record = join(dir, 'record.json');
-  const result = polish('apply', doc, blocksPath, proposals, '--reject', byText('授权拒绝').id,
-    '--record', record, '--model', 'deepseek/deepseek-flash');
-  assert.equal(result.status, 0, result.stderr);
-  const after = readFileSync(doc, 'utf8');
-  assert.match(after, /^- 疑似欺诈响应并非已确认的损失。$/m);
-  assert.match(after, /^- 授权拒绝 1,111 笔，`C1` 为高风险。$/m, 'a caller-rejected block stays unchanged');
-  assert.ok(after.includes('| 授权尝试 | 4,946 |\n'));
-  assert.ok(after.includes('print("本期授权尝试 4,946 笔")'));
-  assert.ok(after.endsWith('```\n'), 'the trailing newline survives');
-  const written = JSON.parse(readFileSync(record, 'utf8'));
-  assert.equal(written.model, 'deepseek/deepseek-flash');
-  assert.equal(written.applied.length, 1);
-  assert.deepEqual(written.notes, ['note kept']);
-});
-
-test('apply keeps CRLF line endings and leaves HTML comments intact', () => {
-  const crlf = ['正文 123。', '', '<!--', '注释 99', '-->', '', '结尾。', ''].join('\r\n');
-  const { doc, blocksPath, propose } = setup(crlf);
-  const result = polish('apply', doc, blocksPath, propose([['正文', '正文为 123。']]));
-  assert.equal(result.status, 0, result.stderr);
-  assert.equal(readFileSync(doc, 'utf8'), crlf.replace('正文 123。', '正文为 123。'));
-});
-
-test('apply refuses an incomplete proposal file and a document changed after extract', () => {
-  const { doc, blocksPath, propose } = setup();
-  const incomplete = polish('apply', doc, blocksPath, propose([['疑似欺诈', '疑似欺诈响应并非已确认的损失。']], { partial: true }));
-  assert.equal(incomplete.status, 1);
-  assert.match(incomplete.stderr, /every block needs/);
-
-  const proposals = propose([['疑似欺诈', '疑似欺诈响应并非已确认的损失。']]);
-  writeFileSync(doc, REPORT.replace('疑似欺诈响应不是', '疑似欺诈响应并不是'));
-  const result = polish('apply', doc, blocksPath, proposals);
-  assert.equal(result.status, 1);
-  assert.match(result.stderr, /changed since extract/);
-  assert.ok(readFileSync(doc, 'utf8').includes('疑似欺诈响应并不是'));
-});

package/skills/core/reg-parser/scripts/reg-parser.test.mjs

@@ -1,165 +0,0 @@
-import assert from 'node:assert/strict';
-import { spawnSync } from 'node:child_process';
-import { mkdtempSync, readdirSync, readFileSync, writeFileSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { dirname, join } from 'node:path';
-import { fileURLToPath } from 'node:url';
-import test from 'node:test';
-
-const scriptsDir = dirname(fileURLToPath(import.meta.url));
-const skillDir = join(scriptsDir, '..');
-const checklistsDir = join(skillDir, 'references', 'checklists');
-const mandatesDir = join(skillDir, 'references', 'mandates');
-
-function run(cmd, args) {
-  return spawnSync(cmd, args, { encoding: 'utf8' });
-}
-
-function py(script, ...args) {
-  return run('python3', [join(scriptsDir, script), ...args]);
-}
-
-function scratch(name, content) {
-  const path = join(mkdtempSync(join(tmpdir(), 'reg-parser-test-')), name);
-  writeFileSync(path, content);
-  return path;
-}
-
-const HEADER = `\`\`\`yaml
-name: "Test Checklist"
-regulation: "Directive 2015/849"
-version: "1.0"
-scope: "Test"
-date: "2026-10-19"
-summary:
-  categories: 1
-  total_items: 1
-  critical: 1
-  major: 0
-  minor: 0
-\`\`\`
-
-## CAT-01: Customer Due Diligence
-
-`;
-
-// The item template in SKILL.md, filled in. If the template drifts from what
-// the validator accepts, every checklist an agent builds from it fails.
-function templateItem() {
-  const skill = readFileSync(join(skillDir, 'SKILL.md'), 'utf8');
-  const block = skill.match(/Each obligation from Step 2 becomes one checklist item:\n\n```markdown\n([\s\S]*?)```/)[1];
-  return block
-    .split('\n')
-    .filter((line) => !line.includes('# source-native scheme'))
-    .map((line) => line.replace(/\s+# cat scheme$/, ''))
-    .join('\n')
-    .replace('CAT-XX-NNN', 'CAT-01-001')
-    .replace('[Requirement title]', 'Verify identity')
-    .replace('[plain-language obligation]', 'Verify customer identity before onboarding.')
-    .replace('section [ref], [regulation]', 'Art. 13(1)(a), Directive 2015/849')
-    .replace('critical | major | minor', 'critical')
-    .replace('[Concrete observable step]', 'ID checked against a reliable source')
-    .replace('[Another step]', 'Check date recorded')
-    .replace('[Document or record that proves compliance]', 'Verification log');
-}
-
-test('every bundled checklist validates', () => {
-  for (const name of readdirSync(checklistsDir).filter((f) => f.endsWith('.md'))) {
-    const result = py('validate_checklist.py', join(checklistsDir, name));
-    assert.equal(result.status, 0, `${name}\n${result.stdout}${result.stderr}`);
-  }
-});
-
-test('every bundled mandate validates', () => {
-  for (const name of readdirSync(mandatesDir).filter((f) => f.endsWith('.md'))) {
-    const result = py('validate_mandate.py', join(mandatesDir, name));
-    assert.equal(result.status, 0, `${name}\n${result.stdout}${result.stderr}`);
-  }
-});
-
-test('every bundled checklist JSON equals a fresh dump of its markdown', () => {
-  for (const name of readdirSync(checklistsDir).filter((f) => f.endsWith('.json'))) {
-    const md = join(checklistsDir, name.replace(/\.json$/, '.md'));
-    const result = py('dump_checklist.py', md);
-    assert.equal(result.status, 0, result.stderr);
-    assert.deepEqual(JSON.parse(result.stdout), JSON.parse(readFileSync(join(checklistsDir, name), 'utf8')), name);
-  }
-});
-
-test('the SKILL.md item template passes the validator', () => {
-  const result = py('validate_checklist.py', scratch('ok.md', HEADER + templateItem() + '\n'));
-  assert.equal(result.status, 0, result.stdout + result.stderr);
-});
-
-test('a checklist with colons outside the bold labels fails', () => {
-  const item = templateItem().replace(/\*\*(\w+):\*\*/g, '**$1**:');
-  const result = py('validate_checklist.py', scratch('bad.md', HEADER + item + '\n'));
-  assert.equal(result.status, 1, result.stdout);
-});
-
-test('a checklist item with empty fields fails', () => {
-  const item = `### CAT-01-001: Empty
-
-- **Requirement:**
-- **Source:**
-- **Severity:** critical
-- **Verification:**
-- **Evidence:**
-`;
-  const result = py('validate_checklist.py', scratch('empty.md', HEADER + item));
-  assert.equal(result.status, 1, result.stdout);
-  assert.match(result.stdout + result.stderr, /field Requirement is empty/);
-  assert.match(result.stdout + result.stderr, /Verification has no '- \[ \]' steps/);
-});
-
-test('a raw preprocess output fails mandate validation', () => {
-  const source = scratch('src.md', 'Section 1 The licensee must keep records.\n```\nfenced\n```\n');
-  const out = join(dirname(source), 'mandate.md');
-  const pre = run('bash', [join(scriptsDir, 'preprocess-mandate.sh'), source, '--source', 'Test', '--out', out]);
-  assert.equal(pre.status, 0, pre.stderr);
-  const result = py('validate_mandate.py', out);
-  assert.equal(result.status, 1, result.stdout);
-  assert.match(result.stdout + result.stderr, /Status: RAW/);
-});
-
-test('preprocess refuses to overwrite a finished mandate', () => {
-  const finished = join(mandatesDir, 'visa-vars.md');
-  const copy = scratch('visa-vars.md', readFileSync(finished, 'utf8'));
-  const source = scratch('src.md', 'Section 1 must.\n');
-  const pre = run('bash', [join(scriptsDir, 'preprocess-mandate.sh'), source, '--source', 'Test', '--out', copy]);
-  assert.equal(pre.status, 1, pre.stdout);
-  assert.equal(readFileSync(copy, 'utf8'), readFileSync(finished, 'utf8'));
-});
-
-test('a mandate section with only prose or a hollow entry fails', () => {
-  const sections = (body) => `# Test — Contract Obligation Extract
-
-> **Source:** Test Act 2026
-
-## Mandatory Downstream Agreement Clauses
-${body}
-
-## Prohibited Conduct
-_None_
-
-## Thresholds & Triggers
-_None_
-
-## Audit & Oversight Rights
-_None_
-
-## Notes
-Jurisdiction: test only. Effective 2026-10-19. Version 1. Caveat: synthetic fixture for the validator.
-`;
-  assert.equal(py('validate_mandate.py', scratch('ok.md', sections('**section 4(1) Records**: Keep records five years.'))).status, 0);
-  assert.equal(py('validate_mandate.py', scratch('prose.md', sections('This part has no obligations worth noting, n/a mostly.'))).status, 1);
-  assert.equal(py('validate_mandate.py', scratch('hollow.md', sections('**section **:'))).status, 1);
-});
-
-test('compare_checklist counts item headings, not ID mentions', () => {
-  const target = join(skillDir, 'references', 'tests', 'vars-checklist.md');
-  const canonical = py('compare_checklist.py', join(checklistsDir, 'visa-vars-checklist.md'), target);
-  assert.equal(canonical.status, 0, canonical.stdout);
-  const ids = [...new Set(readFileSync(target, 'utf8').match(/\b[A-Z]{4}\.C\d+\.\d+\b/g))].join(' ');
-  assert.equal(py('compare_checklist.py', scratch('fake.md', ids), target).status, 1);
-});

package/skills/engineering/better-review/scripts/pr-context.test.mjs

@@ -1,168 +0,0 @@
-import { test } from 'node:test'
-import assert from 'node:assert/strict'
-import { threeDotRange, parseNameStatus, parseLog, fastForwardable, isPromotionTarget, isIntegrationBranch, isPromotion, reviewNote, normalizeHeadName } from './pr-context.mjs'
-
-test('threeDotRange uses three dots (the merge diff, not two-dot)', () => {
-  assert.equal(threeDotRange('abc123'), 'abc123...HEAD')
-  assert.equal(threeDotRange('abc', 'feat'), 'abc...feat')
-})
-
-test('parseNameStatus handles add/modify/delete and rename', () => {
-  const text = ['M\tsrc/a.ts', 'A\tsrc/new.ts', 'D\tsrc/old.ts', 'R100\tsrc/from.ts\tsrc/to.ts'].join('\n')
-  assert.deepEqual(parseNameStatus(text), [
-    { status: 'M', path: 'src/a.ts' },
-    { status: 'A', path: 'src/new.ts' },
-    { status: 'D', path: 'src/old.ts' },
-    { status: 'R100', path: 'src/to.ts' },
-  ])
-})
-
-test('parseNameStatus empty → []', () => {
-  assert.deepEqual(parseNameStatus(''), [])
-})
-
-test('parseLog splits sha + subject', () => {
-  const text = '1a2b3c add export\n4d5e6f fix expired coupon'
-  assert.deepEqual(parseLog(text), [
-    { sha: '1a2b3c', subject: 'add export' },
-    { sha: '4d5e6f', subject: 'fix expired coupon' },
-  ])
-})
-
-test('fastForwardable: empty = FF possible; non-empty (base has commits head lacks) = not FF', () => {
-  assert.equal(fastForwardable([]), true)
-  assert.equal(fastForwardable([{ sha: 'h0tf1x', subject: 'hotfix prod' }]), false)
-  assert.equal(fastForwardable(undefined), true)
-})
-
-test('isPromotionTarget: production branches only (the base half of a promotion)', () => {
-  assert.equal(isPromotionTarget('main'), true)
-  assert.equal(isPromotionTarget('master'), true)
-  assert.equal(isPromotionTarget('production'), true)
-  assert.equal(isPromotionTarget('origin/main'), true)
-  assert.equal(isPromotionTarget('dev'), false)
-  assert.equal(isPromotionTarget('feature/x'), false)
-  assert.equal(isPromotionTarget('release/1.2'), false)
-})
-
-test('isIntegrationBranch: only integration/release-shaped heads (the head half of a promotion)', () => {
-  // integration branches
-  assert.equal(isIntegrationBranch('dev'), true)
-  assert.equal(isIntegrationBranch('develop'), true)
-  assert.equal(isIntegrationBranch('origin/dev'), true)
-  assert.equal(isIntegrationBranch('refs/heads/develop'), true)
-  // release / hotfix / staging shapes
-  assert.equal(isIntegrationBranch('release/1.4'), true)
-  assert.equal(isIntegrationBranch('release/2.0.0'), true)
-  assert.equal(isIntegrationBranch('hotfix/urgent-fix'), true)
-  assert.equal(isIntegrationBranch('staging'), true)
-  assert.equal(isIntegrationBranch('staging/eu'), true)
-  assert.equal(isIntegrationBranch('origin/release/1.4'), true)
-  // ordinary feature branches — NOT integration
-  assert.equal(isIntegrationBranch('feature/x'), false)
-  assert.equal(isIntegrationBranch('chore/388-doc-code-prose-audit'), false)
-  assert.equal(isIntegrationBranch('issue-392-promotion-heuristic'), false)
-  assert.equal(isIntegrationBranch('main'), false)
-  assert.equal(isIntegrationBranch('develop-tweaks'), false) // not exactly `develop`
-  assert.equal(isIntegrationBranch('feature/release-notes'), false) // `release` not its own segment
-  // unresolvable head (a bare SHA, or empty) — NOT integration → safe default
-  assert.equal(isIntegrationBranch('a1b2c3d'), false)
-  assert.equal(isIntegrationBranch(''), false)
-  assert.equal(isIntegrationBranch(undefined), false)
-})
-
-test('isPromotion: true ONLY when base is production AND head is integration/release-shaped', () => {
-  // the ~15x false-positive this fixes: a feature branch → main is NOT a promotion
-  assert.equal(isPromotion('main', 'chore/388-doc-code-prose-audit'), false)
-  assert.equal(isPromotion('main', 'feature/x'), false)
-  assert.equal(isPromotion('main', 'issue-392-promotion-heuristic'), false)
-  // genuine promotions: integration/release branch → production
-  assert.equal(isPromotion('main', 'dev'), true)
-  assert.equal(isPromotion('main', 'release/1.4'), true)
-  assert.equal(isPromotion('main', 'hotfix/x'), true)
-  assert.equal(isPromotion('master', 'develop'), true)
-  assert.equal(isPromotion('origin/main', 'origin/dev'), true)
-  // base is not a production branch → never a promotion, even from an integration head
-  assert.equal(isPromotion('dev', 'feature/x'), false)
-  assert.equal(isPromotion('release/1.4', 'dev'), false)
-  // unresolvable head (bare SHA / empty) → false (documented safe default)
-  assert.equal(isPromotion('main', 'a1b2c3d'), false)
-  assert.equal(isPromotion('main', ''), false)
-  assert.equal(isPromotion('main', undefined), false)
-})
-
-test('normalizeHeadName strips ONLY known ref prefixes — never an arbitrary leading segment', () => {
-  // known ref prefixes → stripped to the bare name
-  assert.equal(normalizeHeadName('refs/heads/develop'), 'develop')
-  assert.equal(normalizeHeadName('refs/remotes/origin/release/1.4'), 'release/1.4')
-  assert.equal(normalizeHeadName('origin/dev'), 'dev')
-  assert.equal(normalizeHeadName('origin/release/1.4'), 'release/1.4')
-  assert.equal(normalizeHeadName('upstream/staging'), 'staging')
-  // NOT a known prefix → left intact (the any-prefix hole this closes)
-  assert.equal(normalizeHeadName('jsmith/dev'), 'jsmith/dev')
-  assert.equal(normalizeHeadName('feature/release/notes'), 'feature/release/notes')
-  assert.equal(normalizeHeadName('team/staging/eu'), 'team/staging/eu')
-  // plain names pass through
-  assert.equal(normalizeHeadName('dev'), 'dev')
-  assert.equal(normalizeHeadName('release/1.4'), 'release/1.4')
-  assert.equal(normalizeHeadName(''), '')
-  assert.equal(normalizeHeadName(undefined), '')
-})
-
-// LEFT-BOUNDARY REGRESSION (issue #392, fix round): the integration/release name must
-// open the branch (after ref-prefix stripping), never be buried under a namespace. A
-// nested-prefix feature branch is an ordinary feature PR, not a promotion — the unsafe
-// direction if mis-classified. Removing the `^`/normalization makes these go RED.
-test('isIntegrationBranch: nested-prefix feature branches are NOT integration (left-boundary hole)', () => {
-  // line-79 (release/hotfix/staging path) hole
-  assert.equal(isIntegrationBranch('feature/release/notes'), false)
-  assert.equal(isIntegrationBranch('feature/hotfix/x'), false)
-  assert.equal(isIntegrationBranch('team/staging/eu'), false)
-  // line-77 (exact dev/develop/staging) hole
-  assert.equal(isIntegrationBranch('jsmith/dev'), false)
-  assert.equal(isIntegrationBranch('alice/develop'), false)
-  assert.equal(isIntegrationBranch('wip/staging'), false)
-})
-
-test('isIntegrationBranch: supported ref prefixes still resolve to TRUE after normalization', () => {
-  assert.equal(isIntegrationBranch('origin/dev'), true)
-  assert.equal(isIntegrationBranch('refs/heads/develop'), true)
-  assert.equal(isIntegrationBranch('refs/remotes/origin/release/1.4'), true)
-})
-
-test('isPromotion: nested-prefix feature branches → main are NOT promotions (left-boundary hole)', () => {
-  // these all classified promotion=true before the anchor fix (the fail-open direction)
-  assert.equal(isPromotion('main', 'feature/release/notes'), false)
-  assert.equal(isPromotion('main', 'feature/hotfix/x'), false)
-  assert.equal(isPromotion('main', 'team/staging/eu'), false)
-  assert.equal(isPromotion('main', 'jsmith/dev'), false)
-  assert.equal(isPromotion('main', 'alice/develop'), false)
-  assert.equal(isPromotion('main', 'wip/staging'), false)
-  // supported prefixes remain genuine promotions
-  assert.equal(isPromotion('main', 'refs/heads/develop'), true)
-  assert.equal(isPromotion('main', 'refs/remotes/origin/release/1.4'), true)
-})
-
-test('reviewNote: PROMOTION + NOT fast-forwardable note still fires for a genuine promotion that is not FF', () => {
-  const note = reviewNote({
-    promotion: true,
-    fastForwardable: false,
-    base: 'main',
-    head: 'release/1.4',
-    commitsOnBaseNotInHead: [{ sha: 'h0tf1x', subject: 'hotfix prod' }],
-  })
-  assert.match(note, /PROMOTION \+ NOT fast-forwardable/)
-  assert.match(note, /main has 1 commit\(s\) not in release\/1\.4/)
-  assert.match(note, /HOLD/)
-})
-
-test('reviewNote: fast-forwardable promotion → release-readiness note (not a re-review)', () => {
-  const note = reviewNote({ promotion: true, fastForwardable: true, base: 'main', head: 'dev', commitsOnBaseNotInHead: [] })
-  assert.match(note, /PROMOTION \(fast-forwardable\)/)
-  assert.doesNotMatch(note, /NOT fast-forwardable/)
-})
-
-test('reviewNote: not a promotion → review the NET diff (the common feature-PR path)', () => {
-  const note = reviewNote({ promotion: false, fastForwardable: true, base: 'main', head: 'feature/x', commitsOnBaseNotInHead: [] })
-  assert.match(note, /Review the NET diff/)
-})