@nitra/cursor 1.8.105 → 1.8.106

package/scripts/check-bun.mjs

@@ -50,6 +50,91 @@ async function loadNCursorRules() {
   }
 }
 
+/**
+ * @param {{ pass: (msg: string) => void, fail: (msg: string) => void }} reporter репортер для збору результатів
+ * @param {Record<string, unknown>} pkg розібраний package.json
+ */
+function checkDevDependencies(reporter, pkg) {
+  const { pass, fail } = reporter
+  const dev = pkg.devDependencies
+  if (dev === undefined) {
+    pass('Кореневий package.json без devDependencies')
+    return
+  }
+  if (dev === null || typeof dev !== 'object' || Array.isArray(dev)) {
+    fail(
+      'Кореневий package.json: `devDependencies` має бути object з ключами пакетів і діапазонами версій (не null, не масив)'
+    )
+    return
+  }
+  const bad = Object.keys(/** @type {object} */ (dev)).filter(n => !isAllowedRootDevDependency(n))
+  if (bad.length > 0) {
+    fail(`Кореневі devDependencies: дозволені лише @nitra/* — прибери або перенеси: ${bad.join(', ')} (bun.mdc)`)
+    return
+  }
+  const n = Object.keys(/** @type {object} */ (dev)).length
+  pass(
+    n === 0
+      ? 'Кореневі devDependencies порожні або відсутні (лише @nitra/*)'
+      : `Кореневі devDependencies: лише @nitra/* (${n} пак.)`
+  )
+}
+
+/**
+ * @param {{ pass: (msg: string) => void, fail: (msg: string) => void }} reporter репортер для збору результатів
+ * @param {Record<string, string>} scripts scripts з package.json
+ */
+function checkLintAggregate(reporter, scripts) {
+  const { pass, fail } = reporter
+  const lintPrefixed = Object.keys(scripts).filter(name => name.startsWith('lint-'))
+  if (lintPrefixed.length === 0) return
+  const aggregate = typeof scripts.lint === 'string' ? scripts.lint : ''
+  if (!aggregate.trim()) {
+    const scriptList = lintPrefixed.map(s => `\`${s}\``).join(', ')
+    fail(
+      `У package.json є скрипти ${scriptList}, але немає агрегованого \`lint\` — додай скрипт, який запускає їх через \`bun run\``
+    )
+    return
+  }
+  const missing = lintPrefixed.filter(name => !aggregate.includes(`bun run ${name}`))
+  if (missing.length > 0) {
+    const missingList = missing.map(s => '`' + s + '`').join(', ')
+    fail(`Скрипт \`lint\` має викликати всі lint-* через bun run; відсутньо: ${missingList}`)
+    return
+  }
+  pass('package.json: агрегований `lint` покриває всі `lint-*` скрипти')
+  if (OXFMT_END_RE.test(aggregate.trim())) {
+    pass('package.json: `lint` завершується `&& oxfmt .`')
+  } else {
+    fail('Скрипт `lint` має закінчуватися на `&& oxfmt .`')
+  }
+}
+
+/**
+ * @param {{ pass: (msg: string) => void, fail: (msg: string) => void }} reporter репортер для збору результатів
+ * @param {Record<string, string>} scripts scripts з package.json
+ * @param {Set<string>} cursorRules активні правила з .n-cursor.json
+ */
+function checkCursorRuleScripts(reporter, scripts, cursorRules) {
+  const { pass, fail } = reporter
+  /** @type {Array<{rule: string, script: string, doc: string}>} */
+  const ruleScripts = [
+    { rule: 'docker', script: 'lint-docker', doc: 'docker.mdc' },
+    { rule: 'k8s', script: 'lint-k8s', doc: 'k8s.mdc' }
+  ]
+  for (const { rule, script, doc } of ruleScripts) {
+    if (cursorRules.has(rule)) {
+      if (scripts[script]) {
+        pass(`package.json: є \`${script}\` (правило ${rule} у .n-cursor.json)`)
+      } else {
+        fail(
+          `У .n-cursor.json є правило \`${rule}\` — додай скрипт \`${script}\` у кореневий package.json (див. ${doc})`
+        )
+      }
+    }
+  }
+}
+
 /**
  * Перевіряє відповідність проєкту правилам bun.mdc
  * @returns {Promise<number>} 0 — все OK, 1 — є проблеми
@@ -58,8 +143,7 @@ export async function check() {
   const reporter = createCheckReporter()
   const { pass, fail } = reporter
 
-  const forbidden = ['package-lock.json', 'yarn.lock', 'pnpm-lock.yaml', '.yarnrc.yml']
-  for (const f of forbidden) {
+  for (const f of ['package-lock.json', 'yarn.lock', 'pnpm-lock.yaml', '.yarnrc.yml']) {
     if (existsSync(f)) {
       fail(`Знайдено заборонений файл: ${f} — видали його`)
     } else {
@@ -80,86 +164,30 @@ export async function check() {
 
   const cursorRules = await loadNCursorRules()
 
-  if (existsSync('package.json')) {
-    const pkg = JSON.parse(await readFile('package.json', 'utf8'))
-    if (pkg.packageManager) {
-      fail(`package.json містить поле packageManager: "${pkg.packageManager}" — видали його`)
-    } else {
-      pass('package.json не містить packageManager')
-    }
-
-    if (pkg.dependencies === undefined) {
-      pass('Кореневий package.json без поля `dependencies`')
-    } else {
-      fail(
-        'Кореневий package.json не повинен містити поле `dependencies` — додай залежності в workspace-пакети (bun.mdc)'
-      )
-    }
+  if (!existsSync('package.json')) {
+    return reporter.getExitCode()
+  }
 
-    const dev = pkg.devDependencies
-    if (dev === undefined) {
-      pass('Кореневий package.json без devDependencies')
-    } else if (dev === null || typeof dev !== 'object' || Array.isArray(dev)) {
-      fail(
-        'Кореневий package.json: `devDependencies` має бути object з ключами пакетів і діапазонами версій (не null, не масив)'
-      )
-    } else {
-      const bad = Object.keys(dev).filter(n => !isAllowedRootDevDependency(n))
-      if (bad.length > 0) {
-        fail(`Кореневі devDependencies: дозволені лише @nitra/* — прибери або перенеси: ${bad.join(', ')} (bun.mdc)`)
-      } else {
-        const n = Object.keys(dev).length
-        pass(
-          n === 0
-            ? 'Кореневі devDependencies порожні або відсутні (лише @nitra/*)'
-            : `Кореневі devDependencies: лише @nitra/* (${n} пак.)`
-        )
-      }
-    }
+  const pkg = JSON.parse(await readFile('package.json', 'utf8'))
+  if (pkg.packageManager) {
+    fail(`package.json містить поле packageManager: "${pkg.packageManager}" — видали його`)
+  } else {
+    pass('package.json не містить packageManager')
+  }
 
-    const scripts = pkg.scripts && typeof pkg.scripts === 'object' ? pkg.scripts : {}
+  if (pkg.dependencies === undefined) {
+    pass('Кореневий package.json без поля `dependencies`')
+  } else {
+    fail(
+      'Кореневий package.json не повинен містити поле `dependencies` — додай залежності в workspace-пакети (bun.mdc)'
+    )
+  }
 
-    if (cursorRules.has('docker')) {
-      if (scripts['lint-docker']) {
-        pass('package.json: є `lint-docker` (правило docker у .n-cursor.json)')
-      } else {
-        fail(
-          'У .n-cursor.json є правило `docker` — додай скрипт `lint-docker` у кореневий package.json (див. docker.mdc)'
-        )
-      }
-    }
+  checkDevDependencies(reporter, pkg)
 
-    if (cursorRules.has('k8s')) {
-      if (scripts['lint-k8s']) {
-        pass('package.json: є `lint-k8s` (правило k8s у .n-cursor.json)')
-      } else {
-        fail('У .n-cursor.json є правило `k8s` — додай скрипт `lint-k8s` у кореневий package.json (див. k8s.mdc)')
-      }
-    }
-    const lintPrefixed = Object.keys(scripts).filter(name => name.startsWith('lint-'))
-    if (lintPrefixed.length > 0) {
-      const aggregate = typeof scripts.lint === 'string' ? scripts.lint : ''
-      if (aggregate.trim()) {
-        const missing = lintPrefixed.filter(name => !aggregate.includes(`bun run ${name}`))
-        if (missing.length > 0) {
-          const missingList = missing.map(s => `\`${s}\``).join(', ')
-          fail(`Скрипт \`lint\` має викликати всі lint-* через bun run; відсутньо: ${missingList}`)
-        } else {
-          pass('package.json: агрегований `lint` покриває всі `lint-*` скрипти')
-          if (OXFMT_END_RE.test(aggregate.trim())) {
-            pass('package.json: `lint` завершується `&& oxfmt .`')
-          } else {
-            fail('Скрипт `lint` має закінчуватися на `&& oxfmt .`')
-          }
-        }
-      } else {
-        const scriptList = lintPrefixed.map(s => `\`${s}\``).join(', ')
-        fail(
-          `У package.json є скрипти ${scriptList}, але немає агрегованого \`lint\` — додай скрипт, який запускає їх через \`bun run\``
-        )
-      }
-    }
-  }
+  const scripts = pkg.scripts && typeof pkg.scripts === 'object' ? pkg.scripts : {}
+  checkCursorRuleScripts(reporter, scripts, cursorRules)
+  checkLintAggregate(reporter, scripts)
 
   return reporter.getExitCode()
 }

package/scripts/check-ga.mjs

@@ -118,31 +118,135 @@ function verifyNoDirectBunOrCache(relPath, content, failFn, passFn) {
 }
 
 /**
- * Перевіряє відповідність проєкту правилам ga.mdc
- * @returns {Promise<number>} 0 — все OK, 1 — є проблеми
+ * Перевіряє apply-workflow на наявність paths trigger.
+ * @param {string} wfDir директорія workflows
+ * @param {string[]} files список файлів у директорії
+ * @param {string} filename параметр filename
+ * @param {string} expectedPath параметр expectedPath
+ * @param {(msg: string) => void} passFn callback при успішній перевірці
+ * @param {(msg: string) => void} failFn callback при помилці
  */
-export async function check() {
-  const reporter = createCheckReporter()
-  const { pass, fail } = reporter
+async function checkApplyWorkflow(wfDir, files, filename, expectedPath, passFn, failFn) {
+  if (!files.includes(filename)) return
+  const content = await readFile(`${wfDir}/${filename}`, 'utf8')
+  const root = parseWorkflowYaml(content)
+  const ok = root ? eventPathsIncludeExact(root, 'push', expectedPath) : content.includes(expectedPath)
+  if (ok) {
+    passFn(`${filename} має правильний paths trigger`)
+  } else {
+    failFn(`${filename} не містить paths: ${expectedPath}`)
+  }
+}
 
-  const wfDir = '.github/workflows'
+/**
+ * Перевіряє відсутність MegaLinter у workflows та конфіг-файлах.
+ * @param {string} wfDir директорія workflows
+ * @param {string[]} ymlWorkflows параметр ymlWorkflows
+ * @param {(msg: string) => void} passFn callback при успішній перевірці
+ * @param {(msg: string) => void} failFn callback при помилці
+ */
+async function checkMegalinter(wfDir, ymlWorkflows, passFn, failFn) {
+  let found = false
+  for (const f of ymlWorkflows) {
+    const content = await readFile(join(wfDir, f), 'utf8')
+    if (MEGALINTER_USE_PATTERNS.some(re => re.test(content))) {
+      found = true
+      failFn(`MegaLinter у workflow ${wfDir}/${f} — видали інтеграцію (ga.mdc: MegaLinter)`)
+    }
+  }
+  for (const name of MEGALINTER_CONFIG_NAMES) {
+    if (existsSync(name)) {
+      found = true
+      failFn(`Файл ${name} — видали конфіг MegaLinter (ga.mdc: MegaLinter)`)
+    }
+  }
+  if (!found) passFn('Залишків MegaLinter не виявлено')
+}
 
-  if (!existsSync(wfDir)) {
-    fail(`Директорія ${wfDir} не існує`)
-    return reporter.getExitCode()
+/**
+ * Перевіряє zizmor конфіг.
+ * @param {(msg: string) => void} passFn callback при успішній перевірці
+ * @param {(msg: string) => void} failFn callback при помилці
+ */
+async function checkZizmor(passFn, failFn) {
+  const zizmorPath = '.github/zizmor.yml'
+  if (!existsSync(zizmorPath)) {
+    failFn(`Відсутній ${zizmorPath} — потрібен для zizmor (ga.mdc)`)
+    return
+  }
+  const z = await readFile(zizmorPath, 'utf8')
+  passFn(`${zizmorPath} існує`)
+  if (z.includes('ref-pin')) {
+    passFn(`${zizmorPath} містить політику ref-pin (zizmor)`)
+  } else {
+    failFn(`${zizmorPath}: додай policies ref-pin для unpinned-uses (ga.mdc)`)
   }
+}
 
-  const setupBunDepsAction = '.github/actions/setup-bun-deps/action.yml'
-  if (existsSync(setupBunDepsAction)) {
-    pass(`${setupBunDepsAction} існує`)
+/**
+ * Перевіряє скрипт lint-ga в package.json.
+ * @param {(msg: string) => void} passFn callback при успішній перевірці
+ * @param {(msg: string) => void} failFn callback при помилці
+ */
+async function checkLintGaScript(passFn, failFn) {
+  if (!existsSync('package.json')) {
+    failFn('package.json не існує — потрібен lint-ga у scripts')
+    return
+  }
+  const pkg = JSON.parse(await readFile('package.json', 'utf8'))
+  const lg = pkg.scripts?.['lint-ga']
+  if (typeof lg !== 'string') {
+    failFn('package.json: додай скрипт "lint-ga" (ga.mdc)')
+    return
+  }
+  passFn('package.json містить lint-ga')
+  if (lg.includes('node-actionlint')) {
+    passFn('lint-ga викликає node-actionlint')
   } else {
-    fail(
-      `Відсутній ${setupBunDepsAction} — запустіть npx @nitra/cursor або скопіюйте з пакету (ga.mdc: composite setup-bun-deps)`
-    )
+    failFn('lint-ga має містити bunx node-actionlint (ga.mdc)')
+  }
+  if (lg.includes('zizmor') && lg.includes('--offline')) {
+    passFn('lint-ga викликає zizmor з --offline')
+  } else {
+    failFn('lint-ga має містити zizmor і --offline (ga.mdc)')
   }
+}
 
-  const files = await readdir(wfDir)
+/**
+ * Перевіряє lint-ga.yml workflow.
+ * @param {string} wfDir директорія workflows
+ * @param {(msg: string) => void} passFn callback при успішній перевірці
+ * @param {(msg: string) => void} failFn callback при помилці
+ */
+async function checkLintGaWorkflow(wfDir, passFn, failFn) {
+  const lintGaWf = join(wfDir, 'lint-ga.yml')
+  if (!existsSync(lintGaWf)) return
+  const lgContent = await readFile(lintGaWf, 'utf8')
+  const root = parseWorkflowYaml(lgContent)
+  const hasBunRun = root ? anyRunStepIncludes(root, 'bun run lint-ga') : lgContent.includes('bun run lint-ga')
+  const hasSetupUv = root
+    ? hasAnyStepUsesContaining(root, ['astral-sh/setup-uv']) || lgContent.includes('astral-sh/setup-uv')
+    : lgContent.includes('astral-sh/setup-uv')
+  if (hasBunRun) {
+    passFn('lint-ga.yml викликає bun run lint-ga')
+  } else {
+    failFn('lint-ga.yml: крок має містити bun run lint-ga')
+  }
+  if (hasSetupUv) {
+    passFn('lint-ga.yml містить astral-sh/setup-uv')
+  } else {
+    failFn('lint-ga.yml: додай astral-sh/setup-uv для uvx zizmor (ga.mdc)')
+  }
+}
 
+/**
+ * Перевіряє розширення workflow-файлів і наявність обов'язкових workflow.
+ * @param {string} wfDir шлях до директорії workflows
+ * @param {string[]} files список файлів у wfDir
+ * @param {(msg: string) => void} pass callback при успішній перевірці
+ * @param {(msg: string) => void} fail callback при помилці
+ */
+function checkGaWorkflowFiles(wfDir, files, pass, fail) {
   const yamlFiles = files.filter(f => f.endsWith('.yaml'))
   if (yamlFiles.length > 0) {
     for (const f of yamlFiles) {
@@ -159,31 +263,38 @@ export async function check() {
       fail(`Відсутній ${wfDir}/${f}`)
     }
   }
+}
 
-  if (files.includes('apply-k8s.yml')) {
-    const content = await readFile(`${wfDir}/apply-k8s.yml`, 'utf8')
-    const root = parseWorkflowYaml(content)
-    const ok =
-      root && eventPathsIncludeExact(root, 'push', '**/k8s/**/*.yaml') ? true : content.includes('**/k8s/**/*.yaml')
-    if (ok) {
-      pass('apply-k8s.yml має правильний paths trigger')
-    } else {
-      fail('apply-k8s.yml не містить paths: **/k8s/**/*.yaml')
-    }
+/**
+ * Перевіряє відповідність проєкту правилам ga.mdc
+ * @returns {Promise<number>} 0 — все OK, 1 — є проблеми
+ */
+export async function check() {
+  const reporter = createCheckReporter()
+  const { pass, fail } = reporter
+
+  const wfDir = '.github/workflows'
+
+  if (!existsSync(wfDir)) {
+    fail(`Директорія ${wfDir} не існує`)
+    return reporter.getExitCode()
   }
 
-  if (files.includes('apply-nats-consumer.yml')) {
-    const content = await readFile(`${wfDir}/apply-nats-consumer.yml`, 'utf8')
-    const root = parseWorkflowYaml(content)
-    const ok =
-      root && eventPathsIncludeExact(root, 'push', '**/consumer.yaml') ? true : content.includes('**/consumer.yaml')
-    if (ok) {
-      pass('apply-nats-consumer.yml має правильний paths trigger')
-    } else {
-      fail('apply-nats-consumer.yml не містить paths: **/consumer.yaml')
-    }
+  const setupBunDepsAction = '.github/actions/setup-bun-deps/action.yml'
+  if (existsSync(setupBunDepsAction)) {
+    pass(`${setupBunDepsAction} існує`)
+  } else {
+    fail(
+      `Відсутній ${setupBunDepsAction} — запустіть npx @nitra/cursor або скопіюйте з пакету (ga.mdc: composite setup-bun-deps)`
+    )
   }
 
+  const files = await readdir(wfDir)
+  checkGaWorkflowFiles(wfDir, files, pass, fail)
+
+  await checkApplyWorkflow(wfDir, files, 'apply-k8s.yml', '**/k8s/**/*.yaml', pass, fail)
+  await checkApplyWorkflow(wfDir, files, 'apply-nats-consumer.yml', '**/consumer.yaml', pass, fail)
+
   if (existsSync('.vscode/extensions.json')) {
     const ext = JSON.parse(await readFile('.vscode/extensions.json', 'utf8'))
     if (ext.recommendations?.includes('github.vscode-github-actions')) {
@@ -196,25 +307,7 @@ export async function check() {
   }
 
   const ymlWorkflows = files.filter(f => f.endsWith('.yml'))
-  let foundMegalinter = false
-  for (const f of ymlWorkflows) {
-    const content = await readFile(join(wfDir, f), 'utf8')
-    if (MEGALINTER_USE_PATTERNS.some(re => re.test(content))) {
-      foundMegalinter = true
-      fail(`MegaLinter у workflow ${wfDir}/${f} — видали інтеграцію (ga.mdc: MegaLinter)`)
-    }
-  }
-
-  for (const name of MEGALINTER_CONFIG_NAMES) {
-    if (existsSync(name)) {
-      foundMegalinter = true
-      fail(`Файл ${name} — видали конфіг MegaLinter (ga.mdc: MegaLinter)`)
-    }
-  }
-
-  if (!foundMegalinter) {
-    pass('Залишків MegaLinter не виявлено')
-  }
+  await checkMegalinter(wfDir, ymlWorkflows, pass, fail)
 
   for (const f of ymlWorkflows) {
     const content = await readFile(join(wfDir, f), 'utf8')
@@ -222,70 +315,9 @@ export async function check() {
     verifyNoDirectBunOrCache(`${wfDir}/${f}`, content, fail, pass)
   }
 
-  const zizmorPath = '.github/zizmor.yml'
-  if (existsSync(zizmorPath)) {
-    const z = await readFile(zizmorPath, 'utf8')
-    pass(`${zizmorPath} існує`)
-    if (z.includes('ref-pin')) {
-      pass(`${zizmorPath} містить політику ref-pin (zizmor)`)
-    } else {
-      fail(`${zizmorPath}: додай policies ref-pin для unpinned-uses (ga.mdc)`)
-    }
-  } else {
-    fail(`Відсутній ${zizmorPath} — потрібен для zizmor (ga.mdc)`)
-  }
-
-  if (existsSync('package.json')) {
-    const pkg = JSON.parse(await readFile('package.json', 'utf8'))
-    const lg = pkg.scripts?.['lint-ga']
-    if (typeof lg === 'string') {
-      pass('package.json містить lint-ga')
-      if (lg.includes('node-actionlint')) {
-        pass('lint-ga викликає node-actionlint')
-      } else {
-        fail('lint-ga має містити bunx node-actionlint (ga.mdc)')
-      }
-      if (lg.includes('zizmor') && lg.includes('--offline')) {
-        pass('lint-ga викликає zizmor з --offline')
-      } else {
-        fail('lint-ga має містити zizmor і --offline (ga.mdc)')
-      }
-    } else {
-      fail('package.json: додай скрипт "lint-ga" (ga.mdc)')
-    }
-  } else {
-    fail('package.json не існує — потрібен lint-ga у scripts')
-  }
-
-  const lintGaWf = join(wfDir, 'lint-ga.yml')
-  if (existsSync(lintGaWf)) {
-    const lgContent = await readFile(lintGaWf, 'utf8')
-    const root = parseWorkflowYaml(lgContent)
-    if (root) {
-      if (anyRunStepIncludes(root, 'bun run lint-ga')) {
-        pass('lint-ga.yml викликає bun run lint-ga')
-      } else {
-        fail('lint-ga.yml: крок має містити bun run lint-ga')
-      }
-      const usesFlat = hasAnyStepUsesContaining(root, ['astral-sh/setup-uv'])
-      if (usesFlat || lgContent.includes('astral-sh/setup-uv')) {
-        pass('lint-ga.yml містить astral-sh/setup-uv')
-      } else {
-        fail('lint-ga.yml: додай astral-sh/setup-uv для uvx zizmor (ga.mdc)')
-      }
-    } else {
-      if (lgContent.includes('bun run lint-ga')) {
-        pass('lint-ga.yml викликає bun run lint-ga')
-      } else {
-        fail('lint-ga.yml: крок має містити bun run lint-ga')
-      }
-      if (lgContent.includes('astral-sh/setup-uv')) {
-        pass('lint-ga.yml містить astral-sh/setup-uv')
-      } else {
-        fail('lint-ga.yml: додай astral-sh/setup-uv для uvx zizmor (ga.mdc)')
-      }
-    }
-  }
+  await checkZizmor(pass, fail)
+  await checkLintGaScript(pass, fail)
+  await checkLintGaWorkflow(wfDir, pass, fail)
 
   return reporter.getExitCode()
 }