@itcase/config 1.7.15 → 1.7.17

package/README.md

@@ -103,7 +103,7 @@ npm install --save-dev commitizen @commitlint/cz-commitlint
 
 ## Smart commit
 
-`smart-commit` — Cursor Agent (`ask`) планирует коммиты по `SCOPES`, печатает сообщения, спрашивает `[y/N]`. Перед коммитом ждёт и снимает устаревший `.git/index.lock`, если его никто не держит. Lockfile (`package-lock.json` и аналоги) всегда кладётся в тот же коммит, что и соседний `package.json`; при одном только bump deps агент не вызывается.
+`smart-commit` — Cursor Agent (`ask`) планирует коммиты по `SCOPES`, печатает сообщения, спрашивает `[y/N]`. Перед вызовом агента запускается та же проверка типов, что у husky в lint-staged: `tsc --noEmit --pretty`, либо `npm run typecheck`, если конфиг — `lintStagedMonorepo`. Проверка идёт только когда среди изменений есть существующие `.ts`/`.tsx`. Если типы не сходятся, агент не вызывается. `--no-verify` пропускает и эту проверку, и хуки. Перед коммитом CLI ждёт и снимает устаревший `.git/index.lock`, если его никто не держит. Lockfile (`package-lock.json` и аналоги) всегда кладётся в тот же коммит, что и соседний `package.json`; при одном только bump deps агент не вызывается.
 
 ```text
 [random pixel animation]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@itcase/config",
-  "version": "1.7.15",
+  "version": "1.7.17",
   "author": "ITCase",
   "description": "ITCase Config",
   "engines": {

package/smart-commit/cli.js

@@ -50,6 +50,15 @@ const UPYACHKA_FRAME_MS = 100
 const INDEX_LOCK_RETRIES = 8
 const INDEX_LOCK_WAIT_MS = 250
 const STALE_LOCK_MS = 1500
+const LINT_STAGED_CONFIGS = [
+  'lint-staged.config.mjs',
+  'lint-staged.config.js',
+  'lint-staged.config.cjs',
+  '.lintstagedrc.mjs',
+  '.lintstagedrc.js',
+  '.lintstagedrc.cjs',
+]
+const LINT_STAGED_TS_KEY = '*.{js,jsx,mjs,cjs,ts,tsx}'
 const AGENT_WAIT_STEPS = [
   'Asking Cursor Agent',
   'Still thinking',
@@ -67,6 +76,11 @@ Default model: ${DEFAULT_MODEL} (override with --model / SMART_COMMIT_MODEL).
 (husky pre-commit / lint-staged and commit-msg). Commit message
 is still checked by smart-commit before git runs.
 
+Before the agent, the same typecheck as husky lint-staged runs
+when changes include .ts/.tsx (tsc --noEmit, or npm run typecheck
+from lint-staged config). A type error exits before the agent
+is called. --no-verify skips this check too.
+
 Progress sprite is random (nyan cat, banana, or upyachka).
 Force with --cat, --banana, or --upyachka. Agent log is below.
 `
@@ -104,6 +118,11 @@ async function main(argv) {
     return
   }
 
+  // Husky typechecks in pre-commit, after the agent has already spent tokens.
+  if (!option.noVerify) {
+    await runHuskyTypecheck(cwd, changedFiles)
+  }
+
   let plans = tryFastPlan(changedFiles, cwd, scopes, types)
   if (!plans) {
     plans = await planWithAgent({
@@ -371,6 +390,82 @@ function readReleaseBranches(cwd) {
   return match[1].split(/\s+/).filter(Boolean)
 }
 
+async function runHuskyTypecheck(cwd, files) {
+  const command = await typecheckCommandForFiles(cwd, files)
+  if (!command) return
+
+  process.stdout.write(`${paint('1', 'Typecheck')} ${dim(command)}\n`)
+  const bin = path.join(cwd, 'node_modules', '.bin')
+  const result = spawnSync(command, {
+    cwd: cwd,
+    env: {
+      ...process.env,
+      PATH: `${bin}${path.delimiter}${process.env.PATH || ''}`,
+    },
+    shell: true,
+    stdio: 'inherit',
+  })
+  if (result.error) {
+    throw new Error(`Typecheck failed to start: ${result.error.message}`)
+  }
+  if (result.status !== 0) {
+    throw new Error(
+      'Typecheck failed. Fix the errors above — the agent was not called.',
+    )
+  }
+  process.stdout.write('Typecheck passed.\n\n')
+}
+
+async function typecheckCommandForFiles(cwd, files) {
+  const tsFiles = files.filter(
+    (file) =>
+      /\.(ts|tsx)$/.test(file) && fs.existsSync(path.resolve(cwd, file)),
+  )
+  if (!tsFiles.length) return ''
+
+  const config = await loadLintStagedConfig(cwd)
+  const task = config?.[LINT_STAGED_TS_KEY]
+  if (typeof task !== 'function') return 'tsc --noEmit --pretty'
+
+  const absolute = tsFiles.map((file) => path.resolve(cwd, file))
+  const commands = await task(absolute)
+  return pickTypecheckCommand(commands)
+}
+
+function pickTypecheckCommand(commands) {
+  const list = (Array.isArray(commands) ? commands : [commands]).filter(
+    (command) => typeof command === 'string' && command.trim(),
+  )
+  const explicit = list.find((command) =>
+    /^(?:tsc|npm(?:\s+run)?\s+typecheck|pnpm(?:\s+run)?\s+typecheck|yarn(?:\s+run)?\s+typecheck)\b/.test(
+      command.trim(),
+    ),
+  )
+  if (explicit) return explicit.trim()
+
+  const other = list.find(
+    (command) => !/^\s*"?(?:eslint|stylelint)\b/.test(command),
+  )
+  return other ? other.trim() : ''
+}
+
+async function loadLintStagedConfig(cwd) {
+  const file = LINT_STAGED_CONFIGS.map((name) => path.join(cwd, name)).find(
+    (candidate) => fs.existsSync(candidate),
+  )
+  if (file) {
+    const loaded = await import(pathToFileURL(file).href)
+    return loaded.default ?? loaded
+  }
+
+  const packageJsonPath = path.join(cwd, 'package.json')
+  if (!fs.existsSync(packageJsonPath)) return null
+  const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'))
+  const field = packageJson['lint-staged']
+  if (field && typeof field === 'object') return field
+  return null
+}
+
 function listFiles(cwd, option) {
   if (option.staged) {
     return lines(cwd, [
@@ -1447,9 +1542,14 @@ function stageFiles(cwd, files) {
 async function confirm(count) {
   if (!input.isTTY) throw new Error('No TTY — use --yes or --dry-run')
   const rl = readline.createInterface({ input, output })
+  const prompt = `Create ${count} commit(s)? [y/N] `
   try {
-    const answer = await rl.question(`Create ${count} commit(s)? [y/N] `)
-    return /^y(es)?$/i.test(answer.trim())
+    for (;;) {
+      const answer = (await rl.question(prompt)).trim()
+      if (/^y(es)?$/i.test(answer)) return true
+      if (/^(n(o)?)?$/i.test(answer)) return false
+      output.write('Enter y or n.\n')
+    }
   } finally {
     rl.close()
   }