@atelier-ui/create-workspace 0.2.27 → 0.2.36

package/CHANGELOG.md

@@ -1,3 +1,61 @@
+## 0.2.36 (2026-09-06)
+
+This was a version bump only for create-workspace to align it with other projects, there were no code changes.
+
+## 0.2.35 (2026-09-06)
+
+### 🩹 Fixes
+
+- **deps:** the manifests the sweep left behind, and one it must not touch ([460bdf9](https://github.com/DominikPieper/atelier-ui/commit/460bdf9))
+
+### ❤️ Thank You
+
+- Claude Opus 5 (1M context)
+- Dominik Pieper @DominikPieper
+
+## 0.2.34 (2026-09-05)
+
+This was a version bump only for create-workspace to align it with other projects, there were no code changes.
+
+## 0.2.33 (2026-09-05)
+
+This was a version bump only for create-workspace to align it with other projects, there were no code changes.
+
+## 0.2.32 (2026-09-05)
+
+### 🩹 Fixes
+
+- **preflight:** make the sync claim true instead of just writing it down ([b1b52d9](https://github.com/DominikPieper/atelier-ui/commit/b1b52d9))
+- **preflight:** check the ports of the environment you are actually in ([a2ff354](https://github.com/DominikPieper/atelier-ui/commit/a2ff354))
+
+### ❤️ Thank You
+
+- Claude Opus 5 (1M context)
+- Dominik Pieper @DominikPieper
+
+## 0.2.31 (2026-08-29)
+
+This was a version bump only for create-workspace to align it with other projects, there were no code changes.
+
+## 0.2.30 (2026-08-29)
+
+### 🚀 Features
+
+- **tokens:** decide the counting rule, then mint the two roles it earns ([93323ec](https://github.com/DominikPieper/atelier-ui/commit/93323ec))
+
+### ❤️ Thank You
+
+- Claude Opus 5 (1M context)
+- Dominik Pieper @DominikPieper
+
+## 0.2.29 (2026-08-29)
+
+This was a version bump only for create-workspace to align it with other projects, there were no code changes.
+
+## 0.2.28 (2026-08-29)
+
+This was a version bump only for create-workspace to align it with other projects, there were no code changes.
+
 ## 0.2.27 (2026-08-29)
 
 This was a version bump only for create-workspace to align it with other projects, there were no code changes.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atelier-ui/create-workspace",
-  "version": "0.2.27",
+  "version": "0.2.36",
   "description": "Nx preset for Atelier workshop workspaces",
   "keywords": [
     "nx",
@@ -21,7 +21,7 @@
   "types": "./src/index.d.ts",
   "generators": "./generators.json",
   "dependencies": {
-    "@nx/devkit": "23.1.1",
+    "@nx/devkit": "23.2.0",
     "prettier": "^3.0.0",
     "tslib": "^2.3.0"
   },

package/src/generators/preset/files/styles/tokens.css

@@ -143,6 +143,23 @@
   --ui-type-action: var(--ui-font-weight-semibold) var(--ui-font-size-md) /
     var(--ui-line-height-tight) var(--ui-font-family);
 
+  /* The row roles (ADR-0085). Where `control` is the label ON a control and
+   * `action` is the text of a control that acts, these two are the text IN a
+   * row: a menu item, an option, a table cell, a checkbox or radio label. They
+   * are regular weight, not medium, and they sit at the two default sizes the
+   * row ladder uses — `row` at md for the standard rung, `row-sm` at sm for the
+   * compact one. Same tight leading as the rest of the ladder: a row's height
+   * comes from --ui-row-height-*, so its leading must not grow with the text.
+   *
+   * The ADR-0073 carve-out applies here exactly as it does above: a control's
+   * own VALUE text keeps its longhands. `row` describes what the row DISPLAYS,
+   * not what an input CONTAINS.
+   */
+  --ui-type-row: var(--ui-font-weight-normal) var(--ui-font-size-md) /
+    var(--ui-line-height-tight) var(--ui-font-family);
+  --ui-type-row-sm: var(--ui-font-weight-normal) var(--ui-font-size-sm) /
+    var(--ui-line-height-tight) var(--ui-font-family);
+
   /* ── Colours · light mode (default) ────────────────────────── */
 
   /* Primary — Conciso deep teal */

package/src/generators/preset/files/tools/scripts/preflight.mjs

@@ -13,7 +13,7 @@
 
 import { execSync, spawnSync } from 'node:child_process';
 import { createServer } from 'node:net';
-import { readFileSync, existsSync } from 'node:fs';
+import { readFileSync, existsSync, statSync } from 'node:fs';
 import { homedir } from 'node:os';
 import { fileURLToPath } from 'node:url';
 import { dirname, join, resolve } from 'node:path';
@@ -155,25 +155,25 @@ async function probeMcp(url) {
   // 3. On an Atelier Storybook MCP, exercise one real tool call: manifests
   //    are only fetched inside tool calls, and a broken manifest fetch comes
   //    back as HTTP 200 + isError — invisible to anything shallower.
-  if (tools.some((t) => t?.name === 'list-all-documentation')) {
+  if (tools.some((t) => t?.name === 'docs-list')) {
     const call = await mcpPost(
       url,
       {
         jsonrpc: '2.0',
         id: 3,
         method: 'tools/call',
-        params: { name: 'list-all-documentation', arguments: {} },
+        params: { name: 'docs-list', arguments: {} },
       },
       session,
     );
-    if (call.error) return { level: 'broken', detail: `list-all-documentation failed (${call.error})` };
-    if (call.status !== 200) return { level: 'broken', detail: `list-all-documentation → HTTP ${call.status}` };
+    if (call.error) return { level: 'broken', detail: `docs-list failed (${call.error})` };
+    if (call.status !== 200) return { level: 'broken', detail: `docs-list → HTTP ${call.status}` };
     const callMsg = parseJsonRpc(call.contentType, call.text, 3);
     if (!callMsg?.result || callMsg.result.isError) {
       const reason =
         callMsg?.result?.content?.[0]?.text ?? callMsg?.error?.message ?? 'no parseable result';
       const oneLine = String(reason).replace(/\s+/g, ' ').trim();
-      return { level: 'broken', detail: `list-all-documentation → ${oneLine.slice(0, 120)}` };
+      return { level: 'broken', detail: `docs-list → ${oneLine.slice(0, 120)}` };
     }
   }
 
@@ -324,13 +324,15 @@ async function checkFigmaSetup() {
   ok('FIGMA_ACCESS_TOKEN', `set (${token.length} chars, REST reads enabled)`);
 }
 
-async function checkMcpEndpoints() {
+async function checkMcpEndpoints(env) {
   const endpoints = loadMcpEndpoints();
   if (endpoints.length === 0) {
     warn(
       'MCP endpoints',
       'none configured in .mcp.json',
-      'Scaffold a workspace with `npx create-atelier-ui-workspace`',
+      env === 'clone'
+        ? 'The clone ships .mcp.json at the repo root with these servers — check it was not deleted, renamed, or run from the wrong directory'
+        : 'Scaffold a workspace with `npx create-atelier-ui-workspace`',
     );
     return;
   }
@@ -354,15 +356,88 @@ async function checkMcpEndpoints() {
   }
 }
 
-async function checkPorts() {
-  // 4200 = the dev server (every framework serves on the @nx/vite default),
-  // 6006 = local Storybook. Nothing in the scaffold uses 4201/4202 — the old
-  // multi-framework rig did, and checking them only confused participants.
-  const ports = [4200, 6006];
-  for (const p of ports) {
-    const free = await portFree(p);
-    if (free) ok(`Port ${p}`, 'free');
-    else warn(`Port ${p}`, 'in use', `Run \`lsof -ti :${p} | xargs kill\` (macOS/Linux)`);
+// ── Environment detection ───────────────────────────────────────────
+// This file ships identically into two trees: the atelier monorepo that the
+// two-day cohort clones, and the single-framework workspace `create-workspace`
+// scaffolds for everyone else (ADR-0084). `ROOT` sits at the same relative
+// depth (two levels above tools/scripts) in both, so the ports to check
+// differ by tree even though the code is one file — the environment logic
+// has to live in here, not as a divergence between the two copies. The two
+// copies are kept byte-identical by `npm run check:preflight`
+// (tools/scripts/sync-preflight.mjs --check) — NOT by `check:sync`, which
+// only checks Angular/React/Vue component drift and never looks at
+// `libs/create-workspace/`.
+//
+// Discriminator: both `libs/spec/` (the framework-agnostic spec contract,
+// libs/spec/src/index.ts — see CLAUDE.md) and `plan/adr/` (the decision log)
+// are directories that exist only in the source monorepo — `create-workspace`'s
+// preset generator provably never writes either one into a scaffolded
+// workspace (it writes a single `workshop-<fw>` app instead; see preset.ts).
+// Requiring both, rather than either alone, is deliberate: a single directory
+// name is something a team could plausibly recreate inside a scaffold they
+// extend (e.g. adding their own `libs/spec` package), and a false "clone"
+// verdict then goes silent — checkPorts skips 4200/6006 entirely and reports
+// an all-clear on ports it never looked at. Two thematically-linked,
+// distinctively-named directories are a much smaller false-positive target,
+// at the cost of one extra `statSync` call — still two stats, not workspace
+// introspection.
+//
+// This does not eliminate the risk, only shrinks it, and there is no cheap
+// way to shrink it further: a tree that genuinely has both directories for
+// unrelated reasons is indistinguishable from a real clone without deeper
+// introspection than this check is willing to pay for. That residual case
+// gets no separate warning branch — a "some checks may be wrong" message
+// with no actionable next step is not worth the extra state. The reported
+// `Environment` line is the mitigation: it prints which verdict was reached
+// so a participant who is not where they expect notices immediately (the
+// motivation for this whole change). Fall back to `scaffold` when neither
+// directory (or only one) is present: the generated workspace is the copy
+// that ships to strangers, so an undetectable or partial-match tree should
+// behave exactly as it did before this change.
+function isDir(path) {
+  try {
+    return statSync(path).isDirectory();
+  } catch {
+    return false;
+  }
+}
+
+function detectEnvironment() {
+  const isClone = isDir(resolve(ROOT, 'libs/spec')) && isDir(resolve(ROOT, 'plan/adr'));
+  return isClone ? 'clone' : 'scaffold';
+}
+
+const PORTS_BY_ENV = {
+  // The atelier monorepo clone: the docs app and one Storybook per framework
+  // (libs/{angular,react,vue}/project.json's `storybook dev … --port 44xx`,
+  // docs/project.json:21). Nothing in the clone runs on 4200/6006.
+  clone: [
+    { port: 4300, label: 'docs' },
+    { port: 4400, label: 'Storybook (angular)' },
+    { port: 4401, label: 'Storybook (react)' },
+    { port: 4402, label: 'Storybook (vue)' },
+  ],
+  // The scaffolded workspace: 4200 = the dev server (every framework serves
+  // on the @nx/vite default), 6006 = its single local Storybook. Nothing in
+  // the scaffold uses 4201/4202 — the old multi-framework rig did, and
+  // checking them only confused participants.
+  scaffold: [
+    { port: 4200, label: 'dev server' },
+    { port: 6006, label: 'Storybook' },
+  ],
+};
+
+async function checkPorts(env) {
+  ok(
+    'Environment',
+    env === 'clone'
+      ? 'atelier monorepo clone — checking docs (4300) + Storybook (4400/4401/4402)'
+      : 'scaffolded workspace — checking dev server (4200) + Storybook (6006)',
+  );
+  for (const { port, label } of PORTS_BY_ENV[env]) {
+    const free = await portFree(port);
+    if (free) ok(`Port ${port} (${label})`, 'free');
+    else warn(`Port ${port} (${label})`, 'in use', `Run \`lsof -ti :${port} | xargs kill\` (macOS/Linux)`);
   }
 }
 
@@ -377,6 +452,8 @@ async function main() {
   console.log(bold('Atelier UI Preflight'));
   console.log(dim('Checking your environment for the workshop…'));
 
+  const env = detectEnvironment();
+
   header('Runtime');
   checkNode();
   checkNpm();
@@ -389,10 +466,10 @@ async function main() {
   await checkFigmaSetup();
 
   header('MCP reachability');
-  await checkMcpEndpoints();
+  await checkMcpEndpoints(env);
 
   header('Local ports');
-  await checkPorts();
+  await checkPorts(env);
 
   // Summary
   const failures = results.filter((r) => r.level === 'fail').length;

package/src/generators/preset/preset.js

@@ -124,9 +124,9 @@ function presetGenerator(tree, options) {
         const mcpSection = frameworks
             .map((f) => `### \`storybook-${f}\` MCP
 Before using any component:
-1. Call \`list-all-documentation\` to get valid component IDs
-2. Call \`get-documentation\` with the ID — never invent props
-3. Call \`get-documentation-for-story\` for a specific variant
+1. Call \`docs-list\` to get valid component IDs
+2. Call \`docs-show\` with the ID — never invent props
+3. Call \`docs-show-story\` for a specific variant
 4. Do not use a component that is not in the docs`)
             .join('\n\n');
         tree.write('CLAUDE.md', `# Atelier Workshop