Skip to content

fix(BA-6922): use the scoped search route shape the convention documents#13035

Closed
jopemachine wants to merge 2 commits into
feat/BA-6922-app-config-fragment-search-rest-v2from
fix/fragment-scoped-search-route
Closed

fix(BA-6922): use the scoped search route shape the convention documents#13035
jopemachine wants to merge 2 commits into
feat/BA-6922-app-config-fragment-search-rest-v2from
fix/fragment-scoped-search-route

Conversation

@jopemachine

Copy link
Copy Markdown
Member

📚 Stacked PRs

Merge in order, bottom-up. Each PR is based on the one above it, so its diff shows only its own layer.

Related: #12929 (BA-6922)

Summary

POST /v2/app-config-fragments/scoped-search matches neither documented route shape:

shape
api/rest/v2/AGENTS.md, current POST /v2/{entity}/{scope_type}/{scope_id}/search
api/rest/v2/AGENTS.md, forward direction POST /v2/{entity}/scoped/search
before POST /v2/app-config-fragments/scoped-search
after POST /v2/app-config-fragments/scoped/search

scheduling-history already ships the forward form as POST /v2/scheduling-history/kernels/scoped/search, so this follows it rather than inventing a third spelling.

The route is introduced by #13020 and does not exist on main, so this is based on that branch.

Test plan

  • openapi.json regenerated with ./backend.ai mgr api dump-openapi; the delta is the single path string
  • CI verification

🤖 Generated with Claude Code

`POST /scoped-search` matches neither documented form: the current
`/{entity}/{scope_type}/{scope_id}/search` nor the forward
`/{entity}/scoped/search`. `scheduling-history` already ships the latter as
`/kernels/scoped/search`, so follow it.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jopemachine
jopemachine requested a review from a team as a code owner July 22, 2026 08:03
@github-actions github-actions Bot added size:XS ~10 LoC area:docs Documentations comp:manager Related to Manager component labels Jul 22, 2026
@jopemachine
jopemachine marked this pull request as draft July 22, 2026 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:docs Documentations comp:manager Related to Manager component size:XS ~10 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant