moved to a webhook#13836
Conversation
sayakpaul
left a comment
There was a problem hiding this comment.
I have configured the secrets.
| SIG="sha256=$(openssl dgst -sha256 -hmac "${WEBHOOK_SECRET}" payload.json | awk '{print $NF}')" | ||
|
|
||
| - name: Strip fork-supplied reviewer/agent config | ||
| # ai-reviewer fetches its config (.ai/review-rules.md, .ai/review-tools.json, |
There was a problem hiding this comment.
How does it get fetched then? It's important that these files always get fetched from the upstream main.
There was a problem hiding this comment.
exactly like the github action : the repo is cloned in the githubapp server and it reads it
There was a problem hiding this comment.
For review rules, we always use upstream main right?
There was a problem hiding this comment.
no we may use the forked / PR one. I can change that
There was a problem hiding this comment.
correction: it is the case, but we do get the context-script from the PR/fork, which is a big issue. Fixing
| llm_api_key: ${{ secrets.ANTHROPIC_API_KEY }} | ||
| llm_api_base: https://api.anthropic.com | ||
| llm_model: claude-opus-4-6 | ||
| llm_stream: 'true' | ||
| mention_trigger: '@askserge' |
There was a problem hiding this comment.
How do we configure these things?
There was a problem hiding this comment.
in the admin UI. the webook then picks the right now depending on what org/repo calls the service
sayakpaul
left a comment
There was a problem hiding this comment.
Good to merge after the conflicts are resolved. Excited to try it out.
What does this PR do?
Moves the serge bot into using the github app deployed on our infra