declapract-typescript-ehmpathy 0.49.12 → 0.49.14

package/dist/.test/assets/expo-migration/declarations/practices/pnpm/bad-practices/override-undercuts-dependency/.declapract.readme.md

@@ -0,0 +1,5 @@
+a `pnpm.overrides` entry below the version the repo declares for that same package.
+
+pnpm installs the override, never the dependency it shadows. so a floor check that reads
+`dependencies` reads green while the tree runs the lower copy. the fix raises each such override to
+the declared version; every other override is kept.

package/dist/.test/assets/expo-migration/declarations/practices/pnpm/bad-practices/override-undercuts-dependency/package.json.declapract.ts

@@ -0,0 +1,67 @@
+import type { FileCheckFunction, FileFixFunction } from 'declapract';
+
+import { asPackageJSON } from '../../../../utils/asPackageJSON';
+import { isVersionBelowFloor } from '../../../../utils/isVersionBelowFloor';
+
+const PRACTICE = 'pnpm/override-undercuts-dependency';
+
+/**
+ * .what = each `pnpm.overrides` key whose version sits below the version the repo itself declares
+ *         for that package, paired with the declared version it must rise to.
+ * .why  = pnpm installs the override, never the dependency it shadows. a floor check reads the
+ *         dependency, so an override below it reads green while the tree runs the lower copy
+ *         (#624). an override is usually added to settle a contention by someone with no floor in
+ *         mind, so the louder its reason, the quieter the loss.
+ * .note = only a plain package-name key is judged; a selector key (`foo@1>bar`) or a non-numeric
+ *         value (`$foo`, `-`, `link:…`) cannot be compared, so it is never flagged.
+ */
+const getUndercutOverrides = (input: {
+  packageJSON: Record<string, any>;
+}): Array<{ name: string; declared: string }> => {
+  const overrides: Record<string, unknown> =
+    input.packageJSON.pnpm?.overrides ?? {};
+  const declaredOf = (name: string): string | undefined =>
+    input.packageJSON.dependencies?.[name] ??
+    input.packageJSON.devDependencies?.[name];
+  return Object.entries(overrides).flatMap(([name, override]) => {
+    const declared = declaredOf(name);
+    if (typeof override !== 'string' || !declared) return [];
+    return isVersionBelowFloor({ version: override, floor: declared })
+      ? [{ name, declared }]
+      : [];
+  });
+};
+
+export const check: FileCheckFunction = (contents) => {
+  if (!contents)
+    throw new Error('does not match bad practice: package.json absent');
+  const packageJSON = asPackageJSON({ contents }, { practice: PRACTICE });
+  if (getUndercutOverrides({ packageJSON }).length > 0) return; // bad practice detected
+  throw new Error(
+    'does not match bad practice: no pnpm.overrides entry sits below its declared dependency',
+  );
+};
+
+/**
+ * .what = raise each undercut override to the version the repo declares for that package.
+ * .why  = so the version a floor check reads is the version pnpm installs. every other override
+ *         and key is kept as-is.
+ */
+export const fix: FileFixFunction = (contents) => {
+  if (!contents) return { contents };
+  const packageJSON = asPackageJSON({ contents }, { practice: PRACTICE });
+  const raised = Object.fromEntries(
+    getUndercutOverrides({ packageJSON }).map(({ name, declared }) => [
+      name,
+      declared,
+    ]),
+  );
+  const updated = {
+    ...packageJSON,
+    pnpm: {
+      ...packageJSON.pnpm,
+      overrides: { ...packageJSON.pnpm?.overrides, ...raised },
+    },
+  };
+  return { contents: JSON.stringify(updated, null, 2) };
+};

package/dist/.test/assets/expo-migration/declarations/practices/pnpm/bad-practices/stale-patched-dependency/.declapract.readme.md

@@ -0,0 +1,22 @@
+# bad-practice: stale-patched-dependency
+
+## .what
+
+a `pnpm.patchedDependencies` entry (and its `patches/<name>@<version>.patch` file) whose pinned
+version the lockfile no longer resolves.
+
+## .why it is bad
+
+a patch pins one exact version. the next upgrade of that dependency orphans it: the patch
+applies to no install. without `pnpm.allowUnusedPatches` pnpm refuses the whole install
+(`ERR_PNPM_UNUSED_PATCH`); with it, pnpm warns on every install. either way the patch is dead
+weight that a human must notice and retire by hand.
+
+## .the fix
+
+drop each stale key from `pnpm.patchedDependencies` (and an empty map left behind), and delete
+the patch file. the verdict keys off the lockfile-resolved version, so:
+
+- no `pnpm-lock.yaml` → no verdict; no patch is judged stale
+- a range key (`name@^1.0.0`) or a bare name key (`name`) → never judged stale
+- the fix is a fixed point: once the stale keys are gone, the check no longer detects the pattern

package/dist/.test/assets/expo-migration/declarations/practices/pnpm/bad-practices/stale-patched-dependency/package.json.declapract.ts

@@ -0,0 +1,74 @@
+import type { FileCheckFunction, FileFixFunction } from 'declapract';
+
+import { asPackageJSON } from '../../../../utils/asPackageJSON';
+import { getAllStalePatchKeys } from '../../../../utils/getAllStalePatchKeys';
+import { getFileTextOrNull } from '../../../../utils/getFileTextOrNull';
+
+const PRACTICE = 'pnpm/stale-patched-dependency';
+
+/**
+ * .what = the repo's pnpm lockfile, or null when it has none
+ */
+const getLockfile = (input: { root: string }): string | null =>
+  getFileTextOrNull({ root: input.root, relativePath: 'pnpm-lock.yaml' });
+
+/**
+ * .what = detects a package.json whose `pnpm.patchedDependencies` pins a version the lockfile no
+ *         longer resolves (#680)
+ * .why  = such a patch applies to no install: pnpm warns on it (under `allowUnusedPatches`) or
+ *         refuses the install (without it). the patch is dead weight a human must retire by hand
+ * .note = bad-practice semantics: a RETURN means detected, a THROW means not detected
+ */
+export const check: FileCheckFunction = (contents, context) => {
+  if (!contents) throw new Error('does not match bad practice');
+  const packageJSON = asPackageJSON({ contents }, { practice: PRACTICE });
+  const keys = getAllStalePatchKeys({
+    patchedDependencies: packageJSON.pnpm?.patchedDependencies ?? {},
+    lockfile: getLockfile({ root: context.getProjectRootDirectory() }),
+  });
+  if (keys.length) return; // bad practice detected
+  throw new Error('does not match bad practice');
+};
+
+/**
+ * .what = drops each stale patch key, and an empty map or pnpm block left behind
+ * .why  = the patch file itself is deleted by the `patches/` declaration beside this one
+ * .note = a fixed point: once the stale keys are gone, the check no longer detects the pattern
+ */
+export const fix: FileFixFunction = (contents, context) => {
+  if (!contents) return {};
+  const packageJSON = asPackageJSON({ contents }, { practice: PRACTICE });
+  const patchedDependencies: Record<string, string> =
+    packageJSON.pnpm?.patchedDependencies ?? {};
+  const keysStale = getAllStalePatchKeys({
+    patchedDependencies,
+    lockfile: getLockfile({ root: context.getProjectRootDirectory() }),
+  });
+  if (!keysStale.length) return { contents };
+
+  // the map without the stale keys
+  const patchedKept = Object.fromEntries(
+    Object.entries(patchedDependencies).filter(
+      ([key]) => !keysStale.includes(key),
+    ),
+  );
+
+  // an empty map is dropped, and so is an empty pnpm block
+  const pnpmKept = {
+    ...packageJSON.pnpm,
+    patchedDependencies: Object.keys(patchedKept).length
+      ? patchedKept
+      : undefined,
+  };
+  const pnpmHasKeys = Object.values(pnpmKept).some(
+    (value) => value !== undefined,
+  );
+
+  return {
+    contents: `${JSON.stringify(
+      { ...packageJSON, pnpm: pnpmHasKeys ? pnpmKept : undefined },
+      null,
+      2,
+    )}\n`,
+  };
+};

package/dist/.test/assets/expo-migration/declarations/practices/pnpm/bad-practices/stale-patched-dependency/patches/<star>.patch.declapract.ts

@@ -0,0 +1,41 @@
+import type { FileCheckFunction, FileFixFunction } from 'declapract';
+
+import path from 'node:path';
+import { getAllStalePatchKeys } from '../../../../../utils/getAllStalePatchKeys';
+import { getFileTextOrNull } from '../../../../../utils/getFileTextOrNull';
+
+/**
+ * .what = the `pnpm.patchedDependencies` key a patch file stands for, from its path
+ * .why  = pnpm names a patch `<name>@<version>.patch`, with a scope slash encoded as `__`
+ */
+const asPatchKey = (input: { relativeFilePath: string }): string =>
+  path
+    .basename(input.relativeFilePath)
+    .replace(/\.patch$/, '')
+    .replace(/^(@[^_]+)__/, '$1/');
+
+/**
+ * .what = detects a patch file whose pinned version the lockfile no longer resolves (#680)
+ * .why  = the patch applies to no install; its key is retired by the package.json declaration
+ *         beside this one, and the file goes with it
+ * .note = bad-practice semantics: a RETURN means detected, a THROW means not detected
+ */
+export const check: FileCheckFunction = (contents, context) => {
+  if (contents === null) throw new Error('does not match bad practice');
+  const lockfile = getFileTextOrNull({
+    root: context.getProjectRootDirectory(),
+    relativePath: 'pnpm-lock.yaml',
+  });
+  const key = asPatchKey({ relativeFilePath: context.relativeFilePath });
+  const keysStale = getAllStalePatchKeys({
+    patchedDependencies: { [key]: context.relativeFilePath },
+    lockfile,
+  });
+  if (keysStale.length) return; // bad practice detected
+  throw new Error('does not match bad practice');
+};
+
+/**
+ * .what = deletes the stale patch file
+ */
+export const fix: FileFixFunction = () => ({ contents: null });

package/dist/.test/assets/expo-migration/declarations/practices/pnpm/best-practice/package.json

@@ -1,3 +1,7 @@
 {
-  "packageManager": "pnpm@10.34.5"
+  "packageManager": "pnpm@10.34.5",
+  "pnpm": {
+    "allowUnusedPatches": true,
+    "ignorePatchFailures": true
+  }
 }

package/dist/.test/assets/expo-migration/declarations/practices/pnpm/best-practice/package.json.declapract.ts

@@ -13,5 +13,12 @@ import { FileCheckType } from 'declapract';
  * .note = a consumer ahead of the pin (e.g. on 10.32.1) is pinned BACK to the blessed version on
  *         `declapract fix` — that convergence-to-one-version is the intent, not a regression. when
  *         the org bumps the blessed version, this pin bumps and every consumer follows.
+ * .what = also declares `pnpm.allowUnusedPatches` + `pnpm.ignorePatchFailures` (#680).
+ * .why  = a patch pins one exact version; the next upgrade orphans it, and pnpm then refuses the
+ *         whole install (`ERR_PNPM_UNUSED_PATCH`, `ERR_PNPM_PATCH_FAILED`). both keys turn that
+ *         refusal into a warn, so patch-then-upgrade never halts an install. a repo with no patch
+ *         gets them too, so its first patch is already safe to upgrade past. the CONTAINS fix
+ *         deep-merges, so a repo's other `pnpm.*` keys (overrides, patchedDependencies, the build
+ *         allowlist) are kept.
  */
 export const check = FileCheckType.CONTAINS;

package/dist/.test/assets/expo-migration/declarations/practices/pnpm/stale-patched-dependency.declapract.integration.test.ts

@@ -0,0 +1,82 @@
+import fs from 'fs/promises';
+import { existsSync } from 'node:fs';
+import path from 'node:path';
+
+import { executeApply } from 'declapract';
+import { given, then, useBeforeAll, when } from 'test-fns';
+
+import { genTempRepoWithDeclarationsOutside } from '../../utils/genTempRepoWithDeclarationsOutside';
+
+// executeApply evaluates the whole practice, so widen the jest budget
+jest.setTimeout(180_000);
+
+/**
+ * .what = pipeline test for the pnpm patch settings + the stale-patched-dependency bad practice (#680)
+ * .why  = only executeApply proves declapract merges `allowUnusedPatches` + `ignorePatchFailures`
+ *         into a repo's extant `pnpm` field (its other keys kept), reads the repo's lockfile to
+ *         judge a patch stale, drops the stale key, and deletes the stale patch file — and that a
+ *         second apply is a no-op
+ */
+describe('pnpm — a stale patch never blocks an install', () => {
+  given('[case1] a repo with one patch the lockfile moved past and one it still resolves', () => {
+    const tempDir = genTempRepoWithDeclarationsOutside({
+      slug: 'pnpm-stale-patch',
+      clone: './src/practices/pnpm/.test/assets/repo-stale-patch',
+      declarations: './src/.test/assets/pnpm/declarations',
+      symlink: [{ at: 'node_modules', to: 'node_modules' }],
+    });
+    const stalePatch = 'patches/rhachet-roles-ehmpathy@1.39.1.patch';
+    const keptPatch = 'patches/type-fns@1.21.3.patch';
+
+    const state = useBeforeAll(async () => {
+      const config = path.join(tempDir, 'declapract.use.yml');
+      const read = () =>
+        fs.readFile(path.join(tempDir, 'package.json'), 'utf-8');
+      const before = await read();
+      await executeApply({ config });
+      const after1 = await read();
+      const exists1 = {
+        stale: existsSync(path.join(tempDir, stalePatch)),
+        kept: existsSync(path.join(tempDir, keptPatch)),
+      };
+      await executeApply({ config });
+      const after2 = await read();
+      return { before, after1, after2, exists1 };
+    }, 170_000);
+
+    when('[t0] the pnpm practice is applied', () => {
+      then('pnpm declares allowUnusedPatches + ignorePatchFailures', () => {
+        const pnpm = JSON.parse(state.after1).pnpm;
+        expect(pnpm.allowUnusedPatches).toEqual(true);
+        expect(pnpm.ignorePatchFailures).toEqual(true);
+      });
+
+      then("the repo's other pnpm keys are kept", () => {
+        const pnpm = JSON.parse(state.after1).pnpm;
+        expect(pnpm.overrides).toEqual({ 'react-refresh': '0.14.2' });
+      });
+
+      then('the stale patch key is retired, the resolved one kept', () => {
+        const patched = JSON.parse(state.after1).pnpm.patchedDependencies;
+        expect(patched).toEqual({
+          'type-fns@1.21.3': 'patches/type-fns@1.21.3.patch',
+        });
+      });
+
+      then('the stale patch file is deleted, the resolved one kept', () => {
+        expect(state.exists1).toEqual({ stale: false, kept: true });
+      });
+
+      then('the before + after contents match snapshot', () => {
+        expect(state.before).toMatchSnapshot('package.json — before');
+        expect(state.after1).toMatchSnapshot('package.json — after');
+      });
+    });
+
+    when('[t1] the practice is applied a second time', () => {
+      then('the second apply is a no-op', () => {
+        expect(state.after2).toEqual(state.after1);
+      });
+    });
+  });
+});

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/.declapract.integration.test.ts

@@ -24,7 +24,7 @@ describe('rhachet/prod-deps relocation', () => {
     const tempDir = genTempDir({
       slug: 'declapract-rhachet-prod-deps',
       clone: './src/practices/rhachet/bad-practices/prod-deps/.test/assets/demo-repo-with-rhachet-prod-deps',
-      symlink: [{ at: 'src', to: 'src' }],
+      symlink: [{ at: 'declarations', to: 'src' }],
     });
 
     const pkgPath = 'package.json';
@@ -56,7 +56,7 @@ describe('rhachet/prod-deps relocation', () => {
           expect(pkg.dependencies?.['rhachet']).toBeUndefined();
           expect(pkg.dependencies?.['rhachet-brains-anthropic']).toBeUndefined();
           // and now sit in dev deps (the -brains- match is the D26 proof)
-          expect(pkg.devDependencies['rhachet']).toEqual('1.47.5');
+          expect(pkg.devDependencies['rhachet']).toEqual('1.49.2');
           expect(pkg.devDependencies['rhachet-brains-anthropic']).toEqual(
             '0.4.3',
           );
@@ -88,7 +88,7 @@ describe('rhachet/prod-deps relocation', () => {
       });
 
       then('the relocated + retained deps survive the re-apply', () => {
-        expect(pkg.devDependencies['rhachet']).toEqual('1.47.5');
+        expect(pkg.devDependencies['rhachet']).toEqual('1.49.2');
         expect(pkg.devDependencies['rhachet-brains-anthropic']).toEqual('0.4.3');
         expect(pkg.dependencies.uuid).toEqual('9.0.0');
         expect(pkg.devDependencies.jest).toEqual('29.3.1');

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/.test/assets/demo-repo-with-consumer-boot/.agent/repo=.this/role=any/boot.yml

@@ -5,3 +5,5 @@ briefs:
     - briefs/my-own-brief.md
   ref:
     - briefs/my-own-deep-variant.md
+    # the old practice shape put this declared glob under ref; the fix moves it to not (#689)
+    - briefs/domain.terms/term=*._.choice.reason.md

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/.test/assets/demo-repo-with-consumer-boot/declapract.declare.yml

@@ -1,4 +1,4 @@
 declare:
-  examples: src/examples
-  practices: src/practices
-  use-cases: src/useCases.yml
+  examples: declarations/examples
+  practices: declarations/practices
+  use-cases: declarations/useCases.yml

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/.test/assets/repo-keyrack-extant/.agent/keyrack.yml

@@ -0,0 +1,9 @@
+org: ehmpathy
+extends:
+  - .agent/repo=ehmpathy/role=mechanic/keyrack.yml
+env.prod: null
+env.prep:
+  - OPENROUTER_API_KEY
+env.test:
+  # the local aws login, from before is-optional-if-has
+  - AWS_PROFILE

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/.test/assets/repo-keyrack-extant/declapract.use.node.yml

@@ -0,0 +1,4 @@
+# .note = declarations dir is symlinked by the test to src/.test/assets/rhachet/declarations
+# .why  = the same repo with no aws practice: the keyrack.yml is left alone
+declarations: 'declarations/declapract.declare.yml'
+useCase: rhachet-node

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/.test/assets/repo-keyrack-extant/declapract.use.yml

@@ -0,0 +1,4 @@
+# .note = declarations dir is symlinked by the test to src/.test/assets/rhachet/declarations
+# .why  = an aws repo whose keyrack.yml holds a bare AWS_PROFILE (#491)
+declarations: 'declarations/declapract.declare.yml'
+useCase: rhachet-aws

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/.test/assets/repo-keyrack-extant/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "test-project",
+  "version": "1.0.0"
+}

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/.test/assets/repo-self-linked/declapract.use.yml

@@ -0,0 +1,4 @@
+# .note = declarations dir is symlinked by the test to src/.test/assets/rhachet/declarations
+# .why  = a rhachet roles package that self-links its roles via `link:.` (#520)
+declarations: 'declarations/declapract.declare.yml'
+useCase: rhachet-node

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/.test/assets/repo-self-linked/package.json

@@ -0,0 +1,22 @@
+{
+  "name": "rhachet-roles-demo",
+  "version": "1.0.0",
+  "scripts": {
+    "build": "tsc -p ./tsconfig.build.json",
+    "prepare:rhachet": "rhachet init --keys --hooks --roles mechanic behaver driver reviewer librarian ergonomist architect dispatcher learner achiever telepath operator observer deployer playwright enroller",
+    "upgrade:rhachet": "rhachet upgrade"
+  },
+  "devDependencies": {
+    "rhachet": "1.49.2",
+    "rhachet-brains-anthropic": "0.4.3",
+    "rhachet-brains-fireworksai": "0.1.6",
+    "rhachet-brains-xai": "0.3.3",
+    "rhachet-roles-bhrain": "0.34.1",
+    "rhachet-roles-bhrowser": "0.1.0",
+    "rhachet-roles-bhuild": "0.21.36",
+    "rhachet-roles-demo": "link:.",
+    "rhachet-roles-ehmpathy": "1.38.12",
+    "rhachet-roles-ghlitch": "0.3.0",
+    "rhachet-roles-rhachet": "0.1.7"
+  }
+}

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/__snapshots__/.declapract.integration.test.ts.snap

@@ -25,14 +25,15 @@ exports[`rhachet/prod-deps relocation given: [case1] a repo with rhachet + a rha
   },
   "devDependencies": {
     "jest": "29.3.1",
-    "rhachet": "1.47.5",
+    "rhachet": "1.49.2",
     "rhachet-brains-anthropic": "0.4.3",
-    "rhachet-brains-fireworksai": "@declapract{check.minVersion('0.1.6')}",
+    "rhachet-brains-fireworksai": "@declapract{check.minVersion('0.2.3')}",
+    "rhachet-brains-openrouter": "@declapract{check.minVersion('0.1.3')}",
     "rhachet-brains-xai": "@declapract{check.minVersion('0.3.3')}",
-    "rhachet-roles-bhrain": "@declapract{check.minVersion('0.34.1')}",
+    "rhachet-roles-bhrain": "@declapract{check.minVersion('0.40.0')}",
     "rhachet-roles-bhrowser": "@declapract{check.minVersion('0.1.0')}",
-    "rhachet-roles-bhuild": "@declapract{check.minVersion('0.21.36')}",
-    "rhachet-roles-ehmpathy": "@declapract{check.minVersion('1.38.12')}",
+    "rhachet-roles-bhuild": "@declapract{check.minVersion('0.23.2')}",
+    "rhachet-roles-ehmpathy": "@declapract{check.minVersion('1.39.5')}",
     "rhachet-roles-ghlitch": "@declapract{check.minVersion('0.3.0')}",
     "rhachet-roles-rhachet": "@declapract{check.minVersion('0.1.7')}"
   },

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/__snapshots__/boot-yml-findsert.declapract.integration.test.ts.snap

@@ -8,6 +8,8 @@ briefs:
     - briefs/my-own-brief.md
   ref:
     - briefs/my-own-deep-variant.md
+    # the old practice shape put this declared glob under ref; the fix moves it to not (#689)
+    - briefs/domain.terms/term=*._.choice.reason.md
 "
 `;
 
@@ -19,8 +21,12 @@ briefs:
     - briefs/my-own-brief.md
   ref:
     - briefs/my-own-deep-variant.md
+  not:
     - briefs/domain.terms/term=*._.choice.reason.md
     - briefs/domain.terms/term=*._.choice.example=*.md
+# budget = the token cap on the say payload; a boot over it halts.
+# to fix a halt: demote a say glob to ref, use a .min brief, or raise the cap.
+# plain \`5000\`, never \`5_000\`: yaml 1.2 reads an underscore integer as a string.
 budget:
   tokens: 5000
 "

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/__snapshots__/keyrack-yml-findsert.declapract.integration.test.ts.snap

@@ -0,0 +1,28 @@
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
+
+exports[`rhachet — keyrack.yml AWS_PROFILE findsert given: [case1] an aws repo whose keyrack.yml holds a bare AWS_PROFILE when: [t0] the rhachet practice is applied then: the before + after contents match snapshot: keyrack.yml — after 1`] = `
+"org: ehmpathy
+extends:
+  - .agent/repo=ehmpathy/role=mechanic/keyrack.yml
+env.prod: null
+env.prep:
+  - OPENROUTER_API_KEY
+env.test:
+  # the local aws login, from before is-optional-if-has
+  - key: AWS_PROFILE
+    is-optional-if-has: AWS_ACCESS_KEY_ID
+"
+`;
+
+exports[`rhachet — keyrack.yml AWS_PROFILE findsert given: [case1] an aws repo whose keyrack.yml holds a bare AWS_PROFILE when: [t0] the rhachet practice is applied then: the before + after contents match snapshot: keyrack.yml — before 1`] = `
+"org: ehmpathy
+extends:
+  - .agent/repo=ehmpathy/role=mechanic/keyrack.yml
+env.prod: null
+env.prep:
+  - OPENROUTER_API_KEY
+env.test:
+  # the local aws login, from before is-optional-if-has
+  - AWS_PROFILE
+"
+`;

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/__snapshots__/self-link-build-order.declapract.integration.test.ts.snap

@@ -0,0 +1,81 @@
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
+
+exports[`rhachet — prepare:rhachet builds first in a self-linked repo given: [case1] a rhachet roles package with a link:. self-dep and no build before init when: [t0] the rhachet practice is applied then: the before + after contents match snapshot: package.json — after 1`] = `
+"{
+  "name": "rhachet-roles-demo",
+  "version": "1.0.0",
+  "scripts": {
+    "build": "tsc -p ./tsconfig.build.json",
+    "prepare:rhachet": "npm run build && rhachet init --keys --hooks --roles mechanic behaver driver reviewer librarian ergonomist architect dispatcher learner achiever telepath operator observer deployer playwright enroller",
+    "upgrade:rhachet": "rhachet upgrade"
+  },
+  "devDependencies": {
+    "rhachet": "1.49.2",
+    "rhachet-brains-anthropic": "0.4.3",
+    "rhachet-brains-fireworksai": "0.2.3",
+    "rhachet-brains-xai": "0.3.3",
+    "rhachet-roles-bhrain": "0.40.0",
+    "rhachet-roles-bhrowser": "0.1.0",
+    "rhachet-roles-bhuild": "0.23.2",
+    "rhachet-roles-demo": "link:.",
+    "rhachet-roles-ehmpathy": "1.39.5",
+    "rhachet-roles-ghlitch": "0.3.0",
+    "rhachet-roles-rhachet": "0.1.7",
+    "rhachet-brains-openrouter": "@declapract{check.minVersion('0.1.3')}"
+  }
+}
+"
+`;
+
+exports[`rhachet — prepare:rhachet builds first in a self-linked repo given: [case1] a rhachet roles package with a link:. self-dep and no build before init when: [t0] the rhachet practice is applied then: the before + after contents match snapshot: package.json — before 1`] = `
+"{
+  "name": "rhachet-roles-demo",
+  "version": "1.0.0",
+  "scripts": {
+    "build": "tsc -p ./tsconfig.build.json",
+    "prepare:rhachet": "rhachet init --keys --hooks --roles mechanic behaver driver reviewer librarian ergonomist architect dispatcher learner achiever telepath operator observer deployer playwright enroller",
+    "upgrade:rhachet": "rhachet upgrade"
+  },
+  "devDependencies": {
+    "rhachet": "1.49.2",
+    "rhachet-brains-anthropic": "0.4.3",
+    "rhachet-brains-fireworksai": "0.1.6",
+    "rhachet-brains-xai": "0.3.3",
+    "rhachet-roles-bhrain": "0.34.1",
+    "rhachet-roles-bhrowser": "0.1.0",
+    "rhachet-roles-bhuild": "0.21.36",
+    "rhachet-roles-demo": "link:.",
+    "rhachet-roles-ehmpathy": "1.38.12",
+    "rhachet-roles-ghlitch": "0.3.0",
+    "rhachet-roles-rhachet": "0.1.7"
+  }
+}
+"
+`;
+
+exports[`rhachet — prepare:rhachet builds first in a self-linked repo given: [case1] a rhachet roles package with a link:. self-dep and no build before init when: [t1] the practice is applied a second time then: every raw marker resolves to a version, and the build step is kept: package.json — after the second apply 1`] = `
+"{
+  "name": "rhachet-roles-demo",
+  "version": "1.0.0",
+  "scripts": {
+    "build": "tsc -p ./tsconfig.build.json",
+    "prepare:rhachet": "npm run build && rhachet init --keys --hooks --roles mechanic behaver driver reviewer librarian ergonomist architect dispatcher learner achiever telepath operator observer deployer playwright enroller",
+    "upgrade:rhachet": "rhachet upgrade"
+  },
+  "devDependencies": {
+    "rhachet": "1.49.2",
+    "rhachet-brains-anthropic": "0.4.3",
+    "rhachet-brains-fireworksai": "0.2.3",
+    "rhachet-brains-xai": "0.3.3",
+    "rhachet-roles-bhrain": "0.40.0",
+    "rhachet-roles-bhrowser": "0.1.0",
+    "rhachet-roles-bhuild": "0.23.2",
+    "rhachet-roles-demo": "link:.",
+    "rhachet-roles-ehmpathy": "1.39.5",
+    "rhachet-roles-ghlitch": "0.3.0",
+    "rhachet-roles-rhachet": "0.1.7",
+    "rhachet-brains-openrouter": "0.1.3"
+  }
+}
+"
+`;

package/dist/.test/assets/expo-migration/declarations/practices/rhachet/bad-practices/prod-deps/.test/assets/demo-repo-with-rhachet-prod-deps/declapract.declare.yml

@@ -1,4 +1,4 @@
 declare:
-  examples: src/examples
-  practices: src/practices
-  use-cases: src/useCases.yml
+  examples: declarations/examples
+  practices: declarations/practices
+  use-cases: declarations/useCases.yml