Repository navigation
fix(console): drop the three retired /api/v1/ai/assistant try-it rows from the Developer API console (objectui#11704) - #11730
Merged
objectstack-fleet[bot] merged 1 commit intoOct 6, 2026
Conversation
… from the Developer API console (objectui#11704) The server retired GET /assistant, GET /assistant/skills and POST /assistant/chat (objectstack-ai/cloud#2621, ruling A); each now answers 404, so the three hand-written rows in the static `ai` catalog group could only fail. The rows, their "Ambient assistant" comment and the count comments they falsified are gone; the named-agent chat route stays as the one chat door. A pin in useApiDiscovery.test.ts keeps the agent route and refuses an /assistant row. Claude-Session: https://claude.ai/code/session_01DrKzdPdyLLBW3qpZ4vtk7z Co-authored-by: Claude <noreply@anthropic.com>
Contributor
✅ Console Performance Budget
The eager closure is every chunk the entry reaches through static imports — what the browser fetches and parses before the app renders. The entry chunk on its own is a small fraction of it. 📦 Bundle Size Report
Size Limits
|
objectstack-fleet
Bot
deleted the
claude/issue-11704-drop-retired-assistant-rows
branch
October 6, 2026 16:47
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
2 participants
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.
Fixes #11704
Clause-②: no
The Developer API console's AI group no longer lists the three retired ambient-assistant routes as try-it rows:
GET /api/v1/ai/assistant,GET /api/v1/ai/assistant/skillsandPOST /api/v1/ai/assistant/chat. The maintainer ruled them retired on objectstack-ai/cloud#2621 (ruling A). A server that carries that change answers each one with 404, so every row was a try-it button that could only fail.POST /api/v1/ai/agents/:agentName/chatstays in the group as the one chat door.Dispatched by the
domain:uiseat 1 PM (claim comment 6019944567 and its addendum 6020010808 on #11704); sessionsession_01DrKzdPdyLLBW3qpZ4vtk7z.What changed
apps/console/src/pages/developer/hooks/useApiDiscovery.ts: the three rows and the "Ambient assistant" comment above them are removed. The comment that counts the conditionally mounted builders now says three, with/assistant/*dropped from its list. The agent row, the tools rows and everything else in the group are untouched.apps/console/src/pages/developer/hooks/useApiDiscovery.test.ts: no test pinned the AI group before, so one new pin is added. It renders the hook against a discovery answer that reports theaislot usable, asserts the named-agent chat row is present, and asserts no row carries/assistant. The presence assertion is the positive control that makes the empty list a measurement..changeset/11704-drop-retired-assistant-rows.md: apatchfor@object-ui/console.apps/consoleis in the changesetfixedgroup and the edit is under itssrc/, so AGENTS.md section 9 requires a declaration. The change is user-visible (rows leave the shipped console), so it is a real patch entry, not the empty-frontmatter form.node scripts/check-changeset-presence.mjsreads it as satisfied.Premises measured on origin/main before editing
aigroup ofSERVICE_ENDPOINT_CATALOG, whosedefaultRouteis/api/v1/ai: confirmed. The group is gated on/discoveryreporting theaislot usable and is not served from a route listing.git grepforai/assistanthad no code hit, and a wider scan for a/assistantpath (quote, backtick or slash led, imports androleliterals filtered out) hit only the three rows and the count comment in this one file. Positive control in the same file: the/api/v1/aidefault-route string hits.Acceptance notes
/assistantrow planted back into the catalog turned the new pin red (1 failed, 14 passed in the file,expected [ 'GET /api/v1/ai/assistant' ] to deeply equal []), and the byte-identical restore was proven (blob equal to HEAD,git diff HEADempty). The first attempt was a no-op: the tool refused it because the replacement text still contained the anchor, so nothing ran; it was redone with a replaced anchor. The tested module is imported by relative path fromsrc, so the ablation has nodist/leg and needed no build.Done whenand agrees with the rows left in this file (/agents/*,/evals/runs,/conversations/:id/debug).mainmay drop the rows now. A deployment pinned to an older cloud keeps its old console.26f43d6: the console dependency-closure build, the full@object-ui/consoletest run (154 files, 1799 tests) andtype-check, pluscheck:control-bytes,check:new-line-citations,check:changeset-claims,check:pending-changeset-literals, the fourcheck-changeset-*scripts andcheck:test-path-roots. Repo-wide lint and the rest of the farm are left to CI.Generated by Claude Code