Skip to content

fix: health payload includes indexer check - #1415

Merged
Junirezz merged 5 commits into
Junirezz:mainfrom
m1s0g1:combo-issues-1330-1324-1362-1325
Oct 3, 2026
Merged

Junirezz merged 5 commits into
Junirezz:mainfrom
m1s0g1:combo-issues-1330-1324-1362-1325

Conversation

@m1s0g1

@m1s0g1 m1s0g1 commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

Issue #1330: Health payload missing indexer check

File: backend/src/tests/issues711.test.ts

  • Added indexer: 'up' to the health checks fixture (line 64)
  • Fixture now includes all optional-service check fields

Issue #1324: Event outbox initialization validation

File: backend/src/eventOutbox.ts

  • Added validation check before prisma.eventOutbox.create() (line 127-129)
  • Now fails fast with clear error message if Prisma schema is missing eventOutbox model

Issue #1362: Orphaned idempotency store exports

File: backend/src/idempotency.ts

  • Added import NodeCache from 'node-cache' and Redis support imports
  • Restored complete IdempotencyStore class with:
    • Redis-backed storage with local fallback
    • execute() method for idempotent operation handling
    • inspectKeys() for monitoring
    • deleteKey() and clear() for administration
    • getMetrics() for observability
    • pruneStaleKeys() for retention cleanup
  • Exported singleton: export const idempotencyStore
  • Exported helper: export function buildIdempotencyFingerprint()
  • Exported error class: export class IdempotencyConflictError
  • Exported types: IdempotentOperationResult, IdempotencyKeyInfo, IdempotencyMetrics

Issue #1325: Cache invalidation hook validation

File: backend/src/middleware/cache.ts

  • Added array type validation in triggerCacheInvalidation() (lines 335-342)
  • Logs error and skips if hook returns non-array instead of crashing

Closes #1325
Closes #1362
Closes #1324
Closes #1330

@drips-wave

drips-wave Bot commented Sep 25, 2026

Copy link
Copy Markdown

@m1s0g1 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Junirezz
Junirezz merged commit ae82ae8 into Junirezz:main Oct 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants