Skip to content

quest: review keeps a reviewer's environment credentials - #295

Merged
moq-bot[bot] merged 1 commit into
mainfrom
quest-plan/review-env-login
Oct 30, 2026
Merged

moq-bot[bot] merged 1 commit into
mainfrom
quest-plan/review-env-login

Conversation

@kixelated

Copy link
Copy Markdown
Collaborator

Extends quest/m0/review-login.md (now [S]) so that quest review still works when a reviewer's credentials come only from environment variables.

strippedEnvironment removes every ANTHROPIC_* and CLAUDE* variable except CLAUDE_CONFIG_DIR. Sessions on 2026-10-30 that authenticated Claude Code only through the environment got "claude is not logged in", so Codex-written PRs (moq-dev/moq#7084, moq-dev/moq#7251) had no Claude review. The plan lets a reviewer keep the credential its own CLI reads from the environment, still strips variables that would redirect billing or configuration, and names the variable when one is stripped on purpose.

Decided 2026-10-30 (user, /quest-plan of a moq-dev/moq /quest-complete session's follow-ups): ✅ extend review-login.md / drop.

Review

quest review, Codex gpt-6.1-sol, medium effort, round 1 on f1859d4: 0 findings.

(Written by Claude Opus 5.5)

🤖 Generated with Claude Code

@kixelated
kixelated enabled auto-merge (squash) October 30, 2026 17:19
* quest: review keeps a reviewer's environment credentials

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
@moq-bot
moq-bot Bot force-pushed the quest-plan/review-env-login branch from f1859d4 to eeec0c6 Compare October 30, 2026 17:25
@moq-bot
moq-bot Bot merged commit eeec0c6 into main Oct 30, 2026
@moq-bot
moq-bot Bot deleted the quest-plan/review-env-login branch October 30, 2026 17:25
@coderabbitai

coderabbitai Bot commented Oct 30, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 8badbcd4-2451-4f7d-90eb-41d4ba212022
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

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.

1 participant