@pracht/cli 1.1.5 → 1.2.1

package/{lib/verification.js → src/verification.ts}

@@ -9,6 +9,7 @@ import {
   listFilesRecursively,
   readProjectConfig,
   resolveProjectPath,
+  type ProjectConfig,
 } from "./project.js";
 
 const CONFIG_FILE_NAMES = new Set([
@@ -23,7 +24,30 @@ const CONFIG_FILE_NAMES = new Set([
 const MODULE_SOURCE_RE = /\.(ts|tsx|js|jsx)$/;
 const PAGE_SOURCE_RE = /\.(ts|tsx|js|jsx|md|mdx)$/;
 
-export function runDoctor(root) {
+export interface Check {
+  message: string;
+  status: "ok" | "warning" | "error";
+}
+
+export interface DoctorReport {
+  checks: Check[];
+  configFile: string | null;
+  mode: "manifest" | "pages";
+  ok: boolean;
+}
+
+export interface VerificationReport {
+  changedFiles: string[];
+  checks: Check[];
+  configFile: string | null;
+  frameworkFiles: string[];
+  mode: "manifest" | "pages";
+  ok: boolean;
+  requestedScope: string;
+  scope: string;
+}
+
+export function runDoctor(root: string): DoctorReport {
   const report = runVerification(root);
 
   return {
@@ -34,9 +58,12 @@ export function runDoctor(root) {
   };
 }
 
-export function runVerification(root, options = {}) {
+export function runVerification(
+  root: string,
+  options: { changed?: boolean } = {},
+): VerificationReport {
   const project = readProjectConfig(root);
-  const checks = [];
+  const checks: Check[] = [];
   const packageJsonPath = resolve(project.root, "package.json");
   const configDisplayPath = project.configFile
     ? displayPath(root, project.configFile)
@@ -45,7 +72,7 @@ export function runVerification(root, options = {}) {
 
   collectConfigChecks(project, checks, configDisplayPath);
 
-  let changedInfo = {
+  let changedInfo: { files: string[]; warning: string | null } = {
     files: [],
     warning: null,
   };
@@ -92,7 +119,11 @@ export function runVerification(root, options = {}) {
   };
 }
 
-function collectConfigChecks(project, checks, configDisplayPath) {
+function collectConfigChecks(
+  project: ProjectConfig,
+  checks: Check[],
+  configDisplayPath: string,
+): void {
   if (!project.configFile) {
     checks.push(createCheck("error", "Missing vite config."));
   } else {
@@ -106,7 +137,11 @@ function collectConfigChecks(project, checks, configDisplayPath) {
   }
 }
 
-function collectManifestVerification(project, checks, { changedFiles, scope }) {
+function collectManifestVerification(
+  project: ProjectConfig,
+  checks: Check[],
+  { changedFiles, scope }: { changedFiles: string[]; scope: string },
+): void {
   const manifestPath = resolveProjectPath(project.root, project.appFile);
   if (!existsSync(manifestPath)) {
     checks.push(createCheck("error", `App manifest is missing at ${project.appFile}.`));
@@ -187,12 +222,12 @@ function collectManifestVerification(project, checks, { changedFiles, scope }) {
 }
 
 function collectChangedManifestModuleChecks(
-  project,
-  checks,
-  manifestPath,
-  relativeModules,
-  changedFiles,
-) {
+  project: ProjectConfig,
+  checks: Check[],
+  manifestPath: string,
+  relativeModules: string[],
+  changedFiles: string[],
+): void {
   const manifestDir = dirname(manifestPath);
   const referencedModules = new Set(relativeModules.map(normalizePath));
   const moduleDirectories = [
@@ -241,7 +276,11 @@ function collectChangedManifestModuleChecks(
   }
 }
 
-function collectPagesVerification(project, checks, { changedFiles, scope }) {
+function collectPagesVerification(
+  project: ProjectConfig,
+  checks: Check[],
+  { changedFiles, scope }: { changedFiles: string[]; scope: string },
+): void {
   const pagesDir = resolveProjectPath(project.root, project.pagesDir);
   if (!existsSync(pagesDir)) {
     checks.push(createCheck("error", `Pages directory is missing at ${project.pagesDir}.`));
@@ -250,7 +289,7 @@ function collectPagesVerification(project, checks, { changedFiles, scope }) {
 
   const pages = scanPagesDirectory(pagesDir);
   const routes = pages.filter((page) => page.kind === "route");
-  const duplicates = collectDuplicateRoutePaths(routes).map((entry) => ({
+  const duplicates = collectDuplicateRoutePaths(routes as PagesRoute[]).map((entry) => ({
     ...entry,
     files: entry.files.map((file) => displayPath(project.root, file)),
   }));
@@ -298,7 +337,12 @@ function collectPagesVerification(project, checks, { changedFiles, scope }) {
   }
 }
 
-function collectChangedPagesChecks(project, checks, pagesDir, changedFiles) {
+function collectChangedPagesChecks(
+  project: ProjectConfig,
+  checks: Check[],
+  pagesDir: string,
+  changedFiles: string[],
+): void {
   for (const file of changedFiles) {
     if (!isWithinDirectory(file, pagesDir)) continue;
     if (!PAGE_SOURCE_RE.test(file)) continue;
@@ -344,7 +388,11 @@ function collectChangedPagesChecks(project, checks, pagesDir, changedFiles) {
   }
 }
 
-function collectApiVerification(project, checks, { changedFiles, scope }) {
+function collectApiVerification(
+  project: ProjectConfig,
+  checks: Check[],
+  { changedFiles, scope }: { changedFiles: string[]; scope: string },
+): void {
   const apiDir = resolveProjectPath(project.root, project.apiDir);
   const changedApiFiles = changedFiles.filter((file) => isWithinDirectory(file, apiDir));
   if (scope === "changed" && changedApiFiles.length === 0) {
@@ -364,7 +412,7 @@ function collectApiVerification(project, checks, { changedFiles, scope }) {
   }
 
   const apiFiles = listFilesRecursively(apiDir).filter((file) => MODULE_SOURCE_RE.test(file));
-  const routeMap = new Map();
+  const routeMap = new Map<string, string[]>();
 
   for (const file of apiFiles) {
     const routePath = resolveApiRoutePath(apiDir, file);
@@ -422,14 +470,18 @@ function collectApiVerification(project, checks, { changedFiles, scope }) {
   }
 }
 
-function collectPackageChecks(project, checks, packageJsonPath) {
+function collectPackageChecks(
+  project: ProjectConfig,
+  checks: Check[],
+  packageJsonPath: string,
+): void {
   if (!existsSync(packageJsonPath)) {
     checks.push(createCheck("warning", "No package.json found in the current app root."));
     return;
   }
 
   const packageJson = JSON.parse(readFileSync(packageJsonPath, "utf-8"));
-  const deps = {
+  const deps: Record<string, string> = {
     ...packageJson.dependencies,
     ...packageJson.devDependencies,
   };
@@ -455,7 +507,7 @@ function collectPackageChecks(project, checks, packageJsonPath) {
   }
 }
 
-function collectChangedFiles(root) {
+function collectChangedFiles(root: string): { files: string[]; warning: string | null } {
   try {
     const repoRoot = execFileSync("git", ["rev-parse", "--show-toplevel"], {
       cwd: root,
@@ -469,7 +521,7 @@ function collectChangedFiles(root) {
       cwd: repoRoot,
       encoding: "utf-8",
     });
-    const files = new Set();
+    const files = new Set<string>();
 
     for (const line of output.split(/\r?\n/).filter(Boolean)) {
       const record = line.slice(3);
@@ -496,7 +548,12 @@ function collectChangedFiles(root) {
   }
 }
 
-function addChangedFile(files, repoRoot, prefix, repoRelativePath) {
+function addChangedFile(
+  files: Set<string>,
+  repoRoot: string,
+  prefix: string,
+  repoRelativePath: string,
+): void {
   if (prefix && !repoRelativePath.startsWith(prefix)) {
     return;
   }
@@ -509,7 +566,11 @@ function addChangedFile(files, repoRoot, prefix, repoRelativePath) {
   files.add(resolve(repoRoot, projectRelativePath));
 }
 
-function filterFrameworkFiles(project, files, packageJsonPath) {
+function filterFrameworkFiles(
+  project: ProjectConfig,
+  files: string[],
+  packageJsonPath: string,
+): string[] {
   const appFile = resolveProjectPath(project.root, project.appFile);
   const routesDir = resolveProjectPath(project.root, project.routesDir);
   const shellsDir = resolveProjectPath(project.root, project.shellsDir);
@@ -532,7 +593,7 @@ function filterFrameworkFiles(project, files, packageJsonPath) {
   });
 }
 
-function requiresFullVerification(project, changedFiles) {
+function requiresFullVerification(project: ProjectConfig, changedFiles: string[]): boolean {
   const packageJsonPath = resolve(project.root, "package.json");
   const appFile = resolveProjectPath(project.root, project.appFile);
 
@@ -545,13 +606,21 @@ function requiresFullVerification(project, changedFiles) {
   });
 }
 
-function scanPagesDirectory(pagesDir) {
+type PagesFile = { file: string; kind: "shell" } | { file: string; kind: "ignored" } | PagesRoute;
+
+interface PagesRoute {
+  file: string;
+  kind: "route";
+  routePath: string;
+}
+
+function scanPagesDirectory(pagesDir: string): PagesFile[] {
   return listFilesRecursively(pagesDir)
     .filter((file) => PAGE_SOURCE_RE.test(file))
     .map((file) => describePagesFile(pagesDir, file));
 }
 
-function describePagesFile(pagesDir, file) {
+function describePagesFile(pagesDir: string, file: string): PagesFile {
   const relativePath = relative(pagesDir, file).replace(/\\/g, "/");
   const routePath = relativePath.replace(/\.(tsx?|jsx?|mdx?)$/, "");
   const name = basename(routePath);
@@ -580,8 +649,8 @@ function describePagesFile(pagesDir, file) {
   };
 }
 
-function collectDuplicateRoutePaths(routes) {
-  const routeMap = new Map();
+function collectDuplicateRoutePaths(routes: PagesRoute[]): { files: string[]; path: string }[] {
+  const routeMap = new Map<string, string[]>();
 
   for (const route of routes) {
     const files = routeMap.get(route.routePath) ?? [];
@@ -594,7 +663,7 @@ function collectDuplicateRoutePaths(routes) {
     .map(([path, files]) => ({ files, path }));
 }
 
-function resolveApiRoutePath(apiDir, file) {
+function resolveApiRoutePath(apiDir: string, file: string): string {
   let relativePath = relative(apiDir, file).replace(/\\/g, "/");
   relativePath = relativePath.replace(/\.(ts|tsx|js|jsx)$/, "");
 
@@ -609,7 +678,7 @@ function resolveApiRoutePath(apiDir, file) {
   return normalizeRoutePath(relativePath ? `/api/${relativePath}` : "/api");
 }
 
-function toModuleSpecifier(fromDir, filePath) {
+function toModuleSpecifier(fromDir: string, filePath: string): string {
   const relativePath = relative(fromDir, filePath).replace(/\\/g, "/");
   if (relativePath.startsWith(".")) {
     return relativePath;
@@ -617,7 +686,7 @@ function toModuleSpecifier(fromDir, filePath) {
   return `./${relativePath}`;
 }
 
-function normalizeRoutePath(path) {
+function normalizeRoutePath(path: string): string {
   if (!path || path === "/") {
     return "/";
   }
@@ -627,15 +696,15 @@ function normalizeRoutePath(path) {
   return collapsed.length > 1 && collapsed.endsWith("/") ? collapsed.slice(0, -1) : collapsed;
 }
 
-function isWithinDirectory(filePath, directoryPath) {
+function isWithinDirectory(filePath: string, directoryPath: string): boolean {
   const relativePath = relative(directoryPath, filePath);
   return relativePath === "" || (!relativePath.startsWith("..") && !relativePath.startsWith("../"));
 }
 
-function normalizePath(value) {
+function normalizePath(value: string): string {
   return value.replace(/\\/g, "/");
 }
 
-function createCheck(status, message) {
+function createCheck(status: Check["status"], message: string): Check {
   return { message, status };
 }

package/tsdown.config.ts

@@ -0,0 +1,8 @@
+import { defineConfig } from "tsdown";
+
+export default defineConfig({
+  clean: true,
+  entry: ["src/index.ts"],
+  format: "esm",
+  external: [/^@pracht\//, /^node:/, "vite", "citty"],
+});

package/lib/build-metadata.js

@@ -1,73 +0,0 @@
-import { existsSync, readFileSync } from "node:fs";
-import { resolve } from "node:path";
-
-const MANIFEST_PATHS = ["dist/client/.vite/manifest.json", "dist/.vite/manifest.json"];
-
-export function readClientBuildAssets(root = process.cwd()) {
-  const manifestPath = MANIFEST_PATHS.map((candidate) => resolve(root, candidate)).find((path) =>
-    existsSync(path),
-  );
-
-  if (!manifestPath) {
-    return {
-      clientEntryUrl: null,
-      cssManifest: {},
-      jsManifest: {},
-    };
-  }
-
-  const rawManifest = readFileSync(manifestPath, "utf-8");
-  const manifest = JSON.parse(rawManifest);
-  const clientEntry = manifest["virtual:pracht/client"];
-
-  function collectTransitiveDeps(key) {
-    const css = new Set();
-    const js = new Set();
-    const visited = new Set();
-
-    function collect(currentKey) {
-      if (visited.has(currentKey)) return;
-      visited.add(currentKey);
-
-      const entry = manifest[currentKey];
-      if (!entry) return;
-
-      for (const cssFile of entry.css ?? []) {
-        css.add(cssFile);
-      }
-
-      js.add(entry.file);
-
-      for (const importedKey of entry.imports ?? []) {
-        collect(importedKey);
-      }
-    }
-
-    collect(key);
-    return {
-      css: [...css],
-      js: [...js],
-    };
-  }
-
-  const cssManifest = {};
-  const jsManifest = {};
-
-  for (const [key, entry] of Object.entries(manifest)) {
-    if (!entry.src) continue;
-
-    const deps = collectTransitiveDeps(key);
-    if (deps.css.length > 0) {
-      cssManifest[key] = deps.css.map((file) => `/${file}`);
-    }
-    if (deps.js.length > 0) {
-      jsManifest[key] = deps.js.map((file) => `/${file}`);
-    }
-  }
-
-  return {
-    clientEntryUrl: clientEntry ? `/${clientEntry.file}` : null,
-    cssManifest,
-    jsManifest,
-  };
-}

package/lib/cli.js

@@ -1,164 +0,0 @@
-import { HTTP_METHODS, VERSION } from "./constants.js";
-
-export function printHelp() {
-  console.log(`pracht ${VERSION}
-
-Usage:
-  pracht dev [port]                 Start development server with HMR
-  pracht build                      Production build (client + server)
-  pracht generate <kind> [flags]    Scaffold framework files
-  pracht verify [--changed] [--json] Fast framework-aware verification
-  pracht inspect [target] [--json]  Inspect resolved app graph
-  pracht doctor [--json]            Validate app wiring
-
-Generate kinds:
-  route       --path /dashboard [--render ssr|spa|ssg|isg] [--shell app] [--middleware auth] [--loader]
-  shell       --name app
-  middleware  --name auth
-  api         --path /health [--methods GET,POST]
-`);
-}
-
-export function printInspectHelp() {
-  console.log(`Usage:
-  pracht inspect [routes|api|build] [--json]
-  pracht inspect --json
-`);
-}
-
-export function printGenerateHelp() {
-  console.log(`Usage:
-  pracht generate route --path /dashboard [--render ssr|spa|ssg|isg] [--shell app] [--middleware auth] [--loader] [--json]
-  pracht generate shell --name app [--json]
-  pracht generate middleware --name auth [--json]
-  pracht generate api --path /health [--methods GET,POST] [--json]
-`);
-}
-
-export function parseFlags(args) {
-  const options = { _: [] };
-
-  for (let index = 0; index < args.length; index += 1) {
-    const token = args[index];
-    if (!token.startsWith("--")) {
-      options._.push(token);
-      continue;
-    }
-
-    if (token.startsWith("--no-")) {
-      options[token.slice(5)] = false;
-      continue;
-    }
-
-    const equalsIndex = token.indexOf("=");
-    if (equalsIndex !== -1) {
-      const key = token.slice(2, equalsIndex);
-      const value = token.slice(equalsIndex + 1);
-      assignOption(options, key, value);
-      continue;
-    }
-
-    const key = token.slice(2);
-    const next = args[index + 1];
-    if (next && !next.startsWith("--")) {
-      assignOption(options, key, next);
-      index += 1;
-      continue;
-    }
-
-    assignOption(options, key, true);
-  }
-
-  return options;
-}
-
-export function requireStringOption(options, key) {
-  const value = requireOptionalString(options, key);
-  if (!value) {
-    throw new Error(`Missing required flag --${key}.`);
-  }
-  return value;
-}
-
-export function requireOptionalString(options, key) {
-  const value = options[key];
-  if (Array.isArray(value)) {
-    return String(value[value.length - 1]);
-  }
-  if (typeof value === "string") {
-    return value;
-  }
-  return null;
-}
-
-export function requireEnumOption(options, key, allowed, fallback) {
-  const value = requireOptionalString(options, key) ?? fallback;
-  if (!allowed.includes(value)) {
-    throw new Error(`Invalid value for --${key}. Expected one of ${allowed.join(", ")}.`);
-  }
-  return value;
-}
-
-export function requirePositiveIntegerOption(options, key, fallback) {
-  const raw = requireOptionalString(options, key);
-  const value = raw == null ? fallback : Number.parseInt(raw, 10);
-  if (!Number.isInteger(value) || value <= 0) {
-    throw new Error(`--${key} must be a positive integer.`);
-  }
-  return value;
-}
-
-export function parseCommaList(value) {
-  if (!value) return [];
-  const values = Array.isArray(value) ? value : [value];
-  return values
-    .flatMap((entry) => String(entry).split(","))
-    .map((entry) => entry.trim())
-    .filter(Boolean);
-}
-
-export function parseApiMethods(value) {
-  const methods = parseCommaList(value);
-  const normalized = methods.length === 0 ? ["GET"] : methods.map((entry) => entry.toUpperCase());
-
-  for (const method of normalized) {
-    if (!HTTP_METHODS.has(method)) {
-      throw new Error(`Unsupported HTTP method "${method}".`);
-    }
-  }
-
-  return [...new Set(normalized)];
-}
-
-export function quote(value) {
-  return JSON.stringify(value);
-}
-
-export function ensureTrailingNewline(value) {
-  return value.endsWith("\n") ? value : `${value}\n`;
-}
-
-export function handleCliError(error, { json }) {
-  const message = error instanceof Error ? error.message : String(error);
-  if (json) {
-    console.error(JSON.stringify({ ok: false, error: message }, null, 2));
-  } else {
-    console.error(message);
-    if (error instanceof Error && error.stack && process.env.DEBUG) {
-      console.error(error.stack);
-    }
-  }
-  process.exit(1);
-}
-
-function assignOption(options, key, value) {
-  if (!(key in options)) {
-    options[key] = value;
-    return;
-  }
-
-  if (!Array.isArray(options[key])) {
-    options[key] = [options[key]];
-  }
-  options[key].push(value);
-}

package/lib/commands/build.js

@@ -1,120 +0,0 @@
-import { cpSync, existsSync, mkdirSync, readdirSync, rmSync, writeFileSync } from "node:fs";
-import { dirname, join, resolve } from "node:path";
-
-import { build as viteBuild } from "vite";
-
-import { readClientBuildAssets } from "../build-metadata.js";
-import { writeVercelBuildOutput } from "../build-shared.js";
-
-export async function buildCommand() {
-  const root = process.cwd();
-
-  console.log("\n  Building client...\n");
-  await viteBuild({
-    root,
-    build: {
-      outDir: "dist",
-      manifest: true,
-      rollupOptions: {
-        input: "virtual:pracht/client",
-      },
-    },
-  });
-
-  console.log("\n  Building server...\n");
-  await viteBuild({
-    root,
-    build: {
-      outDir: "dist/server",
-      ssr: "virtual:pracht/server",
-    },
-  });
-
-  const serverEntry = resolve(root, "dist/server/server.js");
-  let clientDir;
-  if (existsSync(resolve(root, "dist/client/.vite/manifest.json"))) {
-    clientDir = resolve(root, "dist/client");
-  } else {
-    clientDir = resolve(root, "dist/client");
-    const distRoot = resolve(root, "dist");
-    mkdirSync(clientDir, { recursive: true });
-    for (const entry of readdirSync(distRoot)) {
-      if (entry === "server" || entry === "client") continue;
-      const sourcePath = join(distRoot, entry);
-      const destinationPath = join(clientDir, entry);
-      cpSync(sourcePath, destinationPath, { recursive: true });
-      rmSync(sourcePath, { force: true, recursive: true });
-    }
-  }
-
-  if (existsSync(serverEntry)) {
-    const serverMod = await import(serverEntry);
-    const { prerenderApp } = serverMod;
-    const { clientEntryUrl, cssManifest, jsManifest } = readClientBuildAssets(root);
-
-    const { pages, isgManifest } = await prerenderApp({
-      app: serverMod.resolvedApp,
-      clientEntryUrl: clientEntryUrl ?? undefined,
-      cssManifest,
-      jsManifest,
-      registry: serverMod.registry,
-      withISGManifest: true,
-    });
-    const headersManifest = Object.fromEntries(
-      pages.map((page) => [page.path, page.headers ?? {}]),
-    );
-
-    if (pages.length > 0) {
-      console.log(`\n  Prerendering ${pages.length} SSG/ISG route(s)...\n`);
-      for (const page of pages) {
-        const filePath =
-          page.path === "/"
-            ? join(clientDir, "index.html")
-            : join(clientDir, page.path, "index.html");
-
-        mkdirSync(dirname(filePath), { recursive: true });
-        writeFileSync(filePath, page.html, "utf-8");
-        console.log(`    ${page.path} → ${filePath.replace(root + "/", "")}`);
-      }
-    }
-
-    if (Object.keys(headersManifest).length > 0) {
-      const headersManifestJson = `${JSON.stringify(headersManifest, null, 2)}\n`;
-      writeFileSync(
-        resolve(root, "dist/server/headers-manifest.json"),
-        headersManifestJson,
-        "utf-8",
-      );
-      mkdirSync(resolve(clientDir, "_pracht"), { recursive: true });
-      writeFileSync(resolve(clientDir, "_pracht/headers.json"), headersManifestJson, "utf-8");
-    }
-
-    if (Object.keys(isgManifest).length > 0) {
-      const isgManifestPath = resolve(root, "dist/server/isg-manifest.json");
-      writeFileSync(isgManifestPath, JSON.stringify(isgManifest, null, 2), "utf-8");
-      console.log(
-        `\n  ISG manifest → dist/server/isg-manifest.json (${Object.keys(isgManifest).length} route(s))\n`,
-      );
-    }
-
-    if (serverMod.buildTarget === "cloudflare") {
-      console.log("\n  Cloudflare worker → dist/server/server.js\n");
-      console.log("  Deploy with: wrangler deploy\n");
-    }
-
-    if (serverMod.buildTarget === "vercel") {
-      const outputPath = writeVercelBuildOutput({
-        functionName: serverMod.vercelFunctionName,
-        isgRoutes: Object.keys(isgManifest),
-        headersManifest,
-        regions: serverMod.vercelRegions,
-        root,
-        staticRoutes: pages.map((page) => page.path).filter((path) => !(path in isgManifest)),
-      });
-
-      console.log(`\n  Vercel build output → ${outputPath}\n`);
-    }
-  }
-
-  console.log("\n  Build complete.\n");
-}

package/lib/commands/dev.js

@@ -1,16 +0,0 @@
-import { createServer } from "vite";
-
-import { parseFlags } from "../cli.js";
-
-export async function devCommand(args) {
-  const options = parseFlags(args);
-  const port = parseInt(process.env.PORT || options._[0] || "3000", 10);
-
-  const server = await createServer({
-    root: process.cwd(),
-    server: { port },
-  });
-
-  await server.listen();
-  server.printUrls();
-}