@pippit-dev/cli 0.0.17 → 0.0.18

package/README.md

@@ -41,6 +41,11 @@ Only `.doc`, `.docx`, and `.txt` file extensions are supported.
 `+download-result` downloads the result URL to
 the `--output-path` file path.
 
+Short drama command errors are appended to a daily local log file under
+`~/.pippit_tool_cli/logs/yyyy-mm-dd.log`. The path is built with the current
+user home directory and the platform path separator, so it works on macOS,
+Linux, and Windows.
+
 ## HTTP Client
 
 Command modules should receive `common.Runner` for service calls. Runtime

package/checksums.txt

@@ -1,6 +1,6 @@
-dc08d1d25fe9fb2d2ed4dc5c2dec42b999ffe9f42f7dfac723d50f85d9107099  pippit-tool-cli-0.0.17-darwin-amd64.tar.gz
-f696c2aec0901638716b5037231fd47916c557b6157c1dcff559c894d7992c5a  pippit-tool-cli-0.0.17-darwin-arm64.tar.gz
-ee4386da9ba02e5dfd93786b319e42af2a41c1fa01c20151232b87ebc81ef343  pippit-tool-cli-0.0.17-linux-amd64.tar.gz
-795c624e273acb9edb0f4851d767d4f58b947e8dc056a7246a64bbb30b085bc1  pippit-tool-cli-0.0.17-linux-arm64.tar.gz
-0e3eb11d38fd04686317df1185195e12180265b9bd7bbb1de2b68a145d72f339  pippit-tool-cli-0.0.17-windows-amd64.zip
-acccf51f98a58f3d887a6e2c9131b8acbbc246b6c518ebe8c26b28ad20ed0a54  pippit-tool-cli-0.0.17-windows-arm64.zip
+2cd3bb5e214d85004196806643c3b7d0a8ceb141ef3df58d1bbfc7aa9642f6ab  pippit-tool-cli-0.0.18-darwin-amd64.tar.gz
+8fb57bef9d7e81b8bcc3421f2179879b656134c23387104e01a1090022f6e7c9  pippit-tool-cli-0.0.18-darwin-arm64.tar.gz
+bdde278cbee36f2b90602b1b58f7be05790a186b3adbc90b8dd8fd970b48b20c  pippit-tool-cli-0.0.18-linux-amd64.tar.gz
+878c6b0cd41a2cafdd62f99be98968eee05f0d16de0c6b173f84bb0328c1bef6  pippit-tool-cli-0.0.18-linux-arm64.tar.gz
+e4423154ba1a52625f255e11060459844a4c5bcf3aab21af682f3e1adcce82cd  pippit-tool-cli-0.0.18-windows-amd64.zip
+cca47eab19bdc7200aabcea2d59a535c63267fe293f993b1013aa049a4be16b4  pippit-tool-cli-0.0.18-windows-arm64.zip

package/cmd/short_drama/short_drama.go

@@ -3,6 +3,8 @@ package short_drama
 import (
 	"fmt"
 	"io"
+	"path/filepath"
+	"strconv"
 	"strings"
 
 	"github.com/Pippit-dev/pippit-cli/internal/common"
@@ -35,7 +37,12 @@ func newShortDramaSubmitRunCommand(stdout, stderr io.Writer, runner *common.Runn
 		Use:   "+submit-run",
 		Short: "Submit a Run task for the short drama scene",
 		Args:  cobra.NoArgs,
-		RunE: func(cmd *cobra.Command, _ []string) error {
+		RunE: withErrorLog("short-drama +submit-run", func() map[string]string {
+			return map[string]string{
+				"thread_id":   opts.ThreadID,
+				"asset_count": strconv.Itoa(len(opts.AssetIDs)),
+			}
+		}, func(cmd *cobra.Command, _ []string) error {
 			opts.Message = strings.TrimSpace(opts.Message)
 			opts.ThreadID = strings.TrimSpace(opts.ThreadID)
 
@@ -48,7 +55,7 @@ func newShortDramaSubmitRunCommand(stdout, stderr io.Writer, runner *common.Runn
 				return err
 			}
 			return writeJSON(stdout, result)
-		},
+		}),
 	}
 	cmd.SetOut(stdout)
 	cmd.SetErr(stderr)
@@ -65,7 +72,11 @@ func newShortDramaUploadFileCommand(stdout, stderr io.Writer, runner *common.Run
 		Use:   "+upload-file",
 		Short: "Upload a file for the short drama scene",
 		Args:  cobra.NoArgs,
-		RunE: func(cmd *cobra.Command, _ []string) error {
+		RunE: withErrorLog("short-drama +upload-file", func() map[string]string {
+			return map[string]string{
+				"file_name": fileNameForLog(opts.Path),
+			}
+		}, func(cmd *cobra.Command, _ []string) error {
 			opts.Path = strings.TrimSpace(opts.Path)
 
 			if opts.Path == "" {
@@ -77,7 +88,7 @@ func newShortDramaUploadFileCommand(stdout, stderr io.Writer, runner *common.Run
 				return err
 			}
 			return writeJSON(stdout, result)
-		},
+		}),
 	}
 	cmd.SetOut(stdout)
 	cmd.SetErr(stderr)
@@ -92,7 +103,13 @@ func newShortDramaDownloadResultCommand(stdout, stderr io.Writer, runner *common
 		Use:   "+download-result",
 		Short: "Download a generated result URL",
 		Args:  cobra.NoArgs,
-		RunE: func(cmd *cobra.Command, _ []string) error {
+		RunE: withErrorLog("short-drama +download-result", func() map[string]string {
+			return map[string]string{
+				"output_path": opts.OutputPath,
+				"has_url":     strconv.FormatBool(strings.TrimSpace(opts.URL) != ""),
+				"workers":     strconv.Itoa(opts.Workers),
+			}
+		}, func(cmd *cobra.Command, _ []string) error {
 			opts.OutputPath = strings.TrimSpace(opts.OutputPath)
 			if opts.OutputPath == "" {
 				return fmt.Errorf("--output-path is required")
@@ -110,7 +127,7 @@ func newShortDramaDownloadResultCommand(stdout, stderr io.Writer, runner *common
 				return err
 			}
 			return writeJSON(stdout, result)
-		},
+		}),
 	}
 	cmd.SetOut(stdout)
 	cmd.SetErr(stderr)
@@ -127,7 +144,12 @@ func newShortDramaGetThreadCommand(stdout, stderr io.Writer, runner *common.Runn
 		Use:   "+get-thread",
 		Short: "Get a short drama thread detail",
 		Args:  cobra.NoArgs,
-		RunE: func(cmd *cobra.Command, _ []string) error {
+		RunE: withErrorLog("short-drama +get-thread", func() map[string]string {
+			return map[string]string{
+				"thread_id": opts.ThreadID,
+				"run_id":    opts.RunID,
+			}
+		}, func(cmd *cobra.Command, _ []string) error {
 			opts.ThreadID = strings.TrimSpace(opts.ThreadID)
 			if opts.ThreadID == "" {
 				return fmt.Errorf("--thread-id is required")
@@ -140,7 +162,7 @@ func newShortDramaGetThreadCommand(stdout, stderr io.Writer, runner *common.Runn
 			}
 			_, err = fmt.Fprintln(stdout, result.ReadableText)
 			return err
-		},
+		}),
 	}
 	cmd.SetOut(stdout)
 	cmd.SetErr(stderr)
@@ -156,7 +178,13 @@ func newShortDramaListThreadFileCommand(stdout, stderr io.Writer, runner *common
 		Use:   "+list-thread-file",
 		Short: "List files in a short drama thread",
 		Args:  cobra.NoArgs,
-		RunE: func(cmd *cobra.Command, _ []string) error {
+		RunE: withErrorLog("short-drama +list-thread-file", func() map[string]string {
+			return map[string]string{
+				"thread_id": opts.ThreadID,
+				"page_num":  strconv.Itoa(opts.PageNum),
+				"page_size": strconv.Itoa(opts.PageSize),
+			}
+		}, func(cmd *cobra.Command, _ []string) error {
 			opts.ThreadID = strings.TrimSpace(opts.ThreadID)
 			if opts.ThreadID == "" {
 				return fmt.Errorf("--thread-id is required")
@@ -173,7 +201,7 @@ func newShortDramaListThreadFileCommand(stdout, stderr io.Writer, runner *common
 				return err
 			}
 			return writeJSON(stdout, result)
-		},
+		}),
 	}
 	cmd.SetOut(stdout)
 	cmd.SetErr(stderr)
@@ -183,6 +211,28 @@ func newShortDramaListThreadFileCommand(stdout, stderr io.Writer, runner *common
 	return cmd
 }
 
+func withErrorLog(command string, fields func() map[string]string, run func(*cobra.Command, []string) error) func(*cobra.Command, []string) error {
+	return func(cmd *cobra.Command, args []string) error {
+		err := run(cmd, args)
+		if err != nil {
+			logFields := map[string]string(nil)
+			if fields != nil {
+				logFields = fields()
+			}
+			_ = common.AppendDailyErrorLog(command, err, logFields)
+		}
+		return err
+	}
+}
+
+func fileNameForLog(path string) string {
+	path = strings.TrimSpace(path)
+	if path == "" {
+		return ""
+	}
+	return filepath.Base(path)
+}
+
 func writeJSON(w io.Writer, v any) error {
 	data, err := sonic.Marshal(v)
 	if err != nil {

package/cmd/short_drama_test.go

@@ -9,6 +9,7 @@ import (
 	"path/filepath"
 	"strings"
 	"testing"
+	"time"
 
 	"github.com/Pippit-dev/pippit-cli/internal/common"
 	"github.com/Pippit-dev/pippit-cli/internal/config"
@@ -17,6 +18,19 @@ import (
 	"github.com/spf13/cobra"
 )
 
+func TestMain(m *testing.M) {
+	home, err := os.MkdirTemp("", "pippit-cli-test-home")
+	if err != nil {
+		panic(err)
+	}
+	_ = os.Setenv("HOME", home)
+	_ = os.Setenv("USERPROFILE", home)
+
+	code := m.Run()
+	_ = os.RemoveAll(home)
+	os.Exit(code)
+}
+
 func TestShortDramaSubmitRun(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		if r.Method != http.MethodPost {
@@ -597,6 +611,8 @@ func TestShortDramaGetThread(t *testing.T) {
 }
 
 func TestShortDramaGetThreadRequiresThreadID(t *testing.T) {
+	clearDailyErrorLog(t)
+
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
 	root.SetArgs([]string{"short-drama", "+get-thread"})
@@ -608,6 +624,17 @@ func TestShortDramaGetThreadRequiresThreadID(t *testing.T) {
 	if !strings.Contains(err.Error(), "--thread-id is required") {
 		t.Fatalf("error = %q, want thread-id validation", err)
 	}
+
+	entries := readDailyErrorLog(t)
+	if len(entries) != 1 {
+		t.Fatalf("log entries = %d, want 1: %#v", len(entries), entries)
+	}
+	if entries[0]["command"] != "short-drama +get-thread" {
+		t.Fatalf("command = %v, want get-thread", entries[0]["command"])
+	}
+	if entries[0]["error"] != "--thread-id is required" {
+		t.Fatalf("error = %v, want thread-id validation", entries[0]["error"])
+	}
 }
 
 func TestShortDramaGetThreadRequiresAccessKey(t *testing.T) {
@@ -787,6 +814,41 @@ func decodeJSON(t *testing.T, data []byte) map[string]any {
 	return got
 }
 
+func clearDailyErrorLog(t *testing.T) {
+	t.Helper()
+	if err := os.RemoveAll(filepath.Join(testHomeDir(t), ".pippit_tool_cli", "logs")); err != nil {
+		t.Fatalf("RemoveAll logs: %v", err)
+	}
+}
+
+func readDailyErrorLog(t *testing.T) []map[string]any {
+	t.Helper()
+	path := filepath.Join(testHomeDir(t), ".pippit_tool_cli", "logs", time.Now().Format("2006-01-02")+".log")
+	data, err := os.ReadFile(path)
+	if err != nil {
+		t.Fatalf("ReadFile(%s): %v", path, err)
+	}
+	lines := strings.Split(strings.TrimSpace(string(data)), "\n")
+	entries := make([]map[string]any, 0, len(lines))
+	for _, line := range lines {
+		var entry map[string]any
+		if err := sonic.Unmarshal([]byte(line), &entry); err != nil {
+			t.Fatalf("decode log line: %v\n%s", err, line)
+		}
+		entries = append(entries, entry)
+	}
+	return entries
+}
+
+func testHomeDir(t *testing.T) string {
+	t.Helper()
+	home, err := os.UserHomeDir()
+	if err != nil {
+		t.Fatalf("UserHomeDir(): %v", err)
+	}
+	return home
+}
+
 func chdirTemp(t *testing.T) string {
 	t.Helper()
 	cwd := t.TempDir()

package/internal/common/error_log.go

@@ -0,0 +1,91 @@
+package common
+
+import (
+	"os"
+	"path/filepath"
+	"strings"
+	"time"
+
+	"github.com/bytedance/sonic"
+)
+
+const (
+	errorLogDirName = ".pippit_tool_cli"
+	errorLogSubdir  = "logs"
+)
+
+type errorLogEntry struct {
+	Time    string            `json:"time"`
+	Command string            `json:"command"`
+	Fields  map[string]string `json:"fields,omitempty"`
+	Error   string            `json:"error"`
+}
+
+// AppendDailyErrorLog appends one CLI error record to ~/.pippit_tool_cli/logs/yyyy-mm-dd.log.
+func AppendDailyErrorLog(command string, err error, fields map[string]string) error {
+	if err == nil {
+		return nil
+	}
+	now := time.Now()
+	path, pathErr := dailyErrorLogPath(now)
+	if pathErr != nil {
+		return pathErr
+	}
+	if mkErr := os.MkdirAll(filepath.Dir(path), 0o700); mkErr != nil {
+		return mkErr
+	}
+
+	entry := errorLogEntry{
+		Time:    now.Format(time.RFC3339),
+		Command: strings.TrimSpace(command),
+		Fields:  cleanErrorLogFields(fields),
+		Error:   err.Error(),
+	}
+	data, marshalErr := sonic.Marshal(entry)
+	if marshalErr != nil {
+		return marshalErr
+	}
+	data = append(data, '\n')
+	file, openErr := os.OpenFile(path, os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0o600)
+	if openErr != nil {
+		return openErr
+	}
+	defer file.Close()
+	_, writeErr := file.Write(data)
+	return writeErr
+}
+
+func dailyErrorLogPath(now time.Time) (string, error) {
+	home, err := os.UserHomeDir()
+	if err != nil {
+		return "", err
+	}
+	return filepath.Join(home, errorLogDirName, errorLogSubdir, now.Format("2006-01-02")+".log"), nil
+}
+
+func cleanErrorLogFields(fields map[string]string) map[string]string {
+	if len(fields) == 0 {
+		return nil
+	}
+	cleaned := make(map[string]string, len(fields))
+	for k, v := range fields {
+		key := strings.TrimSpace(k)
+		value := strings.TrimSpace(v)
+		if key == "" || value == "" || isSensitiveLogField(key) {
+			continue
+		}
+		cleaned[key] = value
+	}
+	if len(cleaned) == 0 {
+		return nil
+	}
+	return cleaned
+}
+
+func isSensitiveLogField(key string) bool {
+	key = strings.ToLower(key)
+	return strings.Contains(key, "access") ||
+		strings.Contains(key, "authorization") ||
+		strings.Contains(key, "secret") ||
+		strings.Contains(key, "token")
+}

package/internal/common/error_log_test.go

@@ -0,0 +1,61 @@
+package common
+
+import (
+	"fmt"
+	"os"
+	"path/filepath"
+	"strings"
+	"testing"
+	"time"
+
+	"github.com/bytedance/sonic"
+)
+
+func TestAppendDailyErrorLog(t *testing.T) {
+	home := t.TempDir()
+	t.Setenv("HOME", home)
+	t.Setenv("USERPROFILE", home)
+
+	err := AppendDailyErrorLog("short-drama +get-thread", fmt.Errorf("request failed"), map[string]string{
+		"thread_id":  "thread_123",
+		"access_key": "secret",
+		"empty":      "",
+	})
+	if err != nil {
+		t.Fatalf("AppendDailyErrorLog(): %v", err)
+	}
+	if err := AppendDailyErrorLog("short-drama +get-thread", fmt.Errorf("second failure"), nil); err != nil {
+		t.Fatalf("AppendDailyErrorLog() second call: %v", err)
+	}
+
+	path := filepath.Join(home, ".pippit_tool_cli", "logs", time.Now().Format("2006-01-02")+".log")
+	data, err := os.ReadFile(path)
+	if err != nil {
+		t.Fatalf("ReadFile(%s): %v", path, err)
+	}
+	lines := strings.Split(strings.TrimSpace(string(data)), "\n")
+	if len(lines) != 2 {
+		t.Fatalf("log lines = %d, want 2\n%s", len(lines), string(data))
+	}
+
+	var first map[string]any
+	if err := sonic.Unmarshal([]byte(lines[0]), &first); err != nil {
+		t.Fatalf("decode first log line: %v\n%s", err, lines[0])
+	}
+	if first["command"] != "short-drama +get-thread" {
+		t.Fatalf("command = %v, want get-thread command", first["command"])
+	}
+	if first["error"] != "request failed" {
+		t.Fatalf("error = %v, want request failed", first["error"])
+	}
+	fields, ok := first["fields"].(map[string]any)
+	if !ok {
+		t.Fatalf("fields = %#v, want object", first["fields"])
+	}
+	if fields["thread_id"] != "thread_123" {
+		t.Fatalf("thread_id = %v, want thread_123", fields["thread_id"])
+	}
+	if _, ok := fields["access_key"]; ok {
+		t.Fatalf("access_key should be omitted from log fields: %#v", fields)
+	}
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pippit-dev/cli",
-  "version": "0.0.17",
+  "version": "0.0.18",
   "description": "Pippit CLI",
   "bin": {
     "pippit-tool-cli": "scripts/run.js"