Skip to content

chore: add git worktree docs and roborev configuration#92

Open
ian-flores wants to merge 2 commits intomainfrom
add-worktree-docs-and-roborev
Open

chore: add git worktree docs and roborev configuration#92
ian-flores wants to merge 2 commits intomainfrom
add-worktree-docs-and-roborev

Conversation

@ian-flores
Copy link
Collaborator

Summary

  • Add Git Worktrees section to CLAUDE.md formalizing the namespaced worktree convention (team-operator-<branch>)
  • Add .roborev.toml configuring automatic code reviews with claude-code agent

Test plan

  • Verify worktree creation with namespaced path works
  • Verify just build and just test work in a worktree
  • Verify roborev triggers on commit (roborev status)

@claude
Copy link

claude bot commented Feb 18, 2026

Claude finished @ian-flores's task —— View job


Review Complete

  • Gather context and read review guidelines
  • Analyze the diff and changed files
  • Submit review with inline feedback

Reviewed both changed files (.roborev.toml and CLAUDE.md). Left one minor suggestion about the worktree path documentation clarity. No blocking issues found.

Copy link

@claude claude bot left a comment

Choose a reason for hiding this comment

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

Review Summary

Clean, well-structured additions. Two minor observations:

CLAUDE.md — worktree path clarity
The relative path ../../.worktrees/ and the description referencing ptd-workspace/.worktrees/ assume a specific parent directory layout. If someone clones to a different location, ../../.worktrees/ won't resolve to ptd-workspace/.worktrees/. Consider either:

  • Adding a note that the repo is expected to live at ptd-workspace/team-operator/, or
  • Removing the ptd-workspace/ mention and relying solely on the relative ../../.worktrees/ form with a brief explanation of the assumed structure.

.roborev.toml
Minimal and straightforward — no concerns.

Overall this looks good to merge.

@ian-flores ian-flores marked this pull request as ready for review February 18, 2026 22:11
@ian-flores ian-flores requested a review from statik February 18, 2026 22:11
Copy link
Collaborator

@statik statik left a comment

Choose a reason for hiding this comment

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

@ian-flores this looks good to me. would you be up for adding a note to CONTRIBUTING.md?

I was thinking something like

roborev code review

If you would like to use roborev continuous code review, you will need to run roborev install-hook to install the commit hooks into your copy of the repo.


or, alternatively, should we put in CLAUDE.md a check or prompt asking the user if they want to install and use roborev?

Copy link

@Lytol Lytol left a comment

Choose a reason for hiding this comment

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

You could argue that the worktree stuff better belongs in a CLAUDE.local.md since it's a personal workflow thing, but I'm also fine with having it be a project norm. Your call, approved!

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

Comments