@natjswenson/press 0.6.1 → 0.7.1

package/CHANGELOG.md

@@ -5,6 +5,61 @@ All notable changes to the **press** skill are documented here.
 The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) and
 this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.7.1] - 2026-08-01
+
+### Fixed
+
+- **Declaring a new target broke every consumer's CI.** 0.7.0 added README
+  targets for the external repos without seeding the regions, so the moment a
+  consumer's pin reached 0.7.0 its `check` failed `missing`. `propagate` now
+  **seeds** a region the consumer does not have yet, in the same pull request
+  that raises the pin — the two are one change, not two. Only targets declaring
+  an `init` anchor are seedable, so a deliberately deleted region is still
+  reported rather than silently recreated.
+- **`check` now enforces the recorded version, not just the values.** In-repo
+  regions sat at `v0.1.0` through six releases with a green check, because only
+  content was compared. One version per consumer means `check` has to say so.
+- New `init.insertAfter` anchor, so a seeded README badge lands under the title
+  instead of at the bottom of the file.
+
+### Added
+
+- Version badges for the in-repo skills' READMEs (`city-report`, `resume`,
+  `ghostwriter`, `ghostwriter-x`, `devlog`), and `press` itself is now listed in
+  the monorepo's root README skill table.
+- `natejswenson` has no README target on purpose: its README is generated by
+  `build_svg.py` and a region there would be clobbered on the next build.
+
+## [0.7.0] - 2026-08-01
+
+### Added
+
+- **`version-badge` emitter** — a generated `press:version` line for a
+  consumer's README. Until now a repo's press version appeared only in a CI pin
+  and a comment marker, neither of which anyone reads when landing on the repo.
+- Emitters now receive a small run context (`{ version }`), kept separate from
+  target `params` because it is a property of the invocation, not the config.
+
+### Changed
+
+- **Propagation now covers every long-lived branch, not just `dev`.** Landing on
+  the integration branch alone left `main` — the branch that actually ships —
+  stale until someone remembered to promote, and made the next promotion carry
+  an unrelated brand diff. Branch names are suffixed per base so the two pushes
+  cannot collide.
+
+### Fixed
+
+- **Target selection could match the wrong repository.** Selection was by file
+  presence alone, and `README.md` exists everywhere — so a README target
+  selected inside *any* checkout and would have been compared against the wrong
+  file. Selection now checks the checkout's own `origin` remote against the
+  target's `github`/`repo`, falling back to presence only when there is no
+  remote.
+- Goldens pin the **shape** of each emitter's output using a fixed placeholder
+  version. Using the real release would rewrite every golden on every release
+  and turn a drift detector into noise.
+
 ## [0.6.1] - 2026-08-01
 
 ### Fixed
@@ -264,6 +319,8 @@ source of truth and a CI drift gate.
   rasterised cards, whose eyebrow legitimately runs at `.16em`. Scoped rather
   than waived, and caught by linting the real shipped corpus.
 
+[0.7.1]: https://github.com/natejswenson/claude-skills/releases/tag/press-v0.7.1
+[0.7.0]: https://github.com/natejswenson/claude-skills/releases/tag/press-v0.7.0
 [0.6.1]: https://github.com/natejswenson/claude-skills/releases/tag/press-v0.6.1
 [0.6.0]: https://github.com/natejswenson/claude-skills/releases/tag/press-v0.6.0
 [0.5.1]: https://github.com/natejswenson/claude-skills/releases/tag/press-v0.5.1

package/SKILL.md

@@ -2,7 +2,7 @@
 name: press
 description: The one brand system for everything produced in Claude — design tokens, the visual laws, the run-presentation contract, and the universal voice core. Use when composing or restyling any artifact (report, résumé, card, cover, PDF, HTML page, chart), when asked about brand colors, fonts, the accent law or "the PRESS look", when adding a new skill that renders anything, or when a brand value needs to change everywhere at once. Also handles "check the brand is in sync", "why do my colors differ", and onboarding a new consumer repo.
 user_invocable: true
-version: 0.6.1
+version: 0.7.1
 ---
 
 # /press — the brand system
@@ -121,8 +121,14 @@ is that the PR says which kind it is:
 | **Brand values moved** | `press vX — BRAND VALUES CHANGED` | read the diff as a design change |
 | **Version only** | `adopt press vX` | one line; nothing renders differently |
 
-`.github/workflows/press-propagate.yml` runs this on every release and weekly,
-opening a PR in each consumer repo whose bytes actually moved.
+`.github/workflows/press-propagate.yml` runs this on every release and weekly.
+It propagates to **every long-lived branch** a consumer has — `dev` *and* the
+default branch — because landing on `dev` alone leaves the branch that actually
+ships stale until someone remembers to promote, and makes the next promotion
+carry an unrelated brand diff.
+
+Each consumer's README carries a generated `press:version` line, so the release
+a repo is on is visible without opening a workflow file.
 
 ## Changing a brand value
 

package/bin/press.js

@@ -86,7 +86,7 @@ function cmdEmit({ tokens, root, values }) {
   const rows = [];
   for (const target of targets) {
     const path = targetPath(target, root);
-    const body = emitBody(tokens, target.emitter, target.params ?? {});
+    const body = emitBody(tokens, target.emitter, target.params ?? {}, { version: VERSION });
     const before = readFileSync(path, 'utf8');
     const has = findRegion(before, target.region, target.syntax);
 

package/lib/check.mjs

@@ -37,12 +37,21 @@ export function checkTarget(target, root, tokens, version) {
   }
   if (!found) return { target, path, status: 'missing' };
 
-  const expected = emitBody(tokens, target.emitter, target.params ?? {}).replace(/\s+$/, '');
+  const expected = emitBody(tokens, target.emitter, target.params ?? {}, { version })
+    .replace(/\s+$/, '');
   const actual = found.body.replace(/\s+$/, '');
-  if (expected === actual) {
-    return { target, path, status: 'ok', writtenBy: found.version };
+  if (expected !== actual) {
+    return { target, path, status: 'drift', diff: lineDiff(expected, actual), writtenBy: found.version };
   }
-  return { target, path, status: 'drift', diff: lineDiff(expected, actual), writtenBy: found.version };
+  // The values match, but the region must also record the release it belongs
+  // to. Without this the version is decoration: in-repo regions sat at v0.1.0
+  // through six releases with a green check, which is precisely the "three
+  // different version numbers and no way to read them" problem this policy
+  // exists to end. One version per consumer means check has to say so.
+  if (found.version !== version) {
+    return { target, path, status: 'stale-version', writtenBy: found.version };
+  }
+  return { target, path, status: 'ok', writtenBy: found.version };
 }
 
 export function checkAll({ tokens, targets, root, ids, version }) {
@@ -85,6 +94,7 @@ export function lineDiff(expected, actual) {
 export const EXPLAIN = {
   ok: 'in sync',
   drift: 'region content differs from what the tokens produce',
+  'stale-version': 'values are correct but the region records an older press release — re-emit',
   missing: 'file has no press region — the generated block was removed',
   absent: 'declared file does not exist at this path',
   corrupt: 'region markers are malformed',

package/lib/emit.mjs

@@ -292,23 +292,49 @@ function pythonConsts(tokens, params) {
   return out.join('\n');
 }
 
+
+/**
+ * A short "this repo is on press vX" note for a consumer's README.
+ *
+ * The version is the point: without it, the only places a repo's press version
+ * appears are a CI pin and a comment marker, neither of which anyone reads. A
+ * reader landing on the repo should be able to see which brand release it is on
+ * without opening a workflow file.
+ */
+function versionBadge(tokens, params, ctx) {
+  const version = ctx.version ?? '0.0.0';
+  const repo = params.repo_url ?? 'https://github.com/natejswenson/claude-skills/tree/main/skills/press';
+  const what = params.what ?? 'Design tokens, brand laws and run presentation';
+  return [
+    `**Brand:** PRESS v${version} — ${what} are generated from the`,
+    `[\`press\`](${repo}) skill, not maintained here. The token blocks marked`,
+    '`press:tokens` are generated; edit `press/brand/tokens.json` and re-emit instead.',
+  ].join('\n');
+}
+
 export const EMITTERS = {
   'python-theme': pythonTheme,
   'python-consts': pythonConsts,
+  'version-badge': versionBadge,
   'css-vars': cssVars,
   'md-palette': mdPalette,
   'markdown-block': markdownBlock,
   json: jsonTokens,
 };
 
-export function emitBody(tokens, emitter, params = {}) {
+/**
+ * @param ctx  Run context available to emitters — currently `{ version }`.
+ *             Kept separate from `params` because it is a property of the
+ *             invocation, not of the target's configuration.
+ */
+export function emitBody(tokens, emitter, params = {}, ctx = {}) {
   const fn = EMITTERS[emitter];
   if (!fn) {
     throw new EmitError(
       `unknown emitter "${emitter}" (expected one of: ${Object.keys(EMITTERS).join(', ')})`,
     );
   }
-  return fn(tokens, params);
+  return fn(tokens, params, ctx);
 }
 
 // --------------------------------------------------------------------------

package/lib/propagate.mjs

@@ -25,7 +25,7 @@ import { readFileSync, writeFileSync, existsSync, readdirSync } from 'node:fs';
 import { join } from 'node:path';
 
 import { emitBody } from './emit.mjs';
-import { findRegion, spliceRegion } from './region.mjs';
+import { findRegion, initRegion, spliceRegion } from './region.mjs';
 import { selectTargets, targetPath } from './targets.mjs';
 
 /** The npm reference consumers pin in CI, e.g. `@natjswenson/press@0.3.0`. */
@@ -47,10 +47,35 @@ export function propagate({ tokens, targets, root, version, dryRun = false }) {
     const before = readFileSync(path, 'utf8');
     const found = findRegion(before, target.region, target.syntax);
     if (!found) {
-      regions.push({ id: target.id, path: target.path, status: 'missing' });
+      // A newly declared target has no region in the consumer yet, and that
+      // consumer's `check` starts failing the moment its pin reaches the
+      // release that declares it. So the PR that raises the pin must also
+      // create the region — the two are one change, not two.
+      //
+      // This is not "silently creating": it lands in a reviewed pull request
+      // alongside the pin bump. A region that was deliberately DELETED is a
+      // different case, and one this cannot distinguish, which is why the
+      // target must declare an `init` anchor to be seedable at all.
+      if (!target.init) {
+        regions.push({ id: target.id, path: target.path, status: 'missing' });
+        continue;
+      }
+      const body = emitBody(tokens, target.emitter, target.params ?? {}, { version });
+      if (!dryRun) {
+        writeFileSync(path, initRegion(before, target.region, target.syntax, body, version, target.init), 'utf8');
+      }
+      regions.push({
+        id: target.id,
+        path: target.path,
+        changed: true,
+        brandChanged: false,
+        versionChanged: true,
+        status: dryRun ? 'would seed' : 'seeded',
+        wroteBy: null,
+      });
       continue;
     }
-    const body = emitBody(tokens, target.emitter, target.params ?? {});
+    const body = emitBody(tokens, target.emitter, target.params ?? {}, { version });
     const after = spliceRegion(before, target.region, target.syntax, body, version);
     // Two different kinds of "changed", kept apart on purpose.
     //

package/lib/region.mjs

@@ -153,7 +153,22 @@ export function initRegion(text, region, syntaxName, body, version, anchor = {})
     throw new RegionError(`file already has a press:${region} region — use emit without --init`);
   }
   const block = renderRegion(region, syntaxName, body, version);
-  const { replaceFrom, replaceTo } = anchor;
+  const { replaceFrom, replaceTo, insertAfter } = anchor;
+
+  // Seeding a brand-new region (a README badge) rather than taking over a
+  // hand-written block: place it after the first line matching `insertAfter`,
+  // usually the title, instead of appending to the bottom where nobody looks.
+  if (insertAfter) {
+    const lines = text.split('\n');
+    const re = new RegExp(insertAfter);
+    const at = lines.findIndex((l) => re.test(l));
+    if (at === -1) throw new RegionError(`anchor insertAfter /${insertAfter}/ matched no line`);
+    let i = at + 1;
+    while (i < lines.length && lines[i].trim() === '') i += 1;
+    lines.splice(i, 0, ...block.split('\n'), '');
+    return lines.join('\n');
+  }
+
   if (!replaceFrom) return `${text.replace(/\s+$/, '')}\n\n${block}\n`;
 
   const lines = text.split('\n');

package/lib/targets.mjs

@@ -6,6 +6,7 @@
  * `check`, which is why `doctor` exists to show the whole registry rather than
  * only what resolved locally.
  */
+import { execFileSync } from 'node:child_process';
 import { existsSync, readFileSync } from 'node:fs';
 import { fileURLToPath } from 'node:url';
 import { dirname, isAbsolute, join, resolve } from 'node:path';
@@ -43,6 +44,26 @@ export function repoRoot(start = process.cwd()) {
 export const targetPath = (target, root) =>
   isAbsolute(target.path) ? target.path : join(root, target.path);
 
+/**
+ * The repository a checkout actually is, from its origin remote.
+ *
+ * File presence alone cannot identify a consumer: `README.md` exists in every
+ * repo, so a README target would select inside any checkout and be checked
+ * against the wrong file. Returns null when there is no remote (a temp dir in a
+ * test), in which case callers fall back to presence.
+ */
+export function repoIdentity(root) {
+  try {
+    const url = execFileSync('git', ['-C', root, 'config', '--get', 'remote.origin.url'], {
+      encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'],
+    }).trim();
+    const m = /([^/:]+?)(?:\.git)?$/.exec(url);
+    return m ? m[1] : null;
+  } catch {
+    return null;
+  }
+}
+
 /**
  * Which targets this invocation is responsible for.
  *
@@ -63,5 +84,12 @@ export function selectTargets(targets, { root, ids }) {
       return found;
     });
   }
-  return targets.filter((t) => existsSync(targetPath(t, root)));
+  const identity = repoIdentity(root);
+  return targets.filter((t) => {
+    if (!existsSync(targetPath(t, root))) return false;
+    // When the checkout names itself, trust that over a path that happens to
+    // exist — otherwise every repo's README matches every README target.
+    if (identity) return identity === (t.github ?? t.repo);
+    return true;
+  });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@natjswenson/press",
-  "version": "0.6.1",
+  "version": "0.7.1",
   "description": "One brand system — tokens, laws, run presentation and voice core — generated into every consumer with a CI drift gate",
   "license": "MIT",
   "author": "Nate Swenson",

package/targets.json

@@ -395,9 +395,122 @@
         ]
       },
       "init": {
-        "replaceFrom": "^# --- PRESS tokens",
-        "replaceTo": "^\\}$"
+        "insertAfter": "^# "
+      }
+    },
+    {
+      "id": "natejswenson.io-readme",
+      "repo": "natejswenson.io",
+      "path": "README.md",
+      "region": "version",
+      "syntax": "md",
+      "emitter": "version-badge",
+      "params": {
+        "what": "The site's colour, type and panel tokens"
+      },
+      "init": {
+        "insertAfter": "^# "
+      }
+    },
+    {
+      "id": "budget-readme",
+      "repo": "budget",
+      "path": "README.md",
+      "region": "version",
+      "syntax": "md",
+      "emitter": "version-badge",
+      "params": {
+        "what": "The report's colour and type tokens"
+      },
+      "github": "local-budget",
+      "init": {
+        "insertAfter": "^# "
+      }
+    },
+    {
+      "id": "local-fitness-readme",
+      "repo": "local-fitness",
+      "path": "README.md",
+      "region": "version",
+      "syntax": "md",
+      "emitter": "version-badge",
+      "params": {
+        "what": "The brief and report-card colour and type tokens"
+      },
+      "init": {
+        "insertAfter": "^# "
+      }
+    },
+    {
+      "id": "city-report-readme",
+      "repo": "claude-skills",
+      "path": "skills/city-report/README.md",
+      "region": "version",
+      "syntax": "md",
+      "emitter": "version-badge",
+      "params": {
+        "what": "The report's colour and type tokens"
+      },
+      "init": {
+        "insertAfter": "^# "
+      }
+    },
+    {
+      "id": "resume-readme",
+      "repo": "claude-skills",
+      "path": "skills/resume/README.md",
+      "region": "version",
+      "syntax": "md",
+      "emitter": "version-badge",
+      "params": {
+        "what": "The résumé themes' colour and type tokens"
+      },
+      "init": {
+        "insertAfter": "^# "
+      }
+    },
+    {
+      "id": "ghostwriter-readme",
+      "repo": "claude-skills",
+      "path": "skills/ghostwriter/README.md",
+      "region": "version",
+      "syntax": "md",
+      "emitter": "version-badge",
+      "params": {
+        "what": "The card system's colour, type and terminal tokens"
+      },
+      "init": {
+        "insertAfter": "^# "
+      }
+    },
+    {
+      "id": "ghostwriter-x-readme",
+      "repo": "claude-skills",
+      "path": "skills/ghostwriter-x/README.md",
+      "region": "version",
+      "syntax": "md",
+      "emitter": "version-badge",
+      "params": {
+        "what": "The card system's colour, type and terminal tokens"
+      },
+      "init": {
+        "insertAfter": "^# "
+      }
+    },
+    {
+      "id": "devlog-readme",
+      "repo": "claude-skills",
+      "path": "skills/devlog/README.md",
+      "region": "version",
+      "syntax": "md",
+      "emitter": "version-badge",
+      "params": {
+        "what": "The cover-image palette"
+      },
+      "init": {
+        "insertAfter": "^# "
       }
     }
-  ]
+  ],
+  "$readme_note": "natejswenson has no README target on purpose: its README.md is generated by build_svg.py, so a region there would be clobbered on the next build. Its press version is visible in build_svg.py's own region marker instead."
 }