Skip to content

add error context to ctf contract calls#77

Closed
notyrjo wants to merge 1 commit into
Polymarket:mainfrom
notyrjo:fix/ctf-error-context
Closed

add error context to ctf contract calls#77
notyrjo wants to merge 1 commit into
Polymarket:mainfrom
notyrjo:fix/ctf-error-context

Conversation

@notyrjo
Copy link
Copy Markdown

@notyrjo notyrjo commented May 28, 2026

The three contract read calls in ctf.rs (getConditionId, getCollectionId, getPositionId) propagate errors with ? but without context, so when they fail the user sees a raw RPC or transport error with no indication of which operation failed.

This adds .context() messages consistent with the error handling pattern used elsewhere in the codebase.


Note

Low Risk
Read-only CLI error messaging only; no behavior, auth, or on-chain logic changes.

Overview
Adds anyhow::Context messages on the three read-only CTF contract calls in ctf.rs (getConditionId, getCollectionId, getPositionId) so RPC/transport failures name the operation instead of surfacing a bare error.

This matches the existing pattern on write paths (e.g. "Split position failed") and on input parsing in the same file.

Reviewed by Cursor Bugbot for commit 3baa0d4. Bugbot is set up for automated code reviews on this repo. Configure here.

@notyrjo notyrjo closed this by deleting the head repository May 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants