[$40 BOUNTY] [Go] Add order book cancellation depth tests

[jackjin1997]

Bounty: $40 (LT)

Area: market/orderbook/orderbook.go

Current state: The order book supports adding and cancelling orders, but focused tests are needed around depth updates after cancellation, closed-book behavior, and snapshot immutability.

What is needed: Add Go tests for order cancellation and snapshot/depth behavior in the order book package.

Acceptance criteria:

• Cover cancelling a bid removes the matching bid level and deletes the order id.
• Cover cancelling an ask removes the matching ask level.
• Cover cancelling an unknown order returns ErrOrderNotFound.
• Cover closed books rejecting add/cancel operations with ErrBookClosed.
• Cover snapshots returning copies so callers cannot mutate internal bid/ask slices.

Required validation:

• Run go test ./market/orderbook or the closest package-level Go test command available in this repo.
• Run python3 build.py.
• Include the generated diagnostic/build-*.logd artifact from diagnostic/; include the matching diagnostic/build-*.json if present.
• Use .github/pull_request_template.md for the submission.

[Ojas2095]

/claim

I'd like to take this on! Reviewing the repository right now and will submit a PR shortly.

[poorandhungrybutbuthappy]

/claim #2

I am taking this on with a focused test-only change for the Go orderbook package: cover bid/ask cancellation depth updates, unknown-order and closed-book errors, snapshot copy immutability, then run the package-level Go tests and the required build validation.

[poorandhungrybutbuthappy]

/attempt #2

Submitted PR: #3

Implemented the focused orderbook cancellation/depth test coverage:

• cancelling bids removes bid depth and deletes the order id from the internal index
• cancelling asks removes ask depth
• unknown cancellations return ErrOrderNotFound
• closed books reject add/cancel with ErrBookClosed
• snapshots return copies so caller mutations do not alter internal book state

Validation:

• go test ./orderbook from market/ passed
• python build.py -m market passed locally

Note: build diagnostics generated large encrypted split .logd artifacts with local absolute paths, so I did not commit them publicly; I noted that in the PR.

[Nexussyn]

Bounty Submission — Runtime Open Federation

Payment address (USDC on Base): 0xb438d36b425b504724a1c72aa0941c80cb940995
We can complete this task and deliver within 24h. Please assign to confirm.