@nitra/cursor 1.13.89 → 1.14.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (75) hide show
  1. package/CHANGELOG.md +27 -0
  2. package/package.json +1 -1
  3. package/rules/abie/abie.mdc +8 -8
  4. package/rules/abie/js/{applies/check.mjs → applies.mjs} +2 -2
  5. package/rules/abie/js/{env_dns/check.mjs → env_dns.mjs} +3 -3
  6. package/rules/abie/js/{firebase_hosting/check.mjs → firebase_hosting.mjs} +1 -1
  7. package/rules/abie/js/{hc_pairing/check.mjs → hc_pairing.mjs} +4 -4
  8. package/rules/abie/js/{ua_http_route/check.mjs → ua_http_route.mjs} +6 -6
  9. package/rules/abie/js/{ua_node_selector/check.mjs → ua_node_selector.mjs} +5 -5
  10. package/rules/abie/policy/base_deployment_preem/base_deployment_preem.rego +2 -2
  11. package/rules/abie/policy/health_check_policy/health_check_policy.rego +2 -2
  12. package/rules/abie/policy/http_route_base/http_route_base.rego +1 -1
  13. package/rules/abie/utils/enabled.mjs +1 -1
  14. package/rules/abie/utils/k8s-tree.mjs +1 -1
  15. package/rules/adr/js/{hooks/check.mjs → hooks.mjs} +1 -1
  16. package/rules/bun/js/{layout/check.mjs → layout.mjs} +1 -1
  17. package/rules/capacitor/js/{platforms/check.mjs → platforms.mjs} +1 -1
  18. package/rules/changelog/changelog.mdc +1 -1
  19. package/rules/changelog/js/{consistency/check.mjs → consistency.mjs} +2 -2
  20. package/rules/changelog/{js/consistency → utils}/package-manifest.mjs +1 -1
  21. package/rules/docker/js/{lint/check.mjs → lint.mjs} +5 -5
  22. package/rules/docker/lint/lint.mjs +1 -1
  23. package/rules/docker/{js/lint → utils}/docker-hadolint.mjs +1 -1
  24. package/rules/feedback/feedback.mdc +1 -1
  25. package/rules/ga/js/{workflows/check.mjs → workflows.mjs} +5 -5
  26. package/rules/ga/lint/lint.mjs +1 -1
  27. package/rules/graphql/js/{tooling/check.mjs → tooling.mjs} +5 -5
  28. package/rules/hasura/js/{internal_urls/check.mjs → internal_urls.mjs} +4 -4
  29. package/rules/hasura/policy/svc_hl/svc_hl.rego +1 -1
  30. package/rules/image-avif/js/{avif_generation/check.mjs → avif_generation.mjs} +5 -5
  31. package/rules/image-compress/js/{package_setup/check.mjs → package_setup.mjs} +1 -1
  32. package/rules/js-bun-db/js/{safety/check.mjs → safety.mjs} +5 -5
  33. package/rules/js-bun-db/{js/safety → utils}/bun-sql-scan.mjs +1 -1
  34. package/rules/js-bun-redis/js/{imports/check.mjs → imports.mjs} +4 -4
  35. package/rules/js-bun-redis/policy/package_json/package_json.rego +1 -1
  36. package/rules/js-lint/js/{tooling/check.mjs → tooling.mjs} +8 -4
  37. package/rules/js-lint/js-lint.mdc +11 -1
  38. package/rules/js-lint/{js/tooling → utils}/rebuild-oxlint-canonical.mjs +2 -2
  39. package/rules/js-mssql/js/{deps/check.mjs → deps.mjs} +5 -5
  40. package/rules/js-mssql/{js/deps → utils}/mssql-pool-scan.mjs +1 -1
  41. package/rules/js-run/js/{runtime/check.mjs → runtime.mjs} +10 -10
  42. package/rules/js-run/{js/runtime → utils}/bunyan-imports.mjs +1 -1
  43. package/rules/js-run/{js/runtime → utils}/check-env-scan.mjs +1 -1
  44. package/rules/js-run/{js/runtime → utils}/conn-file-rules.mjs +1 -1
  45. package/rules/js-run/{js/runtime → utils}/conn-imports-scan.mjs +1 -1
  46. package/rules/js-run/{js/runtime → utils}/promise-settimeout-scan.mjs +1 -1
  47. package/rules/k8s/js/{manifests/check.mjs → manifests.mjs} +4 -4
  48. package/rules/k8s/k8s.mdc +1 -1
  49. package/rules/nginx-default-tpl/js/{template/check.mjs → template.mjs} +5 -5
  50. package/rules/npm-module/js/{package_structure/check.mjs → package_structure.mjs} +4 -4
  51. package/rules/php/js/{tooling/check.mjs → tooling.mjs} +1 -1
  52. package/rules/rego/js/{applies/check.mjs → applies.mjs} +3 -3
  53. package/rules/security/js/{sample_secret/check.mjs → sample_secret.mjs} +2 -2
  54. package/rules/security/js/{trufflehog/check.mjs → trufflehog.mjs} +3 -3
  55. package/rules/security/security.mdc +2 -2
  56. package/rules/style-lint/js/{tooling/check.mjs → tooling.mjs} +1 -1
  57. package/rules/tauri/js/{tooling/check.mjs → tooling.mjs} +2 -2
  58. package/rules/test/js/{location/check.mjs → location.mjs} +3 -3
  59. package/rules/text/js/{formatting/check.mjs → formatting.mjs} +2 -2
  60. package/rules/vue/js/{packages/check.mjs → packages.mjs} +5 -5
  61. package/scripts/auto-rules.mjs +3 -3
  62. package/scripts/sync-claude-config.mjs +1 -1
  63. package/scripts/utils/discover-checkable-rules.mjs +20 -27
  64. package/scripts/utils/inline-template-links.mjs +1 -1
  65. package/scripts/utils/run-rule.mjs +18 -23
  66. /package/rules/adr/js/{hooks/template → templates/hooks}/.gitignore.snippet +0 -0
  67. /package/rules/docker/{js/lint → utils}/docker-mirror.mjs +0 -0
  68. /package/rules/graphql/{js/tooling → utils}/graphql-gql-scan.mjs +0 -0
  69. /package/rules/js-lint/js/{tooling → data/tooling}/knip-canonical.json +0 -0
  70. /package/rules/js-lint/js/{tooling → data/tooling}/oxlint-canonical-skeleton.json +0 -0
  71. /package/rules/js-lint/js/{tooling → data/tooling}/oxlint-canonical.json +0 -0
  72. /package/rules/js-lint/js/{tooling → data/tooling}/oxlint-rules.tsv +0 -0
  73. /package/rules/k8s/js/{kubescape_exceptions/template → templates/kubescape_exceptions}/.kubescape-exceptions.json.snippet.json +0 -0
  74. /package/rules/security/js/{trufflehog/template → templates/trufflehog}/.trufflehog-exclude.snippet.txt +0 -0
  75. /package/rules/vue/{js/packages → utils}/vue-forbidden-imports.mjs +0 -0
@@ -2,7 +2,7 @@
2
2
  description: Перевірка JavaScript коду
3
3
  globs: "**/{.oxlintrc.json,eslint.config.js,.jscpd.json,knip.json,package.json},**/*.{js,mjs,cjs,jsx,ts,tsx}"
4
4
  alwaysApply: false
5
- version: '1.23'
5
+ version: '1.24'
6
6
  ---
7
7
 
8
8
  **oxlint**, **ESLint**, **jscpd**, **knip**. У скрипті **`lint-js`** і в CI — **`bunx oxlint`**, **`bunx eslint`**, **`bunx jscpd`**, **`bunx knip`** (у CI без **`--fix`** для oxlint/eslint — див. приклад workflow нижче). Без **prettier** і **@nitra/prettier-config**. У **`devDependencies`** має бути **`@nitra/eslint-config` мінімум `^3.9.2`** (з **3.8.0** правило `no-restricted-syntax` забороняє `for...in`; з **3.9.2** у `getConfig` вбудовано ignore для **`**/adr/**`** — ADR-документи не валідуються ESLint, локально цей glob додавати не потрібно; також транзитивно йде **`@e18e/eslint-plugin`** для oxlint); пакет **`@e18e/eslint-plugin`** окремо не додавай. Пакети oxlint/eslint/jscpd/knip не додавай без потреби монорепо.
@@ -79,6 +79,16 @@ version: '1.23'
79
79
 
80
80
  Але обов'язково перед рефакторингом перевір чи є тести на блоки які підлягають зміні, а саме Bun.test для js та playwright для vue. Якщо є, то перевір чи вони покривають блоки які підлягають зміні. Якщо не покривають або тестів немає — спочатку створи їх, перевір що вони покривають і відпрацьовують коректно, а потім роби рефакторинг і ще раз запускай тести, але якщо тести не відпрацьовують коректно після рефакторингу, то не роби рефакторинг.
81
81
 
82
+ ## Структура спільних модулів: `utils/` vs `lib/`
83
+
84
+ Коли спільні методи виносяться з кількох JS-файлів в окремий каталог — назву каталога обирай за призначенням модулів усередині.
85
+
86
+ - **`utils/`** — низькорівневі, чисті, generic helpers без бізнес-логіки і без залежностей від домену проєкту. Те, що могло б жити в окремому npm-пакеті. Приклади: `formatDate()`, `chunk(array, size)`, `retry(fn, opts)`, `deepMerge()`, `parseDuration('5m')`, `sleep(ms)`.
87
+
88
+ - **`lib/`** — внутрішні модулі / підсистеми проєкту, які знають про домен. Більші за `utils/`, часто з власним state, конфігом або side effects. Приклади: `lib/gmail-client.js`, `lib/circuit-breaker.js`, `lib/skill-loader.js`, `lib/safety/dry-run.js`.
89
+
90
+ Швидкий тест: якщо файл завтра можна опублікувати окремим npm-пакетом без переписування — це `utils/`; якщо він тримає domain-state, читає конфіг проєкту або викликає зовнішні сервіси/файли — це `lib/`. Не плутай із чужими каталогами на кшталт `shared/` чи `common/`: канонічні назви — лише `utils/` і `lib/`.
91
+
82
92
  Додай workflow:
83
93
 
84
94
  ```yaml title=".github/workflows/lint-js.yml"
@@ -2,14 +2,14 @@
2
2
  * Збирає `oxlint-canonical.json` з `oxlint-canonical-skeleton.json` (без поля rules) та списку
3
3
  * правил у `oxlint-rules.tsv` (колонки: ім’я правила, TAB, severity: deny | off | error).
4
4
  *
5
- * Після змін у TSV або скелеті запускай з каталогу пакета: `bun ./rules/js-lint/js/tooling/rebuild-oxlint-canonical.mjs`,
5
+ * Після змін у TSV або скелеті запускай з каталогу пакета: `bun ./rules/js-lint/utils/rebuild-oxlint-canonical.mjs`,
6
6
  * потім скопіюй оновлений канон у корінь споживача як `.oxlintrc.json` за потреби.
7
7
  */
8
8
  import { readFileSync, writeFileSync } from 'node:fs'
9
9
  import { dirname, join } from 'node:path'
10
10
  import { fileURLToPath } from 'node:url'
11
11
 
12
- const dir = dirname(fileURLToPath(import.meta.url))
12
+ const dir = join(dirname(fileURLToPath(import.meta.url)), '..', 'js', 'data', 'tooling')
13
13
  const rules = {}
14
14
  for (const line of readFileSync(join(dir, 'oxlint-rules.tsv'), 'utf8').split('\n')) {
15
15
  const t = line.trim()
@@ -12,8 +12,8 @@ import { existsSync } from 'node:fs'
12
12
  import { readFile } from 'node:fs/promises'
13
13
  import { join, relative } from 'node:path'
14
14
 
15
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
16
- import { findAllPackageJsonPaths } from '../../../../scripts/utils/find-package-json-paths.mjs'
15
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
16
+ import { findAllPackageJsonPaths } from '../../../scripts/utils/find-package-json-paths.mjs'
17
17
  import {
18
18
  findMssqlPerRequestConnectionInText,
19
19
  findSharedMssqlRequestInText,
@@ -22,9 +22,9 @@ import {
22
22
  findUnsafeMssqlInListUnparsedInText,
23
23
  findUnsafeMssqlInListMissingEmptyGuardInText,
24
24
  isMssqlScanSourceFile
25
- } from './mssql-pool-scan.mjs'
26
- import { loadCursorIgnorePaths } from '../../../../scripts/utils/load-cursor-config.mjs'
27
- import { walkDir } from '../../../../scripts/utils/walkDir.mjs'
25
+ } from '../utils/mssql-pool-scan.mjs'
26
+ import { loadCursorIgnorePaths } from '../../../scripts/utils/load-cursor-config.mjs'
27
+ import { walkDir } from '../../../scripts/utils/walkDir.mjs'
28
28
 
29
29
  const VERSION_PREFIX_RE = /^[\^~>=<]+\s*/u
30
30
  const SEMVER_RE = /^(\d+)\.(\d+)\.(\d+)/u
@@ -30,7 +30,7 @@ import {
30
30
  normalizeSnippet,
31
31
  offsetToLine,
32
32
  walkAstWithAncestors
33
- } from '../../../../scripts/utils/ast-scan-utils.mjs'
33
+ } from '../../../scripts/utils/ast-scan-utils.mjs'
34
34
 
35
35
  const SOURCE_FILE_RE = /\.([cm]?[jt]sx?)$/
36
36
  const IN_PLACEHOLDER_END_RE = /\bin\s*\(\s*$/iu
@@ -40,24 +40,24 @@ import {
40
40
  findBunyanImportsInText,
41
41
  isBunyanScanSourceFile,
42
42
  shouldSkipFileForBunyanScan
43
- } from './bunyan-imports.mjs'
44
- import { findUncheckedProcessEnvInText, isCheckEnvScanSourceFile } from './check-env-scan.mjs'
45
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
46
- import { runConftestBatch } from '../../../../scripts/utils/run-conftest-batch.mjs'
47
- import { findConnFileRuleViolations, isConnFileRulesSourceFile } from './conn-file-rules.mjs'
43
+ } from '../utils/bunyan-imports.mjs'
44
+ import { findUncheckedProcessEnvInText, isCheckEnvScanSourceFile } from '../utils/check-env-scan.mjs'
45
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
46
+ import { runConftestBatch } from '../../../scripts/utils/run-conftest-batch.mjs'
47
+ import { findConnFileRuleViolations, isConnFileRulesSourceFile } from '../utils/conn-file-rules.mjs'
48
48
  import {
49
49
  findConnFactoryImportsInText,
50
50
  isConnImportsScanSourceFile,
51
51
  isInsideConnDir,
52
52
  resolveConnDirFromPackageJson
53
- } from './conn-imports-scan.mjs'
54
- import { loadCursorIgnorePaths } from '../../../../scripts/utils/load-cursor-config.mjs'
53
+ } from '../utils/conn-imports-scan.mjs'
54
+ import { loadCursorIgnorePaths } from '../../../scripts/utils/load-cursor-config.mjs'
55
55
  import {
56
56
  findPromiseSetTimeoutInText,
57
57
  isPromiseSetTimeoutScanSourceFile
58
- } from './promise-settimeout-scan.mjs'
59
- import { walkDir } from '../../../../scripts/utils/walkDir.mjs'
60
- import { getMonorepoPackageRootDirs } from '../../../../scripts/utils/workspaces.mjs'
58
+ } from '../utils/promise-settimeout-scan.mjs'
59
+ import { walkDir } from '../../../scripts/utils/walkDir.mjs'
60
+ import { getMonorepoPackageRootDirs } from '../../../scripts/utils/workspaces.mjs'
61
61
 
62
62
  /**
63
63
  * Чи існує непорожній за змістом маркер каталогу `src/` (рекомендована структура js-run).
@@ -18,7 +18,7 @@ import {
18
18
  offsetToLine,
19
19
  requireCallModule,
20
20
  walkAstWithAncestors
21
- } from '../../../../scripts/utils/ast-scan-utils.mjs'
21
+ } from '../../../scripts/utils/ast-scan-utils.mjs'
22
22
 
23
23
  const SOURCE_FILE_RE = /\.([cm]?[jt]sx?)$/u
24
24
  const FORBIDDEN_MODULES = new Set(['@nitra/bunyan', 'bunyan'])
@@ -29,7 +29,7 @@
29
29
  * Якщо ключ обчислюваний (`process.env[varName]`) — пропускаємо без помилки,
30
30
  * бо за статичним AST неможливо встановити, яка саме змінна оточення використовується.
31
31
  */
32
- import { offsetToLine, parseProgramOrNull, walkAstWithAncestors } from '../../../../scripts/utils/ast-scan-utils.mjs'
32
+ import { offsetToLine, parseProgramOrNull, walkAstWithAncestors } from '../../../scripts/utils/ast-scan-utils.mjs'
33
33
 
34
34
  const SOURCE_FILE_RE = /\.([cm]?[jt]sx?)$/u
35
35
  const IGNORE_DIRECTIVE_RE = /\/\/\s*@nitra\/cursor\s+ignore-next-line\s+checkEnv\b/u
@@ -14,7 +14,7 @@
14
14
  * Парсимо через oxc-parser; коли файл не парситься — повертаємо порожні результати, щоб
15
15
  * не змішувати помилки синтаксису з порушеннями цього правила.
16
16
  */
17
- import { parseProgramOrNull } from '../../../../scripts/utils/ast-scan-utils.mjs'
17
+ import { parseProgramOrNull } from '../../../scripts/utils/ast-scan-utils.mjs'
18
18
 
19
19
  const SOURCE_FILE_RE = /\.([cm]?[jt]sx?)$/u
20
20
 
@@ -16,7 +16,7 @@
16
16
  * використовується. Якщо файл не парситься — повертаємо порожній результат, спочатку
17
17
  * треба полагодити синтаксис.
18
18
  */
19
- import { langFromPath, normalizeSnippet, offsetToLine } from '../../../../scripts/utils/ast-scan-utils.mjs'
19
+ import { langFromPath, normalizeSnippet, offsetToLine } from '../../../scripts/utils/ast-scan-utils.mjs'
20
20
  import { parseSync } from 'oxc-parser'
21
21
 
22
22
  const SOURCE_FILE_RE = /\.([cm]?[jt]sx?)$/u
@@ -12,7 +12,7 @@
12
12
  * Сканер не вимагає, щоб файл компілювався: при синтаксичних помилках повертається
13
13
  * порожній результат (як інші сканери — спочатку треба полагодити синтаксис).
14
14
  */
15
- import { normalizeSnippet, offsetToLine, parseProgramOrNull } from '../../../../scripts/utils/ast-scan-utils.mjs'
15
+ import { normalizeSnippet, offsetToLine, parseProgramOrNull } from '../../../scripts/utils/ast-scan-utils.mjs'
16
16
 
17
17
  const SOURCE_FILE_RE = /\.([cm]?[jt]sx?)$/
18
18
 
@@ -140,10 +140,10 @@ import { basename, dirname, join, relative, resolve } from 'node:path'
140
140
 
141
141
  import { isSeq, parseAllDocuments, parseDocument } from 'yaml'
142
142
 
143
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
144
- import { loadCursorIgnorePaths } from '../../../../scripts/utils/load-cursor-config.mjs'
145
- import { runConftestBatch } from '../../../../scripts/utils/run-conftest-batch.mjs'
146
- import { walkDir } from '../../../../scripts/utils/walkDir.mjs'
143
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
144
+ import { loadCursorIgnorePaths } from '../../../scripts/utils/load-cursor-config.mjs'
145
+ import { runConftestBatch } from '../../../scripts/utils/run-conftest-batch.mjs'
146
+ import { walkDir } from '../../../scripts/utils/walkDir.mjs'
147
147
 
148
148
  /** Версія набору схем yannh — узгоджено з k8s.mdc */
149
149
  const YAML_LS_MODELINE_RE = /^# yaml-language-server: \$schema=.*\n/
package/rules/k8s/k8s.mdc CHANGED
@@ -39,7 +39,7 @@ alwaysApply: false
39
39
 
40
40
  Якщо в **корені проєкту** є файл **`.kubescape-exceptions.json`** — `lint-k8s` автоматично передає його в `kubescape scan` через **`--exceptions`** ([postureExceptionPolicy](https://github.com/kubescape/kubescape/blob/master/docs/exceptions.md)). Файл — JSON-масив об'єктів з полями `name`, `policyType: "postureExceptionPolicy"`, `actions` (`["alertOnly"]` — знижує fail до alert, не блокує lint), `resources` (resource designator) і `posturePolicies` (масив `controlID`).
41
41
 
42
- Канонічний кейс — **C-0012** (`Applications credentials in configuration files`, High): control тригериться на **імʼя** env, що містить підрядок `secret`/`password`/`key`/`token`, а **не** на значення. Для `HASURA_GRAPHQL_JWT_SECRET` у ConfigMap значення — публічний JWT-конфіг (`jwk_url`, `issuer`), не credentials, але kubescape падає лише через імʼя. Точкове виключення для ConfigMap із цим env — канон: [.kubescape-exceptions.json.snippet.json](./js/kubescape_exceptions/template/.kubescape-exceptions.json.snippet.json)
42
+ Канонічний кейс — **C-0012** (`Applications credentials in configuration files`, High): control тригериться на **імʼя** env, що містить підрядок `secret`/`password`/`key`/`token`, а **не** на значення. Для `HASURA_GRAPHQL_JWT_SECRET` у ConfigMap значення — публічний JWT-конфіг (`jwk_url`, `issuer`), не credentials, але kubescape падає лише через імʼя. Точкове виключення для ConfigMap із цим env — канон: [.kubescape-exceptions.json.snippet.json](./js/templates/kubescape_exceptions/.kubescape-exceptions.json.snippet.json)
43
43
 
44
44
  Підстав свою `attributes.name` (рядок або regex), якщо ConfigMap зветься інакше; виключай контрольно, а не глобально (не додавай винятки без `attributes.name`/`labels`, бо тоді C-0012 знімається для усіх ConfigMap-ів проєкту і реальні витоки credentials теж пройдуть).
45
45
 
@@ -17,11 +17,11 @@ import { existsSync } from 'node:fs'
17
17
  import { readdir, readFile, rename, unlink, writeFile } from 'node:fs/promises'
18
18
  import { basename, dirname, join, relative } from 'node:path'
19
19
 
20
- import { findDockerfilePaths } from '../../../docker/js/lint/check.mjs'
21
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
22
- import { loadCursorIgnorePaths } from '../../../../scripts/utils/load-cursor-config.mjs'
23
- import { runConftestBatch } from '../../../../scripts/utils/run-conftest-batch.mjs'
24
- import { walkDir } from '../../../../scripts/utils/walkDir.mjs'
20
+ import { findDockerfilePaths } from '../../docker/js/lint.mjs'
21
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
22
+ import { loadCursorIgnorePaths } from '../../../scripts/utils/load-cursor-config.mjs'
23
+ import { runConftestBatch } from '../../../scripts/utils/run-conftest-batch.mjs'
24
+ import { walkDir } from '../../../scripts/utils/walkDir.mjs'
25
25
 
26
26
  const LINE_SPLIT_RE = /\r?\n/u
27
27
  const INI_KEY_RE = /^([A-Za-z_]\w*)\s*=/u
@@ -37,10 +37,10 @@ import {
37
37
  langFromPath,
38
38
  requireCallModule,
39
39
  walkAstWithAncestors
40
- } from '../../../../scripts/utils/ast-scan-utils.mjs'
41
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
42
- import { loadCursorIgnorePaths } from '../../../../scripts/utils/load-cursor-config.mjs'
43
- import { walkDir } from '../../../../scripts/utils/walkDir.mjs'
40
+ } from '../../../scripts/utils/ast-scan-utils.mjs'
41
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
42
+ import { loadCursorIgnorePaths } from '../../../scripts/utils/load-cursor-config.mjs'
43
+ import { walkDir } from '../../../scripts/utils/walkDir.mjs'
44
44
 
45
45
  const execFileAsync = promisify(execFile)
46
46
 
@@ -11,7 +11,7 @@
11
11
  */
12
12
  import { existsSync } from 'node:fs'
13
13
 
14
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
14
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
15
15
 
16
16
  /**
17
17
  * Перевіряє відповідність проєкту правилам php.mdc.
@@ -9,9 +9,9 @@
9
9
  * JS тут лишається лише як cross-file гейт: walkDir не виразити декларативно через `target.json`.
10
10
  * Друк короткого pass-повідомлення з контекстом робить `check()` (необовʼязковий).
11
11
  */
12
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
13
- import { loadCursorIgnorePaths } from '../../../../scripts/utils/load-cursor-config.mjs'
14
- import { walkDir } from '../../../../scripts/utils/walkDir.mjs'
12
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
13
+ import { loadCursorIgnorePaths } from '../../../scripts/utils/load-cursor-config.mjs'
14
+ import { walkDir } from '../../../scripts/utils/walkDir.mjs'
15
15
 
16
16
  /**
17
17
  * Чи є хоча б один `.rego`-файл у дереві від `cwd`. Зупиняється на першому матчі.
@@ -29,8 +29,8 @@
29
29
  import { readFile } from 'node:fs/promises'
30
30
  import { relative, sep } from 'node:path'
31
31
 
32
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
33
- import { walkDir } from '../../../../scripts/utils/walkDir.mjs'
32
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
33
+ import { walkDir } from '../../../scripts/utils/walkDir.mjs'
34
34
 
35
35
  /** Суфікс basename'а прикладного файлу (`config.example`, `.env.dist`). */
36
36
  const EXAMPLE_SUFFIX_RE = /\.(?:example|sample|template|dist)$/iu
@@ -11,11 +11,11 @@ import { readFile } from 'node:fs/promises'
11
11
  import { dirname, join } from 'node:path'
12
12
  import { fileURLToPath } from 'node:url'
13
13
 
14
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
15
- import { checkTextSubset } from '../../../../scripts/utils/template.mjs'
14
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
15
+ import { checkTextSubset } from '../../../scripts/utils/template.mjs'
16
16
 
17
17
  const HERE = dirname(fileURLToPath(import.meta.url))
18
- const SNIPPET_PATH = join(HERE, 'template', '.trufflehog-exclude.snippet.txt')
18
+ const SNIPPET_PATH = join(HERE, 'templates', 'trufflehog', '.trufflehog-exclude.snippet.txt')
19
19
 
20
20
  /**
21
21
  * @returns {Promise<number>} exit-код перевірки
@@ -24,7 +24,7 @@ version: '2.1'
24
24
 
25
25
  ## `.trufflehog-exclude` (рекомендована основа)
26
26
 
27
- Канон (допускає розширення): [.trufflehog-exclude.snippet.txt](./js/trufflehog/template/.trufflehog-exclude.snippet.txt)
27
+ Канон (допускає розширення): [.trufflehog-exclude.snippet.txt](./js/templates/trufflehog/.trufflehog-exclude.snippet.txt)
28
28
 
29
29
  **Важливо:** один regex-pattern на рядок, без TOML-обгортки; коментарі починаються з `#`.
30
30
 
@@ -45,7 +45,7 @@ Workflow обовʼязковий — забезпечує незалежний
45
45
  - Правильно: `DB_PASSWORD=sample-secret`, `password: "sample-secret"`
46
46
  - Неправильно: `DB_PASSWORD=secret`, `password: "secret"`
47
47
 
48
- Перевіряється лише `secret` у позиції значення (після `=`, `:`, `=>`); імена ключів на кшталт `client_secret` не чіпаються. Concern `security.sample_secret` — деталі скану в `js/sample_secret/check.mjs`.
48
+ Перевіряється лише `secret` у позиції значення (після `=`, `:`, `=>`); імена ключів на кшталт `client_secret` не чіпаються. Concern `security.sample_secret` — деталі скану в `js/sample_secret.mjs`.
49
49
 
50
50
  ## Перевірка
51
51
 
@@ -19,7 +19,7 @@
19
19
  import { existsSync } from 'node:fs'
20
20
  import { readFile } from 'node:fs/promises'
21
21
 
22
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
22
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
23
23
 
24
24
  /**
25
25
  * Альтернатива полю `stylelint` у `package.json` — зовнішній файл конфігу. Якщо
@@ -18,8 +18,8 @@
18
18
  import { existsSync, statSync } from 'node:fs'
19
19
  import { readFile } from 'node:fs/promises'
20
20
 
21
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
22
- import { runConftestBatch } from '../../../../scripts/utils/run-conftest-batch.mjs'
21
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
22
+ import { runConftestBatch } from '../../../scripts/utils/run-conftest-batch.mjs'
23
23
 
24
24
  /**
25
25
  * Чи є префікс `@tauri-apps/` у ключах `dependencies` або `devDependencies`.
@@ -9,9 +9,9 @@
9
9
  */
10
10
  import { basename, dirname, relative } from 'node:path'
11
11
 
12
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
13
- import { loadCursorIgnorePaths } from '../../../../scripts/utils/load-cursor-config.mjs'
14
- import { walkDir } from '../../../../scripts/utils/walkDir.mjs'
12
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
13
+ import { loadCursorIgnorePaths } from '../../../scripts/utils/load-cursor-config.mjs'
14
+ import { walkDir } from '../../../scripts/utils/walkDir.mjs'
15
15
 
16
16
  const TESTS_DIR_NAME = 'tests'
17
17
 
@@ -32,8 +32,8 @@
32
32
  import { existsSync } from 'node:fs'
33
33
  import { readFile } from 'node:fs/promises'
34
34
 
35
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
36
- import { anyRunStepIncludes, parseWorkflowYaml } from '../../../../scripts/utils/gha-workflow.mjs'
35
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
36
+ import { anyRunStepIncludes, parseWorkflowYaml } from '../../../scripts/utils/gha-workflow.mjs'
37
37
 
38
38
  /** Заголовок абзацу про апостроф у text.mdc / n-text.mdc. */
39
39
  const UK_APOSTROPHE_HEADING = '**Український апостроф:**'
@@ -25,16 +25,16 @@ import { existsSync } from 'node:fs'
25
25
  import { readFile } from 'node:fs/promises'
26
26
  import { join, relative } from 'node:path'
27
27
 
28
- import { createCheckReporter } from '../../../../scripts/utils/check-reporter.mjs'
28
+ import { createCheckReporter } from '../../../scripts/utils/check-reporter.mjs'
29
29
  import {
30
30
  findForbiddenNodeImportsInVueFile,
31
31
  findForbiddenVueImportsInSourceFile,
32
32
  isVueImportScanSourceFile,
33
33
  shouldSkipFileForVueImportScan
34
- } from './vue-forbidden-imports.mjs'
35
- import { loadCursorIgnorePaths } from '../../../../scripts/utils/load-cursor-config.mjs'
36
- import { walkDir } from '../../../../scripts/utils/walkDir.mjs'
37
- import { getMonorepoPackageRootDirs } from '../../../../scripts/utils/workspaces.mjs'
34
+ } from '../utils/vue-forbidden-imports.mjs'
35
+ import { loadCursorIgnorePaths } from '../../../scripts/utils/load-cursor-config.mjs'
36
+ import { walkDir } from '../../../scripts/utils/walkDir.mjs'
37
+ import { getMonorepoPackageRootDirs } from '../../../scripts/utils/workspaces.mjs'
38
38
 
39
39
  const ESBUILD_RE = /\besbuild\b/
40
40
 
@@ -16,13 +16,13 @@ import { existsSync } from 'node:fs'
16
16
  import { readdir, readFile } from 'node:fs/promises'
17
17
  import { basename, join, relative } from 'node:path'
18
18
 
19
- import { textHasBunSqlImport } from '../rules/js-bun-db/js/safety/bun-sql-scan.mjs'
19
+ import { textHasBunSqlImport } from '../rules/js-bun-db/utils/bun-sql-scan.mjs'
20
20
  import {
21
21
  isGqlScanSourceFile,
22
22
  shouldSkipFileForGqlScan,
23
23
  sourceFileHasGqlTaggedTemplate
24
- } from '../rules/graphql/js/tooling/graphql-gql-scan.mjs'
25
- import { contentForVueImportScan } from '../rules/vue/js/packages/vue-forbidden-imports.mjs'
24
+ } from '../rules/graphql/utils/graphql-gql-scan.mjs'
25
+ import { contentForVueImportScan } from '../rules/vue/utils/vue-forbidden-imports.mjs'
26
26
 
27
27
  /** Порядок автододавання правил відповідно до `rules/<rule>/auto.md`. */
28
28
  export const AUTO_RULE_ORDER = Object.freeze([
@@ -57,7 +57,7 @@ const ADR_HOOK_SCRIPT_NAME = 'capture-decisions.sh'
57
57
  const ADR_NORMALIZE_HOOK_SCRIPT_NAME = 'normalize-decisions.sh'
58
58
  const TEMPLATE_DIR_NAME = '.claude-template'
59
59
  /** Відносний шлях до канонічного фрагмента `.gitignore` для ADR Stop-hook'ів у tarball пакета. */
60
- export const ADR_GITIGNORE_SNIPPET_REL = 'rules/adr/js/hooks/template/.gitignore.snippet'
60
+ export const ADR_GITIGNORE_SNIPPET_REL = 'rules/adr/js/templates/hooks/.gitignore.snippet'
61
61
  const GITIGNORE_FILE = '.gitignore'
62
62
  const EOL_RE = /\r?\n/u
63
63
 
@@ -1,29 +1,26 @@
1
1
  /**
2
- * Discovery rules для CLI `check`. Шукає правила, для яких є щось «прогонне»:
3
- * - JS concerns: `rules/<id>/js/<concern>/<check.mjs | check-*.mjs>` кожен concern окремий вузол.
2
+ * Discovery rules для CLI `fix`. Шукає правила, для яких є щось «прогонне»:
3
+ * - JS concerns: `rules/<id>/js/<concern>.mjs` один файл = один concern.
4
4
  * - Policy concerns: `rules/<id>/policy/<concern>/target.json` — пара з `<concern>.rego`.
5
5
  *
6
- * Каталог `js/utils/` свідомо пропускається — це хелпери, не концерни.
7
- * Файли `*.test.mjs` фільтруються regex (`^check(?:-.+)?\.mjs$`).
6
+ * Файли з префіксом `_` (зокрема каталог `_lib/`) і `*.test.mjs` пропускаються — це хелпери й тести.
8
7
  *
9
8
  * Намеренно НЕ парсимо `target.json` тут (це робить runner). Discovery — швидкий скан структури:
10
9
  * шляхи + назви, без I/O вмісту.
11
10
  *
12
- * Історичний контекст: convention пройшла еволюцію `js/` (до 1.11.12) → `fix/` (1.11.10–1.13.79)
13
- * → знову `js/` (1.13.80+, після появи кореневого `fix.mjs` як entry-point правила, щоб не плутати
14
- * з папкою `fix/`). Convention тепер за технологією: `js/` ↔ `policy/`.
11
+ * Історичний контекст: convention пройшла еволюцію
12
+ * `js/<concern>/check.mjs` (1.13.80–1.13.89)
13
+ * `js/<concern>.mjs` (1.13.90+, flat: концерн = файл, не каталог)
14
+ * Helpers, tests, templates і data винесені в окремі топ-level папки правила (`js/_lib/`,
15
+ * `tests/`, `templates/`, `data/`).
15
16
  */
16
17
  import { existsSync } from 'node:fs'
17
18
  import { readdir } from 'node:fs/promises'
18
19
  import { join } from 'node:path'
19
20
 
20
- const CHECK_FILENAME_RE = /^check(?:-.+)?\.mjs$/u
21
- const TEST_SUFFIX = '.test.mjs'
22
-
23
21
  /**
24
22
  * @typedef {object} JsConcern
25
- * @property {string} name імʼя концерну (`<name>` у `js/<name>/`)
26
- * @property {string[]} files імена `check*.mjs` у концерні (відсортовані алфавітно)
23
+ * @property {string} name імʼя концерну (= basename файла `js/<name>.mjs` без розширення)
27
24
  */
28
25
 
29
26
  /**
@@ -39,30 +36,26 @@ const TEST_SUFFIX = '.test.mjs'
39
36
  */
40
37
 
41
38
  /**
42
- * Перелічує JS-концерни одного правила: підкаталоги `js/<name>/` з принаймні одним `check*.mjs`.
39
+ * Перелічує JS-концерни одного правила: файли `js/<name>.mjs` (один файл один concern).
43
40
  *
44
- * `js/utils/` свідомо пропускається це хелпери, а не концерни.
41
+ * Файли з префіксом `_` (наприклад каталог `_lib/`) і `*.test.mjs` пропускаються.
45
42
  * @param {string} jsDir абсолютний шлях `rules/<id>/js/`
46
43
  * @returns {Promise<JsConcern[]>} концерни в алфавітному порядку
47
44
  */
48
45
  async function listJsConcerns(jsDir) {
49
46
  if (!existsSync(jsDir)) return []
50
- const topLevel = await readdir(jsDir, { withFileTypes: true })
51
-
47
+ const entries = await readdir(jsDir, { withFileTypes: true })
52
48
  /** @type {JsConcern[]} */
53
49
  const concerns = []
54
- for (const entry of topLevel) {
55
- if (!entry.isDirectory() || entry.name === 'utils' || entry.name.startsWith('.')) continue
56
- const concernDir = join(jsDir, entry.name)
57
- const dirContents = await readdir(concernDir)
58
- const files = dirContents
59
- .filter(n => CHECK_FILENAME_RE.test(n) && !n.endsWith(TEST_SUFFIX))
60
- .toSorted((a, b) => a.localeCompare(b))
61
- if (files.length > 0) {
62
- concerns.push({ name: entry.name, files })
63
- }
50
+ for (const entry of entries) {
51
+ if (!entry.isFile()) continue
52
+ if (!entry.name.endsWith('.mjs')) continue
53
+ if (entry.name.endsWith('.test.mjs')) continue
54
+ if (entry.name.startsWith('_')) continue
55
+ if (entry.name.startsWith('.')) continue
56
+ const name = entry.name.slice(0, -'.mjs'.length)
57
+ concerns.push({ name })
64
58
  }
65
-
66
59
  return concerns.toSorted((a, b) => a.name.localeCompare(b.name))
67
60
  }
68
61
 
@@ -3,7 +3,7 @@ import { readFile } from 'node:fs/promises'
3
3
  import { basename, extname, join } from 'node:path'
4
4
 
5
5
  const MD_LINK_RE = /\[([^\]]{1,200})\]\((\.\/[^)]{1,500})\)/g
6
- const TEMPLATE_SEGMENT_RE = /\/template\//
6
+ const TEMPLATE_SEGMENT_RE = /\/templates?\//
7
7
  /** Статичні regexp-літерали `^(.+)\.<slot>\.<ext>$` — без `RegExp(variable)`. */
8
8
  const SLOT_SUFFIX_RES = [/^(.+)\.snippet\.[^.]+$/, /^(.+)\.deny\.[^.]+$/, /^(.+)\.contains\.[^.]+$/]
9
9
 
@@ -1,10 +1,10 @@
1
1
  /**
2
- * Оркестратор одного правила під CLI `check`.
2
+ * Оркестратор одного правила під CLI `fix`.
3
3
  *
4
4
  * Послідовність (concerns у межах правила — алфавітно):
5
- * 1. **applies-гейт** з `js/applies/check.mjs`. Якщо модуль експортує `applies()` і вона повертає
5
+ * 1. **applies-гейт** з `js/applies.mjs`. Якщо модуль експортує `applies()` і вона повертає
6
6
  * false — друкуємо `✅ правило не застосовне` і завершуємо без подальших викликів.
7
- * 2. **JS-концерни** — кожен `check*.mjs` у `js/<concern>/`. Concern `applies` теж може мати
7
+ * 2. **JS-концерни** — кожен файл `js/<concern>.mjs`. Concern `applies` теж може мати
8
8
  * `check()` для друку контексту (його `applies()` уже відпрацював на кроці 1, він не повторюється).
9
9
  * 3. **Policy-концерни** — кожен `policy/<concern>/target.json` через `runConftestBatch`.
10
10
  * Резолвер `resolveTargetFiles` ділить cache (`walkCache`) між концернами.
@@ -24,32 +24,29 @@ import { resolveConcernTemplateData } from './template.mjs'
24
24
  const APPLIES_CONCERN_NAME = 'applies'
25
25
 
26
26
  /**
27
- * Обчислює абсолютний шлях до файла-check у JS-концерні: `rules/<id>/js/<concern>/<file>`.
28
- * Convention `js/` (за технологією) повернулась у 1.13.80 після короткої епохи `fix/`
29
- * (1.11.10–1.13.79) — щоб не плутати з кореневим `fix.mjs` entry-point'ом.
27
+ * Обчислює абсолютний шлях до файла-концерну: `rules/<id>/js/<concern>.mjs`.
28
+ * Flat-convention з 1.14.0 концерн = файл, не каталог.
30
29
  * @param {string} bundledRulesDir абсолютний `rules/`
31
30
  * @param {string} ruleId id правила
32
31
  * @param {import('./discover-checkable-rules.mjs').JsConcern} concern опис концерну
33
- * @param {string} fileName імʼя файла з `concern.files`
34
32
  * @returns {string} абсолютний шлях
35
33
  */
36
- function resolveJsCheckPath(bundledRulesDir, ruleId, concern, fileName) {
37
- return join(bundledRulesDir, ruleId, 'js', concern.name, fileName)
34
+ function resolveJsCheckPath(bundledRulesDir, ruleId, concern) {
35
+ return join(bundledRulesDir, ruleId, 'js', `${concern.name}.mjs`)
38
36
  }
39
37
 
40
38
  /**
41
- * Спробувати викликати applies() гейт з `js/applies/check.mjs` правила.
42
- * Гейт активний лише за наявності концерну з імʼям `applies` і експортом-функцією `applies` у його
43
- * першому check-файлі (алфавіт).
39
+ * Спробувати викликати applies() гейт з `js/applies.mjs` правила.
40
+ * Гейт активний лише за наявності концерну з імʼям `applies` і експортом-функцією `applies`.
44
41
  * @param {string} bundledRulesDir абсолютний `rules/`
45
42
  * @param {import('./discover-checkable-rules.mjs').CheckableRule} rule опис правила
46
43
  * @returns {Promise<boolean>} `true` — правило застосовне (або гейту немає); `false` — пропустити
47
44
  */
48
45
  async function evaluateAppliesGate(bundledRulesDir, rule) {
49
46
  const concern = rule.jsConcerns.find(c => c.name === APPLIES_CONCERN_NAME)
50
- if (!concern || concern.files.length === 0) return true
51
- const path = resolveJsCheckPath(bundledRulesDir, rule.id, concern, concern.files[0])
52
- // eslint-disable-next-line no-unsanitized/method -- path будується з discovered concern/file, які пройшли regex CHECK_FILENAME_RE
47
+ if (!concern) return true
48
+ const path = resolveJsCheckPath(bundledRulesDir, rule.id, concern)
49
+ // eslint-disable-next-line no-unsanitized/method -- path з discovered concern, файл з whitelist'у readdir
53
50
  const mod = await import(path)
54
51
  if (typeof mod.applies !== 'function') return true
55
52
  return Boolean(await mod.applies())
@@ -116,14 +113,12 @@ export async function runRule(rule, bundledRulesDir, walkCache) {
116
113
  let totalCode = 0
117
114
 
118
115
  for (const concern of rule.jsConcerns) {
119
- for (const fileName of concern.files) {
120
- const path = resolveJsCheckPath(bundledRulesDir, rule.id, concern, fileName)
121
- // eslint-disable-next-line no-unsanitized/method -- path будується з discovered concern/file, які пройшли regex CHECK_FILENAME_RE
122
- const mod = await import(path)
123
- if (typeof mod.check === 'function') {
124
- const code = await mod.check()
125
- if (code !== 0) totalCode = 1
126
- }
116
+ const path = resolveJsCheckPath(bundledRulesDir, rule.id, concern)
117
+ // eslint-disable-next-line no-unsanitized/method -- path з discovered concern, файл з whitelist'у readdir
118
+ const mod = await import(path)
119
+ if (typeof mod.check === 'function') {
120
+ const code = await mod.check()
121
+ if (code !== 0) totalCode = 1
127
122
  }
128
123
  }
129
124