Skip to content

add error context to ctf contract calls#77

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

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

Conversation

@ClawdisAI

@ClawdisAI ClawdisAI commented May 28, 2026

Copy link
Copy Markdown

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.

@ClawdisAI ClawdisAI 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.

1 participant