Complete DeepSeek integration documentation#22
Conversation
Co-authored-by: JetSquirrel <20291255+JetSquirrel@users.noreply.github.com> Agent-Logs-Url: https://github.com/JetSquirrel/cloudbridge/sessions/8aaf5f07-6cc9-45cf-9f31-add3deb8fccb
Co-authored-by: JetSquirrel <20291255+JetSquirrel@users.noreply.github.com> Agent-Logs-Url: https://github.com/JetSquirrel/cloudbridge/sessions/85fb11e4-6e1a-4468-b2c8-013f77116ef3
Co-authored-by: JetSquirrel <20291255+JetSquirrel@users.noreply.github.com> Agent-Logs-Url: https://github.com/JetSquirrel/cloudbridge/sessions/7dd84240-1bcb-4152-a94a-a29181f4a04e
There was a problem hiding this comment.
Pull request overview
Updates repository documentation and release metadata to reflect the already-implemented DeepSeek provider, clarifying that DeepSeek surfaces account balance (not cost history) and updating roadmap/changelog accordingly.
Changes:
- Documented DeepSeek as a fully supported provider in README (setup steps + balance vs. cost semantics).
- Updated roadmap/changelog entries to include/mark DeepSeek support for the 0.2.0 cycle.
- Bumped crate version to
0.2.0-dev(and updated lockfile package version).
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| README.md | Adds DeepSeek provider docs (configuration + usage semantics) and marks DeepSeek complete in roadmap. |
| Cargo.toml | Bumps application version to 0.2.0-dev. |
| Cargo.lock | Updates the cloudbridge package version entry to match Cargo.toml. |
| CHANGELOG.md | Adds a 0.2.0 section describing DeepSeek integration and planned items. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - Current month total cost (or balance for DeepSeek accounts) | ||
| - Last month total cost | ||
| - Month-over-month change | ||
| - Active accounts count |
There was a problem hiding this comment.
This bullet list implies the overview always includes meaningful “Last month total cost” and “Month-over-month change”, but DeepSeek accounts have no historical data (in code these values are effectively 0). Consider clarifying that those comparison stats are only meaningful for cost-based providers, while DeepSeek shows current balance and a balance breakdown.
| [package] | ||
| name = "cloudbridge" | ||
| version = "0.1.1" | ||
| version = "0.2.0-dev" |
There was a problem hiding this comment.
The PR is described as documenting an already-complete integration, but this change also bumps the crate version to 0.2.0-dev (and updates Cargo.lock). If the intent is documentation-only, consider reverting the version bump; if the intent is to start a 0.2.0 dev cycle, please call that out explicitly in the PR description/release notes so downstream packaging and tagging expectations are clear.
| ## [0.2.0] - TBD | ||
|
|
||
| ### Planned | ||
| - Azure support | ||
| - Google Cloud Platform support | ||
| - Cost alerts and notifications | ||
| - Budget tracking | ||
|
|
There was a problem hiding this comment.
Changelog structure may be misleading here: ## [0.2.0] - TBD reads like a released version section, but the project version is set to a pre-release (0.2.0-dev) and the Unreleased section is empty. Consider keeping these entries under Unreleased until 0.2.0 is actually cut, or renaming the section to reflect a pre-release/development cycle to avoid confusion.
| 3. Click on any account card to expand service-level details | ||
| 4. Click **Trend** to view the 30-day cost chart | ||
| 3. Click on any account card to expand service-level details (or balance breakdown for DeepSeek) | ||
| 4. Click **Trend** to view the 30-day cost chart (not available for DeepSeek) |
There was a problem hiding this comment.
The Usage docs say to click Trend, but the Dashboard UI appears to load/render trend data automatically when an account card is expanded (there isn’t a visible “Trend” button/CTA). Reword this step to match the actual interaction (e.g., expand an account card and view the “Cost Trend” section), and keep the DeepSeek exception note.
| 4. Click **Trend** to view the 30-day cost chart (not available for DeepSeek) | |
| 4. After expanding an account card, view the **Cost Trend** section to see the 30-day cost chart (not available for DeepSeek) |
The DeepSeek integration code was complete but undocumented. README.md made no mention of the new provider despite full implementation in codebase and CHANGELOG.
Changes
Implementation Notes
DeepSeek differs from AWS/Aliyun by displaying account balance rather than cost history:
/user/balanceendpointThe provider-specific behavior is already handled in
src/cloud/deepseek.rsand UI components (dashboard.rs:327,346,accounts.rs:130,275).