Skip to content

Ignore it, it was opened by mistake so I have closed it myself, Sorry. - #303

Closed
dotrahat wants to merge 1 commit into
LoganRupe:multi-repo-workspacesfrom
dotrahat:fix/multi-repo-absolute-file-paths
Closed

dotrahat wants to merge 1 commit into
LoganRupe:multi-repo-workspacesfrom
dotrahat:fix/multi-repo-absolute-file-paths

Conversation

@dotrahat

@dotrahat dotrahat commented Sep 24, 2026 •

Copy link
Copy Markdown

Sorry, it was a mistake. Didn't mean to publish it yet

In a multi-repo project, agents quote paths the way git prints them,
relative to the repository they ran in. The client resolves a relative
path against the workspace root, so a link like `.graft/config.json`
failed with "Failed to read workspace file". The path is also ambiguous
when several repos hold a file at the same relative path.

Runtime instructions now tell a session that spans more than one repo
root to write absolute paths. Every provider adapter passes the flag.
Inline-code file links also get the repo roots, so an absolute path in a
repo outside the workspace root opens in that repo instead of as a
read-only host file.

Fixes #302

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UJCWNTsJkzwPag1ZX8GtRq
@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:M labels Sep 24, 2026
@dotrahat
dotrahat marked this pull request as draft September 24, 2026 18:56
@dotrahat dotrahat closed this Sep 24, 2026
@dotrahat
dotrahat deleted the fix/multi-repo-absolute-file-paths branch September 24, 2026 19:08
@dotrahat dotrahat changed the title fix: multi-repo agents name files by absolute path so links open Ignore it, it was opened by mistake so I have closed it myself, Sorry. Sep 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant