Clarify checkpointing git phrasing

[jfcantu]

Related GitHub Issue

Addresses: #8164

(I discussed this with @hannesrudolph on Discord, who indicated not to worry about creating a new issue.)

Roo Code Task Context (Optional)

Description

What:

• Clarifies the error message shown when checkpointing cannot be enabled due to a Git repository existing below the workspace root.
• Replace the term "nested" in checkpointing-related code with "child".

Why:
The term "nesting" can be confusing, as it implies the issue is "git repository inside a git repository", which may not be the case.

How:

• Replaced the term "nested" with "child", which is clearer.
• Rephrased the error message to be more informational, and added a link pointing to the Roo documentation.

Test Procedure

I ran pnpm test as described.

I received this error several times, but it appears to be related to my local build environment, not to my changes:

roo-cline:test: Error: require() of ES Module C:\jfc\src\roo\Roo-Code\node_modules\.pnpm\[email protected]\node_modules\brace-expansion\index.js from C:\jfc\src\roo\Roo-Code\node_modules\.pnpm\[email protected]\node_modules\minimatch\minimatch.js not supported.

Pre-Submission Checklist

• [N/A] Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
• [Y] Scope: My changes are focused on the linked issue (one major feature/fix per PR).
• [Y] Self-Review: I have performed a thorough self-review of my code.
• [Y] Testing: New and/or updated tests have been added to cover my changes (if applicable).
• [Y] Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
• [Y] Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

Documentation Updates

PR for documentation update has been submitted: RooCodeInc/Roo-Code-Docs#373

Additional Notes

Get in Touch

---

Important

Clarified Git repository detection terminology in checkpointing by replacing 'nested' with 'child' and updated related error messages and tests.

• Behavior:
  • Rephrased error message in common.json for Git repository detection, replacing 'nested' with 'child' and adding a documentation link.
  • Updated ShadowCheckpointService.ts to use 'child' instead of 'nested' in error handling and logging.
• Functions:
  • Renamed getNestedGitRepository() to getChildGitRepository() in ShadowCheckpointService.ts.
• Tests:
  • Updated test descriptions and expectations in ShadowCheckpointService.spec.ts to reflect terminology change from 'nested' to 'child'.

^{This description was created by for 3e1b9fc. You can customize this summary. It will automatically update as commits are pushed.}

[Copilot]

Pull Request Overview

This PR clarifies the error message and terminology used when checkpointing cannot be enabled due to Git repositories existing below the workspace root. The changes replace the potentially confusing term "nested" with "child" throughout the codebase and improve the error message clarity.

• Replaces "nested" terminology with "child" for better clarity
• Updates error message to be more informative and includes documentation link
• Updates corresponding test descriptions and variable names

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
src/services/checkpoints/ShadowCheckpointService.ts	Updates method names, variable names, and error messages from "nested" to "child" terminology
src/services/checkpoints/tests/ShadowCheckpointService.spec.ts	Updates test descriptions, variable names, and expected error message patterns to match new terminology
src/i18n/locales/en/common.json	Updates the internationalized error message with clearer wording and adds documentation link

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

The translation key references 'child_git_repos_warning' but the JSON file still uses 'nested_git_repos_warning'. This will cause the translation to fail.

[ellipsis-dev]

Typographical consistency: The string uses "Git repository" earlier but refers to it as "git repository" later on this line. Consider using a consistent capitalization (e.g., "Git repository") throughout.

Suggested change

	"nested_git_repos_warning": "Checkpoints are disabled because a Git repository was detected below the workspace root at: {{path}}. To use checkpoints, please remove or relocate this git repository, or open a Git repository as the workspace root.\n\nFor more information, see https://docs.roocode.com/features/checkpoints#git-repositories-below-the-workspace-root",
	"nested_git_repos_warning": "Checkpoints are disabled because a Git repository was detected below the workspace root at: {{path}}. To use checkpoints, please remove or relocate this Git repository, or open a Git repository as the workspace root.\n\nFor more information, see https://docs.roocode.com/features/checkpoints#git-repositories-below-the-workspace-root",