kodelyth-ecc 1.8.0 → 1.8.2

package/tests/hooks/branch-name-check.test.js

@@ -1,184 +0,0 @@
-'use strict'
-
-/**
- * Tests for scripts/hooks/branch-name-check.js
- * Tests the pure logic functions directly.
- */
-
-const assert = require('node:assert/strict')
-const { test } = require('node:test')
-
-// ── Replicate pure functions ───────────────────────────────────────────────
-
-const VALID_PREFIXES = [
-  'feat', 'fix', 'chore', 'docs', 'refactor',
-  'test', 'perf', 'ci', 'hotfix', 'release',
-  'experiment', 'spike', 'wip',
-]
-
-const BRANCH_REGEX = new RegExp(
-  `^(${VALID_PREFIXES.join('|')})\\/[a-z0-9][a-z0-9-._/]{1,60}$`
-)
-
-const PROTECTED_BRANCHES = ['main', 'master', 'develop', 'staging', 'production']
-
-function isValid(name) {
-  return BRANCH_REGEX.test(name)
-}
-
-function isProtected(name) {
-  return PROTECTED_BRANCHES.includes(name)
-}
-
-function extractBranchName(command) {
-  const checkoutMatch = command.match(/git\s+checkout\s+(?:-b|-B)\s+([^\s]+)/)
-  if (checkoutMatch) return checkoutMatch[1]
-
-  const switchMatch = command.match(/git\s+switch\s+(?:-c|-C)\s+([^\s]+)/)
-  if (switchMatch) return switchMatch[1]
-
-  return null
-}
-
-// ── isValid ────────────────────────────────────────────────────────────────
-
-test('isValid — accepts feat/add-auth', () => {
-  assert.equal(isValid('feat/add-auth'), true)
-})
-
-test('isValid — accepts fix/null-pointer-checkout', () => {
-  assert.equal(isValid('fix/null-pointer-checkout'), true)
-})
-
-test('isValid — accepts hotfix/payment-double-charge', () => {
-  assert.equal(isValid('hotfix/payment-double-charge'), true)
-})
-
-test('isValid — accepts release/2.1.0', () => {
-  assert.equal(isValid('release/2.1.0'), true)
-})
-
-test('isValid — accepts chore/upgrade-typescript-5', () => {
-  assert.equal(isValid('chore/upgrade-typescript-5'), true)
-})
-
-test('isValid — accepts wip/experiments', () => {
-  assert.equal(isValid('wip/experiments'), true)
-})
-
-test('isValid — rejects no-prefix branch', () => {
-  assert.equal(isValid('my-feature'), false)
-})
-
-test('isValid — rejects uppercase type', () => {
-  assert.equal(isValid('FEAT/add-auth'), false)
-})
-
-test('isValid — rejects unknown type prefix', () => {
-  assert.equal(isValid('feature/add-auth'), false)
-})
-
-test('isValid — rejects empty description', () => {
-  assert.equal(isValid('feat/'), false)
-})
-
-test('isValid — rejects description starting with dash', () => {
-  assert.equal(isValid('feat/-bad-start'), false)
-})
-
-test('isValid — rejects bare branch name with no slash', () => {
-  assert.equal(isValid('my-branch'), false)
-})
-
-// ── isProtected ────────────────────────────────────────────────────────────
-
-test('isProtected — main is protected', () => {
-  assert.equal(isProtected('main'), true)
-})
-
-test('isProtected — master is protected', () => {
-  assert.equal(isProtected('master'), true)
-})
-
-test('isProtected — staging is protected', () => {
-  assert.equal(isProtected('staging'), true)
-})
-
-test('isProtected — feat/something is not protected', () => {
-  assert.equal(isProtected('feat/something'), false)
-})
-
-// ── extractBranchName ──────────────────────────────────────────────────────
-
-test('extractBranchName — git checkout -b', () => {
-  assert.equal(
-    extractBranchName('git checkout -b feat/add-login'),
-    'feat/add-login'
-  )
-})
-
-test('extractBranchName — git checkout -B', () => {
-  assert.equal(
-    extractBranchName('git checkout -B hotfix/urgent-fix'),
-    'hotfix/urgent-fix'
-  )
-})
-
-test('extractBranchName — git switch -c', () => {
-  assert.equal(
-    extractBranchName('git switch -c fix/broken-button'),
-    'fix/broken-button'
-  )
-})
-
-test('extractBranchName — git switch -C', () => {
-  assert.equal(
-    extractBranchName('git switch -C release/1.2.0'),
-    'release/1.2.0'
-  )
-})
-
-test('extractBranchName — unrelated command returns null', () => {
-  assert.equal(
-    extractBranchName('git status'),
-    null
-  )
-})
-
-test('extractBranchName — git pull returns null', () => {
-  assert.equal(
-    extractBranchName('git pull origin main'),
-    null
-  )
-})
-
-test('extractBranchName — git commit returns null', () => {
-  assert.equal(
-    extractBranchName('git commit -m "fix: broken auth"'),
-    null
-  )
-})
-
-// ── End-to-end logic ──────────────────────────────────────────────────────
-
-test('should block — extracts invalid branch name from command', () => {
-  const command = 'git checkout -b my-random-branch'
-  const name = extractBranchName(command)
-  assert.ok(name)
-  assert.equal(isProtected(name), false)
-  assert.equal(isValid(name), false)
-})
-
-test('should pass — extracts valid branch name from command', () => {
-  const command = 'git checkout -b feat/user-authentication'
-  const name = extractBranchName(command)
-  assert.ok(name)
-  assert.equal(isValid(name), true)
-})
-
-test('should pass — protected branch always passes', () => {
-  const command = 'git checkout -b main'
-  const name = extractBranchName(command)
-  assert.ok(name)
-  assert.equal(isProtected(name), true)
-})

package/tests/hooks/smart-suggest.test.js

@@ -1,149 +0,0 @@
-'use strict'
-
-/**
- * Tests for scripts/hooks/smart-suggest.js
- * Tests signal detection logic.
- */
-
-const assert = require('node:assert/strict')
-const { test } = require('node:test')
-
-// ── Replicate detection logic ─────────────────────────────────────────────
-
-const SUGGESTION_RULES = [
-  {
-    signals: ['file_path', 'new_string', 'old_string', 'content'],
-    keywords: ['.ts', '.tsx', '.js', '.py', '.go', '.rs', '.java', '.kt'],
-    notKeywords: ['test', 'spec', '__test__'],
-    suggestion: 'use code-reviewer',
-    reason: 'Code was written — review before committing',
-  },
-  {
-    signals: ['error', 'bug', 'crash', 'exception', 'undefined', 'null', 'failed', 'broken'],
-    suggestion: 'use debug-detective',
-    reason: 'A bug or error is present — trace it to root cause',
-  },
-  {
-    signals: ['pull_request', 'gh pr create', 'created pr', 'pull request'],
-    suggestion: 'use security-reviewer',
-    reason: 'PR was created — run a security check before merge',
-  },
-  {
-    signals: ['build succeeded', 'compiled successfully', 'build passed', 'tests passed'],
-    suggestion: 'use e2e-runner',
-    reason: 'Build passed — verify critical user flows with E2E tests',
-  },
-  {
-    signals: ['route', 'endpoint', 'controller', 'handler', '/api/'],
-    suggestion: 'use api-guardian',
-    reason: 'API was modified — check for breaking changes',
-  },
-  {
-    signals: ['i want to', 'i need to', 'how do i', 'help me build', 'implement'],
-    suggestion: 'use pair-programmer',
-    reason: 'Planning a feature — think it through before coding',
-  },
-  {
-    signals: ['.tsx', '.jsx', 'component', '<button', '<form', '<input', 'onclick', 'onchange'],
-    suggestion: 'use ux-reviewer',
-    reason: 'UI was built — check UX and accessibility',
-  },
-  {
-    signals: ['upgrade', 'migrate', 'version', 'breaking change', 'deprecated'],
-    suggestion: 'use migration-guide',
-    reason: 'Migration or upgrade detected — get a phased plan',
-  },
-]
-
-function detectSuggestion(content) {
-  const lower = content.toLowerCase()
-
-  for (const rule of SUGGESTION_RULES) {
-    const hasSignal = (rule.signals || []).some(s => lower.includes(s.toLowerCase()))
-    const hasKeyword = !rule.keywords || rule.keywords.some(k => lower.includes(k.toLowerCase()))
-    const hasNoExclusion = !rule.notKeywords || !rule.notKeywords.some(k => lower.includes(k.toLowerCase()))
-
-    if (hasSignal && hasKeyword && hasNoExclusion) {
-      return { suggestion: rule.suggestion, reason: rule.reason }
-    }
-  }
-
-  return null
-}
-
-// ── detectSuggestion ──────────────────────────────────────────────────────
-
-test('detects code edit → suggests code-reviewer', () => {
-  const result = detectSuggestion('file_path: src/auth.ts new_string: ...')
-  assert.ok(result)
-  assert.equal(result.suggestion, 'use code-reviewer')
-})
-
-test('detects error → suggests debug-detective', () => {
-  const result = detectSuggestion('TypeError: Cannot read property of undefined')
-  assert.ok(result)
-  assert.equal(result.suggestion, 'use debug-detective')
-})
-
-test('detects crash → suggests debug-detective', () => {
-  const result = detectSuggestion('app crash on startup')
-  assert.ok(result)
-  assert.equal(result.suggestion, 'use debug-detective')
-})
-
-test('detects PR creation → suggests security-reviewer', () => {
-  const result = detectSuggestion('gh pr create --title "feat: add auth"')
-  assert.ok(result)
-  assert.equal(result.suggestion, 'use security-reviewer')
-})
-
-test('detects build passed → suggests e2e-runner', () => {
-  const result = detectSuggestion('build succeeded — all 24 tests passed')
-  assert.ok(result)
-  assert.equal(result.suggestion, 'use e2e-runner')
-})
-
-test('detects API route → suggests api-guardian', () => {
-  const result = detectSuggestion('created new endpoint /api/users')
-  assert.ok(result)
-  assert.equal(result.suggestion, 'use api-guardian')
-})
-
-test('detects feature planning → suggests pair-programmer', () => {
-  const result = detectSuggestion('i want to build a notification system')
-  assert.ok(result)
-  assert.equal(result.suggestion, 'use pair-programmer')
-})
-
-test('detects UI component → suggests ux-reviewer', () => {
-  const result = detectSuggestion('created a new component with <button onClick...')
-  assert.ok(result)
-  assert.equal(result.suggestion, 'use ux-reviewer')
-})
-
-test('detects migration → suggests migration-guide', () => {
-  const result = detectSuggestion('upgrade from Next.js 13 to 15 — breaking change')
-  assert.ok(result)
-  assert.equal(result.suggestion, 'use migration-guide')
-})
-
-test('returns null for unrelated content', () => {
-  const result = detectSuggestion('hello world nothing special here')
-  assert.equal(result, null)
-})
-
-test('does not suggest code-reviewer for test file edits', () => {
-  const result = detectSuggestion('file_path: src/auth.test.ts new_string: ...')
-  // test keyword exclusion means code-reviewer is skipped
-  // but another rule might match — we just verify it is NOT code-reviewer
-  if (result) {
-    assert.notEqual(result.suggestion, 'use code-reviewer')
-  }
-})
-
-test('returns first matching rule — error takes priority over API route', () => {
-  const result = detectSuggestion('error in the route handler: exception thrown')
-  assert.ok(result)
-  // error rule comes before route rule in the list
-  assert.equal(result.suggestion, 'use debug-detective')
-})

package/tests/hooks/test-reminder.test.js

@@ -1,181 +0,0 @@
-'use strict'
-
-/**
- * Tests for scripts/hooks/test-reminder.js
- * Tests the pure logic functions directly without spawning a process.
- */
-
-const assert = require('node:assert/strict')
-const { test } = require('node:test')
-
-// ── Extract pure functions for testing ────────────────────────────────────────
-
-const TEST_FILE_PATTERNS = [
-  /\.(test|spec)\.(ts|tsx|js|jsx|mjs)$/,
-  /\.(test|spec)\.py$/,
-  /__tests__/,
-  /\/tests?\//,
-  /_test\.go$/,
-  /_spec\.rb$/,
-  /\.test\.rs$/,
-  /Test\.java$/,
-  /Spec\.kt$/,
-]
-
-const CODE_FILE_PATTERNS = [
-  /\.(ts|tsx|js|jsx|mjs)$/,
-  /\.py$/,
-  /\.go$/,
-  /\.rs$/,
-  /\.java$/,
-  /\.kt$/,
-  /\.rb$/,
-  /\.cs$/,
-  /\.cpp$|\.cc$|\.cxx$/,
-]
-
-const IGNORED_PATHS = [
-  /node_modules/,
-  /\.git\//,
-  /dist\//,
-  /build\//,
-  /coverage\//,
-  /\.next\//,
-  /\.nuxt\//,
-  /migrations\//,
-  /generated\//,
-]
-
-function isTestFile(filePath) {
-  return TEST_FILE_PATTERNS.some(p => p.test(filePath))
-}
-
-function isIgnored(filePath) {
-  return IGNORED_PATHS.some(p => p.test(filePath))
-}
-
-function isCodeFile(filePath) {
-  return CODE_FILE_PATTERNS.some(p => p.test(filePath)) && !isTestFile(filePath)
-}
-
-// ── isTestFile ─────────────────────────────────────────────────────────────
-
-test('isTestFile — detects .test.ts', () => {
-  assert.equal(isTestFile('src/utils.test.ts'), true)
-})
-
-test('isTestFile — detects .spec.js', () => {
-  assert.equal(isTestFile('src/api.spec.js'), true)
-})
-
-test('isTestFile — detects __tests__ directory', () => {
-  assert.equal(isTestFile('src/__tests__/auth.ts'), true)
-})
-
-test('isTestFile — detects /tests/ directory', () => {
-  assert.equal(isTestFile('tests/hook.test.js'), true)
-})
-
-test('isTestFile — detects _test.go', () => {
-  assert.equal(isTestFile('server/handler_test.go'), true)
-})
-
-test('isTestFile — detects Test.java', () => {
-  assert.equal(isTestFile('src/AuthServiceTest.java'), true)
-})
-
-test('isTestFile — detects Spec.kt', () => {
-  assert.equal(isTestFile('src/UserSpec.kt'), true)
-})
-
-test('isTestFile — does not flag regular .ts file', () => {
-  assert.equal(isTestFile('src/utils.ts'), false)
-})
-
-test('isTestFile — does not flag regular .py file', () => {
-  assert.equal(isTestFile('src/service.py'), false)
-})
-
-// ── isCodeFile ─────────────────────────────────────────────────────────────
-
-test('isCodeFile — detects .ts', () => {
-  assert.equal(isCodeFile('src/auth.ts'), true)
-})
-
-test('isCodeFile — detects .py', () => {
-  assert.equal(isCodeFile('app/service.py'), true)
-})
-
-test('isCodeFile — detects .go', () => {
-  assert.equal(isCodeFile('server/handler.go'), true)
-})
-
-test('isCodeFile — detects .rs', () => {
-  assert.equal(isCodeFile('src/main.rs'), true)
-})
-
-test('isCodeFile — detects .java', () => {
-  assert.equal(isCodeFile('src/UserService.java'), true)
-})
-
-test('isCodeFile — does not classify a test file as code', () => {
-  assert.equal(isCodeFile('src/auth.test.ts'), false)
-})
-
-test('isCodeFile — does not classify .md as code', () => {
-  assert.equal(isCodeFile('README.md'), false)
-})
-
-test('isCodeFile — does not classify .json as code', () => {
-  assert.equal(isCodeFile('config.json'), false)
-})
-
-// ── isIgnored ──────────────────────────────────────────────────────────────
-
-test('isIgnored — ignores node_modules', () => {
-  assert.equal(isIgnored('node_modules/lodash/index.js'), true)
-})
-
-test('isIgnored — ignores .next build output', () => {
-  assert.equal(isIgnored('.next/server/app.js'), true)
-})
-
-test('isIgnored — ignores dist/', () => {
-  assert.equal(isIgnored('dist/bundle.js'), true)
-})
-
-test('isIgnored — ignores generated/', () => {
-  assert.equal(isIgnored('generated/graphql.ts'), true)
-})
-
-test('isIgnored — does not ignore src/', () => {
-  assert.equal(isIgnored('src/auth.ts'), false)
-})
-
-// ── Logic: should reminder fire? ───────────────────────────────────────────
-
-test('reminder fires when code edited with no test touched', () => {
-  const files = ['src/auth.ts']
-  const codeFiles = files.filter(f => !isIgnored(f) && isCodeFile(f))
-  const testFiles = files.filter(f => !isIgnored(f) && isTestFile(f))
-  assert.equal(codeFiles.length > 0 && testFiles.length === 0, true)
-})
-
-test('reminder does not fire when test file also touched', () => {
-  const files = ['src/auth.ts', 'src/auth.test.ts']
-  const codeFiles = files.filter(f => !isIgnored(f) && isCodeFile(f))
-  const testFiles = files.filter(f => !isIgnored(f) && isTestFile(f))
-  assert.equal(codeFiles.length > 0 && testFiles.length === 0, false)
-})
-
-test('reminder does not fire when only markdown edited', () => {
-  const files = ['README.md']
-  const codeFiles = files.filter(f => !isIgnored(f) && isCodeFile(f))
-  assert.equal(codeFiles.length, 0)
-})
-
-test('reminder does not fire for ignored paths', () => {
-  const files = ['node_modules/express/index.js']
-  const codeFiles = files.filter(f => !isIgnored(f) && isCodeFile(f))
-  assert.equal(codeFiles.length, 0)
-})

package/tests/mcp/catalog.test.js

@@ -1,98 +0,0 @@
-// Tests for scripts/mcp/catalog.js — read-only loader for ECC catalog files.
-'use strict';
-
-const test   = require('node:test');
-const assert = require('node:assert/strict');
-
-const catalog = require('../../scripts/mcp/catalog');
-
-test('parseFrontmatter parses simple key/value', () => {
-  const { meta, body } = catalog.parseFrontmatter('---\nname: foo\ndescription: bar\n---\nhello');
-  assert.equal(meta.name, 'foo');
-  assert.equal(meta.description, 'bar');
-  assert.equal(body, 'hello');
-});
-
-test('parseFrontmatter handles folded scalar (>)', () => {
-  const raw = '---\nname: x\ndescription: >\n  line one\n  line two\n---\n# body';
-  const { meta, body } = catalog.parseFrontmatter(raw);
-  assert.equal(meta.name, 'x');
-  assert.match(meta.description, /line one/);
-  assert.match(meta.description, /line two/);
-  assert.equal(body, '# body');
-});
-
-test('parseFrontmatter handles missing frontmatter', () => {
-  const { meta, body } = catalog.parseFrontmatter('# just markdown');
-  assert.deepEqual(meta, {});
-  assert.equal(body, '# just markdown');
-});
-
-test('loadAgents finds at least one known agent', () => {
-  const agents = catalog.loadAgents();
-  assert.ok(Array.isArray(agents));
-  assert.ok(agents.length >= 60, `expected >=60 agents, got ${agents.length}`);
-  const dd = agents.find(a => a.name === 'debug-detective');
-  assert.ok(dd, 'debug-detective agent must be present');
-  assert.ok(dd.description.length > 0);
-  assert.ok(dd.body.length > 100);
-});
-
-test('loadAgents includes the new devil-mode crew', () => {
-  const agents = catalog.loadAgents();
-  const names = new Set(agents.map(a => a.name));
-  for (const expected of [
-    'prompt-injection-hunter',
-    'supply-chain-auditor',
-    'jailbreak-tester',
-    'backdoor-hunter',
-    'chaos-engineer',
-  ]) {
-    assert.ok(names.has(expected), `missing devil-mode agent: ${expected}`);
-  }
-});
-
-test('loadSkills returns a populated list with descriptions', () => {
-  const skills = catalog.loadSkills();
-  assert.ok(skills.length >= 100, `expected >=100 skills, got ${skills.length}`);
-  const ae = skills.find(s => s.name === 'agentic-engineering');
-  assert.ok(ae, 'agentic-engineering skill must be present');
-  assert.ok(ae.description.length > 0);
-});
-
-test('loadCommands returns commands and supports findCommand with leading slash', () => {
-  const cmds = catalog.loadCommands();
-  assert.ok(cmds.length >= 50);
-  const found1 = catalog.findCommand('code-review');
-  const found2 = catalog.findCommand('/code-review');
-  assert.ok(found1 && found2 && found1.name === found2.name);
-});
-
-test('loadAllRules returns the intent routing rule', () => {
-  const rules = catalog.loadAllRules();
-  const r = rules.find(x => x.name === 'agent-intent-routing');
-  assert.ok(r, 'agent-intent-routing rule must be present');
-  assert.ok(r.body.length > 500);
-});
-
-test('loadBundles returns the three power bundles', () => {
-  const bundles = catalog.loadBundles();
-  const names = new Set(bundles.map(b => b.name));
-  assert.ok(names.has('indie-hacker'));
-  assert.ok(names.has('red-team'));
-  assert.ok(names.has('enterprise'));
-});
-
-test('stats returns expected fields', () => {
-  const s = catalog.stats();
-  assert.ok(typeof s.agents === 'number' && s.agents > 0);
-  assert.ok(typeof s.skills === 'number' && s.skills > 0);
-  assert.ok(typeof s.commands === 'number' && s.commands > 0);
-  assert.ok(typeof s.rules === 'number' && s.rules > 0);
-  assert.ok(typeof s.bundles === 'number');
-  assert.ok(typeof s.root === 'string');
-});
-
-test('findAgent returns null for unknown name', () => {
-  assert.equal(catalog.findAgent('this-agent-does-not-exist'), null);
-});