spine-rigc 0.35.1 → 0.36.1

package/README.md

@@ -590,8 +590,6 @@ the file are **findings** with codes rather than plausible values: a construct t
 spec format cannot hold (`point`, a `sequence` block, an unknown field
 on a bone, slot or constraint) is a blocker, the command exits non-zero, and both
 specs are still written — a spec plus a list of what is missing from it beats no spec.
-One thing it drops on purpose and says so: a path attachment's `lengths`, which is
-`PathConstraint`'s own measurement and which rigc re-measures.
 
 ⚠️ **Two values are not in a skeleton at all.**
 
@@ -633,9 +631,12 @@ bun tools/editor_roundtrip.ts --build build/ --editor /Applications/Spine.app/Co
 
 It prints the import and export exit codes, the validator's verdict on the
 export, every `diff` measure that moved, `check`'s mean MAE and worst drift per
-animation **for each skin the build declares** — one render-and-check block per
-skin, with a per-skin roll-up under them, because a rig's contested art lives in
-its named skins and a single un-skinned check draws none of it — and a
+animation **for each skin the build and the export both declare** — one
+render-and-check block per skin, with a per-skin roll-up under them, because a
+rig's contested art lives in its named skins and a single un-skinned check draws
+none of it; a skin only one side declares is a FAIL naming it as **lost** (or
+**added**) **by the export**, with `diff`'s `attachments.skins` beside it, and is
+rendered on neither side ([#801](https://github.com/firejune/rigc/issues/801)) — and a
 field-by-field list of what the editor rewrote. Every step quotes what its child
 said when that child did not do what it was for, the renderers included; a skin
 **neither** side can draw — a hit-box rig, say — is a **SKIP** naming that, not a

package/cli.ts

@@ -2836,6 +2836,17 @@ function cmdExplain(flags: Record<string, string>): void {
   // relying on. Reading it by name means a change to the skins ORDER cannot turn
   // this line into a report about some other skin.
   const defaultSkin = result.skeleton.skins.find((skin) => skin.name === 'default');
+  // ...and a skeleton may declare none (issue #801), in which case every
+  // `attachments=[]` below is true and says nothing — so the line above the
+  // column says where the placeholders are instead of letting it read as
+  // "this rig has no art".
+  if (defaultSkin === undefined) {
+    const names = result.skeleton.skins.map((skin) => JSON.stringify(skin.name));
+    console.log(
+      `  (this skeleton declares no default skin, so the attachments column lists none; its placeholders are in ` +
+        `${names.length === 0 ? 'no skin at all' : `the named skin(s) ${names.join(', ')}`})`,
+    );
+  }
   for (const s of result.skeleton.slots) {
     const atts = Object.keys(defaultSkin?.attachments[s.name] ?? {});
     console.log(
@@ -3039,7 +3050,10 @@ function cmdExplain(flags: Record<string, string>): void {
       const curve = attachments.find((att) => (att as { type?: string }).type === 'path') as
         | { lengths?: number[]; closed?: boolean; constantSpeed?: boolean }
         | undefined;
-      const lengths = curve?.lengths ?? [];
+      // `vertexCount / 3` entries on both shapes since issue #804, so an open
+      // path's curves are one fewer than its entries and its length is the last
+      // CURVE's entry — the trailing one is the wrap-around curve nothing reads.
+      const lengths = (curve?.lengths ?? []).slice(0, curve?.closed ? undefined : -1);
       console.log(
         `  ${c.name.padEnd(12)} slot=${slot.padEnd(12)} bones=[${(c.bones as string[]).join(', ')}] ` +
           `position=${c.position ?? 0} ${String(c.positionMode ?? 'percent')}/${String(c.spacingMode ?? 'length')}/${String(c.rotateMode ?? 'tangent')}`,