Skip to content

Reuse the JSON pointer token buffer - #2041

Merged
Aaronontheweb merged 1 commit into
devfrom
fix/json-pointer-token-allocation
Aug 20, 2026
Merged

Reuse the JSON pointer token buffer#2041
Aaronontheweb merged 1 commit into
devfrom
fix/json-pointer-token-allocation

Conversation

@Aaronontheweb

Copy link
Copy Markdown
Collaborator

Summary

  • Reuse one StringBuilder across JSON pointer token decoding.
  • Preserve strict RFC 6901 validation and projection behavior.
  • Resolve the allocation alert from PR Add bounded workspace tools #2037.

Validation

  • dotnet build Netclaw.slnx -c Release --no-restore
  • dotnet test src/Netclaw.Actors.Tests/Netclaw.Actors.Tests.csproj -c Release --no-build --filter FullyQualifiedName~StructuredWorkspaceToolTests (18 passed)
  • dotnet slopwatch analyze --fail-on warning
  • pwsh ./scripts/Add-FileHeaders.ps1 -Verify
  • git diff --check

The eval suite was not run. This change does not alter a tool schema or agent behavior.

@Aaronontheweb Aaronontheweb added performance Memory, CPU, and I/O optimization issues. tools Issues related to agent tools: file_read, web_search, shell_execute, image processing, etc. labels Aug 20, 2026
@Aaronontheweb
Aaronontheweb merged commit 686670a into dev Aug 20, 2026
23 checks passed
@Aaronontheweb
Aaronontheweb deleted the fix/json-pointer-token-allocation branch August 20, 2026 14:05
@Aaronontheweb Aaronontheweb mentioned this pull request Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance Memory, CPU, and I/O optimization issues. tools Issues related to agent tools: file_read, web_search, shell_execute, image processing, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant