Skip to content

test(mcp): smoke-test fixture for run_secret_scanning [do not merge] - #785

Closed
don-petry wants to merge 2 commits into
mainfrom
claude/mcp-scan-smoketest
Closed

don-petry wants to merge 2 commits into
mainfrom
claude/mcp-scan-smoketest

Conversation

@don-petry

@don-petry don-petry commented Jun 19, 2026 •

Copy link
Copy Markdown
Collaborator

⚠️ Throwaway smoke test — do not merge

This PR exists only to exercise the run_secret_scanning MCP tool end-to-end in the pr-review workflow, then it will be closed without merging.

It adds a placeholder-only .env.example. The .env* path is a HIGH-risk triage trigger, so a forced pr-review dry-run will escalate to tier-2 deep-review (where the secret scan is wired) with a non-empty, scannable diff. All values are obvious <placeholder> text, so neither gitleaks CI nor run_secret_scanning should flag anything — giving a clean positive execution that confirms the GHAS round-trip works.

Pilot validation for #681. Will be closed once the run log is captured.


Generated by Claude Code

Summary by CodeRabbit

  • Chores
    • Added example environment configuration file with template entries for required API credentials and database settings
    • Included guidance on setting up local development environment and protecting sensitive credentials

Adds a placeholder-only .env.example so a forced pr-review dry-run escalates
to tier-2 deep-review (the `.env*` path is a HIGH-risk triage trigger) with a
non-empty, scannable diff. Used to confirm the run_secret_scanning MCP tool
actually executes against real content and returns a clean result (the GHAS
round-trip). This PR is a smoke test and will be closed without merging.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015VRMMWmqW9nW81jygmea3e
@don-petry
don-petry requested a review from a team as a code owner June 19, 2026 00:46
@coderabbitai

coderabbitai Bot commented Jun 19, 2026 •

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@don-petry, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 55 minutes and 54 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits.

🚦 How do rate limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: ebbc3bd9-220c-4eeb-b18e-10c9d6fa7a9c

📥 Commits

Reviewing files that changed from the base of the PR and between 313d5a1 and c85eebc.

📒 Files selected for processing (1)
  • .env.example
📝 Walkthrough

Walkthrough

A new .env.example file is added with 15 lines providing placeholder values and explanatory comments for four local-development configuration variables: GITHUB_TOKEN, CLAUDE_CODE_OAUTH_TOKEN, GEMINI_API_KEY, and DATABASE_URL.

Changes

Local development environment template

Layer / File(s) Summary
.env.example placeholder configuration
.env.example
New file with header comments instructing users to copy to .env without committing real secrets, and placeholder entries for GITHUB_TOKEN (GitHub classic PAT), CLAUDE_CODE_OAUTH_TOKEN (Anthropic/Claude), GEMINI_API_KEY (Google/Gemini), and DATABASE_URL (local database).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

needs-human-review

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title describes a smoke test for the run_secret_scanning MCP tool, but the actual changeset only adds a .env.example file with placeholder configuration entries. The title does not reflect the primary content change. Update the title to reflect the actual changes, such as 'Add .env.example with placeholder configuration' or 'chore: add example environment configuration file', or reframe to accurately describe what the changeset contains.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/mcp-scan-smoketest

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 and usage tips.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a .env.example file containing placeholder environment variables for local development, such as GITHUB_TOKEN, CLAUDE_CODE_OAUTH_TOKEN, GEMINI_API_KEY, and DATABASE_URL. I have no feedback to provide as there are no review comments.

@don-petry

Copy link
Copy Markdown
Collaborator Author

Dev-Lead — waiting on PR blockers (intent: review-changes)

PR: #785
No changes were committed, but the PR still has blocking checks or reviews (failing or cancelled checks, or changes-requested reviews). The retry cron will re-attempt automatically. Next attempt after: 2026-06-19T01:17:48Z

@don-petry

Copy link
Copy Markdown
Collaborator Author

Note

@don-petry I reviewed this PR and no code changes were needed, but it still has blocking checks or reviews (failing or cancelled checks, or changes-requested reviews), so I cannot mark it done yet. I'll re-check automatically.
Next attempt after: 2026-06-19T01:17:48Z

@don-petry
don-petry enabled auto-merge (squash) June 19, 2026 00:47
@don-petry
don-petry disabled auto-merge June 19, 2026 00:48

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.env.example:
- Around line 5-15: The environment variable names in .env.example do not match
what the actual scripts and workflows expect. Replace GITHUB_TOKEN with GH_TOKEN
(required by scripts/verify-auth-scopes.sh), replace GEMINI_API_KEY with
GOOGLE_API_KEY (required by scripts/validate-engines.sh), and add a new
environment variable GEMINI_CLI_TRUST_WORKSPACE with an example value so that
local setup matches the actual contract used by the scripts and workflows.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: e475d196-0f3c-4605-8cf2-aaa68dbceb9d

📥 Commits

Reviewing files that changed from the base of the PR and between 1bd5ab2 and 313d5a1.

📒 Files selected for processing (1)
  • .env.example

Comment thread .env.example Outdated
coderabbitai[bot]
coderabbitai Bot previously approved these changes Jun 19, 2026
@don-petry

Copy link
Copy Markdown
Collaborator Author

Dev-Lead — fix-reviews (applied)

Changes committed and pushed.

@sonarqubecloud

Copy link
Copy Markdown

@don-petry

Copy link
Copy Markdown
Collaborator Author

Dev-Lead — waiting on PR blockers (intent: review-changes)

PR: #785
No changes were committed, but the PR still has blocking checks or reviews (failing or cancelled checks, or changes-requested reviews). The retry cron will re-attempt automatically. Next attempt after: 2026-06-19T01:23:19Z

@don-petry

Copy link
Copy Markdown
Collaborator Author

Note

@don-petry I reviewed this PR and no code changes were needed, but it still has blocking checks or reviews (failing or cancelled checks, or changes-requested reviews), so I cannot mark it done yet. I'll re-check automatically.
Next attempt after: 2026-06-19T01:23:19Z

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