@nitra/cursor 3.2.1 → 3.2.3

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## [3.2.3] - 2026-06-01
+
+### Fixed
+
+- k8s `workloadAppLabel`: CronJob/workload без `spec` (чи `spec: null`) повертає `null` замість TypeError — `check k8s` більше не падає на неповному маніфесті
+
+## [3.2.2] - 2026-06-01
+
+### Changed
+
+- js-lint: додано e18e/\* deny-правила та канонічні ignorePatterns у .oxlintrc.json
+
+### Fixed
+
+- k8s NetworkPolicy: мітку `app` для Job беремо з `spec.template.metadata.labels.app`, для CronJob — з `spec.jobTemplate.spec.template.metadata.labels.app` (ручний `spec.selector` у Job/CronJob невалідний без `manualSelector: true`)
+
 ## [3.2.1] - 2026-06-01
 
 ### Fixed

package/package.json

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

package/rules/k8s/js/manifests.mjs

@@ -4009,23 +4009,49 @@ function appLabelFromSpecSelector(spec) {
   return typeof app === 'string' && app.trim() !== '' ? app : null
 }
 
+/**
+ * Витягує мітку `app` з `spec.template.metadata.labels.app` (pod-template labels).
+ *
+ * Джерело для Job і CronJob: у Job/CronJob **не можна** задавати ручний `spec.selector`
+ * без `manualSelector: true` — apiserver сам генерує селектор з `controller-uid`, тож
+ * `spec.selector.matchLabels.app` там невалідне джерело. Pod-template labels — валідне
+ * й завжди присутнє поле.
+ * @param {Record<string, unknown>} spec об'єкт `spec` (Job — `spec`, CronJob — `jobTemplate.spec`)
+ * @returns {string | null} непорожнє значення `app` або null
+ */
+function appLabelFromPodTemplate(spec) {
+  const template = getNestedObject(spec, 'template')
+  if (template === null) return null
+  const metadata = getNestedObject(template, 'metadata')
+  if (metadata === null) return null
+  const labels = getNestedObject(metadata, 'labels')
+  if (labels === null) return null
+  const app = labels.app
+  return typeof app === 'string' && app.trim() !== '' ? app : null
+}
+
 /**
  * Витягує мітку `app` для workload, для якого потрібен NetworkPolicy.
- * Deployment / StatefulSet / DaemonSet / Job — `spec.selector.matchLabels.app`;
- * CronJob — `spec.jobTemplate.spec.selector.matchLabels.app`.
+ * Deployment / StatefulSet / DaemonSet — `spec.selector.matchLabels.app`;
+ * Job — `spec.template.metadata.labels.app`;
+ * CronJob — `spec.jobTemplate.spec.template.metadata.labels.app`.
+ *
+ * Job і CronJob читаються з pod-template labels, бо ручний `spec.selector` у них невалідний
+ * без `manualSelector: true` (селектор генерує контролер). Див. `appLabelFromPodTemplate`.
  * @param {Record<string, unknown>} manifest AST workload
  * @returns {string | null} непорожнє значення `app` або null
  */
 export function workloadAppLabel(manifest) {
   const kind = manifest.kind
   if (typeof kind !== 'string') return null
+  const spec = getNestedObject(manifest, 'spec')
+  if (spec === null) return null
   if (kind === 'CronJob') {
-    const jobTemplate = getNestedObject(getNestedObject(manifest, 'spec'), 'jobTemplate')
+    const jobTemplate = getNestedObject(spec, 'jobTemplate')
     const jobSpec = jobTemplate === null ? null : getNestedObject(jobTemplate, 'spec')
-    return jobSpec === null ? null : appLabelFromSpecSelector(jobSpec)
+    return jobSpec === null ? null : appLabelFromPodTemplate(jobSpec)
   }
-  const spec = getNestedObject(manifest, 'spec')
-  if (spec === null) return null
+  if (kind === 'Job') return appLabelFromPodTemplate(spec)
   return appLabelFromSpecSelector(spec)
 }
 
@@ -5631,7 +5657,7 @@ async function validateNetworkPoliciesForK8sWorkloads(root, yamlFilesAbs, fail,
       }
       if (appLabel === null) {
         fail(
-          `${deployRel}: ${workloadKind} '${workloadName}' без мітки app у selector (spec.selector.matchLabels.app або jobTemplate для CronJob) (k8s.mdc)`
+          `${deployRel}: ${workloadKind} '${workloadName}' без мітки app (spec.selector.matchLabels.app; Job — spec.template.metadata.labels.app; CronJob — spec.jobTemplate.spec.template.metadata.labels.app) (k8s.mdc)`
         )
         continue
       }

package/rules/k8s/k8s.mdc

@@ -385,7 +385,7 @@ images:
 
 Для **кожного** `kind: Deployment` у каталозі **`…/k8s/…/base/`** (у будь-якому файлі `.yaml`, наприклад **`deploy.yaml`**, **`deployment.yaml`**) сам Deployment має канонічні **`spec.template.spec.topologySpreadConstraints`**, а **HPA і PDB** живуть у **sibling каталозі** **`…/k8s/…/components/`** (Kustomize Component, фіксована назва каталогу — `components`). У `base/` локальні `hpa.yaml` і `pdb.yaml` **заборонені** (file-existence error). У дереві base-kustomize HPA / PDB також **не дозволені** через `resources` / `components` / `bases`.
 
-**NetworkPolicy** — інша історія: оскільки обмеження мережі мають діяти **і на dev**, NP лежить **у `base/`** (а не в `components/`). Для **кожного** з **`Deployment`**, **`StatefulSet`**, **`DaemonSet`**, **`Job`**, **`CronJob`** під `k8s` обов'язковий **NetworkPolicy**: у **`…/k8s/…/base/`** — у **`base/networkpolicy.yaml`** поруч з workload-маніфестом (multi-doc, якщо workload-ів кілька); у **не-base** — **`networkpolicy.yaml`** поруч із маніфестом workload у тому ж каталозі (overlay-specific override). `metadata.name` NetworkPolicy **= `metadata.name`** workload; `spec.podSelector.matchLabels.app` **= мітка `app`** з `spec.selector.matchLabels` (для **CronJob** — з `spec.jobTemplate.spec.selector.matchLabels`). У `base/kustomization.yaml` `resources:` має бути `networkpolicy.yaml`. Відсутні документи **`check k8s`** створює автоматично і додає `networkpolicy.yaml` у `base/kustomization.yaml` `resources:`.
+**NetworkPolicy** — інша історія: оскільки обмеження мережі мають діяти **і на dev**, NP лежить **у `base/`** (а не в `components/`). Для **кожного** з **`Deployment`**, **`StatefulSet`**, **`DaemonSet`**, **`Job`**, **`CronJob`** під `k8s` обов'язковий **NetworkPolicy**: у **`…/k8s/…/base/`** — у **`base/networkpolicy.yaml`** поруч з workload-маніфестом (multi-doc, якщо workload-ів кілька); у **не-base** — **`networkpolicy.yaml`** поруч із маніфестом workload у тому ж каталозі (overlay-specific override). `metadata.name` NetworkPolicy **= `metadata.name`** workload; `spec.podSelector.matchLabels.app` **= мітка `app`** з `spec.selector.matchLabels` (для **Job** — з `spec.template.metadata.labels.app`, для **CronJob** — з `spec.jobTemplate.spec.template.metadata.labels.app`; у Job/CronJob ручний `spec.selector` невалідний без `manualSelector: true`, тож джерело — pod-template labels). У `base/kustomization.yaml` `resources:` має бути `networkpolicy.yaml`. Відсутні документи **`check k8s`** створює автоматично і додає `networkpolicy.yaml` у `base/kustomization.yaml` `resources:`.
 
 **Канонічна структура `<pkg>/k8s/components/`** (sibling до `base/`) — лише HPA і PDB: