@markdy/core 0.7.10 → 0.7.12

package/dist/index.d.ts

@@ -7,7 +7,7 @@ type AssetDef = {
     value: string;
 };
 type ActorDef = {
-    type: "sprite" | "text" | "box" | "figure" | "caption";
+    type: ActorType;
     /** Constructor arguments: asset name for sprite, display text for text/caption actors. */
     args: string[];
     x: number;
@@ -26,6 +26,8 @@ type ActorDef = {
      */
     anchor?: "top" | "bottom" | "center";
 };
+type BuiltinActorType = "sprite" | "text" | "box" | "figure" | "caption";
+type ActorType = BuiltinActorType | (string & {});
 type TimelineEvent = {
     time: number;
     actor: string;
@@ -89,7 +91,7 @@ type Chapter = {
  * renderer otherwise no-ops the offending statement.
  */
 type ParseWarning = {
-    kind: "unknown-action" | "unknown-modifier" | "unknown-scene-key" | "unknown-camera-action" | "unknown-preset" | "import-unresolved" | "preset-mixed";
+    kind: "unknown-action" | "unknown-modifier" | "unknown-scene-key" | "unknown-camera-action" | "unknown-preset" | "import-unresolved" | "preset-mixed" | "actor-count-threshold" | "label-overflow";
     message: string;
     line: number;
 };
@@ -137,6 +139,17 @@ interface ParseOptions {
      * `<ns>.<name>` prefix. Missing namespaces produce a soft warning.
      */
     imports?: Record<string, SceneAST>;
+    /**
+     * Emit a non-fatal warning when actor count exceeds this threshold.
+     * Set to <= 0 to disable.
+     */
+    actorCountWarningThreshold?: number;
+    /**
+     * Emit a non-fatal warning when actor labels exceed this length.
+     * Applies to the first constructor arg when it is a string.
+     * Set to <= 0 to disable.
+     */
+    labelLengthWarningThreshold?: number;
     /**
      * Internal flag — distinguishes a top-level call from a recursive
      * call used to expand a `preset`. Preset expansion bypasses the
@@ -167,4 +180,11 @@ type PresetFn = (args: string[]) => string;
 declare const PRESETS: Record<string, PresetFn>;
 declare const PRESET_NAMES: readonly string[];
 
-export { type ActorDef, type AssetDef, type Chapter, type ImportDecl, PRESETS, PRESET_NAMES, ParseError, type ParseOptions, type ParseWarning, type PresetFn, type SceneAST, type SceneMeta, type SequenceDef, type TemplateDef, type TimelineEvent, parse };
+type ActorPack = {
+    name: string;
+    actors: readonly string[];
+    actions?: Record<string, readonly string[]>;
+};
+declare function registerActorPack(pack: ActorPack): void;
+
+export { type ActorDef, type ActorPack, type ActorType, type AssetDef, type BuiltinActorType, type Chapter, type ImportDecl, PRESETS, PRESET_NAMES, ParseError, type ParseOptions, type ParseWarning, type PresetFn, type SceneAST, type SceneMeta, type SequenceDef, type TemplateDef, type TimelineEvent, parse, registerActorPack };

package/dist/index.js

@@ -224,6 +224,69 @@ actor title = caption("tier list") at top
 };
 var PRESET_NAMES = Object.keys(PRESETS);
 
+// src/registry.ts
+var BUILTIN_ACTOR_TYPES = [
+  "sprite",
+  "text",
+  "box",
+  "figure",
+  "caption"
+];
+var UNIVERSAL_ACTIONS = /* @__PURE__ */ new Set([
+  "enter",
+  "exit",
+  "move",
+  "fade_in",
+  "fade_out",
+  "scale",
+  "rotate",
+  "shake",
+  "say",
+  "throw",
+  "play"
+]);
+var FIGURE_ONLY_ACTIONS = /* @__PURE__ */ new Set([
+  "punch",
+  "kick",
+  "wave",
+  "nod",
+  "jump",
+  "bounce",
+  "face",
+  "rotate_part",
+  "pose"
+]);
+var CAMERA_ACTIONS = /* @__PURE__ */ new Set(["pan", "zoom", "shake"]);
+var actorTypes = new Set(BUILTIN_ACTOR_TYPES);
+var actorActions = /* @__PURE__ */ new Map();
+function registerActorPack(pack) {
+  for (const actor of pack.actors) {
+    actorTypes.add(actor);
+    if (!actorActions.has(actor)) actorActions.set(actor, /* @__PURE__ */ new Set());
+  }
+  if (!pack.actions) return;
+  for (const [actorType, actions] of Object.entries(pack.actions)) {
+    actorTypes.add(actorType);
+    const known = actorActions.get(actorType) ?? /* @__PURE__ */ new Set();
+    for (const action of actions) known.add(action);
+    actorActions.set(actorType, known);
+  }
+}
+function isKnownActorType(type) {
+  return actorTypes.has(type);
+}
+function isFigureOnlyAction(action) {
+  return FIGURE_ONLY_ACTIONS.has(action);
+}
+function isCameraAction(action) {
+  return CAMERA_ACTIONS.has(action);
+}
+function isKnownAction(actorType, action) {
+  if (UNIVERSAL_ACTIONS.has(action)) return true;
+  if (actorType === "figure" && FIGURE_ONLY_ACTIONS.has(action)) return true;
+  return actorActions.get(actorType)?.has(action) ?? false;
+}
+
 // src/parser.ts
 var ParseError = class extends Error {
   constructor(message, line) {
@@ -369,36 +432,6 @@ function parseActionParams(action, raw) {
   }
   return params;
 }
-var UNIVERSAL_ACTIONS = /* @__PURE__ */ new Set([
-  "enter",
-  "exit",
-  "move",
-  "fade_in",
-  "fade_out",
-  "scale",
-  "rotate",
-  "shake",
-  "say",
-  "throw",
-  "play"
-]);
-var FIGURE_ONLY_ACTIONS = /* @__PURE__ */ new Set([
-  "punch",
-  "kick",
-  "wave",
-  "nod",
-  "jump",
-  "bounce",
-  "face",
-  "rotate_part",
-  "pose"
-]);
-var CAMERA_ACTIONS = /* @__PURE__ */ new Set(["pan", "zoom", "shake"]);
-function isKnownAction(actorType, action) {
-  if (actorType === "camera") return CAMERA_ACTIONS.has(action);
-  if (UNIVERSAL_ACTIONS.has(action)) return true;
-  return FIGURE_ONLY_ACTIONS.has(action);
-}
 var MODIFIER_KEYS = /* @__PURE__ */ new Set(["scale", "rotate", "opacity", "size", "z"]);
 function parseSpaceModifiers(raw, lineNum, warnings) {
   const result = {};
@@ -469,14 +502,13 @@ function parseActorTrailer(trailerRaw, lineNum, warnings) {
   return { ...fromSpace, ...fromWith };
 }
 var ASSET_RE = /^asset\s+(\w+)\s*=\s*(image|icon)\("([^"]+)"\)$/;
-var BUILTIN_ACTOR_TYPES = /* @__PURE__ */ new Set(["sprite", "text", "box", "figure", "caption"]);
 var ACTOR_NUM_POS_RE = /^actor\s+(\w+)\s*=\s*([\w.]+)\(([^)]*)\)\s+at\s+\(\s*(-?[\d.]+)\s*,\s*(-?[\d.]+)\s*\)(.*)$/;
 var ACTOR_ANCHOR_POS_RE = /^actor\s+(\w+)\s*=\s*([\w.]+)\(([^)]*)\)\s+at\s+(top|bottom|center)\b(.*)$/;
 var EVENT_RE = /^@([\d.]+):\s+(\w+)\.(!?\w+)\((.*)\)$/;
 var REL_EVENT_RE = /^@\+([\d.]+):\s+(\w+)\.(!?\w+)\((.*)\)$/;
 var VAR_RE = /^var\s+(\w+)\s*=\s*(.+)$/;
 var DEF_HEADER_RE = /^def\s+(\w+)\(([^)]*)\)\s*\{$/;
-var DEF_BODY_RE = /^\s*(sprite|text|box|figure|caption)\(([^)]*)\)\s*$/;
+var DEF_BODY_RE = /^\s*([\w]+)\(([^)]*)\)\s*$/;
 var SEQ_HEADER_RE = /^seq\s+(\w+)(?:\(([^)]*)\))?\s*\{$/;
 var SEQ_EVENT_RE = /^@\+([\d.]+):\s+\$\.(!?\w+)\((.*)\)$/;
 var CHAPTER_HEADER_RE = /^scene\s+"([^"]+)"\s*\{$/;
@@ -527,6 +559,9 @@ function parse(source, opts = {}) {
     warnings: [],
     imports: []
   };
+  const actorCountWarningThreshold = opts.actorCountWarningThreshold ?? 10;
+  const labelLengthWarningThreshold = opts.labelLengthWarningThreshold ?? 36;
+  let actorCountWarned = false;
   let sceneFound = false;
   const lines = source.split(/\r?\n/);
   let inDef = null;
@@ -584,6 +619,9 @@ function parse(source, opts = {}) {
         throw new ParseError(`Invalid def body (expected "type(args)"): ${raw}`, lineNum);
       }
       const [, actorType, bodyArgsRaw] = bm;
+      if (!isKnownActorType(actorType)) {
+        throw new ParseError(`Unknown actor type in def body: "${actorType}"`, lineNum);
+      }
       const bodyArgs = bodyArgsRaw.trim() ? splitByComma(bodyArgsRaw).map((a) => a.trim()) : [];
       ast.defs[inDef.name] = {
         params: inDef.params,
@@ -738,7 +776,28 @@ function parse(source, opts = {}) {
       continue;
     }
     if (raw.startsWith("actor ")) {
-      parseActorLine(raw, lineNum, ast);
+      const actorInfo = parseActorLine(raw, lineNum, ast);
+      if (!actorCountWarned && actorCountWarningThreshold > 0) {
+        const actorCount = Object.keys(ast.actors).length;
+        if (actorCount > actorCountWarningThreshold) {
+          actorCountWarned = true;
+          ast.warnings.push({
+            kind: "actor-count-threshold",
+            message: `scene has ${actorCount} actors (threshold ${actorCountWarningThreshold}); consider splitting into multiple scenes`,
+            line: lineNum
+          });
+        }
+      }
+      if (labelLengthWarningThreshold > 0 && typeof actorInfo.args[0] === "string") {
+        const label = actorInfo.args[0];
+        if (label.length > labelLengthWarningThreshold) {
+          ast.warnings.push({
+            kind: "label-overflow",
+            message: `actor "${actorInfo.name}" label length ${label.length} exceeds ${labelLengthWarningThreshold}; consider shortening to reduce overlap`,
+            line: lineNum
+          });
+        }
+      }
       continue;
     }
     if (raw.startsWith("@")) {
@@ -780,9 +839,7 @@ function parseActorCallArgs(argsRaw) {
 function parseActorLine(raw, lineNum, ast) {
   const amAnchor = ACTOR_ANCHOR_POS_RE.exec(raw);
   const amNum = amAnchor ? null : ACTOR_NUM_POS_RE.exec(raw);
-  if (!amAnchor && !amNum) {
-    throw new ParseError(`Invalid actor declaration: ${raw}`, lineNum);
-  }
+  if (!amAnchor && !amNum) throw new ParseError(`Invalid actor declaration: ${raw}`, lineNum);
   let name;
   let typeName;
   let argsRaw;
@@ -827,7 +884,7 @@ function parseActorLine(raw, lineNum, ast) {
   const rawArgs = parseActorCallArgs(argsRaw);
   let resolvedType;
   let resolvedArgs;
-  if (BUILTIN_ACTOR_TYPES.has(typeName)) {
+  if (isKnownActorType(typeName)) {
     resolvedType = typeName;
     resolvedArgs = rawArgs;
   } else if (ast.defs[typeName]) {
@@ -868,6 +925,7 @@ function parseActorLine(raw, lineNum, ast) {
     ...modifiers,
     ...anchor ? { anchor } : {}
   };
+  return { name, args: resolvedArgs };
 }
 function parseEventLine(raw, lineNum, ast, inChapter, topScope) {
   const scope = inChapter?.scope ?? topScope;
@@ -909,7 +967,7 @@ function parseEventLine(raw, lineNum, ast, inChapter, topScope) {
   const mustUnderstand = actionToken.startsWith("!");
   const action = mustUnderstand ? actionToken.slice(1) : actionToken;
   if (actor === "camera") {
-    if (!CAMERA_ACTIONS.has(action)) {
+    if (!isCameraAction(action)) {
       if (mustUnderstand) {
         throw new ParseError(`Unknown camera action "${action}"`, lineNum);
       }
@@ -998,7 +1056,7 @@ function pushEvent(ast, scope, ev) {
   if (endTime > scope.prevEnd) scope.prevEnd = endTime;
 }
 function validateActionForActor(action, actorDef, mustUnderstand, lineNum, warnings) {
-  if (FIGURE_ONLY_ACTIONS.has(action)) {
+  if (isFigureOnlyAction(action)) {
     if (actorDef.type !== "figure") {
       throw new ParseError(
         `action "${action}" is figure-only; actor type is "${actorDef.type}"`,
@@ -1050,5 +1108,6 @@ export {
   PRESETS,
   PRESET_NAMES,
   ParseError,
-  parse
+  parse,
+  registerActorPack
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@markdy/core",
-  "version": "0.7.10",
+  "version": "0.7.12",
   "description": "MarkdyScript parser and AST types — zero runtime dependencies.",
   "license": "MIT",
   "type": "module",