Skip to content

Add recordable trash, archive, and restore subcommands#259

Merged
jeremy merged 1 commit intomainfrom
pr/recordable-status-actions
Mar 11, 2026
Merged

Add recordable trash, archive, and restore subcommands#259
jeremy merged 1 commit intomainfrom
pr/recordable-status-actions

Conversation

@jeremy
Copy link
Member

@jeremy jeremy commented Mar 11, 2026

Summary

  • add shared trash, archive, and restore helpers for recordable entities
  • wire the new actions into todos, todolists, messages, cards, files, and comments
  • update the command catalog and CLI surface to match the expanded action set

Testing

  • bin/ci

Summary by cubic

Add trash, archive, and restore commands for recordable items to manage status from the CLI. Applies to todos, todolists, messages, cards, files, comments, docs, uploads, and vaults, and updates the command catalog and CLI surface.

  • New Features
    • Added shared helpers: newRecordableTrashCmd, newRecordableArchiveCmd, newRecordableRestoreCmd (uses runRecordingsStatus with "trashed"/"archived"/"active").
    • Wired into: todos, todolists, messages, cards, files, comments, docs, uploads, vaults.
    • Commands accept an ID or Basecamp URL; flags and help are consistent across resources.
    • Updated CommandCategories and regenerated .surface to include the new actions in listings and flags, and to show "download" where applicable.

Written for commit 5e9baa0. Summary will update on new commits.

@jeremy jeremy requested a review from a team as a code owner March 11, 2026 03:50
Copilot AI review requested due to automatic review settings March 11, 2026 03:50
@github-actions github-actions bot added the commands CLI command implementations label Mar 11, 2026
@github-actions github-actions bot added the enhancement New feature or request label Mar 11, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 9 files

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds shared “trash / archive / restore” subcommands for recordable entities and wires them into multiple existing command groups, updating the command catalog and CLI surface snapshot accordingly.

Changes:

  • Introduces reusable newRecordableTrashCmd / newRecordableArchiveCmd / newRecordableRestoreCmd helpers backed by runRecordingsStatus.
  • Adds trash, archive, and restore subcommands to todos, todolists, messages, cards, files (and thus docs/uploads/vaults via aliases), and comments.
  • Updates the command catalog (CommandCategories) and .surface snapshot to reflect the expanded command surface.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
internal/commands/todos.go Adds trash/archive/restore subcommands to the todos command group.
internal/commands/todolists.go Adds trash/archive/restore subcommands to the todolists command group.
internal/commands/recordings.go Adds shared recordable command constructors that call runRecordingsStatus.
internal/commands/messages.go Adds trash/archive/restore subcommands to messages.
internal/commands/files.go Adds trash/archive/restore subcommands to files (and aliases inherit them).
internal/commands/comment.go Adds trash/archive/restore subcommands to comments.
internal/commands/commands.go Updates command catalog action lists to include trash/archive/restore for affected commands.
internal/commands/cards.go Adds trash/archive/restore subcommands to cards.
.surface Updates CLI surface snapshot for new subcommands/flags.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jeremy jeremy force-pushed the pr/recordable-status-actions branch from 7b474fe to 5e9baa0 Compare March 11, 2026 04:29
@jeremy jeremy merged commit 821d97d into main Mar 11, 2026
25 checks passed
@jeremy jeremy deleted the pr/recordable-status-actions branch March 11, 2026 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commands CLI command implementations enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants