feat(coprocessor): speed up CI

[antoniupop]

To speed up compilation times in CI

[github-actions]

Changed Lines Coverage

Coverage of added/modified lines: N/A

[antoniupop]

@claude /pr-review

[claude]

Claude finished @antoniupop's request. — View run

[claude]

[AUTOMATED]

Code Review

Summary

This PR improves CI performance through two key optimizations:

1. Mold linker — Uses mold (via clang) as the linker for the cargo-tests workflow, which is significantly faster than the default ld.
2. Parallelized DB migration — Starts database migration in the background early, compiles tests with SQLX_OFFLINE=true while the DB initializes, then waits for migration before running tests.

The approach is sound — it mirrors the existing make init_db target (docker compose up -d --build db-migration && docker wait db-migration) but splits it into start/wait phases with compilation in between.

Reviewed For

• ✅ Bugs and runtime correctness
• ✅ Error handling and silent failures
• ✅ Guideline compliance and pattern consistency

Findings

One minor observation posted inline. No blocking issues found.

Overall this is a clean, well-structured CI optimization. The wait logic has proper timeout handling, exit code checking, and diagnostic log output.

[claude]

The review has been posted successfully. Here's a summary of the review:

PR Review Summary — PR #2174: feat(coprocessor): speed up CI

What the PR Does

Two CI optimizations:

1. Mold linker — Configures the faster mold linker (via clang) for the cargo-tests workflow
2. Parallelized DB migration — Starts database migration in the background early, compiles tests with SQLX_OFFLINE=true while the DB initializes, then waits for migration completion before running tests

Review Results

• Bugs: No blocking bugs found. The container name db-migration is correctly referenced (confirmed via container_name: in docker-compose.yml). The parallelization strategy is sound — SQLX_OFFLINE=true correctly allows compilation without a live DB.
• Error Handling: The wait logic has proper timeout handling (15 min), exit code checking, and diagnostic log output on failure. Well done.
• Guideline Compliance: No violations found. No CLAUDE.md or style guide exists in the repo.
• 1 Nit posted inline: The nohup wrapper around docker compose up -d --build is redundant in CI — nohup guards against HUP signals which don't apply in GitHub Actions. The & is the useful part. Non-blocking suggestion.

[antoniupop]

@Mergifyio queue

[mergify]

Merge Queue Status

• ✅ Entered queue — 2026-03-26 06:14 UTC · Rule: main
• ✅ Checks passed · on draft merge queue: embarking main (04bd48a) and #2174 together #2176
• ✅ Merged — 2026-03-26 08:55 UTC · at aef53df9fb7f0734a5590bfa3051dd0434776a20

This pull request spent 2 hours 41 minutes 10 seconds in the queue, including 1 hour 39 minutes 52 seconds running CI.

Required conditions to merge

• #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • feat(coprocessor): speed up CI #2174
• #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • feat(coprocessor): speed up CI #2174
• #review-threads-unresolved = 0 [🛡 GitHub branch protection]
  • feat(coprocessor): speed up CI #2174
• branch-protection-review-decision = APPROVED [🛡 GitHub branch protection]
  • feat(coprocessor): speed up CI #2174
• check-success = run-e2e-tests / fhevm-e2e-test
• any of [🛡 GitHub branch protection]:
  • check-success = common-pull-request/lint (bpr)
  • check-neutral = common-pull-request/lint (bpr)
  • check-skipped = common-pull-request/lint (bpr)
• any of [🛡 GitHub branch protection]:
  • check-success = coprocessor-cargo-test/cargo-tests (bpr)
  • check-neutral = coprocessor-cargo-test/cargo-tests (bpr)
  • check-skipped = coprocessor-cargo-test/cargo-tests (bpr)
• any of [🛡 GitHub branch protection]:
  • check-skipped = coprocessor-dependency-analysis/dependencies-check (bpr)
  • check-neutral = coprocessor-dependency-analysis/dependencies-check (bpr)
  • check-success = coprocessor-dependency-analysis/dependencies-check (bpr)
• any of [🛡 GitHub branch protection]:
  • check-skipped = gateway-contracts-deployment-tests/sc-deploy (bpr)
  • check-neutral = gateway-contracts-deployment-tests/sc-deploy (bpr)
  • check-success = gateway-contracts-deployment-tests/sc-deploy (bpr)
• any of [🛡 GitHub branch protection]:
  • check-skipped = kms-connector-tests/test-connector (bpr)
  • check-neutral = kms-connector-tests/test-connector (bpr)
  • check-success = kms-connector-tests/test-connector (bpr)