-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
Skill Workshop apply popup carries no proposal details + fixed 60s TTL #93173
Copy link
Copy link
Closed
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:needs-security-reviewClawSweeper marked this issue as needing security-sensitive review.ClawSweeper marked this issue as needing security-sensitive review.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:securitySecurity boundary, credential, authz, sandbox, or sensitive-data risk.Security boundary, credential, authz, sandbox, or sensitive-data risk.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:needs-security-reviewClawSweeper marked this issue as needing security-sensitive review.ClawSweeper marked this issue as needing security-sensitive review.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:securitySecurity boundary, credential, authz, sandbox, or sensitive-data risk.Security boundary, credential, authz, sandbox, or sensitive-data risk.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Type
Fields
Priority
None yet
Title: Skill Workshop apply popup carries no proposal details + fixed 60s TTL
Version: OpenClaw 2026.6.1 (2e08f0f)
Doc: https://docs.openclaw.ai/tools/skill-workshop
When the agent calls skill_workshop with action: apply, the approval popup only shows fixed text. It does not include the proposal id, target skill name, 160-byte description, or body preview. The popup also auto-expires in 60 seconds, which is too short to read a multi-file skill diff.
If I leave the chat for an hour, I cannot decide from the popup alone. I have to drop into a terminal and run "openclaw skills workshop inspect " to see anything.
Ask 1: Enrich Skill Workshop lifecycle approval requests with proposal metadata.
What I would like the apply popup to show:
Currently the popup only shows:
The hard-coded text comes from the action-level lifecycle approval text. The plugin approval plumbing supports carrying extra fields, but the Skill Workshop path does not pass them through.
Ask 2: Add a configurable (or longer default) timeout for Skill Workshop approvals.
For a skill that touches ~10 files, 60 seconds is not enough.
What I would like:
The generic plugin approval plumbing supports a per-request timeoutMs internally, capped at 10 minutes, but Skill Workshop does not set one today and there is no documented config knob.
Current workaround:
None of these fix the popup itself.