Skip to content

feat: log workspace file tool outcomes - #12

Merged
bonigo1998 merged 1 commit into
mainfrom
codex/file-tool-logs
Sep 27, 2026
Merged

bonigo1998 merged 1 commit into
mainfrom
codex/file-tool-logs

Conversation

@bonigo1998

Copy link
Copy Markdown
Owner

Adds tool_start, tool_success, and tool_error events for read_file calls executed by AgentLoop.

Records the model step, tool name, byte count, and error reason. Distinguishes file-read failures from oversized contents withheld from the model.

Validation:

  • Application builds and both existing CTest suites pass.
  • Successful read logged 103 bytes.
  • Missing-file case logged read_failed.
  • A 4,001-byte file logged file_too_large with the correct byte count.

These events cover model-requested file reads. The separate /read and /askfile commands are not instrumented.

@bonigo1998
bonigo1998 merged commit 7813f26 into main Sep 27, 2026
1 check 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.

1 participant