Skip to content

refactor(bayn): check every production index access - #13616

Merged
gregkonush merged 1 commit into
codex/bayn-effect-tsgo-strict-profilefrom
codex/bayn-tsgo-unchecked-indexes
Aug 10, 2026
Merged

refactor(bayn): check every production index access#13616
gregkonush merged 1 commit into
codex/bayn-effect-tsgo-strict-profilefrom
codex/bayn-tsgo-unchecked-indexes

Conversation

@gregkonush

@gregkonush gregkonush commented Aug 10, 2026

Copy link
Copy Markdown
Member

Summary

  • adds a production-only TSGo profile with noUncheckedIndexedAccess while retaining the full profile for tests and fixtures
  • replaces unchecked production indexing with explicit fail-closed domain results across market data, reconciliation, execution, persistence, and forward-performance paths
  • adds typed missing-reference failures instead of allowing implicit undefined to cross domain boundaries

Related Issues

None

Testing

  • bun run --filter @proompteng/bayn tsc
  • bun run --filter @proompteng/bayn lint
  • bun run --filter @proompteng/bayn lint:oxlint:type
  • bun run --filter @proompteng/bayn test (1,054 pass, 160 environment-gated skips, 0 fail)
  • bun run --filter @proompteng/bayn build (793 modules)

Breaking Changes

None

Checklist

  • Testing section documents the exact validation performed (or N/A with justification).
  • Screenshots and Breaking Changes sections are handled appropriately (removed or filled in).
  • Documentation, release notes, and follow-ups are updated or tracked.

@gregkonush
gregkonush force-pushed the codex/bayn-tsgo-unchecked-indexes branch from 8b96691 to 570676a Compare August 10, 2026 02:55
@gregkonush

Copy link
Copy Markdown
Member Author

:shipit:

@gregkonush
gregkonush merged commit efcea37 into main Aug 10, 2026
16 checks passed
@gregkonush
gregkonush deleted the codex/bayn-tsgo-unchecked-indexes branch August 10, 2026 11:50
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