Skip to content

🧶 fix: Large Tool Inputs Break Bash PTC Replay and Pending-Call Serialization - #75

Merged
danny-avila merged 3 commits into
mainfrom
danny-avila/ptc-large-tool-input
Sep 2, 2026
Merged

🧶 fix: Large Tool Inputs Break Bash PTC Replay and Pending-Call Serialization#75
danny-avila merged 3 commits into
mainfrom
danny-avila/ptc-large-tool-input

Conversation

@danny-avila

Copy link
Copy Markdown
Collaborator

I replayed the original three large Bash PTC commits from #48 onto the latest main while preserving the newer configuration coverage.

  • Retain hashed oversized tool inputs so Bash replay matches cached calls without re-invoking tools.
  • Reject multi-document tool input before pending-call serialization.
  • Configure positive per-result and aggregate PTC history byte caps with safe fallbacks.
  • Combine the PR's positive-integer configuration tests with the remote-bridge configuration coverage now on main.
  • Supersede fix: large tool inputs break bash PTC replay and pending-call serialization #48 because its contributor branch has maintainer edits disabled and cannot be updated by repository maintainers.

Change Type

  • Bug fix (non-breaking change which fixes an issue)

Testing

  • Ran bun test src/config.test.ts (10 passed).
  • Ran bun run scripts/test-ptc-replay-bash-smoke.ts (101 passed).
  • Ran bunx tsc --noEmit --pretty false; it reports four existing repository type/configuration errors unrelated to these five changed files.

Test Configuration:

  • Bun 1.3.13
  • Dependencies installed from service/bun.lock with bun install --frozen-lockfile

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • My changes do not introduce new warnings
  • I have written tests demonstrating that my changes are effective or that my feature works
  • Local focused tests pass with my changes

@danny-avila
danny-avila merged commit a7afac5 into main Sep 2, 2026
5 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.

3 participants