Skip to content

[codex] Simplify backtesting helpers#13

Closed
david30907d wants to merge 1 commit into
mainfrom
simplify/backtesting-mechanical
Closed

[codex] Simplify backtesting helpers#13
david30907d wants to merge 1 commit into
mainfrom
simplify/backtesting-mechanical

Conversation

@david30907d
Copy link
Copy Markdown
Member

Summary

  • Factor the small useBacktestConfiguration regression test setup into local helpers and type the strategy config fixture.
  • Derive valid backtesting actions from ActionType instead of duplicating the literal set.
  • Extract the repeated asset bucket clamp in PortfolioBuckets.asset_allocation.
  • Cache nested public params schema generation with lru_cache.

Validation

  • pnpm --filter @zapengine/frontend exec vitest run tests/unit/hooks/backtesting/useBacktestConfiguration.test.ts
  • pnpm --filter @zapengine/frontend type-check
  • pnpm --filter @zapengine/frontend lint
  • pnpm --filter @zapengine/analytics-engine test
  • pnpm --filter @zapengine/analytics-engine type-check
  • pnpm --filter @zapengine/analytics-engine lint
  • Pre-commit pnpm check:local passed during commit, including affected frontend and analytics tests plus strategy snapshot check with 0 drift.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 14, 2026

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

Project Deployment Actions Updated (UTC)
zap-engine-frontend Ready Ready Preview, Comment May 14, 2026 0:26am
zap-engine-landing-page Ready Ready Preview, Comment May 14, 2026 0:26am

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