Skip to content

Commit 070ddb6

Browse files
committed
Merge branch 'main' into add-tracker
2 parents 4429aaf + da669f6 commit 070ddb6

File tree

30 files changed

+1494
-1428
lines changed

30 files changed

+1494
-1428
lines changed

.cursor/BUGBOT.md

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# BugBot configuration
2+
3+
This file contains the configuration for the BugBot.
4+
5+
## PR description
6+
7+
- Don't list all the changes by files/change types, just very briefly summarize what is the PR trying to accomplish.
8+
- Don't list the changes files/dirs.
9+
- Don't use sections, or lists, ideally just one short paragraph.
10+
- Don't use emojis.
11+
12+
## PR review
13+
14+
Please review this pull request and provide feedback on:
15+
16+
- Potential bugs or issues
17+
- Performance considerations
18+
- Important security concerns
19+
20+
Be constructive and helpful in your feedback and be **very conscise**.
21+
Skip general recommendations, skip summarizing the changes, and don't make any recommendations on code style.
22+
Also skip any summarization about why the PR was done well, focus only on the code changes and the potential issues.
23+
Don't output final summaries, or final lists of action items.
Lines changed: 63 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,63 @@
1+
name: Claude Code Review
2+
3+
on:
4+
pull_request:
5+
types: [opened]
6+
7+
jobs:
8+
claude-review:
9+
name: Claude Code Review
10+
11+
runs-on: ubuntu-latest
12+
permissions:
13+
contents: read
14+
pull-requests: read
15+
issues: read
16+
id-token: write
17+
18+
steps:
19+
- name: Checkout repository
20+
uses: actions/checkout@v4
21+
with:
22+
fetch-depth: 1
23+
24+
- name: Run Claude Code Review
25+
id: claude-review
26+
uses: anthropics/claude-code-action@v1
27+
with:
28+
anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
29+
prompt: |
30+
REPO: ${{ github.repository }}
31+
PR NUMBER: ${{ github.event.pull_request.number }}
32+
33+
Please review this pull request and provide feedback on:
34+
- Potential bugs or issues
35+
- Performance considerations
36+
- Important security concerns
37+
38+
Be constructive and helpful in your feedback and be **very conscise**.
39+
Skip general recommendations, skip summarizing the changes, and don't make any recommendations on code style.
40+
Also skip any summarization about why the PR was done well, focus only on the code changes and the potential issues.
41+
Don't output final summaries, or final lists of action items.
42+
Reduce false positives—try to validate if the issue is really a valid problem in the changes.
43+
44+
When you find a *specific issue* in the code (bug, performance concern, security risk, etc.),
45+
leave an **inline comment** on that exact file and line.
46+
If no issues are found, do not post anything.
47+
48+
To comment inline, use this format exactly:
49+
50+
```bash
51+
gh api \
52+
--method POST \
53+
-H "Accept: application/vnd.github+json" \
54+
-H "X-GitHub-Api-Version: 2022-11-28" \
55+
/repos/${{ github.repository }}/pulls/${{ github.event.pull_request.number }}/comments \
56+
-f body='${BODY}' \
57+
-f commit_id="${{ github.event.pull_request.head.sha }}" \
58+
-f path='${PATH}' \
59+
-F line=${LINE} \
60+
-f side='RIGHT'
61+
```
62+
63+
claude_args: '--allowed-tools "Bash(gh api:*), Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*)"'

.github/workflows/lint.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,6 @@ on: [workflow_call]
44

55
permissions:
66
contents: read
7-
env:
8-
GOLANGCI_LINT_VERSION: v2.4.0
97

108
jobs:
119
detect-modules:
@@ -30,8 +28,12 @@ jobs:
3028
uses: ./.github/actions/go-setup-cache
3129
with:
3230
job_key: lint-${{ matrix.modules }}
31+
32+
- name: Parse .tool-versions
33+
uses: wistia/parse-tool-versions@v2.1.1
34+
3335
- name: golangci-lint ${{ matrix.modules }}
3436
uses: golangci/golangci-lint-action@v8
3537
with:
36-
version: ${{ env.GOLANGCI_LINT_VERSION }}
38+
version: "v${{ env.GOLANGCI_LINT }}"
3739
working-directory: ${{ matrix.modules }}

.github/workflows/periodic-test.yml

Lines changed: 26 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,17 +56,37 @@ jobs:
5656
name: "pause and resume sandbox"
5757
- command: "tests/periodic-test/internet-works.ts"
5858
name: "internet connectivity inside of sandbox"
59-
# - command: "tests/periodic-test/cli-logs.ts"
60-
# name: "cli logs"
6159

6260
steps:
6361
- name: Checkout repository
6462
uses: actions/checkout@v4
6563

66-
- name: Install deno
67-
uses: denoland/setup-deno@v2
64+
- name: Parse .tool-versions
65+
uses: wistia/parse-tool-versions@v2.1.1
6866
with:
69-
cache: true
67+
filename: ".tool-versions"
68+
uppercase: "true"
69+
prefix: "tool_version_"
70+
71+
- name: Install Bun
72+
uses: oven-sh/setup-bun@v2
73+
with:
74+
bun-version: "${{ env.TOOL_VERSION_BUN }}"
75+
76+
- name: Cache Bun dependencies (periodic-test)
77+
id: bun-cache
78+
uses: actions/cache@v4
79+
with:
80+
path: |
81+
~/.bun/install/cache
82+
tests/periodic-test/node_modules
83+
key: ${{ runner.os }}-bun-periodic-${{ hashFiles('tests/periodic-test/bun.lock') }}
84+
restore-keys: |
85+
${{ runner.os }}-bun-periodic-
86+
87+
- name: Install dependencies (tests/periodic-test)
88+
working-directory: tests/periodic-test
89+
run: bun install --frozen-lockfile
7090

7191
- name: Set env variables from secrets
7292
if: always()
@@ -75,7 +95,7 @@ jobs:
7595
echo "E2B_ACCESS_TOKEN=${{ secrets[format('{0}', matrix.access_token)] }}" >> $GITHUB_ENV
7696
7797
- name: ${{ matrix.test-command.name }}
78-
run: deno run --allow-all ${{ matrix.test-command.command }}
98+
run: bun run ${{ matrix.test-command.command }}
7999
working-directory: ${{ matrix.test-command.dir }}
80100
env:
81101
E2B_API_KEY: ${{ env.E2B_API_KEY }}

.github/workflows/pull-request.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ jobs:
2121
unit-tests:
2222
uses: ./.github/workflows/pr-tests.yml
2323
integration-tests:
24-
needs: [ lint, out-of-order-migrations ]
24+
needs: [out-of-order-migrations]
2525
uses: ./.github/workflows/integration_tests.yml
2626
publish-test-results:
2727
needs:
@@ -46,5 +46,5 @@ jobs:
4646
if: always()
4747
with:
4848
comment_mode: off
49-
fail_on: 'errors'
49+
fail_on: "errors"
5050
files: "artifacts/**/*.xml"

.tool-versions

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,3 +3,4 @@ golang 1.24.7
33
golangci-lint 2.4.0
44
packer 1.13.1
55
terraform 1.5.7
6+
bun 1.3.2

0 commit comments

Comments
 (0)