@ethlete/agent-rules 0.1.0-next.11 → 0.1.0-next.13

package/content/skills/sdk-local-build/sdk-local-baseline.mjs

@@ -0,0 +1,64 @@
+import { copyFileSync, existsSync, mkdirSync, readFileSync } from 'node:fs';
+import { fileURLToPath } from 'node:url';
+import { join, resolve } from 'node:path';
+
+const baselinePaths = (baselineDir) => ({
+  manifest: join(baselineDir, 'package.json.baseline'),
+  lockfile: join(baselineDir, 'yarn.lock.baseline'),
+});
+
+export const captureBaseline = (manifestPath, lockfilePath, baselineDir) => {
+  const baseline = baselinePaths(baselineDir);
+
+  mkdirSync(baselineDir, { recursive: true });
+
+  if (existsSync(baseline.manifest) || existsSync(baseline.lockfile)) {
+    throw new Error(`Baseline already exists in ${baselineDir}.`);
+  }
+
+  copyFileSync(manifestPath, baseline.manifest);
+  copyFileSync(lockfilePath, baseline.lockfile);
+};
+
+export const restoreBaseline = (manifestPath, lockfilePath, baselineDir) => {
+  const baseline = baselinePaths(baselineDir);
+
+  copyFileSync(baseline.manifest, manifestPath);
+  copyFileSync(baseline.lockfile, lockfilePath);
+};
+
+export const verifyBaseline = (manifestPath, lockfilePath, baselineDir) => {
+  const baseline = baselinePaths(baselineDir);
+
+  for (const [current, recorded] of [
+    [manifestPath, baseline.manifest],
+    [lockfilePath, baseline.lockfile],
+  ]) {
+    if (!readFileSync(current).equals(readFileSync(recorded))) {
+      throw new Error(`${current} does not match its recorded baseline.`);
+    }
+  }
+};
+
+const run = () => {
+  const [mode, manifestPath, lockfilePath, baselineDir] = process.argv.slice(2);
+
+  if (!mode || !manifestPath || !lockfilePath || !baselineDir) {
+    throw new Error('Usage: sdk-local-baseline.mjs <capture|restore|verify> <manifest> <lockfile> <baseline-dir>');
+  }
+
+  const actions = {
+    capture: captureBaseline,
+    restore: restoreBaseline,
+    verify: verifyBaseline,
+  };
+  const action = actions[mode];
+
+  if (!action) throw new Error(`Unknown mode "${mode}".`);
+
+  action(manifestPath, lockfilePath, baselineDir);
+};
+
+if (process.argv[1] && fileURLToPath(import.meta.url) === resolve(process.argv[1])) {
+  run();
+}

package/content/skills/sdk-source/SKILL.md

@@ -21,7 +21,7 @@ first. Reach for the source when the docs genuinely cannot answer the question:
 
 ## 1. Resolve the checkout
 
-The path is per machine, so it lives in the gitignored `ethlete-agents.config.local.json`
+The path is per machine, so it lives in the gitignored `ethlete.config.local.json`
 at the repo root:
 
 ```json
@@ -30,6 +30,9 @@ at the repo root:
 }
 ```
 
+Older checkouts still keep this key in `ethlete-agents.config.local.json`. Read that file as a
+fallback when `ethlete.config.local.json` is absent.
+
 Read that file before searching anywhere. A relative path is resolved from the repo root.
 
 If the file or the key is missing, **do not guess a path** and do not clone the
@@ -38,48 +41,48 @@ holds the full `.d.ts` surface of exactly the version this repo runs - and to
 {%docsBaseUrl%}. Then tell the user a local checkout would help, and offer the snippet
 above (the file is gitignored, so adding it changes nothing for anyone else).
 
-## 2. Check the branch before you read anything
+## 2. Match evidence to the installed package
 
-A checkout sits on whatever branch the developer left it on, so the first command you
-run against it is the one that tells you what you are looking at:
+Start with this routing:
 
-```bash
-git -C <sdkSourcePath> fetch --quiet          # read-only, safe
-git -C <sdkSourcePath> status -sb             # branch, ahead/behind, dirty files
-```
+- Installed `.d.ts` files are authoritative for the public type surface available to
+  this consumer.
+- Source matching the installed build is authoritative for runtime implementation
+  details.
+- Docs explain intended usage and defaults, but can drift from a prerelease.
+- A dirty or ahead `next` checkout is not automatically source for the installed build.
 
-**Unless the user says otherwise, the expected state is `next`, up to date with
-`origin/next`.** That is the branch the SDK develops on, and the one the `-next`
-prereleases are cut from. Anything else and you are reading a different SDK than the
-one you are about to describe.
+Compare the installed package version with the checkout version before attributing
+runtime behavior to the checkout:
 
-When it is not in that state, **say so and ask** - never switch, pull, stash or reset
-it yourself. It is someone's working tree, and the read-only rule below applies:
+```bash
+grep '"@ethlete/' package.json
+grep -m1 '"version"' <sdkSourcePath>/libs/<lib>/package.json
+```
 
-- **On another branch** - name it and ask whether to read it as-is or whether they want
-  `next`. A feature branch may be exactly what you were sent to look at.
-- **Behind `origin/next`** - report how far. The code you would quote may already be
-  fixed upstream, so read the missing commits before calling anything a bug:
-  `git -C <sdkSourcePath> log --oneline HEAD..origin/next`.
-- **Dirty** - it may contain someone's work in progress. Say so rather than quoting it
-  as SDK behaviour.
+Matching version strings are necessary but not sufficient when the checkout is ahead of
+the published commit. Never write consumer code against source that the installed types
+do not expose.
 
-## 3. Check the checkout matches what is installed
+## 3. Check only the paths that answer the question
 
-Even on a clean `next`, the checkout can be months of work ahead of the installed
-package - or behind it:
+Resolve the implementation, public barrel, specs, and docs relevant to the question.
+Then inspect state for those paths:
 
 ```bash
-grep '"@ethlete/' package.json                                 # what this repo runs
-grep -m1 '"version"' <sdkSourcePath>/libs/<lib>/package.json   # what the checkout is at
+git -C <sdkSourcePath> status -sb
+git -C <sdkSourcePath> status --short -- <relevant-paths>
+git -C <sdkSourcePath> diff -- <relevant-paths>
 ```
 
-Rules when they differ:
+Unrelated dirty files, another branch, or unrelated ahead commits do not block a source
+lookup. If relevant paths are dirty or differ in the relevant commit range, label the
+worktree behavior separately from published behavior. Ask only when that distinction
+changes the answer.
 
-- **The installed package wins** for anything about how this repo behaves today. The
-  `.d.ts` in `node_modules` is the truth about the API you are calling.
-- Source that is ahead describes an **unreleased** API. Never write consumer code
-  against it, and never assume it is available - say what release it needs.
+Use existing remote refs first. `git fetch` preserves worktree files but uses the network
+and mutates remote refs. Fetch only when freshness is material, and request any approval
+the environment requires. Never switch, pull, stash, or reset the checkout yourself.
 
 ## 4. Where things live
 
@@ -118,8 +121,8 @@ implementation does.
 
 It is a different repository with its own branch, lint, docs and changeset workflow.
 Never edit it while working on a task in this repo - that covers its git state too, so
-no `checkout`, `pull`, `stash` or `reset` without the user asking for it (`fetch` is
-fine). Never copy its internals into consumer code either - a private helper is not a
+no `checkout`, `pull`, `stash` or `reset` without the user asking for it. Never copy its
+internals into consumer code either - a private helper is not a
 supported API and disappears without a major version (the same goes for anything under
 a `subtle` namespace).
 

package/content/skills/story-styling/SKILL.md

@@ -62,7 +62,7 @@ paints must come from those tokens - the same rule as component CSS
 
 - A tinted panel: put `etAutoSurface` on it (next elevation up) and read the token:
   `style="background: var(--et-surface-background-solid)"`.
-- An accent: scope it with `[etProvideColor]="'brand'"` and read
+- An accent: scope it with `[etProvideColor]="storyAccentTheme"` and read
   `--et-theme-color-primary-solid` / `--et-theme-color-ink-solid`. Theme names are
   whatever **your app** registers - fine to name in a story, never in library code.
 - Better still: don't hand-paint. Compose the component that already does it
@@ -80,4 +80,5 @@ paints must come from those tokens - the same rule as component CSS
 - Text sizes from your theme's scale, not Tailwind's default `text-sm`/`text-xs`.
 - No `dark:` variants.
 - Width/height that matters is px, not the rem container scale.
-- Then verify it actually renders - {%skill:verify-in-storybook%}.
+- Then verify it actually renders with the repository's installed Storybook verification
+  workflow, or inspect the story directly when no such guide was emitted.

package/content/skills/styleguide/SKILL.md

@@ -18,14 +18,19 @@ ship auto-fixers, so most violations are corrected for you:
 {%lintCommand%}      # then re-run to see what needs a manual fix
 ```
 
-Full prose reference, including the rule → lint-rule table and the worked
-file-structure example: {%resource:STYLEGUIDE.md%}.
+Load only the reference needed for the current question:
 
-## Focused guides
+- {%resource:lint-rule-lookup.md%} when a lint fix or diagnostic needs explanation.
+- {%resource:file-structure.md%} when placing routes, views, reusable components,
+  providers, or local utilities.
+- {%resource:assets.md%} when deciding where build, serve, shared, or Storybook assets
+  belong.
+- {%resource:storybook-structure.md%} when placing a story file or its private helpers.
+- {%resource:changesets.md%} when deciding whether an application or library change
+  needs a release note and no repository-specific release skill exists.
 
-- {%skill:rxjs-signals%} - synchronous state vs async, subscriptions, effects.
-- {%skill:angular-patterns%} - components, directives, services, pipes, templates,
-  lifecycle.
+For reactive state or Angular building blocks, load the corresponding focused repository
+guide if one was emitted. This entry point does not duplicate those guides.
 
 The rest of this guide is the cross-cutting judgment that doesn't belong to one
 of those.

package/content/skills/styleguide/assets.md

@@ -0,0 +1,54 @@
+# Assets
+
+Read this file when deciding whether an asset belongs to production, local serve, a
+specific app, Storybook, or shared output.
+
+Keep assets in the workspace assets library and separate them by consumer and purpose:
+
+```plaintext
+assets/
+├── my-app/
+│   ├── build/
+│   ├── serve/
+│   └── storybook/
+├── other-app/
+│   ├── build/
+│   ├── serve/
+│   └── storybook/
+└── shared/
+    ├── build/
+    ├── serve/
+    └── storybook/
+```
+
+- `build/` contains production assets.
+- `serve/` contains development-only placeholders or fixtures.
+- `storybook/` contains assets used only by stories.
+- `shared/` contains assets genuinely used by more than one app.
+- Preserve the intended public URL through the project's `assets` configuration. Do not
+  expose a `build` directory segment in the final URL.
+
+```json
+{
+  "assets": [
+    {
+      "input": "libs/assets/src/my-app/build",
+      "glob": "**/*",
+      "output": "assets"
+    },
+    {
+      "input": "libs/assets/src/my-app/serve",
+      "glob": "**/*",
+      "output": "assets/serve"
+    },
+    {
+      "input": "libs/assets/src/shared/build",
+      "glob": "**/*",
+      "output": "assets/shared"
+    }
+  ]
+}
+```
+
+Add only build assets to a production configuration. Add serve and Storybook assets
+only to the configurations that need them.

package/content/skills/styleguide/changesets.md

@@ -0,0 +1,17 @@
+# Changesets
+
+Read this fallback only when the repository has no more specific release workflow.
+
+- Add a changeset for a consumer-visible change to a published package.
+- Do not add one for formatting, comments, tests, or an internal refactor with no
+  consumer-visible effect.
+- Use `patch` for fixes, `minor` for backward-compatible API additions, and `major` for
+  breaking API or behavior changes.
+- Keep one logical release note focused. Split unrelated changes.
+- Write the note for the package consumer: state what they now get, not how the change
+  was implemented.
+- Keep the note to one sentence, two at most, under 40 words.
+- Do not run an interactive changeset command in a non-interactive agent environment;
+  write the repository's changeset file format directly.
+
+When a repository supplies its own release skill or policy, that local guidance wins.

package/content/skills/styleguide/file-structure.md

@@ -0,0 +1,73 @@
+# File structure
+
+Read this file when placing routes, views, components, providers, or local utilities.
+
+Mirror the route tree in folders and suffix routing components with `-view`:
+
+```ts
+import { Routes } from '@angular/router';
+
+export const SHOP_ROUTES: Routes = [
+  {
+    path: 'items',
+    loadComponent: () =>
+      import('./items-list-view/items-list-view.component').then((module) => module.ItemsListViewComponent),
+  },
+  {
+    path: 'items/:id',
+    loadComponent: () =>
+      import('./item-detail-host-view/item-detail-host-view.component').then(
+        (module) => module.ItemDetailHostViewComponent,
+      ),
+    children: [
+      {
+        path: '',
+        loadComponent: () =>
+          import('./item-detail-host-view/item-detail-view/item-detail-view.component').then(
+            (module) => module.ItemDetailViewComponent,
+          ),
+      },
+      {
+        path: 'reviews',
+        loadComponent: () =>
+          import('./item-detail-host-view/item-reviews-view/item-reviews-view.component').then(
+            (module) => module.ItemReviewsViewComponent,
+          ),
+      },
+    ],
+  },
+];
+```
+
+Use these placement rules:
+
+- Put reusable view-level pieces in `components/`.
+- Put children used only by one parent in that parent's `partials/`.
+- Put generic presentation-only pieces in `uikit/`.
+- Put application-shell pieces in `shell/`.
+- Keep a directive, pipe, provider, form configuration, or utility beside its narrowest
+  consumer. Move it upward only when more consumers appear.
+- Use simple utility names such as `items-list-filter-form.ts`, not `.utils.ts`.
+- Give exportable folders an `index.ts`, but import source files directly. Do not import
+  a parent barrel from its subdirectory.
+- Keep application projects thin; move shared or domain logic into buildable libraries
+  with clear import paths.
+
+```plaintext
+shop/
+├── items-list-view/
+│   ├── components/
+│   │   └── item-card/
+│   │       ├── partials/
+│   │       │   ├── item-image/
+│   │       │   └── item-price/
+│   │       ├── item-card.component.ts
+│   │       └── item-card.component.html
+│   ├── items-list-filter-form.ts
+│   └── items-list-view.component.ts
+└── item-detail-host-view/
+    ├── item-detail-view/
+    ├── item-reviews-view/
+    ├── item-detail.provider.ts
+    └── item-detail-host-view.component.ts
+```

package/content/skills/styleguide/lint-rule-lookup.md

@@ -0,0 +1,50 @@
+# Lint rule lookup
+
+Read this file only to explain a lint result or identify which rule owns a mechanical
+convention. Run lint with `--fix`; do not hand-check this table before linting.
+
+| Convention                                                           | Rule                                                                                                                                                                                                                |
+| -------------------------------------------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| No `any` / `$any()`; narrow `unknown`                                | `@typescript-eslint/no-explicit-any`, `@angular-eslint/template/no-any`                                                                                                                                             |
+| Static strings and booleans use attributes                           | `@angular-eslint/template/prefer-static-string-properties`, `ethlete/prefer-static-boolean-properties`                                                                                                              |
+| `type`, not `interface`                                              | `@typescript-eslint/consistent-type-definitions`                                                                                                                                                                    |
+| No `enum` or `const enum`                                            | `ethlete/no-enum`                                                                                                                                                                                                   |
+| `const` by default; one declaration per statement                    | `no-var`, `prefer-const`, `one-var`                                                                                                                                                                                 |
+| Strict equality                                                      | `eqeqeq`                                                                                                                                                                                                            |
+| Maximum two positional parameters                                    | `max-params`                                                                                                                                                                                                        |
+| No type-only imports                                                 | `ethlete/no-type-only-import`                                                                                                                                                                                       |
+| Descriptive `T`-prefixed generic names                               | `@typescript-eslint/naming-convention`                                                                                                                                                                              |
+| No `async` / `await`                                                 | `ethlete/no-async-await`                                                                                                                                                                                            |
+| Arrow functions standalone; methods in classes                       | `no-restricted-syntax`                                                                                                                                                                                              |
+| Blank line before a multiline guard return                           | `ethlete/guard-return-newline`                                                                                                                                                                                      |
+| No trivially inferable return type                                   | `ethlete/no-trivial-return-type`                                                                                                                                                                                    |
+| Member and local naming conventions                                  | `@typescript-eslint/naming-convention`, `ethlete/no-leading-underscore-class-member`, `ethlete/no-screaming-case-local`                                                                                             |
+| Class constants are `readonly` and upper case                        | `ethlete/class-constant-property`                                                                                                                                                                                   |
+| No `readonly` on reactive members                                    | `ethlete/no-readonly-signal`                                                                                                                                                                                        |
+| Injected and template-visible member accessibility                   | `ethlete/inject-member-accessibility`, `ethlete/template-member-accessibility`                                                                                                                                      |
+| No inject chains, pure aliases, or trivial wrappers                  | `ethlete/no-inject-chain`, `ethlete/no-member-alias`, `ethlete/no-trivial-wrapper-method`                                                                                                                           |
+| No unused private or protected members                               | `ethlete/no-unused-class-member`                                                                                                                                                                                    |
+| Pure module-scope provider calls                                     | `ethlete/no-impure-top-level-provider`                                                                                                                                                                              |
+| No redundant `@internal`                                             | `ethlete/no-redundant-internal`                                                                                                                                                                                     |
+| Observable names end in `$`                                          | `ethlete/require-dollar-suffix`                                                                                                                                                                                     |
+| No subscribe body, subscribe in pipe, or RxJS in signal derivations  | `ethlete/no-subscribe-with-body`, `ethlete/no-subscribe-in-pipe`, `ethlete/no-rxjs-in-effect`                                                                                                                       |
+| Effect teardown uses `onCleanup` or `DestroyRef`                     | `ethlete/no-effect-cleanup-return`                                                                                                                                                                                  |
+| Prefer `linkedSignal` for writable derived state                     | `ethlete/prefer-linked-signal`                                                                                                                                                                                      |
+| Global component CSS encapsulation                                   | `ethlete/require-view-encapsulation-none`                                                                                                                                                                           |
+| Angular-version-correct change detection metadata                    | `ethlete/require-on-push-change-detection`, `ethlete/no-redundant-on-push-change-detection`                                                                                                                         |
+| No redundant standalone flag or empty metadata arrays                | `ethlete/no-standalone-flag`, `ethlete/no-empty-angular-metadata-arrays`                                                                                                                                            |
+| No legacy lifecycle hooks or decorators                              | `no-restricted-syntax`, `ethlete/no-legacy-angular-decorators`                                                                                                                                                      |
+| No injectable services, guards, or resolvers                         | `no-restricted-syntax`                                                                                                                                                                                              |
+| Output naming                                                        | `@angular-eslint/no-output-on-prefix`, `@angular-eslint/no-output-native`, `ethlete/prefer-present-tense-output`                                                                                                    |
+| Inputs do not collide with global HTML attributes                    | `ethlete/no-native-html-input-name`                                                                                                                                                                                 |
+| No pipe logic                                                        | `ethlete/no-pipe-logic`                                                                                                                                                                                             |
+| Class members and decorator metadata stay ordered                    | `ethlete/class-member-order`, `ethlete/angular-decorator-property-order`                                                                                                                                            |
+| Routing views use `-view` / `ViewComponent`                          | `ethlete/enforce-routing-view-naming`                                                                                                                                                                               |
+| No direct browser globals, DOM queries, native observers, or cookies | `no-restricted-globals`, `ethlete/no-direct-dom-manipulation`, `ethlete/no-dom-query`, `ethlete/no-native-observers`, `ethlete/no-document-cookie`                                                                  |
+| No barrel imports                                                    | `no-restricted-syntax`                                                                                                                                                                                              |
+| Prefer the SDK wrappers for platform utilities                       | `ethlete/prefer-clone-equal`, `ethlete/prefer-rxjs-timer`, `ethlete/prefer-match-media`, `ethlete/prefer-viewport-size`, `ethlete/no-angular-seo-services`, `ethlete/no-locale-id`, `ethlete/no-angular-router-api` |
+
+Judgment still belongs outside lint: name things after their behavior, use an object
+parameter after two positional parameters, expose injected members directly instead of
+aliasing them, and use `/** @internal */` only on technically public library API that
+must be stripped from declarations.

package/content/skills/styleguide/storybook-structure.md

@@ -0,0 +1,25 @@
+# Storybook structure
+
+Read this file when placing a `*.stories.ts` file or code used only to render stories.
+
+- Keep the `*.stories.ts` file beside the component it documents.
+- Put private host components, dummy data, and story-only helpers in `storybook/`.
+- Never export Storybook-only code from the component's public barrel.
+- A private `storybook/index.ts` may simplify imports into the adjacent story, but no
+  production source may import it.
+
+```plaintext
+settings-form/
+├── storybook/
+│   ├── settings-form-storybook.component.ts
+│   ├── settings-form-storybook.component.html
+│   ├── settings-form-dummy-data.ts
+│   └── index.ts
+├── settings-form.component.ts
+├── settings-form.component.html
+├── settings-form.component.stories.ts
+└── index.ts
+```
+
+The story file stays at the component root; only its private support code goes into
+`storybook/`.

package/content/skills/theming/SKILL.md

@@ -37,16 +37,16 @@ The generated CSS defines the raw vars on `.et-surface--<name>` / `.et-color--<n
 classes (plus `:root` defaults) and derives the public tokens below on `:root` **and**
 every scope class - so the tokens are always resolvable and components just read them.
 
-**Theme names are yours.** `brand`, `danger`, `neutral`, `dark-elevated` are only
-examples. Never hardcode a theme-name union in a component type, a doc or an example.
+**Theme names are yours.** Never hardcode them as an SDK-defined union or reusable API
+contract. App-specific docs may name their own themes when they label them as app values.
 The portable handle is the theme **`type`**: `injectErrorTheme()` finds whichever theme
 the app registered with `type: 'error'`.
 
 ## Tokens components may consume
 
 Use these **derived** tokens - never the raw `--et-surface-background` /
-`--et-color-primary` channel triplets, and never a hardcoded color (a static fallback
-_after_ the token is fine).
+`--et-color-primary` channel triplets. Never use a hardcoded color as the primary value.
+A static fallback inside `var(--token, <fallback>)` is permitted, but not required.
 
 Surface (each exists as `-solid` = usable color, `-rgb` = `R G B` channels):
 
@@ -75,8 +75,8 @@ the interactive element itself, never on a wrapper.
 
 ## Providing context
 
-- `[etProvideSurface]="'dark-elevated'"` / `[etAutoSurface]` (auto-picks the next
-  elevation) set the surface scope; `[etProvideColor]="'brand'"` sets the color scope.
+- `[etProvideSurface]="surfaceThemeName"` / `[etAutoSurface]` (auto-picks the next
+  elevation) set the surface scope; `[etProvideColor]="colorThemeName"` sets the color scope.
   A component that owns a themed region usually adds `ProvideColorDirective` /
   `ProvideSurfaceDirective` as `hostDirectives`.
 - **Detached overlay panes (menu, dialog, tooltip…) do not inherit DOM context.**
@@ -106,12 +106,12 @@ the interactive element itself, never on a wrapper.
 - `--et-theme-color-primary-*` always resolves to the **nearest color scope**. A
   hardcoded semantic color in CSS can't be replaced by it unless the right theme is
   provided on that element.
-- Keep a static fallback (`var(--et-surface-border-solid, rgb(255 255 255 / 0.1))`) so
-  components degrade in theme-less setups. `injectErrorTheme()` is the exception: it is
-  a hard requirement wherever it is used.
+- A static fallback (`var(--et-surface-border-solid, rgb(255 255 255 / 0.1))`) is
+  permitted for theme-less setups, but themes make it unnecessary. `injectErrorTheme()`
+  is a hard requirement wherever it is used.
 - **Cascade layers, not `:where()`, are what let a Tailwind utility override component
   CSS.** Wrap every component CSS file in `@layer components { … }`; `:where()` only
   flattens a component's own modifiers to single-class weight. See the `styling` rule.
 
-Styling a **story** file rather than a component? The colour rule is the same, but the
-Tailwind side has its own traps - {%skill:story-styling%}.
+Styling a **story** file rather than a component? The colour rule is the same. Check the
+repository's Storybook guidance before assuming a Tailwind utility exists.

package/content/skills/timetrack/SKILL.md

@@ -38,7 +38,7 @@ nobody can rotate.
 | `log --issue <KEY> --minutes <n>` | The user asks to record time that nothing observed                      |
 
 `git-flow start` uses the same channel, so a branch is named from the real issue rather than
-from a key you typed - see the `git-flow` skill.
+from a key you typed. Follow the repository's branch workflow when creating a branch.
 
 ## Writes
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ethlete/agent-rules",
-  "version": "0.1.0-next.11",
+  "version": "0.1.0-next.13",
   "license": "MIT",
   "type": "commonjs",
   "exports": {

package/src/lib/config.d.ts

@@ -43,11 +43,9 @@ export type SyncConfig = {
  * - `disableAutoHandoffSave: true` keeps the context-warning hook's normal tiered messages (including
  *   in auto mode), but at the critical tier in auto mode it falls back to just recommending
  *   a handoff instead of writing the handoff file automatically.
- * - `sdkSourcePath` points at a local `ethlete-sdk` checkout, which the SDK source and local-build
- *   skills read when they need the SDK's own sources instead of the published package.
- * - `apiRepoPaths` maps an app in this repo to the checkout of the API it talks to (`{ "hub":
- *   "../fut-hub-backend" }`), which the api-source skill reads to answer a question about the
- *   server instead of guessing it from the client.
+ *
+ * Where the sibling checkouts live (`sdkSourcePath`, `apiRepoPaths`, `apiRepoBranches`) is not here:
+ * it moved to `@ethlete/cli`'s `ethlete.config.local.json`, because `et` reads the same values.
  *
  * No secret belongs here any more. Jira is reached through the Timetrack app, which holds the token
  * in the machine's keychain — one secret per machine rather than one per checkout.
@@ -55,8 +53,6 @@ export type SyncConfig = {
 export type LocalConfig = {
     disableHooks?: boolean | string[];
     disableAutoHandoffSave?: boolean;
-    sdkSourcePath?: string;
-    apiRepoPaths?: Record<string, string>;
 };
 export type LocalConfigState = {
     exists: false;
@@ -68,7 +64,17 @@ export type LocalConfigState = {
     valid: true;
     config: LocalConfig;
     unknownKeys: string[];
+    movedKeys: string[];
 };
+export declare const LOCAL_CONFIG_KEYS: (keyof LocalConfig)[];
+/**
+ * Repo topology moved to `@ethlete/cli`'s `ethlete.config.local.json`, because `et` needs it too.
+ * Still recognised here so a checkout that has not moved them yet gets told where they went
+ * instead of an "unsupported key" warning.
+ */
+export declare const MOVED_LOCAL_CONFIG_KEYS: string[];
+/** Named only to point a migration at it. `@ethlete/cli` owns the file and validates its values. */
+export declare const TOPOLOGY_CONFIG_FILE_NAME = "ethlete.config.local.json";
 export declare const readLocalConfig: (root: string) => LocalConfigState;
 /**
  * Emit for the agents a repo already uses. `codex` is always included: `AGENTS.md` is the

package/src/lib/config.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.loadConfig = exports.detectTargets = exports.readLocalConfig = exports.LOCAL_CONFIG_FILE_NAME = exports.CONFIG_FILE_NAME = exports.AGENT_TARGETS = void 0;
+exports.loadConfig = exports.detectTargets = exports.readLocalConfig = exports.TOPOLOGY_CONFIG_FILE_NAME = exports.MOVED_LOCAL_CONFIG_KEYS = exports.LOCAL_CONFIG_KEYS = exports.LOCAL_CONFIG_FILE_NAME = exports.CONFIG_FILE_NAME = exports.AGENT_TARGETS = void 0;
 const fs_1 = require("fs");
 const path_1 = require("path");
 const commitlint_1 = require("./commitlint");
@@ -15,12 +15,15 @@ const readRawConfig = (root) => {
         return {};
     return JSON.parse((0, fs_1.readFileSync)(path, 'utf8'));
 };
-const LOCAL_CONFIG_KEYS = [
-    'disableHooks',
-    'disableAutoHandoffSave',
-    'sdkSourcePath',
-    'apiRepoPaths',
-];
+exports.LOCAL_CONFIG_KEYS = ['disableHooks', 'disableAutoHandoffSave'];
+/**
+ * Repo topology moved to `@ethlete/cli`'s `ethlete.config.local.json`, because `et` needs it too.
+ * Still recognised here so a checkout that has not moved them yet gets told where they went
+ * instead of an "unsupported key" warning.
+ */
+exports.MOVED_LOCAL_CONFIG_KEYS = ['sdkSourcePath', 'apiRepoPaths', 'apiRepoBranches'];
+/** Named only to point a migration at it. `@ethlete/cli` owns the file and validates its values. */
+exports.TOPOLOGY_CONFIG_FILE_NAME = 'ethlete.config.local.json';
 const readLocalConfig = (root) => {
     const path = (0, path_1.join)(root, exports.LOCAL_CONFIG_FILE_NAME);
     if (!(0, fs_1.existsSync)(path))
@@ -35,8 +38,10 @@ const readLocalConfig = (root) => {
     if (typeof parsed !== 'object' || parsed === null || Array.isArray(parsed))
         return { exists: true, valid: false };
     const config = parsed;
-    const unknownKeys = Object.keys(config).filter((key) => !LOCAL_CONFIG_KEYS.includes(key));
-    return { exists: true, valid: true, config, unknownKeys };
+    const keys = Object.keys(config);
+    const movedKeys = keys.filter((key) => exports.MOVED_LOCAL_CONFIG_KEYS.includes(key));
+    const unknownKeys = keys.filter((key) => !exports.LOCAL_CONFIG_KEYS.includes(key) && !movedKeys.includes(key));
+    return { exists: true, valid: true, config, unknownKeys, movedKeys };
 };
 exports.readLocalConfig = readLocalConfig;
 /**

package/src/lib/config.js.map

@@ -1 +1 @@
-{"version":3,"file":"config.js","sourceRoot":"","sources":["../../../../../libs/agent-rules/src/lib/config.ts"],"names":[],"mappings":";;;AAAA,2BAA8C;AAC9C,+BAA4B;AAC5B,6CAAuE;AAEvE,yCAAmF;AACnF,iDAAiD;AAEpC,QAAA,aAAa,GAAG,CAAC,QAAQ,EAAE,OAAO,EAAE,QAAQ,EAAE,SAAS,CAAU,CAAC;AAIlE,QAAA,gBAAgB,GAAG,4BAA4B,CAAC;AAEhD,QAAA,sBAAsB,GAAG,kCAAkC,CAAC;AA6CzE,MAAM,aAAa,GAAG,CAAC,IAAY,EAAE,EAAE;IACrC,MAAM,IAAI,GAAG,IAAA,WAAI,EAAC,IAAI,EAAE,wBAAgB,CAAC,CAAC;IAE1C,IAAI,CAAC,IAAA,eAAU,EAAC,IAAI,CAAC;QAAE,OAAO,EAAE,CAAC;IAEjC,OAAO,IAAI,CAAC,KAAK,CAAC,IAAA,iBAAY,EAAC,IAAI,EAAE,MAAM,CAAC,CAAc,CAAC;AAC7D,CAAC,CAAC;AAiCF,MAAM,iBAAiB,GAA0B;IAC/C,cAAc;IACd,wBAAwB;IACxB,eAAe;IACf,cAAc;CACf,CAAC;AAEK,MAAM,eAAe,GAAG,CAAC,IAAY,EAAoB,EAAE;IAChE,MAAM,IAAI,GAAG,IAAA,WAAI,EAAC,IAAI,EAAE,8BAAsB,CAAC,CAAC;IAEhD,IAAI,CAAC,IAAA,eAAU,EAAC,IAAI,CAAC;QAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC;IAEhD,IAAI,MAAe,CAAC;IAEpB,IAAI,CAAC;QACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,IAAA,iBAAY,EAAC,IAAI,EAAE,MAAM,CAAC,CAAC,CAAC;IAClD,CAAC;IAAC,WAAM,CAAC;QACP,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC;IACxC,CAAC;IAED,IAAI,OAAO,MAAM,KAAK,QAAQ,IAAI,MAAM,KAAK,IAAI,IAAI,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC;QAAE,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC;IAElH,MAAM,MAAM,GAAG,MAAqB,CAAC;IACrC,MAAM,WAAW,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,iBAAiB,CAAC,QAAQ,CAAC,GAAwB,CAAC,CAAC,CAAC;IAE/G,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE,CAAC;AAC5D,CAAC,CAAC;AAnBW,QAAA,eAAe,mBAmB1B;AAEF;;;;;GAKG;AACI,MAAM,aAAa,GAAG,CAAC,IAAY,EAAiB,EAAE;IAC3D,MAAM,QAAQ,GAAkB,CAAC,OAAO,CAAC,CAAC;IAE1C,IAAI,IAAA,eAAU,EAAC,IAAA,WAAI,EAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QAAE,QAAQ,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAC/D,IAAI,IAAA,eAAU,EAAC,IAAA,WAAI,EAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QAAE,QAAQ,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAC/D,IAAI,IAAA,eAAU,EAAC,IAAA,WAAI,EAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QAAE,QAAQ,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IAEhE,OAAO,QAAQ,CAAC;AAClB,CAAC,CAAC;AARW,QAAA,aAAa,iBAQxB;AAEF,MAAM,kBAAkB,GAAG,CAAC,OAAsB,EAAE,EAAE;IACpD,MAAM,OAAO,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC,qBAAa,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC;IAE5E,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACvB,MAAM,IAAI,KAAK,CAAC,sBAAsB,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,oBAAoB,qBAAa,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IAC3G,CAAC;AACH,CAAC,CAAC;AAEF;;;;GAIG;AACH,MAAM,WAAW,GAAG,CAAC,OAAsB,EAAqC,EAAE,CAAC,CAAC;IAClF,wBAAwB,EAAE,OAAO,CAAC,YAAY,CAAC,WAAW;IAC1D,uBAAuB,EAAE,OAAO,CAAC,YAAY,CAAC,UAAU;IACxD,YAAY,EAAE,OAAO,CAAC,KAAK;IAC3B,kBAAkB,EAAE,OAAO,CAAC,WAAW;IACvC,gBAAgB,EAAE,OAAO,CAAC,SAAS;CACpC,CAAC,CAAC;AAEI,MAAM,UAAU,GAAG,CAAC,OAAyD,EAAc,EAAE;;IAClG,MAAM,EAAE,IAAI,EAAE,cAAc,EAAE,GAAG,OAAO,CAAC;IACzC,MAAM,GAAG,GAAG,aAAa,CAAC,IAAI,CAAC,CAAC;IAChC,MAAM,UAAU,GAAG,GAAG,CAAC,OAAO,IAAI,GAAG,CAAC,OAAO,KAAK,MAAM,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,SAAS,CAAC;IACnF,MAAM,OAAO,GAAG,MAAA,cAAc,aAAd,cAAc,cAAd,cAAc,GAAI,UAAU,mCAAI,IAAA,qBAAa,EAAC,IAAI,CAAC,CAAC;IACpE,MAAM,OAAO,GAAG,IAAA,+BAAoB,EAAC,GAAG,CAAC,OAAO,CAAC,CAAC;IAElD,kBAAkB,CAAC,OAAO,CAAC,CAAC;IAE5B,OAAO;QACL,IAAI;QACJ,OAAO;QACP,MAAM,EAAE,GAAG,CAAC,OAAO,KAAK,KAAK,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,UAAU,EAAE,MAAM,CAAC;QAC/D,IAAI,8DACC,IAAA,8BAAe,GAAE,GACjB,WAAW,CAAC,OAAO,CAAC,GACpB,IAAA,8BAAiB,EAAC,IAAA,iCAAoB,EAAC,IAAI,CAAC,CAAC,GAC7C,CAAC,MAAA,GAAG,CAAC,IAAI,mCAAI,EAAE,CAAC,CACpB;QACD,OAAO,EAAE,MAAA,GAAG,CAAC,OAAO,mCAAI,EAAE;QAC1B,uBAAuB,EAAE,MAAA,GAAG,CAAC,uBAAuB,mCAAI,KAAK;QAC7D,KAAK,EAAE,MAAA,GAAG,CAAC,KAAK,mCAAI,EAAE;QACtB,QAAQ,EAAE,MAAA,GAAG,CAAC,QAAQ,mCAAI,EAAE;QAC5B,OAAO;QACP,IAAI,EAAE,MAAA,GAAG,CAAC,IAAI,mCAAI,EAAE;KACrB,CAAC;AACJ,CAAC,CAAC;AA1BW,QAAA,UAAU,cA0BrB"}
+{"version":3,"file":"config.js","sourceRoot":"","sources":["../../../../../libs/agent-rules/src/lib/config.ts"],"names":[],"mappings":";;;AAAA,2BAA8C;AAC9C,+BAA4B;AAC5B,6CAAuE;AAEvE,yCAAmF;AACnF,iDAAiD;AAEpC,QAAA,aAAa,GAAG,CAAC,QAAQ,EAAE,OAAO,EAAE,QAAQ,EAAE,SAAS,CAAU,CAAC;AAIlE,QAAA,gBAAgB,GAAG,4BAA4B,CAAC;AAEhD,QAAA,sBAAsB,GAAG,kCAAkC,CAAC;AA6CzE,MAAM,aAAa,GAAG,CAAC,IAAY,EAAE,EAAE;IACrC,MAAM,IAAI,GAAG,IAAA,WAAI,EAAC,IAAI,EAAE,wBAAgB,CAAC,CAAC;IAE1C,IAAI,CAAC,IAAA,eAAU,EAAC,IAAI,CAAC;QAAE,OAAO,EAAE,CAAC;IAEjC,OAAO,IAAI,CAAC,KAAK,CAAC,IAAA,iBAAY,EAAC,IAAI,EAAE,MAAM,CAAC,CAAc,CAAC;AAC7D,CAAC,CAAC;AA6BW,QAAA,iBAAiB,GAA0B,CAAC,cAAc,EAAE,wBAAwB,CAAC,CAAC;AAEnG;;;;GAIG;AACU,QAAA,uBAAuB,GAAG,CAAC,eAAe,EAAE,cAAc,EAAE,iBAAiB,CAAC,CAAC;AAE5F,oGAAoG;AACvF,QAAA,yBAAyB,GAAG,2BAA2B,CAAC;AAE9D,MAAM,eAAe,GAAG,CAAC,IAAY,EAAoB,EAAE;IAChE,MAAM,IAAI,GAAG,IAAA,WAAI,EAAC,IAAI,EAAE,8BAAsB,CAAC,CAAC;IAEhD,IAAI,CAAC,IAAA,eAAU,EAAC,IAAI,CAAC;QAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC;IAEhD,IAAI,MAAe,CAAC;IAEpB,IAAI,CAAC;QACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,IAAA,iBAAY,EAAC,IAAI,EAAE,MAAM,CAAC,CAAC,CAAC;IAClD,CAAC;IAAC,WAAM,CAAC;QACP,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC;IACxC,CAAC;IAED,IAAI,OAAO,MAAM,KAAK,QAAQ,IAAI,MAAM,KAAK,IAAI,IAAI,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC;QAAE,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC;IAElH,MAAM,MAAM,GAAG,MAAqB,CAAC;IACrC,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IACjC,MAAM,SAAS,GAAG,IAAI,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,+BAAuB,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC;IAC9E,MAAM,WAAW,GAAG,IAAI,CAAC,MAAM,CAC7B,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,yBAAiB,CAAC,QAAQ,CAAC,GAAwB,CAAC,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,GAAG,CAAC,CAC3F,CAAC;IAEF,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE,SAAS,EAAE,CAAC;AACvE,CAAC,CAAC;AAvBW,QAAA,eAAe,mBAuB1B;AAEF;;;;;GAKG;AACI,MAAM,aAAa,GAAG,CAAC,IAAY,EAAiB,EAAE;IAC3D,MAAM,QAAQ,GAAkB,CAAC,OAAO,CAAC,CAAC;IAE1C,IAAI,IAAA,eAAU,EAAC,IAAA,WAAI,EAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QAAE,QAAQ,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAC/D,IAAI,IAAA,eAAU,EAAC,IAAA,WAAI,EAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QAAE,QAAQ,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAC/D,IAAI,IAAA,eAAU,EAAC,IAAA,WAAI,EAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QAAE,QAAQ,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IAEhE,OAAO,QAAQ,CAAC;AAClB,CAAC,CAAC;AARW,QAAA,aAAa,iBAQxB;AAEF,MAAM,kBAAkB,GAAG,CAAC,OAAsB,EAAE,EAAE;IACpD,MAAM,OAAO,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC,qBAAa,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC;IAE5E,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACvB,MAAM,IAAI,KAAK,CAAC,sBAAsB,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,oBAAoB,qBAAa,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IAC3G,CAAC;AACH,CAAC,CAAC;AAEF;;;;GAIG;AACH,MAAM,WAAW,GAAG,CAAC,OAAsB,EAAqC,EAAE,CAAC,CAAC;IAClF,wBAAwB,EAAE,OAAO,CAAC,YAAY,CAAC,WAAW;IAC1D,uBAAuB,EAAE,OAAO,CAAC,YAAY,CAAC,UAAU;IACxD,YAAY,EAAE,OAAO,CAAC,KAAK;IAC3B,kBAAkB,EAAE,OAAO,CAAC,WAAW;IACvC,gBAAgB,EAAE,OAAO,CAAC,SAAS;CACpC,CAAC,CAAC;AAEI,MAAM,UAAU,GAAG,CAAC,OAAyD,EAAc,EAAE;;IAClG,MAAM,EAAE,IAAI,EAAE,cAAc,EAAE,GAAG,OAAO,CAAC;IACzC,MAAM,GAAG,GAAG,aAAa,CAAC,IAAI,CAAC,CAAC;IAChC,MAAM,UAAU,GAAG,GAAG,CAAC,OAAO,IAAI,GAAG,CAAC,OAAO,KAAK,MAAM,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,SAAS,CAAC;IACnF,MAAM,OAAO,GAAG,MAAA,cAAc,aAAd,cAAc,cAAd,cAAc,GAAI,UAAU,mCAAI,IAAA,qBAAa,EAAC,IAAI,CAAC,CAAC;IACpE,MAAM,OAAO,GAAG,IAAA,+BAAoB,EAAC,GAAG,CAAC,OAAO,CAAC,CAAC;IAElD,kBAAkB,CAAC,OAAO,CAAC,CAAC;IAE5B,OAAO;QACL,IAAI;QACJ,OAAO;QACP,MAAM,EAAE,GAAG,CAAC,OAAO,KAAK,KAAK,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,UAAU,EAAE,MAAM,CAAC;QAC/D,IAAI,8DACC,IAAA,8BAAe,GAAE,GACjB,WAAW,CAAC,OAAO,CAAC,GACpB,IAAA,8BAAiB,EAAC,IAAA,iCAAoB,EAAC,IAAI,CAAC,CAAC,GAC7C,CAAC,MAAA,GAAG,CAAC,IAAI,mCAAI,EAAE,CAAC,CACpB;QACD,OAAO,EAAE,MAAA,GAAG,CAAC,OAAO,mCAAI,EAAE;QAC1B,uBAAuB,EAAE,MAAA,GAAG,CAAC,uBAAuB,mCAAI,KAAK;QAC7D,KAAK,EAAE,MAAA,GAAG,CAAC,KAAK,mCAAI,EAAE;QACtB,QAAQ,EAAE,MAAA,GAAG,CAAC,QAAQ,mCAAI,EAAE;QAC5B,OAAO;QACP,IAAI,EAAE,MAAA,GAAG,CAAC,IAAI,mCAAI,EAAE;KACrB,CAAC;AACJ,CAAC,CAAC;AA1BW,QAAA,UAAU,cA0BrB"}