Skip to content

Bound retained recipe command evidence - #1947

Merged
chubes4 merged 2 commits into
mainfrom
fix/1945-recipe-range-attribution
Jul 21, 2026
Merged

Bound retained recipe command evidence#1947
chubes4 merged 2 commits into
mainfrom
fix/1945-recipe-range-attribution

Conversation

@chubes4

@chubes4 chubes4 commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • bound retained command strings, structure nodes, records, and truncation diagnostics before writing commands.jsonl and logs/commands.log
  • apply the same bounded execution projection to recipe-run --json while preserving recipe phase, step, and command attribution
  • report residual artifact allocation failures with operation, workflow step, command, artifact path, payload bytes, configured limits, and retained cause stack

Verification

  • npm run build
  • npm run test:runtime-command-artifact-bounds
  • npm run test:trusted-apply-artifact-channel
  • npm run test:artifact-result-envelope
  • npx tsx tests/recipe-run-summary-output.test.ts
  • npm run check reaches the pre-existing unrelated command-registry-smoke failure: wordpress.collect-workload-result outputShape should mention outputSchema id

Closes #1945

@chubes4

chubes4 commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator Author

Lab verification against the exact preserved failure recipe passed on 901af29a.

Reproduction:

node packages/cli/dist/index.js recipe-run \
  --recipe wp-codebox-static-site-fixture-matrix-batch-001-recovery-fse-pilot-build-theme.json \
  --artifacts wp-codebox-pr1947-replay-v2-artifacts \
  --json

Evidence:

  • Original recipe SHA-256: 24b68541f09f0f7eca623d080db53f73cac8b96590f24682f30d0807522af09e
  • Run: run_824bc4f471654260aa928c29e42444a3 on the first pass; corrected per-command-budget replay also exited 0
  • 10/10 recipe executions exited 0; all retained phase/step/command attribution
  • commands.jsonl: 2.4 MB, logs/commands.log: 1.1 MB, final JSON output: 2.8 MB
  • Only the oversized import command was truncated; later editor and visual evidence remained intact
  • Editor validation: 139 blocks, 0 invalid
  • Visual comparison completed and reported its independent site-quality result (13.0532%, 738,663 pixels) instead of failing artifact collection

This reproduces and clears the original Invalid string length boundary with the original 47 MB intake assets.

@chubes4
chubes4 merged commit fe3b781 into main Jul 21, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Attribute recipe-run Invalid string length failures to the failing step and payload

1 participant