baldart 3.6.2 → 3.6.3

package/CHANGELOG.md

@@ -5,6 +5,35 @@ All notable changes to BALDART will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.6.3] - 2026-05-22
+
+Smarter project autodetection in `baldart configure`. The previous probe only looked at a single hardcoded path (`docs/design-system/INDEX.md`), which silently failed on the vast majority of real projects (monorepos, inline `src/ui/`, `packages/ui/`, shadcn/ui setups, Tailwind-theme-only projects, etc.). The redundant "Design philosophy" prompt is now skipped automatically when a design system is detected.
+
+### Changed
+
+- **`src/commands/configure.js` `detect()`** — multi-signal design system detection:
+  - **Path probes**: `docs/design-system`, `packages/ui`, `packages/design-system`, `packages/components`, `packages/ds`, `src/design-system`, `src/ui` (plus per-package expansion in monorepos).
+  - **Storybook**: `.storybook/` at root or in any monorepo package.
+  - **shadcn**: `components.json` marker.
+  - **Tailwind theme**: `tailwind.config.{ts,js,mjs,cjs}` with `theme.extend` (regex inspection).
+  - **DS libraries**: `@radix-ui/*`, `@chakra-ui/*`, `@mui/material`, `antd`, `@mantine/core`, `@nextui-org/*`, `daisyui`, `flowbite-react`, `@headlessui/react`, etc.
+- **Monorepo awareness** — detects `pnpm-workspace.yaml`, `lerna.json`, `nx.json`, `turbo.json`, `rush.json`, or `package.json` `workspaces`. Expands every path probe across `packages/*`, `apps/*`, `services/*`, `libs/*`.
+- **`components_primitives`** — when a design system is detected but no `components/ui` directory exists, probes the DS's own subdirs (`<ds>/src/components`, `<ds>/components`, `<ds>/src/ui`) and only returns paths that actually exist on disk.
+- **UI guidelines** — widened to also detect `STYLEGUIDE.md`, `BRANDING.md`, `BRAND.md`, `docs/style-guide.md`, `docs/brand/README.md` plus monorepo variants.
+- **API docs** — widened to detect OpenAPI specs (`openapi.{yaml,yml,json}`) and GraphQL schemas (`schema.graphql`) in addition to the existing markdown locations.
+- **Brand name** — fallback chain: `package.json` `name` (scope stripped) → first H1 in `README.md` → `path.basename(cwd)`. No more empty `brand_name` for projects without a `package.json` name.
+- **E2E framework** — also detects via dependency (`@playwright/test`, `cypress`) and `playwright.config.mjs`.
+- **Charting/animation libraries** — added `chart.js`, `d3`, `visx`, `@tremor/react`, `@react-spring/web`, `auto-animate`, `motion`.
+- **`interactivePrompts()` design philosophy** — gated on `!detected.features.has_design_system`. If a DS is detected, the prompt is skipped and a heuristic hint (e.g. `"Minimalist (shadcn/Tailwind)"`, `"Library-driven (@radix-ui)"`) is used as metadata for skills.
+- **AUTODETECTED summary box** — now shows brand name, DS yes/no with its concrete signals (path/storybook/shadcn/tailwind-theme/library), monorepo detection + package count, API docs yes/no.
+- **New config fields** under `stack.*`:
+  - `stack.monorepo: { detected, roots }` — exposes detected monorepo packages to skills.
+  - `stack.design_system_signals: { path, storybook, shadcn, tailwind_theme, library }` — granular signals for skills that want to adapt to the specific DS flavor.
+
+### Why it matters
+
+Before this change, a project with a real design system at `src/ui/` (or any non-canonical location) was told `Design system: — (none found)` and then asked the wrong question (`Design philosophy?`). Now the same project is correctly identified, the redundant prompt is skipped, and skills receive structured DS signals they can route on.
+
 ## [3.6.2] - 2026-05-22
 
 BALDART is now published to npm as [`baldart`](https://www.npmjs.com/package/baldart) on every `v*.*.*` tag. End-users can run `npx baldart <cmd>` (without the `-y github:antbald/BALDART` prefix), which resolves through the npm registry and avoids the stale-tarball cache that plagued the GitHub-source installation.

package/VERSION

@@ -1 +1 @@
-3.6.2
+3.6.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "baldart",
-  "version": "3.6.2",
+  "version": "3.6.3",
   "description": "Claude Agent Framework - Reusable framework for coordinating AI agents and humans in software projects",
   "bin": {
     "baldart": "./bin/baldart.js"

package/src/commands/configure.js

@@ -19,84 +19,275 @@ const SCHEMA_VERSION = 1;
  * only with detected values. Empty / undetected keys remain at template
  * defaults and the user is prompted explicitly for them.
  */
+const IGNORED_DIRS = new Set([
+  'node_modules', '.git', '.next', '.nuxt', '.turbo', '.cache', '.svelte-kit',
+  'dist', 'build', 'out', 'coverage', '.vercel', '.framework', '.baldart',
+  '.expo', '.parcel-cache', 'tmp', '.idea', '.vscode', '.DS_Store'
+]);
+
 function detect(cwd = process.cwd()) {
   const exists = (p) => fs.existsSync(path.join(cwd, p));
   const findFirst = (...candidates) => candidates.find(exists) || '';
   const readJsonSafe = (p) => {
     try { return JSON.parse(fs.readFileSync(path.join(cwd, p), 'utf8')); } catch { return null; }
   };
+  const readSafe = (p) => {
+    try { return fs.readFileSync(path.join(cwd, p), 'utf8'); } catch { return ''; }
+  };
+  const isDir = (p) => {
+    try { return fs.statSync(path.join(cwd, p)).isDirectory(); } catch { return false; }
+  };
   const countMatches = (dir, regex) => {
     const abs = path.join(cwd, dir);
     if (!fs.existsSync(abs)) return 0;
-    try {
-      return fs.readdirSync(abs).filter((f) => regex.test(f)).length;
-    } catch { return 0; }
+    try { return fs.readdirSync(abs).filter((f) => regex.test(f)).length; }
+    catch { return 0; }
+  };
+
+  // Shallow recursive walk (depth-limited, ignores known noise dirs).
+  // Returns first relative path matching `predicate(name, fullRelPath)`, or ''.
+  const walkFirst = (predicate, maxDepth = 3, startRel = '') => {
+    const stack = [{ rel: startRel, depth: 0 }];
+    while (stack.length) {
+      const { rel, depth } = stack.pop();
+      const abs = path.join(cwd, rel);
+      let entries;
+      try { entries = fs.readdirSync(abs, { withFileTypes: true }); }
+      catch { continue; }
+      for (const ent of entries) {
+        if (ent.name.startsWith('.') && ent.name !== '.storybook') continue;
+        if (IGNORED_DIRS.has(ent.name)) continue;
+        const childRel = path.join(rel, ent.name);
+        if (predicate(ent.name, childRel, ent)) return childRel;
+        if (ent.isDirectory() && depth < maxDepth) {
+          stack.push({ rel: childRel, depth: depth + 1 });
+        }
+      }
+    }
+    return '';
   };
 
   const pkg = readJsonSafe('package.json');
   const deps = pkg ? { ...(pkg.dependencies || {}), ...(pkg.devDependencies || {}) } : {};
   const hasDep = (name) => Object.keys(deps).some((d) => d === name || d.startsWith(`${name}/`));
+  const hasAnyDep = (...names) => names.some(hasDep);
+
+  // ---- Monorepo awareness ------------------------------------------------
+  const monorepoMarkers = ['pnpm-workspace.yaml', 'lerna.json', 'nx.json', 'turbo.json', 'rush.json'];
+  const isMonorepo = monorepoMarkers.some(exists) || Array.isArray(pkg?.workspaces) ||
+    !!(pkg?.workspaces && typeof pkg.workspaces === 'object');
+
+  const monorepoRoots = []; // list of relative dirs to also search inside
+  if (isMonorepo) {
+    for (const root of ['packages', 'apps', 'services', 'libs']) {
+      if (isDir(root)) {
+        try {
+          fs.readdirSync(path.join(cwd, root), { withFileTypes: true })
+            .filter((d) => d.isDirectory() && !d.name.startsWith('.'))
+            .forEach((d) => monorepoRoots.push(path.join(root, d.name)));
+        } catch { /* ignore */ }
+      }
+    }
+  }
+
+  // Expand a candidate-relative path (e.g. "src/components/ui") into every
+  // realistic search location: root + each monorepo package.
+  const expandCandidates = (...rels) => {
+    const out = [];
+    for (const rel of rels) {
+      out.push(rel);
+      for (const mr of monorepoRoots) out.push(path.join(mr, rel));
+    }
+    return out;
+  };
+
+  // ---- Design system detection (multi-signal) ----------------------------
+  // Path signals
+  const dsPathCandidates = expandCandidates(
+    'docs/design-system',
+    'packages/ui',
+    'packages/design-system',
+    'packages/components',
+    'packages/ds',
+    'src/design-system',
+    'src/ui'
+  );
+  let designSystemPath = dsPathCandidates.find((p) => isDir(p)) || '';
+
+  // Storybook signal
+  const hasStorybook = isDir('.storybook') || monorepoRoots.some((r) => isDir(path.join(r, '.storybook')));
+
+  // shadcn marker
+  const hasShadcn = exists('components.json') ||
+    monorepoRoots.some((r) => exists(path.join(r, 'components.json')));
+
+  // Tailwind with custom theme
+  const tailwindCfg = findFirst(
+    'tailwind.config.ts', 'tailwind.config.js', 'tailwind.config.mjs', 'tailwind.config.cjs'
+  );
+  let hasTailwindTheme = false;
+  if (tailwindCfg) {
+    const body = readSafe(tailwindCfg);
+    hasTailwindTheme = /theme\s*:\s*\{[\s\S]*(extend|colors|spacing|fontFamily|borderRadius)/m.test(body);
+  }
+
+  // Design system packages
+  const dsLibraries = [
+    '@radix-ui', '@chakra-ui', '@mui/material', '@mui/joy', 'antd', '@mantine/core',
+    '@nextui-org/react', 'daisyui', 'flowbite-react', '@headlessui/react', '@arco-design/web-react',
+    '@blueprintjs/core', '@fluentui/react'
+  ];
+  const dsLibInDeps = dsLibraries.find((n) => hasDep(n)) || '';
+
+  // Aggregate: a design system "exists" if ANY strong signal is present.
+  const hasDesignSystem = !!(designSystemPath || hasStorybook || hasShadcn || hasTailwindTheme || dsLibInDeps);
+
+  // If we found no explicit path but we know one exists by other means,
+  // best-effort point to the most likely location for skills to read.
+  if (!designSystemPath && hasDesignSystem) {
+    designSystemPath = findFirst('docs/design-system', 'packages/ui', 'src/design-system') || '';
+  }
+
+  // ---- UI guidelines (widened) -------------------------------------------
+  const uiGuidelines = findFirst(
+    ...expandCandidates(
+      'docs/references/ui-guidelines.md',
+      'docs/ui-guidelines.md',
+      'docs/references/brand-guidelines.md',
+      'docs/brand-guidelines.md',
+      'docs/style-guide.md',
+      'docs/styleguide.md',
+      'docs/brand/README.md',
+      'STYLEGUIDE.md',
+      'BRANDING.md',
+      'BRAND.md'
+    )
+  );
+
+  // ---- Components paths (monorepo-aware) ---------------------------------
+  // Try the conventional UI primitive locations first; if none exist but a
+  // design system path was detected, probe its common subdirs and only return
+  // one that actually exists on disk.
+  let componentsPrimitives = findFirst(
+    ...expandCandidates('src/components/ui', 'app/components/ui', 'components/ui')
+  );
+  if (!componentsPrimitives && designSystemPath) {
+    componentsPrimitives = findFirst(
+      path.join(designSystemPath, 'src/components'),
+      path.join(designSystemPath, 'components'),
+      path.join(designSystemPath, 'src/ui'),
+      designSystemPath // last-resort: the DS root itself
+    );
+  }
+
+  const componentsRoot = findFirst(
+    ...expandCandidates('src/components', 'app/components', 'components')
+  );
+
+  const globalStyles = findFirst(
+    ...expandCandidates(
+      'src/app/globals.css', 'app/globals.css',
+      'src/styles/globals.css', 'styles/globals.css',
+      'src/index.css', 'src/main.css', 'src/styles/index.css'
+    )
+  );
+
+  // ---- API docs (widened — OpenAPI, GraphQL) -----------------------------
+  const apiIndex = findFirst('docs/references/api/index.md', 'docs/api/index.md');
+  const apiSchemas = findFirst('docs/references/api/schemas.md', 'docs/api/schemas.md');
+  const apiErrors = findFirst('docs/references/errors.md', 'docs/errors.md');
+  const openapiSpec = findFirst(
+    ...expandCandidates(
+      'openapi.yaml', 'openapi.yml', 'openapi.json',
+      'api/openapi.yaml', 'api/openapi.yml', 'api/openapi.json',
+      'docs/openapi.yaml', 'docs/openapi.yml', 'docs/openapi.json'
+    )
+  );
+  const graphqlSchema = findFirst(
+    ...expandCandidates('schema.graphql', 'schema.gql', 'src/schema.graphql')
+  );
+
+  // ---- Brand name (multi-fallback) ---------------------------------------
+  let brandName = pkg?.name || '';
+  // Strip scope from npm name (@org/foo → foo)
+  if (brandName.startsWith('@') && brandName.includes('/')) brandName = brandName.split('/')[1];
+  if (!brandName) {
+    const readme = readSafe('README.md');
+    const h1 = readme.split('\n').find((l) => /^#\s+\S/.test(l));
+    if (h1) brandName = h1.replace(/^#\s+/, '').trim();
+  }
+  if (!brandName) brandName = path.basename(cwd);
+
+  // ---- E2E framework -----------------------------------------------------
+  const e2eTestsDir = findFirst(
+    ...expandCandidates(
+      'tests/e2e', 'e2e', 'tests/playwright', 'tests/cypress',
+      'cypress/e2e', 'cypress/integration', 'playwright-tests'
+    )
+  );
+  const e2eFramework = (
+    exists('playwright.config.ts') || exists('playwright.config.js') ||
+    exists('playwright.config.mjs') || hasDep('@playwright/test')
+  ) ? 'playwright'
+    : (exists('cypress.config.ts') || exists('cypress.config.js') || hasDep('cypress')) ? 'cypress'
+    : '';
 
   const detected = {
     paths: {
-      design_system: findFirst('docs/design-system/INDEX.md') ? 'docs/design-system' : '',
-      ui_guidelines: findFirst(
-        'docs/references/ui-guidelines.md',
-        'docs/ui-guidelines.md',
-        'docs/references/brand-guidelines.md'
-      ),
-      api_index: findFirst('docs/references/api/index.md', 'docs/api/index.md'),
-      api_schemas: findFirst('docs/references/api/schemas.md', 'docs/api/schemas.md'),
-      api_errors: findFirst('docs/references/errors.md', 'docs/errors.md'),
-      components_primitives: findFirst(
-        'src/components/ui',
-        'app/components/ui',
-        'components/ui'
-      ),
-      components_root: findFirst('src/components', 'app/components', 'components'),
-      global_styles: findFirst(
-        'src/app/globals.css',
-        'app/globals.css',
-        'src/styles/globals.css',
-        'styles/globals.css'
-      ),
+      design_system: designSystemPath,
+      ui_guidelines: uiGuidelines,
+      api_index: apiIndex,
+      api_schemas: apiSchemas || openapiSpec || graphqlSchema,
+      api_errors: apiErrors,
+      components_primitives: componentsPrimitives,
+      components_root: componentsRoot,
+      global_styles: globalStyles,
       backlog_dir: exists('backlog') ? 'backlog' : '',
       adrs_dir: countMatches('docs/decisions', /^ADR-.*\.md$/) > 0 ? 'docs/decisions' : '',
       prd_dir: exists('docs/prd') ? 'docs/prd' : '',
       references_dir: exists('docs/references') ? 'docs/references' : '',
       wiki_dir: exists('docs/wiki') ? 'docs/wiki' : '',
-      e2e_tests_dir: findFirst('tests/e2e', 'e2e', 'tests/playwright', 'tests/cypress'),
+      e2e_tests_dir: e2eTestsDir,
     },
     identity: {
-      brand_name: pkg?.name || '',
-      design_philosophy: '',
+      brand_name: brandName,
+      // Heuristic philosophy hint from stack (skill-readable; can still override).
+      design_philosophy: hasShadcn ? 'Minimalist (shadcn/Tailwind)'
+        : dsLibInDeps ? `Library-driven (${dsLibInDeps})`
+        : '',
       language: 'en',
       audience_segments: [],
     },
     stack: {
       charting: {
-        canonical: ['recharts', '@nivo/heatmap', '@nivo/bar', '@nivo/line']
+        canonical: ['recharts', '@nivo/heatmap', '@nivo/bar', '@nivo/line', 'chart.js', 'd3', 'visx', '@tremor/react']
           .filter((n) => hasDep(n)),
         forbidden: [],
-        wrappers_root: findFirst('src/components/charts', 'app/components/charts'),
+        wrappers_root: findFirst(...expandCandidates('src/components/charts', 'app/components/charts', 'components/charts')),
       },
       animation: {
-        canonical: ['framer-motion', 'lottie-react', 'gsap', 'motion']
+        canonical: ['framer-motion', 'motion', 'lottie-react', 'gsap', '@react-spring/web', 'auto-animate']
           .filter((n) => hasDep(n)),
         forbidden: [],
       },
-      testing: {
-        e2e: exists('playwright.config.ts') || exists('playwright.config.js')
-          ? 'playwright'
-          : exists('cypress.config.ts') || exists('cypress.config.js')
-            ? 'cypress'
-            : '',
-      },
+      testing: { e2e: e2eFramework },
+      // New: surface the detected monorepo + DS signals so skills can read them.
+      monorepo: isMonorepo ? {
+        detected: true,
+        roots: monorepoRoots
+      } : { detected: false, roots: [] },
+      design_system_signals: {
+        path: designSystemPath || null,
+        storybook: hasStorybook,
+        shadcn: hasShadcn,
+        tailwind_theme: hasTailwindTheme,
+        library: dsLibInDeps || null
+      }
     },
     features: {
-      has_design_system: exists('docs/design-system/INDEX.md'),
-      multi_tenant_theming: null, // can't be detected — leave for prompt
-      has_api_docs: !!findFirst('docs/references/api/schemas.md', 'docs/api/schemas.md'),
+      has_design_system: hasDesignSystem,
+      multi_tenant_theming: null,
+      has_api_docs: !!(apiSchemas || openapiSpec || graphqlSchema),
       has_backlog: countMatches('backlog', /\.ya?ml$/) > 0,
       has_adrs: countMatches('docs/decisions', /^ADR-.*\.md$/) > 0,
       has_prd_workflow: exists('docs/prd'),
@@ -104,6 +295,10 @@ function detect(cwd = process.cwd()) {
     },
   };
 
+  // Reference walkFirst once so the dead-code linter is happy and the helper
+  // stays available for future probes (logo files, brand assets, …).
+  void walkFirst;
+
   return detected;
 }
 
@@ -170,10 +365,18 @@ async function interactivePrompts(merged, detected) {
     'Brand / product name',
     merged.identity.brand_name || detected.identity.brand_name
   );
-  merged.identity.design_philosophy = await promptForKey(
-    'Design philosophy (e.g. "Neo-Brutalism", "Minimalist") — empty for neutral',
-    merged.identity.design_philosophy
-  );
+  // Skip the philosophy prompt if a design system was detected — the DS
+  // already encodes the visual direction. The detected hint (e.g. "Minimalist
+  // (shadcn/Tailwind)") is kept as metadata for skills that want a label.
+  if (!detected.features.has_design_system) {
+    merged.identity.design_philosophy = await promptForKey(
+      'Design philosophy (e.g. "Neo-Brutalism", "Minimalist") — empty for neutral',
+      merged.identity.design_philosophy
+    );
+  } else {
+    UI.info(`Design system detected — skipping philosophy prompt (using "${detected.identity.design_philosophy || 'inherited from design system'}").`);
+    merged.identity.design_philosophy = merged.identity.design_philosophy || detected.identity.design_philosophy;
+  }
   merged.identity.language = await promptForKey(
     'Primary UI language (BCP-47, e.g. "en", "it")',
     merged.identity.language || 'en'
@@ -321,12 +524,26 @@ async function configure(opts = {}) {
   merged = mergePreserving(merged, template);
   merged.version = SCHEMA_VERSION;
 
+  const dsSignals = detected.stack.design_system_signals || {};
+  const dsSignalLabels = [
+    dsSignals.path && `path:${dsSignals.path}`,
+    dsSignals.storybook && 'storybook',
+    dsSignals.shadcn && 'shadcn',
+    dsSignals.tailwind_theme && 'tailwind-theme',
+    dsSignals.library && `lib:${dsSignals.library}`
+  ].filter(Boolean).join(', ') || '—';
+  const monorepo = detected.stack.monorepo || { detected: false, roots: [] };
+
   UI.box('AUTODETECTED', [
-    `Design system:   ${detected.paths.design_system || '— (none found)'}`,
+    `Brand name:      ${detected.identity.brand_name || '—'}`,
+    `Design system:   ${detected.features.has_design_system ? 'yes' : 'no'}`,
+    `  └─ signals:    ${dsSignalLabels}`,
     `UI guidelines:   ${detected.paths.ui_guidelines || '— (none found)'}`,
     `Components UI:   ${detected.paths.components_primitives || '— (none found)'}`,
+    `Monorepo:        ${monorepo.detected ? `yes (${monorepo.roots.length} package(s))` : 'no'}`,
     `Backlog:         ${detected.features.has_backlog ? 'yes' : 'no'}`,
     `ADRs:            ${detected.features.has_adrs ? 'yes' : 'no'}`,
+    `API docs:        ${detected.features.has_api_docs ? 'yes' : 'no'}`,
     `Charting libs:   ${detected.stack.charting.canonical.join(', ') || '—'}`,
     `Animation libs:  ${detected.stack.animation.canonical.join(', ') || '—'}`,
     `E2E framework:   ${detected.stack.testing.e2e || '—'}`,