@pippit-dev/cli 1.0.1 → 1.0.3

package/cmd/short_drama/short_drama.go

@@ -3,12 +3,12 @@ package short_drama
 import (
 	"fmt"
 	"io"
+	"path/filepath"
 	"strconv"
 	"strings"
 
 	"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"
 )
 
@@ -22,6 +22,7 @@ func NewCommand(stdout, stderr io.Writer, runner *common.Runner) *cobra.Command
 	cmd.SetOut(stdout)
 	cmd.SetErr(stderr)
 	cmd.AddCommand(newShortDramaSubmitRunCommand(stdout, stderr, runner))
+	cmd.AddCommand(newShortDramaUploadFileCommand(stdout, stderr, runner))
 	return cmd
 }
 
@@ -42,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)
@@ -60,6 +61,40 @@ func newShortDramaSubmitRunCommand(stdout, stderr io.Writer, runner *common.Runn
 	return cmd
 }
 
+func newShortDramaUploadFileCommand(stdout, stderr io.Writer, runner *common.Runner) *cobra.Command {
+	var opts common.UploadFileOptions
+
+	cmd := &cobra.Command{
+		Use:   "+upload-file",
+		Short: "Upload a file for the short drama scene",
+		Args:  cobra.NoArgs,
+		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 == "" {
+				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 common.WriteJSON(stdout, result)
+		}),
+	}
+	cmd.SetOut(stdout)
+	cmd.SetErr(stderr)
+	cmd.Flags().StringVar(&opts.Path, "path", "", "local file path to upload")
+	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)
@@ -74,11 +109,19 @@ func withErrorLog(command string, fields func() map[string]string, run func(*cob
 	}
 }
 
-func writeJSON(w io.Writer, v any) error {
-	data, err := sonic.Marshal(v)
-	if err != nil {
-		return err
+func fileNameForLog(path string) string {
+	path = strings.TrimSpace(path)
+	if path == "" {
+		return ""
+	}
+	return filepath.Base(path)
+}
+
+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,12 +124,13 @@ 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",
-		"upload-file",
 		"update",
 		"--version",
 	} {
@@ -158,7 +159,6 @@ func TestShortDramaDoesNotIncludeCommonThreadCommands(t *testing.T) {
 		"+download-result":  true,
 		"+get-thread":       true,
 		"+list-thread-file": true,
-		"+upload-file":      true,
 	}
 	for _, child := range cmd.Commands() {
 		if removedCommands[child.Name()] {
@@ -189,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)
 	}
 }
@@ -214,7 +214,7 @@ func TestShortDramaSubmitRunRequiresAccessKey(t *testing.T) {
 	assertAccessKeyGuidance(t, err)
 }
 
-func TestUploadFile(t *testing.T) {
+func TestShortDramaUploadFile(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)
@@ -265,7 +265,7 @@ func TestUploadFile(t *testing.T) {
 
 	var stdout, stderr bytes.Buffer
 	root := newTestRootCommand(t, &stdout, &stderr, server.URL)
-	root.SetArgs([]string{"upload-file", "--path", path})
+	root.SetArgs([]string{"short-drama", "+upload-file", "--path", path})
 
 	if err := root.Execute(); err != nil {
 		t.Fatalf("Execute() error = %v, stderr = %s", err, stderr.String())
@@ -277,10 +277,10 @@ func TestUploadFile(t *testing.T) {
 	}
 }
 
-func TestUploadFileRequiresPath(t *testing.T) {
+func TestShortDramaUploadFileRequiresPath(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
-	root.SetArgs([]string{"upload-file"})
+	root.SetArgs([]string{"short-drama", "+upload-file"})
 
 	err := root.Execute()
 	if err == nil {
@@ -291,7 +291,7 @@ func TestUploadFileRequiresPath(t *testing.T) {
 	}
 }
 
-func TestUploadFileRequiresAccessKey(t *testing.T) {
+func TestShortDramaUploadFileRequiresAccessKey(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")
 	}))
@@ -305,7 +305,7 @@ func TestUploadFileRequiresAccessKey(t *testing.T) {
 
 	var stdout, stderr bytes.Buffer
 	root := newTestRootCommandWithAccessKey(t, &stdout, &stderr, server.URL, "")
-	root.SetArgs([]string{"upload-file", "--path", path})
+	root.SetArgs([]string{"short-drama", "+upload-file", "--path", path})
 
 	err := root.Execute()
 	if err == nil {
@@ -314,7 +314,7 @@ func TestUploadFileRequiresAccessKey(t *testing.T) {
 	assertAccessKeyGuidance(t, err)
 }
 
-func TestUploadFileRejectsUnsupportedFileType(t *testing.T) {
+func TestShortDramaUploadFileRejectsUnsupportedFileType(t *testing.T) {
 	cwd := chdirTemp(t)
 	path := filepath.Join(cwd, "story.png")
 	if err := os.WriteFile(path, []byte("png-data"), 0o644); err != nil {
@@ -323,13 +323,13 @@ func TestUploadFileRejectsUnsupportedFileType(t *testing.T) {
 
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
-	root.SetArgs([]string{"upload-file", "--path", path})
+	root.SetArgs([]string{"short-drama", "+upload-file", "--path", path})
 
 	err := root.Execute()
 	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)
 	}
 }
@@ -601,7 +601,7 @@ func TestDownloadResultRejectsOutputDirFlag(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)
 	}
 }
@@ -629,7 +629,7 @@ func TestDownloadResultRejectsInvalidScheme(t *testing.T) {
 	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)
 	}
 }
@@ -654,7 +654,7 @@ func TestDownloadResultAllFailed(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)
@@ -843,7 +843,7 @@ func TestListThreadFile(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"])
 	}
@@ -893,7 +893,7 @@ func TestListThreadFileMessageWhenPageFull(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"])
 	}
@@ -969,7 +969,7 @@ 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") {
+	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") {

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
 }

package/internal/common/client.go

@@ -20,6 +20,7 @@ import (
 
 type Client interface {
 	SendRequest(ctx context.Context, path string, body any, out any) error
+	SendRequestWithHeaders(ctx context.Context, path string, body any, headers map[string]string, out any) error
 	SendMultipartRequest(ctx context.Context, path string, fields map[string]string, file MultipartFile, out any) error
 }
 
@@ -53,6 +54,10 @@ func NewHTTPClient(baseURL string, timeout time.Duration, authorizer RequestAuth
 }
 
 func (c *httpClient) SendRequest(ctx context.Context, path string, body any, out any) error {
+	return c.SendRequestWithHeaders(ctx, path, body, nil, out)
+}
+
+func (c *httpClient) SendRequestWithHeaders(ctx context.Context, path string, body any, headers map[string]string, out any) error {
 	method := http.MethodPost
 	if body == nil {
 		method = http.MethodGet
@@ -67,27 +72,27 @@ func (c *httpClient) SendRequest(ctx context.Context, path string, body any, out
 	if body != nil {
 		payload, err := sonic.Marshal(body)
 		if err != nil {
-			return fmt.Errorf("encode body: %w", err)
+			return fmt.Errorf("编码请求体失败: %w", err)
 		}
 		reader = bytes.NewReader(payload)
 	}
 
 	req, err := http.NewRequestWithContext(ctx, method, reqURL, reader)
 	if err != nil {
-		return fmt.Errorf("build %s request: %w", method, err)
+		return fmt.Errorf("构造 %s 请求失败: %w", method, err)
 	}
 	if body != nil {
 		req.Header.Set("Content-Type", "application/json")
 	}
 
 	if c.authorizer == nil {
-		return fmt.Errorf("authorized request requires authorizer")
+		return fmt.Errorf("授权请求缺少认证器")
 	}
 	if err := c.authorizer.Inject(ctx, req); err != nil {
-		return fmt.Errorf("inject auth headers: %w", err)
+		return fmt.Errorf("写入认证请求头失败: %w", err)
 	}
 
-	c.injectHeaders(req)
+	c.injectHeaders(req, headers)
 
 	// If out is **http.Response, return the raw response for streaming (e.g. file download).
 	if out != nil {
@@ -95,11 +100,11 @@ func (c *httpClient) SendRequest(ctx context.Context, path string, body any, out
 			if rv.Elem().Type().Elem() == reflect.TypeOf(http.Response{}) {
 				resp, err := c.httpClient.Do(req)
 				if err != nil {
-					return fmt.Errorf("%s %s failed: %w", method, reqURL, err)
+					return fmt.Errorf("%s %s 请求失败: %w", method, reqURL, err)
 				}
 				if resp.StatusCode >= 400 {
 					defer resp.Body.Close()
-					return fmt.Errorf("%s %s returned HTTP %d", method, reqURL, resp.StatusCode)
+					return fmt.Errorf("%s %s 返回 HTTP %d", method, reqURL, resp.StatusCode)
 				}
 				rv.Elem().Set(reflect.ValueOf(resp))
 				return nil
@@ -118,7 +123,7 @@ func (c *httpClient) SendMultipartRequest(ctx context.Context, path string, fiel
 		return err
 	}
 	if file.FieldName == "" {
-		return fmt.Errorf("multipart file field name is required")
+		return fmt.Errorf("multipart 文件字段名不能为空")
 	}
 	if file.FileName == "" {
 		file.FileName = filepath.Base(file.Path)
@@ -133,7 +138,7 @@ func (c *httpClient) SendMultipartRequest(ctx context.Context, path string, fiel
 	if err != nil {
 		_ = pr.Close()
 		_ = pw.Close()
-		return fmt.Errorf("build POST request: %w", err)
+		return fmt.Errorf("构造 POST 请求失败: %w", err)
 	}
 	req.Header.Set("Content-Type", writer.FormDataContentType())
 	req.Header.Set("Accept", "application/json")
@@ -141,14 +146,14 @@ func (c *httpClient) SendMultipartRequest(ctx context.Context, path string, fiel
 	if c.authorizer == nil {
 		_ = pr.Close()
 		_ = pw.Close()
-		return fmt.Errorf("authorized request requires authorizer")
+		return fmt.Errorf("授权请求缺少认证器")
 	}
 	if err := c.authorizer.Inject(ctx, req); err != nil {
 		_ = pr.Close()
 		_ = pw.Close()
-		return fmt.Errorf("inject auth headers: %w", err)
+		return fmt.Errorf("写入认证请求头失败: %w", err)
 	}
-	c.injectHeaders(req)
+	c.injectHeaders(req, nil)
 
 	go func() {
 		err := writeMultipartBody(writer, fields, file)
@@ -172,7 +177,7 @@ func writeMultipartBody(writer *multipart.Writer, fields map[string]string, file
 
 	f, err := os.Open(file.Path)
 	if err != nil {
-		return fmt.Errorf("open upload file: %w", err)
+		return fmt.Errorf("打开上传文件失败: %w", err)
 	}
 	defer f.Close()
 
@@ -184,7 +189,7 @@ func writeMultipartBody(writer *multipart.Writer, fields map[string]string, file
 		return err
 	}
 	if _, err := io.Copy(part, f); err != nil {
-		return fmt.Errorf("write upload file: %w", err)
+		return fmt.Errorf("写入上传文件失败: %w", err)
 	}
 	return nil
 }
@@ -195,38 +200,41 @@ func escapeQuotes(s string) string {
 	return quotesReplacer.Replace(s)
 }
 
-func (c *httpClient) injectHeaders(req *http.Request) {
+func (c *httpClient) injectHeaders(req *http.Request, headers map[string]string) {
 	for k, values := range c.headers {
 		for _, v := range values {
 			req.Header.Add(k, v)
 		}
 	}
 	req.Header.Set("User-Agent", "Pippit-CLI/1.0")
+	for k, v := range headers {
+		req.Header.Set(k, v)
+	}
 }
 
 func (c *httpClient) do(req *http.Request, out any) error {
 	resp, err := c.httpClient.Do(req)
 	if err != nil {
-		return fmt.Errorf("%s %s failed: %w", req.Method, req.URL.String(), err)
+		return fmt.Errorf("%s %s 请求失败: %w", req.Method, req.URL.String(), err)
 	}
 	defer resp.Body.Close()
 
 	data, err := io.ReadAll(resp.Body)
 	if err != nil {
-		return fmt.Errorf("read response body: %w", err)
+		return fmt.Errorf("读取响应体失败: %w", err)
 	}
 	if resp.StatusCode >= 400 {
 		msg := strings.TrimSpace(string(data))
 		if msg == "" {
 			msg = http.StatusText(resp.StatusCode)
 		}
-		return fmt.Errorf("%s %s returned HTTP %d: %s", req.Method, req.URL.String(), resp.StatusCode, msg)
+		return fmt.Errorf("%s %s 返回 HTTP %d: %s", req.Method, req.URL.String(), resp.StatusCode, msg)
 	}
 	if out == nil || len(data) == 0 {
 		return nil
 	}
 	if err := sonic.Unmarshal(data, out); err != nil {
-		return fmt.Errorf("decode response body: %w", err)
+		return fmt.Errorf("解析响应体失败: %w", err)
 	}
 	return nil
 }
@@ -236,7 +244,7 @@ func (c *httpClient) resolveURL(path string, query map[string]string) (string, e
 		return appendQuery(path, query)
 	}
 	if c.baseURL == "" {
-		return "", fmt.Errorf("base URL is required for relative path %q", path)
+		return "", fmt.Errorf("相对路径 %q 需要配置 base URL", path)
 	}
 	if !strings.HasPrefix(path, "/") {
 		path = "/" + path
@@ -247,7 +255,7 @@ func (c *httpClient) resolveURL(path string, query map[string]string) (string, e
 func appendQuery(raw string, query map[string]string) (string, error) {
 	u, err := url.Parse(raw)
 	if err != nil {
-		return "", fmt.Errorf("parse URL: %w", err)
+		return "", fmt.Errorf("解析 URL 失败: %w", err)
 	}
 	values := u.Query()
 	for k, v := range query {

package/internal/common/download_results.go

@@ -58,20 +58,26 @@ type downloadTaskResult struct {
 
 func DownloadResult(ctx context.Context, opts DownloadResultOptions, runner *Runner) (*DownloadResultResponse, error) {
 	if err := ctx.Err(); err != nil {
-		return nil, err
+		if errors.Is(err, context.Canceled) {
+			return nil, fmt.Errorf("下载已取消")
+		}
+		if errors.Is(err, context.DeadlineExceeded) {
+			return nil, fmt.Errorf("下载超时")
+		}
+		return nil, fmt.Errorf("下载上下文异常: %w", err)
 	}
 	rawURL := strings.TrimSpace(opts.URL)
 	if rawURL == "" {
-		return nil, fmt.Errorf("download url is required")
+		return nil, fmt.Errorf("下载 URL 不能为空")
 	}
 
 	outputPath := strings.TrimSpace(opts.OutputPath)
 	if outputPath == "" {
-		return nil, fmt.Errorf("output_path is required")
+		return nil, fmt.Errorf("输出路径不能为空")
 	}
 	if info, err := os.Stat(outputPath); err == nil {
 		if info.IsDir() {
-			return nil, fmt.Errorf("output path %q is a directory", outputPath)
+			return nil, fmt.Errorf("输出路径 %q 是目录，请指定文件", outputPath)
 		}
 		if shouldSkipExistingFile(info, opts.UpdatedAt) {
 			return &DownloadResultResponse{
@@ -80,19 +86,19 @@ func DownloadResult(ctx context.Context, opts DownloadResultOptions, runner *Run
 			}, nil
 		}
 	} else if !os.IsNotExist(err) {
-		return nil, fmt.Errorf("stat output path: %w", err)
+		return nil, fmt.Errorf("获取输出路径信息失败: %w", err)
 	}
 	outputDir := filepath.Dir(outputPath)
 	if err := os.MkdirAll(outputDir, 0o755); err != nil {
-		return nil, fmt.Errorf("create output dir: %w", err)
+		return nil, fmt.Errorf("创建输出目录失败: %w", err)
 	}
 
 	parsed, err := url.Parse(rawURL)
 	if err != nil {
-		return nil, fmt.Errorf("invalid url %q: %w", rawURL, err)
+		return nil, fmt.Errorf("URL %q 不合法: %w", rawURL, err)
 	}
 	if parsed.Scheme != "http" && parsed.Scheme != "https" {
-		return nil, fmt.Errorf("invalid url scheme %q in %q, only http and https are allowed", parsed.Scheme, rawURL)
+		return nil, fmt.Errorf("URL %q 的协议 %q 不支持，仅支持 http 和 https", rawURL, parsed.Scheme)
 	}
 	tasks := []downloadTask{
 		{
@@ -166,7 +172,7 @@ func DownloadResult(ctx context.Context, opts DownloadResultOptions, runner *Run
 		Errors:     errorList,
 	}
 	if len(downloaded) == 0 && len(errorList) > 0 {
-		return result, fmt.Errorf("all %d download(s) failed", len(errorList))
+		return result, fmt.Errorf("全部 %d 个下载任务失败", len(errorList))
 	}
 	return result, nil
 }
@@ -200,7 +206,7 @@ func downloadFileWithRetry(ctx context.Context, client Client, rawURL string, ta
 			return err
 		}
 	}
-	return fmt.Errorf("failed after %d retries: %w", maxDownloadRetries, lastErr)
+	return fmt.Errorf("重试 %d 次后仍失败: %w", maxDownloadRetries, lastErr)
 }
 
 func isRetryableError(err error) bool {
@@ -230,18 +236,18 @@ func downloadFile(ctx context.Context, client Client, rawURL string, targetPath
 	}()
 	out, err := os.Create(tmpPath)
 	if err != nil {
-		return fmt.Errorf("create temp file: %w", err)
+		return fmt.Errorf("创建临时文件失败: %w", err)
 	}
 	_, copyErr := io.Copy(out, resp.Body)
 	closeErr := out.Close()
 	if copyErr != nil {
-		return fmt.Errorf("write temp file: %w", copyErr)
+		return fmt.Errorf("写入临时文件失败: %w", copyErr)
 	}
 	if closeErr != nil {
-		return fmt.Errorf("close temp file: %w", closeErr)
+		return fmt.Errorf("关闭临时文件失败: %w", closeErr)
 	}
 	if err := os.Rename(tmpPath, targetPath); err != nil {
-		return fmt.Errorf("replace target file: %w", err)
+		return fmt.Errorf("替换目标文件失败: %w", err)
 	}
 	tempActive = false
 	if updatedAt > 0 {