gateway: plan housekeeping — tick phase 8/9 checklists, renumber phase 9 - #1693
Merged
Merged
Conversation
…e 9, drop dead useSpendByAgent param Docs reflect what actually shipped, judged from code rather than the tickboxes: - phase-8 and phase-9 wire-up checklists ticked; compound lines split so unshipped items (Config page + /_plugin/config/*, Sessions, Runs list, BudgetEditor, Bucket/DimensionPicker) stay unticked. DataTable noted as sortable-not-paginated: the run call log truncates at the server's 50-row default with no pager. - phase-9 doc was titled and self-referenced as "Phase 8" from before the numbering shift; renumbered, with a status note at the top. References to the cookie/session layer stay "phase 8" because that layer shipped there. - llm-governance-v2 spec table now points at phases 8 and 9; phase-3 notes the admin pair also signs into /_plugin/ui; phase-12 records the unsigned /_plugin/tlog/status route the status card uses. - README page list updated (Canvas, People, UserDetail, catalog tabs, kill switches) with what phase 9 still owes. - Sidebar comment no longer promises Sessions/Config as present. - useSpendByAgent's userID param removed: no caller passed it and UserDetail reads agents_used from /users/:id instead.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Follow-up to a plan-vs-UI audit. Docs-only apart from one dead parameter.
/_plugin/config/*override layer, Sessions, a/runslist, BudgetEditor, Bucket/DimensionPicker.llm-governance-v2spec table gains phase 8 and 9 rows. Phase 3 notes the admin pair also signs into/_plugin/ui. Phase 12 records the unsigned/_plugin/tlog/statusroute behind the status card.useSpendByAgent(window, userID?)losesuserID: nothing passed it, UserDetail readsagents_usedfrom/users/:id.npm run typecheckclean.