fix(branch): -y flag was shadowed by same-named global option (+ bump 0.1.69)

[jwfing]

Summary

• npx @insforge/cli branch merge <name> -y (and branch reset / branch delete) still prompted for confirmation, blocking agent-driven automation. Commander v13 assigns the value to the parent program's option when both parent and subcommand declare the same flag, leaving the subcommand's opts.yes undefined. -y, --yes is already a global option on the root program, so the redundant local declarations on these subcommands were what triggered the shadowing.
• Fix: drop the local -y, --yes declarations and read yes from getRootOpts(cmd), which already walks to the root program.
• Bumps version to 0.1.69.

Test plan

• npx vitest run — 188 passed, 13 skipped
• New regression test in merge.test.ts nests under a branch parent (matching the real CLI wiring) and asserts clack.confirm is not called when -y is passed without --json. The prior --yes test paired with --json, where --json alone bypassed the prompt — so it never actually exercised -y.
• Manual verification after release: npx @insforge/cli@0.1.69 branch merge <name> -y exits without prompting.

🤖 Generated with Claude Code

---

Summary by cubic

Fixes -y/--yes being ignored on branch merge, branch reset, and branch delete, so these commands no longer prompt when -y is passed. Bumps @insforge/cli to 0.1.69.

• Bug Fixes
  • Removed local -y, --yes from branch subcommands and read yes from root via getRootOpts(cmd).
  • Added a regression test (nested under branch) to ensure no confirmation prompt when -y is used without --json.

^{Written for commit 053a308. Summary will update on new commits.}

Summary by CodeRabbit

• Chores

  • Bumped package version to 0.1.69

• Tests

  • Added regression test for the --yes flag on branch merge subcommand

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d1151d66-1eaf-41e6-91db-b982f70f980b

📥 Commits

Reviewing files that changed from the base of the PR and between e94452a and 053a308.

📒 Files selected for processing (5)

• package.json
• src/commands/branch/delete.ts
• src/commands/branch/merge.test.ts
• src/commands/branch/merge.ts
• src/commands/branch/reset.ts

---

Walkthrough

CLI package version incremented to 0.1.69. Branch delete, merge, and reset commands refactored to source the yes confirmation-bypass flag from root CLI options via getRootOpts(cmd) instead of command-specific opts parameters. Regression test added for merge command verifies --yes flag skips confirmation prompt and executes merge.

Changes

Branch Command Yes-Flag Consolidation

Layer / File(s)	Summary
Package Version package.json	Package version bumped from 0.1.68 to 0.1.69.
Option Sourcing Refactor src/commands/branch/delete.ts, src/commands/branch/merge.ts, src/commands/branch/reset.ts	Delete, merge, and reset commands updated to destructure yes from getRootOpts(cmd) instead of reading from command-specific opts; confirmation gates updated to reference root-sourced yes flag across all three commands.
Test Coverage src/commands/branch/merge.test.ts	Added hoisted @clack/prompts mock with reset in beforeEach; new regression test verifies branch merge -y skips confirmation prompt and invokes merge execute API.

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 With whiskers twitching and nose held high,
We consolidate the yes flag up on high,
From commands scattered to root so neat,
One source of truth, the refactor's sweet! ✨
Tests ensure the prompt takes flight,
When -y is passed, all ends right!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main problem being fixed: a shadowing issue with the -y flag in branch subcommands, and includes the version bump.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/branching

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cubic-dev-ai]

No issues found across 5 files