@nitra/cursor 1.9.8 → 1.9.12
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.
- package/CHANGELOG.md +37 -2
- package/mdc/js-lint.mdc +6 -12
- package/package.json +1 -1
- package/policy/ga/workflow_common/workflow_common.rego +6 -1
- package/policy/js_lint/package_json/package_json.rego +1 -1
- package/policy/js_lint/package_json/package_json_test.rego +130 -0
- package/scripts/build-agents-commands.mjs +2 -1
- package/scripts/check-ga.mjs +0 -1
- package/scripts/check-js-lint.mjs +21 -61
- package/scripts/check-npm-module.mjs +4 -0
- package/scripts/claude-stop-hook.mjs +1 -1
- package/scripts/lint-ga.mjs +2 -2
- package/scripts/utils/knip-canonical.json +29 -0
package/CHANGELOG.md
CHANGED
|
@@ -4,11 +4,46 @@
|
|
|
4
4
|
|
|
5
5
|
Формат — [Keep a Changelog](https://keepachangelog.com/uk/1.1.0/), нумерація — [SemVer](https://semver.org/lang/uk/).
|
|
6
6
|
|
|
7
|
+
## [1.9.12] - 2026-05-13
|
|
8
|
+
|
|
9
|
+
### Removed
|
|
10
|
+
|
|
11
|
+
- **`check-js-lint.mjs` — видалено дубльовані JS-копії канону `lint-js`:** експорти `CANONICAL_LINT_JS`, `isCanonicalLintJs`, `normalizeLintJsScript`, `nitraEslintConfigMeetsMinVersion` і константу `WHITESPACE_RE` видалено. Ці функції експортувалися, але **не використовувалися** у `check()` — пер-документна перевірка кореневого `package.json` (канонічний `lint-js`, `@nitra/eslint-config ≥ 3.9.2`, `type: "module"`, `engines.{node,bun}`) давно мігровано до rego-полісі `npm/policy/js_lint/package_json/`. Залишені експорти створювали два джерела істини: при оновленні канону (`bunx knip --no-config-hints`) доводилось правити і `.rego`, і `.mjs`-константу, що відкриває дрифт. Тепер канон лише в rego, JS-копії немає. У `check-js-lint.mjs` додано коментар з посиланням на rego-полісі.
|
|
12
|
+
|
|
13
|
+
### Added
|
|
14
|
+
|
|
15
|
+
- **`npm/policy/js_lint/package_json/package_json_test.rego` — rego-тести для канону `lint-js`:** перенесено покриття з JS-тестів (`normalizeLintJsScript`, `isCanonicalLintJs`, `nitraEslintConfigMeetsMinVersion`) у rego через `json.patch`-фікстури: 16 нових `test_*`-кейсів — happy path, неправильний порядок команд, відсутність `bunx knip`, відсутність `--no-config-hints`, `type` не `module`, `engines.node < 24`, `engines.bun < 1.3`, `@nitra/eslint-config` < 3.9.2, `workspace:*` дозволено. Покривається `bun run lint-rego` (`conftest verify`).
|
|
16
|
+
|
|
17
|
+
### Changed
|
|
18
|
+
|
|
19
|
+
- **`npm/tests/check-js-lint.test.mjs` — прибрано тести для видалених JS-копій:** видалено блоки `describe('normalizeLintJsScript / isCanonicalLintJs')` і `describe('nitraEslintConfigMeetsMinVersion')` разом з імпортами. Залишилися тести `verifyOxlintRcAgainstCanonical` (там JS усе ще authoritative — потрібен readFile + рекурсивне порівняння канон-блока).
|
|
20
|
+
|
|
21
|
+
## [1.9.11] - 2026-05-13
|
|
22
|
+
|
|
23
|
+
### Changed
|
|
24
|
+
|
|
25
|
+
- **`lint-js` — `bunx knip --no-config-hints` у каноні (js-lint.mdc `1.20 → 1.21`):** до канонічного `lint-js`-скрипта додано прапор `--no-config-hints`, щоб knip не друкував щоразу інформаційну секцію «Configuration hints» (`Remove from ignoreDependencies/ignoreBinaries`). Hints не впливають на exit code і часто стосуються свідомо доданих ignore (`graphql` як peer-залежність) — щоразу їх читати немає сенсу. Оновлено: `CANONICAL_LINT_JS` у `check-js-lint.mjs`, `canonical_lint_js` у `npm_module_unused.../js_lint.package_json.rego`, приклади `lint-js`-скрипта і CI-блока у `npm/mdc/js-lint.mdc` + дзеркало `.cursor/rules/n-js-lint.mdc`, реальний `package.json#scripts.lint-js` у корені, реальний `.github/workflows/lint-js.yml`. Rego-deny `lint-js.yml: у run немає bunx knip` ловиться через `contains` — новий рядок з прапором проходить як раніше.
|
|
26
|
+
|
|
27
|
+
## [1.9.10] - 2026-05-13
|
|
28
|
+
|
|
29
|
+
### Added
|
|
30
|
+
|
|
31
|
+
- **`npm/scripts/utils/knip-canonical.json` — канонічний baseline `knip.json` для проєктів-споживачів:** покриває типові false-positives `bunx knip` для наших правил — `entry` зі CLI-конфігами (eslint/stylelint/oxlint/jscpd/markdownlint-cli2/commitlint), `project` для `**/*.{js,mjs,cjs,jsx,ts,tsx,mts,cts}`, `ignore` для `**/__fixtures__/**`, `ignoreDependencies` (`@nitra/cspell-dict`, `/@cspell\/dict-.+/`), `ignoreBinaries` для CLI з канону `bunx`/`npx` (`actionlint`, `cspell`, `depcheck`, `eslint`, `git-ai`, `jscpd`, `markdownlint-cli2`, `oxfmt`, `oxlint`, `shellcheck`, `uvx`, `v8r`, `zizmor`). Структурно поряд з `oxlint-canonical.json` / `oxlint-canonical-skeleton.json` у `npm/scripts/utils/`.
|
|
32
|
+
- **`js-lint.mdc` — секція про канонічний `knip.json` (версія `1.19 → 1.20`):** правило тепер вимагає `knip.json` у корені проєкту як стартовий baseline з канонічного файлу пакета. Перевіряється **лише наявність** — зміст подальших модифікацій локально не валідується (`entry` / `project` / `ignore` / `ignoreDependencies` / `ignoreBinaries` дозволені будь-які). Дзеркало — `.cursor/rules/n-js-lint.mdc`. Прибрано згадку про обовʼязковий `ignoreDependencies: ["graphql"]` як зміст-вимогу (тепер це лише стартова рекомендація через канон).
|
|
33
|
+
- **`check-js-lint.mjs::checkKnipConfig` — auto-create з канону:** якщо `knip.json` відсутній у корені, чек копіює `KNIP_CANONICAL_JSON_PATH` у `knip.json` і повідомляє pass про створення. Раніше функція падала з fail на відсутність і додатково перевіряла `ignoreDependencies ∋ "graphql"` — обидві перевірки замінено на FS-existence + копію канону (side effect, описано у `js-lint.mdc`).
|
|
34
|
+
|
|
35
|
+
## [1.9.9] - 2026-05-13
|
|
36
|
+
|
|
37
|
+
### Changed
|
|
38
|
+
|
|
39
|
+
- **AGENTS.md — додано `bunx knip` до секції Commands:** `build-agents-commands.mjs` тепер завжди додає рядок `- **knip (невикористані залежності та експорти)**: \`bunx knip\``після`npx @nitra/cursor check`; оновлено тест (`items.length`3 → 4, перевірка`toContain('bunx knip')`).
|
|
40
|
+
- **`knip.json` — `graphql` у `ignoreDependencies`:** повернуто `graphql` до кореневого `ignoreDependencies` (peer-залежність, яку knip фолсово репортить як unused; вимога `js-lint.mdc` / `check-js-lint.mjs`).
|
|
41
|
+
|
|
7
42
|
## [1.9.8] - 2026-05-12
|
|
8
43
|
|
|
9
44
|
### Changed
|
|
10
45
|
|
|
11
|
-
- **Корінь монорепо:**
|
|
46
|
+
- **Корінь монорепо:** локальне перевизначення `sonarjs/cognitive-complexity` у `eslint.config.js` прибрано — поріг і severity задаються в `@nitra/eslint-config`.
|
|
12
47
|
|
|
13
48
|
## [1.9.7] - 2026-05-12
|
|
14
49
|
|
|
@@ -18,7 +53,7 @@
|
|
|
18
53
|
|
|
19
54
|
- **ga (mdc v1.8 → v1.9) — заборона `depcheck` у workflow-файлах:** додано полісі `ga.workflow_common.deny` на будь-який виклик `depcheck` (через `npx`/`bunx`/`npm exec`/`pnpm exec` чи як standalone-команду) у `run:` кроку `.github/workflows/*.yml`. Перевірка невикористаних залежностей виконується разом з рештою лінтерів у `bun run lint-js` (`bunx knip`), окремий depcheck-крок у workflow зайвий. У `npm/mdc/ga.mdc` додано буліт «`depcheck`: не використовувати» з посиланням на `js-lint.mdc` і `ga.workflow_common`.
|
|
20
55
|
|
|
21
|
-
- **ci (тільки в цьому репо) — lint-ga встановлює conftest; knip.json налаштовано під монорепо:** `.github/workflows/lint-ga.yml` отримав крок `Install conftest` (curl
|
|
56
|
+
- **ci (тільки в цьому репо) — lint-ga встановлює conftest; knip.json налаштовано під монорепо:** `.github/workflows/lint-ga.yml` отримав крок `Install conftest` (curl-витяг релізу), бо `check-ga.mjs::runAllGaRego` ходить у `runConftestBatch` і hard-fail без бінарника. Кореневий `knip.json` розширено `workspaces.npm.entry` (всі CLI/scripts/tests як entry points — інакше knip false-positive репортить їх як unused), `ignoreBinaries` для `cspell`/`oxfmt`/`stylelint`/`vite` (всі через `bunx`/`npx`, не з deps), і `ignoreDependencies` для workspace self-refs. Це налаштування специфічне для цього репо; інші проєкти налаштовують `knip.json` під свою структуру.
|
|
22
57
|
|
|
23
58
|
### Removed
|
|
24
59
|
|
package/mdc/js-lint.mdc
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
description: Перевірка JavaScript коду
|
|
3
3
|
alwaysApply: true
|
|
4
|
-
version: '1.
|
|
4
|
+
version: '1.21'
|
|
5
5
|
---
|
|
6
6
|
|
|
7
7
|
**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 не додавай без потреби монорепо.
|
|
@@ -22,10 +22,10 @@ version: '1.19'
|
|
|
22
22
|
{
|
|
23
23
|
"type": "module",
|
|
24
24
|
"scripts": {
|
|
25
|
-
"lint-js": "bunx oxlint --fix && bunx eslint --fix . && bunx jscpd . && bunx knip"
|
|
25
|
+
"lint-js": "bunx oxlint --fix && bunx eslint --fix . && bunx jscpd . && bunx knip --no-config-hints"
|
|
26
26
|
},
|
|
27
27
|
"devDependencies": {
|
|
28
|
-
"@nitra/eslint-config": "^3.9.
|
|
28
|
+
"@nitra/eslint-config": "^3.9.4"
|
|
29
29
|
}
|
|
30
30
|
}
|
|
31
31
|
```
|
|
@@ -65,15 +65,9 @@ version: '1.19'
|
|
|
65
65
|
|
|
66
66
|
Перевірку невикористаних залежностей і експортів виконує **knip** (заміна `depcheck`). Викликається у скрипті `lint-js` і в CI разом з oxlint/eslint/jscpd — окремий крок у CI не потрібен.
|
|
67
67
|
|
|
68
|
-
У корені проєкту має бути `knip.json`
|
|
68
|
+
У корені проєкту має бути **`knip.json`**, який стартує з канонічного baseline з пакета `@nitra/cursor` — файл [`npm/scripts/utils/knip-canonical.json`](../scripts/utils/knip-canonical.json). Він покриває типові false-positives для наших правил: `entry` зі CLI-конфігами (eslint, stylelint, oxlint, jscpd, markdownlint-cli2, `commitlint`), `project` для `**/*.{js,mjs,cjs,jsx,ts,tsx,mts,cts}`, `ignore` для `**/__fixtures__/**`, `ignoreDependencies` для пакетів, посилання на які є лише в не-JS-конфігах (`@nitra/cspell-dict`, `/@cspell\/dict-.+/`), і `ignoreBinaries` для CLI, які канон вимагає викликати через `npx`/`bunx` і яких заборонено додавати в `devDependencies` (`actionlint`, `cspell`, `depcheck`, `eslint`, `git-ai`, `jscpd`, `markdownlint-cli2`, `oxfmt`, `oxlint`, `shellcheck`, `uvx`, `v8r`, `zizmor`).
|
|
69
69
|
|
|
70
|
-
|
|
71
|
-
{
|
|
72
|
-
"ignoreDependencies": [
|
|
73
|
-
"graphql"
|
|
74
|
-
]
|
|
75
|
-
}
|
|
76
|
-
```
|
|
70
|
+
Якщо `knip.json` відсутній — `npx @nitra/cursor check js-lint` копіює канон у корінь проєкту (side effect). Після створення модифікуй файл під свій проєкт як завгодно: перевіряємо лише наявність, зміст подальших змін не валідується.
|
|
77
71
|
|
|
78
72
|
Пакет `knip` окремо в `devDependencies` не додавай — `bunx knip` тягне його ad-hoc, як oxlint/eslint/jscpd.
|
|
79
73
|
|
|
@@ -134,7 +128,7 @@ jobs:
|
|
|
134
128
|
bunx oxlint
|
|
135
129
|
bunx eslint .
|
|
136
130
|
bunx jscpd .
|
|
137
|
-
bunx knip
|
|
131
|
+
bunx knip --no-config-hints
|
|
138
132
|
```
|
|
139
133
|
|
|
140
134
|
Перед **`./.github/actions/setup-bun-deps`** — **`actions/checkout@v6`** (див. **ga.mdc**). Composite: Node 24, Bun, кеш, `bun install --frozen-lockfile`.
|
package/package.json
CHANGED
|
@@ -55,6 +55,11 @@ setup_bun_no_checkout_template := concat(" ", [
|
|
|
55
55
|
"інакше runner не знайде action.yml (ga.mdc)",
|
|
56
56
|
])
|
|
57
57
|
|
|
58
|
+
forbidden_run_command_template := concat(" ", [
|
|
59
|
+
"jobs.%s.steps[%d]: `%s` заборонено у workflow —",
|
|
60
|
+
"мігровано на knip (js-lint.mdc, ga.mdc)",
|
|
61
|
+
])
|
|
62
|
+
|
|
58
63
|
# ── Аліаси на input ────────────────────────────────────────────────────────
|
|
59
64
|
|
|
60
65
|
# Усі jobs (з гарантією, що це обʼєкт) — щоб не падати на нетипових YAML.
|
|
@@ -88,7 +93,7 @@ deny contains msg if {
|
|
|
88
93
|
some entry in all_flat_steps
|
|
89
94
|
some name, pattern in forbidden_run_command_patterns
|
|
90
95
|
regex.match(pattern, step_run_text(entry.step))
|
|
91
|
-
msg := sprintf(
|
|
96
|
+
msg := sprintf(forbidden_run_command_template, [entry.job_id, entry.step_index, name])
|
|
92
97
|
}
|
|
93
98
|
|
|
94
99
|
# ── deny: shell-продовження `\` перед переносом рядка у `run:` ─────────────
|
|
@@ -17,7 +17,7 @@ package js_lint.package_json
|
|
|
17
17
|
|
|
18
18
|
import rego.v1
|
|
19
19
|
|
|
20
|
-
canonical_lint_js := "bunx oxlint --fix && bunx eslint --fix . && bunx jscpd . && bunx knip"
|
|
20
|
+
canonical_lint_js := "bunx oxlint --fix && bunx eslint --fix . && bunx jscpd . && bunx knip --no-config-hints"
|
|
21
21
|
|
|
22
22
|
# ── deny: `lint-js` скрипт ─────────────────────────────────────────────────
|
|
23
23
|
|
|
@@ -0,0 +1,130 @@
|
|
|
1
|
+
# Тести для `js_lint.package_json`. Запуск:
|
|
2
|
+
# conftest verify -p npm/policy/js_lint/package_json
|
|
3
|
+
package js_lint.package_json_test
|
|
4
|
+
|
|
5
|
+
import rego.v1
|
|
6
|
+
|
|
7
|
+
import data.js_lint.package_json
|
|
8
|
+
|
|
9
|
+
canonical_lint_js := "bunx oxlint --fix && bunx eslint --fix . && bunx jscpd . && bunx knip --no-config-hints"
|
|
10
|
+
|
|
11
|
+
valid_pkg := {
|
|
12
|
+
"type": "module",
|
|
13
|
+
"scripts": {"lint-js": canonical_lint_js},
|
|
14
|
+
"engines": {"node": ">=24", "bun": ">=1.3"},
|
|
15
|
+
"devDependencies": {"@nitra/eslint-config": "^3.9.2"},
|
|
16
|
+
}
|
|
17
|
+
|
|
18
|
+
# ── happy path ────────────────────────────────────────────────────────────
|
|
19
|
+
|
|
20
|
+
test_allow_canonical if {
|
|
21
|
+
count(package_json.deny) == 0 with input as valid_pkg
|
|
22
|
+
}
|
|
23
|
+
|
|
24
|
+
test_allow_workspace_eslint_config if {
|
|
25
|
+
pkg := json.patch(
|
|
26
|
+
valid_pkg,
|
|
27
|
+
[{"op": "replace", "path": "/devDependencies/@nitra~1eslint-config", "value": "workspace:*"}],
|
|
28
|
+
)
|
|
29
|
+
count(package_json.deny) == 0 with input as pkg
|
|
30
|
+
}
|
|
31
|
+
|
|
32
|
+
# ── lint-js ───────────────────────────────────────────────────────────────
|
|
33
|
+
|
|
34
|
+
test_deny_missing_lint_js if {
|
|
35
|
+
pkg := json.patch(valid_pkg, [{"op": "remove", "path": "/scripts/lint-js"}])
|
|
36
|
+
count(package_json.deny) > 0 with input as pkg
|
|
37
|
+
}
|
|
38
|
+
|
|
39
|
+
test_deny_lint_js_without_knip if {
|
|
40
|
+
pkg := json.patch(
|
|
41
|
+
valid_pkg,
|
|
42
|
+
[{"op": "replace", "path": "/scripts/lint-js", "value": "bunx oxlint --fix && bunx eslint --fix . && bunx jscpd ."}],
|
|
43
|
+
)
|
|
44
|
+
count(package_json.deny) > 0 with input as pkg
|
|
45
|
+
}
|
|
46
|
+
|
|
47
|
+
test_deny_lint_js_without_no_config_hints if {
|
|
48
|
+
without_flag := "bunx oxlint --fix && bunx eslint --fix . && bunx jscpd . && bunx knip"
|
|
49
|
+
pkg := json.patch(valid_pkg, [{"op": "replace", "path": "/scripts/lint-js", "value": without_flag}])
|
|
50
|
+
count(package_json.deny) > 0 with input as pkg
|
|
51
|
+
}
|
|
52
|
+
|
|
53
|
+
test_deny_lint_js_wrong_order if {
|
|
54
|
+
wrong_order := "bunx eslint --fix . && bunx oxlint --fix && bunx jscpd . && bunx knip --no-config-hints"
|
|
55
|
+
pkg := json.patch(valid_pkg, [{"op": "replace", "path": "/scripts/lint-js", "value": wrong_order}])
|
|
56
|
+
count(package_json.deny) > 0 with input as pkg
|
|
57
|
+
}
|
|
58
|
+
|
|
59
|
+
test_allow_lint_js_with_extra_whitespace if {
|
|
60
|
+
pkg := json.patch(
|
|
61
|
+
valid_pkg,
|
|
62
|
+
[{"op": "replace", "path": "/scripts/lint-js", "value": concat(" ", ["", canonical_lint_js, ""])}],
|
|
63
|
+
)
|
|
64
|
+
count(package_json.deny) == 0 with input as pkg
|
|
65
|
+
}
|
|
66
|
+
|
|
67
|
+
# ── type: module ──────────────────────────────────────────────────────────
|
|
68
|
+
|
|
69
|
+
test_deny_type_not_module if {
|
|
70
|
+
pkg := json.patch(valid_pkg, [{"op": "replace", "path": "/type", "value": "commonjs"}])
|
|
71
|
+
count(package_json.deny) > 0 with input as pkg
|
|
72
|
+
}
|
|
73
|
+
|
|
74
|
+
test_deny_type_missing if {
|
|
75
|
+
pkg := json.patch(valid_pkg, [{"op": "remove", "path": "/type"}])
|
|
76
|
+
count(package_json.deny) > 0 with input as pkg
|
|
77
|
+
}
|
|
78
|
+
|
|
79
|
+
# ── engines ──────────────────────────────────────────────────────────────
|
|
80
|
+
|
|
81
|
+
test_deny_node_below_24 if {
|
|
82
|
+
pkg := json.patch(valid_pkg, [{"op": "replace", "path": "/engines/node", "value": ">=22"}])
|
|
83
|
+
count(package_json.deny) > 0 with input as pkg
|
|
84
|
+
}
|
|
85
|
+
|
|
86
|
+
test_allow_node_above_24 if {
|
|
87
|
+
pkg := json.patch(valid_pkg, [{"op": "replace", "path": "/engines/node", "value": ">=25"}])
|
|
88
|
+
count(package_json.deny) == 0 with input as pkg
|
|
89
|
+
}
|
|
90
|
+
|
|
91
|
+
test_deny_bun_below_1_3 if {
|
|
92
|
+
pkg := json.patch(valid_pkg, [{"op": "replace", "path": "/engines/bun", "value": ">=1.2"}])
|
|
93
|
+
count(package_json.deny) > 0 with input as pkg
|
|
94
|
+
}
|
|
95
|
+
|
|
96
|
+
test_allow_bun_2_x if {
|
|
97
|
+
pkg := json.patch(valid_pkg, [{"op": "replace", "path": "/engines/bun", "value": "^2.0.0"}])
|
|
98
|
+
count(package_json.deny) == 0 with input as pkg
|
|
99
|
+
}
|
|
100
|
+
|
|
101
|
+
# ── @nitra/eslint-config ─────────────────────────────────────────────────
|
|
102
|
+
|
|
103
|
+
test_deny_eslint_config_below_3_9_2 if {
|
|
104
|
+
cases := [
|
|
105
|
+
"^3.9.1",
|
|
106
|
+
"^3.8.0",
|
|
107
|
+
"^3.6.12",
|
|
108
|
+
"^3.4.3",
|
|
109
|
+
]
|
|
110
|
+
some bad in cases
|
|
111
|
+
pkg := json.patch(valid_pkg, [{"op": "replace", "path": "/devDependencies/@nitra~1eslint-config", "value": bad}])
|
|
112
|
+
count(package_json.deny) > 0 with input as pkg
|
|
113
|
+
}
|
|
114
|
+
|
|
115
|
+
test_allow_eslint_config_above_3_9_2 if {
|
|
116
|
+
cases := [
|
|
117
|
+
"^3.9.2",
|
|
118
|
+
"^3.9.10",
|
|
119
|
+
"^3.10.0",
|
|
120
|
+
"^4.0.0",
|
|
121
|
+
]
|
|
122
|
+
some good in cases
|
|
123
|
+
pkg := json.patch(valid_pkg, [{"op": "replace", "path": "/devDependencies/@nitra~1eslint-config", "value": good}])
|
|
124
|
+
count(package_json.deny) == 0 with input as pkg
|
|
125
|
+
}
|
|
126
|
+
|
|
127
|
+
test_deny_missing_eslint_config if {
|
|
128
|
+
pkg := json.patch(valid_pkg, [{"op": "remove", "path": "/devDependencies/@nitra~1eslint-config"}])
|
|
129
|
+
count(package_json.deny) > 0 with input as pkg
|
|
130
|
+
}
|
|
@@ -81,7 +81,8 @@ export async function buildAgentsCommandBulletItems(projectRoot) {
|
|
|
81
81
|
{
|
|
82
82
|
name: `- **Оновити правила та ${AGENTS_MD}** (після змін у правилах/шаблоні CLI): \`npx ${PACKAGE_NAME}\``
|
|
83
83
|
},
|
|
84
|
-
{ name: `- **Перевірки правил (programmatic)**: \`npx ${PACKAGE_NAME} check\`` }
|
|
84
|
+
{ name: `- **Перевірки правил (programmatic)**: \`npx ${PACKAGE_NAME} check\`` },
|
|
85
|
+
{ name: `- **knip (невикористані залежності та експорти)**: \`bunx knip\`` }
|
|
85
86
|
)
|
|
86
87
|
|
|
87
88
|
return items
|
package/scripts/check-ga.mjs
CHANGED
|
@@ -48,7 +48,6 @@ function gitHasAnyTrackedFileMatchingGlob(globPattern) {
|
|
|
48
48
|
if (!p) return false
|
|
49
49
|
if (p.startsWith('!')) return true
|
|
50
50
|
try {
|
|
51
|
-
// eslint-disable-next-line sonarjs/no-os-command-from-path -- git як стандартне dev-середовище через PATH; альтернативи (хардкод шляху) непортативні
|
|
52
51
|
const out = execFileSync('git', ['ls-files', '-z', '--', `:(glob)${p}`], { encoding: 'utf8' })
|
|
53
52
|
return out.length > 0
|
|
54
53
|
} catch {
|
|
@@ -12,7 +12,7 @@
|
|
|
12
12
|
* `engines.bun` >= 1.3, `"type": "module"` у кореневому і всіх workspace `package.json`. Дубль перевірки JS у `lint.yml` — заборонено.
|
|
13
13
|
*/
|
|
14
14
|
import { existsSync } from 'node:fs'
|
|
15
|
-
import { readFile } from 'node:fs/promises'
|
|
15
|
+
import { copyFile, readFile } from 'node:fs/promises'
|
|
16
16
|
import { dirname, join } from 'node:path'
|
|
17
17
|
import { fileURLToPath } from 'node:url'
|
|
18
18
|
|
|
@@ -25,55 +25,18 @@ export const OXLINT_CANONICAL_JSON_PATH = join(
|
|
|
25
25
|
'oxlint-canonical.json'
|
|
26
26
|
)
|
|
27
27
|
|
|
28
|
-
/**
|
|
29
|
-
export const
|
|
28
|
+
/** Шлях до канонічного knip JSON у цьому пакеті — копіюється у корінь проєкту-споживача, якщо відсутній. */
|
|
29
|
+
export const KNIP_CANONICAL_JSON_PATH = join(dirname(fileURLToPath(import.meta.url)), 'utils', 'knip-canonical.json')
|
|
30
30
|
|
|
31
31
|
/** Мінімальні рекомендації розширень редактора з js-lint.mdc (eslint, oxlint, GA). */
|
|
32
32
|
export const REQUIRED_VSCODE_EXTENSIONS = ['dbaeumer.vscode-eslint', 'github.vscode-github-actions', 'oxc.oxc-vscode']
|
|
33
33
|
|
|
34
|
-
const WHITESPACE_RE = /\s+/gu
|
|
35
34
|
const NON_DIGITS_RE = /\D+/u
|
|
36
35
|
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
*/
|
|
42
|
-
export function normalizeLintJsScript(s) {
|
|
43
|
-
return String(s).trim().replaceAll(WHITESPACE_RE, ' ')
|
|
44
|
-
}
|
|
45
|
-
|
|
46
|
-
/**
|
|
47
|
-
* Чи рядок `lint-js` збігається з каноном (`bunx oxlint`, `bunx eslint`, `bunx jscpd`).
|
|
48
|
-
* @param {string} script значення `scripts.lint-js` з package.json
|
|
49
|
-
* @returns {boolean} true, якщо рядок канонічний
|
|
50
|
-
*/
|
|
51
|
-
export function isCanonicalLintJs(script) {
|
|
52
|
-
return normalizeLintJsScript(script) === CANONICAL_LINT_JS
|
|
53
|
-
}
|
|
54
|
-
|
|
55
|
-
/**
|
|
56
|
-
* Чи діапазон `@nitra/eslint-config` у `package.json` задовольняє мінімум `>= 3.9.2`
|
|
57
|
-
* (заборона `for...in` через `no-restricted-syntax` з 3.8.0 + вбудований ignore для ADR-каталогів
|
|
58
|
-
* у `getConfig` з 3.9.2 + транзитивний `@e18e/eslint-plugin` для oxlint).
|
|
59
|
-
* @param {unknown} versionSpec значення `devDependencies['@nitra/eslint-config']`
|
|
60
|
-
* @returns {boolean} true для `workspace:*` або першої semver у рядку >= 3.9.2
|
|
61
|
-
*/
|
|
62
|
-
export function nitraEslintConfigMeetsMinVersion(versionSpec) {
|
|
63
|
-
const s = String(versionSpec).trim()
|
|
64
|
-
if (s.startsWith('workspace:')) {
|
|
65
|
-
return true
|
|
66
|
-
}
|
|
67
|
-
const parts = s.split(NON_DIGITS_RE).filter(Boolean)
|
|
68
|
-
if (parts.length < 3) {
|
|
69
|
-
return false
|
|
70
|
-
}
|
|
71
|
-
const [major, minor, patch] = parts.slice(0, 3).map(Number)
|
|
72
|
-
if ([major, minor, patch].some(n => Number.isNaN(n))) {
|
|
73
|
-
return false
|
|
74
|
-
}
|
|
75
|
-
return major > 3 || (major === 3 && (minor > 9 || (minor === 9 && patch >= 2)))
|
|
76
|
-
}
|
|
36
|
+
// Канонічний рядок `lint-js`-скрипта і мінімальна версія `@nitra/eslint-config` —
|
|
37
|
+
// у rego (`npm/policy/js_lint/package_json/`). JS-копії (`CANONICAL_LINT_JS`,
|
|
38
|
+
// `isCanonicalLintJs`, `nitraEslintConfigMeetsMinVersion`) видалено, щоб не
|
|
39
|
+
// було двох джерел істини й ризику дрифту.
|
|
77
40
|
|
|
78
41
|
/**
|
|
79
42
|
* Рекурсивне порівняння фрагментів канону oxlint (масиви — порядок як у каноні; об’єкти — той самий набір ключів і вкладеність).
|
|
@@ -473,31 +436,28 @@ async function checkJscpdConfig(passFn, failFn) {
|
|
|
473
436
|
}
|
|
474
437
|
|
|
475
438
|
/**
|
|
476
|
-
* Перевіряє `knip.json
|
|
477
|
-
*
|
|
478
|
-
*
|
|
439
|
+
* Перевіряє наявність `knip.json` у корені проєкту. Якщо файл відсутній —
|
|
440
|
+
* копіює канонічний `knip-canonical.json` з пакета `@nitra/cursor` як стартовий
|
|
441
|
+
* baseline; зміст подальших модифікацій локально не валідується (`entry` /
|
|
442
|
+
* `project` / `ignore` / `ignoreDependencies` / `ignoreBinaries` дозволені
|
|
443
|
+
* будь-які; це side effect — описано у js-lint.mdc).
|
|
479
444
|
* @param {(msg: string) => void} passFn callback при успішній перевірці
|
|
480
445
|
* @param {(msg: string) => void} failFn callback при помилці
|
|
481
446
|
*/
|
|
482
447
|
async function checkKnipConfig(passFn, failFn) {
|
|
483
|
-
if (
|
|
484
|
-
|
|
448
|
+
if (existsSync('knip.json')) {
|
|
449
|
+
passFn('knip.json існує')
|
|
485
450
|
return
|
|
486
451
|
}
|
|
487
|
-
|
|
488
|
-
|
|
489
|
-
|
|
490
|
-
|
|
491
|
-
|
|
452
|
+
if (!existsSync(KNIP_CANONICAL_JSON_PATH)) {
|
|
453
|
+
failFn(
|
|
454
|
+
`knip.json відсутній, і канонічний шаблон у пакеті не знайдено (${KNIP_CANONICAL_JSON_PATH}) — ` +
|
|
455
|
+
'перевстанови @nitra/cursor'
|
|
456
|
+
)
|
|
492
457
|
return
|
|
493
458
|
}
|
|
494
|
-
|
|
495
|
-
|
|
496
|
-
if (Array.isArray(ignoreDeps) && ignoreDeps.includes('graphql')) {
|
|
497
|
-
passFn('knip.json: ignoreDependencies містить "graphql"')
|
|
498
|
-
} else {
|
|
499
|
-
failFn('knip.json: ignoreDependencies має містити "graphql" (peer-залежність) (js-lint.mdc)')
|
|
500
|
-
}
|
|
459
|
+
await copyFile(KNIP_CANONICAL_JSON_PATH, 'knip.json')
|
|
460
|
+
passFn('knip.json створено з канонічного npm/scripts/utils/knip-canonical.json (js-lint.mdc)')
|
|
501
461
|
}
|
|
502
462
|
|
|
503
463
|
/**
|
|
@@ -366,6 +366,10 @@ export function globToRegex(glob) {
|
|
|
366
366
|
re = re.replace(GLOBSTAR_LEADING_RE, '(?:.*/)?')
|
|
367
367
|
re = re.replace(GLOBSTAR_TRAILING_RE, '(?:/.*)?')
|
|
368
368
|
re = re.replaceAll('__GLOBSTAR__', '.*')
|
|
369
|
+
// Дозволено: уся функція існує саме для конструкції RegExp з glob-pattern
|
|
370
|
+
// у `files` (значення з npm/package.json, не від кінцевого користувача), і
|
|
371
|
+
// спецсимволи вже екрановано через `REGEX_SPECIAL_IN_GLOB` вище.
|
|
372
|
+
// eslint-disable-next-line security/detect-non-literal-regexp
|
|
369
373
|
return new RegExp(`^${re}$`, 'u')
|
|
370
374
|
}
|
|
371
375
|
|
|
@@ -62,7 +62,7 @@ export async function runStopHookCli() {
|
|
|
62
62
|
if (isRecursiveStopHookCall(stdin)) {
|
|
63
63
|
return 0
|
|
64
64
|
}
|
|
65
|
-
|
|
65
|
+
|
|
66
66
|
const child = spawn('npx', ['--no', '@nitra/cursor', 'check'], { stdio: 'inherit' })
|
|
67
67
|
try {
|
|
68
68
|
const [code] = await once(child, 'exit')
|
package/scripts/lint-ga.mjs
CHANGED
|
@@ -8,7 +8,7 @@
|
|
|
8
8
|
* `check-ga.mjs::check()` як перший крок — `lint-ga.mjs` про це не знає. Раніше `lint-ga.mjs` сам
|
|
9
9
|
* спавнив conftest для `ga.<name>` per-workflow і `ga.workflow_common` (PoC); тепер ця логіка
|
|
10
10
|
* централізована у `check-ga.mjs`, тож одне джерело істини, без дублювання між
|
|
11
|
-
* `lint-ga` і `npx
|
|
11
|
+
* `lint-ga` і `npx \@nitra/cursor check ga`.
|
|
12
12
|
*
|
|
13
13
|
* Без preflight `actionlint` (через `bunx github-actionlint`) мовчки пропускає shell-перевірки в
|
|
14
14
|
* `run:` блоках, коли `shellcheck` відсутній у PATH; локально `bun lint-ga` лишається зеленим, а CI
|
|
@@ -143,7 +143,7 @@ function runStep(title, cmd, args) {
|
|
|
143
143
|
* 2) `bunx github-actionlint`;
|
|
144
144
|
* 3) `uvx zizmor --offline --collect=workflows .`;
|
|
145
145
|
* 4) `check-ga.mjs::check()` — Rego-полісі (батч conftest з `npm/policy/ga/`) + JS cross-file
|
|
146
|
-
* перевірки правил `ga.mdc`. Це **те саме**, що робить `npx
|
|
146
|
+
* перевірки правил `ga.mdc`. Це **те саме**, що робить `npx \@nitra/cursor check ga`, тож
|
|
147
147
|
* `lint-ga` тепер є суперсетом перевірки правила: external-tools + check.
|
|
148
148
|
*
|
|
149
149
|
* Якщо хоча б один preflight не пройшов — виходимо одразу з кодом 1, **до** запуску actionlint/zizmor,
|
|
@@ -0,0 +1,29 @@
|
|
|
1
|
+
{
|
|
2
|
+
"$schema": "https://unpkg.com/knip@5/schema.json",
|
|
3
|
+
"entry": [
|
|
4
|
+
"eslint.config.{js,mjs,cjs}",
|
|
5
|
+
"stylelint.config.{js,cjs,mjs}",
|
|
6
|
+
"oxlint.config.{js,mjs}",
|
|
7
|
+
"jscpd.config.{js,mjs}",
|
|
8
|
+
"markdownlint-cli2.config.{js,mjs,cjs,jsonc}",
|
|
9
|
+
".markdownlint-cli2.{jsonc,cjs,mjs}",
|
|
10
|
+
"commitlint.config.{js,cjs,mjs}"
|
|
11
|
+
],
|
|
12
|
+
"project": ["**/*.{js,mjs,cjs,jsx,ts,tsx,mts,cts}"],
|
|
13
|
+
"ignore": ["**/__fixtures__/**"],
|
|
14
|
+
"ignoreDependencies": ["@nitra/cspell-dict", "/@cspell\\/dict-.+/", "graphql"],
|
|
15
|
+
"ignoreBinaries": [
|
|
16
|
+
"actionlint",
|
|
17
|
+
"cspell",
|
|
18
|
+
"eslint",
|
|
19
|
+
"git-ai",
|
|
20
|
+
"jscpd",
|
|
21
|
+
"markdownlint-cli2",
|
|
22
|
+
"oxfmt",
|
|
23
|
+
"oxlint",
|
|
24
|
+
"shellcheck",
|
|
25
|
+
"uvx",
|
|
26
|
+
"v8r",
|
|
27
|
+
"zizmor"
|
|
28
|
+
]
|
|
29
|
+
}
|