@astryxdesign/cli 0.1.7-canary.fb9c68c → 0.1.7

package/src/utils/package-manager.test.mjs

@@ -4,16 +4,9 @@ import {describe, it, expect, afterEach, vi} from 'vitest';
 import * as fs from 'node:fs';
 import * as path from 'node:path';
 import * as os from 'node:os';
-import {
-  detectPackageManager,
-  getDlxPrefix,
-  isCliOneOff,
-  getCliInvocation,
-  formatCliCommand,
-} from './package-manager.mjs';
+import {detectPackageManager} from './package-manager.mjs';
 
 let tmpDir;
-const ORIGINAL_ARGV1 = process.argv[1];
 
 afterEach(() => {
   if (tmpDir) {
@@ -22,7 +15,6 @@ afterEach(() => {
   }
   vi.restoreAllMocks();
   delete process.env.npm_config_user_agent;
-  process.argv[1] = ORIGINAL_ARGV1;
 });
 
 function makeTmpDir() {
@@ -119,102 +111,3 @@ describe('detectPackageManager', () => {
     expect(detectPackageManager(dir)).toBe('bun');
   });
 });
-
-describe('getDlxPrefix', () => {
-  it('returns "pnpm dlx" for pnpm projects', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'pnpm-lock.yaml'), '');
-    expect(getDlxPrefix(dir)).toBe('pnpm dlx');
-  });
-
-  it('returns "yarn dlx" for yarn projects', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'yarn.lock'), '');
-    expect(getDlxPrefix(dir)).toBe('yarn dlx');
-  });
-
-  it('returns "bunx" for bun projects', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'bun.lockb'), '');
-    expect(getDlxPrefix(dir)).toBe('bunx');
-  });
-
-  it('falls back to "npx" with no signals', () => {
-    const dir = makeTmpDir();
-    delete process.env.npm_config_user_agent;
-    expect(getDlxPrefix(dir)).toBe('npx');
-  });
-});
-
-describe('isCliOneOff', () => {
-  it('detects an npm npx cache entry', () => {
-    process.argv[1] = '/home/u/.npm/_npx/a1b2/node_modules/.bin/astryx';
-    expect(isCliOneOff()).toBe(true);
-  });
-
-  it('detects a pnpm dlx cache entry', () => {
-    process.argv[1] = '/home/u/.cache/pnpm/dlx/9f/node_modules/@astryxdesign/cli/bin/astryx.mjs';
-    expect(isCliOneOff()).toBe(true);
-  });
-
-  it('detects a bunx cache entry', () => {
-    process.argv[1] = '/home/u/.bun/install/cache/@astryxdesign/cli/bin/astryx.mjs';
-    expect(isCliOneOff()).toBe(true);
-  });
-
-  it('is false for an installed node_modules entry', () => {
-    process.argv[1] = '/proj/node_modules/@astryxdesign/cli/bin/astryx.mjs';
-    expect(isCliOneOff()).toBe(false);
-  });
-
-  it('is false for a source checkout (dev) entry', () => {
-    process.argv[1] = '/repo/packages/cli/bin/astryx.mjs';
-    expect(isCliOneOff()).toBe(false);
-  });
-});
-
-describe('getCliInvocation', () => {
-  it('uses the run-prefix + bare bin when installed (not one-off)', () => {
-    process.argv[1] = '/proj/node_modules/@astryxdesign/cli/bin/astryx.mjs';
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'pnpm-lock.yaml'), '');
-    expect(getCliInvocation(dir)).toBe('pnpm exec astryx');
-  });
-
-  it('uses the dlx runner + scoped package when run one-off', () => {
-    process.argv[1] = '/home/u/.npm/_npx/a1b2/node_modules/.bin/astryx';
-    const dir = makeTmpDir();
-    delete process.env.npm_config_user_agent;
-    expect(getCliInvocation(dir)).toBe('npx @astryxdesign/cli');
-  });
-
-  it('pairs the dlx runner with the scoped package for pnpm one-off', () => {
-    process.argv[1] = '/home/u/.cache/pnpm/dlx/9f/node_modules/@astryxdesign/cli/bin/astryx.mjs';
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'pnpm-lock.yaml'), '');
-    expect(getCliInvocation(dir)).toBe('pnpm dlx @astryxdesign/cli');
-  });
-});
-
-describe('formatCliCommand', () => {
-  it('strips a leading "astryx" token and prepends the invocation stem', () => {
-    process.argv[1] = '/proj/node_modules/@astryxdesign/cli/bin/astryx.mjs';
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'pnpm-lock.yaml'), '');
-    expect(formatCliCommand('astryx component Button', dir)).toBe('pnpm exec astryx component Button');
-  });
-
-  it('accepts a bare subcommand (no leading astryx)', () => {
-    process.argv[1] = '/proj/node_modules/@astryxdesign/cli/bin/astryx.mjs';
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'package-lock.json'), '{}');
-    expect(formatCliCommand('docs tokens', dir)).toBe('npx astryx docs tokens');
-  });
-
-  it('rewrites to the scoped package for one-off invocations', () => {
-    process.argv[1] = '/home/u/.npm/_npx/a1b2/node_modules/.bin/astryx';
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'package-lock.json'), '{}');
-    expect(formatCliCommand('astryx component Button', dir)).toBe('npx @astryxdesign/cli component Button');
-  });
-});

package/src/utils/path-safety.mjs

@@ -165,3 +165,21 @@ export function isFilePathArg(pathArg) {
   const ext = path.extname(base).toLowerCase();
   return ext.length > 0 && FILE_EXTENSIONS.has(ext);
 }
+
+/**
+ * True when the process is running non-interactively (no TTY) or when the
+ * caller has signaled JSON / scripted use. Commands consult this before
+ * prompting for confirmation; in scripted mode they require an explicit
+ * `--overwrite` flag instead.
+ *
+ * @param {object} [options]
+ * @param {boolean} [options.json] - Caller's --json flag.
+ * @returns {boolean}
+ */
+export function isNonInteractive({json = false} = {}) {
+  if (json) return true;
+  // stdin not a TTY means piped input or scripted execution.
+  if (process.stdin && process.stdin.isTTY === false) return true;
+  if (process.stdout && process.stdout.isTTY === false) return true;
+  return false;
+}

package/src/utils/update-check.mjs

@@ -14,7 +14,6 @@
 import * as fs from 'node:fs';
 import * as path from 'node:path';
 import {semverGt} from './semver.mjs';
-import {getCliInvocation} from './package-manager.mjs';
 
 /**
  * Read the latest available version from local signals.
@@ -76,7 +75,7 @@ export function checkForUpdate(cwd = process.cwd()) {
   // Use semver-aware comparison so '0.0.20' is correctly treated as greater
   // than '0.0.5' (lexicographic compare gets that backwards).
   if (semverGt(latest, installed)) {
-    return `FYI: A newer version of @astryxdesign/core (${latest}) is available. Install the new package version, then run: ${getCliInvocation()} upgrade --from <old-version> --apply`;
+    return `FYI: A newer version of @astryxdesign/core (${latest}) is available. Install the new package version, then run: astryx upgrade --from <old-version> --apply`;
   }
 
   return null;

package/templates/blocks/components/TabList/TabListTabsWithActions.doc.mjs

@@ -7,7 +7,7 @@ export const doc = {
   name: 'TabList — With Actions',
   displayName: 'TabList — With Actions',
   description:
-    'Page header pattern with tabs on the left and action buttons pushed to the right. When hasDivider is true, match the Button size to the TabList size so the tabs and actions align to a shared baseline above the divider.',
+    'Page header pattern with tabs on the left and action buttons pushed to the right. When hasDivider is true, pair with a smaller button size (sm) so actions don\'t overpower the tab row.',
   isReady: true,
   aspectRatio: 4 / 3,
   componentsUsed: ['TabList', 'Tab', 'Button'],

package/templates/blocks/components/TabList/TabListTabsWithActions.tsx

@@ -55,11 +55,16 @@ export default function TabListTabsWithActions() {
         <Button
           label="Filter"
           variant="ghost"
-          size="lg"
+          size="sm"
           icon={FilterIcon}
           isIconOnly
         />
-        <Button label="New item" variant="primary" size="lg" icon={PlusIcon} />
+        <Button
+          label="New item"
+          variant="primary"
+          size="sm"
+          icon={PlusIcon}
+        />
       </div>
     </TabList>
   );

package/docs/cli-integrations.doc.mjs

@@ -1,150 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/** @type {import('../../core/src/docs-types').ReferenceDoc} */
-
-export const docs = {
-  name: 'cli-integrations',
-  title: 'CLI Integrations',
-  category: 'guide',
-  description:
-    'Author an npm package that contributes components, templates, and upgrade codemods to Astryx.',
-
-  sections: [
-    {
-      title: 'Overview',
-      category: 'guide',
-      content: [
-        {
-          type: 'prose',
-          text: 'An integration is an npm package that contributes components, templates, and/or upgrade codemods to a consumer\u2019s design-system workflow. Consumers install the package and add it to their `astryx.config`; from then on the integration\u2019s contributions show up alongside core\u2019s in the same CLI commands.',
-        },
-        {
-          type: 'prose',
-          text: 'The system runs on two files. The consumer writes `astryx.config.{ts,mjs,js}` at their project root to list which packages to load. The author writes `astryx.integration.{ts,mjs,js}` at the package root to declare what the package contributes. This page is the author\u2019s guide. For the consumer side, run `npx astryx docs getting-started`.',
-        },
-        {
-          type: 'prose',
-          text: 'On the consumer side, adding your package is one line:',
-        },
-        {
-          type: 'code',
-          lang: 'typescript',
-          code: "import {createConfig} from '@astryxdesign/core/config';\n\nexport default createConfig({\n  integrations: ['@acme/astryx-widgets'],\n});",
-        },
-        {
-          type: 'prose',
-          text: 'Your components and templates then appear next to core\u2019s:',
-        },
-        {
-          type: 'code',
-          lang: 'bash',
-          code: 'astryx component --list --package @acme/astryx-widgets\nastryx component AcmeCarousel --props',
-        },
-      ],
-    },
-    {
-      title: 'The Integration File',
-      category: 'guide',
-      content: [
-        {
-          type: 'prose',
-          text: 'To register your package as an integration, add an `astryx.integration.{ts,mjs,js}` file as a sibling of your `package.json`. It tells the CLI where to find your components, templates, and codemods. Identity (name, version) comes from your `package.json`, not this file.',
-        },
-        {
-          type: 'code',
-          lang: 'typescript',
-          code: "// astryx.integration.ts\nimport {createIntegration} from '@astryxdesign/core/authoring';\n\nexport default createIntegration({\n  components: './components',\n  templates: './templates',\n  codemods: './codemods',\n  issuesUrl: 'https://github.com/acme/widgets/issues',\n});",
-        },
-        {
-          type: 'prose',
-          text: 'Every field is optional. Declare only the contribution roots your package ships. `createIntegration` is a type-preserving helper for editor autocomplete and type-checking. It lives in `@astryxdesign/core/authoring` and is also re-exported from `@astryxdesign/cli/integration` for back-compat.',
-        },
-      ],
-    },
-    {
-      title: 'Components',
-      category: 'guide',
-      content: [
-        {
-          type: 'prose',
-          text: 'Export your components from your library however you like, and consumers still import them from your package. For each component the CLI should document, ship a `.doc.{ts,mjs,js}` file with the same stem, for example `AcmeCarousel.tsx` alongside `AcmeCarousel.doc.ts`.',
-        },
-        {
-          type: 'code',
-          lang: 'typescript',
-          code: "// AcmeCarousel.doc.ts\nimport {createComponentDoc} from '@astryxdesign/core/authoring';\n\nexport default createComponentDoc({\n  name: 'AcmeCarousel',\n  description: 'A carousel that cycles through slides.',\n  // props, usage, examples, ...\n});",
-        },
-      ],
-    },
-    {
-      title: 'Templates',
-      category: 'guide',
-      content: [
-        {
-          type: 'prose',
-          text: 'Templates are usually not exported from the package directly. Instead, consumers browse them through the CLI and materialize them into their app. Define a template with `createPageTemplate` (full pages) or `createBlockTemplate` (smaller chunks) in a `.template.{ts,mjs,js}` file next to the source, for example `AcmeLandingPage.tsx` and `AcmeLandingPage.template.ts`.',
-        },
-        {
-          type: 'code',
-          lang: 'typescript',
-          code: "// AcmeLandingPage.template.ts\nimport {createPageTemplate} from '@astryxdesign/core/authoring';\n\nexport default createPageTemplate({\n  // name, description, preview, ...\n});",
-        },
-        {
-          type: 'prose',
-          text: 'The CLI needs the template source at consume time, so make sure it is included in your published package. This is typically done via the `exports` key in `package.json`. It also lets the docsite render template previews in the future.',
-        },
-        {
-          type: 'code',
-          lang: 'jsonc',
-          code: '{\n  "exports": {\n    // ...\n    "./templates/*.tsx": "./templates/*.tsx"\n  }\n}',
-        },
-        {
-          type: 'prose',
-          text: 'To verify it resolves, try importing the template component with its `.tsx` extension. An extensionless specifier will not resolve under `moduleResolution: bundler`, and the extensionful export above is what lets this type-check without consumers enabling `allowImportingTsExtensions`.',
-        },
-        {
-          type: 'code',
-          lang: 'typescript',
-          code: "import('@acme/astryx-widgets/templates/AcmeLandingPage.tsx');",
-        },
-      ],
-    },
-    {
-      title: 'Codemods',
-      category: 'guide',
-      content: [
-        {
-          type: 'prose',
-          text: 'Ship codemods so `astryx upgrade` can migrate consumers across breaking changes in your package. Point the integration file\u2019s `codemods` field at your codemods root, and author each one with `createCodemod` (transforms source files) or `createConfigCodemod` (rewrites the consumer\u2019s `astryx.config`).',
-        },
-        {
-          type: 'code',
-          lang: 'typescript',
-          code: "// codemods/v2-rename-prop.ts\nimport {createCodemod} from '@astryxdesign/cli/codemod';\n\nexport default createCodemod({\n  // version, description, transform, ...\n});",
-        },
-        {
-          type: 'prose',
-          text: 'The codemod helpers live in `@astryxdesign/cli/codemod`, not `@astryxdesign/core/authoring` like the doc, integration, and template helpers. Consumers can also run their own post-codemod hooks, such as a reinstall or rebuild, via `hooks.postCodemod` in their `astryx.config`.',
-        },
-      ],
-    },
-    {
-      title: 'How It Works',
-      category: 'guide',
-      content: [
-        {
-          type: 'prose',
-          text: 'Every CLI command loads the consumer\u2019s `astryx.config`, resolves each listed integration\u2019s manifest from `node_modules`, and discovers its contributions. Everything is validated against one strict schema at the load boundary. The `create*` helpers do not validate. They are identity functions whose value is their TypeScript surface, so validation happens when the CLI loads the file, not when you author it.',
-        },
-        {
-          type: 'prose',
-          text: 'Discovery is resilient. A broken or misconfigured integration is skipped with a single non-blocking warning on stderr instead of crashing the CLI, and it never corrupts a `--json` stdout envelope. Everyday commands keep working with the remaining valid contributions.',
-        },
-        {
-          type: 'prose',
-          text: 'To inspect problems, run `astryx validate-integration <package>` for a detailed report on one package, or `astryx doctor` for an overall health check of the setup.',
-        },
-      ],
-    },
-  ],
-};

package/scripts/postinstall.mjs

@@ -1,74 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file @astryxdesign/cli postinstall — nudge to run `astryx init`.
- *
- * Enforcement layer 2 of making `astryx init` foolproof: when the CLI is
- * installed as a project dependency and the project hasn't run init yet, print a
- * one-line next-step so agents/humans discover it.
- *
- * Reuses the ONE setup check (isAstryxInitialized from agent-docs.mjs — a
- * dep-free import chain, safe at install time). Non-interactive, never fails the
- * install, and stays quiet:
- *   - in the monorepo/source build (not under node_modules),
- *   - during npx's transient fetch (npx runs the bin — likely `init` — right
- *     after, so nudging here would double up), and
- *   - once setup is already done.
- */
-
-import {fileURLToPath, pathToFileURL} from 'node:url';
-
-const HERE = fileURLToPath(import.meta.url);
-
-/**
- * Pure decision: should the postinstall print the setup nudge? Split out so the
- * matrix is unit-testable without an actual npm install.
- *
- * @param {object} opts
- * @param {string} opts.scriptPath - Absolute path of this script (location tells
- *   us dependency vs monorepo vs npx-cache).
- * @param {string} [opts.npmCommand] - process.env.npm_command ('install', 'exec', …).
- * @param {boolean} opts.isSetUp - Whether the project already ran init.
- * @returns {boolean}
- */
-export function shouldNudge({scriptPath, npmCommand, isSetUp} = {}) {
-  if (!scriptPath || !scriptPath.includes('node_modules')) return false; // monorepo/source build
-  if (scriptPath.includes('_npx') || npmCommand === 'exec') return false; // npx transient — init runs next
-  if (isSetUp) return false; // already set up — stay quiet
-  return true;
-}
-
-/** @param {string} root @returns {Promise<boolean>} */
-async function projectIsSetUp(root) {
-  try {
-    const {isAstryxInitialized} = await import('../src/commands/agent-docs.mjs');
-    return isAstryxInitialized(root);
-  } catch {
-    return false; // best-effort — if the check can't load, fall through and nudge
-  }
-}
-
-async function main() {
-  const root = process.env.INIT_CWD || process.cwd();
-  const nudge = shouldNudge({
-    scriptPath: HERE,
-    npmCommand: process.env.npm_command,
-    isSetUp: await projectIsSetUp(root),
-  });
-  if (nudge) {
-    // Scoped package form (`@astryxdesign/cli`) — always resolves to us. Bare
-    // `npx astryx` would fetch an unrelated look-alike package (see PR #4151).
-    // After that lands, switch this to its getCliInvocation() single source of truth.
-    process.stdout.write(
-      '\nNext step: run `npx @astryxdesign/cli init` to finish setup and install the Astryx agent prompt.\n\n',
-    );
-  }
-}
-
-// Run only when executed directly (`node scripts/postinstall.mjs`), never when
-// imported by tests.
-if (import.meta.url === pathToFileURL(process.argv[1] || '').href) {
-  main()
-    .catch(() => {})
-    .finally(() => process.exit(0));
-}

package/src/commands/cli-postinstall.test.mjs

@@ -1,42 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file Guardrail tests for the @astryxdesign/cli postinstall nudge (layer 2).
- *
- * Tests the pure decision matrix (shouldNudge) — nudges for a real dependency
- * install when not set up; stays quiet in the monorepo/source, during npx's
- * transient fetch (path _npx or npm_command=exec), and once set up.
- */
-
-import {describe, it, expect} from 'vitest';
-import {shouldNudge} from '../../scripts/postinstall.mjs';
-
-const DEP = '/proj/node_modules/@astryxdesign/cli/scripts/postinstall.mjs'; // real dep install
-const NPX = '/Users/x/.npm/_npx/a1b2/node_modules/@astryxdesign/cli/scripts/postinstall.mjs';
-const REPO = '/repo/packages/cli/scripts/postinstall.mjs'; // monorepo/source
-
-describe('cli postinstall — shouldNudge', () => {
-  it('nudges for a real dependency install when not set up', () => {
-    expect(shouldNudge({scriptPath: DEP, npmCommand: 'install', isSetUp: false})).toBe(true);
-  });
-
-  it('quiet in the monorepo/source (not under node_modules)', () => {
-    expect(shouldNudge({scriptPath: REPO, npmCommand: 'install', isSetUp: false})).toBe(false);
-  });
-
-  it('quiet during npx transient install (path contains _npx)', () => {
-    expect(shouldNudge({scriptPath: NPX, npmCommand: 'install', isSetUp: false})).toBe(false);
-  });
-
-  it('quiet during npx (npm_command=exec) — avoids double-nudge before init', () => {
-    expect(shouldNudge({scriptPath: DEP, npmCommand: 'exec', isSetUp: false})).toBe(false);
-  });
-
-  it('quiet once the project is already set up', () => {
-    expect(shouldNudge({scriptPath: DEP, npmCommand: 'install', isSetUp: true})).toBe(false);
-  });
-
-  it('quiet with no script path (defensive)', () => {
-    expect(shouldNudge({})).toBe(false);
-  });
-});

package/src/commands/setup-nudge.test.mjs

@@ -1,108 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file Guardrail tests — the centralized setup check + enforcement layer 3.
- *
- * Unit: isAstryxInitialized() detects the Astryx marker across EVERY agent-doc
- * location (including the previously-missed Hermes files) and legacy XDS blocks.
- *
- * Integration: the CLI nudges (stderr) before a command when the project hasn't
- * run init — INCLUDING in --json mode (agents pass --json, and stderr never
- * corrupts the stdout JSON envelope). Quiet once set up, outside a project, and
- * for the installer command itself.
- */
-
-import {describe, it, expect, beforeEach, afterEach} from 'vitest';
-import {spawnSync} from 'node:child_process';
-import * as fs from 'node:fs';
-import * as path from 'node:path';
-import * as os from 'node:os';
-import {fileURLToPath} from 'node:url';
-import {isAstryxInitialized} from './agent-docs.mjs';
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const CLI = path.resolve(__dirname, '..', '..', 'bin', 'astryx.mjs');
-const MARKER = '<!-- ASTRYX:START -->';
-const NUDGE = /finish setup and install the Astryx agent prompt/;
-
-let tmp;
-beforeEach(() => {
-  tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'astryx-setup-nudge-'));
-});
-afterEach(() => {
-  fs.rmSync(tmp, {recursive: true, force: true});
-});
-
-function write(rel, body) {
-  const p = path.join(tmp, rel);
-  fs.mkdirSync(path.dirname(p), {recursive: true});
-  fs.writeFileSync(p, body);
-}
-
-function run(args, cwd = tmp) {
-  return spawnSync(process.execPath, [CLI, ...args], {
-    cwd,
-    encoding: 'utf8',
-    timeout: 30_000,
-    env: {...process.env, FORCE_COLOR: '0'},
-  });
-}
-
-describe('isAstryxInitialized — centralized setup check (one place)', () => {
-  it('is false in an empty project', () => {
-    expect(isAstryxInitialized(tmp)).toBe(false);
-  });
-
-  // Covers EVERY location a preset can write — .hermes.md/HERMES.md were the gap.
-  it.each(['AGENTS.md', 'CLAUDE.md', '.claude/CLAUDE.md', '.cursorrules', '.hermes.md', 'HERMES.md'])(
-    'detects the marker in %s',
-    file => {
-      write(file, `# doc\n${MARKER}\nbody`);
-      expect(isAstryxInitialized(tmp)).toBe(true);
-    },
-  );
-
-  it('detects the legacy XDS marker for back-compat', () => {
-    write('AGENTS.md', '<!-- XDS:START -->');
-    expect(isAstryxInitialized(tmp)).toBe(true);
-  });
-
-  it('is false when a doc file exists WITHOUT the marker', () => {
-    write('AGENTS.md', 'project notes, no astryx block');
-    expect(isAstryxInitialized(tmp)).toBe(false);
-  });
-});
-
-describe('enforcement layer 3 — per-command setup nudge', () => {
-  const asProject = () => write('package.json', '{"name":"t"}');
-
-  it('nudges on stderr after a command when not set up', () => {
-    asProject();
-    const r = run(['docs', 'tokens']);
-    expect(r.stderr).toMatch(NUDGE);
-  });
-
-  it('is suppressed in --json (machine mode stays clean), stdout valid JSON', () => {
-    asProject();
-    const r = run(['--json', 'docs', 'tokens']);
-    // --json is machine output with a clean stdout+stderr contract; the human
-    // nudge must NOT leak into it (json-shim: error envelopes have empty stderr).
-    expect(r.stderr).not.toMatch(NUDGE);
-    expect(() => JSON.parse(r.stdout)).not.toThrow();
-  });
-
-  it('is quiet once set up (marker present)', () => {
-    asProject();
-    write('AGENTS.md', MARKER);
-    expect(run(['docs', 'tokens']).stderr).not.toMatch(NUDGE);
-  });
-
-  it('is quiet outside a project (no package.json)', () => {
-    expect(run(['docs', 'tokens']).stderr).not.toMatch(NUDGE);
-  });
-
-  it('does not nudge for the installer command itself', () => {
-    asProject();
-    expect(run(['init']).stderr).not.toMatch(NUDGE);
-  });
-});

package/src/lib/term-log.mjs

@@ -1,48 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file Minimal non-interactive terminal logger.
- *
- * @input  message strings from CLI commands/codemods
- * @output plain lines on stdout via humanLog (suppressed in --json mode)
- * @position src/lib — shared output helper, no side effects on import
- *
- * The CLI is fully non-interactive: it never prompts, so it only needs plain,
- * unbuffered output. This provides the *output-only* surface (`log.*`, `intro`,
- * `outro`) the CLI needs, so it has no dependency on any prompt library.
- *
- * All output is routed through `humanLog`, the CLI's stdout-discipline
- * primitive, which is a no-op in `--json` mode — so these human logs can never
- * corrupt a JSON envelope.
- *
- * Call sites use it as `import * as p from '../lib/term-log.mjs'` and call
- * `p.log.info(...)`, `p.intro(...)`, `p.outro(...)`.
- */
-
-import {humanLog} from './json.mjs';
-
-const toStr = (msg) => (msg === undefined || msg === null ? '' : String(msg));
-
-/**
- * Human-facing log surface (the small `log` API the CLI uses). All lines go to
- * stdout via humanLog; the level prefixes are cosmetic. `--json` mode suppresses
- * every one of these, keeping machine-readable stdout clean.
- */
-export const log = {
-  message: (msg) => humanLog(toStr(msg)),
-  info: (msg) => humanLog(toStr(msg)),
-  step: (msg) => humanLog(toStr(msg)),
-  success: (msg) => humanLog(`✓ ${toStr(msg)}`),
-  warn: (msg) => humanLog(`⚠ ${toStr(msg)}`),
-  error: (msg) => humanLog(`✗ ${toStr(msg)}`),
-};
-
-/** Banner printed at the start of a multi-step command. */
-export function intro(title) {
-  humanLog(`\n${toStr(title)}`);
-}
-
-/** Footer printed at the end of a multi-step command. */
-export function outro(message) {
-  humanLog(`${toStr(message)}\n`);
-}