Repository navigation
feat: remove native chat cost tracking in favor of AI Gateway cost data #27330
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
4d333ea
feat: remove native chat cost tracking in favor of AI Gateway cost data
ibetitsmike b342a41
fix(site/src/pages/AgentsPage): correct mobileBack prop doc
ibetitsmike ce810ad
fix(site/src/pages/AgentsPage): attach the label docblock to its func…
ibetitsmike 662482d
docs: apply spend management review feedback
ibetitsmike b830ac6
fix: reconcile native cost removal with upstream changes
ibetitsmike 1f2e0e2
docs: mark spend management premium and document CSV output
ibetitsmike 2eb64d5
docs: document native cost migration
ibetitsmike 065ef43
docs: document native cost export pagination
ibetitsmike fb6c996
docs: clarify native cost pagination stop
ibetitsmike File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
docs: document native cost migration
- Loading branch information
commit 2eb64d5dfa892ae01a12dd3c446f610abb77f5fa
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As a total aside, I was confused when I tried to search for "usage insights" in the docs, and I had to go to
manifest.jsonto figure out that the page's title is actually "Spend management" now. Should we rename and redirect this page? Is "Spend management" closer to what we want to convey for this page? If so, the URL should also be updated.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Accepted the rename in
533779f7cf8: the page is nowspend-management.md, and the manifest plus all in-repo links use the new path. The public redirect from the old URL must be added tocoder/coder.com'sredirects.json, so I am leaving this thread open until that companion change lands.\n\n> Mux applied this feedback on Mike's behalf.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nick, could you add the companion redirect in
coder/coder.com? Thecoder/coderrename and link updates are already pushed.{"source":"/docs/ai-coder/agents/platform-controls/usage-insights","destination":"/docs/ai-coder/agents/platform-controls/spend-management","permanent":true}There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done — companion redirect is up in coder/coder.com#1001, and I've added you as reviewer there. It adds the permanent
/docs/ai-coder/agents/platform-controls/usage-insights→/docs/ai-coder/agents/platform-controls/spend-managementredirect.I flagged on that PR that it should not merge until this one lands, so we don't redirect the still-live page to a path that doesn't exist yet. Leaving this thread open until both merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR here for your review: https://github.com/coder/coder.com/pull/1001