refactor: Make force mode DEFAULT ON with --no-force flag to disable
BREAKING CHANGE: Force mode is now ON by default (was OFF by default) User requested: "make this default on with skip flag only" Changes: -------- - Force mode is now ON by default (skip all confirmations) - New flag: `--no-force` to disable force mode (enable confirmations) - Old flag: `--force` removed (force is always ON now) Rationale: ---------- - Maximizes automation out-of-the-box - Better UX for CI/CD and batch processing (no extra flags needed) - Aligns with "dangerously skip mode" user request - Explicit opt-out is better than hidden opt-in for automation tools Migration: ---------- - Before: `skill-seekers enhance output/react/ --force` - After: `skill-seekers enhance output/react/` (force ON by default!) - To disable: `skill-seekers enhance output/react/ --no-force` Behavior: --------- - Default: `LocalSkillEnhancer(skill_dir, force=True)` - With --no-force: `LocalSkillEnhancer(skill_dir, force=False)` CLI Examples: ------------- # Force ON (default - no flag needed) skill-seekers enhance output/react/ # Force OFF (enable confirmations) skill-seekers enhance output/react/ --no-force # Background with force (force already ON by default) skill-seekers enhance output/react/ --background # Background without force (need --no-force) skill-seekers enhance output/react/ --background --no-force Files Changed: -------------- - src/skill_seekers/cli/enhance_skill_local.py - Changed default: force=False → force=True - Changed flag: --force → --no-force - Updated docstring - Updated help text - src/skill_seekers/cli/main.py - Changed flag: --force → --no-force - Updated argument forwarding - docs/ENHANCEMENT_MODES.md - Updated Force Mode section (default ON) - Updated examples (removed unnecessary --force flags) - Updated batch enhancement example - Updated CI/CD example - CHANGELOG.md - Updated "Force Mode" description (Default ON) - Clarified no flag needed Impact: ------- - ✅ CI/CD pipelines: No extra flags needed (force ON by default) - ✅ Batch processing: Cleaner commands - ✅ Manual users: Use --no-force if they want confirmations - ✅ Backward compatible: Old behavior available via --no-force 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -137,7 +137,7 @@ For more information: https://github.com/yusufkaraaslan/Skill_Seekers
|
||||
enhance_parser.add_argument("skill_directory", help="Skill directory path")
|
||||
enhance_parser.add_argument("--background", action="store_true", help="Run in background")
|
||||
enhance_parser.add_argument("--daemon", action="store_true", help="Run as daemon")
|
||||
enhance_parser.add_argument("--force", "-f", action="store_true", help="Force mode (skip confirmations)")
|
||||
enhance_parser.add_argument("--no-force", action="store_true", help="Disable force mode (enable confirmations)")
|
||||
enhance_parser.add_argument("--timeout", type=int, default=600, help="Timeout in seconds")
|
||||
|
||||
# === enhance-status subcommand ===
|
||||
@@ -376,8 +376,8 @@ def main(argv: Optional[List[str]] = None) -> int:
|
||||
sys.argv.append("--background")
|
||||
if args.daemon:
|
||||
sys.argv.append("--daemon")
|
||||
if args.force:
|
||||
sys.argv.append("--force")
|
||||
if args.no_force:
|
||||
sys.argv.append("--no-force")
|
||||
if args.timeout:
|
||||
sys.argv.extend(["--timeout", str(args.timeout)])
|
||||
return enhance_main() or 0
|
||||
|
||||
Reference in New Issue
Block a user