@funkai/cli 0.1.2 → 0.1.4

package/src/lib/prompts/paths.ts

@@ -23,7 +23,7 @@ function extractName(content: string): string | undefined {
     return undefined;
   }
 
-  const frontmatter = match[1];
+  const [, frontmatter] = match;
   const nameLine = frontmatter.split("\n").find((line) => line.startsWith("name:"));
   if (!nameLine) {
     return undefined;
@@ -74,7 +74,7 @@ function scanDirectory(dir: string, depth: number): DiscoveredPrompt[] {
 
       if (entry.isFile() && extname(entry.name) === PROMPT_EXT) {
         // oxlint-disable-next-line security/detect-non-literal-fs-filename -- safe: reading prompt file content for name extraction
-        const content = readFileSync(fullPath, "utf-8");
+        const content = readFileSync(fullPath, "utf8");
         const name = extractName(content) ?? deriveNameFromPath(fullPath);
 
         if (!NAME_RE.test(name)) {

package/src/lib/prompts/pipeline.ts

@@ -1,15 +1,30 @@
 import { existsSync, readFileSync } from "node:fs";
 import { resolve } from "node:path";
 
-import { clean, PARTIALS_DIR } from "@funkai/prompts";
+import { clean, PARTIALS_DIR } from "@funkai/prompts/cli";
 
-import { type ParsedPrompt } from "./codegen.js";
+import type { ParsedPrompt } from "./codegen.js";
 import { extractVariables } from "./extract-variables.js";
 import { flattenPartials } from "./flatten.js";
 import { parseFrontmatter } from "./frontmatter.js";
-import { lintPrompt, type LintResult } from "./lint.js";
+import { lintPrompt } from "./lint.js";
+import type { LintResult } from "./lint.js";
 import { discoverPrompts } from "./paths.js";
 
+/**
+ * Resolve the list of partial directories to search.
+ *
+ * @param customDir - Custom partials directory path.
+ * @returns Array of directories to search for partials.
+ */
+// oxlint-disable-next-line security/detect-non-literal-fs-filename -- safe: checking custom partials directory from CLI config
+function resolvePartialsDirs(customDir: string): readonly string[] {
+  if (existsSync(customDir)) {
+    return [customDir, PARTIALS_DIR];
+  }
+  return [PARTIALS_DIR];
+}
+
 /**
  * Options for the prompts lint pipeline.
  */
@@ -35,14 +50,11 @@ export interface LintPipelineResult {
 export function runLintPipeline(options: LintPipelineOptions): LintPipelineResult {
   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 = existsSync(customPartialsDir)
-    ? [customPartialsDir, PARTIALS_DIR]
-    : [PARTIALS_DIR];
+  const partialsDirs = resolvePartialsDirs(customPartialsDir);
 
   const results = discovered.map((d) => {
     // oxlint-disable-next-line security/detect-non-literal-fs-filename -- safe: reading discovered prompt file
-    const raw = readFileSync(d.filePath, "utf-8");
+    const raw = readFileSync(d.filePath, "utf8");
     const frontmatter = parseFrontmatter(raw, d.filePath);
     const template = flattenPartials(clean(raw), partialsDirs);
     const templateVars = extractVariables(template);
@@ -81,14 +93,11 @@ export interface GeneratePipelineResult {
 export function runGeneratePipeline(options: GeneratePipelineOptions): GeneratePipelineResult {
   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 = existsSync(customPartialsDir)
-    ? [customPartialsDir, PARTIALS_DIR]
-    : [PARTIALS_DIR];
+  const partialsDirs = resolvePartialsDirs(customPartialsDir);
 
   const processed = discovered.map((d) => {
     // oxlint-disable-next-line security/detect-non-literal-fs-filename -- safe: reading discovered prompt file
-    const raw = readFileSync(d.filePath, "utf-8");
+    const raw = readFileSync(d.filePath, "utf8");
     const frontmatter = parseFrontmatter(raw, d.filePath);
     const template = flattenPartials(clean(raw), partialsDirs);
     const templateVars = extractVariables(template);