@pippit-dev/cli 1.0.0 → 1.0.2

package/cmd/short_drama/short_drama.go

@@ -9,7 +9,6 @@ import (
 
 	"github.com/Pippit-dev/pippit-cli/internal/common"
 	"github.com/Pippit-dev/pippit-cli/internal/short_drama"
-	"github.com/bytedance/sonic"
 	"github.com/spf13/cobra"
 )
 
@@ -24,9 +23,6 @@ func NewCommand(stdout, stderr io.Writer, runner *common.Runner) *cobra.Command
 	cmd.SetErr(stderr)
 	cmd.AddCommand(newShortDramaSubmitRunCommand(stdout, stderr, runner))
 	cmd.AddCommand(newShortDramaUploadFileCommand(stdout, stderr, runner))
-	cmd.AddCommand(newShortDramaDownloadResultCommand(stdout, stderr, runner))
-	cmd.AddCommand(newShortDramaGetThreadCommand(stdout, stderr, runner))
-	cmd.AddCommand(newShortDramaListThreadFileCommand(stdout, stderr, runner))
 	return cmd
 }
 
@@ -47,14 +43,14 @@ func newShortDramaSubmitRunCommand(stdout, stderr io.Writer, runner *common.Runn
 			opts.ThreadID = strings.TrimSpace(opts.ThreadID)
 
 			if opts.Message == "" {
-				return fmt.Errorf("--message is required")
+				return fmt.Errorf("缺少必填参数 --message")
 			}
 
 			result, err := short_drama.SubmitRun(cmd.Context(), &opts, runner)
 			if err != nil {
 				return err
 			}
-			return writeJSON(stdout, result)
+			return common.WriteJSON(stdout, result)
 		}),
 	}
 	cmd.SetOut(stdout)
@@ -82,12 +78,15 @@ func newShortDramaUploadFileCommand(stdout, stderr io.Writer, runner *common.Run
 			if opts.Path == "" {
 				return fmt.Errorf("--path is required")
 			}
+			if !isShortDramaUploadFile(opts.Path) {
+				return fmt.Errorf("短剧上传仅支持上传 .doc、.docx 和 .txt 文件")
+			}
 
 			result, err := common.UploadFile(cmd.Context(), opts, runner)
 			if err != nil {
 				return err
 			}
-			return writeJSON(stdout, result)
+			return common.WriteJSON(stdout, result)
 		}),
 	}
 	cmd.SetOut(stdout)
@@ -96,126 +95,6 @@ func newShortDramaUploadFileCommand(stdout, stderr io.Writer, runner *common.Run
 	return cmd
 }
 
-func newShortDramaDownloadResultCommand(stdout, stderr io.Writer, runner *common.Runner) *cobra.Command {
-	var opts common.DownloadResultOptions
-
-	cmd := &cobra.Command{
-		Use:   "+download-result",
-		Short: "Download a generated result URL",
-		Args:  cobra.NoArgs,
-		RunE: withErrorLog("short-drama +download-result", func() map[string]string {
-			fields := map[string]string{
-				"output_path": opts.OutputPath,
-				"has_url":     strconv.FormatBool(strings.TrimSpace(opts.URL) != ""),
-				"workers":     strconv.Itoa(opts.Workers),
-			}
-			if opts.UpdatedAt > 0 {
-				fields["updated_at"] = strconv.FormatInt(opts.UpdatedAt, 10)
-			}
-			return fields
-		}, func(cmd *cobra.Command, _ []string) error {
-			opts.OutputPath = strings.TrimSpace(opts.OutputPath)
-			if opts.OutputPath == "" {
-				return fmt.Errorf("--output-path is required")
-			}
-			opts.URL = strings.TrimSpace(opts.URL)
-			if opts.URL == "" {
-				return fmt.Errorf("--url is required")
-			}
-			if opts.Workers <= 0 {
-				return fmt.Errorf("--workers must be greater than 0")
-			}
-
-			result, err := common.DownloadResult(cmd.Context(), opts, runner)
-			if err != nil {
-				return err
-			}
-			return writeJSON(stdout, result)
-		}),
-	}
-	cmd.SetOut(stdout)
-	cmd.SetErr(stderr)
-	cmd.Flags().StringVar(&opts.URL, "url", "", "URL to download")
-	cmd.Flags().StringVar(&opts.OutputPath, "output-path", "", "local output file path")
-	cmd.Flags().Int64Var(&opts.UpdatedAt, "updated-at", 0, "remote file update time as a Unix timestamp")
-	cmd.Flags().IntVar(&opts.Workers, "workers", 5, "parallel download workers")
-	return cmd
-}
-
-func newShortDramaGetThreadCommand(stdout, stderr io.Writer, runner *common.Runner) *cobra.Command {
-	var opts common.GetThreadOptions
-
-	cmd := &cobra.Command{
-		Use:   "+get-thread",
-		Short: "Get a short drama thread detail",
-		Args:  cobra.NoArgs,
-		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")
-			}
-			opts.RunID = strings.TrimSpace(opts.RunID)
-
-			result, err := common.GetThread(cmd.Context(), &opts, runner)
-			if err != nil {
-				return err
-			}
-			_, err = fmt.Fprintln(stdout, result.ReadableText)
-			return err
-		}),
-	}
-	cmd.SetOut(stdout)
-	cmd.SetErr(stderr)
-	cmd.Flags().StringVar(&opts.ThreadID, "thread-id", "", "thread ID to fetch")
-	cmd.Flags().StringVar(&opts.RunID, "run-id", "", "run ID to fetch")
-	return cmd
-}
-
-func newShortDramaListThreadFileCommand(stdout, stderr io.Writer, runner *common.Runner) *cobra.Command {
-	var opts common.ListThreadFileOptions
-
-	cmd := &cobra.Command{
-		Use:   "+list-thread-file",
-		Short: "List files in a short drama thread",
-		Args:  cobra.NoArgs,
-		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")
-			}
-			if opts.PageSize <= 0 || opts.PageSize > common.MaxListThreadFilePageSize {
-				return fmt.Errorf("--page-size must be between 1 and %d", common.MaxListThreadFilePageSize)
-			}
-			if opts.PageNum <= 0 {
-				return fmt.Errorf("--page-num must be greater than 0")
-			}
-
-			result, err := common.ListThreadFile(cmd.Context(), &opts, runner)
-			if err != nil {
-				return err
-			}
-			return writeJSON(stdout, result)
-		}),
-	}
-	cmd.SetOut(stdout)
-	cmd.SetErr(stderr)
-	cmd.Flags().StringVar(&opts.ThreadID, "thread-id", "", "thread ID to list files for")
-	cmd.Flags().IntVar(&opts.PageNum, "page-num", 1, "page number (1-based)")
-	cmd.Flags().IntVar(&opts.PageSize, "page-size", common.MaxListThreadFilePageSize, fmt.Sprintf("number of files per page (between 1 and %d)", common.MaxListThreadFilePageSize))
-	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)
@@ -238,11 +117,11 @@ func fileNameForLog(path string) string {
 	return filepath.Base(path)
 }
 
-func writeJSON(w io.Writer, v any) error {
-	data, err := sonic.Marshal(v)
-	if err != nil {
-		return err
+func isShortDramaUploadFile(path string) bool {
+	switch strings.ToLower(filepath.Ext(path)) {
+	case ".doc", ".docx", ".txt":
+		return true
+	default:
+		return false
 	}
-	_, err = fmt.Fprintln(w, string(data))
-	return err
 }

package/cmd/short_drama_test.go

@@ -124,7 +124,12 @@ func TestRootHelpListsSupportedCommands(t *testing.T) {
 	}
 	got := stdout.String()
 	for _, want := range []string{
-		"Pippit CLI submits short-drama workflows",
+		"Pippit CLI generates videos",
+		"generate-video",
+		"download-result",
+		"get-thread",
+		"list-thread-file",
+		"query-result",
 		"short-drama",
 		"update",
 		"--version",
@@ -140,6 +145,28 @@ func TestRootHelpListsSupportedCommands(t *testing.T) {
 	}
 }
 
+func TestShortDramaDoesNotIncludeCommonThreadCommands(t *testing.T) {
+	var stdout, stderr bytes.Buffer
+	root := NewRootCommand(&stdout, &stderr)
+	cmd, _, err := root.Find([]string{"short-drama"})
+	if err != nil {
+		t.Fatalf("Find(short-drama) error = %v", err)
+	}
+	if cmd.CommandPath() != "pippit-tool-cli short-drama" {
+		t.Fatalf("CommandPath() = %q, want short-drama command", cmd.CommandPath())
+	}
+	removedCommands := map[string]bool{
+		"+download-result":  true,
+		"+get-thread":       true,
+		"+list-thread-file": true,
+	}
+	for _, child := range cmd.Commands() {
+		if removedCommands[child.Name()] {
+			t.Fatalf("short-drama child %s = %#v, want nil", child.Name(), child)
+		}
+	}
+}
+
 func TestVersionFlagPrintsVersion(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
@@ -162,7 +189,7 @@ func TestShortDramaSubmitRunRequiresMessage(t *testing.T) {
 	if err == nil {
 		t.Fatal("Execute() error = nil, want validation error")
 	}
-	if !strings.Contains(err.Error(), "--message is required") {
+	if !strings.Contains(err.Error(), "缺少必填参数 --message") {
 		t.Fatalf("error = %q, want message validation", err)
 	}
 }
@@ -302,12 +329,12 @@ func TestShortDramaUploadFileRejectsUnsupportedFileType(t *testing.T) {
 	if err == nil {
 		t.Fatal("Execute() error = nil, want file type validation error")
 	}
-	if !strings.Contains(err.Error(), "only .doc, .docx, and .txt uploads are supported") {
+	if !strings.Contains(err.Error(), "仅支持上传 .doc、.docx 和 .txt") {
 		t.Fatalf("error = %q, want unsupported type validation", err)
 	}
 }
 
-func TestShortDramaDownloadResult(t *testing.T) {
+func TestDownloadResult(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		if r.Header.Get("User-Agent") != "Pippit-CLI/1.0" {
 			t.Fatalf("User-Agent = %q, want Pippit-CLI/1.0", r.Header.Get("User-Agent"))
@@ -326,7 +353,7 @@ func TestShortDramaDownloadResult(t *testing.T) {
 	root := NewRootCommand(&stdout, &stderr)
 	outputPath := filepath.Join("results", "cover.jpeg")
 	root.SetArgs([]string{
-		"short-drama", "+download-result",
+		"download-result",
 		"--output-path", outputPath,
 		"--workers", "2",
 		"--url", server.URL + "/image?filename=ignored.jpeg",
@@ -358,7 +385,7 @@ func TestShortDramaDownloadResult(t *testing.T) {
 	assertFileContent(t, wantFiles[0], "image-data")
 }
 
-func TestShortDramaDownloadResultSkipsExistingFile(t *testing.T) {
+func TestDownloadResultSkipsExistingFile(t *testing.T) {
 	serverCalled := false
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		serverCalled = true
@@ -378,7 +405,7 @@ func TestShortDramaDownloadResultSkipsExistingFile(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
 	root.SetArgs([]string{
-		"short-drama", "+download-result",
+		"download-result",
 		"--output-path", outputPath,
 		"--url", server.URL + "/image",
 	})
@@ -404,7 +431,7 @@ func TestShortDramaDownloadResultSkipsExistingFile(t *testing.T) {
 	assertFileContent(t, outputPath, "existing-data")
 }
 
-func TestShortDramaDownloadResultSkipsExistingFileWhenLocalIsFresh(t *testing.T) {
+func TestDownloadResultSkipsExistingFileWhenLocalIsFresh(t *testing.T) {
 	serverCalled := false
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		serverCalled = true
@@ -429,7 +456,7 @@ func TestShortDramaDownloadResultSkipsExistingFileWhenLocalIsFresh(t *testing.T)
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
 	root.SetArgs([]string{
-		"short-drama", "+download-result",
+		"download-result",
 		"--output-path", outputPath,
 		"--updated-at", "1779716734",
 		"--url", server.URL + "/image",
@@ -450,7 +477,7 @@ func TestShortDramaDownloadResultSkipsExistingFileWhenLocalIsFresh(t *testing.T)
 	assertFileContent(t, outputPath, "existing-data")
 }
 
-func TestShortDramaDownloadResultOverwritesStaleExistingFile(t *testing.T) {
+func TestDownloadResultOverwritesStaleExistingFile(t *testing.T) {
 	serverCalled := false
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		serverCalled = true
@@ -478,7 +505,7 @@ func TestShortDramaDownloadResultOverwritesStaleExistingFile(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
 	root.SetArgs([]string{
-		"short-drama", "+download-result",
+		"download-result",
 		"--output-path", outputPath,
 		"--updated-at", "1779716734",
 		"--url", server.URL + "/image",
@@ -509,7 +536,7 @@ func TestShortDramaDownloadResultOverwritesStaleExistingFile(t *testing.T) {
 	}
 }
 
-func TestShortDramaDownloadResultDownloadsMetaJSON(t *testing.T) {
+func TestDownloadResultDownloadsMetaJSON(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		if r.URL.Path != "/meta.json" {
 			t.Fatalf("path = %s, want meta.json", r.URL.Path)
@@ -524,7 +551,7 @@ func TestShortDramaDownloadResultDownloadsMetaJSON(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
 	root.SetArgs([]string{
-		"short-drama", "+download-result",
+		"download-result",
 		"--output-path", outputPath,
 		"--url", server.URL + "/meta.json",
 	})
@@ -547,10 +574,10 @@ func TestShortDramaDownloadResultDownloadsMetaJSON(t *testing.T) {
 	assertFileContent(t, outputPath, `{"ok":true}`)
 }
 
-func TestShortDramaDownloadResultRequiresOutputPath(t *testing.T) {
+func TestDownloadResultRequiresOutputPath(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
-	root.SetArgs([]string{"short-drama", "+download-result", "--url", "https://example.com/image.png"})
+	root.SetArgs([]string{"download-result", "--url", "https://example.com/image.png"})
 
 	err := root.Execute()
 	if err == nil {
@@ -561,11 +588,11 @@ func TestShortDramaDownloadResultRequiresOutputPath(t *testing.T) {
 	}
 }
 
-func TestShortDramaDownloadResultRejectsOutputDirFlag(t *testing.T) {
+func TestDownloadResultRejectsOutputDirFlag(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
 	root.SetArgs([]string{
-		"short-drama", "+download-result",
+		"download-result",
 		"--output-dir", filepath.Join("results", "image.png"),
 		"--url", "https://example.com/image.png",
 	})
@@ -574,15 +601,15 @@ func TestShortDramaDownloadResultRejectsOutputDirFlag(t *testing.T) {
 	if err == nil {
 		t.Fatal("Execute() error = nil, want unknown flag error")
 	}
-	if !strings.Contains(err.Error(), "unknown flag: --output-dir") {
+	if !strings.Contains(err.Error(), "未知参数: --output-dir") {
 		t.Fatalf("error = %q, want output-dir rejection", err)
 	}
 }
 
-func TestShortDramaDownloadResultRequiresURL(t *testing.T) {
+func TestDownloadResultRequiresURL(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
-	root.SetArgs([]string{"short-drama", "+download-result", "--output-path", filepath.Join("results", "image.png")})
+	root.SetArgs([]string{"download-result", "--output-path", filepath.Join("results", "image.png")})
 
 	err := root.Execute()
 	if err == nil {
@@ -593,21 +620,21 @@ func TestShortDramaDownloadResultRequiresURL(t *testing.T) {
 	}
 }
 
-func TestShortDramaDownloadResultRejectsInvalidScheme(t *testing.T) {
+func TestDownloadResultRejectsInvalidScheme(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
-	root.SetArgs([]string{"short-drama", "+download-result", "--output-path", filepath.Join("results", "image.png"), "--url", "file:///etc/passwd"})
+	root.SetArgs([]string{"download-result", "--output-path", filepath.Join("results", "image.png"), "--url", "file:///etc/passwd"})
 
 	err := root.Execute()
 	if err == nil {
 		t.Fatal("Execute() error = nil, want scheme validation error")
 	}
-	if !strings.Contains(err.Error(), "only http and https are allowed") {
+	if !strings.Contains(err.Error(), "仅支持 http 和 https") {
 		t.Fatalf("error = %q, want scheme validation", err)
 	}
 }
 
-func TestShortDramaDownloadResultAllFailed(t *testing.T) {
+func TestDownloadResultAllFailed(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		w.WriteHeader(http.StatusNotFound)
 	}))
@@ -618,7 +645,7 @@ func TestShortDramaDownloadResultAllFailed(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
 	root.SetArgs([]string{
-		"short-drama", "+download-result",
+		"download-result",
 		"--output-path", filepath.Join("results", "missing.png"),
 		"--url", server.URL + "/notfound",
 	})
@@ -627,7 +654,7 @@ func TestShortDramaDownloadResultAllFailed(t *testing.T) {
 	if err == nil {
 		t.Fatal("Execute() error = nil, want all-failed error")
 	}
-	if !strings.Contains(err.Error(), "all 1 download(s) failed") {
+	if !strings.Contains(err.Error(), "全部 1 个下载任务失败") {
 		t.Fatalf("error = %q, want all-failed message", err)
 	}
 	logs := readDailyErrorLog(t)
@@ -643,7 +670,7 @@ func TestShortDramaDownloadResultAllFailed(t *testing.T) {
 	}
 }
 
-func TestShortDramaDownloadResultOutputPath(t *testing.T) {
+func TestDownloadResultOutputPath(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		_, _ = w.Write([]byte("image-data"))
 	}))
@@ -655,7 +682,7 @@ func TestShortDramaDownloadResultOutputPath(t *testing.T) {
 	root := NewRootCommand(&stdout, &stderr)
 	outputPath := filepath.Join("custom", "nested", "cover.png")
 	root.SetArgs([]string{
-		"short-drama", "+download-result",
+		"download-result",
 		"--output-path", outputPath,
 		"--url", server.URL + "/image.png",
 	})
@@ -671,7 +698,7 @@ func TestShortDramaDownloadResultOutputPath(t *testing.T) {
 	assertFileContent(t, outputPath, "image-data")
 }
 
-func TestShortDramaGetThread(t *testing.T) {
+func TestGetThread(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		if r.Method != http.MethodPost {
 			t.Fatalf("method = %s, want POST", r.Method)
@@ -709,7 +736,7 @@ func TestShortDramaGetThread(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := newTestRootCommand(t, &stdout, &stderr, server.URL)
 	root.SetArgs([]string{
-		"short-drama", "+get-thread",
+		"get-thread",
 		"--thread-id", "thread_123",
 		"--run-id", "run_456",
 	})
@@ -723,12 +750,12 @@ func TestShortDramaGetThread(t *testing.T) {
 	}
 }
 
-func TestShortDramaGetThreadRequiresThreadID(t *testing.T) {
+func TestGetThreadRequiresThreadID(t *testing.T) {
 	clearDailyErrorLog(t)
 
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
-	root.SetArgs([]string{"short-drama", "+get-thread"})
+	root.SetArgs([]string{"get-thread"})
 
 	err := root.Execute()
 	if err == nil {
@@ -742,7 +769,7 @@ func TestShortDramaGetThreadRequiresThreadID(t *testing.T) {
 	if len(entries) != 1 {
 		t.Fatalf("log entries = %d, want 1: %#v", len(entries), entries)
 	}
-	if entries[0]["command"] != "short-drama +get-thread" {
+	if entries[0]["command"] != "get-thread" {
 		t.Fatalf("command = %v, want get-thread", entries[0]["command"])
 	}
 	if entries[0]["error"] != "--thread-id is required" {
@@ -750,7 +777,7 @@ func TestShortDramaGetThreadRequiresThreadID(t *testing.T) {
 	}
 }
 
-func TestShortDramaGetThreadRequiresAccessKey(t *testing.T) {
+func TestGetThreadRequiresAccessKey(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		t.Fatal("server should not receive request without access key")
 	}))
@@ -758,7 +785,7 @@ func TestShortDramaGetThreadRequiresAccessKey(t *testing.T) {
 
 	var stdout, stderr bytes.Buffer
 	root := newTestRootCommandWithAccessKey(t, &stdout, &stderr, server.URL, "")
-	root.SetArgs([]string{"short-drama", "+get-thread", "--thread-id", "thread_123"})
+	root.SetArgs([]string{"get-thread", "--thread-id", "thread_123"})
 
 	err := root.Execute()
 	if err == nil {
@@ -767,7 +794,7 @@ func TestShortDramaGetThreadRequiresAccessKey(t *testing.T) {
 	assertAccessKeyGuidance(t, err)
 }
 
-func TestShortDramaListThreadFile(t *testing.T) {
+func TestListThreadFile(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		if r.Method != http.MethodPost {
 			t.Fatalf("method = %s, want POST", r.Method)
@@ -802,7 +829,7 @@ func TestShortDramaListThreadFile(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := newTestRootCommand(t, &stdout, &stderr, server.URL)
 	root.SetArgs([]string{
-		"short-drama", "+list-thread-file",
+		"list-thread-file",
 		"--thread-id", "thread_123",
 		"--page-num", "2",
 		"--page-size", "10",
@@ -816,7 +843,7 @@ func TestShortDramaListThreadFile(t *testing.T) {
 	if got["total"] != float64(1) {
 		t.Fatalf("total = %v, want 1", got["total"])
 	}
-	wantMessage := "<system-remind>\n- total is below 200; continue querying with current --page-num 2\n</system-remind>"
+	wantMessage := "<system-remind>\n- 文件总数小于 200；继续使用当前 --page-num 2 查询\n</system-remind>"
 	if got["message"] != wantMessage {
 		t.Fatalf("message = %v, want current page hint", got["message"])
 	}
@@ -843,7 +870,7 @@ func TestShortDramaListThreadFile(t *testing.T) {
 	}
 }
 
-func TestShortDramaListThreadFileMessageWhenPageFull(t *testing.T) {
+func TestListThreadFileMessageWhenPageFull(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		if !strings.Contains(r.URL.Path, "list_thread_file") {
 			t.Fatalf("path = %s, want list_thread_file path", r.URL.Path)
@@ -855,7 +882,7 @@ func TestShortDramaListThreadFileMessageWhenPageFull(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := newTestRootCommand(t, &stdout, &stderr, server.URL)
 	root.SetArgs([]string{
-		"short-drama", "+list-thread-file",
+		"list-thread-file",
 		"--thread-id", "thread_123",
 		"--page-num", "2",
 		"--page-size", "200",
@@ -866,16 +893,16 @@ func TestShortDramaListThreadFileMessageWhenPageFull(t *testing.T) {
 	}
 
 	got := decodeJSON(t, stdout.Bytes())
-	wantMessage := "<system-remind>\n- total reached 200; query the next page with --page-num 3\n</system-remind>"
+	wantMessage := "<system-remind>\n- 文件总数已达到 200；请使用 --page-num 3 查询下一页\n</system-remind>"
 	if got["message"] != wantMessage {
 		t.Fatalf("message = %v, want next page hint", got["message"])
 	}
 }
 
-func TestShortDramaListThreadFileRequiresThreadID(t *testing.T) {
+func TestListThreadFileRequiresThreadID(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
-	root.SetArgs([]string{"short-drama", "+list-thread-file"})
+	root.SetArgs([]string{"list-thread-file"})
 
 	err := root.Execute()
 	if err == nil {
@@ -886,7 +913,7 @@ func TestShortDramaListThreadFileRequiresThreadID(t *testing.T) {
 	}
 }
 
-func TestShortDramaListThreadFileRequiresAccessKey(t *testing.T) {
+func TestListThreadFileRequiresAccessKey(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		t.Fatal("server should not receive request without access key")
 	}))
@@ -894,7 +921,7 @@ func TestShortDramaListThreadFileRequiresAccessKey(t *testing.T) {
 
 	var stdout, stderr bytes.Buffer
 	root := newTestRootCommandWithAccessKey(t, &stdout, &stderr, server.URL, "")
-	root.SetArgs([]string{"short-drama", "+list-thread-file", "--thread-id", "thread_123"})
+	root.SetArgs([]string{"list-thread-file", "--thread-id", "thread_123"})
 
 	err := root.Execute()
 	if err == nil {
@@ -903,11 +930,11 @@ func TestShortDramaListThreadFileRequiresAccessKey(t *testing.T) {
 	assertAccessKeyGuidance(t, err)
 }
 
-func TestShortDramaListThreadFileRejectsPageSizeAboveMax(t *testing.T) {
+func TestListThreadFileRejectsPageSizeAboveMax(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
 	root.SetArgs([]string{
-		"short-drama", "+list-thread-file",
+		"list-thread-file",
 		"--thread-id", "thread_123",
 		"--page-size", "201",
 	})
@@ -942,13 +969,13 @@ func assertAccessKeyGuidance(t *testing.T, err error) {
 		t.Fatal("error = nil, want access key guidance")
 	}
 	msg := err.Error()
-	if !strings.Contains(msg, "XYQ_ACCESS_KEY is required") {
+	if !strings.Contains(msg, "XYQ_ACCESS_KEY 缺失") {
 		t.Fatalf("error = %q, want access key guidance", err)
 	}
 	if !strings.Contains(msg, "https://xyq.jianying.com/home?tab_name=home") {
 		t.Fatalf("error = %q, want access key settings URL", err)
 	}
-	if !strings.Contains(msg, `export XYQ_ACCESS_KEY="<access-key>"`) {
+	if !strings.Contains(msg, `export XYQ_ACCESS_KEY="<your-access-key>"`) {
 		t.Fatalf("error = %q, want setup command guidance", err)
 	}
 }

package/cmd/update/update.go

@@ -60,21 +60,21 @@ func runUpdate(stdout, stderr io.Writer) error {
 	fmt.Fprintf(stderr, "Updating pippit-tool-cli via npm: %s\n", pkg)
 	restore, err := prepareSelfReplace()
 	if err != nil {
-		return fmt.Errorf("prepare self replace: %w", err)
+		return fmt.Errorf("准备替换当前可执行文件失败: %w", err)
 	}
 	if err := runInheritEnv(stderr, []string{"PIPPIT_CLI_SKIP_SKILLS=1"}, "npm", "install", "-g", pkg); err != nil {
 		restore()
-		return fmt.Errorf("update pippit-tool-cli: %w", err)
+		return fmt.Errorf("更新 pippit-tool-cli 失败: %w", err)
 	}
 
 	root, err := globalPackageRoot(defaultPackage)
 	if err != nil {
-		return fmt.Errorf("locate global package: %w", err)
+		return fmt.Errorf("定位全局 npm 包失败: %w", err)
 	}
 
 	fmt.Fprintln(stderr, "Updating pippit-tool-cli skills...")
 	if err := installSkills(root, stderr); err != nil {
-		return fmt.Errorf("update pippit-tool-cli skills: %w", err)
+		return fmt.Errorf("更新 pippit-tool-cli skills 失败: %w", err)
 	}
 
 	reportBundledSkillTelemetry("update", "cli_update", stderr)
@@ -89,7 +89,7 @@ func globalPackageRoot(pkg string) (string, error) {
 	}
 	npmRoot := strings.TrimSpace(string(out))
 	if npmRoot == "" {
-		return "", fmt.Errorf("npm root -g returned empty output")
+		return "", fmt.Errorf("npm root -g 输出为空")
 	}
 	return filepath.Join(append([]string{npmRoot}, strings.Split(pkg, "/")...)...), nil
 }
@@ -98,7 +98,7 @@ func installSkills(root string, stderr io.Writer) error {
 	if info, err := os.Stat(filepath.Join(root, "skills")); err != nil {
 		return err
 	} else if !info.IsDir() {
-		return fmt.Errorf("%s is not a directory", filepath.Join(root, "skills"))
+		return fmt.Errorf("%s 不是目录", filepath.Join(root, "skills"))
 	}
 	if err := runInherit(stderr, "npx", "-y", "skills", "add", root, "-g", "-y", "--skill", "*"); err != nil {
 		return err
@@ -153,7 +153,7 @@ func reportBundledSkillTelemetry(event string, source string, stderr io.Writer)
 		go func(payload telemetryPayload) {
 			defer wg.Done()
 			if err := reportSkillTelemetry(payload); err != nil && os.Getenv("PIPPIT_CLI_DEBUG_TELEMETRY") == "1" {
-				fmt.Fprintf(stderr, "[pippit-tool-cli] telemetry failed: %v\n", err)
+				fmt.Fprintf(stderr, "[pippit-tool-cli] 埋点上报失败: %v\n", err)
 			}
 		}(payload)
 	}
@@ -188,7 +188,7 @@ func reportSkillTelemetry(payload telemetryPayload) error {
 	defer resp.Body.Close()
 	_, _ = io.Copy(io.Discard, resp.Body)
 	if resp.StatusCode >= 400 {
-		return fmt.Errorf("telemetry returned HTTP %d", resp.StatusCode)
+		return fmt.Errorf("埋点请求返回 HTTP %d", resp.StatusCode)
 	}
 	return nil
 }