Skip to content

Commit d750241

Browse files
Oba-Oneclaude
andcommitted
fix(ci): resolve contract test, type check, and Vercel deployment failures
- Use FOUNDRY_PROFILE=dev in CI workflows to match local build scripts - Add fetch-depth: 0 for proper submodule resolution - Fix TypeScript error in useJarCreation.ts (widen options array type) - Add vercel.json to specify bun as install command Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent a55a220 commit d750241

File tree

4 files changed

+28
-8
lines changed

4 files changed

+28
-8
lines changed

.github/workflows/contract-tests.yml

Lines changed: 18 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ jobs:
2424
uses: actions/checkout@v4
2525
with:
2626
submodules: recursive
27+
fetch-depth: 0
2728

2829
- name: Install Foundry
2930
uses: foundry-rs/foundry-toolchain@v1
@@ -36,15 +37,18 @@ jobs:
3637
run: |
3738
echo "🔧 Building contracts..."
3839
cd contracts
39-
40+
41+
# Use dev profile (matches local package.json scripts)
42+
export FOUNDRY_PROFILE=dev
43+
4044
# Build contracts with size reporting
4145
forge build --sizes
42-
46+
4347
if [ $? -ne 0 ]; then
4448
echo "❌ Contract compilation failed"
4549
exit 1
4650
fi
47-
51+
4852
echo "✅ Contracts compiled successfully"
4953
echo ""
5054
echo "📊 Contract sizes:"
@@ -54,7 +58,10 @@ jobs:
5458
run: |
5559
echo "🧪 Running Foundry test suite..."
5660
cd contracts
57-
61+
62+
# Use dev profile (matches local package.json scripts)
63+
export FOUNDRY_PROFILE=dev
64+
5865
# Run tests with detailed output and gas reporting
5966
forge test -vvv --gas-report
6067
@@ -72,7 +79,10 @@ jobs:
7279
run: |
7380
echo "📊 Generating coverage report..."
7481
cd contracts
75-
82+
83+
# Use dev profile (matches local package.json scripts)
84+
export FOUNDRY_PROFILE=dev
85+
7686
# Generate coverage in lcov format for better tooling support
7787
forge coverage --report lcov
7888
@@ -105,7 +115,9 @@ jobs:
105115
run: |
106116
echo "🔍 Running basic security analysis..."
107117
cd contracts
108-
118+
119+
export FOUNDRY_PROFILE=dev
120+
109121
echo "📋 Checking compilation warnings:"
110122
forge build 2>&1 | tee build-warnings.log
111123

.github/workflows/integration-tests.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ jobs:
2121
uses: actions/checkout@v4
2222
with:
2323
submodules: recursive
24+
fetch-depth: 0
2425

2526
- name: Setup Node.js and bun
2627
uses: ./.github/actions/setup-node-pnpm
@@ -74,7 +75,10 @@ jobs:
7475
run: |
7576
echo "🚀 Deploying contracts for integration testing..."
7677
cd contracts
77-
78+
79+
# Use dev profile (matches local package.json scripts)
80+
export FOUNDRY_PROFILE=dev
81+
7882
# Build contracts first
7983
forge build
8084

client/hooks/jar/useJarCreation.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -593,7 +593,7 @@ export const useJarCreation = () => {
593593

594594
// Currency options for the supported network
595595
const getCurrencyOptions = () => {
596-
const options = [
596+
const options: Array<{ value: string; label: string; description: string }> = [
597597
{
598598
value: ETH_ADDRESS,
599599
label: "ETH (Native)",

vercel.json

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
{
2+
"installCommand": "bun install --no-frozen-lockfile",
3+
"framework": "nextjs"
4+
}

0 commit comments

Comments
 (0)