Skip to content

fix: sanitize hidden terminal strings in web evidence - #705

Open
outlier27-cell wants to merge 2 commits into
openpi-dev:mainfrom
outlier27-cell:fix/web-evidence-terminal-strings
Open

outlier27-cell wants to merge 2 commits into
openpi-dev:mainfrom
outlier27-cell:fix/web-evidence-terminal-strings

Conversation

@outlier27-cell

Copy link
Copy Markdown
Contributor

Problem

Fixes #700.

Value

This is an evidence-backed source-boundary fix with deterministic regressions; malformed input no longer becomes a misleading successful projection.

Approach

Require the existing canonical types and limits at the Web boundary. Add regression tests that fail on origin/main and pass after the fix. See the linked docs/research record in this branch.

Validation

  • Targeted regression tests pass after failing on origin/main.
  • bun run check passed.
  • Full local bun run test was interrupted during machine-wide Windows resource contention; no full local pass is claimed. CI is the authoritative full matrix.

Impact

Existing valid behavior is preserved. Malformed evidence/configuration is rejected or remains unknown. No provider, Session, or orchestration surface is added.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Oct 7, 2026
@outlier27-cell

Copy link
Copy Markdown
Contributor Author

@tt-a1i 请审核此 PR。最新提交已通过完整 GitHub CI,当前可合并;如有需要调整的地方,我会在原分支及时修复。

This branch has not been deployed

No deployments
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(web): strip hidden terminal strings from displayed tool evidence

1 participant