From 875258d6c594f36c38508a33d3916cab3fbf8d4e Mon Sep 17 00:00:00 2001 From: Amolith Date: Thu, 30 Oct 2025 12:45:29 -0600 Subject: [PATCH] refactor(cmd/t/u): simplify completion messages Remove redundant else branches and duplicate instructions when all tasks are complete. The completion message now only advises escalating to operator without repeating usage patterns already shown elsewhere. Co-authored-by: Crush --- cmd/t/u.go | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/cmd/t/u.go b/cmd/t/u.go index 73f86b34b968f27f7a3e41b6b2f86ff1f2d2d9bf..1aed8c886fabbe7a36f240189964e43ce711e2bb 100644 --- a/cmd/t/u.go +++ b/cmd/t/u.go @@ -216,9 +216,7 @@ func runBatchStatusUpdate(cmd *cobra.Command, env *cli.Environment, sid string, } if allComplete { - _, _ = fmt.Fprintln(out, "No pending tasks remaining. If you've completed the goal, escalate to the operator and obtain confirmation before archiving the session with `np a`.") - } else { - _, _ = fmt.Fprintln(out, "Continue working through remaining tasks. Use `np t u -i -s -i -s ` to update multiple tasks at once.") + _, _ = fmt.Fprintln(out, "No pending tasks remaining. If you've completed the goal, escalate to the operator and ask whether to archive the session (`np a`).") } return nil @@ -343,9 +341,7 @@ func runSingleTaskUpdate(cmd *cobra.Command, env *cli.Environment, sid string, t } if len(pending) == 0 && len(inProgress) == 0 { - _, _ = fmt.Fprintln(out, "No pending tasks remaining. If you've completed the goal, escalate to the operator and obtain confirmation before archiving the session with `np a`.") - } else { - _, _ = fmt.Fprintln(out, "Task marked completed. Continue working through remaining tasks. Use `np t u -i -s -i -s ` to update multiple at once.") + _, _ = fmt.Fprintln(out, "No pending tasks remaining. If you've completed the goal, escalate to the operator and ask whether to archive the session (`np a`).") } } else { _, _ = fmt.Fprintln(out, "Task updated. Use `np p` to review the full plan.")