-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Labels
tech-debtTechnical debt items tracked for future cleanupTechnical debt items tracked for future cleanup
Description
Context
Greptile self-review comment on PR #94. Pre-existing pattern deviation, not introduced by the PR.
Problem
All other repositories use operationErrorHandler() from core/errors.js as the centralized error mapping function. WorkerRepository manually constructs BackbeatError inline in every method.
Note: register() has special UNIQUE constraint detection logic which may justify a custom error mapper for that one method. The other 6 methods should use operationErrorHandler.
Fix
Import and use operationErrorHandler for all methods except register().
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
tech-debtTechnical debt items tracked for future cleanupTechnical debt items tracked for future cleanup