@xulthekl/team-flow 0.31.0 → 0.32.1

package/tests/lib/token-baseline.test.mjs

@@ -1,72 +0,0 @@
-// tests/lib/token-baseline.test.mjs
-// Tests for scripts/token-baseline.mjs — measurement and comparison
-
-import { describe, it } from 'node:test';
-import assert from 'node:assert/strict';
-import { execSync } from 'node:child_process';
-import { writeFileSync, unlinkSync, readFileSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-
-const BASELINE_PATH = join(process.cwd(), 'scripts/token-baseline.mjs');
-const TMP = tmpdir();
-
-describe('token-baseline: measurement', () => {
-  it('outputs valid JSON with components and totals', () => {
-    const out = execSync(`node "${BASELINE_PATH}" --files hooks/session-start`, { encoding: 'utf-8' });
-    const data = JSON.parse(out);
-    assert.ok(Array.isArray(data.components), 'components should be an array');
-    assert.ok(data.components.length > 0, 'should have at least one component');
-    const c = data.components[0];
-    assert.ok(typeof c.lines === 'number' && c.lines > 0, 'lines should be positive');
-    assert.ok(typeof c.chars === 'number' && c.chars > 0, 'chars should be positive');
-    assert.ok(typeof c.estimatedTokens === 'number' && c.estimatedTokens > 0, 'estimatedTokens should be positive');
-    assert.ok(typeof data.totals === 'object', 'totals should be an object');
-    assert.ok(data.totals.estimatedTokens > 0, 'total estimatedTokens should be positive');
-  });
-
-  it('--files filter limits output', () => {
-    const out = execSync(`node "${BASELINE_PATH}" --files hooks/session-start`, { encoding: 'utf-8' });
-    const data = JSON.parse(out);
-    assert.equal(data.components.length, 1, 'should only have one component');
-    assert.ok(data.components[0].path.includes('hooks/session-start'), 'should be the filtered file');
-  });
-
-  it('--output writes to file', () => {
-    const outPath = join(TMP, 'token-baseline-test.json');
-    execSync(`node "${BASELINE_PATH}" --files hooks/session-start --output "${outPath}"`, { encoding: 'utf-8' });
-    const data = JSON.parse(readFileSync(outPath, 'utf-8'));
-    assert.ok(data.totals.estimatedTokens > 0, 'file should contain valid data');
-    unlinkSync(outPath);
-  });
-});
-
-describe('token-baseline: comparison', () => {
-  it('--compare produces change deltas', () => {
-    const baselinePath = join(TMP, 'baseline-cmp-test.json');
-    // Create a baseline with slightly different values
-    const baseline = {
-      timestamp: '2026-01-01T00:00:00Z',
-      components: [
-        { path: 'hooks/session-start', label: 'hooks/session-start', lines: 30, chars: 2000, estimatedTokens: 500 },
-      ],
-      totals: { lines: 30, chars: 2000, estimatedTokens: 500 },
-    };
-    writeFileSync(baselinePath, JSON.stringify(baseline));
-
-    const out = execSync(`node "${BASELINE_PATH}" --files hooks/session-start --compare "${baselinePath}"`, { encoding: 'utf-8' });
-    const data = JSON.parse(out);
-    assert.ok(Array.isArray(data.changes), 'should have changes array');
-    assert.ok(data.changes.length > 0, 'should have at least one change');
-    assert.ok(typeof data.summary.totalDelta === 'number', 'should have totalDelta');
-    assert.ok(typeof data.summary.totalPct === 'string', 'should have totalPct');
-
-    unlinkSync(baselinePath);
-  });
-
-  it('missing file in baseline reports error', () => {
-    const out = execSync(`node "${BASELINE_PATH}" --files hooks/session-start --compare /nonexistent/baseline.json 2>&1 || true`, { encoding: 'utf-8' });
-    // Should fail gracefully — error message from file not found
-    assert.ok(out.length > 0, 'should produce output even on error');
-  });
-});

package/tests/lib/token-rules.test.mjs

@@ -1,94 +0,0 @@
-// tests/lib/token-rules.test.mjs
-// Tests for scripts/lint/rules/token-rules.mjs — lint rule functions
-
-import { describe, it } from 'node:test';
-import assert from 'node:assert/strict';
-import { checkMaxLines, checkMaxChars, checkMaxEmphasisMarkers, checkMaxCodeBlockLength } from '../../scripts/lint/rules/token-rules.mjs';
-
-const CTX = { skillDirs: ['test-skill'], skillsDir: '/fake/skills' };
-
-describe('token-rules: checkMaxLines', () => {
-  it('passes when under limit', async () => {
-    const issues = await checkMaxLines('test', 'line1\nline2\nline3', CTX, 10);
-    assert.equal(issues.length, 0);
-  });
-
-  it('fails when over limit', async () => {
-    const content = Array(251).fill('line').join('\n');
-    const issues = await checkMaxLines('test', content, CTX, 250);
-    assert.equal(issues.length, 1);
-    assert.equal(issues[0].severity, 'error');
-    assert.ok(issues[0].message.includes('251'));
-  });
-});
-
-describe('token-rules: checkMaxChars', () => {
-  it('passes when under limit', async () => {
-    const issues = await checkMaxChars('test', 'short', CTX, 100);
-    assert.equal(issues.length, 0);
-  });
-
-  it('warns when over limit', async () => {
-    const content = 'x'.repeat(10001);
-    const issues = await checkMaxChars('test', content, CTX, 10000);
-    assert.equal(issues.length, 1);
-    assert.equal(issues[0].severity, 'warning');
-  });
-});
-
-describe('token-rules: checkMaxEmphasisMarkers', () => {
-  it('passes for content without banned markers', async () => {
-    const issues = await checkMaxEmphasisMarkers('test', 'normal text with **some** emphasis', CTX, 30);
-    assert.equal(issues.length, 0);
-  });
-
-  it('detects EXTREMELY_IMPORTANT', async () => {
-    const issues = await checkMaxEmphasisMarkers('test', 'This is EXTREMELY_IMPORTANT content', CTX, 30);
-    const eiIssue = issues.find(i => i.message.includes('EXTREMELY_IMPORTANT'));
-    assert.ok(eiIssue, 'should flag EXTREMELY_IMPORTANT');
-    assert.equal(eiIssue.severity, 'error');
-  });
-
-  it('detects CRITICAL', async () => {
-    const issues = await checkMaxEmphasisMarkers('test', 'CRITICAL: do not skip', CTX, 30);
-    const crIssue = issues.find(i => i.message.includes('CRITICAL'));
-    assert.ok(crIssue, 'should flag CRITICAL');
-    assert.equal(crIssue.severity, 'error');
-  });
-
-  it('warns on too many IMPORTANT occurrences', async () => {
-    const content = 'IMPORTANT\n'.repeat(5) + 'text';
-    const issues = await checkMaxEmphasisMarkers('test', content, CTX, 30);
-    const impIssue = issues.find(i => i.message.includes('IMPORTANT'));
-    assert.ok(impIssue, 'should flag excessive IMPORTANT');
-  });
-
-  it('warns on too many emphasis markers', async () => {
-    const content = '**a** **b** '.repeat(31);
-    const issues = await checkMaxEmphasisMarkers('test', content, CTX, 30);
-    const empIssue = issues.find(i => i.message.includes('emphasis markers'));
-    assert.ok(empIssue, 'should flag excessive emphasis');
-    assert.equal(empIssue.severity, 'warning');
-  });
-});
-
-describe('token-rules: checkMaxCodeBlockLength', () => {
-  it('passes for short code blocks', async () => {
-    const issues = await checkMaxCodeBlockLength('test', '```\nline1\nline2\n```', CTX, 5);
-    assert.equal(issues.length, 0);
-  });
-
-  it('warns for long code blocks', async () => {
-    const lines = Array(17).fill('code').join('\n');
-    const content = '```\n' + lines + '\n```';
-    const issues = await checkMaxCodeBlockLength('test', content, CTX, 15);
-    assert.equal(issues.length, 1);
-    assert.equal(issues[0].severity, 'warning');
-    assert.ok(issues[0].message.includes('19'), `expected message to include '19', got: ${issues[0].message}`);
-  });
-
-  it('handles content with no code blocks', async () => {
-    const issues = await checkMaxCodeBlockLength('test', 'just text, no code', CTX, 15);
-    assert.equal(issues.length, 0);
-  });
-});

package/tests/lib/verify-marketplace-release.test.mjs

@@ -1,128 +0,0 @@
-import { describe, it } from 'node:test';
-import assert from 'node:assert/strict';
-import { spawnSync } from 'node:child_process';
-import {
-  main,
-  parseReleaseCheckArgs,
-  verifyMarketplaceRelease,
-} from '../../scripts/verify-marketplace-release.mjs';
-
-const response = ({ ok = true, status = 200, body = { version: '0.9.0' } } = {}) => ({
-  ok,
-  status,
-  async json() { return body; },
-});
-
-describe('verify marketplace release', () => {
-  it('accepts a matching HTTPS manifest version', async () => {
-    const result = await verifyMarketplaceRelease({
-      manifestUrl: 'https://example.test/plugin.json',
-      expectedVersion: '0.9.0',
-      fetchImpl: async () => response(),
-    });
-    assert.equal(result.actualVersion, '0.9.0');
-  });
-
-  it('rejects HTTP, malformed, network, and version-drift responses', async () => {
-    await assert.rejects(
-      () => verifyMarketplaceRelease({
-        manifestUrl: 'https://example.test/plugin.json',
-        expectedVersion: '0.9.0',
-        fetchImpl: async () => response({ ok: false, status: 404 }),
-      }),
-      /HTTP 404/,
-    );
-    await assert.rejects(
-      () => verifyMarketplaceRelease({
-        manifestUrl: 'https://example.test/plugin.json',
-        expectedVersion: '0.9.0',
-        fetchImpl: async () => response({ body: { name: 'team-flow' } }),
-      }),
-      /missing a string version/,
-    );
-    await assert.rejects(
-      () => verifyMarketplaceRelease({
-        manifestUrl: 'https://example.test/plugin.json',
-        expectedVersion: '0.9.0',
-        fetchImpl: async () => ({
-          ok: true,
-          status: 200,
-          async json() { throw new Error('unexpected token'); },
-        }),
-      }),
-      /marketplace manifest is not valid JSON: unexpected token/,
-    );
-    await assert.rejects(
-      () => verifyMarketplaceRelease({
-        manifestUrl: 'https://example.test/plugin.json',
-        expectedVersion: '0.9.0',
-        fetchImpl: async () => { throw new Error('socket closed'); },
-      }),
-      /network request failed: socket closed/,
-    );
-    await assert.rejects(
-      () => verifyMarketplaceRelease({
-        manifestUrl: 'https://example.test/plugin.json',
-        expectedVersion: '0.9.0',
-        fetchImpl: async () => response({ body: { version: '0.8.17' } }),
-      }),
-      /expected 0.9.0, found 0.8.17/,
-    );
-  });
-
-  it('requires one HTTPS URL and one complete semver argument', () => {
-    assert.throws(() => parseReleaseCheckArgs([]), /Usage:/);
-    assert.throws(
-      () => parseReleaseCheckArgs([
-        '--manifest-url', 'http://example.test/plugin.json',
-        '--expected-version', '0.9.0',
-      ]),
-      /HTTPS/,
-    );
-    assert.throws(
-      () => parseReleaseCheckArgs([
-        '--manifest-url', 'https://example.test/plugin.json',
-        '--expected-version', '0.9',
-      ]),
-      /x\.y\.z/,
-    );
-  });
-
-  it('writes exact success and error messages through injected streams', async () => {
-    const stdout = [];
-    const stderr = [];
-    const args = [
-      '--manifest-url', 'https://example.test/plugin.json',
-      '--expected-version', '0.9.0',
-    ];
-    const streams = {
-      stdout: { write: (message) => stdout.push(message) },
-      stderr: { write: (message) => stderr.push(message) },
-    };
-
-    assert.equal(await main(args, {
-      ...streams,
-      fetchImpl: async () => response(),
-    }), 0);
-    assert.deepEqual(stdout, ['Marketplace manifest verified: 0.9.0 (https://example.test/plugin.json)\n']);
-    assert.deepEqual(stderr, []);
-
-    assert.equal(await main(args, {
-      ...streams,
-      fetchImpl: async () => response({ body: { version: '0.8.17' } }),
-    }), 1);
-    assert.deepEqual(stderr, ['Error: marketplace version mismatch: expected 0.9.0, found 0.8.17\n']);
-  });
-
-  it('does not set process.exitCode when imported', () => {
-    const scriptUrl = new URL('../../scripts/verify-marketplace-release.mjs', import.meta.url).href;
-    const result = spawnSync(process.execPath, [
-      '--input-type=module',
-      '--eval',
-      `await import(${JSON.stringify(scriptUrl)}); if (process.exitCode !== undefined) throw new Error('process.exitCode was set');`,
-    ], { encoding: 'utf8' });
-
-    assert.equal(result.status, 0, result.stderr);
-    assert.equal(result.stderr, '');
-  });
-});

package/tests/tsconfig.json

@@ -1,12 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "ES2022",
-    "module": "NodeNext",
-    "moduleResolution": "NodeNext",
-    "strict": true,
-    "esModuleInterop": true,
-    "skipLibCheck": true,
-    "noEmit": true
-  },
-  "include": ["./**/*.ts", "../src/**/*.ts"]
-}