Skip to content

feat: add CLI falsifiers and exit tables to the remaining twelve examples - #144

Merged
TMHSDigital merged 2 commits into
mainfrom
feat/falsifier-retrofit-batch-4
Sep 13, 2026
Merged

feat: add CLI falsifiers and exit tables to the remaining twelve examples#144
TMHSDigital merged 2 commits into
mainfrom
feat/falsifier-retrofit-batch-4

Conversation

@TMHSDigital

Copy link
Copy Markdown
Owner

Summary

Closes the example-falsifier retrofit on both axes: 55/55 flags and 55/55 README exit tables. Two commits: B1 behavior, B2 docs-only.

Issue #138's checker is still out of scope. This PR is what makes it buildable as an enforcing gate.

Part A already landed (context, not this PR)

#143 squash-merged as 89ae4eb (feat: add CLI falsifiers and exit tables to thirteen rigging examples). Release cut v0.60.0. Pages deployed. Smoke did not re-run on push (intentional). #131 and #138 remain OPEN.

B1 — final 12 falsifiers

Same treatment as batches 1–3: a flag that changes the input so an existing assertion fails. No assertion skipped, disabled, or bypassed. No exit-code value changed. Grease Pencil version shims were left alone; both GP flags are version-independent RNA (use_contour, cyclic). --check-pixels on vse-cut-list stays a second witness; the falsifier beside it is --swap-inputs.

Live-run-proven on local binaries (default must be 0; falsifier must be the documented code on all three). Falsifier was never 0.

Example Flag Lands 4.5.11 (4db51e9d1e1e) 5.1.2 (ec6e62d40fa9) 5.2.1 (9e2066aef7ef)
compositor-glare --threshold-high 3 0 / 3 0 / 3 0 / 3
curve-bevel-arc --no-caps 6 0 / 6 0 / 6 0 / 6
gp-lineart-contour --no-contour 4 0 / 4 0 / 4 0 / 4
grease-pencil-rosette --open-strokes 5 0 / 5 0 / 5 0 / 5
light-link-studio --skip-link 5 0 / 5 0 / 5 0 / 5
png-exr-alpha --opaque-alpha 5 0 / 5 0 / 5 0 / 5
shader-node-group --same-tint 6 0 / 6 0 / 6 0 / 6
sky-texture-sun-elevation --unlink-sky 6 0 / 6 0 / 6 0 / 6
text-version-stamp --wrong-body 4 0 / 4 0 / 4 0 / 4
turntable --no-keys 3 0 / 3 0 / 3 0 / 3
vse-cut-list --swap-inputs 7 0 / 7 0 / 7 0 / 7
vse-gamma-cross --swap-inputs 4 0 / 4 0 / 4 0 / 4

Binaries: .scratch/blender-4.5.11-windows-x64/blender.exe, .scratch/blender-5.1.2-windows-x64/blender.exe, .scratch/blender-5.2.1-windows-x64/blender.exe. 5.1 evidence here is local; CI 5.1 is the labeled job.

Falsifiers are not on the default smoke path. No catalog.json edits.

B2 — thirteen README exit tables (inspection-only)

No flags, no behavior, no code-value changes, no local Blender runs. Codes copied from each script as they are. export-preset-axis exiting 9 is correct.

Pathology / sidecar (no gallery, by design):

  • exit-pre-sidecar — 0, 1, 2, 77 (bullet list converted to the numbered table)
  • coincident-vert-weld — 0–4
  • gn-bundle-roundtrip — 0–5, 77
  • ngon-triangulate — 0–4
  • unapplied-scale-gltf — 0–4

Existing flags, missing tables:

  • cross-version-property-delete — 0–7, 10, 12
  • export-preset-axis — 0–11 (including 9)
  • gn-zone-iterate — 0–6, 10, 12
  • lightmap-uv-channel — 0–11
  • modular-kit-snap — 0–9, 11 (check_framing is called with a documented deviation, so 10 is not reachable)
  • socket-attach-points — 0–11
  • usd-export-evaluation-mode — 0–6, 10, 12
  • vertex-color-ao — 0–11

Final counts (this branch, before opening)

Falsifier flags 55/55:

python .scratch/count_falsifiers.py
# WITH extra flags 55
# WITHOUT 0
# total dirs 55

README exit tables matching CONTRIBUTING “Exit codes” / bake-normal (## Exit codes + | Code | Meaning |) 55/55:

python .scratch/count_exit_tables.py
# WITH exit table 55 / 55
# WITHOUT 0

Test plan

  • B1 default=0 and falsifier=documented code on 4.5.11, 5.1.2, 5.2.1 (live)
  • GP falsifiers did not need a per-version flag (live)
  • B2 tables inspection-only against the scripts (no Blender)
  • CI 4.5 + 5.2 smoke on the PR
  • CI 5.1 smoke via needs-5.1
  • Do not merge from this session

TMHSDigital and others added 2 commits September 13, 2026 10:27
…ples

Closes the last behavior gap in the retrofit: every example now has a flag that
makes an existing assertion fail, without touching Grease Pencil version shims
or changing any exit code value.

Signed-off-by: TMHSDigital <154358121+TMHSDigital@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Inspection-only documentation so every shipped example lists its file-local
nonzero codes. No flags, no behavior, no code-value changes.

Signed-off-by: TMHSDigital <154358121+TMHSDigital@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@TMHSDigital TMHSDigital added the needs-5.1 Opt-in Blender 5.1 smoke on this PR. Default matrix stays 4.5 + 5.2. Auto-label will not apply this. label Sep 13, 2026
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Sep 13, 2026
@TMHSDigital
TMHSDigital merged commit c4714a7 into main Sep 13, 2026
13 checks passed
@TMHSDigital
TMHSDigital deleted the feat/falsifier-retrofit-batch-4 branch September 13, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation needs-5.1 Opt-in Blender 5.1 smoke on this PR. Default matrix stays 4.5 + 5.2. Auto-label will not apply this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant