Skip to content

Conversation

@jonhickman
Copy link
Contributor

@jonhickman jonhickman commented Oct 9, 2025

  • Add new remove-tags command following add-tags pattern
  • Support filtering by content type, visibility, existing tags, authors, and dates
  • Include both CLI and interactive modes
  • Interactive mode uses searchable tag selection
  • Add comprehensive documentation and examples

- Add new remove-tags command following add-tags pattern
- Support filtering by content type, visibility, existing tags, authors, and dates
- Include both CLI and interactive modes
- Interactive mode uses searchable tag selection
- Add comprehensive documentation and examples
- Fix tag filtering pattern consistency across tools
cursor[bot]

This comment was marked as outdated.

- Add 'Leave blank for all' pattern to tags to be removed field
- Fix date picker defaults to use logical 1-year cleanup window (2 years ago to 1 year ago)
- Ensure consistent UX patterns with other GCTools
- Handle empty tag selection to remove all tags from posts/pages
}

// Report success
ui.log.ok(`Successfully updated ${context.updated.length} posts in ${Date.now() - timer}ms.`);
Copy link

Choose a reason for hiding this comment

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

Bug: Context Management and Success Message Errors

The context object's errors and updated properties aren't reliably managed. The initialise task overwrites ctx.errors, preventing collected errors from appearing in the final log. context.updated isn't initialized in the command, which can lead to a runtime error if tasks fail before it's populated. Also, the success message incorrectly states "posts" regardless of the content type updated.

Additional Locations (3)

Fix in Cursor Fix in Web

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