feat(channel): add delete, archive, unarchive subcommands

[scottlovegrove]

Summary

Ports the remaining channel-lifecycle commands from Doist/twist-cli#246. create already existed in comms-cli, so this adds the missing three:

tdc channel delete <ch-ref> --yes        # permanently delete (usually admin-only)
tdc channel archive <ch-ref>             # archive (no-op if already archived)
tdc channel unarchive id:<id>            # unarchive

All three support --workspace, --dry-run and --json.

• delete requires --yes to mutate; short-circuits a structured MISSING_YES_FLAG CliError in --json mode before any lookup, and translates a Comms 403 into a clear FORBIDDEN CliError (channel deletion is typically restricted to workspace admins).
• archive / unarchive share a setArchiveState helper and skip the API call when the channel is already in the target state. Name-ref resolution only finds active channels, so unarchive documents passing id:/numeric refs.

Adaptations from the twist-cli source

comms-cli diverges, so this is not a 1:1 copy:

• String channel IDs — comms channel id is a base58 string; the SDK's deleteChannel/archiveChannel/unarchiveChannel take id: string.
• Direct client calls — getCommsClient().channels.X(...) (matching sibling create.ts/update.ts), not api.ts wrappers. Only spinner-message entries were added to api.ts.
• CommsRequestError (from @doist/comms-sdk) for the 403 check, instead of twist's TwistRequestError.
• tdc naming throughout.

No scope changes needed — channels:write/channels:remove already landed with the membership PR (#7). The read-only write-gate applies automatically via the safe-by-default isMutatingMethod.

Files

Layer	File
Delete	src/commands/channel/delete.ts (new)
Archive/Unarchive	src/commands/channel/archive.ts (new)
Spinners	src/lib/api.ts
Error code	src/lib/errors.ts (FORBIDDEN)
Registration	src/commands/channel/index.ts
Skill docs	src/lib/skills/content.ts + regenerated skills/comms-cli/SKILL.md
Tests	src/commands/channel/channel.test.ts (+13)

Test plan

• npm run type-check
• npm run lint
• npm test — 668/668 passing (13 new across delete/archive/unarchive)
• npm run sync:skill (SKILL.md regenerated)
• --help smoke for all three commands
• Live end-to-end mutation against a real workspace (create → archive → list --state archived → unarchive → delete)

🤖 Generated with Claude Code

[doistbot]

Thanks scottlovegrove for your contribution 🤗. The implementation for these new lifecycle commands is excellent, particularly the shared archive state mutation and clean error mapping.

Few things worth tightening:

• Bypass workspace resolution for direct channel refs (id: or URLs) using getDirectChannelId(), matching how channel update handles cross-workspace channels.
• Add test coverage for the --workspace flag across these new subcommands to verify the correct workspace ID is passed to the resolver.

I also included a few optional follow-up notes in the details below.

Optional follow-up note (1)

• [P3] src/commands/channel/delete.ts:19: This does the full workspace/channel resolution before the plain !options.yes early return below. On a name ref, resolveChannelRef() loads the channel list for the workspace, so tdc channel delete <name> without --yes still pays for multiple API calls just to print Use --yes to confirm. Consider moving the non---yes/non---dry-run return above the lookup and using the raw ref in that message so the no-op path stays local.

_{Share Feedback • Review Logs}

[doist-release-bot]

🎉 This PR is included in version 1.2.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[henningmu]

✅