Skip to content

Commit f10c0a7

Browse files
authored
ci: activate RELEASE_BUILD in upgrade-deps (#516)
<!-- CURSOR_SUMMARY --> > [!NOTE] > **Low Risk** > Low risk CI-only change that adjusts build flags and workflow steps; main risk is slightly different build artifacts/caching behavior during automated dependency upgrades. > > **Overview** > **Enables release-mode builds in the deps upgrade workflow.** `upgrade-deps.yml` now sets `RELEASE_BUILD: 'true'` for the `build-upstream` composite action and for the Claude fallback step so the upgrade validation matches release build conditions. > > **Stabilizes PR output.** After updating the lockfile, the workflow checks out `packages/cli/binding/index.cjs` and `packages/cli/binding/index.d.cts` to discard generated binding changes before formatting/creating the PR. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 9e435c9. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
1 parent 40918e0 commit f10c0a7

1 file changed

Lines changed: 12 additions & 3 deletions

File tree

.github/workflows/upgrade-deps.yml

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,13 @@ jobs:
5757
continue-on-error: true
5858
with:
5959
target: x86_64-unknown-linux-gnu
60+
env:
61+
RELEASE_BUILD: 'true'
6062

6163
- uses: anthropics/claude-code-action@a017b830c03e23789b11fb69ed571ea61c12e45c # v1.0.0
6264
if: steps.build-upstream.outcome == 'failure'
65+
env:
66+
RELEASE_BUILD: 'true'
6367
with:
6468
claude_code_oauth_token: ${{ secrets.ANTHROPIC_API_KEY }}
6569
github_token: ${{ secrets.GITHUB_TOKEN }}
@@ -73,14 +77,14 @@ jobs:
7377
- We are using `pnpm` as the package manager
7478
- We are aiming to upgrade all dependencies to the latest versions in this workflow, so don't downgrade any dependencies.
7579
- Check `.claude/agents/cargo-workspace-merger.md` if rolldown hash is changed.
76-
- If deps in our `Cargo.toml` need to be upgraded, you can refer to the `./.claude/agents/cargo-workspace-merger.md`
77-
- If `Cargo.toml` has been modified, you need to run `cargo shear` to ensure there is nothing wrong with our dependencies.
78-
- Run `cargo check --all-targets --all-features` to ensure everything works fine if any Rust related codes are modified.
7980
- Run the steps in `build-upstream` action.yml after your fixing. If no errors are found, you can safe to exit.
8081
- Your final step is to run `just build` to ensure all builds are successful.
8182
- Run `pnpm run lint` to check if there are any issues after the build, if has, deep investigate it and fix it. You need to run `just build` before you can run `pnpm run lint`.
8283
- Run `pnpm run test` after `just build` to ensure all tests are successful.
8384
- The snapshot tests in `pnpm run test` are always successful, you need to check the updated snapshot to see if there is anything wrong after our deps upgrade.
85+
- If deps in our `Cargo.toml` need to be upgraded, you can refer to the `./.claude/agents/cargo-workspace-merger.md`
86+
- If `Cargo.toml` has been modified, you need to run `cargo shear` to ensure there is nothing wrong with our dependencies.
87+
- Run `cargo check --all-targets --all-features` to ensure everything works fine if any Rust related codes are modified.
8488
8589
Help me fix the errors in `build-upstream` steps, no need to commit changes after your fixing.
8690
claude_args: |
@@ -93,6 +97,11 @@ jobs:
9397
pnpm install --no-frozen-lockfile
9498
pnpm dedupe
9599
100+
- name: Checkout binding files
101+
run: |
102+
git checkout packages/cli/binding/index.cjs
103+
git checkout packages/cli/binding/index.d.cts
104+
96105
- name: Format code
97106
run: pnpm fmt
98107

0 commit comments

Comments
 (0)