Skip to content

ctx-mcp-bridge: normalize tool arg paths (path/under + glob) to workspace-relative#80

Merged
m1rl0k merged 2 commits intoContext-Engine-AI:testfrom
voarsh2:mcp-bridge-path-normalisation
Dec 18, 2025
Merged

ctx-mcp-bridge: normalize tool arg paths (path/under + glob) to workspace-relative#80
m1rl0k merged 2 commits intoContext-Engine-AI:testfrom
voarsh2:mcp-bridge-path-normalisation

Conversation

@voarsh2
Copy link
Copy Markdown
Contributor

@voarsh2 voarsh2 commented Dec 18, 2025

  • Add request-side argument remapping in the MCP bridge so tool calls accept repo-prefixed and host-absolute workspace paths.
  • Normalize path/under/root/subdir plus path_glob/not_glob (preserve leading '!').
  • Exclude workspace/workspace_path/workspacePath from mapping to avoid semantic ambiguity.
  • Add TODO note about making glob normalization more conservative if needed.

…pace-relative

- Add request-side argument remapping in the MCP bridge so tool calls accept repo-prefixed and host-absolute workspace paths.
- Normalize path/under/root/subdir plus path_glob/not_glob (preserve leading '!').
- Exclude workspace/workspace_path/workspacePath from mapping to avoid semantic ambiguity.
- Add TODO note about making glob normalization more conservative if needed.
- Add a shared helper to parse /work/<workspace>/... into repo-relative paths
- Reuse it across arg normalization and result path remapping
- No behavioral change intended; reduces duplicated string-splitting logic
@voarsh2 voarsh2 marked this pull request as ready for review December 18, 2025 07:51
@m1rl0k m1rl0k merged commit 3f5eaf7 into Context-Engine-AI:test Dec 18, 2025
1 check passed
@voarsh2 voarsh2 deleted the mcp-bridge-path-normalisation branch December 18, 2025 20:36
m1rl0k added a commit that referenced this pull request Mar 1, 2026
ctx-mcp-bridge: normalize tool arg paths (path/under + glob) to workspace-relative
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.

3 participants