Skip to content

fix(core): update contract error resolution signatures and add error …#279

Open
Bilalishaq7 wants to merge 1 commit into
Toolbox-Lab:mainfrom
Bilalishaq7:feat/error-documentation-links
Open

fix(core): update contract error resolution signatures and add error …#279
Bilalishaq7 wants to merge 1 commit into
Toolbox-Lab:mainfrom
Bilalishaq7:feat/error-documentation-links

Conversation

@Bilalishaq7

Copy link
Copy Markdown

Closes #263

Overview

I'm all done with this implementation! This PR introduces the missing learn_more error documentation links to our diagnostic reports and resolves the compilation issues that cropped up from the recent context updates.

What's Done

  • Error Links Added: Embedded the unified Stellar developer documentation URL directly into the DiagnosticReport struct generation logic inside report.rs.
  • Module Resolution Fix: Corrected the module layout in crates/core/src/decode/mod.rs to map correctly to decode_context.rs instead of the legacy context.rs name.
  • Type Signature Fix: Updated the operational filtering loop to construct a proper DecodeContext from the network configurations and pass its reference cleanly into contract_error::resolve.
  • Code Cleanup: Removed an obsolete duplicate call to enrich_report.

Verification

  • Ran local testing suite via cargo test and everything builds and passes cleanly without diagnostics errors.

Closes #263

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.

Attach a "Learn More" Documentation Link to Every Decoded Error Code

1 participant