docs: update CLAUDE.md, README, and .env.template for write test requirements

Document RUN_WRITE_TESTS and TEST_PR_ID env vars across all three files.
Fix .env.template to keep write-test vars commented out. Reference .env.template
as the starting point for local development setup.

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

.env.template

@@ -4,8 +4,12 @@ DEFAULT_WORKSPACE=your-workspace
 DEFAULT_REPO=your-repo
 
 # Integration test write operations
+# ---------------------------------------------------------------------------------
 # Set RUN_WRITE_TESTS=true to enable tests that create/modify/delete data.
+#
 # TEST_PR_ID must also be set — write tests will only run against this specific PR
 # to avoid accidentally modifying unrelated pull requests.
+# ---------------------------------------------------------------------------------
 # RUN_WRITE_TESTS=true
 # TEST_PR_ID=123
+# ---------------------------------------------------------------------------------

CLAUDE.md

@@ -43,7 +43,13 @@ The server is a 3-layer pipeline: **MCP protocol** -> **Router** -> **Bitbucket
 | `BITBUCKET_MCP_TOKEN` | Bitbucket API token (Basic Auth password) |
 | `DEFAULT_WORKSPACE` | Optional default workspace slug |
 | `DEFAULT_REPO` | Optional default repository slug |
+| `RUN_WRITE_TESTS` | Set to `true` to enable write integration tests |
+| `TEST_PR_ID` | PR ID to use for write integration tests (required when `RUN_WRITE_TESTS=true`) |
+
+Copy `.env.template` to `.env` and fill in your values for local development and testing.
 
 ## Testing
 
 Tests use vitest with 30s timeouts. Unit tests mock the Bitbucket client; integration tests call the real API and require valid credentials in the environment.
+
+Write integration tests (approve/unapprove, comments, tasks) are skipped by default. They require both `RUN_WRITE_TESTS=true` and `TEST_PR_ID` to be set. Write tests never pick a random PR — if `TEST_PR_ID` is missing the test is skipped with a warning.

README.md

@@ -225,13 +225,17 @@ Response:
 
 ### Running Locally
 
-For development, you can use a `.env` file instead of configuring credentials in the MCP client:
+For development, you can use a `.env` file instead of configuring credentials in the MCP client. Copy `.env.template` to `.env` and fill in your values:
 
 ```env
 BITBUCKET_MCP_EMAIL=your_email@example.com
 BITBUCKET_MCP_TOKEN=your_api_token
 DEFAULT_WORKSPACE=my-workspace
 DEFAULT_REPO=my-repo
+
+# Only needed when running write integration tests (see Testing below)
+# RUN_WRITE_TESTS=true
+# TEST_PR_ID=123
 ```
 
 ```bash
@@ -261,14 +265,14 @@ npx tsc --noEmit
 npm run test:coverage
 ```
 
-**Write operation tests** (approve/unapprove, comments, tasks) are disabled by default. To enable them, set two variables in your `.env`:
+**Write operation tests** (approve/unapprove, comments, tasks) are disabled by default and require two additional variables in your `.env`:
 
-```env
+| Variable | Description |
-RUN_WRITE_TESTS=true
+|----------|-------------|
-TEST_PR_ID=123   # ID of a PR you own and can safely modify
+| `RUN_WRITE_TESTS=true` | Opt in to running write tests |
-```
+| `TEST_PR_ID=<id>` | ID of a PR you own and can safely modify |
 
-`TEST_PR_ID` is required — write tests will not fall back to a random PR. If it is not set, each write test is skipped with a warning. This prevents accidental modifications to production PRs.
+`TEST_PR_ID` is mandatory — write tests will never fall back to a random PR from the repository. If it is not set, each write test is skipped with a warning. This prevents accidental modifications to PRs you don't intend to touch.
 
 ### Project Structure