fix(budget): flatten limit commands to 2-token leaves (limit-list/limit-set)

The CLI dispatcher (cli.py _build_parser) and completion generator both
partition the command name on the first space, nesting only two levels
(group + leaf). A three-token 'budget limit set' registers but is
unreachable via the CLI (invalid choice: 'limit'); the mocked unit suite
missed this because it calls handlers directly, bypassing dispatch.

Flatten to 'budget limit-list' / 'budget limit-set' (hyphenated 2-token
leaf), matching every other command. Logged the n-level dispatcher
refactor in TODO.md for when a future group needs real sub-subcommands.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/firefly_cli/commands/budget.py b/firefly_cli/commands/budget.py
index 733d092..f9999a5 100644
--- a/firefly_cli/commands/budget.py
+++ b/firefly_cli/commands/budget.py
@@ -107,7 +107,7 @@ def cmd_disable(args, ctx):
     return _set_active(ctx, args.ref, False)
 
 
-@registry.command("budget limit list", help="list a budget's limits (name or id)", args=_ref_arg)
+@registry.command("budget limit-list", help="list a budget's limits (name or id)", args=_ref_arg)
 def cmd_limit_list(args, ctx):
     b = ctx.resolver.budget(args.ref)
     resp = ctx.client.request("GET", f"/api/v1/budgets/{b['id']}/limits")
@@ -121,7 +121,7 @@ def _limit_set_args(p):
     p.add_argument("--end", default=None, help="YYYY-MM-DD (default: last of this month)")
     p.add_argument("--currency", default=None, help="currency code, e.g. EUR")
 
-@registry.command("budget limit set", help="set (create) a spending limit for a budget over a period", args=_limit_set_args)
+@registry.command("budget limit-set", help="set (create) a spending limit for a budget over a period", args=_limit_set_args)
 def cmd_limit_set(args, ctx):
     b = ctx.resolver.budget(args.ref)
     first, last = _current_month()