@xgent-ai/skills 0.7.4 → 0.7.6

package/skills/archify/test/community-proof-intake.test.mjs

@@ -1,125 +0,0 @@
-import assert from 'node:assert/strict';
-import fs from 'node:fs';
-import path from 'node:path';
-import test from 'node:test';
-import { fileURLToPath } from 'node:url';
-
-const here = path.dirname(fileURLToPath(import.meta.url));
-const repoRoot = path.resolve(here, '../..');
-
-function read(relativePath) {
-  return fs.readFileSync(path.join(repoRoot, relativePath), 'utf8');
-}
-
-test('showcase intake requires reproducible proof, redaction, and explicit publication permission', () => {
-  const template = read('.github/ISSUE_TEMPLATE/showcase.yml');
-
-  for (const field of [
-    'id: diagram_type',
-    'id: archify_version',
-    'id: agent',
-    'id: model',
-    'id: prompt',
-    'id: source_json',
-    'id: artifact',
-    'id: validation_receipt',
-    'id: visual_review',
-    'id: sensitive_data',
-    'id: sharing_rights',
-    'id: public_permission',
-  ]) {
-    assert.match(template, new RegExp(field), field);
-  }
-  assert.match(template, /access tokens/i);
-  assert.match(template, /personal or customer data/i);
-  assert.match(template, /repository, documentation, gallery, and project website/i);
-  assert.match(template, /required:\s*true/g);
-
-  const submissionUrl = 'https://github.com/tt-a1i/archify/issues/new?template=showcase.yml';
-  for (const readme of ['README.md', 'README_EN.md', 'README_ZH.md']) {
-    assert.match(read(readme), new RegExp(submissionUrl.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')), `${readme}: direct showcase link`);
-  }
-});
-
-test('bug intake captures a minimal deterministic reproduction before visual diagnosis', () => {
-  const template = read('.github/ISSUE_TEMPLATE/bug-report.yml');
-
-  for (const field of [
-    'id: archify_version',
-    'id: install_method',
-    'id: diagram_type',
-    'id: command',
-    'id: minimal_json',
-    'id: validation_receipt',
-    'id: expected',
-    'id: actual',
-    'id: visual_evidence',
-    'id: environment',
-    'id: sensitive_data',
-  ]) {
-    assert.match(template, new RegExp(field), field);
-  }
-  assert.ok(
-    template.indexOf('id: validation_receipt') < template.indexOf('id: visual_evidence'),
-    'deterministic evidence should be requested before visual evidence',
-  );
-  const evidenceBlock = template.slice(
-    template.indexOf('id: visual_evidence'),
-    template.indexOf('id: environment'),
-  );
-  assert.match(template, /type: upload\s+id: visual_evidence/);
-  assert.match(evidenceBlock, /required:\s*false/);
-  assert.match(evidenceBlock, /screenshots or recordings/i);
-  assert.match(evidenceBlock, /Reproducible steps above may be enough/i);
-  assert.doesNotMatch(evidenceBlock, /Required for visual problems/i);
-  assert.doesNotMatch(evidenceBlock, /accept:/);
-});
-
-test('contributor and pull-request guides keep proof changes reproducible and stability-first', () => {
-  const contributing = read('CONTRIBUTING.md');
-  const pullRequest = read('.github/PULL_REQUEST_TEMPLATE.md');
-
-  for (const required of [
-    '.github/ISSUE_TEMPLATE/showcase.yml',
-    '.github/ISSUE_TEMPLATE/bug-report.yml',
-    'npm test',
-    'node scripts/build-gallery.mjs docs',
-    'Do not include secrets',
-    'Agent-first',
-    'diagnostics[]',
-    'Start from the latest `main`',
-    'tracked-only, symlink-safe',
-    'is **skipped**, not passed',
-  ]) {
-    assert.match(contributing, new RegExp(required.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')), required);
-  }
-  assert.match(
-    contributing,
-    /(?:^|\n)scripts\/build-zip\.sh \/tmp\/archify-contrib\.zip(?:\n|$)/,
-    'the archive builder must be documented as an executable shell script',
-  );
-  assert.doesNotMatch(
-    contributing,
-    /\bnode\s+scripts\/build-zip\.sh\b/,
-    'the shell archive builder must not be documented as a Node.js command',
-  );
-  for (const required of [
-    'Stability impact',
-    'Tests run',
-    'Generated artifacts',
-    'Visual evidence',
-    'Evidence provided:',
-    'Automated or browser checks:',
-    'Perceptual visual review: passed / failed / skipped / Not applicable',
-  ]) {
-    assert.match(pullRequest, new RegExp(required), required);
-  }
-  assert.match(contributing, /enough evidence to evaluate whether the intended user value was achieved/i);
-  assert.match(contributing, /screenshots, recordings, or reproducible steps/i);
-  assert.match(contributing, /same input/i);
-  assert.match(contributing, /automated or browser evidence separately from perceptual review/i);
-  assert.match(pullRequest, /screenshots, recordings, or reproducible steps/i);
-  assert.match(pullRequest, /automated or browser evidence separately from perceptual review/i);
-  assert.doesNotMatch(contributing, /must reach `passed` before final review or merge/i);
-  assert.doesNotMatch(pullRequest, /must reach `passed` before final review or merge/i);
-});

package/skills/archify/test/cursor-onboarding.test.mjs

@@ -1,74 +0,0 @@
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { execFileSync } from 'node:child_process';
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const skillRoot = path.resolve(__dirname, '..');
-const repoRoot = path.resolve(skillRoot, '..');
-const cursorCommand = 'npx -y skills add tt-a1i/archify --skill archify --agent cursor --global --copy --yes';
-
-test('Cursor onboarding stays explicit, bilingual, and backed by the same Skill', () => {
-  const english = fs.readFileSync(path.join(repoRoot, 'README.md'), 'utf8');
-  const englishMirror = fs.readFileSync(path.join(repoRoot, 'README_EN.md'), 'utf8');
-  const chinese = fs.readFileSync(path.join(repoRoot, 'README_ZH.md'), 'utf8');
-  const start = fs.readFileSync(path.join(repoRoot, 'docs', 'start.html'), 'utf8');
-  const landing = fs.readFileSync(path.join(repoRoot, 'docs', 'index.html'), 'utf8');
-
-  assert.equal(english, englishMirror, 'English README mirrors must stay synchronized');
-  assert.match(english, /Cursor, Claude Code, Codex CLI, and OpenCode/);
-  assert.match(chinese, /Cursor、Claude Code、Codex CLI 和 OpenCode/);
-  for (const surface of [english, chinese, landing]) assert.ok(surface.includes(cursorCommand));
-  for (const surface of [english, chinese, start, landing]) {
-    assert.doesNotMatch(surface, /skills use[^\n<]*--agent cursor/);
-    assert.doesNotMatch(surface, /~\/\.cursor\/skills\/archify/);
-    assert.doesNotMatch(surface, /all Cursor models|every Cursor model/i);
-  }
-
-  assert.match(start, /data-agent="cursor">Cursor<\/button>/);
-  assert.match(start, /data-agent="codex">Codex<\/button>/);
-  assert.match(start, /data-agent="claude-code">Claude Code<\/button>/);
-  assert.match(start, /data-agent="opencode">OpenCode<\/button>/);
-  assert.match(start, /KNOWN_AGENTS\.has\(requestedAgent\)/);
-  assert.match(start, /same Skill/);
-  assert.match(start, /同一份 Skill/);
-  assert.doesNotMatch(start, /vendor-specific (?:renderer|schema|skill)/i);
-});
-
-test('the zero-dependency archive works from the canonical Cursor-visible agent path', () => {
-  const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'archify-cursor-package-'));
-  const agentSkills = path.join(tmp, '.agents', 'skills');
-  try {
-    fs.mkdirSync(agentSkills, { recursive: true });
-    execFileSync('unzip', ['-q', path.join(repoRoot, 'archify.zip'), '-d', agentSkills]);
-    const installed = path.join(agentSkills, 'archify');
-    const cli = path.join(installed, 'bin', 'archify.mjs');
-    const doctor = execFileSync(process.execPath, [cli, 'doctor'], { encoding: 'utf8' });
-    assert.match(doctor, /Archify is ready\./);
-
-    const fixtures = {
-      architecture: 'web-app.architecture.json',
-      workflow: 'agent-tool-call.workflow.json',
-      sequence: 'cache-miss-request.sequence.json',
-      dataflow: 'product-analytics.dataflow.json',
-      lifecycle: 'agent-run.lifecycle.json',
-    };
-    for (const [type, fixture] of Object.entries(fixtures)) {
-      const output = execFileSync(process.execPath, [
-        cli,
-        'validate',
-        type,
-        path.join(installed, 'examples', fixture),
-        '--json',
-      ], { encoding: 'utf8' });
-      const receipt = JSON.parse(output);
-      assert.equal(receipt.ok, true, `${type}: installed package validation failed`);
-      assert.equal(receipt.type, type);
-    }
-  } finally {
-    fs.rmSync(tmp, { recursive: true, force: true });
-  }
-});

package/skills/archify/test/degraded.test.mjs

@@ -1,161 +0,0 @@
-// Installation contract: the shipped skill performs full JSON Schema
-// validation without node_modules. AJV is a build-time dependency only; its
-// standalone validators are committed and included in the distribution.
-//
-// A malformed-but-JSON-legal document must EXIT
-// NON-ZERO with a friendly message — never crash (TypeError / is not a
-// function) and never write NaN/undefined into the HTML. A random VALID
-// perturbation of an example must still render (exit 0, no NaN).
-//
-//   node --test test/*.test.mjs
-
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { execFileSync } from 'node:child_process';
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const skillRoot = path.resolve(__dirname, '..');
-const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'archify-degraded-'));
-
-const EXAMPLES = {
-  workflow: 'agent-tool-call.workflow.json',
-  sequence: 'cache-miss-request.sequence.json',
-  dataflow: 'product-analytics.dataflow.json',
-  lifecycle: 'agent-run.lifecycle.json',
-  architecture: 'web-app.architecture.json',
-};
-
-const installedRoot = path.join(tmp, 'installed-skill');
-fs.cpSync(skillRoot, installedRoot, {
-  recursive: true,
-  filter(source) {
-    const rel = path.relative(skillRoot, source);
-    return rel !== 'node_modules' && !rel.startsWith(`node_modules${path.sep}`)
-      && rel !== 'test' && !rel.startsWith(`test${path.sep}`)
-      // The validator freshness test creates and removes this fixture inside
-      // skillRoot while the test runner executes files concurrently. Exclude
-      // it from the installed-skill copy to avoid a copy/remove race.
-      && !rel.startsWith('.validator-check-');
-  },
-});
-
-function render(mode, doc) {
-  const input = path.join(tmp, `in-${Math.random().toString(36).slice(2)}.json`);
-  const out = path.join(tmp, 'out.html');
-  fs.writeFileSync(input, JSON.stringify(doc));
-  if (fs.existsSync(out)) fs.rmSync(out);
-  let code = 0;
-  let stderr = '';
-  try {
-    execFileSync('node', [path.join(installedRoot, `renderers/${mode}/render-${mode}.mjs`), input, out],
-      { stdio: ['ignore', 'ignore', 'pipe'] });
-  } catch (err) {
-    code = err.status ?? 1;
-    stderr = String(err.stderr || '');
-  }
-  const html = fs.existsSync(out) ? fs.readFileSync(out, 'utf8') : '';
-  return { code, stderr, html };
-}
-
-function assertFriendlyFailure(mode, doc, label) {
-  const { code, stderr, html } = render(mode, doc);
-  assert.notEqual(code, 0, `${label}: expected non-zero exit`);
-  assert.doesNotMatch(stderr, /TypeError|RangeError|is not a function|Cannot read/,
-    `${label}: crashed instead of reporting friendly error:\n${stderr}`);
-  assert.doesNotMatch(html, /NaN|undefined/, `${label}: wrote NaN/undefined into HTML`);
-}
-
-// ---- type-wrong-but-JSON-legal documents per mode ----
-const ARRAY_FIELDS = {
-  workflow: ['lanes', 'phases', 'groups', 'mainPath', 'nodes', 'edges', 'cards'],
-  sequence: ['participants', 'messages', 'segments', 'activations', 'cards'],
-  dataflow: ['stages', 'nodes', 'flows', 'cards'],
-  lifecycle: ['lanes', 'states', 'transitions', 'cards'],
-  architecture: ['components', 'boundaries', 'connections', 'cards'],
-};
-
-for (const [mode, fields] of Object.entries(ARRAY_FIELDS)) {
-  for (const field of fields) {
-    test(`${mode}: ${field} as a string fails friendly`, () => {
-      const doc = JSON.parse(fs.readFileSync(path.join(skillRoot, 'examples', EXAMPLES[mode]), 'utf8'));
-      if (!(field in doc)) return; // optional field absent in this example
-      doc[field] = 'oops';
-      assertFriendlyFailure(mode, doc, `${mode}.${field}`);
-    });
-  }
-  test(`${mode}: scalar meta fails friendly`, () => {
-    const doc = JSON.parse(fs.readFileSync(path.join(skillRoot, 'examples', EXAMPLES[mode]), 'utf8'));
-    doc.meta = 42;
-    assertFriendlyFailure(mode, doc, `${mode}.meta`);
-  });
-}
-
-// ---- missing-coordinate fields must not yield NaN coordinates ----
-test('workflow: node missing col never writes NaN', () => {
-  const doc = JSON.parse(fs.readFileSync(path.join(skillRoot, 'examples', EXAMPLES.workflow), 'utf8'));
-  delete doc.nodes[0].col;
-  assertFriendlyFailure('workflow', doc, 'workflow node no col');
-});
-test('lifecycle: state missing col never writes NaN', () => {
-  const doc = JSON.parse(fs.readFileSync(path.join(skillRoot, 'examples', EXAMPLES.lifecycle), 'utf8'));
-  delete doc.states[0].col;
-  assertFriendlyFailure('lifecycle', doc, 'lifecycle state no col');
-});
-
-// ---- property test: deterministic VALID perturbations always render ----
-// Seeded PRNG (no Math.random — keeps the test reproducible across runs).
-function mulberry32(seed) {
-  return function next() {
-    seed |= 0; seed = (seed + 0x6D2B79F5) | 0;
-    let t = Math.imul(seed ^ (seed >>> 15), 1 | seed);
-    t = (t + Math.imul(t ^ (t >>> 7), 61 | t)) ^ t;
-    return ((t ^ (t >>> 14)) >>> 0) / 4294967296;
-  };
-}
-
-test('property: shuffling node/state order still renders (order-independence)', () => {
-  for (const mode of ['workflow', 'dataflow', 'lifecycle']) {
-    const arrKey = mode === 'lifecycle' ? 'states' : 'nodes';
-    for (let seed = 1; seed <= 8; seed += 1) {
-      const doc = JSON.parse(fs.readFileSync(path.join(skillRoot, 'examples', EXAMPLES[mode]), 'utf8'));
-      const rng = mulberry32(seed);
-      // Fisher–Yates with the seeded PRNG.
-      const a = doc[arrKey];
-      for (let i = a.length - 1; i > 0; i -= 1) {
-        const j = Math.floor(rng() * (i + 1));
-        [a[i], a[j]] = [a[j], a[i]];
-      }
-      const { code, html } = render(mode, doc);
-      assert.equal(code, 0, `${mode} seed ${seed}: valid shuffle should render (exit 0)`);
-      assert.doesNotMatch(html, /NaN|undefined>/, `${mode} seed ${seed}: NaN in output`);
-    }
-  }
-});
-
-test('installed skill rejects unknown fields without node_modules', () => {
-  const doc = JSON.parse(fs.readFileSync(path.join(skillRoot, 'examples', EXAMPLES.workflow), 'utf8'));
-  doc.nodes[0].colour = 'cyan';
-  const { code, stderr } = render('workflow', doc);
-  assert.notEqual(code, 0);
-  assert.match(stderr, /workflow schema validation failed/);
-  assert.match(stderr, /\/nodes\/0 \(id\/label: "user"\) must NOT have additional properties/);
-  assert.match(stderr, /"additionalProperty":"colour"/);
-  assert.doesNotMatch(stderr, /ajv is not installed|skipping JSON-schema validation/);
-});
-
-for (const mode of Object.keys(EXAMPLES)) {
-  test(`installed skill retains full ${mode} schema without node_modules`, () => {
-    const doc = JSON.parse(fs.readFileSync(path.join(skillRoot, 'examples', EXAMPLES[mode]), 'utf8'));
-    doc.unknownField = true;
-    const { code, stderr } = render(mode, doc);
-    assert.notEqual(code, 0);
-    assert.match(stderr, new RegExp(`${mode} schema validation failed`));
-    assert.match(stderr, /"additionalProperty":"unknownField"/);
-  });
-}
-
-process.on('exit', () => fs.rmSync(tmp, { recursive: true, force: true }));

package/skills/archify/test/delivery-contract.test.mjs

@@ -1,52 +0,0 @@
-import { readFileSync } from 'node:fs';
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-import test from 'node:test';
-import assert from 'node:assert/strict';
-
-const here = path.dirname(fileURLToPath(import.meta.url));
-const skill = readFileSync(path.join(here, '..', 'SKILL.md'), 'utf8');
-const delivery = readFileSync(path.join(here, '..', 'references', 'delivery-contract.md'), 'utf8');
-
-test('skill requires a bounded and truthful perceptual delivery receipt', () => {
-  assert.match(delivery, /browser_evidence: passed\|failed\|skipped/);
-  assert.match(delivery, /visual_review: passed/);
-  assert.match(delivery, /visual_review: skipped \(image reader unavailable\)/);
-  assert.match(delivery, /correction_rounds: [0-2]/);
-  assert.match(delivery, /maximum of two focused correction rounds/i);
-  assert.match(delivery, /never report `visual_review: passed` without inspecting/i);
-});
-
-test('skill keeps deterministic delivery, automated browser evidence, and perceptual review distinct', () => {
-  for (const [name, source] of [['SKILL.md', skill], ['delivery contract', delivery]]) {
-    assert.match(source, /deliver[\s\S]*deterministic/i, name);
-    assert.match(source, /visual-check[\s\S]*automated browser evidence/i, name);
-    assert.match(source, /human|perceptual visual review/i, name);
-  }
-  assert.match(delivery, /manual browser record[\s\S]*all four exact viewport measurements, both endpoint themes, and an artifact-bound record/i);
-});
-
-test('handoff browser evidence mirrors only the automated visual-check outcome', () => {
-  assert.match(delivery, /`browser_evidence`[\s\S]*records only the outcome of this automated command/i);
-  assert.match(delivery, /`passed`[\s\S]*exit 0[\s\S]*receipt `status: "pass"`/i);
-  assert.match(delivery, /`failed`[\s\S]*exit 1[\s\S]*receipt `status: "fail"`/i);
-  assert.match(delivery, /`skipped`[\s\S]*exit 2[\s\S]*receipt `status: "skipped"`/i);
-  assert.match(delivery, /runtime or capture failures[\s\S]*must not be normalized to `skipped`/i);
-  assert.match(delivery, /remains `skipped` even when[\s\S]*`visual_review: passed`/i);
-  assert.match(delivery, /manual browser record[\s\S]*never changes `browser_evidence`/i);
-});
-
-test('skill uses atomic verified delivery for the final artifact', () => {
-  assert.match(delivery, /archify\.mjs deliver <type>/);
-  assert.match(delivery, /same-directory candidate/i);
-  assert.match(delivery, /only replaces the target after.*artifact checks pass/i);
-  assert.match(delivery, /never claim that the deterministic receipt includes visual review/i);
-});
-
-test('skill keeps optional opening behind the verified commit and outside automation', () => {
-  assert.match(delivery, /Add `--open` only when the user wants an immediate local preview/);
-  assert.match(delivery, /runs after that atomic commit/);
-  assert.match(delivery, /Keep it off for CI, unattended agents, and non-interactive environments/);
-  assert.match(delivery, /never invokes an opener/);
-  assert.match(delivery, /status proves only whether the local opener invocation succeeded/);
-});

package/skills/archify/test/desktop-reader-browser.test.mjs

@@ -1,80 +0,0 @@
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { execFileSync } from 'node:child_process';
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-import { findChrome, runVisualCheck } from '../bin/visual-check.mjs';
-import { DESKTOP_READABILITY_VIEWPORT, MIN_PROJECTED_NODE_TEXT_PX } from '../renderers/shared/desktop-readability.mjs';
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const skillRoot = path.resolve(__dirname, '..');
-const chromePath = process.env.ARCHIFY_CHROME ? findChrome() : null;
-const packagedHtmlExamples = fs.readdirSync(path.join(skillRoot, 'examples'))
-  .filter((name) => name.endsWith('.html') && !name.endsWith('.visual-check.html'))
-  .sort();
-
-test('all packaged HTML examples pass the real visual-check desktop gate', {
-  skip: chromePath ? false : 'Set ARCHIFY_CHROME to run the real browser regression.',
-}, async () => {
-  const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'archify-packaged-examples-'));
-  try {
-    assert.ok(packagedHtmlExamples.length > 0, 'expected at least one packaged HTML example');
-    for (const name of packagedHtmlExamples) {
-      const artifact = path.join(tmp, name);
-      fs.copyFileSync(path.join(skillRoot, 'examples', name), artifact);
-      const result = await runVisualCheck({ artifactPath: artifact, chromePath });
-      assert.equal(result.exitCode, 0, `${name}: ${JSON.stringify(result.receipt, null, 2)}`);
-      assert.equal(result.receipt.containment.status, 'pass', name);
-      assert.equal(result.receipt.containment.viewports.every((viewport) => viewport.ok), true, name);
-    }
-  } finally {
-    fs.rmSync(tmp, { recursive: true, force: true });
-  }
-});
-
-test('production showcase is readable in the real 1440 by 900 adaptive reader', {
-  skip: chromePath ? false : 'Set ARCHIFY_CHROME to run the real browser regression.',
-}, async () => {
-  const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'archify-desktop-reader-'));
-  const artifact = path.join(tmp, 'production-deployment.html');
-  try {
-    execFileSync(process.execPath, [
-      path.join(skillRoot, 'bin', 'archify.mjs'),
-      'render',
-      'architecture',
-      path.join(skillRoot, 'examples', 'production-deployment.architecture.json'),
-      artifact,
-      '--quality',
-      'showcase',
-    ], { cwd: skillRoot, encoding: 'utf8' });
-
-    for (let attempt = 1; attempt <= 3; attempt += 1) {
-      const result = await runVisualCheck({ artifactPath: artifact, chromePath });
-      assert.equal(result.exitCode, 0, `attempt ${attempt}: ${JSON.stringify(result.receipt, null, 2)}`);
-      assert.equal(result.receipt.readability.status, 'pass', `attempt ${attempt}: ${JSON.stringify(result.receipt, null, 2)}`);
-      const desktop = result.receipt.readability.viewports.find(({ width, height }) => (
-        width === DESKTOP_READABILITY_VIEWPORT.width && height === DESKTOP_READABILITY_VIEWPORT.height
-      ));
-      const darkDesktop = result.receipt.captures.screenshots.find(({ width, height, theme }) => (
-        width === DESKTOP_READABILITY_VIEWPORT.width
-        && height === DESKTOP_READABILITY_VIEWPORT.height
-        && theme === 'dark'
-      ));
-      for (const observation of [desktop, darkDesktop]) {
-        assert.ok(observation);
-        assert.equal(observation.readerWidth, 960);
-        assert.equal(observation.diagramWidth, 930);
-        assert.ok(observation.minimumProjectedNodeTextPx >= MIN_PROJECTED_NODE_TEXT_PX);
-        assert.equal(observation.minimumProjectedNodeTextDetail, 'boundary');
-        assert.equal(observation.minimumProjectedNodeText, 'AWS eu-west-1 / disaster recovery');
-        assert.equal(observation.readabilityOk, true);
-        assert.equal(observation.scrollHeight, DESKTOP_READABILITY_VIEWPORT.height);
-      }
-    }
-  } finally {
-    fs.rmSync(tmp, { recursive: true, force: true });
-  }
-});

package/skills/archify/test/diagram-guide.test.mjs

@@ -1,92 +0,0 @@
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { execFileSync } from 'node:child_process';
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const skillRoot = path.resolve(__dirname, '..');
-const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'archify-diagram-guide-'));
-
-const CASES = {
-  architecture: 'web-app.architecture.json',
-  workflow: 'agent-tool-call.workflow.json',
-  sequence: 'cache-miss-request.sequence.json',
-  dataflow: 'product-analytics.dataflow.json',
-  lifecycle: 'agent-run.lifecycle.json',
-};
-
-function render(mode, example) {
-  const output = path.join(tmp, `${mode}.html`);
-  execFileSync(process.execPath, [
-    path.join(skillRoot, `renderers/${mode}/render-${mode}.mjs`),
-    path.join(skillRoot, 'examples', example),
-    output,
-  ]);
-  return fs.readFileSync(output, 'utf8');
-}
-
-function canonicalSvg(html) {
-  return html.match(/<svg\b[\s\S]*?<\/svg>/)?.[0] || '';
-}
-
-test('all typed renderers inherit one viewer-only Diagram Guide', () => {
-  for (const [mode, example] of Object.entries(CASES)) {
-    const html = render(mode, example);
-    assert.match(html, /id="diagram-guide" hidden role="dialog" aria-modal="false" aria-labelledby="diagram-guide-title"/, mode);
-    assert.match(html, /id="btn-diagram-guide"[^>]+aria-label="Open diagram guide"[^>]+aria-haspopup="dialog"[^>]+aria-expanded="false"/, mode);
-    assert.match(html, /Archify\.guide = \(function \(\)/, mode);
-    assert.match(html, /Diagram Guide — a factual command deck over existing interactions/, mode);
-    assert.doesNotMatch(canonicalSvg(html), /diagram-guide|Archify\.guide|Explore this system/, mode);
-  }
-});
-
-test('Diagram Guide reports compiled semantic facts and honest story availability', () => {
-  const html = render('workflow', CASES.workflow);
-  assert.match(html, /svg\.querySelectorAll\('\[data-node-id\]'\)\.length/);
-  assert.match(html, /svg\.querySelectorAll\('\[data-edge-from\]\[data-edge-to\]'\)/);
-  assert.match(html, /edge\.getAttribute\('data-edge-key'\)/);
-  assert.match(html, /return Archify\.guidedViews && Number\(Archify\.guidedViews\.count\) \|\| 0/);
-  assert.match(html, /storyBtn\.disabled = views === 0/);
-  assert.match(html, /viewerCount\('viewer\.guide\.fact\.view', views\)/);
-  assert.match(html, /viewerText\('viewer\.guide\.story\.unavailable'\)/);
-});
-
-test('Diagram Guide delegates its task rows to existing production interactions', () => {
-  const html = render('architecture', CASES.architecture);
-  assert.match(html, /if \(action === 'find'\) return Archify\.finder\.open\(\)/);
-  assert.match(html, /if \(action === 'route'\) return Archify\.routeProbe\.begin\(\{ focusNode: true \}\)/);
-  assert.match(html, /if \(action === 'map'\) return Archify\.radar\.open\(\)/);
-  assert.match(html, /if \(action === 'story'\) return Archify\.guidedViews\.play\(\)/);
-  assert.match(html, /if \(action === 'present'\) return Archify\.presentation\.enter\(\)/);
-  assert.match(html, /if \(action === 'export'\) return Archify\.exportMenu\.open\(\)/);
-  assert.match(html, /if \(action === 'theme'\) return Archify\.theme\.toggle\(\)/);
-  assert.match(html, /if \(action === 'reset'\) return Archify\.view\.reset\(\)/);
-  assert.match(html, /Archify\.guidedViews\.pause\(\)/);
-  assert.match(html, /Archify\.finder\.close\(\{ restoreFocus: false \}\)/);
-  assert.match(html, /Archify\.radar\.close\(\{ restoreFocus: false \}\)/);
-  assert.match(html, /event\.stopPropagation\(\);[\s\S]+execute\(button\.getAttribute\('data-guide-action'\)\)/);
-});
-
-test('Diagram Guide is keyboard-first, mobile-contained, motion-safe, and embed-clean', () => {
-  const html = render('sequence', CASES.sequence);
-  assert.match(html, /e\.key === '\?'/);
-  assert.match(html, /Archify\.guide\.toggle\(\)/);
-  assert.match(html, /event\.key === 'ArrowRight'/);
-  assert.match(html, /event\.key === 'ArrowDown'/);
-  assert.match(html, /event\.key === 'Home'/);
-  assert.match(html, /event\.key === 'End'/);
-  assert.match(html, /event\.key === 'Escape' \|\| event\.key === '\?'/);
-  assert.match(html, /html\.setAttribute\('data-guide-open', 'true'\)/);
-  assert.match(html, /html\.getAttribute\('data-guide-open'\) === 'true'/);
-  assert.match(html, /html\[data-embed="true"\] \.diagram-guide/);
-  assert.match(html, /data-wide-diagram="true"\] \.diagram-guide/);
-  assert.match(html, /\.route-probe\[data-guide-open="true"\]/);
-  assert.match(html, /@media \(prefers-reduced-motion: reduce\)[\s\S]+\.diagram-guide/);
-  assert.match(html, /class="diagram-guide no-print"/);
-  assert.doesNotMatch(canonicalSvg(html), /data-guide-open|diagram-guide/);
-});
-
-process.on('exit', () => fs.rmSync(tmp, { recursive: true, force: true }));