fix(mcp): treat "default" budgetId as fallback to configured default (#36)

stephendolan · claude · web-flow · commit 25b7c8adc309 · 2026-03-10T16:43:24.000-04:00
MCP clients pass budgetId: "default" as a literal string, which the
YNAB API rejects with a 404. Now treated as a signal to use the
configured default budget.

Co-authored-by: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@stephendolan/ynab-cli",
-  "version": "2.8.1",
+  "version": "2.8.2",
   "description": "A command-line interface for You Need a Budget (YNAB)",
   "type": "module",
   "main": "./dist/cli.js",
diff --git a/src/lib/api-client.ts b/src/lib/api-client.ts
@@ -43,7 +43,7 @@ export class YnabClient {
   }
 
   async getBudgetId(budgetIdOrDefault?: string): Promise<string> {
-    const budgetId = budgetIdOrDefault || config.getDefaultBudget() || process.env.YNAB_BUDGET_ID;
+    const budgetId = (budgetIdOrDefault && budgetIdOrDefault !== 'default' ? budgetIdOrDefault : undefined) || config.getDefaultBudget() || process.env.YNAB_BUDGET_ID;
 
     if (!budgetId) {
       throw new YnabCliError(

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@stephendolan/ynab-cli",`
`3`		`- "version": "2.8.1",`
	`3`	`+ "version": "2.8.2",`
`4`	`4`	`"description": "A command-line interface for You Need a Budget (YNAB)",`
`5`	`5`	`"type": "module",`
`6`	`6`	`"main": "./dist/cli.js",`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ export class YnabClient {`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`async getBudgetId(budgetIdOrDefault?: string): Promise<string> {`
`46`		`- const budgetId = budgetIdOrDefault \|\| config.getDefaultBudget() \|\| process.env.YNAB_BUDGET_ID;`
	`46`	`+ const budgetId = (budgetIdOrDefault && budgetIdOrDefault !== 'default' ? budgetIdOrDefault : undefined) \|\| config.getDefaultBudget() \|\| process.env.YNAB_BUDGET_ID;`
`47`	`47`
`48`	`48`	`if (!budgetId) {`
`49`	`49`	`throw new YnabCliError(`