@lingjingai/scriptctl 0.10.3 → 0.11.0

package/dist/domain/script-core.d.ts

@@ -15,7 +15,6 @@ export declare function findScene(script: Dict, epId: string, sceneId: string):
 export declare function validateScript(workspace: string, scriptPath?: string | null, opts?: {
     requireSource?: boolean;
     scriptData?: Dict;
-    persist?: boolean;
 }): Dict;
 export declare function assetKeys(assetType: string): [string, string, string];
 export declare function ensureRefId(script: Dict, assetType: string, rawId: unknown, opts?: {

package/dist/domain/script-core.js

@@ -84,7 +84,7 @@ export function validateScript(workspace, scriptPath = null, opts = {}) {
             exitCode: EXIT_INPUT,
             required: ["workspace/source.txt"],
             received: [sourcePath],
-            nextSteps: ["Run scriptctl import init first."],
+            nextSteps: ["Run scriptctl direct init first."],
         });
     }
     if (exists(sourcePath))
@@ -98,9 +98,9 @@ export function validateScript(workspace, scriptPath = null, opts = {}) {
         if (!exists(finalScriptPath)) {
             throw new CliError("VALIDATE BLOCKED: script not found", "Script JSON not found.", {
                 exitCode: EXIT_INPUT,
-                required: ["script.initial.json"],
+                required: ["script.initial.json or --script-path"],
                 received: [finalScriptPath],
-                nextSteps: ["Run scriptctl import init first."],
+                nextSteps: ["Run scriptctl direct init first."],
             });
         }
         try {
@@ -116,13 +116,8 @@ export function validateScript(workspace, scriptPath = null, opts = {}) {
         }
     }
     const issues = [];
-    // LOW_CONFIDENCE_METADATA reflects the local parse artifact (asset_metadata.json
-    // describes script.initial.json). It is only meaningful when validating that
-    // local script. When validating an in-memory script (scriptData) — i.e. the
-    // DB-backed final script after `import push` — the local metadata file is
-    // unrelated, so skip the check to avoid a permanent spurious error on the DB.
     const metadataPath = path.join(dd, "asset_metadata.json");
-    if (opts.scriptData === undefined && exists(metadataPath)) {
+    if (exists(metadataPath)) {
         let metadata = {};
         try {
             metadata = readJson(metadataPath);
@@ -131,15 +126,11 @@ export function validateScript(workspace, scriptPath = null, opts = {}) {
             metadata = {};
         }
         if (isDict(metadata) && metadata["confidence"] === "low") {
-            // Advisory (warning, not error): this is a parse-time confidence signal on
-            // the initial extraction, surfaced once in push's validation.json for the
-            // review subagent to double-check. It is not a clearable gate — DB edits
-            // don't touch this local artifact — so it must not fail `passed`.
             issues.push({
                 code: "LOW_CONFIDENCE_METADATA",
-                severity: "warning",
+                severity: "error",
                 summary: "AI metadata confidence is low.",
-                repair_hint: "Double-check worldview / role_type / descriptions during review.",
+                repair_hint: "Inspect assets and patch worldview, role_type, or descriptions explicitly.",
             });
         }
     }
@@ -437,11 +428,8 @@ export function validateScript(workspace, scriptPath = null, opts = {}) {
             });
         }
     }
-    // Same as LOW_CONFIDENCE_METADATA above: episode_plan.json is a local parse
-    // artifact that only describes the local script.initial.json, so skip it when
-    // validating an in-memory/DB script (scriptData).
     const planPath = path.join(dd, "episode_plan.json");
-    if (opts.scriptData === undefined && exists(planPath)) {
+    if (exists(planPath)) {
         const plan = readJson(planPath);
         if (plan["boundary_confidence"] === "low") {
             issues.push({
@@ -459,24 +447,7 @@ export function validateScript(workspace, scriptPath = null, opts = {}) {
     let totalScenes = 0;
     let totalActions = 0;
     let lastSceneNum = 0;
-    const episodeIds = new Set();
     for (const ep of episodes) {
-        const epId = strOf(ep["episode_id"]);
-        if (epId) {
-            if (episodeIds.has(epId)) {
-                // Duplicate episode_id is a hard structural error: the script-output DB
-                // enforces a unique (project_group_no, episode_uid) constraint, so a push
-                // would crash with a duplicate-key error. Surface it here instead.
-                issues.push({
-                    code: "DUPLICATE_EPISODE_ID",
-                    severity: "error",
-                    episode: epId,
-                    summary: `Duplicate episode_id: ${epId}`,
-                    repair_hint: "Renumber/rename so every episode_id is unique (rerun import init, or patch the episode).",
-                });
-            }
-            episodeIds.add(epId);
-        }
         const epTitle = strOf(ep["title"]).trim();
         if (!epTitle || isTechnicalEpisodeTitle(epTitle)) {
             issues.push({
@@ -484,7 +455,7 @@ export function validateScript(workspace, scriptPath = null, opts = {}) {
                 severity: "warning",
                 episode: ep["episode_id"],
                 summary: `Episode title is missing or technical: ${epTitle || "<empty>"}`,
-                repair_hint: 'Rerun `import init` to regenerate, or `script patch episode <episode_id> --field title --value "第N集：短标题"`.',
+                repair_hint: 'Rerun `direct init` to regenerate, or `script patch episode <episode_id> --field title --value "第N集：短标题"`.',
             });
         }
         const scenes = asList(ep["scenes"]);
@@ -933,10 +904,7 @@ export function validateScript(workspace, scriptPath = null, opts = {}) {
         script_path: opts.scriptData !== undefined ? "<in-memory>" : (scriptPath ?? path.join(dd, "script.initial.json")),
         validated_at: new Date().toISOString().replace(/\.\d+Z$/, "Z"),
     };
-    // persist defaults true. Callers that own the write (validateSession sets a
-    // nicer script_path label) or want no side effect (patch --dry-run) pass false.
-    if (opts.persist !== false)
-        writeJson(path.join(dd, "validation.json"), validation);
+    writeJson(path.join(dd, "validation.json"), validation);
     return validation;
 }
 // ---------------------------------------------------------------------------