Skip to content

fix: restore missing circuit-breaker closing brace and threshold read from bad merge - #850

Open
adelekevictor12 wants to merge 4 commits into
accesslayerorg:mainfrom
adelekevictor12:feat/slippage-cooldown-property-tests
Open

fix: restore missing circuit-breaker closing brace and threshold read from bad merge#850
adelekevictor12 wants to merge 4 commits into
accesslayerorg:mainfrom
adelekevictor12:feat/slippage-cooldown-property-tests

Conversation

@adelekevictor12

Copy link
Copy Markdown

Closes #835

adelekevictor12 and others added 2 commits September 2, 2026 11:35
… from bad merge

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>
🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>
@Chucks1093

Copy link
Copy Markdown
Member

❌ CI Failed — verify (Contracts CI)

The verify check is failing. The PR title suggests a merge conflict fix — the circuit breaker closing brace or threshold read may still have a compile issue.

Likely causes:

  • Unclosed delimiter or brace mismatch introduced during merge resolution
  • Incorrect storage key used for the threshold read after merge

Steps to fix:

  1. Run cargo build locally and check for any unclosed delimiter or type mismatch errors
  2. Run cargo fmt --all and commit
  3. Push — CI will re-run

@adelekevictor12

Copy link
Copy Markdown
Author

ok

adelekevictor12 and others added 2 commits September 7, 2026 10:31
Resolve merge conflicts with latest upstream main:
- lib.rs: take upstream's whitespace-clean circuit-breaker code (fix already merged upstream)
- test_issues_778_779_781_782.rs: keep .unwrap() variants matching Result-returning fee views
- tests/staking_reward_claim.rs: drop obsolete 2-arg API test deleted upstream

Generated with Codebuff 🤖
Co-Authored-By: Codebuff <noreply@codebuff.com>
The branch's copy of the test file used the old client API (initialize,
with_mut, .unwrap() on auto-unwrapped client returns) that no longer
compiles against the merged lib.rs. Take upstream's version adapted to
the current entrypoints (set_protocol_admin/set_treasury_address/
set_key_price, ledger get/set, direct client values).

Generated with Codebuff 🤖
Co-Authored-By: Codebuff <noreply@codebuff.com>
@adelekevictor12

Copy link
Copy Markdown
Author

@Chucks1093 done

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.

Add property-based tests asserting the slippage protection invariant and cooldown guard across random inputs

2 participants