fix: use cwd-based path resolution to handle Windows and Linux home paths (#42)#54
Merged
fix: use cwd-based path resolution to handle Windows and Linux home paths (#42)#54
Conversation
Windows paths (C:\Users\...) were passing through unsanitized, leaking usernames and directory structures. Normalize backslashes and strip drive-letter prefixes before applying existing Unix path extraction. Closes #42 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…lution Replace standalone toRelativePath/sanitizeResult/transformToolUse/enhanceToolOutput with createTransforms(workingDir) factory. Uses path.posix.relative against the JSONL cwd field instead of regex guessing, properly handling Windows paths (C:\...) and Linux /home/ dirs. Adds extractWorkingDir to scan JSONL for working directory. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
vtemian
added a commit
that referenced
this pull request
Feb 18, 2026
fix: use cwd-based path resolution to handle Windows and Linux home paths (#42)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Replaces the fragile regex-based path stripping in the session pipeline with a
createTransforms(workingDir)factory pattern. Instead of guessing how to strip a path prefix, the factory captures thecwdfield from the JSONL stream and usespath.posix.relativeto produce correct relative paths — regardless of whether the source was Windows (C:\Users\vlad\...) or a non-standard Linux home (/home/vlad/...).Closes #42.
Changes
transforms.ts- Replaced standalone transform functions withcreateTransforms(workingDir)factory; usespath.posix.relative(workingDir, absPath)for all path normalizationpipeline.ts- ThreadsworkingDirinto the factory on stream initialization, sourced from the JSONLcwdfieldindex.ts- Updated public API to expose factory patterntransforms.test.ts- Fully rewritten to cover Windows paths (C:\...), Linux home dirs, edge cases, and the factory contractindex.test.ts- New integration-level tests covering end-to-end path resolution through the pipelineClaudebin Session
🔗 Session Link
Testing
C:\Users\...paths, non-root Linux home dirs (/home/vlad/...), and paths outside the working directoryindex.test.tsintegration tests exercise the full pipeline with realistic JSONL inputbun checkandbun type-checkpass locallyChecklist
bun checkpassesbun type-checkpasses