fix: test runner cleanup + fanout timestamp bounded discovery

[matheus1lva]

Summary

Split from #400 per review feedback — smaller, independently measurable changes.

Test runner cleanup

• Fix mocha arg parsing (split --timeout from value)
• Default to **/*.spec.ts when no spec args provided
• Remove describe.only from strider.spec.ts (was still there on the old branch)
• Remove it.only from strider rollback test
• Fix hook.spec.ts blockTime to be recent (within 7-day lookback window)

Fanout timestamp discovery

• Replace unbounded DISTINCT block_time scan with series_time bounded query on the output table, scoped to the fanout window
• Maps to the top pg_stat_statements offender — the 99%+ row-reduction win
• Deploy and measure before adding more changes

From

Split from #400 (PR 1+2 per review request)

Follow-ups (surfaced during review)

• fix: types.spec.ts uses invalid '0x' address — broken since EvmAddressSchema landed #415 — broken types.spec.ts fixture (pre-existing, unmasked by the .only removal)
• fix: bun --filter ingest test silently skips most spec files due to shell glob #416 — bun --filter ingest test silently skips most specs (shell globstar default-off; pre-existing in packages/ingest/run-tests.ts)

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
kong	Ready	Preview, Comment	May 27, 2026 12:08pm

[murderteeth]

Summary

Two unrelated fixes bundled, split from #400:

• Test runner: fix mocha --timeout arg splitting, default to **/*.spec.ts fallback, drop describe.only/it.only from strider, bump hook.spec.ts blockTime into the 7-day lookback window.
• Fanout timestamp discovery: replace unbounded DISTINCT block_time scan with a series_time range-bounded scan on output, returning epoch seconds directly. Aligns with the hypertable partition column for chunk pruning.

Issues

None blocking.

Follow-ups (not blockers, separate from this PR)

• packages/lib/types.spec.ts is broken on disk — Removing the .only calls in strider.spec.ts correctly unmasks every other lib spec, which surfaces a pre-existing failure: OutputSchema.parse({ address: '0x', ... }) is rejected by EvmAddressSchema → viem.getAddress. The test fixture has address: '0x', which has never been a valid checksum address. Either fix the fixture (use a real address) or loosen the schema. Worth fixing soon so bun --filter lib test goes back to exit 0.
• bun --filter ingest test discovery is broken (pre-existing) — packages/ingest/run-tests.ts (unchanged here) uses the same ['**/*.spec.ts'] fallback with spawn(..., { shell: true }). Bash's globstar is off by default, so ** collapses to * and only depth-2 specs are discovered (abis/index.spec.ts, extract/evmlogs.spec.ts). Top-level (things.spec.ts, prices.spec.ts, abiutil.spec.ts) and depth-3+ specs are silently skipped — including the hook.spec.ts blockTime fix in this PR at abis/yearn/2/strategy/snapshot/hook.spec.ts. The fix itself is correct, but the runner doesn't exercise it. Options: drop shell: true and let mocha's own glob handle **, prefix the shell with shopt -s globstar, or pre-expand with a JS glob lib.

Verdict

APPROVE

---

How This Was Reviewed

This review was conducted using the review-pr skill.