Skip to content

fix(tag-manager): require enabled plugin for sley tag commands#212

Merged
indaco merged 2 commits intomainfrom
fix/tag-manager-require-enabled
Feb 26, 2026
Merged

fix(tag-manager): require enabled plugin for sley tag commands#212
indaco merged 2 commits intomainfrom
fix/tag-manager-require-enabled

Conversation

@indaco
Copy link
Copy Markdown
Owner

@indaco indaco commented Feb 26, 2026

Description

All sley tag subcommands (create, list, push, delete) now require tag-manager.enabled: true in .sley.yaml. Previously, sley tag create would print a warning but proceed with default settings - this warn-and-continue behavior was confusing and inconsistent.

Changes:

  • Add a Before hook on the parent tag command that gates all subcommands behind enabled: true, returning a clear error with setup instructions
  • Remove the warn-and-fallback block from runCreateCmd
  • Rename IsEnabled() to IsAutoCreateEnabled() on TagManagerPlugin for consistency - every other plugin's IsEnabled() checks only Enabled, but this one checked Enabled && AutoCreate

Related Issue

Fixes #211

Notes for Reviewers

  • None

@indaco indaco merged commit 70348e3 into main Feb 26, 2026
7 checks passed
@indaco indaco deleted the fix/tag-manager-require-enabled branch February 26, 2026 09:01
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.

sley tag create warns about disabled plugin but proceeds anyway with defaults

1 participant