novahiz 0.9.7 → 0.9.8

package/CHANGELOG.md

@@ -5,6 +5,33 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [0.9.8] - 2026-10-15
+
+### Removed
+
+- **The 7 `eas-*` skills from the catalogue** - `test` and `devops` listed them
+  as `defaultSkills` (`eas-app-stores`, `eas-hosting`, `eas-observe`,
+  `eas-simulator`, `eas-update`, `eas-update-insights`, `eas-workflows`) while
+  the expo pack deliberately excludes them: `providers.json` says so in the
+  provider's own purpose. No provider could install a single one, so `doctor`
+  named them and `novahiz deps --install` returned `install: []`. 0.9.5 made
+  that row tell the truth instead of blaming the reader; the references
+  themselves stayed, and the row reports without blocking, which is why nothing
+  had failed since. `referencedSkills` now spans 51 ids and none of them is
+  undeliverable, so `doctor` reports `51 present` with no caveat. The seven
+  inert `overrides.json` entries went with them; the `devops` keywords `eas`,
+  `eas build` and `eas update` stay, since they route a deployment question to
+  a roadmap that still applies.
+
+### Added
+
+- **A test holding the catalogue to that invariant** -
+  `undeliverableReferences(spec, packageRootDir)` is exported from
+  `doctor.ts`, and `tests/doctor-referenced-skills.test.ts` runs it against the
+  shipped catalogue: it must return `[]`, no `defaultSkills` may carry an
+  `eas-*`, and a reintroduced one must be named. Checked by injection: putting
+  `eas-simulator` back into `test.defaultSkills` turns both assertions red.
+
 ## [0.9.7] - 2026-10-15
 
 ### Added

package/catalog/categories.json

@@ -554,8 +554,7 @@
     "defaultSkills": [
       "flutter-add-widget-test",
       "dart-add-unit-test",
-      "expo-skill-eval",
-      "eas-simulator"
+      "expo-skill-eval"
     ],
     "roadmap": {
       "id": "testing",
@@ -1329,14 +1328,7 @@
       "tag a version",
       "changelog"
     ],
-    "defaultSkills": [
-      "eas-workflows",
-      "eas-app-stores",
-      "eas-hosting",
-      "eas-observe",
-      "eas-update",
-      "eas-update-insights"
-    ],
+    "defaultSkills": [],
     "roadmap": {
       "id": "devops",
       "steps": [

package/catalog/overrides.json

@@ -634,83 +634,6 @@
         "docs-writing"
       ]
     },
-    "eas-app-stores": {
-      "power": 4,
-      "stars": null,
-      "tags": [
-        "eas",
-        "expo"
-      ],
-      "categories": [
-        "expo"
-      ]
-    },
-    "eas-hosting": {
-      "power": 4,
-      "stars": null,
-      "tags": [
-        "eas",
-        "expo"
-      ],
-      "categories": [
-        "expo"
-      ]
-    },
-    "eas-observe": {
-      "power": 4,
-      "stars": null,
-      "tags": [
-        "eas",
-        "expo"
-      ],
-      "categories": [
-        "expo"
-      ]
-    },
-    "eas-simulator": {
-      "power": 4,
-      "stars": null,
-      "tags": [
-        "eas",
-        "expo"
-      ],
-      "categories": [
-        "expo"
-      ]
-    },
-    "eas-update": {
-      "power": 4,
-      "stars": null,
-      "tags": [
-        "eas",
-        "expo"
-      ],
-      "categories": [
-        "expo"
-      ]
-    },
-    "eas-update-insights": {
-      "power": 4,
-      "stars": null,
-      "tags": [
-        "eas",
-        "expo"
-      ],
-      "categories": [
-        "expo"
-      ]
-    },
-    "eas-workflows": {
-      "power": 4,
-      "stars": null,
-      "tags": [
-        "eas",
-        "expo"
-      ],
-      "categories": [
-        "expo"
-      ]
-    },
     "expo-animation": {
       "power": 4,
       "stars": null,

package/dist/commands/doctor.js

@@ -443,22 +443,15 @@ function referencedSkills(spec) {
  *  `indexSkills` is null when the index is unreadable — the index check already
  *  reports that, so the row stays quiet instead of double-failing.
  *  Only skills the package ships (packageRoot/skills) can make a fresh install
- *  broken. The remaining references — the dart/eas/expo/flutter packs and
- *  impeccable — are optional delivery downloaded on request and are simply not
+ *  broken. The remaining references, the dart/expo/flutter packs and
+ *  impeccable, are optional delivery downloaded on request and are simply not
  *  present on a machine that never asked for them; the gate already degrades
  *  them to "not enforced". Blocking on them turned every healthy fresh install
  *  red (`missing from index: dart-...` right after novahiz-install --yes). */
 export function referencedSkillsCheck(spec, indexSkills, packageRootDir) {
     const referenced = referencedSkills(spec);
     const absent = indexSkills ? referenced.filter((id) => !indexSkills.has(id)) : [];
-    const shipped = new Set();
-    const shippedDir = join(packageRootDir, "skills");
-    if (existsSync(shippedDir)) {
-        for (const entry of readdirSync(shippedDir, { withFileTypes: true })) {
-            if (entry.isDirectory())
-                shipped.add(entry.name);
-        }
-    }
+    const shipped = shippedSkills(packageRootDir);
     const absentShipped = absent.filter((id) => shipped.has(id));
     const absentPack = absent.filter((id) => !shipped.has(id));
     // Only promise the update command when a pack can actually deliver what is
@@ -493,11 +486,14 @@ export function referencedSkillsCheck(spec, indexSkills, packageRootDir) {
  *  argv: an explicit `--skill a b c` names them, and `--skill *` delivers the
  *  whole repo, whose ids start with the repo's leading segment with any hyphen
  *  suffix dropped (`flutter/skills` -> `flutter-`, `dart-lang/skills` ->
- *  `dart-`). Measured 2026-10-15: the 7 skills doctor reported as "pack skills
- *  not installed" are `eas-*`, referenced as `defaultSkills` by the `test` and
- *  `devops` categories while no provider ships them, so
- *  "run /novahiz-skills-update, or novahiz deps --install" could not have
- *  installed a single one. Advice no command can honour is a defect. */
+ *  `dart-`). Measured 2026-10-15 on the 0.9.5 install: the 7 skills doctor
+ *  reported as "pack skills not installed" were `eas-*`, referenced as
+ *  `defaultSkills` by `test` and `devops` while the expo pack deliberately
+ *  excludes them, so "run /novahiz-skills-update, or novahiz deps --install"
+ *  could not have installed a single one. Advice no command can honour is a
+ *  defect. Those references are gone from the catalog; the guard stays, since
+ *  the same situation reappears whenever a pack is disabled or its `--skill`
+ *  list stops naming an id the catalog still asks for. */
 function packDelivery(spec) {
     const exact = new Set();
     const prefixes = [];
@@ -526,6 +522,27 @@ function packDelivery(spec) {
     }
     return { exact, prefixes };
 }
+/** Skill ids the package itself carries in `packageRootDir/skills`. */
+function shippedSkills(packageRootDir) {
+    const shipped = new Set();
+    const shippedDir = join(packageRootDir, "skills");
+    if (existsSync(shippedDir)) {
+        for (const entry of readdirSync(shippedDir, { withFileTypes: true })) {
+            if (entry.isDirectory())
+                shipped.add(entry.name);
+        }
+    }
+    return shipped;
+}
+/** Referenced skill ids nothing in the system can supply: the package ships
+ *  none of them and no enabled pack can install them. Exported so the catalog
+ *  is held to this invariant in tests, since the row above reports it without
+ *  blocking and therefore never failed on it. */
+export function undeliverableReferences(spec, packageRootDir) {
+    const shipped = shippedSkills(packageRootDir);
+    const { exact, prefixes } = packDelivery(spec);
+    return referencedSkills(spec).filter((id) => !shipped.has(id) && !exact.has(id) && !prefixes.some((prefix) => id.startsWith(prefix)));
+}
 /** Impeccable design-context checks: PRODUCT.md and DESIGN.md resolution
  *  (root -> .agents/context -> docs/). Rows appear only when the impeccable
  *  skill is installed (opt-in) and are never blocking: a missing record is a

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "novahiz",
-  "version": "0.9.7",
+  "version": "0.9.8",
   "description": "Enforce AI agent skills before every file edit: prompt classifier, skill catalog, and pre-edit gate.",
   "type": "module",
   "license": "Apache-2.0",

package/src/commands/doctor.ts

@@ -439,8 +439,8 @@ function referencedSkills(spec: ReturnType<typeof loadSpec>): string[] {
  *  `indexSkills` is null when the index is unreadable — the index check already
  *  reports that, so the row stays quiet instead of double-failing.
  *  Only skills the package ships (packageRoot/skills) can make a fresh install
- *  broken. The remaining references — the dart/eas/expo/flutter packs and
- *  impeccable — are optional delivery downloaded on request and are simply not
+ *  broken. The remaining references, the dart/expo/flutter packs and
+ *  impeccable, are optional delivery downloaded on request and are simply not
  *  present on a machine that never asked for them; the gate already degrades
  *  them to "not enforced". Blocking on them turned every healthy fresh install
  *  red (`missing from index: dart-...` right after novahiz-install --yes). */
@@ -451,13 +451,7 @@ export function referencedSkillsCheck(
 ): DoctorCheck {
   const referenced = referencedSkills(spec);
   const absent = indexSkills ? referenced.filter((id) => !indexSkills.has(id)) : [];
-  const shipped = new Set<string>();
-  const shippedDir = join(packageRootDir, "skills");
-  if (existsSync(shippedDir)) {
-    for (const entry of readdirSync(shippedDir, { withFileTypes: true })) {
-      if (entry.isDirectory()) shipped.add(entry.name);
-    }
-  }
+  const shipped = shippedSkills(packageRootDir);
   const absentShipped = absent.filter((id) => shipped.has(id));
   const absentPack = absent.filter((id) => !shipped.has(id));
   // Only promise the update command when a pack can actually deliver what is
@@ -491,11 +485,14 @@ export function referencedSkillsCheck(
  *  argv: an explicit `--skill a b c` names them, and `--skill *` delivers the
  *  whole repo, whose ids start with the repo's leading segment with any hyphen
  *  suffix dropped (`flutter/skills` -> `flutter-`, `dart-lang/skills` ->
- *  `dart-`). Measured 2026-10-15: the 7 skills doctor reported as "pack skills
- *  not installed" are `eas-*`, referenced as `defaultSkills` by the `test` and
- *  `devops` categories while no provider ships them, so
- *  "run /novahiz-skills-update, or novahiz deps --install" could not have
- *  installed a single one. Advice no command can honour is a defect. */
+ *  `dart-`). Measured 2026-10-15 on the 0.9.5 install: the 7 skills doctor
+ *  reported as "pack skills not installed" were `eas-*`, referenced as
+ *  `defaultSkills` by `test` and `devops` while the expo pack deliberately
+ *  excludes them, so "run /novahiz-skills-update, or novahiz deps --install"
+ *  could not have installed a single one. Advice no command can honour is a
+ *  defect. Those references are gone from the catalog; the guard stays, since
+ *  the same situation reappears whenever a pack is disabled or its `--skill`
+ *  list stops naming an id the catalog still asks for. */
 function packDelivery(spec: ReturnType<typeof loadSpec>): { exact: Set<string>; prefixes: string[] } {
   const exact = new Set<string>();
   const prefixes: string[] = [];
@@ -518,6 +515,33 @@ function packDelivery(spec: ReturnType<typeof loadSpec>): { exact: Set<string>;
   return { exact, prefixes };
 }
 
+/** Skill ids the package itself carries in `packageRootDir/skills`. */
+function shippedSkills(packageRootDir: string): Set<string> {
+  const shipped = new Set<string>();
+  const shippedDir = join(packageRootDir, "skills");
+  if (existsSync(shippedDir)) {
+    for (const entry of readdirSync(shippedDir, { withFileTypes: true })) {
+      if (entry.isDirectory()) shipped.add(entry.name);
+    }
+  }
+  return shipped;
+}
+
+/** Referenced skill ids nothing in the system can supply: the package ships
+ *  none of them and no enabled pack can install them. Exported so the catalog
+ *  is held to this invariant in tests, since the row above reports it without
+ *  blocking and therefore never failed on it. */
+export function undeliverableReferences(
+  spec: ReturnType<typeof loadSpec>,
+  packageRootDir: string
+): string[] {
+  const shipped = shippedSkills(packageRootDir);
+  const { exact, prefixes } = packDelivery(spec);
+  return referencedSkills(spec).filter(
+    (id) => !shipped.has(id) && !exact.has(id) && !prefixes.some((prefix) => id.startsWith(prefix))
+  );
+}
+
 /** Impeccable design-context checks: PRODUCT.md and DESIGN.md resolution
  *  (root -> .agents/context -> docs/). Rows appear only when the impeccable
  *  skill is installed (opt-in) and are never blocking: a missing record is a