Skip to content

Commit 5ecbdf5

Browse files
ci(release): separate module-mocking suites from real-plan-store suites
1 parent 2d0bfc3 commit 5ecbdf5

1 file changed

Lines changed: 13 additions & 6 deletions

File tree

.github/workflows/release.yml

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -30,13 +30,20 @@ jobs:
3030
fail-fast: false
3131
matrix:
3232
include:
33-
# `state` isolates the two suites that mutate process-global state
34-
# (the persistence-dir override and the sqlite plan-DB path). They
35-
# are order-sensitive to whatever else shares their process, so they
36-
# get their own shard. Both are excluded from core/agents below;
37-
# keep the three lists in sync or coverage silently drops.
33+
# Bun's mock.module() is process-global and permanent: once a file
34+
# installs one, every module loaded afterwards in that process gets
35+
# the mock. Four trading suites mock the shared plans/trades/executor
36+
# modules, which silently replaces the real plan store for any suite
37+
# that loads later. Whoever loses that race depends on module-eval
38+
# order, which differs per platform, so it passed locally and failed
39+
# on Linux CI. `state` holds the real-plan-store suites and `mocked`
40+
# holds the module-mocking ones, so the mocks cannot reach a suite
41+
# that needs the real store. Keep these two lists disjoint from
42+
# core/agents below or coverage silently drops.
3843
- shard-name: state
39-
paths: src/core/lifecycle src/infra/agents/tools/trading
44+
paths: src/core/lifecycle src/infra/agents/tools/trading/trading.approvalBinding.test.ts
45+
- shard-name: mocked
46+
paths: src/infra/agents/tools/trading/trading.executePlan.test.ts src/infra/agents/tools/trading/trading.terminationLayers.test.ts src/infra/agents/tools/trading/trading.terminationLayers.enforce.test.ts src/infra/agents/tools/trading/trading.wipGate.test.ts
4047
- shard-name: core
4148
paths: src/core/alpha src/core/audit src/core/backtesting src/core/consensus src/core/edge src/core/execution src/core/genome src/core/indicators src/core/learning src/core/market-analysis src/core/memory src/core/microstructure src/core/numerics src/core/orders src/core/pipeline src/core/playbooks src/core/positions src/core/regime src/core/risk-kernel src/core/risk-management src/core/runtime src/core/safety src/core/stats src/core/strategies src/core/volatility
4249
- shard-name: agents

0 commit comments

Comments
 (0)