refactor(cmd/t/u): simplify completion messages

Amolith and Crush created 

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 <crush@charm.land>

Change summary

cmd/t/u.go | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)

Detailed changes

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 <id1> -s <status1> -i <id2> -s <status2>` 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 <id1> -s <status1> -i <id2> -s <status2>` 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.")