Skip to content

Comments

fix(cli): align delete command wording with owner permissions#417

Open
lc0rp wants to merge 1 commit intoopenclaw:mainfrom
lc0rp-contrib:fix/413-delete-owner-wording
Open

fix(cli): align delete command wording with owner permissions#417
lc0rp wants to merge 1 commit intoopenclaw:mainfrom
lc0rp-contrib:fix/413-delete-owner-wording

Conversation

@lc0rp
Copy link

@lc0rp lc0rp commented Feb 18, 2026

Summary

  • update clawhub delete|hide|undelete|unhide command descriptions to state owner/moderator/admin access
  • update delete command confirmation suffix text to match the same permission model

Rationale

Backend soft-delete already allows owners to manage their own skills, and only requires moderator/admin for non-owners (convex/skills.ts, setSkillSoftDeletedInternal). CLI wording was stale and implied owners were blocked.

Testing

  • bunx vitest run packages/clawdhub/src/cli/commands/delete.test.ts
  • bun run lint

Refs #413

Greptile Summary

Updated CLI command descriptions and confirmation prompts to correctly reflect that skill owners can manage their own skills' soft-delete status without requiring moderator/admin privileges. The backend logic in convex/skills.ts (setSkillSoftDeletedInternal) already implements this permission model: it allows owners to manage their own skills and only requires moderator/admin access for non-owners.

Confidence Score: 5/5

  • Safe to merge - straightforward text-only documentation fix
  • Changes are purely textual updates to CLI help strings that align with existing backend implementation, no logic changes, and tests pass
  • No files require special attention

Last reviewed commit: b2e4144

@vercel
Copy link
Contributor

vercel bot commented Feb 18, 2026

@lc0rp is attempting to deploy a commit to the Amantus Machina Team on Vercel.

A member of the Team first needs to authorize it.

@lc0rp lc0rp marked this pull request as ready for review February 18, 2026 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant