Skip to content

Commit d8320a6

Browse files
jaredLundeclaude
andcommitted
fix(ci): install rustfmt explicitly and stabilize generated types
dprint runs rustfmt via the exec plugin, but mise's rust toolchain doesn't reliably install the rustfmt component, so check:fmt fails in CI with "rustfmt is not installed for toolchain 1.92.0". Add an explicit rustup component install step, mirroring ../kv. generate:types ran openapi-typescript (4-space output) without a follow-up format, so the committed file (dprint-formatted, 2-space) always diffed against CI regeneration. Run dprint fmt inside the generate script so local and CI produce the same canonical output. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent f12ab14 commit d8320a6

2 files changed

Lines changed: 3 additions & 0 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@ jobs:
1313
- uses: actions/checkout@v6
1414
- uses: jdx/mise-action@v4
1515
- uses: Swatinem/rust-cache@v2
16+
- name: ensure rustfmt/clippy components
17+
run: rustup component add rustfmt clippy
1618
- name: check:fmt
1719
run: mise run check:fmt
1820
- name: check:rs

sdk/ts/scripts/generate-types.mjs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,3 +9,4 @@ const spec = resolve(root, "openapi/v1.json");
99
const out = resolve(root, "sdk/ts/src/types.ts");
1010

1111
execSync(`npx openapi-typescript ${spec} -o ${out}`, { stdio: "inherit" });
12+
execSync(`dprint fmt ${out}`, { stdio: "inherit", cwd: root });

0 commit comments

Comments
 (0)