create-qa-architect 5.0.7 → 5.4.3

package/lib/project-maturity.js

@@ -157,6 +157,9 @@ class ProjectMaturityDetector {
       hasTests: this.hasTests(),
       hasDependencies: this.hasDependencies(),
       hasCssFiles: this.hasCssFiles(),
+      hasShellScripts: this.hasShellScripts(),
+      shellScriptCount: this.countShellScripts(),
+      isShellProject: this.isShellProject(),
       packageJsonExists: this.packageJsonExists(),
     }
   }
@@ -266,7 +269,7 @@ class ProjectMaturityDetector {
       const deps = packageJson.dependencies || {}
       const devDeps = packageJson.devDependencies || {}
       return Object.keys(deps).length > 0 || Object.keys(devDeps).length > 0
-    } catch {
+    } catch (_error) {
       return false
     }
   }
@@ -287,6 +290,36 @@ class ProjectMaturityDetector {
     )
   }
 
+  /**
+   * Check if project has shell scripts
+   * @returns {boolean} True if shell scripts exist
+   */
+  hasShellScripts() {
+    return this.countShellScripts() > 0
+  }
+
+  /**
+   * Count shell script files
+   * @returns {number} Number of shell script files
+   */
+  countShellScripts() {
+    const extensions = ['.sh', '.bash']
+
+    return this.countFilesRecursive(this.projectPath, {
+      extensions,
+      excludeDirs: EXCLUDE_DIRECTORIES.PROJECT_MATURITY,
+      maxDepth: 4,
+    })
+  }
+
+  /**
+   * Check if this is primarily a shell script project
+   * @returns {boolean} True if shell project (has .sh files, no package.json)
+   */
+  isShellProject() {
+    return this.hasShellScripts() && !this.packageJsonExists()
+  }
+
   /**
    * Check if package.json exists
    * @returns {boolean} True if package.json exists
@@ -405,9 +438,17 @@ class ProjectMaturityDetector {
         }
       }
     } catch (error) {
-      // Silently ignore permission errors
-      if (this.verbose) {
-        console.warn(`Warning: Could not read directory ${dir}:`, error.message)
+      // Always log permission errors (affects maturity detection accuracy)
+      if (error.code === 'EACCES') {
+        console.warn(`⚠️  Permission denied: ${dir} (excluded from analysis)`)
+      } else if (error.code !== 'ENOENT') {
+        console.warn(
+          `⚠️  Could not read ${dir}: ${error.message} (${error.code})`
+        )
+      }
+
+      if (this.verbose && error.stack) {
+        console.warn(`   Stack: ${error.stack}`)
       }
     }
 
@@ -435,7 +476,7 @@ class ProjectMaturityDetector {
 
   /**
    * Generate GitHub Actions outputs for maturity detection
-   * @returns {{maturity: string, sourceCount: number, testCount: number, hasDeps: boolean, hasDocs: boolean, hasCss: boolean, requiredChecks: string, optionalChecks: string, disabledChecks: string}} GitHub Actions output format
+   * @returns {{maturity: string, sourceCount: number, testCount: number, hasDeps: boolean, hasDocs: boolean, hasCss: boolean, hasShell: boolean, shellCount: number, isShellProject: boolean, requiredChecks: string, optionalChecks: string, disabledChecks: string}} GitHub Actions output format
    */
   generateGitHubActionsOutput() {
     const maturity = this.detect()
@@ -449,6 +490,9 @@ class ProjectMaturityDetector {
       hasDeps: stats.hasDependencies,
       hasDocs: stats.hasDocumentation,
       hasCss: stats.hasCssFiles,
+      hasShell: stats.hasShellScripts,
+      shellCount: stats.shellScriptCount,
+      isShellProject: stats.isShellProject,
       requiredChecks: checks.required.join(','),
       optionalChecks: checks.optional.join(','),
       disabledChecks: checks.disabled.join(','),
@@ -472,7 +516,12 @@ class ProjectMaturityDetector {
     console.log(`  • Test files: ${stats.testFiles}`)
     console.log(`  • Documentation: ${stats.hasDocumentation ? 'Yes' : 'No'}`)
     console.log(`  • Dependencies: ${stats.hasDependencies ? 'Yes' : 'No'}`)
-    console.log(`  • CSS files: ${stats.hasCssFiles ? 'Yes' : 'No'}\n`)
+    console.log(`  • CSS files: ${stats.hasCssFiles ? 'Yes' : 'No'}`)
+    console.log(`  • Shell scripts: ${stats.shellScriptCount}`)
+    if (stats.isShellProject) {
+      console.log(`  • Project type: Shell script project`)
+    }
+    console.log()
 
     console.log('Quality Checks:')
     console.log(`  ✅ Required: ${level.checks.required.join(', ')}`)
@@ -504,6 +553,9 @@ if (require.main === module) {
     console.log(`has-deps=${output.hasDeps}`)
     console.log(`has-docs=${output.hasDocs}`)
     console.log(`has-css=${output.hasCss}`)
+    console.log(`has-shell=${output.hasShell}`)
+    console.log(`shell-count=${output.shellCount}`)
+    console.log(`is-shell-project=${output.isShellProject}`)
     console.log(`required-checks=${output.requiredChecks}`)
     console.log(`optional-checks=${output.optionalChecks}`)
     console.log(`disabled-checks=${output.disabledChecks}`)

package/lib/quality-tools-generator.js

@@ -0,0 +1,495 @@
+/**
+ * Quality Tools Generator
+ * Generates configuration for advanced quality tools:
+ * - Lighthouse CI (performance, accessibility, SEO, best practices)
+ * - Bundle size limits (size-limit)
+ * - axe-core accessibility testing
+ * - Conventional commits (commitlint)
+ * - Coverage thresholds
+ */
+
+const fs = require('fs')
+const path = require('path')
+
+/**
+ * Generate Lighthouse CI configuration
+ * @param {Object} options - Configuration options
+ * @param {string} options.projectPath - Path to project
+ * @param {boolean} options.hasThresholds - Whether to include score thresholds (Pro)
+ * @param {string} options.collectUrl - URL to audit (defaults to http://localhost:3000)
+ * @returns {string} lighthouserc.js content
+ */
+function generateLighthouseConfig(options = {}) {
+  const {
+    hasThresholds = false,
+    collectUrl = 'http://localhost:3000',
+    staticDistDir = null,
+  } = options
+
+  const collectConfig = staticDistDir
+    ? `staticDistDir: '${staticDistDir}',`
+    : `url: ['${collectUrl}'],
+      startServerCommand: 'npm run start',
+      startServerReadyPattern: 'ready|listening|started',
+      startServerReadyTimeout: 30000,`
+
+  const assertConfig = hasThresholds
+    ? `
+    assert: {
+      preset: 'lighthouse:recommended',
+      assertions: {
+        // Performance
+        'first-contentful-paint': ['warn', { maxNumericValue: 2000 }],
+        'largest-contentful-paint': ['error', { maxNumericValue: 2500 }],
+        'cumulative-layout-shift': ['error', { maxNumericValue: 0.1 }],
+        'total-blocking-time': ['warn', { maxNumericValue: 300 }],
+
+        // Categories (0-1 scale)
+        'categories:performance': ['warn', { minScore: 0.8 }],
+        'categories:accessibility': ['error', { minScore: 0.9 }],
+        'categories:best-practices': ['warn', { minScore: 0.9 }],
+        'categories:seo': ['warn', { minScore: 0.9 }],
+
+        // Allow some common warnings
+        'unsized-images': 'off',
+        'uses-responsive-images': 'off',
+      },
+    },`
+    : `
+    assert: {
+      // Basic assertions for Free tier - just warnings, no failures
+      assertions: {
+        'categories:accessibility': ['warn', { minScore: 0.7 }],
+        'categories:best-practices': ['warn', { minScore: 0.7 }],
+      },
+    },`
+
+  return `module.exports = {
+  ci: {
+    collect: {
+      ${collectConfig}
+      numberOfRuns: 3,
+    },
+    upload: {
+      target: 'temporary-public-storage',
+    },${assertConfig}
+  },
+}
+`
+}
+
+/**
+ * Generate size-limit configuration for bundle size limits
+ * @param {Object} options - Configuration options
+ * @param {string} options.projectPath - Path to project
+ * @returns {Array} size-limit config array
+ */
+function generateSizeLimitConfig(options = {}) {
+  const { projectPath = process.cwd() } = options
+
+  // Detect build output paths
+  const possibleDists = ['dist', 'build', '.next', 'out', 'public']
+  let distDir = 'dist'
+
+  for (const dir of possibleDists) {
+    if (fs.existsSync(path.join(projectPath, dir))) {
+      distDir = dir
+      break
+    }
+  }
+
+  // Detect if it's a Next.js app
+  const isNextJS =
+    fs.existsSync(path.join(projectPath, 'next.config.js')) ||
+    fs.existsSync(path.join(projectPath, 'next.config.mjs'))
+
+  if (isNextJS) {
+    return [
+      {
+        path: '.next/static/**/*.js',
+        limit: '300 kB',
+        webpack: false,
+      },
+    ]
+  }
+
+  return [
+    {
+      path: `${distDir}/**/*.js`,
+      limit: '250 kB',
+    },
+    {
+      path: `${distDir}/**/*.css`,
+      limit: '50 kB',
+    },
+  ]
+}
+
+/**
+ * Generate commitlint configuration for conventional commits
+ * @returns {string} commitlint.config.js content
+ */
+function generateCommitlintConfig() {
+  return `module.exports = {
+  extends: ['@commitlint/config-conventional'],
+  rules: {
+    // Type must be one of the conventional types
+    'type-enum': [
+      2,
+      'always',
+      [
+        'feat',     // New feature
+        'fix',      // Bug fix
+        'docs',     // Documentation only
+        'style',    // Formatting, no code change
+        'refactor', // Code change that neither fixes a bug nor adds a feature
+        'perf',     // Performance improvement
+        'test',     // Adding tests
+        'build',    // Build system or external dependencies
+        'ci',       // CI configuration
+        'chore',    // Maintenance tasks
+        'revert',   // Revert a previous commit
+      ],
+    ],
+    // Subject line max length
+    'subject-max-length': [2, 'always', 100],
+    // Subject must not be empty
+    'subject-empty': [2, 'never'],
+    // Type must not be empty
+    'type-empty': [2, 'never'],
+  },
+}
+`
+}
+
+/**
+ * Generate commit-msg hook content for commitlint
+ * @returns {string} commit-msg hook content
+ */
+function generateCommitMsgHook() {
+  return `#!/bin/sh
+npx --no -- commitlint --edit "$1"
+`
+}
+
+/**
+ * Generate vitest coverage thresholds configuration
+ * @param {Object} options - Configuration options
+ * @param {number} options.lines - Line coverage threshold (default: 70)
+ * @param {number} options.functions - Function coverage threshold (default: 70)
+ * @param {number} options.branches - Branch coverage threshold (default: 60)
+ * @param {number} options.statements - Statement coverage threshold (default: 70)
+ * @returns {Object} Coverage threshold config object
+ */
+function generateCoverageThresholds(options = {}) {
+  const { lines = 70, functions = 70, branches = 60, statements = 70 } = options
+
+  return {
+    lines,
+    functions,
+    branches,
+    statements,
+  }
+}
+
+/**
+ * Generate vitest.config.ts/js with coverage thresholds
+ * @param {Object} options - Configuration options
+ * @returns {string} vitest.config content
+ */
+function generateVitestConfigWithCoverage(options = {}) {
+  const { typescript = false, thresholds = {} } = options
+  const coverageThresholds = generateCoverageThresholds(thresholds)
+
+  const importStatement = typescript
+    ? "import { defineConfig } from 'vitest/config'"
+    : "const { defineConfig } = require('vitest/config')"
+  const exportStatement = typescript ? 'export default' : 'module.exports ='
+
+  return `${importStatement}
+
+${exportStatement} defineConfig({
+  test: {
+    globals: true,
+    environment: 'node',
+    include: ['tests/**/*.test.{js,ts}', 'src/**/*.test.{js,ts}'],
+    exclude: ['node_modules', 'dist', 'build'],
+    coverage: {
+      provider: 'v8',
+      reporter: ['text', 'json', 'html', 'lcov'],
+      exclude: [
+        'node_modules/**',
+        'tests/**',
+        '**/*.test.{js,ts}',
+        '**/*.config.{js,ts,mjs}',
+        'coverage/**',
+        'dist/**',
+        'build/**',
+      ],
+      thresholds: {
+        lines: ${coverageThresholds.lines},
+        functions: ${coverageThresholds.functions},
+        branches: ${coverageThresholds.branches},
+        statements: ${coverageThresholds.statements},
+      },
+    },
+  },
+})
+`
+}
+
+/**
+ * Generate axe-core accessibility test setup
+ * @returns {string} Accessibility test file content
+ */
+function generateAxeTestSetup() {
+  return `/**
+ * Accessibility Testing with axe-core
+ *
+ * This file sets up automated accessibility testing using axe-core.
+ * It can be run as part of your test suite or CI pipeline.
+ *
+ * Usage:
+ *   npm run test:a11y
+ *
+ * For manual testing, use the browser extension:
+ *   https://www.deque.com/axe/browser-extensions/
+ */
+
+import { describe, it, expect, beforeAll, afterAll } from 'vitest'
+
+// For component testing, use @axe-core/react or @axe-core/playwright
+// This example uses puppeteer for full page testing
+
+let browser
+
+describe('Accessibility Tests', () => {
+  beforeAll(async () => {
+    // If using puppeteer for E2E accessibility testing:
+    // const puppeteer = require('puppeteer')
+    // browser = await puppeteer.launch()
+    // const page = await browser.newPage()
+
+    // For now, this is a placeholder that passes
+    // Replace with actual implementation based on your framework
+  })
+
+  afterAll(async () => {
+    if (browser) {
+      await browser.close()
+    }
+  })
+
+  it.skip('should have no critical accessibility violations on homepage', async () => {
+    // Example with puppeteer + @axe-core/puppeteer:
+    // const { AxePuppeteer } = require('@axe-core/puppeteer')
+    // await page.goto('http://localhost:3000')
+    // const results = await new AxePuppeteer(page).analyze()
+    // expect(results.violations.filter(v => v.impact === 'critical')).toHaveLength(0)
+    expect(true).toBe(true)
+  })
+
+  it.skip('should have proper heading hierarchy', async () => {
+    // Implement heading hierarchy check
+    expect(true).toBe(true)
+  })
+
+  it.skip('should have sufficient color contrast', async () => {
+    // Implement color contrast check
+    expect(true).toBe(true)
+  })
+
+  it.skip('should have accessible form labels', async () => {
+    // Implement form label check
+    expect(true).toBe(true)
+  })
+})
+
+/**
+ * Helper function to run axe-core on a page
+ * @param {Object} page - Puppeteer/Playwright page object
+ * @returns {Promise<Object>} axe results
+ */
+export async function runAxeOnPage(page) {
+  // Inject axe-core into the page
+  await page.addScriptTag({
+    path: require.resolve('axe-core'),
+  })
+
+  // Run axe
+  const results = await page.evaluate(async () => {
+    // eslint-disable-next-line no-undef
+    return await axe.run()
+  })
+
+  return results
+}
+
+/**
+ * Filter axe violations by impact level
+ * @param {Array} violations - axe violations array
+ * @param {Array} levels - Impact levels to include ['critical', 'serious', 'moderate', 'minor']
+ * @returns {Array} Filtered violations
+ */
+export function filterByImpact(violations, levels = ['critical', 'serious']) {
+  return violations.filter(v => levels.includes(v.impact))
+}
+`
+}
+
+/**
+ * Write Lighthouse CI config to project
+ * @param {string} projectPath - Path to project
+ * @param {Object} options - Options for config generation
+ */
+function writeLighthouseConfig(projectPath, options = {}) {
+  const configPath = path.join(projectPath, 'lighthouserc.js')
+  const config = generateLighthouseConfig(options)
+  fs.writeFileSync(configPath, config)
+  return configPath
+}
+
+/**
+ * Write size-limit config to package.json
+ * @param {string} projectPath - Path to project
+ * @param {Object} options - Options for config generation
+ */
+function writeSizeLimitConfig(projectPath, options = {}) {
+  const packageJsonPath = path.join(projectPath, 'package.json')
+  if (!fs.existsSync(packageJsonPath)) {
+    throw new Error('package.json not found')
+  }
+
+  const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'))
+  packageJson['size-limit'] = generateSizeLimitConfig({
+    projectPath,
+    ...options,
+  })
+  fs.writeFileSync(packageJsonPath, JSON.stringify(packageJson, null, 2) + '\n')
+  return packageJsonPath
+}
+
+/**
+ * Write commitlint config to project
+ * @param {string} projectPath - Path to project
+ */
+function writeCommitlintConfig(projectPath) {
+  const configPath = path.join(projectPath, 'commitlint.config.js')
+  const config = generateCommitlintConfig()
+  fs.writeFileSync(configPath, config)
+  return configPath
+}
+
+/**
+ * Write commit-msg hook for commitlint
+ * @param {string} projectPath - Path to project
+ */
+function writeCommitMsgHook(projectPath) {
+  const huskyDir = path.join(projectPath, '.husky')
+  if (!fs.existsSync(huskyDir)) {
+    fs.mkdirSync(huskyDir, { recursive: true })
+  }
+
+  const hookPath = path.join(huskyDir, 'commit-msg')
+  const hook = generateCommitMsgHook()
+  fs.writeFileSync(hookPath, hook)
+  fs.chmodSync(hookPath, 0o755)
+  return hookPath
+}
+
+/**
+ * Write axe test setup to project
+ * @param {string} projectPath - Path to project
+ * @param {Object} options - Options for test generation
+ */
+function writeAxeTestSetup(projectPath, options = {}) {
+  const testsDir = path.join(projectPath, 'tests')
+  if (!fs.existsSync(testsDir)) {
+    fs.mkdirSync(testsDir, { recursive: true })
+  }
+
+  const testPath = path.join(testsDir, 'accessibility.test.js')
+  const content = generateAxeTestSetup(options)
+  fs.writeFileSync(testPath, content)
+  return testPath
+}
+
+/**
+ * Get dev dependencies for quality tools
+ * @param {Object} features - Which features to include
+ * @returns {Object} Dev dependencies object
+ */
+function getQualityToolsDependencies(features = {}) {
+  const deps = {}
+
+  if (features.lighthouse) {
+    deps['@lhci/cli'] = '^0.14.0'
+  }
+
+  if (features.sizeLimit) {
+    deps['size-limit'] = '^11.0.0'
+    deps['@size-limit/file'] = '^11.0.0'
+  }
+
+  if (features.commitlint) {
+    deps['@commitlint/cli'] = '^19.0.0'
+    deps['@commitlint/config-conventional'] = '^19.0.0'
+  }
+
+  if (features.axeCore) {
+    deps['axe-core'] = '^4.10.0'
+  }
+
+  return deps
+}
+
+/**
+ * Get npm scripts for quality tools
+ * @param {Object} features - Which features to include
+ * @returns {Object} Scripts object
+ */
+function getQualityToolsScripts(features = {}) {
+  const scripts = {}
+
+  if (features.lighthouse) {
+    scripts['lighthouse:ci'] = 'lhci autorun'
+    scripts['lighthouse:upload'] = 'lhci upload'
+  }
+
+  if (features.sizeLimit) {
+    scripts['size'] = 'size-limit'
+    scripts['size:why'] = 'size-limit --why'
+  }
+
+  if (features.axeCore) {
+    scripts['test:a11y'] = 'vitest run tests/accessibility.test.js'
+  }
+
+  if (features.coverage) {
+    scripts['test:coverage'] = 'vitest run --coverage'
+    scripts['test:coverage:check'] =
+      'vitest run --coverage --coverage.thresholds.lines=70 --coverage.thresholds.functions=70'
+  }
+
+  return scripts
+}
+
+module.exports = {
+  // Config generators
+  generateLighthouseConfig,
+  generateSizeLimitConfig,
+  generateCommitlintConfig,
+  generateCommitMsgHook,
+  generateCoverageThresholds,
+  generateVitestConfigWithCoverage,
+  generateAxeTestSetup,
+  // Writers
+  writeLighthouseConfig,
+  writeSizeLimitConfig,
+  writeCommitlintConfig,
+  writeCommitMsgHook,
+  writeAxeTestSetup,
+  // Helpers
+  getQualityToolsDependencies,
+  getQualityToolsScripts,
+}