Skip to content

refactor: harden common package boundaries and test coverage - #202

Merged
mosshqq merged 1 commit into
mainfrom
codex/ai-refactor
Aug 28, 2026
Merged

mosshqq merged 1 commit into
mainfrom
codex/ai-refactor

Conversation

@mosshqq

@mosshqq mosshqq commented Aug 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Move React, Router, and i18n runtime dependencies to peer dependencies and externalize them from the common package bundle.
  • Standardize root/common testing on Vitest 3.2.3 with package-local configs, V8 coverage, and packed Core/EVM consumer smoke probes.
  • Fix request timeout notification duplication and pubsub subscription behavior, including Object.prototype-colliding event names.
  • Add regression coverage for cache, RPC, Zustand stores, SDK clients, ABI decoding, and EVM Simulate Trace.
  • Update project handoff, architecture, TODO, cross-repository, and AI instructions; add the required changeset.

Validation

  • pnpm test — 21 test files, 332 tests passed
  • pnpm test:coverage — 24.84% statements/lines, 74.23% branches, 41.63% functions
  • pnpm build
  • pnpm lint
  • pnpm test:consumers
  • pnpm --filter @cfxjs/sirius-next-common exec tsc --noEmit
  • Prettier and git diff --check

Notes

  • The two sibling scan repositories were not modified. Their real package upgrade, page regression, build, and deployment validation remain follow-up work.
  • Existing React act(...) warnings in useTableData tests remain non-blocking.

This change is Reviewable

@mosshqq
mosshqq merged commit e43978e into main Aug 28, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant