Skip to content

Fix tool edge cases - #1

Closed
rizwan3d wants to merge 5 commits into
masterfrom
codex/fix-tool-edge-cases
Closed

rizwan3d wants to merge 5 commits into
masterfrom
codex/fix-tool-edge-cases

Conversation

@rizwan3d

Copy link
Copy Markdown
Owner

Summary

Fix several tool/runtime edge cases:

  • allow empty file writes for truncation and placeholder files
  • cap captured process output before it can grow unbounded in memory
  • preserve final-newline changes in apply_patch updates
  • write workspace files with explicit BOM-less UTF-8
  • harden recursive workspace traversal against unreadable paths

The empty-response fallback/session-history issue was already fixed in the current branch baseline; this branch preserves that behavior.

Validation

  • dotnet test NanoAgent.slnx from a clean temp copy: 274 passed, 0 failed

@rizwan3d rizwan3d closed this Apr 23, 2026
@rizwan3d
rizwan3d deleted the codex/fix-tool-edge-cases branch April 23, 2026 03:16
@rizwan3d rizwan3d changed the title [codex] Fix tool edge cases Fix tool edge cases Apr 25, 2026
@rizwan3d
rizwan3d restored the codex/fix-tool-edge-cases branch August 14, 2026 15:23
@rizwan3d
rizwan3d deleted the codex/fix-tool-edge-cases branch August 14, 2026 15:26
@rizwan3d
rizwan3d restored the codex/fix-tool-edge-cases branch August 14, 2026 15:28
@rizwan3d
rizwan3d deleted the codex/fix-tool-edge-cases branch August 14, 2026 15:29
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