@programinglive/commiter 1.0.11 → 1.1.0

package/SECURITY.md

@@ -1,30 +1,30 @@
-# Security Policy
-
-## Reporting a Vulnerability
-
-If you discover a security vulnerability in `@programinglive/commiter`, please report it responsibly.
-
-Send your report to: [security@programinglive.com](mailto:security@programinglive.com).
-
-Please include:
-- A detailed description of the vulnerability
-- Steps to reproduce the issue
-- Any potential impact
-- Optional: Suggested mitigation or patch
-
-We aim to respond within **48 hours** and will keep you informed of our progress.
-
-## Supported Versions
-
-We support the latest released version of the package. Please ensure you are running the most recent version before reporting issues.
-
-## Disclosure Policy
-
-We follow a coordinated disclosure process:
-1. Acknowledge receipt of your report within 48 hours
-2. Assess the vulnerability and determine severity
-3. Fix the issue and prepare a release
-4. Credit the reporter (if desired)
-5. Publish a security advisory detailing the fix
-
-Thank you for helping keep our project secure!
+# Security Policy
+
+## Reporting a Vulnerability
+
+If you discover a security vulnerability in `@programinglive/commiter`, please report it responsibly.
+
+Send your report to: [security@programinglive.com](mailto:security@programinglive.com).
+
+Please include:
+- A detailed description of the vulnerability
+- Steps to reproduce the issue
+- Any potential impact
+- Optional: Suggested mitigation or patch
+
+We aim to respond within **48 hours** and will keep you informed of our progress.
+
+## Supported Versions
+
+We support the latest released version of the package. Please ensure you are running the most recent version before reporting issues.
+
+## Disclosure Policy
+
+We follow a coordinated disclosure process:
+1. Acknowledge receipt of your report within 48 hours
+2. Assess the vulnerability and determine severity
+3. Fix the issue and prepare a release
+4. Credit the reporter (if desired)
+5. Publish a security advisory detailing the fix
+
+Thank you for helping keep our project secure!

package/commitlint.config.cjs

@@ -1,4 +1,4 @@
-module.exports = {
-  extends: ['@commitlint/config-conventional'],
-  ignores: [(message) => message.startsWith('chore(release):')]
-};
+module.exports = {
+  extends: ['@commitlint/config-conventional'],
+  ignores: [(message) => message.startsWith('chore(release):')]
+};

package/index.js

@@ -1,148 +1,148 @@
-#!/usr/bin/env node
-
-/**
- * Commiter - Commit convention tooling for standard-version releases
- * 
- * This package helps enforce conventional commits and automate releases
- * with beautiful changelogs featuring icons for each commit type.
- */
-
-const { execSync } = require('child_process');
-const fs = require('fs');
-const path = require('path');
-
-function ensureSafeTestScript(scripts = {}) {
-  const defaultFailingTestScript = 'echo "Error: no test specified" && exit 1';
-  if (!scripts.test || scripts.test === defaultFailingTestScript) {
-    scripts.test = 'echo "No tests specified"';
-  }
-  return scripts;
-}
-
-function setupCommiter() {
-  console.log('🚀 Setting up Commiter...\n');
-
-  // Check if package.json exists
-  if (!fs.existsSync('package.json')) {
-    console.error('❌ Error: package.json not found. Please run this in a Node.js project directory.');
-    process.exit(1);
-  }
-
-  try {
-    // Install dependencies
-    console.log('📦 Installing dependencies...');
-    execSync('npm install --save-dev standard-version @commitlint/cli @commitlint/config-conventional husky', {
-      stdio: 'inherit'
-    });
-
-    // Read package.json
-    const packageJsonPath = path.join(process.cwd(), 'package.json');
-    const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-
-    packageJson.scripts = ensureSafeTestScript(packageJson.scripts || {});
-    // Add scripts
-    packageJson.scripts.prepare = 'husky';
-    packageJson.scripts.release = 'node scripts/release.js';
-    packageJson.scripts['release:major'] = 'node scripts/release.js major';
-    packageJson.scripts['release:minor'] = 'node scripts/release.js minor';
-    packageJson.scripts['release:patch'] = 'node scripts/release.js patch';
-
-    // Add standard-version config
-    packageJson['standard-version'] = {
-      releaseCommitMessageFormat: 'chore(release): {{currentTag}} 🚀',
-      types: [
-        { type: 'feat', section: '✨ Features' },
-        { type: 'fix', section: '🐛 Bug Fixes' },
-        { type: 'perf', section: '⚡ Performance' },
-        { type: 'refactor', section: '♻️ Refactors' },
-        { type: 'docs', section: '📝 Documentation' },
-        { type: 'style', section: '💄 Styles' },
-        { type: 'test', section: '✅ Tests' },
-        { type: 'build', section: '🏗️ Build System' },
-        { type: 'ci', section: '👷 Continuous Integration' },
-        { type: 'chore', section: '🧹 Chores' },
-        { type: 'revert', section: '⏪ Reverts' }
-      ]
-    };
-
-    // Write updated package.json
-    fs.writeFileSync(packageJsonPath, JSON.stringify(packageJson, null, 2) + '\n');
-
-    // Create release helper script
-    console.log('🛠️  Creating release helper script...');
-    const releaseScriptDir = path.join(process.cwd(), 'scripts');
-    if (!fs.existsSync(releaseScriptDir)) {
-      fs.mkdirSync(releaseScriptDir, { recursive: true });
-    }
-
-    const releaseScriptPath = path.join(releaseScriptDir, 'release.js');
-    const releaseScriptSource = path.join(__dirname, 'scripts', 'release.js');
-    const releaseScriptContent = fs.readFileSync(releaseScriptSource, 'utf8');
-
-    fs.writeFileSync(releaseScriptPath, releaseScriptContent + '\n');
-    try {
-      fs.chmodSync(releaseScriptPath, 0o755);
-    } catch (error) {
-      // Ignore chmod errors on non-POSIX systems
-    }
-
-    // Determine commitlint config format based on project module type
-    const isEsmProject = packageJson.type === 'module';
-    const commitlintConfigFile = isEsmProject ? 'commitlint.config.js' : 'commitlint.config.cjs';
-    const commitlintConfigContent = isEsmProject
-      ? `export default {
-  extends: ['@commitlint/config-conventional'],
-  ignores: [(message) => message.startsWith('chore(release):')]
-}\n`
-      : `module.exports = {
-  extends: ['@commitlint/config-conventional'],
-  ignores: [(message) => message.startsWith('chore(release):')]
-}\n`;
-
-    const legacyCommitlintConfigFile = isEsmProject ? 'commitlint.config.cjs' : 'commitlint.config.js';
-    if (fs.existsSync(legacyCommitlintConfigFile)) {
-      fs.rmSync(legacyCommitlintConfigFile);
-    }
-
-    console.log(`⚙️  Creating commitlint config (${commitlintConfigFile})...`);
-    fs.writeFileSync(commitlintConfigFile, commitlintConfigContent);
-
-    // Initialize Husky
-    console.log('🐶 Setting up Husky...');
-    execSync('npx husky init', { stdio: 'inherit' });
-
-    // Create commit-msg hook
-    const huskyDir = path.join(process.cwd(), '.husky');
-    if (!fs.existsSync(huskyDir)) {
-      fs.mkdirSync(huskyDir, { recursive: true });
-    }
-
-    const commitMsgHook = `#!/usr/bin/env sh
-
-npx --no -- commitlint --edit "$1"
-`;
-    fs.writeFileSync(path.join(huskyDir, 'commit-msg'), commitMsgHook);
-    fs.chmodSync(path.join(huskyDir, 'commit-msg'), 0o755);
-
-    console.log('\n✅ Commiter setup complete!\n');
-    console.log('📚 Available commands:');
-    console.log('  npm run release major  - Create a major release (1.0.0 → 2.0.0)');
-    console.log('  npm run release minor  - Create a minor release (1.0.0 → 1.1.0)');
-    console.log('  npm run release patch  - Create a patch release (1.0.0 → 1.0.1)');
-    console.log('  npm run release        - Auto-detect version bump');
-    console.log('  npm run release -- --prerelease beta  - Create a beta prerelease\n');
-    console.log('🎯 Commit format: type(scope): subject');
-    console.log('   Example: feat(auth): add user login\n');
-
-  } catch (error) {
-    console.error('❌ Error during setup:', error.message);
-    process.exit(1);
-  }
-}
-
-// Run setup if called directly
-if (require.main === module) {
-  setupCommiter();
-}
-
-module.exports = { setupCommiter, ensureSafeTestScript };
+#!/usr/bin/env node
+
+/**
+ * Commiter - Commit convention tooling for standard-version releases
+ * 
+ * This package helps enforce conventional commits and automate releases
+ * with beautiful changelogs featuring icons for each commit type.
+ */
+
+const { execSync } = require('child_process');
+const fs = require('fs');
+const path = require('path');
+
+function ensureSafeTestScript(scripts = {}) {
+  const defaultFailingTestScript = 'echo "Error: no test specified" && exit 1';
+  if (!scripts.test || scripts.test === defaultFailingTestScript) {
+    scripts.test = 'echo "No tests specified"';
+  }
+  return scripts;
+}
+
+function setupCommiter() {
+  console.log('🚀 Setting up Commiter...\n');
+
+  // Check if package.json exists
+  if (!fs.existsSync('package.json')) {
+    console.error('❌ Error: package.json not found. Please run this in a Node.js project directory.');
+    process.exit(1);
+  }
+
+  try {
+    // Install dependencies
+    console.log('📦 Installing dependencies...');
+    execSync('npm install --save-dev standard-version @commitlint/cli @commitlint/config-conventional husky', {
+      stdio: 'inherit'
+    });
+
+    // Read package.json
+    const packageJsonPath = path.join(process.cwd(), 'package.json');
+    const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+
+    packageJson.scripts = ensureSafeTestScript(packageJson.scripts || {});
+    // Add scripts
+    packageJson.scripts.prepare = 'husky';
+    packageJson.scripts.release = 'node scripts/release.js';
+    packageJson.scripts['release:major'] = 'node scripts/release.js major';
+    packageJson.scripts['release:minor'] = 'node scripts/release.js minor';
+    packageJson.scripts['release:patch'] = 'node scripts/release.js patch';
+
+    // Add standard-version config
+    packageJson['standard-version'] = {
+      releaseCommitMessageFormat: 'chore(release): {{currentTag}} 🚀',
+      types: [
+        { type: 'feat', section: '✨ Features' },
+        { type: 'fix', section: '🐛 Bug Fixes' },
+        { type: 'perf', section: '⚡ Performance' },
+        { type: 'refactor', section: '♻️ Refactors' },
+        { type: 'docs', section: '📝 Documentation' },
+        { type: 'style', section: '💄 Styles' },
+        { type: 'test', section: '✅ Tests' },
+        { type: 'build', section: '🏗️ Build System' },
+        { type: 'ci', section: '👷 Continuous Integration' },
+        { type: 'chore', section: '🧹 Chores' },
+        { type: 'revert', section: '⏪ Reverts' }
+      ]
+    };
+
+    // Write updated package.json
+    fs.writeFileSync(packageJsonPath, JSON.stringify(packageJson, null, 2) + '\n');
+
+    // Create release helper script
+    console.log('🛠️  Creating release helper script...');
+    const releaseScriptDir = path.join(process.cwd(), 'scripts');
+    if (!fs.existsSync(releaseScriptDir)) {
+      fs.mkdirSync(releaseScriptDir, { recursive: true });
+    }
+
+    const releaseScriptPath = path.join(releaseScriptDir, 'release.js');
+    const releaseScriptSource = path.join(__dirname, 'scripts', 'release.js');
+    const releaseScriptContent = fs.readFileSync(releaseScriptSource, 'utf8');
+
+    fs.writeFileSync(releaseScriptPath, releaseScriptContent + '\n');
+    try {
+      fs.chmodSync(releaseScriptPath, 0o755);
+    } catch (error) {
+      // Ignore chmod errors on non-POSIX systems
+    }
+
+    // Determine commitlint config format based on project module type
+    const isEsmProject = packageJson.type === 'module';
+    const commitlintConfigFile = isEsmProject ? 'commitlint.config.js' : 'commitlint.config.cjs';
+    const commitlintConfigContent = isEsmProject
+      ? `export default {
+  extends: ['@commitlint/config-conventional'],
+  ignores: [(message) => message.startsWith('chore(release):')]
+}\n`
+      : `module.exports = {
+  extends: ['@commitlint/config-conventional'],
+  ignores: [(message) => message.startsWith('chore(release):')]
+}\n`;
+
+    const legacyCommitlintConfigFile = isEsmProject ? 'commitlint.config.cjs' : 'commitlint.config.js';
+    if (fs.existsSync(legacyCommitlintConfigFile)) {
+      fs.rmSync(legacyCommitlintConfigFile);
+    }
+
+    console.log(`⚙️  Creating commitlint config (${commitlintConfigFile})...`);
+    fs.writeFileSync(commitlintConfigFile, commitlintConfigContent);
+
+    // Initialize Husky
+    console.log('🐶 Setting up Husky...');
+    execSync('npx husky init', { stdio: 'inherit' });
+
+    // Create commit-msg hook
+    const huskyDir = path.join(process.cwd(), '.husky');
+    if (!fs.existsSync(huskyDir)) {
+      fs.mkdirSync(huskyDir, { recursive: true });
+    }
+
+    const commitMsgHook = `#!/usr/bin/env sh
+
+npx --no -- commitlint --edit "$1"
+`;
+    fs.writeFileSync(path.join(huskyDir, 'commit-msg'), commitMsgHook);
+    fs.chmodSync(path.join(huskyDir, 'commit-msg'), 0o755);
+
+    console.log('\n✅ Commiter setup complete!\n');
+    console.log('📚 Available commands:');
+    console.log('  npm run release major  - Create a major release (1.0.0 → 2.0.0)');
+    console.log('  npm run release minor  - Create a minor release (1.0.0 → 1.1.0)');
+    console.log('  npm run release patch  - Create a patch release (1.0.0 → 1.0.1)');
+    console.log('  npm run release        - Auto-detect version bump');
+    console.log('  npm run release -- --prerelease beta  - Create a beta prerelease\n');
+    console.log('🎯 Commit format: type(scope): subject');
+    console.log('   Example: feat(auth): add user login\n');
+
+  } catch (error) {
+    console.error('❌ Error during setup:', error.message);
+    process.exit(1);
+  }
+}
+
+// Run setup if called directly
+if (require.main === module) {
+  setupCommiter();
+}
+
+module.exports = { setupCommiter, ensureSafeTestScript };

package/package.json

@@ -1,94 +1,94 @@
-{
-  "name": "@programinglive/commiter",
-  "version": "1.0.11",
-  "description": "Commit convention tooling for standard-version releases with beautiful changelogs",
-  "main": "index.js",
-  "bin": {
-    "commiter": "./index.js"
-  },
-  "scripts": {
-    "test": "node test/ensureSafeTestScript.test.js",
-    "prepare": "husky",
-    "release": "node scripts/release.js",
-    "release:major": "node scripts/release.js major",
-    "release:minor": "node scripts/release.js minor",
-    "release:patch": "node scripts/release.js patch"
-  },
-  "keywords": [
-    "commit",
-    "conventional-commits",
-    "standard-version",
-    "changelog",
-    "versioning",
-    "semantic-release",
-    "git-hooks",
-    "husky",
-    "commitlint"
-  ],
-  "author": "Programming Live",
-  "license": "MIT",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/programinglive/commiter.git"
-  },
-  "bugs": {
-    "url": "https://github.com/programinglive/commiter/issues"
-  },
-  "homepage": "https://github.com/programinglive/commiter#readme",
-  "type": "commonjs",
-  "standard-version": {
-    "releaseCommitMessageFormat": "chore(release): {{currentTag}} 🚀",
-    "types": [
-      {
-        "type": "feat",
-        "section": "✨ Features"
-      },
-      {
-        "type": "fix",
-        "section": "🐛 Bug Fixes"
-      },
-      {
-        "type": "perf",
-        "section": "⚡ Performance"
-      },
-      {
-        "type": "refactor",
-        "section": "♻️ Refactors"
-      },
-      {
-        "type": "docs",
-        "section": "📝 Documentation"
-      },
-      {
-        "type": "style",
-        "section": "💄 Styles"
-      },
-      {
-        "type": "test",
-        "section": "✅ Tests"
-      },
-      {
-        "type": "build",
-        "section": "🏗️ Build System"
-      },
-      {
-        "type": "ci",
-        "section": "👷 Continuous Integration"
-      },
-      {
-        "type": "chore",
-        "section": "🧹 Chores"
-      },
-      {
-        "type": "revert",
-        "section": "⏪ Reverts"
-      }
-    ]
-  },
-  "devDependencies": {
-    "@commitlint/cli": "^20.1.0",
-    "@commitlint/config-conventional": "^20.0.0",
-    "husky": "^9.1.7",
-    "standard-version": "^9.5.0"
-  }
-}
+{
+  "name": "@programinglive/commiter",
+  "version": "1.1.0",
+  "description": "Commiter keeps repositories release-ready by enforcing conventional commits, generating icon-rich changelog entries, and orchestrating semantic version bumps without manual toil. It bootstraps Husky hooks, commitlint rules, and release scripts that inspect history, detect framework-specific test commands, run them automatically, tag git releases, coordinate npm publishing, surface release metrics, enforce project-specific checks, and give maintainers observability across distributed teams. Plus!",
+  "main": "index.js",
+  "bin": {
+    "commiter": "./index.js"
+  },
+  "scripts": {
+    "test": "node test/ensureSafeTestScript.test.js",
+    "prepare": "husky",
+    "release": "node scripts/release.js",
+    "release:major": "node scripts/release.js major",
+    "release:minor": "node scripts/release.js minor",
+    "release:patch": "node scripts/release.js patch"
+  },
+  "keywords": [
+    "commit",
+    "conventional-commits",
+    "standard-version",
+    "changelog",
+    "versioning",
+    "semantic-release",
+    "git-hooks",
+    "husky",
+    "commitlint"
+  ],
+  "author": "Programming Live",
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/programinglive/commiter.git"
+  },
+  "bugs": {
+    "url": "https://github.com/programinglive/commiter/issues"
+  },
+  "homepage": "https://github.com/programinglive/commiter#readme",
+  "type": "commonjs",
+  "standard-version": {
+    "releaseCommitMessageFormat": "chore(release): {{currentTag}} 🚀",
+    "types": [
+      {
+        "type": "feat",
+        "section": "✨ Features"
+      },
+      {
+        "type": "fix",
+        "section": "🐛 Bug Fixes"
+      },
+      {
+        "type": "perf",
+        "section": "⚡ Performance"
+      },
+      {
+        "type": "refactor",
+        "section": "♻️ Refactors"
+      },
+      {
+        "type": "docs",
+        "section": "📝 Documentation"
+      },
+      {
+        "type": "style",
+        "section": "💄 Styles"
+      },
+      {
+        "type": "test",
+        "section": "✅ Tests"
+      },
+      {
+        "type": "build",
+        "section": "🏗️ Build System"
+      },
+      {
+        "type": "ci",
+        "section": "👷 Continuous Integration"
+      },
+      {
+        "type": "chore",
+        "section": "🧹 Chores"
+      },
+      {
+        "type": "revert",
+        "section": "⏪ Reverts"
+      }
+    ]
+  },
+  "devDependencies": {
+    "@commitlint/cli": "^20.1.0",
+    "@commitlint/config-conventional": "^20.0.0",
+    "husky": "^9.1.7",
+    "standard-version": "^9.5.0"
+  }
+}

package/scripts/release.js

@@ -1,6 +1,8 @@
 #!/usr/bin/env node
 
 const { spawnSync } = require('child_process');
+const fs = require('fs');
+const path = require('path');
 
 const VALID_RELEASE_TYPES = new Set([
   'major',
@@ -14,6 +16,13 @@ const VALID_RELEASE_TYPES = new Set([
 
 const SEMVER_REGEX = /^\d+\.\d+\.\d+(?:-[0-9A-Za-z.-]+)?$/;
 
+const LOCKFILE_PREFERENCES = [
+  { manager: 'pnpm', file: 'pnpm-lock.yaml' },
+  { manager: 'yarn', file: 'yarn.lock' },
+  { manager: 'bun', file: 'bun.lockb' },
+  { manager: 'npm', file: 'package-lock.json' }
+];
+
 function getCliArguments(argv = process.argv) {
   const rawArgs = argv.slice(2);
   if (rawArgs.length === 0) {
@@ -61,6 +70,63 @@ function buildStandardVersionArgs({ releaseType, extraArgs }) {
   return args;
 }
 
+function loadPackageJson(cwd = process.cwd()) {
+  try {
+    const packageJsonPath = path.join(cwd, 'package.json');
+    const contents = fs.readFileSync(packageJsonPath, 'utf8');
+    return JSON.parse(contents);
+  } catch (error) {
+    return null;
+  }
+}
+
+function detectPackageManager({ env = process.env, cwd = process.cwd() } = {}) {
+  const userAgent = env.npm_config_user_agent || '';
+  if (userAgent.startsWith('pnpm/')) return 'pnpm';
+  if (userAgent.startsWith('yarn/')) return 'yarn';
+  if (userAgent.startsWith('bun/')) return 'bun';
+
+  for (const { manager, file } of LOCKFILE_PREFERENCES) {
+    if (fs.existsSync(path.join(cwd, file))) {
+      return manager;
+    }
+  }
+
+  return 'npm';
+}
+
+function buildTestCommand(packageManager) {
+  switch (packageManager) {
+    case 'pnpm':
+      return { command: 'pnpm', args: ['test'] };
+    case 'yarn':
+      return { command: 'yarn', args: ['test'] };
+    case 'bun':
+      return { command: 'bun', args: ['test'] };
+    default:
+      return { command: 'npm', args: ['test'] };
+  }
+}
+
+function runProjectTests({ spawn = spawnSync, env = process.env, cwd = process.cwd() } = {}) {
+  const packageJson = loadPackageJson(cwd);
+  const scripts = packageJson && packageJson.scripts ? packageJson.scripts : {};
+
+  if (!scripts.test) {
+    console.log('ℹ️  Skipping tests: no "test" script detected in package.json');
+    return { status: 0 };
+  }
+
+  const packageManager = detectPackageManager({ env, cwd });
+  const { command, args } = buildTestCommand(packageManager);
+
+  console.log(`🧪 Running tests with ${packageManager} ${args.join(' ')}`.trim());
+  return spawn(command, args, {
+    stdio: 'inherit',
+    env
+  });
+}
+
 function runRelease({ argv = process.argv, env = process.env, spawn = spawnSync } = {}) {
   const { releaseType: cliReleaseType, extraArgs } = getCliArguments(argv);
   const inferredReleaseType = cliReleaseType || getNpmRunArgument(env);
@@ -69,6 +135,11 @@ function runRelease({ argv = process.argv, env = process.env, spawn = spawnSync
     extraArgs
   });
 
+  const testResult = runProjectTests({ spawn, env });
+  if (testResult && typeof testResult.status === 'number' && testResult.status !== 0) {
+    return testResult;
+  }
+
   const standardVersionBin = require.resolve('standard-version/bin/cli.js');
   return spawn(process.execPath, [standardVersionBin, ...standardVersionArgs], {
     stdio: 'inherit'
@@ -93,5 +164,8 @@ module.exports = {
   getCliArguments,
   getNpmRunArgument,
   buildStandardVersionArgs,
+  loadPackageJson,
+  detectPackageManager,
+  runProjectTests,
   runRelease
 };