Skip to content

ci(repo): enable snapi AI reviewer#8674

Merged
jacekradko merged 1 commit into
mainfrom
jacek/snapi-enable-ai-reviewer
May 27, 2026
Merged

ci(repo): enable snapi AI reviewer#8674
jacekradko merged 1 commit into
mainfrom
jacek/snapi-enable-ai-reviewer

Conversation

@jacekradko

@jacekradko jacekradko commented May 27, 2026

Copy link
Copy Markdown
Member

The snapi report in PR comments currently has no AI section because the Detect API changes step never sees SNAPI_ANTHROPIC_API_KEY, so snapi soft fails to the static rule-based output. This wires the env variable in the workflow

@changeset-bot

changeset-bot Bot commented May 27, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: efe135a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercel Bot commented May 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clerk-js-sandbox Ready Ready Preview, Comment May 27, 2026 7:30pm

Request Review

@coderabbitai

coderabbitai Bot commented May 27, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

This pull request enables AI reviewer support by making two complementary changes. First, a new changeset file documents the snapi-enable-ai-reviewer feature. Second, the API detection workflow step is updated to pass the SNAPI_ANTHROPIC_API_KEY environment variable from repository secrets, providing the required credentials for the AI detection tool at runtime.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: enabling the snapi AI reviewer in CI, which matches the primary objective of wiring the SNAPI_ANTHROPIC_API_KEY environment variable in the workflow.
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.
Description check ✅ Passed The pull request description clearly explains the purpose: enabling AI reviewer functionality by wiring the SNAPI_ANTHROPIC_API_KEY environment variable in the workflow, which directly relates to the changeset modifications.

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


Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown
Contributor

Snapi: no API changes detected in @clerk/backend, @clerk/clerk-js, @clerk/nextjs, @clerk/react, @clerk/shared, @clerk/ui.

@pkg-pr-new

pkg-pr-new Bot commented May 27, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@8674

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@8674

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@8674

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@8674

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@8674

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@8674

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@8674

@clerk/express

npm i https://pkg.pr.new/@clerk/express@8674

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@8674

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@8674

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@8674

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@8674

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@8674

@clerk/react

npm i https://pkg.pr.new/@clerk/react@8674

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@8674

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@8674

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@8674

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@8674

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@8674

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@8674

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@8674

commit: efe135a

@jacekradko jacekradko merged commit 17925fe into main May 27, 2026
49 checks passed
@jacekradko jacekradko deleted the jacek/snapi-enable-ai-reviewer branch May 27, 2026 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants