Skip to content

Fix/directory handling combined#256

Closed
shuv1337 wants to merge 3 commits intoLatitudes-Dev:devfrom
shuv1337:fix/directory-handling-combined
Closed

Fix/directory handling combined#256
shuv1337 wants to merge 3 commits intoLatitudes-Dev:devfrom
shuv1337:fix/directory-handling-combined

Conversation

@shuv1337
Copy link
Collaborator

@shuv1337 shuv1337 commented Jan 3, 2026

No description provided.

shuv1337 and others added 3 commits January 2, 2026 16:20
When attaching to a remote server, the TUI wasn't properly passing the
working directory context. This fix ensures consistent directory
handling by:
- Passing --dir argument from attach command through TUI to SDK client
- Using sync.data.path.directory for file autocomplete URLs
- Using sync.data.path.directory for custom theme discovery
- Using sync.data.path.directory for exports and path normalization
- Adding normalizePathFromDirectory utility function
Tests for the normalizePathFromDirectory helper function covering:
- Converting absolute paths to relative paths
- Preserving already-relative paths unchanged
- Handling input equal to base directory (returns '.')
- Handling empty input (returns '')
@greptile-apps
Copy link

greptile-apps bot commented Jan 4, 2026

Greptile encountered an error while reviewing this PR. Please reach out to support@greptile.com for assistance.

@shuv1337 shuv1337 closed this Jan 4, 2026
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