Skip to content

Commit 53c4c20

Browse files
authored
fix: route ordering for cloud (#499)
Signed-off-by: phernandez <[email protected]>
1 parent b4486d2 commit 53c4c20

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

src/basic_memory/api/app.py

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -92,17 +92,7 @@ async def _file_sync_runner() -> None:
9292
lifespan=lifespan,
9393
)
9494

95-
# Include v1 routers
96-
app.include_router(knowledge.router, prefix="/{project}")
97-
app.include_router(memory.router, prefix="/{project}")
98-
app.include_router(resource.router, prefix="/{project}")
99-
app.include_router(search.router, prefix="/{project}")
100-
app.include_router(project.project_router, prefix="/{project}")
101-
app.include_router(directory_router.router, prefix="/{project}")
102-
app.include_router(prompt_router.router, prefix="/{project}")
103-
app.include_router(importer_router.router, prefix="/{project}")
104-
105-
# Include v2 routers (ID-based paths)
95+
# Include v2 routers FIRST (more specific paths must match before /{project} catch-all)
10696
app.include_router(v2_knowledge, prefix="/v2/projects/{project_id}")
10797
app.include_router(v2_memory, prefix="/v2/projects/{project_id}")
10898
app.include_router(v2_search, prefix="/v2/projects/{project_id}")
@@ -112,6 +102,16 @@ async def _file_sync_runner() -> None:
112102
app.include_router(v2_importer, prefix="/v2/projects/{project_id}")
113103
app.include_router(v2_project, prefix="/v2")
114104

105+
# Include v1 routers (/{project} is a catch-all, must come after specific prefixes)
106+
app.include_router(knowledge.router, prefix="/{project}")
107+
app.include_router(memory.router, prefix="/{project}")
108+
app.include_router(resource.router, prefix="/{project}")
109+
app.include_router(search.router, prefix="/{project}")
110+
app.include_router(project.project_router, prefix="/{project}")
111+
app.include_router(directory_router.router, prefix="/{project}")
112+
app.include_router(prompt_router.router, prefix="/{project}")
113+
app.include_router(importer_router.router, prefix="/{project}")
114+
115115
# Project resource router works across projects
116116
app.include_router(project.project_resource_router)
117117
app.include_router(management.router)

0 commit comments

Comments
 (0)