Commit 3794ab0
committed
fix(transactions): retry submitIntent on server-side stale state race
Bugsnag reports SubmitIntentError.StaleState errors with reason "race
detected: cached balance version is stale". This is a server-side
optimistic version lock — the client sends no balance version. When
concurrent intents advance the version before the DB transaction commits,
the check fails and the transaction rolls back (no nonces consumed).
- Add isRaceCondition property to StaleState, matching the "race
detected:" prefix — the only StaleState variant safe to blindly retry
- Wrap submitIntent in retryableOrThrow with retryIf narrowed to race
conditions only (other StaleState reasons like stale exchange rates,
closed accounts, or claimed gift cards require client-side state
refresh and would fail identically on retry)
- Move gift-card-claimed Bugsnag suppression from
InternalTransactionRepository to ReceiveGiftCardTransactor where the
context-specific handling belongs
Signed-off-by: Brandon McAnsh <git@bmcreations.dev>1 parent 717b430 commit 3794ab0
4 files changed
Lines changed: 45 additions & 8 deletions
File tree
- services/opencode/src
- main/kotlin/com/getcode/opencode
- internal
- domain/repositories
- transactors
- model/core/errors
- test/kotlin/com/getcode/opencode/model/core/errors
Lines changed: 12 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
| 21 | + | |
21 | 22 | | |
22 | 23 | | |
| 24 | + | |
23 | 25 | | |
24 | 26 | | |
25 | 27 | | |
| |||
29 | 31 | | |
30 | 32 | | |
31 | 33 | | |
32 | | - | |
33 | | - | |
34 | | - | |
35 | | - | |
36 | | - | |
37 | | - | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
38 | 41 | | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
39 | 45 | | |
40 | 46 | | |
41 | 47 | | |
| |||
Lines changed: 7 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
| 17 | + | |
17 | 18 | | |
18 | 19 | | |
19 | 20 | | |
| |||
146 | 147 | | |
147 | 148 | | |
148 | 149 | | |
149 | | - | |
| 150 | + | |
150 | 151 | | |
151 | | - | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
152 | 157 | | |
153 | 158 | | |
154 | 159 | | |
| |||
Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
123 | 123 | | |
124 | 124 | | |
125 | 125 | | |
| 126 | + | |
| 127 | + | |
126 | 128 | | |
127 | 129 | | |
128 | 130 | | |
| |||
Lines changed: 24 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
171 | 171 | | |
172 | 172 | | |
173 | 173 | | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
174 | 198 | | |
175 | 199 | | |
176 | 200 | | |
| |||
0 commit comments