spine-rigc 1.0.1 → 1.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spine-rigc",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Rig compiler for Spine — declarative rig specs in, Spine 4.3 skeleton data out, verified by a spine-core round-trip. Built so AI agents can author rigs and check their own work; the output imports into the Spine editor.",
   "type": "module",
   "bin": {

package/src/atlas.ts

@@ -431,11 +431,12 @@ export function rewritePageNames(parsed: ParsedAtlas, rename: (name: string, ind
  *
  * This is what `--atlas-in` re-emits (issue #803). A pack's blank lines are its
  * packer's, not its content: a 3.8-era packer begins every file with one, and
- * `A07_ATLAS_TEXT_SHAPE` — which checks the text rigc writes — refused the
- * re-emission at `line 1`. What makes dropping them safe is what they mean to
- * the reader that owns the format: in `TextureAtlas` a run of blank lines ends a
- * page block exactly as one blank line does, and a run before the first page is
- * read as nothing. Measured through the runtime, `"\n" + text`, `"\n\n" + text`
+ * `A07_ATLAS_TEXT_SHAPE` — which checks the text rigc writes — refuses a blank
+ * line before the first page block, two side by side, and one after the last
+ * page block, each by its own sentence. What makes dropping them safe is what
+ * they mean to the reader that owns the format: in `TextureAtlas` a run of
+ * blank lines ends a page block exactly as one blank line does, and a run before
+ * the first page is read as nothing. Measured through the runtime, `"\n" + text`, `"\n\n" + text`
  * and `text` load to the same pages and regions (`PKR63`, `PKR64`).
  *
  * ⚠️ One reading is NOT the same, and it moves toward rigc's: the runtime's
@@ -523,9 +524,9 @@ export const PACK_NO_ROTATE = 0;
  * the trailing newline was appended unconditionally. That one byte contradicts
  * the paragraph above it: a blank line is the separator that sits BETWEEN page
  * blocks, so a file consisting of one is a separator with nothing on either side.
- * `A07_ATLAS_TEXT_SHAPE` then read it as a malformed page block and refused the
- * compiler's own output, which is how a rig with nothing to draw became a red
- * gate on a file nobody had written wrong.
+ * `A07_ATLAS_TEXT_SHAPE` reads a file with no non-blank line as having no page
+ * block and reports SKIP, and refuses a blank line that trails a page block as
+ * `the file ends with a blank line`.
  *
  * The runtime cannot tell the two apart — `new TextureAtlas('')`,
  * `new TextureAtlas('\n')` and `new TextureAtlas('\n\n')` all come back with

package/src/compile.ts

@@ -3869,7 +3869,7 @@ function deferredAttachmentRefusal(type: string, where: string, how: string): st
   return (
     `${where}: this attachment is a "${type}"${how} — ${DEFERRED_ATTACHMENTS[type]}. ` +
     `rigc does not emit it yet, deliberately: it emits ${EMITTED_ATTACHMENT_TYPES.join(', ')}, and a point ` +
-    'appears nowhere in the benchmark corpus (docs/SPEC_COVERAGE.md parts 3-1 and 4-2), so it is not on the ' +
+    'appears nowhere in the benchmark corpus, so it is not on the ' +
     "ladder's critical path. docs/SPEC_COVERAGE.md part 1-6 is the row this sentence reads " +
     'from, and it is what an implementation would have to carry.'
   );

package/src/generation.ts

@@ -115,7 +115,8 @@ export function spineGeneration(version: string): SpineGeneration | null {
 export const TOPLEVEL_CONSTRAINT_ARRAYS: readonly string[] = ['ik', 'transform', 'path', 'physics', 'slider'];
 
 /**
- * The bone key 4.2 spelled `transform` and 4.3 spells `inherit`.
+ * The bone key 4.0 and 4.1 spelled `transform`, renamed `inherit` in 4.2 and
+ * spelled that way by 4.3.
  *
  * The old key does not throw and does not warn — it is an unknown field, so the
  * bone falls back to Normal inheritance (#706 row 6, and

package/src/ladder.ts

@@ -34,7 +34,7 @@ export interface Rung {
   id: string;
   /** Directory under `examples/`. */
   example: string;
-  /** One line on what this rung is testing, from SPEC_COVERAGE part 4-1. */
+  /** One line on what this rung is testing, from docs/SURVEY_2026-08-22.md part 4-1. */
   gates: string;
   skeletons: RungSkeleton[];
 }

package/src/rig.ts

@@ -989,10 +989,10 @@ export interface RigPathAttachment extends RigVertexGeometry {
  * an unknown `type` and drop the attachment without a word
  * (`SkeletonJson.ts:653`).
  *
- * 🚧 It appears nowhere in the benchmark corpus (SPEC_COVERAGE parts 3-1 and
- * 4-2), so it is not on the ladder's critical path — which is the reason it is
- * deferred rather than an oversight. `linkedmesh` stood here beside it until
- * issue #691; `RigLinkedMeshAttachment` is the shape that replaced it.
+ * 🚧 It appears nowhere in the benchmark corpus, so it is not on the ladder's
+ * critical path — which is the reason it is deferred rather than an oversight.
+ * `linkedmesh` stood here beside it until issue #691; `RigLinkedMeshAttachment`
+ * is the shape that replaced it.
  */
 export interface RigUnimplementedAttachment {
   type: 'point';

package/src/validate.ts

@@ -1205,22 +1205,27 @@ export function validate(input: ValidateInput): ValidateReport {
       // for want of a SUBJECT only when its reason is one of those constants.
       return skip('A07_ATLAS_TEXT_SHAPE', SKIP_NO_ATLAS_PAGE);
     }
-    // A blank line before the first page name is named as that (issue #803). It
-    // used to reach the loop below with `expectPage` already true and be read as
-    // a SECOND blank line — `line 1: consecutive blank lines`, where there is no
-    // line 0 to be consecutive with — and a run of k of them printed k findings.
-    // The verdict is unchanged: rigc writes no leading blank line
-    // (`writeAtlasText`, and `--atlas-in` through `canonicalAtlasShape`), so a file
-    // that has one was written by something else and is still refused. What
-    // changed is that the run is one finding, stating what it is and how long.
+    // A blank line at either END of the file is named as that end (issues #803,
+    // #810), and a run of them is one finding stating its length. They are taken
+    // off both ends before the walk below, so the walk sees the page blocks and
+    // the blank lines between them and nothing else: a blank line after the last
+    // region is not a page block, and "consecutive blank lines" and "the last
+    // page block declares no region" are about blocks. `TextureAtlas` reads a
+    // blank at either end as nothing (`TextureAtlas.js:98-100` skips the leading
+    // run, `:116-118` ends a page block on each blank line), and rigc writes
+    // neither (`writeAtlasText`, and `--atlas-in` through `canonicalAtlasShape`),
+    // so a file that has one was written by something else and is refused.
     let leading = 0;
     while (leading < atlasLines.length && atlasLines[leading].trim().length === 0) leading++;
     if (leading > 0) {
       fail('A07_ATLAS_TEXT_SHAPE', `line 1: the file begins with ${leading === 1 ? 'a blank line' : `${leading} blank lines`}`);
     }
+    let trailing = 0;
+    while (trailing < atlasLines.length - leading && atlasLines[atlasLines.length - 1 - trailing].trim().length === 0) trailing++;
+    const blockEnd = atlasLines.length - trailing;
     let expectPage = true;
     let sawRegionForPage = false;
-    for (let i = leading; i < atlasLines.length; i++) {
+    for (let i = leading; i < blockEnd; i++) {
       const line = atlasLines[i];
       if (line.trim().length === 0) {
         if (expectPage) fail('A07_ATLAS_TEXT_SHAPE', `line ${i + 1}: consecutive blank lines`);
@@ -1242,8 +1247,9 @@ export function validate(input: ValidateInput): ValidateReport {
       }
       sawRegionForPage = true;
     }
-    if (!sawRegionForPage && atlasLines.length) {
-      fail('A07_ATLAS_TEXT_SHAPE', 'the last page block declares no region');
+    if (!sawRegionForPage) fail('A07_ATLAS_TEXT_SHAPE', 'the last page block declares no region');
+    if (trailing > 0) {
+      fail('A07_ATLAS_TEXT_SHAPE', `line ${blockEnd + 1}: the file ends with ${trailing === 1 ? 'a blank line' : `${trailing} blank lines`}`);
     }
   });
 
@@ -1904,14 +1910,14 @@ export function validate(input: ValidateInput): ValidateReport {
   });
 
   // --- A02: no bone.transform key ------------------------------------------
-  // 4.3 renamed it to `inherit`; the old key loads and silently falls back to
-  // Normal inheritance (case 6b). The key itself is `generation.ts`'s, for
+  // 4.2 renamed it to `inherit`, and 4.3 kept that spelling; the old key loads
+  // and silently falls back to Normal inheritance (case 6b). The key itself is `generation.ts`'s, for
   // A01's reason.
   check('A02_NO_BONE_TRANSFORM_KEY', () => {
     const bones = Array.isArray(raw?.bones) ? (raw.bones as unknown[]) : [];
     for (const bone of bones) {
       if (isObj(bone) && LEGACY_BONE_INHERIT_KEY in bone) {
-        fail('A02_NO_BONE_TRANSFORM_KEY', `bone "${String(bone.name)}" uses 4.2's "transform"; 4.3 wants "inherit"`);
+        fail('A02_NO_BONE_TRANSFORM_KEY', `bone "${String(bone.name)}" uses "transform", the key 4.0 and 4.1 spelled; 4.2 and 4.3 spell it "inherit"`);
       }
     }
   });