@moda-ai/cli 1.30.1 → 1.31.1

package/README.md

@@ -126,6 +126,11 @@ Prompt management is code-first. Keep prompts in `prompts/**/*.prompt.md` or
 `prompts/**/*.prompt.json`, sync them with the CLI, and render them through the
 SDK so LLM spans include prompt metadata.
 
+JSON definitions can provide `content`, `systemPrompt`, or `messages`. Existing
+files with a nonempty text `template` also work: the CLI uses it as content when
+none of those fields supplies usable text. Sync preserves placeholders and
+leaves the source file unchanged; empty definitions still produce an error.
+
 ```bash
 moda prompts init
 moda prompts status

package/dist/{cli-mq00fkwt.js → cli-j3rr0nh7.js}

@@ -11,7 +11,7 @@ import {
   renderStatusHuman,
   scanHarness,
   validateHarnessReport
-} from "./cli-js4bmw21.js";
+} from "./cli-p3271yp2.js";
 import {
   isAuthSessionValid,
   loadAuthSession
@@ -772,7 +772,19 @@ function parsePromptFile(absPath) {
   if (absPath.endsWith(".json")) {
     const parsed = JSON.parse(raw);
     const key2 = parsed.key || keyFromPath(sourcePath);
-    const content = parsed.content || JSON.stringify(parsed.messages ?? [], null, 2);
+    const hasContent = typeof parsed.content === "string" && parsed.content.trim().length > 0;
+    const hasSystemPrompt = typeof parsed.systemPrompt === "string" && parsed.systemPrompt.trim().length > 0;
+    const hasTemplate = typeof parsed.template === "string" && parsed.template.trim().length > 0;
+    const hasMessages = Array.isArray(parsed.messages) && parsed.messages.some((message) => {
+      if (!message || typeof message !== "object" || !("role" in message) || !["system", "developer", "user", "assistant", "tool"].includes(String(message.role)) || !("content" in message))
+        return false;
+      const content2 = message.content;
+      return typeof content2 === "string" ? content2.trim().length > 0 : Array.isArray(content2) && content2.some((part) => part && typeof part === "object" && part.type === "text" && typeof part.text === "string" && part.text.trim().length > 0);
+    });
+    if (!hasContent && !hasSystemPrompt && !hasMessages && !hasTemplate) {
+      throw new Error(`${sourcePath}: prompt JSON requires non-empty content, systemPrompt, messages, or a text template.`);
+    }
+    const content = hasContent ? parsed.content : hasSystemPrompt ? parsed.systemPrompt : hasMessages ? JSON.stringify(parsed.messages, null, 2) : parsed.template;
     return normalizePrompt({ ...parsed, key: key2, name: parsed.name || key2, sourcePath, content });
   }
   const { frontmatter, body } = parseFrontmatter(raw);
@@ -931,11 +943,18 @@ function printJson(value) {
 
 // src/init/rules.ts
 import { existsSync as existsSync3, mkdirSync as mkdirSync2, readFileSync as readFileSync2, writeFileSync as writeFileSync3 } from "node:fs";
+import { join as join3 } from "node:path";
+import { confirm } from "@inquirer/prompts";
+
+// src/package-assets.ts
 import { dirname as dirname2, join as join2 } from "node:path";
 import { fileURLToPath } from "node:url";
-import { confirm } from "@inquirer/prompts";
-var PACKAGE_ROOT = join2(dirname2(fileURLToPath(import.meta.url)), "..", "..");
-var SKILLS_SOURCE_DIR = join2(PACKAGE_ROOT, "skills");
+function packageAssetPath(...parts) {
+  return join2(dirname2(fileURLToPath(import.meta.url)), "..", ...parts);
+}
+
+// src/init/rules.ts
+var SKILLS_SOURCE_DIR = packageAssetPath("skills");
 var SKILLS = [
   {
     id: "moda-cli",
@@ -946,7 +965,7 @@ var ADAPTERS = {
   "Claude Code": {
     name: "Claude Code",
     targetFor: (skillId) => ({
-      dir: join2(".claude", "skills", skillId),
+      dir: join3(".claude", "skills", skillId),
       filename: "SKILL.md"
     }),
     transform: (content) => content
@@ -954,7 +973,7 @@ var ADAPTERS = {
   Cursor: {
     name: "Cursor",
     targetFor: (skillId) => ({
-      dir: join2(".cursor", "rules"),
+      dir: join3(".cursor", "rules"),
       filename: `${skillId}.mdc`
     }),
     transform: claudeSkillToCursorRule
@@ -979,7 +998,7 @@ function claudeSkillToCursorRule(skillContent) {
 ${body.replace(/^\s+/, "")}`;
 }
 function loadSkillSource(skillId) {
-  const path = join2(SKILLS_SOURCE_DIR, skillId, "SKILL.md");
+  const path = join3(SKILLS_SOURCE_DIR, skillId, "SKILL.md");
   if (!existsSync3(path)) {
     throw new Error(`Skill source missing: ${path}`);
   }
@@ -987,9 +1006,9 @@ function loadSkillSource(skillId) {
 }
 function installSkillForAgent(skill, adapter, content) {
   const { dir, filename } = adapter.targetFor(skill.id);
-  const dirPath = join2(process.cwd(), dir);
-  const filePath = join2(dirPath, filename);
-  const relativePath = join2(dir, filename);
+  const dirPath = join3(process.cwd(), dir);
+  const filePath = join3(dirPath, filename);
+  const relativePath = join3(dir, filename);
   try {
     mkdirSync2(dirPath, { recursive: true });
     if (existsSync3(filePath)) {
@@ -1075,7 +1094,7 @@ async function installRules(detectedAgents, options = {}) {
 
 // src/init/mcp.ts
 import { existsSync as existsSync4 } from "node:fs";
-import { join as join3 } from "node:path";
+import { join as join4 } from "node:path";
 import { execSync } from "node:child_process";
 function isOnPath(cmd) {
   try {
@@ -1088,10 +1107,10 @@ function isOnPath(cmd) {
 function detectAgents() {
   const cwd = process.cwd();
   const agents = [];
-  if (existsSync4(join3(cwd, ".claude")) || isOnPath("claude")) {
+  if (existsSync4(join4(cwd, ".claude")) || isOnPath("claude")) {
     agents.push("Claude Code");
   }
-  if (existsSync4(join3(cwd, ".cursor"))) {
+  if (existsSync4(join4(cwd, ".cursor"))) {
     agents.push("Cursor");
   }
   return agents;
@@ -1108,7 +1127,7 @@ import {
   readFileSync as readFileSync5,
   writeFileSync as writeFileSync6
 } from "node:fs";
-import { basename, dirname as dirname5, join as join6, relative as relative3 } from "node:path";
+import { basename, dirname as dirname5, join as join7, relative as relative3 } from "node:path";
 
 // src/skills-manifest.ts
 import {
@@ -1120,12 +1139,12 @@ import {
   unlinkSync,
   writeFileSync as writeFileSync4
 } from "node:fs";
-import { dirname as dirname3, isAbsolute as isAbsolute4, join as join4, normalize, relative as relative2, resolve as resolve4 } from "node:path";
-var SKILLS_MANIFEST_PATH = join4(".moda", "skills.yml");
+import { dirname as dirname3, isAbsolute as isAbsolute4, join as join5, normalize, relative as relative2, resolve as resolve4 } from "node:path";
+var SKILLS_MANIFEST_PATH = join5(".moda", "skills.yml");
 var SKILL_SOURCE_HARNESS = "moda";
 var SKILL_SOURCE_INTEGRATION_BUNDLED = "integration-bundled";
 function readSkillsManifest(cwd) {
-  const manifestPath = join4(cwd, SKILLS_MANIFEST_PATH);
+  const manifestPath = join5(cwd, SKILLS_MANIFEST_PATH);
   if (!existsSync5(manifestPath))
     return null;
   let text = "";
@@ -1156,7 +1175,7 @@ function manifestHeaderFrom(manifest) {
   };
 }
 function writeSkillsManifest(cwd, header, records) {
-  const manifestPath = join4(cwd, SKILLS_MANIFEST_PATH);
+  const manifestPath = join5(cwd, SKILLS_MANIFEST_PATH);
   const managedPaths = [...new Set(records.flatMap((record) => record.paths))].sort();
   const lines = [
     "version: 1",
@@ -1211,7 +1230,7 @@ function pruneManagedSkillPaths(cwd, managedPaths, log) {
 function pruneEmptyClaudeSkillDir(cwd, relativePath) {
   if (!relativePath.startsWith(".claude/skills/") || !relativePath.endsWith("/SKILL.md"))
     return;
-  const dir = dirname3(join4(cwd, relativePath));
+  const dir = dirname3(join5(cwd, relativePath));
   try {
     if (existsSync5(dir) && readdirSync2(dir).length === 0) {
       rmdirSync(dir);
@@ -1331,7 +1350,7 @@ function yamlQuote(value) {
 
 // src/agents-md.ts
 import { existsSync as existsSync6, mkdirSync as mkdirSync4, readFileSync as readFileSync4, writeFileSync as writeFileSync5 } from "node:fs";
-import { dirname as dirname4, join as join5 } from "node:path";
+import { dirname as dirname4, join as join6 } from "node:path";
 var AGENTS_MD_FILENAME = "AGENTS.md";
 var FIXES_BLOCK_BEGIN = "<!-- moda:fixes:begin -->";
 var FIXES_BLOCK_END = "<!-- moda:fixes:end -->";
@@ -1392,7 +1411,7 @@ function writeFileIfChanged(filePath, content) {
   }
 }
 function upsertFixesAgentsBlock(cwd) {
-  const filePath = join5(cwd, AGENTS_MD_FILENAME);
+  const filePath = join6(cwd, AGENTS_MD_FILENAME);
   let existing = null;
   if (existsSync6(filePath)) {
     try {
@@ -1408,7 +1427,7 @@ function upsertFixesAgentsBlock(cwd) {
 // src/skills.ts
 var SKILLS_REQUEST_TIMEOUT_MS = 15000;
 var VALID_STATUSES = new Set(["approved", "proposed", "all"]);
-var SKILL_PROPOSALS_MANIFEST_PATH = join6(".moda", "skill-proposals.yml");
+var SKILL_PROPOSALS_MANIFEST_PATH = join7(".moda", "skill-proposals.yml");
 async function fetchSkills(status) {
   const apiKey = resolveApiKey();
   if (!apiKey) {
@@ -1578,7 +1597,7 @@ async function pullSkills(options = {}) {
     }
   }
   const agents = detectAgents();
-  const writeCursor = agents.includes("Cursor") || existsSync7(join6(cwd, ".cursor"));
+  const writeCursor = agents.includes("Cursor") || existsSync7(join7(cwd, ".cursor"));
   const records = [];
   for (const skill of skills) {
     if (!skill.skill_id || !skill.skill_md) {
@@ -1591,7 +1610,7 @@ async function pullSkills(options = {}) {
       log(`  skipped invalid skill_id: ${JSON.stringify(skill.skill_id).slice(0, 80)}`);
       continue;
     }
-    const claudePath = join6(cwd, ".claude", "skills", safeId, "SKILL.md");
+    const claudePath = join7(cwd, ".claude", "skills", safeId, "SKILL.md");
     const claudeStatus = writeIfChanged(claudePath, skill.skill_md);
     const claudeRelativePath = `.claude/skills/${safeId}/SKILL.md`;
     recordOutcome(summary, claudeStatus, log, claudeRelativePath);
@@ -1599,7 +1618,7 @@ async function pullSkills(options = {}) {
     if (claudeStatus !== "failed")
       paths.push(claudeRelativePath);
     if (writeCursor) {
-      const cursorPath = join6(cwd, ".cursor", "rules", `${safeId}.mdc`);
+      const cursorPath = join7(cwd, ".cursor", "rules", `${safeId}.mdc`);
       const cursorStatus = writeIfChanged(cursorPath, claudeSkillToCursorRule(skill.skill_md));
       const cursorRelativePath = `.cursor/rules/${safeId}.mdc`;
       recordOutcome(summary, cursorStatus, log, cursorRelativePath);
@@ -1637,7 +1656,7 @@ function recordOutcome(summary, status, log, relativePath) {
       break;
   }
 }
-var SKILLS_DIR = join6(".claude", "skills");
+var SKILLS_DIR = join7(".claude", "skills");
 var SKILL_FILE = "SKILL.md";
 var MAX_SKILL_WALK_DEPTH = 4;
 async function runSkillSync(flags = {}, profileOptions = {}, options = {}) {
@@ -1668,7 +1687,7 @@ async function runSkillSync(flags = {}, profileOptions = {}, options = {}) {
   return { synced: response.synced ?? [], dryRun, ...agentsMd ? { agentsMd } : {} };
 }
 function discoverLocalSkills(cwd) {
-  const skillsRoot = join6(cwd, SKILLS_DIR);
+  const skillsRoot = join7(cwd, SKILLS_DIR);
   if (!existsSync7(skillsRoot))
     return [];
   const managed = new Set(managedSkillPaths(cwd));
@@ -1717,7 +1736,7 @@ function walkSkillFiles(dir, depth) {
   }
   const files = [];
   for (const entry of entries) {
-    const fullPath = join6(dir, entry);
+    const fullPath = join7(dir, entry);
     let stat;
     try {
       stat = lstatSync2(fullPath);
@@ -1791,7 +1810,7 @@ async function applySkillProposal(proposalId, flags, profileOptions = {}, option
   }
   const safeId = safeSkillId(proposal.skillId);
   const relativePath = `.claude/skills/${safeId}/SKILL.md`;
-  const writeStatus = writeIfChanged(join6(cwd, relativePath), proposal.skillMd);
+  const writeStatus = writeIfChanged(join7(cwd, relativePath), proposal.skillMd);
   if (writeStatus === "failed") {
     throw new Error(`Failed to write ${relativePath}`);
   }
@@ -1824,7 +1843,7 @@ function safeSkillId(skillId) {
   return safeId;
 }
 function writeSkillProposalManifest(cwd, proposal, relativePath) {
-  const manifestPath = join6(cwd, SKILL_PROPOSALS_MANIFEST_PATH);
+  const manifestPath = join7(cwd, SKILL_PROPOSALS_MANIFEST_PATH);
   mkdirSync5(dirname5(manifestPath), { recursive: true });
   const existing = existsSync7(manifestPath) ? readFileSync5(manifestPath, "utf8") : `applied_proposals:
 `;
@@ -2085,7 +2104,7 @@ function sha2562(input) {
 // src/doctor.ts
 import { createHash as createHash3 } from "node:crypto";
 import { existsSync as existsSync8, readFileSync as readFileSync6, statSync } from "node:fs";
-import { join as join7 } from "node:path";
+import { join as join8 } from "node:path";
 async function runDoctorCommand(context) {
   const report = await buildDoctorReport(context, {
     offline: context.flags.offline === "true"
@@ -2208,7 +2227,7 @@ async function buildDoctorReport(context, options) {
   checks.push(urlCheck("base_url_valid", "config", "Base URL is valid", baseUrl));
   checks.push(urlCheck("ingest_url_valid", "config", "Ingest URL is valid", ingestUrl));
   checks.push(localBaseUrlCheck(activeProfile.environment, baseUrl));
-  const manifestPath = join7(context.cwd, ".moda", "harness.json");
+  const manifestPath = join8(context.cwd, ".moda", "harness.json");
   const manifestExists = existsSync8(manifestPath);
   checks.push({
     id: "harness_manifest_exists",
@@ -2498,7 +2517,7 @@ function harnessSyncStateCheck(cwd) {
   };
 }
 function harnessReportStateCheck(cwd) {
-  const reportPath = join7(cwd, HARNESS_REPORT_PATH);
+  const reportPath = join8(cwd, HARNESS_REPORT_PATH);
   if (!existsSync8(reportPath)) {
     return {
       id: "harness_report_approved_synced",
@@ -2592,7 +2611,7 @@ function harnessReportStateCheck(cwd) {
   }
 }
 function readHarnessReportApproval(cwd) {
-  const approvalPath = join7(cwd, HARNESS_REPORT_APPROVAL_PATH);
+  const approvalPath = join8(cwd, HARNESS_REPORT_APPROVAL_PATH);
   if (!existsSync8(approvalPath))
     return null;
   try {
@@ -2610,7 +2629,7 @@ function readHarnessReportApproval(cwd) {
   }
 }
 function discoveryCheck(id, group, title, cwd, relativePaths, nextCommand) {
-  const found = relativePaths.filter((path) => existsSync8(join7(cwd, path)));
+  const found = relativePaths.filter((path) => existsSync8(join8(cwd, path)));
   return {
     id,
     group,
@@ -2626,7 +2645,7 @@ function sdkInitializationCheck(cwd) {
   const candidates = ["src", "app", "agent_harness"];
   let detected = false;
   for (const candidate of candidates) {
-    const path = join7(cwd, candidate);
+    const path = join8(cwd, candidate);
     if (!existsSync8(path))
       continue;
     try {
@@ -2950,4 +2969,4 @@ function isRecord(value) {
   return Boolean(value) && typeof value === "object" && !Array.isArray(value);
 }
 
-export { DEFAULT_POLL_INTERVAL_MS, DEFAULT_WAIT_TIMEOUT_MS, fetchLatestRun, runPromptsCommand, initPrompts, runPromptSync, claudeSkillToCursorRule, installRules, detectAgents, SKILL_SOURCE_INTEGRATION_BUNDLED, readSkillsManifest, upsertSkillManifestRecord, runSkillSync, runSkillsCommand, runDoctorCommand, runStatusCommand, buildDoctorReport };
+export { DEFAULT_POLL_INTERVAL_MS, DEFAULT_WAIT_TIMEOUT_MS, fetchLatestRun, runPromptsCommand, initPrompts, runPromptSync, packageAssetPath, claudeSkillToCursorRule, installRules, detectAgents, SKILL_SOURCE_INTEGRATION_BUNDLED, readSkillsManifest, upsertSkillManifestRecord, runSkillSync, runSkillsCommand, runDoctorCommand, runStatusCommand, buildDoctorReport };