@webjsdev/cli 0.10.39 → 0.10.41

package/bin/webjs.js

@@ -4,7 +4,6 @@ import { spawn } from 'node:child_process';
 import { fileURLToPath } from 'node:url';
 import { resolveBin } from '../lib/resolve-bin.js';
 import { dbGenerateTtyHint } from '../lib/db-hints.js';
-import { DESIGN_REMINDER, hasUiLayout } from '../lib/design-bar.js';
 import { checkNodeInline, nodeInlineMessage } from '../lib/node-preflight.js';
 import { loadAppEnv, resolvePort } from '../lib/port.js';
 import { planDevSupervisor } from '../lib/dev-supervisor.js';
@@ -47,7 +46,7 @@ const USAGE = `webjs commands:
                                                   (--no-hot: run in-process, no hot-reload supervisor)
   webjs start [--port 8080]                       Start production server (serves source directly, no build step)
   webjs test  [--server|--browser]                 Run server + browser tests
-  webjs check [--json] [--clear-placeholders]     Run correctness checks (--json emits structured violations; --clear-placeholders strips scaffold markers)
+  webjs check [--json]                            Run correctness checks (--json emits structured violations)
   webjs mcp                                       Start the read-only MCP server (routes / actions / components / check)
   webjs doctor                                    Verify project health (Node, tsconfig, env, vendor pins, importmap coherence, @webjsdev versions, git hook, page/layout elision)
   webjs types                                     Generate .webjs/routes.d.ts (typed Route union + per-route params)
@@ -412,37 +411,12 @@ async function main() {
     case 'check': {
       const { checkConventions, RULES } = await import('@webjsdev/server/check');
 
-      // --clear-placeholders: acknowledge the whole scaffold gallery in one
-      // command (strip the marker comment lines, keep the demo code), instead of
-      // one hand-edit per file. The gate then reflects only real violations.
-      if (rest.includes('--clear-placeholders')) {
-        const { clearPlaceholders } = await import('../lib/clear-placeholders.js');
-        const report = clearPlaceholders(process.cwd());
-        const total = report.reduce((n, r) => n + r.markers, 0);
-        if (report.length === 0) {
-          console.log('webjs check: no scaffold-placeholder markers found (nothing to clear).');
-        } else {
-          console.log(`webjs check: cleared ${total} scaffold-placeholder marker(s) across ${report.length} file(s):`);
-          for (const r of report) console.log(`  ${r.file}`);
-          console.log('\nThe demo code is kept. Delete any gallery route/module you do not want, then re-run `webjs check`.');
-          // Re-surface the design bar the cleared markers carried. The
-          // layout/home marker was the just-in-time "adapt this chrome" reminder;
-          // stripping it silently is how an app ends up shipping the scaffold
-          // shell. Print the bar so clearing the markers cannot quietly drop it,
-          // but only for a UI app (the api template has no layout / no chrome).
-          if (hasUiLayout(process.cwd())) console.log(DESIGN_REMINDER);
-        }
-        break;
-      }
-
       if (rest.includes('--rules')) {
         console.log('webjs check, correctness rules:');
         console.log('  Every rule catches code that is wrong to ship: a crash, a');
-        console.log('  security leak, a build/type-strip failure, or (the one');
-        console.log('  sentinel-based rule, no-scaffold-placeholder) unreplaced');
-        console.log('  scaffold example content. They always run. Project');
-        console.log('  conventions (layout, style, process) are guidance in');
-        console.log('  CONVENTIONS.md, not rules here.\n');
+        console.log('  security leak, or a build/type-strip failure. They always');
+        console.log('  run. Project conventions (layout, style, process) are');
+        console.log('  guidance in CONVENTIONS.md, not rules here.\n');
         for (const r of RULES) {
           console.log(`  ${r.name.padEnd(30)} ${r.description}`);
         }
@@ -469,28 +443,12 @@ async function main() {
         console.log('webjs check: all checks pass ✓');
       } else {
         console.log(`webjs check: ${violations.length} violation(s) found\n`);
-        // A fresh scaffold trips no-scaffold-placeholder on every unadapted demo
-        // file at once. Printing an identical block per file drowns the real
-        // feature violations, so collapse the sentinel to ONE grouped summary
-        // (with a one-command clear) and print every other rule per-violation.
-        // The returned violations and --json are unchanged; only this human
-        // printout groups, so agents/tools still see one entry per file.
-        const PLACEHOLDER = 'no-scaffold-placeholder';
-        const placeholders = violations.filter((v) => v.rule === PLACEHOLDER);
         for (const v of violations) {
-          if (v.rule === PLACEHOLDER) continue;
           console.log(`  ✗ [${v.rule}] ${v.file}`);
           console.log(`    ${v.message}`);
           if (v.fix) console.log(`    Fix: ${v.fix}`);
           console.log();
         }
-        if (placeholders.length > 0) {
-          console.log(`  ✗ [${PLACEHOLDER}] ${placeholders.length} file(s) still carry scaffold example content:`);
-          for (const v of placeholders) console.log(`      ${v.file}`);
-          console.log('    Fix: adapt or delete each file, then remove its marker comment line.');
-          console.log('         Keeping the gallery? Run `webjs check --clear-placeholders` to clear all markers at once.');
-          console.log();
-        }
         process.exit(1);
       }
       break;

package/lib/app-tasks.js

@@ -4,15 +4,20 @@ import { join } from 'node:path';
 /**
  * Read the dev/start task orchestration from an app's `package.json` `"webjs"`
  * block (#550). This is what lets `webjs dev` / `webjs start` behave identically
- * to `npm run dev` / `npm run start`: the orchestration (Tailwind's watcher, a
- * `db migrate` before prod boot) moves OUT of `concurrently` + `pre*` npm hooks
+ * to `npm run dev` / `npm run start`: the orchestration (a Tailwind compile, a
+ * `db migrate` before boot) moves OUT of `concurrently` + `pre*` npm hooks
  * and INTO the framework primitive, so a bare `webjs dev` is not a degraded run.
  *
+ * Reads only `before` / `parallel` here (the CLI-run tasks). The dev-only
+ * `regenerate` key (#967, on-request rebuilds of a stale served output like
+ * `public/tailwind.css`) is read by the SERVER, not this reader, so the scaffold
+ * keeps its static CSS fresh WITHOUT a `--watch` under `parallel`.
+ *
  * Shape:
  *   "webjs": {
  *     "dev":   {
- *       "before":   ["webjs db migrate"],
- *       "parallel": ["tailwindcss -i ./public/input.css -o ./public/tailwind.css --watch"]
+ *       "before":   ["webjs db migrate", "tailwindcss -i ./public/input.css -o ./public/tailwind.css --minify"]
+ *       // dev.regenerate keeps the CSS fresh on request (server-read, see dev-regenerate.js)
  *     },
  *     "start": { "before": ["webjs db migrate"] }
  *   }
@@ -20,9 +25,10 @@ import { join } from 'node:path';
  * `before` commands run sequentially to completion BEFORE the server boots (the
  * old `predev` / `prestart` hooks: a one-shot `webjs db migrate`).
  * `parallel` (dev only) commands run as long-lived child processes ALONGSIDE the
- * server (the old `concurrently` watchers: Tailwind). Returns normalized arrays
- * (never undefined) so callers iterate without guards, and a missing/empty
- * config yields empty arrays so a plain app runs `webjs dev`/`start` unchanged.
+ * server (the old `concurrently` watchers), for a genuinely long-lived side
+ * process. Returns normalized arrays (never undefined) so callers iterate
+ * without guards, and a missing/empty config yields empty arrays so a plain app
+ * runs `webjs dev`/`start` unchanged.
  *
  * Pure (reads one file, never spawns / prints / exits) so it is unit-testable
  * without a process, matching `lib/port.js` and `lib/dev-supervisor.js`.