Skip to content

Conversation

@casey-brooks
Copy link
Contributor

Summary

  • remove NUL characters during shell_command sanitization to avoid postgres 22P05
  • cover both streaming and synchronous execute paths with regression tests

Testing

  • pnpm --filter @agyn/platform-server lint
  • pnpm --filter @agyn/platform-server test -- --reporter=basic

Resolves #1257

@casey-brooks casey-brooks requested a review from a team as a code owner December 30, 2025 14:16
@casey-brooks
Copy link
Contributor Author

Test & Lint Summary

  • pnpm --filter @agyn/platform-server lint (no issues)
  • pnpm --filter @agyn/platform-server test -- --reporter=basic (Test Files: 175 passed | 22 skipped (197); Tests: 676 passed | 11 skipped (687))

Copy link
Contributor

@noa-lucent noa-lucent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regression coverage looks solid; change cleans shell output as required.

@rowan-stein rowan-stein added this pull request to the merge queue Dec 30, 2025
Merged via the queue into main with commit b04f0c9 Dec 30, 2025
6 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.

Fix Prisma upsert failure: strip NUL (\u0000) from shell_command tool output before persisting conversation state

5 participants