Skip to content

Resolve {project-root} by walking up to the nearest _bmad/ - #49

Merged
bmadcode merged 1 commit into
devfrom
project-root-walk-up
Sep 25, 2026
Merged

bmadcode merged 1 commit into
devfrom
project-root-walk-up

Conversation

@bmadcode

Copy link
Copy Markdown
Contributor

Every skill now says {project-root} is the nearest folder containing _bmad/, starting at the project working directory and moving up through its parents.

The old line said {project-root} paths resolve from the project working directory. That breaks when an agent starts inside a worktree or subfolder that has no _bmad/ of its own, such as oss/.worktrees/<repo>/<branch> with _bmad/ at oss/.

We tested both wordings in sandboxes with identical decoy installs, across Claude Code, Codex, opencode (GLM) and Antigravity models. The new wording found the right _bmad/ 43 of 45 times. The old wording managed 31 of 45 and wrote to the wrong copy 3 times.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Qrjf7zEvqbDcipTM3UYZuP

Skills started inside a worktree or subfolder now find the _bmad/ install above them instead of assuming the working directory.
@coderabbitai

coderabbitai Bot commented Sep 25, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 1130d59d-7ee1-496c-862a-7547e770698c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@bmadcode
bmadcode merged commit 8402061 into dev Sep 25, 2026
21 checks passed
@bmadcode
bmadcode deleted the project-root-walk-up branch September 25, 2026 10:18
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