kodelyth-ecc 1.2.0

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

@@ -0,0 +1,184 @@
+'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

@@ -0,0 +1,149 @@
+'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

@@ -0,0 +1,181 @@
+'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/run-all.js

@@ -0,0 +1,55 @@
+#!/usr/bin/env node
+'use strict'
+
+/**
+ * Kodelyth ECC — Test Runner
+ * Runs all test files using Node.js built-in test runner.
+ */
+
+const { execSync } = require('node:child_process')
+const path = require('node:path')
+const fs = require('node:fs')
+
+const ROOT = path.resolve(__dirname, '..')
+
+function findTests(dir) {
+  const results = []
+  if (!fs.existsSync(dir)) return results
+
+  for (const entry of fs.readdirSync(dir, { withFileTypes: true })) {
+    const full = path.join(dir, entry.name)
+    if (entry.isDirectory()) {
+      results.push(...findTests(full))
+    } else if (entry.name.endsWith('.test.js')) {
+      results.push(full)
+    }
+  }
+  return results
+}
+
+const testFiles = findTests(path.join(ROOT, 'tests'))
+
+if (testFiles.length === 0) {
+  console.log('No test files found.')
+  process.exit(0)
+}
+
+console.log(`\nKodelyth ECC — running ${testFiles.length} test file(s)\n`)
+
+let failed = false
+
+for (const file of testFiles) {
+  const rel = path.relative(ROOT, file)
+  try {
+    execSync(`node --test ${file}`, { stdio: 'inherit' })
+  } catch {
+    failed = true
+  }
+}
+
+if (failed) {
+  console.error('\nSome tests failed.')
+  process.exit(1)
+} else {
+  console.log('\nAll tests passed.')
+}