@nitra/cursor 1.8.99 → 1.8.102

package/mdc/abie.mdc

@@ -115,9 +115,9 @@ spec:
 
 ## k8s: overlay **ru** і **Service** (у т. ч. headless → NodePort)
 
-Для кожного **Service** в YAML під **`…/k8s/…`**, де шлях файлу **не** містить **`k8s/ua/`** чи **`k8s/ru/`** (маніфести base / спільного шару; у т. ч. **headless** з **`spec.clusterIP: None`** і **`-hl`**), якщо ще не **`spec.type: NodePort`** / **`LoadBalancer`** / **`ExternalName`**, у **`k8s/ru/kustomization.yaml`** того ж пакета (overlay **ru**) додай **inline** **JSON6902** у **`patches`**: **`target.kind: Service`**, **`target.name`** як у маніфеста, **`path: /spec/type`**, **`value: NodePort`**. Якщо в base було **`spec.clusterIP: None`** або **`spec.clusterIPs`** з **`None`** (типово **headless**), у тому ж **patch** додай **`op: remove`** для **`/spec/clusterIP`** та **`/spec/clusterIPs`** — інакше API відхилить **NodePort** з помилкою на **`clusterIPs`**. Деталі — **`check-abie.mjs`**.
+Для кожного **Service** в YAML під **`…/k8s/…`**, де шлях файлу **не** містить **`k8s/ua/`** чи **`k8s/ru/`** (маніфести base / спільного шару; у т. ч. **headless** з **`spec.clusterIP: None`** і **`-hl`**), якщо ще не **`spec.type: NodePort`** / **`LoadBalancer`** / **`ExternalName`**, у **`k8s/ru/kustomization.yaml`** того ж пакета (overlay **ru**) додай **inline** **JSON6902** у **`patches`**: **`target.kind: Service`**, **`target.name`** як у маніфеста, **`path: /spec/type`**, **`value: NodePort`**. Якщо в base було **`spec.clusterIP: None`**, у тому ж **patch** додай **`op: remove`** для **`/spec/clusterIP`**. Якщо в base **явно** задано **`spec.clusterIPs`** (зокрема **`['None']`**), додай **`op: remove`** і для **`/spec/clusterIPs`** — інакше **API** може відхилити **NodePort** (*`spec.clusterIPs[0]: Invalid value: "None"`*). **Не** додавай **`remove`** на **`/spec/clusterIPs`**, якщо ключа **немає** в base: **`kubectl kustomize`** тоді падає (*Unable to remove nonexistent key*). Якщо в base лише **`clusterIP: None`**, а помилка лишається після **`kubectl apply -k`** (злиття з уже існуючим **Service** у кластері), тимчасово **видали** **`Service`** у **`ru`** і застосуй знову, або додай у base поле **`clusterIPs`**, щоб **`remove`** у patch був валідний для **kustomize**. Деталі — **`check-abie.mjs`**.
 
-```yaml title="…/ru/kustomization.yaml (фрагмент, headless → NodePort)"
+```yaml title="…/ru/kustomization.yaml (фрагмент, headless → NodePort, без clusterIPs у base)"
 patches:
   - target:
       kind: Service
@@ -128,10 +128,10 @@ patches:
         value: NodePort
       - op: remove
         path: /spec/clusterIP
-      - op: remove
-        path: /spec/clusterIPs
 ```
 
+Якщо в base у цього **Service** уже є **`spec.clusterIPs`**, той самий **patch** розшир **remove** на **`/spec/clusterIPs`** (див. **`check-abie.mjs`**).
+
 ## k8s: overlay **ru** і HealthCheckPolicy
 
 Якщо в дереві **k8s** є **HealthCheckPolicy**, у **`ru/kustomization.yaml`** має бути patch **`$patch: delete`** для політики (узгоджено з **k8s.mdc**; перевірка в **`check-k8s.mjs`**, **`ruKustomizationHasHealthCheckDeletePatch`**). Підстав реальне ім’я замість **`СЕРВІС`**:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nitra/cursor",
-  "version": "1.8.99",
+  "version": "1.8.102",
   "description": "CLI для завантаження cursor-правил (префікс n-) у локальний репозиторій",
   "keywords": [
     "cli",

package/scripts/check-abie.mjs

@@ -34,7 +34,7 @@
  * Вибір **`op`** — **k8s.mdc**.
  *
  * **Service (overlay ru):** для кожного **Service**, оголошеного в YAML під **`…/k8s/…`**, де шлях **не** проходить через **`k8s/ua/`** чи **`k8s/ru/`** (маніфести base / спільного шару, у т. ч. **headless** з **`clusterIP: None`** і **`-hl`**), якщо ще не **NodePort** / **LoadBalancer** / **ExternalName**,
- * у файлі **`k8s/ru/kustomization.yaml`** того ж пакета (overlay середовища **ru**) — inline **JSON6902** на **`kind: Service`** з тим самим **`target.name`**: **`path: /spec/type`**, **`value: NodePort`**; якщо в base було **`spec.clusterIP: None`** або **`spec.clusterIPs`** з **`None`** — у тому ж patch додай **`op: remove`** для **`/spec/clusterIP`** та **`/spec/clusterIPs`** (інакше **NodePort** з **`None`** відхиляє API).
+ * у файлі **`k8s/ru/kustomization.yaml`** того ж пакета (overlay середовища **ru**) — inline **JSON6902** на **`kind: Service`** з тим самим **`target.name`**: **`path: /spec/type`**, **`value: NodePort`**; якщо в base було **`spec.clusterIP: None`** — **`op: remove`** для **`/spec/clusterIP`**; якщо в base **явно** задано **`spec.clusterIPs`** — також **`remove`** для **`/spec/clusterIPs`** (інакше **API** може залишити **`None`** для **NodePort**; без ключа **`clusterIPs`** у base **`remove`** на **`/spec/clusterIPs`** ламає **`kubectl kustomize`**).
  */
 import { existsSync } from 'node:fs'
 import { readFile } from 'node:fs/promises'
@@ -370,9 +370,9 @@ export function serviceDocumentRequiresAbieRuNodePortOverlay(obj) {
 }
 
 /**
- * Чи в base-**Service** є **headless** **`None`**, який треба прибрати в **ru** перед **NodePort** (**clusterIP** / **clusterIPs**).
+ * Чи в base-**Service** задано **headless** через **`spec.clusterIP: None`**, який треба прибрати в **ru** перед **NodePort**.
  * @param {unknown} obj корінь YAML (**Service**)
- * @returns {boolean} **true**, якщо **`spec.clusterIP === 'None'`** або **`spec.clusterIPs`** містить **`'None'`**
+ * @returns {boolean} **true**, якщо **`spec.clusterIP === 'None'`**
  */
 export function serviceDocumentRequiresRuClusterIPNoneRemoval(obj) {
   if (!isServiceDoc(obj)) {
@@ -384,20 +384,31 @@ export function serviceDocumentRequiresRuClusterIPNoneRemoval(obj) {
     return false
   }
   const sp = /** @type {Record<string, unknown>} */ (spec)
-  if (sp.clusterIP === 'None') {
-    return true
+  return sp.clusterIP === 'None'
+}
+
+/**
+ * Чи в base-**Service** у **`spec`** явно задано поле **`clusterIPs`** (тоді **`remove`** на **`/spec/clusterIPs`** безпечний для **`kubectl kustomize`**).
+ * @param {unknown} obj корінь YAML (**Service**)
+ * @returns {boolean} **true**, якщо **`Object.hasOwn(spec, 'clusterIPs')`**
+ */
+export function serviceDocumentBaseDeclaresClusterIPsField(obj) {
+  if (!isServiceDoc(obj)) {
+    return false
   }
-  const cips = sp.clusterIPs
-  if (Array.isArray(cips) && cips.includes('None')) {
-    return true
+  const rec = /** @type {Record<string, unknown>} */ (obj)
+  const spec = rec.spec
+  if (spec === null || typeof spec !== 'object' || Array.isArray(spec)) {
+    return false
   }
-  return false
+  const sp = /** @type {Record<string, unknown>} */ (spec)
+  return Object.hasOwn(sp, 'clusterIPs')
 }
 
 /**
  * Чи **JSON6902**-текст містить **`op: remove`** для заданого **`path`** (порядок ключів **op** / **path** неважливий).
  * @param {string} patchText поле **patch** у kustomization
- * @param {string} posixPath наприклад **`/spec/clusterIP`**
+ * @param {string} posixPath **`/spec/clusterIP`** або **`/spec/clusterIPs`**
  * @returns {boolean} true, якщо знайдено пару **remove** + **path**
  */
 export function jsonPatchRemovesPath(patchText, posixPath) {
@@ -416,12 +427,12 @@ export function jsonPatchRemovesPath(patchText, posixPath) {
 }
 
 /**
- * Чи patch прибирає **headless** поля **`clusterIP`** / **`clusterIPs`**, щоб **NodePort** пройшов валідацію API.
+ * Чи patch містить **`op: remove`** для **`/spec/clusterIP`**, щоб прибрати **headless** перед **NodePort**.
  * @param {string} patchText поле **patch** у kustomization
- * @returns {boolean} true, якщо є **remove** і для **`/spec/clusterIP`**, і для **`/spec/clusterIPs`**
+ * @returns {boolean} true, якщо є **remove** для **`/spec/clusterIP`**
  */
 export function jsonPatchTextClearsHeadlessServiceClusterIPNone(patchText) {
-  return jsonPatchRemovesPath(patchText, '/spec/clusterIP') && jsonPatchRemovesPath(patchText, '/spec/clusterIPs')
+  return jsonPatchRemovesPath(patchText, '/spec/clusterIP')
 }
 
 /**
@@ -516,7 +527,7 @@ function collectAbieRuServicePatchTextByTargetNameFromRaw(raw) {
 /**
  * Повідомлення про порушення patch **Service** у **ru/kustomization.yaml** (abie.mdc).
  * @param {string} raw повний текст **kustomization.yaml**
- * @param {Map<string, { requiresClusterIPNoneClear: boolean }>} targetsByName ім’я **Service** → чи треба прибрати **None**
+ * @param {Map<string, { requiresClusterIPNoneClear: boolean, requiresClusterIPsRemove?: boolean }>} targetsByName ім’я **Service** → прапорці patch
  * @returns {string[]} порожньо, якщо все OK
  */
 export function getAbieRuServiceNodePortPatchErrors(raw, targetsByName) {
@@ -528,7 +539,8 @@ export function getAbieRuServiceNodePortPatchErrors(raw, targetsByName) {
   const errors = []
   for (const name of [...targetsByName.keys()].toSorted((a, b) => a.localeCompare(b))) {
     const flags = targetsByName.get(name)
-    const requiresClear = flags?.requiresClusterIPNoneClear === true
+    const requiresClusterIPRemove = flags?.requiresClusterIPNoneClear === true
+    const requiresClusterIPsRemove = flags?.requiresClusterIPsRemove === true
     const pt = byName.get(name)
     if (pt === undefined || String(pt).trim() === '') {
       errors.push(`${name}: немає inline patch для kind: Service`)
@@ -536,9 +548,14 @@ export function getAbieRuServiceNodePortPatchErrors(raw, targetsByName) {
       if (!jsonPatchTextSetsServiceTypeNodePort(pt)) {
         errors.push(`${name}: потрібен JSON6902 path /spec/type та value NodePort`)
       }
-      if (requiresClear && !jsonPatchTextClearsHeadlessServiceClusterIPNone(pt)) {
+      if (requiresClusterIPRemove && !jsonPatchTextClearsHeadlessServiceClusterIPNone(pt)) {
+        errors.push(
+          `${name}: для spec.clusterIP: None додай у той самий patch op: remove для path /spec/clusterIP (abie.mdc)`
+        )
+      }
+      if (requiresClusterIPsRemove && !jsonPatchRemovesPath(pt, '/spec/clusterIPs')) {
         errors.push(
-          `${name}: для spec.clusterIP/spec.clusterIPs: None додай у той самий patch op: remove для path /spec/clusterIP та /spec/clusterIPs (abie.mdc)`
+          `${name}: у base задано spec.clusterIPs — додай op: remove для path /spec/clusterIPs (інакше NodePort з None у clusterIPs; abie.mdc)`
         )
       }
     }
@@ -551,10 +568,10 @@ export function getAbieRuServiceNodePortPatchErrors(raw, targetsByName) {
  * @param {string} root корінь репозиторію
  * @param {string[]} yamlAbs абсолютні шляхи yaml під **k8s**
  * @param {(msg: string) => void} fail реєстрація помилки читання/парсингу
- * @returns {Promise<Map<string, Map<string, { requiresClusterIPNoneClear: boolean }>>>} **pkgAbs** → (**ім’я** → прапорці)
+ * @returns {Promise<Map<string, Map<string, { requiresClusterIPNoneClear: boolean, requiresClusterIPsRemove: boolean }>>>} **pkgAbs** → (**ім’я** → прапорці)
  */
 async function collectAbieRuNodePortServiceTargetsByPackage(root, yamlAbs, fail) {
-  /** @type {Map<string, Map<string, { requiresClusterIPNoneClear: boolean }>>} */
+  /** @type {Map<string, Map<string, { requiresClusterIPNoneClear: boolean, requiresClusterIPsRemove: boolean }>>} */
   const map = new Map()
   for (const abs of yamlAbs) {
     const rel = relative(root, abs).replaceAll('\\', '/') || abs
@@ -600,9 +617,11 @@ async function collectAbieRuNodePortServiceTargetsByPackage(root, yamlAbs, fail)
                       map.set(pkgAbs, inner)
                     }
                     const needClear = serviceDocumentRequiresRuClusterIPNoneRemoval(obj)
+                    const needClusterIPsRemove = serviceDocumentBaseDeclaresClusterIPsField(obj)
                     const prev = inner.get(n)
                     inner.set(n, {
-                      requiresClusterIPNoneClear: (prev?.requiresClusterIPNoneClear === true) || needClear
+                      requiresClusterIPNoneClear: (prev?.requiresClusterIPNoneClear === true) || needClear,
+                      requiresClusterIPsRemove: (prev?.requiresClusterIPsRemove === true) || needClusterIPsRemove
                     })
                   }
                 }
@@ -617,7 +636,7 @@ async function collectAbieRuNodePortServiceTargetsByPackage(root, yamlAbs, fail)
 }
 
 /**
- * У **`k8s/ru/kustomization.yaml`** для кожного **Service** з YAML **`k8s`**, шлях якого без сегментів **`k8s/ua/`** та **`k8s/ru/`** (у т. ч. **headless** / **`-hl`**) — **JSON6902** **`/spec/type` → NodePort**; якщо в base було **`clusterIP: None`** / **`clusterIPs: None`** — також **`op: remove`** на **`/spec/clusterIP`** та **`/spec/clusterIPs`** (abie.mdc).
+ * У **`k8s/ru/kustomization.yaml`** для кожного **Service** з YAML **`k8s`**, шлях якого без сегментів **`k8s/ua/`** та **`k8s/ru/`** (у т. ч. **headless** / **`-hl`**) — **JSON6902** **`/spec/type` → NodePort**; при **`clusterIP: None`** — **`op: remove`** на **`/spec/clusterIP`**; якщо в base є **`spec.clusterIPs`** — ще **`remove`** на **`/spec/clusterIPs`** (abie.mdc).
  * @param {string} root корінь
  * @param {string[]} yamlFilesAbs yaml під **k8s**
  * @param {(msg: string) => void} fail callback
@@ -637,7 +656,7 @@ async function ensureRuAbieServiceNodePortPatches(root, yamlFilesAbs, fail, pass
     const nameList = [...targetsByName.keys()].toSorted((a, b) => a.localeCompare(b))
     if (!existsSync(ruAbs)) {
       fail(
-        `${relPkg}/k8s: є Service, для overlay ru потрібен patch Service (NodePort; для headless — ще remove clusterIP/clusterIPs): ${nameList.join(', ')} — додай ru/kustomization.yaml (abie.mdc)`
+        `${relPkg}/k8s: є Service, для overlay ru потрібен patch Service (NodePort; для headless — ще remove /spec/clusterIP): ${nameList.join(', ')} — додай ru/kustomization.yaml (abie.mdc)`
       )
       return
     }

package/skills/lint/SKILL.md

@@ -25,7 +25,7 @@ bun run lint
 
 2. **Якщо exit code не 0** — проаналізуй вивід (останній упавший крок у ланцюжку **`lint`** часто видно з stderr / логів):
    - Де скрипт уже робить **auto-fix** (**`--fix`**, **`markdownlint-cli2 --fix`**, **`oxfmt`** тощо) — перезапусти **`bun run lint`** після змін файлів.
-   - Де auto-fix **немає** (наприклад, **jscpd**, **cspell**, **zizmor**, перевірки без прапорця fix) — виправ код, конфіги або винятки **узгоджено з** `.cursor/rules/` (не розширюй ignore лише щоб приховати проблему без причини).
+   - Де auto-fix **немає** (наприклад, **jscpd**, **cspell**, **zizmor**, перевірки без прапорця fix) — **рефактори код проєкту**, щоб усунути порушення: перейменуй ідентифікатори, перепиши логіку, видали дублікати тощо. **Заборонено** додавати `eslint-disable`, `// @ts-ignore`, розширювати `ignores`, **`.cspellignore`**, **`.jscpdignore`** або інші винятки **лише** щоб приховати порушення без обґрунтованої причини — політика узгоджена з **`.cursor/rules/`** (зокрема **n-js-lint** для **jscpd**, **n-text** для **cspell** тощо).
    - Якщо спрацьовує **`sonarjs/cognitive-complexity`** — див. окремий блок нижче.
 
 3. **Цикл** — повторюй кроки 1–2, доки **`bun run lint`** не завершиться успішно. Після суттєвих правок за потреби ще раз **`bun run lint`**, щоб переконатися, що не зламав наступний крок у скрипті **`lint`**.