[mbjorkqvist]

Follow-up to #274, from a consistency pass over the design, the deployment guide, the changelog and the Candid comments of the automatic deposit flow. It only touches documentation and comments.

• The design still described getMultipleAccounts as not yet offered by the SOL RPC canister, while the flow already uses the typed method.
• The changelog said the watchlist is recorded before every upgrade; it is recorded only when it holds entries, which is what keeps an upgrade without the flow enabled reversible.
• The deployment guide said to set the automatic deposit fee "once it is complete", next to the section that enables it in the installing upgrade.
• The manual flow diagram applied the minimum deposit amount to the sweepable amount instead of the balance.
• Smaller alignments: the overview figure lacks watch_deposit_address, the crediting of a sweep is not specific to the manual flow, the Quarantined error of deposit_sol names only one of the two causes, the cycle figures in the constants differ from the design, and the repository structure omits the automatic flow.

🤖 Generated with Claude Code

[zeropath-ai]

✅ No security or compliance issues detected. Reviewed everything up to f2d0a26.

Security Overview

• 🔎 Scanned files: 9 changed file(s)
• 🔗 Scan Link: https://zeropath.com/app/repositories/58f1cdfe-c0c2-41c1-8d28-3d6efcaf3266?scanId=9a63be70-f058-465e-9751-1b378e3d0566&codeScanTypes=PrScan&tab=issues&detectionTypes=traditional,businessLogic,customRule,IAC,SECRETS,CICD,EOL,uncategorized

Detected Code Changes

Change Type	Relevant files
Enhancement	► libs/types-internal/src/event.rs     Update TransactionPurpose description to include deposits swept by balance scans in addition to those queued by deposit_sol
Enhancement	► minter/src/constants.rs     Adjust cycle cost commentary: getBalance, getAccountInfo, and related constants to reflect updated values (2.3B vs 2.1B, 23M per address)
Enhancement	► minter/src/state/event.rs     Update TransactionPurpose description to include deposits swept by balance scans
Enhancement	► minter/CHANGELOG.md     Update note about watchlist upgrade behavior and related wording
Enhancement	► minter/cksol_minter.did     Reflect updated TransactionPurpose documentation for swept deposits (including balance scans) in DID
Enhancement	► minter/src/constants.rs (duplicate entry in diff context)     Align comments for per-address and total cycle estimates with updated values
Enhancement	► libs/types/src/lib.rs     Update error description for quarantined deposits to mention DepositSolStatus and clarify quarantine conditions

[Copilot]

🟢 Approval recommended

No active review issues remain in the documentation and API-comment updates for the automatic deposit flow.

No open issues 🎉

What changed in this PR

Aligns documentation and API comments with the automatic deposit flow introduced in #274. No runtime behavior or interface definitions change.

Changes:

• Updates the design to reflect typed RPC support, balance-based deposit eligibility, and shared sweep processing.
• Clarifies automatic-flow enablement and watchlist persistence during upgrades.
• Aligns quarantine descriptions, cycle-cost comments, and the repository layout documentation.

File	Description
minter/​src/​constants.rs	Aligns RPC cycle-cost comments with the design estimates.
minter/​cksol_minter.did	Documents both causes of deposit quarantine.
minter/​CHANGELOG.md	Clarifies that empty watchlists are not recorded before upgrades.
libs/​types/​src/​lib.rs	Aligns Rust quarantine documentation with Candid comments.
docs/​src/​development/​repository-structure.md	Adds the automatic deposit module to the layout.
docs/​src/​development/​design.md	Updates deposit diagrams, RPC availability, sweep descriptions, and cost-estimate wording.
docs/​src/​development/​deployment.md	Clarifies how setting the automatic deposit fee enables the flow.

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

No active review issues were identified for the documentation updates clarifying the automatic deposit flow.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

No active review issues are reported for the documentation and comment updates describing automatic deposits.

No open issues 🎉

_{Review effort: Balanced}