test(service-cluster-redis): de-flake Redis wiring test (hoist cold imports)

[xuyushun441-sys]

Problem

Test Core intermittently fails on packages/services/service-cluster-redis/src/redis.contract.test.ts:

FAIL  Redis driver — wiring > exports a registerable driver and defineCluster picks it up
Error: Test timed out in 5000ms.
  ❯ redis.contract.test.ts:120  →  await import('./index.js') / await import('@objectstack/service-cluster')

Both wiring tests did two await import(...) inside the test body, under the default 5000ms timeout. On a cold/slow CI runner the one-time module load alone exceeds 5s, so the first test times out (the second only passed because the first had warmed the module cache). It's a flake, unrelated to any product code — it red-flagged #1488's Test Core, for instance.

Fix

Hoist both to top-level static imports:

import './index.js';                              // side-effect: registers the 'redis' driver
import { defineCluster } from '@objectstack/service-cluster';

import './index.js' still registers the driver as a side-effect at module eval, and defineCluster is a normal named export (service-cluster/src/index.ts). The cold import cost now lands once at file load, outside any per-test timeout. The two test bodies drop their local await import(...) lines; all assertions are unchanged.

Verified by inspection (pure import hoist; defineCluster is a confirmed named export, ./testing is a separate subpath import that's untouched). CI exercises the test.

🤖 Generated with Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
spec	Building	Preview, Comment	Jun 1, 2026 9:17pm