vrr/qwen-code
2
0
Fork 0
mirror of https://github.com/QwenLM/qwen-code.git synced 2026-08-21 14:46:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 22
qwen-code/docs/design/daemon-skill-batch-toggle.md
callmeYe c0a2fee4f7
fix(serve): allow preinstall skill batch toggles (#9139) 
Treat names absent from the installed Skill snapshot as valid batch targets so their workspace state can be declared before installation. Preserve the existing persistence semantics: an undeclared enable is a no-op, an existing workspace disable is removed, and disable writes the declaration.
2026-08-14 09:04:09 +00:00

2.4 KiB
Raw Permalink Blame History

Daemon Skill batch toggle

Problem

Remote Skill managers can toggle only one Skill per request. Closing several Skills therefore requires client-side request orchestration and provides no single response that records all target outcomes.

API

Add collection-level mutation routes:

  • POST /workspace/skills/enable
  • POST /workspaces/:workspace/skills/enable

The request body is:

{
  "skillNames": ["review", "deploy", "missing"],
  "enabled": false
}

skillNames is a non-empty string array with at most 100 entries. Names are trimmed and deduplicated case-insensitively while preserving first-seen order. The response is best-effort for expected target errors: installed targets are validated against one status snapshot, all valid names are persisted in one locked write, and changes are applied with one live-session refresh. Names that are not installed remain valid so callers can declare their state before installation. Enabling one removes a matching workspace skills.disabled entry and is otherwise a no-op, except for the existing defaultDisabled override behavior; disabling one writes skills.disabled. Hidden, inactive-extension, and locked targets are returned without blocking valid targets. Unexpected persistence and runtime-generation failures fail the whole request.

{
  "enabled": false,
  "activation": "applied",
  "sessionsRefreshed": 2,
  "sessionsFailed": 0,
  "results": [
    {
      "skillName": "review",
      "enabled": false,
      "changed": true
    },
    {
      "skillName": "deploy",
      "enabled": false,
      "changed": true
    },
    {
      "skillName": "missing",
      "enabled": false,
      "changed": true
    }
  ],
  "errors": []
}

results and errors each preserve request order within their own array; the response does not reconstruct the original mixed ordering, so clients re-match targets by skillName.

Malformed requests still fail as a whole with HTTP 400. Workspace trust, authentication, client identity, and generation ownership use the same gates as the single-Skill route.

Compatibility

Advertise workspace_skill_batch_toggle separately from workspace_skill_toggle. Clients must pre-flight the new capability before calling the collection route. The existing single-Skill route and response remain unchanged. The collection routes are HTTP-only: the ACP _qwen/workspace/skills dispatch surface stays read-only, matching the single-Skill toggle.