cli: newLogger helper, defaultServerURL, shared http client, report command

- Extract newLogger() to remove duplication across run/serve/start
- Add defaultServerURL const ("http://localhost:8484") used by all client commands
- Move http.Client into internal/cli/http.go with 30s timeout
- Add 'report' command for printing execution summaries
- Add test coverage for create and serve commands

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

1 files changed, 32 insertions, 0 deletions

diff --git a/internal/cli/report_test.go b/internal/cli/report_test.go
new file mode 100644
index 0000000..3ef96f4
--- /dev/null
+++ b/internal/cli/report_test.go
@@ -0,0 +1,32 @@
+package cli
+
+import (
+	"strings"
+	"testing"
+)
+
+func TestReportCmd_InvalidFormat(t *testing.T) {
+	cmd := newReportCmd()
+	cmd.SetArgs([]string{"--format", "xml"})
+	err := cmd.Execute()
+	if err == nil {
+		t.Fatal("expected error for invalid format, got nil")
+	}
+	if !strings.Contains(err.Error(), "format") {
+		t.Errorf("expected error to mention 'format', got: %v", err)
+	}
+}
+
+func TestReportCmd_DefaultsRegistered(t *testing.T) {
+	cmd := newReportCmd()
+	f := cmd.Flags()
+	if f.Lookup("format") == nil {
+		t.Error("missing --format flag")
+	}
+	if f.Lookup("limit") == nil {
+		t.Error("missing --limit flag")
+	}
+	if f.Lookup("task") == nil {
+		t.Error("missing --task flag")
+	}
+}