[CI failed] reactnativecn/react-native-update-cli / ci

[github-actions]

@sunnylqm CI run failed.

Repository: reactnativecn/react-native-update-cli
Workflow: ci
Conclusion: failure
Branch: master
Commit: 2bc7645
Actor: sunnylqm
Run: https://github.com/reactnativecn/react-native-update-cli/actions/runs/24381789844

[sunnylqm]

@coderabbitai plz fix this ci error

[coderabbitai]

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Get issue details
gh issue view 28 --repo reactnativecn/react-native-update-cli

# List recent workflow runs
gh run list --repo reactnativecn/react-native-update-cli --limit 10

Length of output: 1899

---

🏁 Script executed:

#!/bin/bash
# Get details of the most recent failed CI run
gh run view 24393007502 --repo reactnativecn/react-native-update-cli --log-failed

echo "---WORKFLOW FILE---"
# Find the CI workflow file
fd -e yml -e yaml . .github/workflows/
cat .github/workflows/ci.yml 2>/dev/null || find .github -name "*.yml" -o -name "*.yaml" | xargs cat

Length of output: 133803

---

@sunnylqm, I've analyzed the CI failures. There are two categories of issues:

1. Test failures (5 tests) — The source code was updated to use i18n keys for error messages, but tests still assert on the old English strings.

2. Lint failures — Biome reports 2 errors (unused imports) and 109 warnings (unused function parameters, template curly-in-string) in example/ and tests/ files.

Here's my plan to fix everything:

CI Fix Plan

• Fix 5 failing tests: update assertions to match new i18n error message keys
• Fix lint errors: remove unused imports in custom-deploy-module.ts and versions.test.ts
• Fix lint warnings: prefix unused params with _ in example modules/scripts
• Fix noTemplateCurlyInString warnings in tests/utils.test.ts with biome-ignore comments

---

🏁 Script executed:

cat tests/app.test.ts

Length of output: 3096

---

🏁 Script executed:

cat tests/diff.test.ts | grep -n "node-hdiffpatch\|nodeHdiffpatch" -A3 -B3

Length of output: 443

---

🏁 Script executed:

cat tests/versions.test.ts

Length of output: 4171

---

🏁 Script executed:

cat example/modules/custom-deploy-module.ts | head -15
cat tests/versions.test.ts | head -15

Length of output: 611

---

🏁 Script executed:

cat tests/utils.test.ts | head -80
cat example/modules/analytics-module.ts | grep -n "execute\|init:" | head -20
cat example/scripts/provider-api-example.ts | grep -n "execute:" | head -20

Length of output: 2665

---

🏁 Script executed:

#!/bin/bash
cat example/modules/analytics-module.ts
echo "====="
cat example/modules/custom-deploy-module.ts | head -20
echo "====="
# Also get the full list of execute calls with context params in custom-deploy-module.ts
grep -n "execute\|init:\|provider" example/modules/custom-deploy-module.ts | head -30
echo "====="
grep -n "execute\|init:\|provider" example/scripts/provider-api-example.ts | head -20
echo "====="
# Check if mock is used in app.test.ts
grep -n "mock" tests/app.test.ts

Length of output: 5983

---

@sunnylqm, here are all the fixes needed. I'll generate the complete diffs:

Fix: CI failures — 5 test failures + lint errors/warnings

• Fix 3 test assertion failures in tests/app.test.ts (i18n key mismatch + unused import)
• Fix 1 test assertion failure in tests/diff.test.ts (i18n key mismatch)
• Fix 1 test assertion failure + unused import in tests/versions.test.ts
• Suppress intentional noTemplateCurlyInString in tests/utils.test.ts with biome-ignore
• Remove unused imports from example/modules/custom-deploy-module.ts
• Prefix unused context/provider params with _ in all three example files

---

💡 For code changes across multiple files, use @coderabbitai plan in an issue comment

[coderabbitai]

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Get the issue details
gh issue view 28 --repo reactnativecn/react-native-update-cli

echo "---COMMENTS---"
gh issue view 28 --repo reactnativecn/react-native-update-cli --comments

Length of output: 22402

---

🏁 Script executed:

#!/bin/bash
# Check the specific CI run referenced in the issue
gh run view 24381789844 --repo reactnativecn/react-native-update-cli --log-failed 2>&1 | head -300

echo "=== RUN STATUS ==="
gh run view 24381789844 --repo reactnativecn/react-native-update-cli 2>&1

Length of output: 28515

---

🏁 Script executed:

#!/bin/bash
echo "=== tests/app.test.ts ==="
cat tests/app.test.ts

echo "=== tests/versions.test.ts ==="
cat tests/versions.test.ts

echo "=== tests/diff.test.ts (around line 457) ==="
sed -n '450,465p' tests/diff.test.ts

echo "=== tests/utils.test.ts ==="
cat tests/utils.test.ts

echo "=== example/modules/analytics-module.ts ==="
cat example/modules/analytics-module.ts

echo "=== example/modules/custom-deploy-module.ts (head 15) ==="
head -15 example/modules/custom-deploy-module.ts

echo "=== example/modules/custom-deploy-module.ts (execute/init lines) ==="
grep -n "execute:\|init:" example/modules/custom-deploy-module.ts

echo "=== example/scripts/provider-api-example.ts (execute lines) ==="
grep -n "execute:" example/scripts/provider-api-example.ts

echo "=== example/scripts/enhanced-workflow-demo.ts (around line 512) ==="
sed -n '508,516p' example/scripts/enhanced-workflow-demo.ts

echo "=== Full biome lint output - get all errors ==="
# Check if there are more lint issues beyond what we saw in the truncated log
grep -n "noUnusedFunctionParameters\|useParseIntRadix\|noUnusedImports\|noTemplateCurlyInString" example/modules/analytics-module.ts example/modules/custom-deploy-module.ts example/scripts/provider-api-example.ts example/scripts/enhanced-workflow-demo.ts tests/utils.test.ts tests/versions.test.ts 2>/dev/null | head -50

Length of output: 22945

---

@sunnylqm, I've analyzed both failing jobs. Here's the full picture:

test job — 5 failing assertions where i18n keys were updated in source but tests still reference old strings.

lint job — Biome errors for unused imports + warnings for unused function parameters.

CI Fix: test failures + lint errors

• Fix 5 test assertion mismatches (i18n key changes)
• Remove unused mock imports from test files
• Remove unused CommandDefinition/CustomWorkflow from custom-deploy-module.ts
• Prefix unused context/provider params with _ across example files
• Add missing radix to parseInt in enhanced-workflow-demo.ts

---

🏁 Script executed:

#!/bin/bash
# Verify which imports in tests/utils.test.ts are actually used in the body
grep -n "afterEach\|beforeEach\|mock\b\|spyOn" tests/utils.test.ts

Length of output: 51

---

@sunnylqm, confirmed — all four (afterEach, beforeEach, mock, spyOn) in tests/utils.test.ts are only in the import line and never used. Here's the complete fix plan:

Fix all CI failures

• test job: fix 5 i18n assertion mismatches
• lint job: remove unused imports (biome errors) + fix unused params & missing radix (biome warnings)

---

💡 For code changes across multiple files, use @coderabbitai plan in an issue comment