vrr/qwen-code
2
0
Fork 0
mirror of https://github.com/QwenLM/qwen-code.git synced 2026-07-10 01:29:17 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 12

fix(cli): preserve workspace selector error contract

Browse source 
Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
This commit is contained in:
doudouOUC 2026-07-09 01:07:05 +08:00
parent 1d9ed7406e
commit 6979afe890
3 changed files with 16 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
docs/developers/qwen-serve-protocol.md
View file

@ -479,7 +479,7 @@ do not mutate state, and do not change the serve protocol version. Workspace
status routes intentionally do **not** start the ACP child process just because
a client polls a GET route: if the daemon is idle, they return
`initialized: false` with an empty snapshot. Session status routes require a
live session and use the standard `404 SessionNotFoundError` shape for unknown
live session and return `404 { code: "session_not_found", ... }` for unknown
ids.
Capability tags:
@ -1329,7 +1329,7 @@ Use `/load` when the client has no history rendered (cold reconnect, picker →
### `GET /workspace/:id/sessions` and `GET /workspaces/:workspace/sessions`
List sessions whose canonical workspace matches `:id` or `:workspace`. The path parameter first resolves as an exact workspace id and then as a URL-encoded absolute cwd. `GET /workspaces/:workspace/sessions` is a plural alias with the same response shape. Primary workspaces include the existing persisted/live merge: the default list is active sessions from `chats/`; pass `archiveState=archived` to list archived sessions from `chats/archive/`. Non-primary workspaces are live-only in the current multi-workspace sessions surface and reject archived, organized, or grouped queries. `archiveState=all` is not supported in v1. The default response and numeric `cursor` semantics are unchanged by `session_organization`.
List sessions whose canonical workspace matches `:id` or `:workspace`. The path parameter first resolves as an exact workspace id and then as a URL-encoded absolute cwd. `GET /workspaces/:workspace/sessions` is a plural alias with the same response shape. Primary workspaces include the existing persisted/live merge: the default list is active sessions from `chats/`; pass `archiveState=archived` to list archived sessions from `chats/archive/`. Non-primary workspaces are live-only in the current multi-workspace sessions surface and reject archived, organized, or grouped queries. Untrusted non-primary workspaces return `403 { code: "untrusted_workspace" }`. `archiveState=all` is not supported in v1. The default response and numeric `cursor` semantics are unchanged by `session_organization`.
```bash
curl http://127.0.0.1:4170/workspace/$(jq -rn --arg c "$PWD" '$c|@uri')/sessions

13
packages/cli/src/serve/multi-workspace-sessions.test.ts
View file

@ -744,6 +744,19 @@ describe('multi-workspace session dispatch', () => {
expect(unknown.body.workspaceCount).toBe(2);
});
it('preserves the legacy invalid workspace selector message', async () => {
const { app } = makeHarness();
const res = await request(app)
.get('/workspace/not:an:absolute:path/sessions')
.set('Host', host());
expect(res.status).toBe(400);
expect(res.body.error).toBe(
'`:id` must decode to a workspace id or absolute path',
);
});
it('rejects untrusted non-primary workspace session listing', async () => {
const daemonLog = makeDaemonLog();
const { app, secondaryBridge } = makeHarness({

2
packages/cli/src/serve/routes/session.ts
View file

@ -230,7 +230,7 @@ export function registerSessionRoutes(
if (byId) return byId;
if (!path.isAbsolute(workspaceParam)) {
res.status(400).json({
error: `:${paramName} must decode to a workspace id or absolute path`,
error: `\`:${paramName}\` must decode to a workspace id or absolute path`,
});
return null;
}