Skip to content

fix: scrub public and persisted workflow output#111

Merged
andrei-hasna merged 3 commits into
mainfrom
fix/pr66-output-scrub-v2
Jul 20, 2026
Merged

fix: scrub public and persisted workflow output#111
andrei-hasna merged 3 commits into
mainfrom
fix/pr66-output-scrub-v2

Conversation

@andrei-hasna

Copy link
Copy Markdown
Contributor

Replacement for the bounded confidentiality gaps identified on stale PR #66.

Summary:

  • re-scrub public show-output and human text stdout/stderr/error surfaces, and describe MCP output as scrubbed
  • route SQLite and Postgres workflow recover/skip/finalize/cancel/skipped-run reasons and errors through shared scrub helpers with parity
  • deep-scrub Postgres goal evidence and raw responses before JSON stringify, matching SQLite
  • add focused public-format, SQLite, Postgres, and MCP schema regressions

Validation:

  • focused: 87 pass, 29 environment-gated Postgres skips, 0 fail
  • full suite: 931 pass, 29 environment-gated Postgres skips, 0 fail
  • bun run typecheck
  • bun run build
  • staged and committed credential-pattern scans passed

Explicitly excluded: historical output quarantine and cross-package follow-ups.

@andrei-hasna
andrei-hasna merged commit 36563ca into main Jul 20, 2026
7 of 8 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.

1 participant