Skip to content

feat: sui staking-api docs#55

Merged
trikunai merged 1 commit intomainfrom
develop
Apr 7, 2026
Merged

feat: sui staking-api docs#55
trikunai merged 1 commit intomainfrom
develop

Conversation

@trikunai
Copy link
Copy Markdown
Member

@trikunai trikunai commented Apr 7, 2026

No description provided.

@trikunai trikunai self-assigned this Apr 7, 2026
@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot bot commented Apr 7, 2026

Code Review Summary

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 2
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
docs/staking-api/sui/signing.md 132 Fireblocks signature formatting is documented with an explicit assumption that may produce invalid Sui signatures; publishing this as-is can cause integration failures in production signing flows.
docs/staking-api/sui/signing.md 85 waitForTxCompletion loops without timeout/backoff, which can cause indefinite polling and rate-limit pressure in real integrations copied from docs.
Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File Line Issue
N/A N/A No additional out-of-diff issues identified.
Files Reviewed (6 files)
  • docs/staking-api/sui/_category_.json - 0 issues
  • docs/staking-api/sui/index.md - 0 issues
  • docs/staking-api/sui/native-staking/_category_.json - 0 issues
  • docs/staking-api/sui/native-staking/endpoints.md - 0 issues
  • docs/staking-api/sui/native-staking/index.md - 0 issues
  • docs/staking-api/sui/signing.md - 2 issues

Fix these issues in Kilo Cloud


Reviewed by gpt-5.3-codex · 48,344 tokens

@trikunai trikunai merged commit 852b6a0 into main Apr 7, 2026
2 checks passed
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