chore(ocp): ensure give polling failures are reported as an error to bugsnag

bmc08gt · bmc08gt · commit ac292a15012d · 2025-11-06T12:09:16.000-05:00
Signed-off-by: Brandon McAnsh &lt;git@bmcreations.dev&gt;
diff --git a/services/opencode/src/main/kotlin/com/getcode/opencode/internal/transactors/GiveBillTransactor.kt b/services/opencode/src/main/kotlin/com/getcode/opencode/internal/transactors/GiveBillTransactor.kt
@@ -134,9 +134,6 @@ internal class GiveBillTransactor(
                 )
             },
             onFailure = {
-                if (it !is GiveTransactorError)  {
-                    ErrorUtils.handleError(it)
-                }
                 logAndFail(it)
             }
         )
diff --git a/services/opencode/src/main/kotlin/com/getcode/opencode/internal/transactors/Transactor.kt b/services/opencode/src/main/kotlin/com/getcode/opencode/internal/transactors/Transactor.kt
@@ -16,7 +16,8 @@ abstract class Transactor<E: CodeServerError>(
             tag = tag,
             message = error.message.orEmpty(),
             metadata = metadata,
-            type = TraceType.Process
+            type = TraceType.Error,
+            error = error
         )
         return Result.failure(error)
     }

Original file line number	Diff line number	Diff line change
`@@ -134,9 +134,6 @@ internal class GiveBillTransactor(`
`134`	`134`	`)`
`135`	`135`	`},`
`136`	`136`	`onFailure = {`
`137`		`- if (it !is GiveTransactorError) {`
`138`		`- ErrorUtils.handleError(it)`
`139`		`- }`
`140`	`137`	`logAndFail(it)`
`141`	`138`	`}`
`142`	`139`	`)`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,8 @@ abstract class Transactor<E: CodeServerError>(`
`16`	`16`	`tag = tag,`
`17`	`17`	`message = error.message.orEmpty(),`
`18`	`18`	`metadata = metadata,`
`19`		`- type = TraceType.Process`
	`19`	`+ type = TraceType.Error,`
	`20`	`+ error = error`
`20`	`21`	`)`
`21`	`22`	`return Result.failure(error)`
`22`	`23`	`}`