rig-parts 2.4.0 → 2.4.1

package/cli.ts

@@ -886,9 +886,12 @@ usage:
       the seam split by pair on a pass too, under check/seam/; the setup-pose
       still under check/setup_still/). check's --rig and --parts are refused:
       build's check reads the rig and the parts it wrote, <out>/rig and <out>.
-      build's --source is the painting assemble reads and is not handed to
-      the check, so check.json carries no SETUP_POSE_VS_SOURCE; run check
-      --rig <out>/rig --parts <out> --source <painting> for that line.
+      build's --source is the painting assemble reads; it is handed to the
+      check only when the requirements file declares setup_painting, which
+      needs it (CHECK_SETUP_PAINTING_SOURCE otherwise), so the verdict is
+      check --source's. Without that declaration check.json carries no
+      SETUP_POSE_VS_SOURCE; run check --rig <out>/rig --parts <out> --source
+      <painting> for that line.
       --idle-keys is forwarded to the rig stage (as for rig) and defaults to
       ctl; the first line names it when it is direct. A config with a
       two-bone ik over chain links the idle keys needs --idle-keys direct:

package/docs/AUTHORING.md

@@ -3580,8 +3580,12 @@ full loader before its assemble stage writes anything.
    `check --seam-pairs` writes on the same rig; `--out` is `build`'s (the check
    writes under `<out>/check`); `--rig` and `--parts` are refused, because the
    check reads the rig and the parts the build wrote (`<out>/rig`, `<out>`);
-   `--source` is the painting assemble reads and is not handed to the check, so
-   `check.json` carries no `SETUP_POSE_VS_SOURCE` — run
+   `--source` is the painting assemble reads; it is handed to the check only
+   when the requirements file (`--requirements` or `config.requirements`)
+   declares `setup_painting`, which cannot be measured without it (issue #978),
+   so that declaration reads the same verdict under `build` as under
+   `check --source`. Without the declaration `check.json` carries no
+   `SETUP_POSE_VS_SOURCE` and no byte moves — run
    `rig-parts check --rig out/rig --parts out --out <dir> --source inputs/painting.png`
    for that line.
    `rig-parts compare --left config.json --right out/rig/rig.json` reads the rig

package/docs/STABILITY.md

@@ -543,7 +543,11 @@ or region `max_pixels` is refused by name as `SETUP_PAINTING_REQUIRED_LIMIT`
 when the file is read, before anything is built, and no limit is derived; a
 fully limited declaration measures, prints and writes the same bytes as
 without the field. Absent or `false`, every reading in this section is
-unchanged. The optional `pinholes` key (issue #944, additive) declares
+unchanged. `build` hands its `--source` painting to its check when, and only
+when, the requirements file it reads declares `setup_painting` (issue #978),
+so the declaration reads the verdict `check --source` reads on the same output;
+a build whose requirements do not declare it, or that reads none, writes the
+bytes it wrote before. The optional `pinholes` key (issue #944, additive) declares
 `assemble`'s pinhole bar: with `{"required": true}`, `assemble` (and `build`'s
 assemble) refuses every pinhole of the base set's flat parts stack by name as
 `ASSEMBLE_PINHOLES` before anything is written — the bar is 0 and takes no

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rig-parts",
-  "version": "2.4.0",
+  "version": "2.4.1",
   "description": "AI-authored 2D character parts from one anime painting and its See-through layer decomposition: measured parts, weighted-mesh rig specs, verified through rig-c before anything is written. A CLI for agents that cannot see the image.",
   "type": "module",
   "bin": {
@@ -68,7 +68,7 @@
   },
   "dependencies": {
     "ag-psd": "^31.0.2",
-    "@rig-c/core": "3.1.1"
+    "@rig-c/core": "3.2.0"
   },
   "devDependencies": {
     "@esotericsoftware/spine-core": "4.3.13",

package/src/build.ts

@@ -2291,7 +2291,7 @@ export const CHECK_FLAGS_IN_BUILD: Readonly<Record<string, CheckFlagInBuild>> =
   '--rig': { as: 'supplied', why: 'its check reads the rig its rig stage writes, <out>/rig; run check --rig to measure another rig' },
   '--out': { as: 'own', why: 'build\'s own --out; its check writes under <out>/check' },
   '--parts': { as: 'supplied', why: 'its check reads the parts its assemble stage writes, <out>; run check --parts to measure against others' },
-  '--source': { as: 'own', why: 'the painting assemble reads; it is not handed to the check, so check.json carries no SETUP_POSE_VS_SOURCE — run check --rig <out>/rig --parts <out> --source <painting> for that line' },
+  '--source': { as: 'own', why: 'the painting assemble reads; it is handed to the check only when the requirements file declares setup_painting (issue #978), so check.json otherwise carries no SETUP_POSE_VS_SOURCE — run check --rig <out>/rig --parts <out> --source <painting> for that line' },
   '--page-edges': { as: 'forwarded' },
   '--pack-shape': { as: 'forwarded' },
   '--requirements': { as: 'forwarded' },
@@ -2434,6 +2434,9 @@ export function build(input: BuildInput, run: BuildRunners, log: Log): BuildResu
 
   let req: BuildRequirements | null = null;
   let configRecord: InputFileRecord | null = null;
+  // Issue #978: the painting is handed to the check only when the requirements file declares setup_painting, which
+  // cannot be measured without it; a build without that declaration hands it nothing, as before, and writes the same bytes.
+  let paintingForCheck: string | null = null;
   try {
     // build runs rig next, which needs the whole config, so the full loader is
     // asked first: a config rig would refuse is refused before assemble writes
@@ -2447,7 +2450,7 @@ export function build(input: BuildInput, run: BuildRunners, log: Log): BuildResu
     configRecord = inputFileRecord(loaded.bytes, input.config);
     req = buildRequirements(input.requirements, loaded.config.requirements, input.config);
     // The same for the requirements file's own shape; its names resolve against the rig, in the check, before it builds.
-    if (req !== null) readRequirements(req.read);
+    if (req !== null && readRequirements(req.read).setup_painting !== undefined) paintingForCheck = input.source;
     // And the schedule (issue #538), by rigc's own reader, as check reads it.
     if (input.environment !== undefined) readCheckEnvironment(input.environment);
     // And the steady-wind band spec (issue #590), as check reads it.
@@ -2469,7 +2472,7 @@ export function build(input: BuildInput, run: BuildRunners, log: Log): BuildResu
 
   let report: CheckReport;
   try {
-    report = stages.check({ rig: join(out, 'rig'), parts: out, out: join(out, 'check'), pageEdges: input.pageEdges, packShape: input.packShape, ...(req === null ? {} : { requirements: req.read, requirementsAs: req.recorded }), seamPairs: input.seamPairs === true, keepStills: input.keepStills === true, noExtremes: input.noExtremes === true, ...(input.set === undefined ? {} : { set: input.set }), allSets: input.allSets === true, ...(input.environment === undefined ? {} : { environment: input.environment }), ...(input.windBand === undefined ? {} : { windBand: input.windBand }), ...(configRecord === null ? {} : { config: configRecord }) }, run.check, run.checkBin, prefixed('check'));
+    report = stages.check({ rig: join(out, 'rig'), parts: out, out: join(out, 'check'), pageEdges: input.pageEdges, packShape: input.packShape, ...(req === null ? {} : { requirements: req.read, requirementsAs: req.recorded }), ...(paintingForCheck === null ? {} : { source: paintingForCheck }), seamPairs: input.seamPairs === true, keepStills: input.keepStills === true, noExtremes: input.noExtremes === true, ...(input.set === undefined ? {} : { set: input.set }), allSets: input.allSets === true, ...(input.environment === undefined ? {} : { environment: input.environment }), ...(input.windBand === undefined ? {} : { windBand: input.windBand }), ...(configRecord === null ? {} : { config: configRecord }) }, run.check, run.checkBin, prefixed('check'));
   } catch (err) {
     return refused('check', err);
   }