Skip to content

Try as claude code app to approve#86

Draft
tcarmet wants to merge 13 commits intomainfrom
feature/auth-as-claude-app
Draft

Try as claude code app to approve#86
tcarmet wants to merge 13 commits intomainfrom
feature/auth-as-claude-app

Conversation

@tcarmet
Copy link
Copy Markdown
Contributor

@tcarmet tcarmet commented Mar 23, 2026

No description provided.

dvasilas and others added 10 commits March 20, 2026 16:41
pull_request_target checks out the base branch by default.
Use ref in actions/checkout so that the dependency review
can analyze the PR's changes and not the target branch.
claude-code-action rejects bot-initiated triggers by default.
Add dependabot[bot] to allowed_bots since this workflow is
specifically for reviewing Dependabot PRs.
claude-code-action validates that marketplace URLs end with .git.
Accept an optional GIT_ACCESS_TOKEN secret and configure git to
use it for github.com URLs. This allows yarn install to fetch
private Scality dependencies and the claude-code-action to clone
the private agent-hub plugin marketplace.
Move the token from job-level env to step-level env on the git
config step only, preventing unnecessary exposure to other steps.
Use a shell conditional instead of a step-level if condition since
secrets context is not available in step conditions.
Use --local instead of --global for git config to limit token
exposure to the checkout directory.

Add --ignore-scripts to yarn install to prevent lifecycle scripts
from untrusted PR code from running in the pull_request_target
context, which has write permissions and access to credentials.
The marketplace clone runs outside the checkout directory, so
--local git config doesn't apply. Revert to --global.
Without this permission the reviewer cannot query check run
results and reports "Unable to verify" CI status.
Use actions/create-github-app-token@v1 instead of the
GIT_ACCESS_TOKEN secret.
@tcarmet tcarmet force-pushed the feature/auth-as-claude-app branch from 9a44096 to 5f8e59b Compare March 23, 2026 17:07
@dvasilas dvasilas force-pushed the fix/dependency-review-dependabot branch from 6c138a0 to 9561dbf Compare March 31, 2026 07:34
Base automatically changed from fix/dependency-review-dependabot to main March 31, 2026 07:44
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.

2 participants