@oh-story/dsh 0.1.17 → 0.1.19

package/lib/drama/skills/short-drama-write/scripts/selftest.py

@@ -1,9 +1,8 @@
 #!/usr/bin/env python3
-"""Offline self-test for screenplay indexing and voice-sheet projection."""
+"""Offline self-test for screenplay indexing and duration estimates."""
 
 from __future__ import annotations
 
-import hashlib
 import json
 import sys
 import tempfile
@@ -11,7 +10,6 @@ from pathlib import Path
 
 from duration_estimate import StaleIndex, estimate, index_blocks
 from screenplay_index import build_index
-from voice_sheet_check import check
 
 MINIMUM_PYTHON = (3, 9)
 if sys.version_info < MINIMUM_PYTHON:
@@ -23,12 +21,6 @@ def require(condition: bool, message: str) -> None:
         raise AssertionError(message)
 
 
-def codes(result: dict[str, object]) -> set[str]:
-    findings = result["findings"]
-    assert isinstance(findings, list)
-    return {finding["code"] for finding in findings}
-
-
 def measure_text(body: str, project: dict[str, object] | None = None) -> dict[str, object]:
     """Index a screenplay the documented way, then estimate from that index.
 
@@ -85,7 +77,7 @@ def test_duration_uses_the_projects_own_rates() -> None:
 
 
 def test_production_tags_are_not_performed_time() -> None:
-    report = measure_text("[画面文字] 账号后台：粉丝 2\n\n他把门关上。\n")
+    report = measure_text("[画面文字] 店门告示：暂停营业\n\n他把门关上。\n")
     require(report["counts"]["production_tag_lines"] == 1, "tag counted separately")
     require(report["counts"]["action_paragraphs"] == 1, "tag is not an action")
 
@@ -146,72 +138,6 @@ def main() -> int:
             speakers=["陈予安"],
         )
         require(summary["review_status"] == "clean", "valid screenplay index")
-        index_bytes = index_path.read_bytes()
-        records = [
-            json.loads(line) for line in index_bytes.decode("utf-8").splitlines()
-        ]
-        dialogue = next(record for record in records if record.get("kind") == "dialogue")
-
-        header = {
-            "record_type": "sources",
-            "schema_version": "1.0.0",
-            "sources": {
-                "screenplay-index": {
-                    "owner": "short-drama-write",
-                    "artifact": "剧集/EP001/screenplay-index.jsonl",
-                    "hash": hashlib.sha256(index_bytes).hexdigest(),
-                }
-            },
-        }
-        line = {
-            "line_id": "LINE-001",
-            "channel": "sync",
-            "speaker_display": "陈予安",
-            "line_text": "我回来了。",
-            "source_ref": {"src": "screenplay-index", "record_id": dialogue["block_id"]},
-        }
-        sheet = [header, line]
-        result = check(sheet, records, screenplay.read_bytes())
-        require(result["status"] == "pass", "faithful voice sheet")
-        require(result["lines"] == 1, "the sources header is not a voice line")
-
-        # A sheet written before the compact form still resolves; real projects
-        # hold both spellings and neither is rewritten.
-        expanded = [
-            dict(
-                line,
-                source_ref={
-                    "owner": "short-drama-write",
-                    "artifact": "剧集/EP001/screenplay-index.jsonl",
-                    "hash": hashlib.sha256(index_bytes).hexdigest(),
-                    "record_id": dialogue["block_id"],
-                },
-            )
-        ]
-        require(
-            check(expanded, records, screenplay.read_bytes())["status"] == "pass",
-            "expanded voice sheet",
-        )
-
-        undeclared = [header, dict(line, source_ref={"src": "screenplay", "record_id": "X"})]
-        require(
-            "VOICE_SOURCE_REF_UNDECLARED"
-            in codes(check(undeclared, records, screenplay.read_bytes())),
-            "src without a sources entry was not detected",
-        )
-
-        unbound = [header, dict(line, source_ref={"record_id": dialogue["block_id"]})]
-        require(
-            "VOICE_SOURCE_REF_MISSING"
-            in codes(check(unbound, records, screenplay.read_bytes())),
-            "source_ref naming no upstream snapshot was not detected",
-        )
-
-        changed = [header, dict(line, line_text="我走了。")]
-        require(
-            check(changed, records, screenplay.read_bytes())["status"] == "fail",
-            "changed voice line was not detected",
-        )
 
     test_duration_reports_counts_without_rates()
     test_duration_uses_the_projects_own_rates()
@@ -221,7 +147,7 @@ def main() -> int:
     test_comments_are_not_production_content()
     test_an_index_from_other_bytes_is_refused()
 
-    print("14 self-tests passed")
+    print("8 self-tests passed")
     return 0
 
 

package/lib/index.js

@@ -129,7 +129,7 @@ var DSH_SKILL_BRIDGE = [
   "Never start another Agent runtime, session transport, Dashboard, SSE stream, polling loop, or model configuration.",
   "All seven upstream Oh Story specialist Roles are bundled. Invoke one with oh_story_role and a self-contained prompt.",
   "Never inspect .claude/agents, .codex/agents, .opencode/agents, .agents/agents, .zcode, or .story-deployed to decide whether a Role is available.",
-  "Oh Story Roles are reached only through oh_story_role. Never start one through a generic subagent or Task tool \u2014 not DSH's subagent or subagent_fork tools and not invoke_subagent \u2014 and never pass a Role name as subagent_type, agent_type, agent, or TypeName, even where the upstream text below describes that for another host.",
+  "Oh Story Roles are reached only through oh_story_role. Never start one through a generic subagent or Task tool \u2014 not DSH's subagent or subagent_fork tools, not invoke_subagent, and not an Agent tool \u2014 and never pass a Role name as subagent_type, agent_type, agent, or TypeName, even where the upstream text below describes that for another host.",
   "Use only DSH-visible tools. DSH sandbox and permission policy remain authoritative.",
   // Upstream deploys these 「与作者协作」 rules only in its CLAUDE.md/AGENTS.md
   // templates, which DSH does not install.
@@ -166,7 +166,7 @@ var DSH_DRAMA_BRIDGE = [
   "The \u77ED\u5267 tab is the creator workspace. Never start dashboard_server.py, another web server, Dashboard, Agent runtime, session transport, or model configuration.",
   "Drama Skills uses a creator-first contract: each episode keeps only the requested documents, up to five creator-facing sources at \u5267\u96C6/<EP>/\u5267\u672C.md, \u89C6\u89C9\u8BBE\u5B9A.md, \u5206\u955C.md, \u56FE\u7247\u63D0\u793A\u8BCD.md, and \u89C6\u9891\u63D0\u793A\u8BCD.md. Never precreate empty documents, backfill nominal stages, or start work the creator did not request. Persisted reviews use creator-readable Markdown under \u5BA1\u67E5/; an oral review writes nothing.",
   "\u5267\u96C6/<EP>/\u526A\u8F91\u5355.md is the v0.7 assembly record, not a sixth creative truth: the five creator documents stay the creative truth. It is written after readable media exists and records what reaches the cut \u2014 which footage or accepted static keyframes are kept or left unused and why, each cut's in/out timing, its subtitles, \u753B\u9762\u6587\u5B57 overlays, \u97F3\u6548 cues, \u914D\u97F3 and \u73AF\u5883\u58F0, post-processing such as shot-match correction or an external mix, and the delivery spec including optional \u9897\u7C92 grain and \u63A5\u955C\u5339\u914D. It never changes a line, a shot's job, or a declared duration; changing any of those means returning to the document that owns it.",
-  "For a v0.6 project, never create a parallel JSON/JSONL lifecycle truth, indexes, fingerprints, coverage tables, or QA records merely because legacy maintenance scripts and templates remain bundled.",
+  "For a v0.6 project, never create a parallel JSON/JSONL lifecycle truth, indexes, fingerprints, coverage tables, or QA records; Drama 0.9.1 no longer bundles the legacy JSON checkers, and creator_markdown_check.py is the structural check.",
   "Never upgrade a v0.5 structured project in place or mix both contracts in one project root. Keep legacy production/audit artifacts read-only and pinned to v0.5; migrate only into a new creator-first root with manual per-episode creator confirmation.",
   "Use only tools visible in the current DSH preset and preserve the upstream project ownership, freshness, review, and explicit production-confirmation contracts.",
   "Use oh_story_production only for semantic production-view intents (open/focus, explicit shot order, or tracking a job that this Agent is actually executing). Cosmetic canvas layout remains creator-controlled. The tool changes only the Session projection: it does not edit creator documents, generate media, or authorize production.",
@@ -489,6 +489,8 @@ function createVideoRecapSkillProvider(skillRoot = defaultVideoRecapSkillRoot())
 import { execFile } from "node:child_process";
 import { promisify } from "node:util";
 var execFileAsync = promisify(execFile);
+var DRAMA_PYTHON = [3, 9];
+var VIDEO_PYTHON = [3, 10];
 async function commandOutput(command, args) {
   try {
     const { stdout, stderr } = await execFileAsync(command, [...args], { encoding: "utf8", timeout: 5e3, maxBuffer: 4 * 1024 * 1024 });
@@ -497,21 +499,22 @@ async function commandOutput(command, args) {
     return void 0;
   }
 }
-function pythonVersion(value) {
+function pythonVersion(value, floor) {
+  const minimum = `${String(floor[0])}.${String(floor[1])}`;
   const match = /Python\s+(\d+)\.(\d+)(?:\.(\d+))?/u.exec(value ?? "");
-  if (match === null) return { ok: false };
+  if (match === null) return { ok: false, minimum };
   const major = Number(match[1]);
   const minor = Number(match[2]);
-  return { ok: major > 3 || major === 3 && minor >= 10, version: match[0].replace(/^Python\s+/u, "") };
+  return { ok: major > floor[0] || major === floor[0] && minor >= floor[1], version: match[0].replace(/^Python\s+/u, ""), minimum };
 }
-async function hostPython(probe = commandOutput) {
+async function hostPython(floor, probe = commandOutput) {
   let fallback;
   for (const command of ["python3", "python"]) {
-    const parsed = pythonVersion(await probe(command, ["--version"]));
+    const parsed = pythonVersion(await probe(command, ["--version"]), floor);
     if (parsed.ok) return { command, probe: parsed };
     if (parsed.version !== void 0) fallback ??= { command, probe: parsed };
   }
-  return fallback ?? { command: "python3", probe: { ok: false } };
+  return fallback ?? { command: "python3", probe: { ok: false, minimum: pythonVersion(void 0, floor).minimum } };
 }
 
 // src/native-hooks.ts
@@ -1482,7 +1485,7 @@ async function dramaPreflight() {
   if (dramaPreflightCache !== void 0 && dramaPreflightCache.expires > Date.now()) return dramaPreflightCache.value;
   dramaPreflightInFlight ??= (async () => {
     try {
-      const python = await hostPython();
+      const python = await hostPython(DRAMA_PYTHON);
       const adapterConfig = await ensureDramaAdapterConfig(defaultDramaSkillRoot(), { python: python.command });
       const value = { python: python.probe, adapterConfig, adapters: dramaAdapterStatuses() };
       dramaPreflightCache = { expires: Date.now() + 3e4, value };
@@ -1495,7 +1498,7 @@ async function dramaPreflight() {
 }
 async function videoPreflight() {
   if (videoPreflightCache !== void 0 && videoPreflightCache.expires > Date.now()) return videoPreflightCache.value;
-  const python = (await hostPython()).probe;
+  const python = (await hostPython(VIDEO_PYTHON)).probe;
   const [ffmpegOutput, ffmpegFilters, ffprobeOutput] = await Promise.all([
     commandOutput("ffmpeg", ["-version"]),
     commandOutput("ffmpeg", ["-hide_banner", "-filters"]),
@@ -1815,9 +1818,10 @@ async function previewDigest(realm, projectRoot) {
   await visit(app, "");
   return { ready, version: createHash2("sha256").update(entries.sort().join("\n")).digest("hex").slice(0, 16) };
 }
-function headingTitle(content, fallback) {
+function gameProjectTitle(content, fallback) {
   const heading = content?.split(/\r?\n/u).find((line) => /^#\s+/u.test(line));
-  return heading?.replace(/^#\s+/u, "").replace(/^PRODUCT_BRIEF\s*[·・:]?\s*/iu, "").trim() || fallback;
+  const title = heading?.replace(/^#\s+/u, "").replace(/^(?:PRODUCT[_ ]BRIEF|产品简报)/iu, "").replace(/^[\s·・:：|—–-]+/u, "").trim().replace(/^《([^《》]+)》$/u, "$1").trim();
+  return title || fallback;
 }
 async function workspaceGameProjects(realm, files, sessionId, maxBytes) {
   const roots = [...new Set(files.flatMap((file) => {
@@ -1843,7 +1847,7 @@ async function workspaceGameProjects(realm, files, sessionId, maxBytes) {
     return {
       id: `workspace:${id}`,
       root,
-      title: headingTitle(brief, id),
+      title: gameProjectTitle(brief, id),
       source: "workspace",
       previewReady: preview.ready,
       previewUrl: preview.ready ? `/oh-story/game-preview/workspace/${token(sessionId)}/${token(root)}/index.html` : void 0,
@@ -2142,7 +2146,7 @@ async function apply(context, config = {}) {
   registerOhStoryProductionTool(context);
   await registerOhStoryRoleTool(context);
   registerWorkspaceRoute(context, { maxBytes: config.editorMaxBytes ?? 2097152, trustedHosts });
-  await ensureDramaAdapterConfig(defaultDramaSkillRoot(), { python: (await hostPython()).command });
+  await ensureDramaAdapterConfig(defaultDramaSkillRoot(), { python: (await hostPython(DRAMA_PYTHON)).command });
 }
 var index_default = { name, inject, Config, apply };
 export {