From 9ba06e4a42bfe187f1264407b02bb5fe4bdfafb0 Mon Sep 17 00:00:00 2001 From: Claude Code Date: Tue, 1 Sep 2026 10:13:23 -0600 Subject: [PATCH] fix(db): check the task error before the ARN print `apppack db load` starts an ECS task with `StartTask`. If the call fails, it returns a nil task and an error. The command printed `*task.TaskArn` before the error check, so the CLI panicked and hid the real cause. Move `checkErr(err)` to the line after the `StartTask` call. This matches the order in `cmd/shell.go`. Co-Authored-By: Claude Opus 5 (1M context) --- cmd/db.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/db.go b/cmd/db.go index cec6e49..4c6cac8 100644 --- a/cmd/db.go +++ b/cmd/db.go @@ -251,11 +251,11 @@ WARNING: This is a destructive action which will delete the contents of your rem taskOverride, true, ) + checkErr(err) ui.Spinner.Stop() fmt.Println(aurora.Faint("starting task " + *task.TaskArn)) ui.StartSpinner() ui.Spinner.Suffix = " loading database" - checkErr(err) exitCode, err := app.WaitForTaskStopped(task) checkErr(err) ui.Spinner.Stop()