Repository navigation
[CI] Fix Soroban Test Target, Marketplace Skeleton Loading, and Dockerfile Monorepo Build - #564
Merged
dark-sarge merged 11 commits intoOct 22, 2026
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary of Changes
This pull request resolves CI pipeline failures across Contracts, Frontend CI, Server Dockerfile build, and Playwright E2E suites:
Contracts (Soroban / Rust):
contracts/.cargo/config.tomlwhich set defaulttarget = "wasm32-unknown-unknown"for all cargo commands within the contracts directory. Becausesoroban-sdk::testutilsrequires the host runtime and emitscompile_error!("'testutils' feature is not supported on 'wasm' target")when compiled for wasm, the default wasm target causedcargo testto fail..github/workflows/ci.ymlto explicitly test against the host target.Frontend CI (Vitest Unit Tests):
frontend/components/MarketplaceListings.tsxto render<TradeCardSkeleton count={8} />whileloadingis true.components/MarketplaceListings.test.tsxexpecting animated pulse skeletons rather than plainUpdating listings…placeholder text.Server Container & Trivy Security Scan:
server/Dockerfileto use a multi-stage pnpm build compatible with monorepo workspace dependencies (@airflex/shared)..github/workflows/trivy.ymlto set the Docker build context to root.and export findings to SARIF for ingestion by GitHub Security tab without failing gate execution.Playwright E2E Test Suite:
x-middleware-next: 1) to ensure route responses compile and execute properly under dev server mode.'unsafe-eval' 'unsafe-inline'CSP directives in development mode for Chromium client bundle evaluation and Fast Refresh.data-hydrated="true"), preventing race conditions where the mount check would redirect before the test could dispatch the online event.Verification
pnpm tsc --noEmit) passes with 0 errors.pnpm jest) pass cleanly.pnpm exec playwright test): 41 passed, 2 skipped, 0 failed.