@funkai/cli 0.1.1 → 0.1.2

package/src/lib/prompts/frontmatter.ts

@@ -1,8 +1,7 @@
-import { match } from "ts-pattern";
 import { parse as parseYaml } from "yaml";
 
-const FRONTMATTER_RE = /^---\r?\n([\s\S]*?)\r?\n---\r?\n?/;
-const NAME_RE = /^[a-z0-9-]+$/;
+export const FRONTMATTER_RE = /^---\r?\n([\s\S]*?)\r?\n---\r?\n?/;
+export const NAME_RE = /^[a-z0-9-]+$/;
 
 /**
  * Parse raw YAML content into a record, wrapping parse errors
@@ -26,20 +25,20 @@ function parseYamlContent(yaml: string, filePath: string): Record<string, unknow
  * A variable declared in the frontmatter `schema` block.
  */
 export interface SchemaVariable {
-  name: string;
-  type: string;
-  required: boolean;
-  description?: string;
+  readonly name: string;
+  readonly type: string;
+  readonly required: boolean;
+  readonly description?: string;
 }
 
 /**
  * Parsed frontmatter from a `.prompt` file.
  */
 export interface ParsedFrontmatter {
-  name: string;
-  group?: string;
-  version?: string;
-  schema: SchemaVariable[];
+  readonly name: string;
+  readonly group?: string;
+  readonly version?: string;
+  readonly schema: readonly SchemaVariable[];
 }
 
 /**
@@ -77,22 +76,21 @@ export function parseFrontmatter(content: string, filePath: string): ParsedFront
     );
   }
 
-  const group = match(typeof parsed.group === "string")
-    .with(true, () => {
-      const g = parsed.group as string;
-      const invalidSegment = g.split("/").find((segment) => !NAME_RE.test(segment));
-      if (invalidSegment !== undefined) {
-        throw new Error(
-          `Invalid group segment "${invalidSegment}" in ${filePath}. ` +
-            "Group segments must be lowercase alphanumeric with hyphens only.",
-        );
-      }
-      return g;
-    })
-    .otherwise(() => undefined);
-  const version = match(parsed.version != null)
-    .with(true, () => String(parsed.version))
-    .otherwise(() => undefined);
+  const group =
+    typeof parsed.group === "string"
+      ? (() => {
+          const g = parsed.group as string;
+          const invalidSegment = g.split("/").find((segment) => !NAME_RE.test(segment));
+          if (invalidSegment !== undefined) {
+            throw new Error(
+              `Invalid group segment "${invalidSegment}" in ${filePath}. ` +
+                "Group segments must be lowercase alphanumeric with hyphens only.",
+            );
+          }
+          return g;
+        })()
+      : undefined;
+  const version = parsed.version != null ? String(parsed.version) : undefined;
 
   const schema = parseSchemaBlock(parsed.schema, filePath);
 
@@ -124,13 +122,10 @@ function parseSchemaBlock(raw: unknown, filePath: string): SchemaVariable[] {
 
     if (typeof value === "object" && value !== null && !Array.isArray(value)) {
       const def = value as Record<string, unknown>;
-      const type = match(typeof def.type === "string")
-        .with(true, () => def.type as string)
-        .otherwise(() => "string");
+      const type = typeof def.type === "string" ? (def.type as string) : "string";
       const required = def.required !== false;
-      const description = match(typeof def.description === "string")
-        .with(true, () => def.description as string)
-        .otherwise(() => undefined);
+      const description =
+        typeof def.description === "string" ? (def.description as string) : undefined;
 
       return { name: varName, type, required, description };
     }

package/src/lib/prompts/lint.ts

@@ -4,17 +4,17 @@ import type { SchemaVariable } from "./frontmatter.js";
  * A single lint diagnostic.
  */
 export interface LintDiagnostic {
-  level: "error" | "warn";
-  message: string;
+  readonly level: "error" | "warn";
+  readonly message: string;
 }
 
 /**
  * Result of linting a single prompt file.
  */
 export interface LintResult {
-  name: string;
-  filePath: string;
-  diagnostics: LintDiagnostic[];
+  readonly name: string;
+  readonly filePath: string;
+  readonly diagnostics: readonly LintDiagnostic[];
 }
 
 /**
@@ -33,8 +33,8 @@ export interface LintResult {
 export function lintPrompt(
   name: string,
   filePath: string,
-  schemaVars: SchemaVariable[],
-  templateVars: string[],
+  schemaVars: readonly SchemaVariable[],
+  templateVars: readonly string[],
 ): LintResult {
   const diagnostics: LintDiagnostic[] = [];
   const declared = new Set(schemaVars.map((v) => v.name));
@@ -69,6 +69,6 @@ export function lintPrompt(
 /**
  * Check whether any lint results contain errors.
  */
-export function hasLintErrors(results: LintResult[]): boolean {
+export function hasLintErrors(results: readonly LintResult[]): boolean {
   return results.some((r) => r.diagnostics.some((d) => d.level === "error"));
 }

package/src/lib/prompts/paths.ts

@@ -1,14 +1,14 @@
 import { existsSync, lstatSync, readdirSync, readFileSync } from "node:fs";
 import { basename, extname, join, resolve } from "node:path";
 
+import { FRONTMATTER_RE, NAME_RE } from "./frontmatter.js";
+
 const MAX_DEPTH = 5;
 const PROMPT_EXT = ".prompt";
-const FRONTMATTER_RE = /^---\r?\n([\s\S]*?)\r?\n---\r?\n?/;
-const NAME_RE = /^[a-z0-9-]+$/;
 
 export interface DiscoveredPrompt {
-  name: string;
-  filePath: string;
+  readonly name: string;
+  readonly filePath: string;
 }
 
 /**
@@ -102,7 +102,7 @@ function scanDirectory(dir: string, depth: number): DiscoveredPrompt[] {
  * @returns Sorted, deduplicated list of discovered prompts.
  * @throws If duplicate prompt names are found across roots.
  */
-export function discoverPrompts(roots: string[]): DiscoveredPrompt[] {
+export function discoverPrompts(roots: readonly string[]): DiscoveredPrompt[] {
   const all = roots.flatMap((root) => scanDirectory(resolve(root), 0));
 
   const byName = Map.groupBy(all, (prompt) => prompt.name);

package/src/lib/prompts/pipeline.ts

@@ -2,7 +2,6 @@ import { existsSync, readFileSync } from "node:fs";
 import { resolve } from "node:path";
 
 import { clean, PARTIALS_DIR } from "@funkai/prompts";
-import { match } from "ts-pattern";
 
 import { type ParsedPrompt } from "./codegen.js";
 import { extractVariables } from "./extract-variables.js";
@@ -37,9 +36,9 @@ export function runLintPipeline(options: LintPipelineOptions): LintPipelineResul
   const discovered = discoverPrompts([...options.roots]);
   const customPartialsDir = resolve(options.partials ?? ".prompts/partials");
   // oxlint-disable-next-line security/detect-non-literal-fs-filename -- safe: checking custom partials directory from CLI config
-  const partialsDirs = match(existsSync(customPartialsDir))
-    .with(true, () => [customPartialsDir, PARTIALS_DIR])
-    .otherwise(() => [PARTIALS_DIR]);
+  const partialsDirs = existsSync(customPartialsDir)
+    ? [customPartialsDir, PARTIALS_DIR]
+    : [PARTIALS_DIR];
 
   const results = discovered.map((d) => {
     // oxlint-disable-next-line security/detect-non-literal-fs-filename -- safe: reading discovered prompt file
@@ -83,9 +82,9 @@ export function runGeneratePipeline(options: GeneratePipelineOptions): GenerateP
   const discovered = discoverPrompts([...options.roots]);
   const customPartialsDir = resolve(options.partials ?? resolve(options.out, "../partials"));
   // oxlint-disable-next-line security/detect-non-literal-fs-filename -- safe: checking custom partials directory from CLI config
-  const partialsDirs = match(existsSync(customPartialsDir))
-    .with(true, () => [customPartialsDir, PARTIALS_DIR])
-    .otherwise(() => [PARTIALS_DIR]);
+  const partialsDirs = existsSync(customPartialsDir)
+    ? [customPartialsDir, PARTIALS_DIR]
+    : [PARTIALS_DIR];
 
   const processed = discovered.map((d) => {
     // oxlint-disable-next-line security/detect-non-literal-fs-filename -- safe: reading discovered prompt file