Skip to content

Add --completed shorthand flag to todos list#258

Merged
jeremy merged 1 commit intomainfrom
pr/todos-completed-flag
Mar 11, 2026
Merged

Add --completed shorthand flag to todos list#258
jeremy merged 1 commit intomainfrom
pr/todos-completed-flag

Conversation

@jeremy
Copy link
Copy Markdown
Member

@jeremy jeremy commented Mar 11, 2026

Summary

  • add --completed as a shorthand for listing completed todos
  • keep it mutually exclusive with --status
  • preserve existing todo list filtering behavior

Testing

  • bin/ci

Summary by cubic

Add --completed to the todos list command as a shorthand for --status completed. It’s mutually exclusive with --status and keeps existing filtering behavior unchanged.

Written for commit ffc517e. 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
Copy Markdown
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 2 files

Copy link
Copy Markdown

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 a convenience flag to the basecamp todos list command to make it easier to list completed todos while keeping existing filtering behavior and compatibility tracking.

Changes:

  • Introduces --completed on todos list as a shorthand for --status completed
  • Enforces mutual exclusivity between --completed and --status
  • Updates the .surface compatibility snapshot to include the new flag

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
internal/commands/todos.go Adds --completed flag plumbing + validation/mapping to the existing status filter.
.surface Records the new --completed flag in the CLI surface snapshot.

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

@jeremy jeremy merged commit 5f2cdc1 into main Mar 11, 2026
30 checks passed
@jeremy jeremy deleted the pr/todos-completed-flag branch March 11, 2026 04:59
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