starknet_transaction_prover: bump prover_panics_total on panic hook fire#14170
starknet_transaction_prover: bump prover_panics_total on panic hook fire#14170avi-starkware wants to merge 1 commit into
Conversation
PR SummaryLow Risk Overview A unit test asserts the hook bumps the counter by one when installed with a shared test recorder. Reviewed by Cursor Bugbot for commit 646fb1e. Bugbot is set up for automated code reviews on this repo. Configure here. |
cbd1def to
e503ebd
Compare
51ebd37 to
6c02d2b
Compare
e503ebd to
db503b7
Compare
2af08ad to
47068b1
Compare
db503b7 to
1da27e9
Compare
47068b1 to
eea5655
Compare
1da27e9 to
ac98d86
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit ac98d86. Configure here.
ac98d86 to
e4bbbdc
Compare
eea5655 to
df42f2f
Compare
df42f2f to
83d004c
Compare
e4bbbdc to
06bb59e
Compare
83d004c to
9122417
Compare
0b2c8cc to
4b1caba
Compare
9122417 to
23731d6
Compare
23731d6 to
145ff10
Compare
05ed9b4 to
72918b7
Compare
130f7f6 to
7fa5a6d
Compare
72918b7 to
74f4f46
Compare
7fa5a6d to
fe8282d
Compare
74f4f46 to
728f22c
Compare
fe8282d to
d9c346a
Compare
728f22c to
966f499
Compare
Bumps the new `prover_panics_total` counter from the panic hook before constructing the backtrace, so dashboards can alert on panic rate without log search. The pre-registered zero observation in `install_exporter` keeps the series visible at scrape time even before the first panic. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
d9c346a to
d3505a7
Compare
966f499 to
646fb1e
Compare


Bumps the new
prover_panics_totalcounter from the panic hook beforeconstructing the backtrace, so dashboards can alert on panic rate without
log search. The pre-registered zero observation in
install_exporterkeeps the series visible at scrape time even before the first panic.
Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com