Skip to content

Include bootstrap file path in truncation warning#133

Merged
dgarson merged 1 commit intodgarson/forkfrom
codex/add-file-path-to-log-message
Feb 24, 2026
Merged

Include bootstrap file path in truncation warning#133
dgarson merged 1 commit intodgarson/forkfrom
codex/add-file-path-to-log-message

Conversation

@dgarson
Copy link
Owner

@dgarson dgarson commented Feb 24, 2026

Motivation

  • The truncation warning emitted when large workspace bootstrap files are injected only included the filename, which made it hard to identify the exact file that was read; include the fully qualified path to make logs actionable.

Description

  • Modified the truncation warning in buildBootstrapContextFiles to include the resolved file path so the message now contains (${pathValue}) alongside the filename.
  • Updated the test src/agents/pi-embedded-helpers.buildbootstrapcontextfiles.test.ts to assert the warning includes the full path (/tmp/AGENTS.md).

Testing

  • Ran pnpm -s vitest run src/agents/pi-embedded-helpers.buildbootstrapcontextfiles.test.ts, but the test run failed to start in this environment due to a missing dependency (ipaddr.js) during module import, so the test could not complete.

Codex Task

@dgarson dgarson merged commit 070ff0d into dgarson/fork Feb 24, 2026
2 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant