fix: set coinbase sandbox correctly

[broody]

No description provided.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
controller-example-next	Ready	Preview	Feb 4, 2026 8:21pm
keychain	Ready	Preview	Feb 4, 2026 8:21pm
keychain-storybook	Ready	Preview	Feb 4, 2026 8:21pm

[claude]

---

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

---

[claude]

Claude finished @broody's task —— View job

---

Documentation Sync Analysis

Analyzing the Coinbase sandbox fix and updating documentation accordingly.

Tasks:

• Analyze changes in the merged PR files
• Explore docs-repo directory structure
• Determine if documentation updates are needed
• Update documentation files if required

Analysis Results

I've analyzed the changes in the merged PR titled "fix: set coinbase sandbox correctly". The key changes are:

Key Changes Made:

• Fixed Coinbase sandbox mode to properly use !isMainnet logic across all Coinbase-related functionality
• Ensures sandbox mode is enabled for testnet/development environments and disabled for mainnet
• Affects Coinbase quote fetching, order creation, and transaction monitoring

Documentation Review:
The existing coinbase-onramp.md documentation already correctly covers:

• Sandbox environment usage with automatic toggling based on network (line 126: "automatically enabled on testnets")
• This matches the implemented behavior where sandbox: !isMainnet

Conclusion:
No documentation updates are needed. The existing documentation at /controller/coinbase-onramp.md already accurately describes the sandbox behavior that was fixed in this PR. The fix ensures the implementation matches the documented behavior.