Repository navigation
Expand --report JSON into a compliance audit sidecar - #83
Merged
Merged
Conversation
Record Dumpling version, run id/timestamp, config path + checksum, streaming I/O SHA-256, dump-seal digest, gate flags, and coverage/scan outcomes in the JSON --report so reviewers do not reconstruct chain-of- custody from CI logs. Instance fields (run_id, started_at) differ across re-runs; seal_sha256 matches the dump-seal sha256= field. Co-authored-by: andy <andy@wearecrew.com>
ababic
marked this pull request as ready for review
August 14, 2026 16:19
Skip the dump-seal SQL comment for stdin/stdout pipelines while still recording seal_sha256 in --report. Document both flags in --help Examples, the configuration guide, README, and CI audit-evidence docs. Co-authored-by: andy <andy@wearecrew.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implements the narrower scope of #13: treat dump seals as provenance on the SQL artifact, and expand
--reportinto the compliance sidecar (not a second copy of the policy fingerprint).--reportaudit sidecarWhen
--reportis set, the JSON file records:dumpling_versionrun_id/started_at(instance metadata; will not match on re-runs)config_source+config_sha256(SHA-256 of the loaded config file bytes)input_sha256/output_sha256of the SQL streams Dumpling actually read/wrote (output_sha256omitted in--check)seal_sha256— the same digest as dump-sealsha256=(cross-link report ↔ dump)flagsfor gates (check,strict_coverage,scan_output,fail_on_findings,no_seal, …)outcomes(strict_coverage_passed,output_scan_passedwhen those gates ran,trusted_passthrough)I/O hashing is enabled only when
--reportis requested.--no-sealSkip writing the
-- dumpling-seal:SQL comment (useful for stdin/stdout pipelines). Incoming seals are still recognized.--reportstill recordsseal_sha256.Docs
dumpling --helplong help for--report/--no-seal, plus an Examples block--no-sealTesting
run_id;--checkomitsoutput_sha256;--no-sealomits the SQL comment but keepsseal_sha256--helpcontains Examples,--no-seal, and--reportcargo fmt --all -- --check,cargo clippy --all-targets --all-features,cargo test --all-targets --all-features(172 tests)Closes #13