Skip to content

fix(guardrails): use project-scoped path for team worktree (Issue #155)#158

Merged
terisuke merged 1 commit intodevfrom
fix/yard-path-issue-155
Apr 10, 2026
Merged

fix(guardrails): use project-scoped path for team worktree (Issue #155)#158
terisuke merged 1 commit intodevfrom
fix/yard-path-issue-155

Conversation

@terisuke
Copy link
Copy Markdown

Summary

  • Fix yard() function in team.ts that was creating worktree directories in the parent directory (e.g. ~/Developer/.opencode-opencode-team) instead of within the project
  • Changed to path.join(dir, ".opencode", "team") to keep team data scoped inside the project's .opencode/ directory, matching the established pattern

Closes #155

Test plan

  • Verify yard() returns <project>/.opencode/team instead of <parent>/.<project>-opencode-team
  • Confirm yardadd() creates worktrees under .opencode/team/ within the project
  • Verify no regressions in team/background tool execution

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

The yard() function was creating directories in the parent directory
(e.g. ~/Developer/.opencode-opencode-team) instead of within the
project. Changed to use .opencode/team inside the project directory,
matching the established project-scoped pattern.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 10, 2026 08:35
@github-actions
Copy link
Copy Markdown

New PR opened -- automated review will run on the next push.

To trigger a manual review, comment /review on this PR.

@github-actions
Copy link
Copy Markdown

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@github-actions
Copy link
Copy Markdown

The following comment was made by an LLM, it may be inaccurate:

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Guardrails team plugin to place delegated git worktrees in a project-scoped directory (<project>/.opencode/team) rather than creating hidden directories alongside the project’s parent folder, reducing filesystem clutter and keeping state local to the project.

Changes:

  • Adjust yard() to return path.join(dir, ".opencode", "team") for team worktree placement.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@terisuke terisuke merged commit c2503fe into dev Apr 10, 2026
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: Team plugin yard() creates directories in parent directory instead of project-scoped location

2 participants