@pippit-dev/cli 0.0.4 → 0.0.6

package/README.md

@@ -13,7 +13,7 @@ export XYQ_ACCESS_KEY="<access-key>"
 pippit-cli short-drama +submit-run --message "写一个赛博朋克短剧开头"
 pippit-cli short-drama +upload-file --path ./story.md
 pippit-cli short-drama +get-thread --thread-id thread_123 --run-id run_456 --after-seq 0
-pippit-cli short-drama +download-result --output-dir ./thread_123/results/result.mp4 --url URL
+pippit-cli short-drama +download-result --output-path ./thread_123/results/result.mp4 --url URL
 ```
 
 NPM package names must be lowercase, so the publishable package name is
@@ -26,7 +26,7 @@ export XYQ_ACCESS_KEY="<access-key>"
 go run . short-drama +submit-run --message "写一个赛博朋克短剧开头"
 go run . short-drama +upload-file --path ./story.md
 go run . short-drama +get-thread --thread-id thread_123 --run-id run_456 --after-seq 0
-go run . short-drama +download-result --output-dir ./thread_123/results/result.mp4 --url URL
+go run . short-drama +download-result --output-path ./thread_123/results/result.mp4 --url URL
 ```
 
 `+submit-run` calls `/api/biz/v1/skill/submit_run` and prints `thread_id`,
@@ -34,7 +34,7 @@ go run . short-drama +download-result --output-dir ./thread_123/results/result.m
 `+get-thread` calls
 `/api/biz/v1/skill/get_thread` and prints extracted `messages`.
 `+download-result` downloads the result URL to
-the `--output-dir` file path. `+upload-file` is still mocked while
+the `--output-path` file path. `+upload-file` is still mocked while
 its real service contract is wired.
 
 ## HTTP Client

package/checksums.txt

@@ -0,0 +1,6 @@
+2d6ebc213dd8401622a5315846a1f2125b321ca55ed3689b661268ae0cdc7bb4  pippit-cli-0.0.6-darwin-amd64.tar.gz
+2bedf7ed2f9afd313e8b983763e90e66a835b2f340fcda905cae8c82ba553735  pippit-cli-0.0.6-darwin-arm64.tar.gz
+37ecf71b5d701dc257c0287983b511e403ff0cfc7690a07a0c4bcd32837c365e  pippit-cli-0.0.6-linux-amd64.tar.gz
+b6e41054960723d18809e1bad498f59fb7f68550e07c11e867f9580ac91a42da  pippit-cli-0.0.6-linux-arm64.tar.gz
+2103cb871134ec47f34b53b53e93d95d7190888bc035b8c78d48b925a3ace9eb  pippit-cli-0.0.6-windows-amd64.zip
+263fbabede0ad9c266ef23cdad2292297a35b67d2301c14f400308198bab4655  pippit-cli-0.0.6-windows-arm64.zip

package/cmd/short_drama/short_drama.go

@@ -96,9 +96,9 @@ func newShortDramaDownloadResultCommand(stdout, stderr io.Writer, runner *common
 		Short: "Download a generated result URL",
 		Args:  cobra.NoArgs,
 		RunE: func(cmd *cobra.Command, _ []string) error {
-			opts.OutputDir = strings.TrimSpace(opts.OutputDir)
-			if opts.OutputDir == "" {
-				return fmt.Errorf("--output-dir is required")
+			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 == "" {
@@ -118,7 +118,7 @@ func newShortDramaDownloadResultCommand(stdout, stderr io.Writer, runner *common
 	cmd.SetOut(stdout)
 	cmd.SetErr(stderr)
 	cmd.Flags().StringVar(&opts.URL, "url", "", "URL to download")
-	cmd.Flags().StringVar(&opts.OutputDir, "output-dir", "", "output file path")
+	cmd.Flags().StringVar(&opts.OutputPath, "output-path", "", "local output file path")
 	cmd.Flags().IntVar(&opts.Workers, "workers", 5, "parallel download workers")
 	return cmd
 }

package/cmd/short_drama_test.go

@@ -181,7 +181,7 @@ func TestShortDramaDownloadResult(t *testing.T) {
 	outputPath := filepath.Join("results", "cover.jpeg")
 	root.SetArgs([]string{
 		"short-drama", "+download-result",
-		"--output-dir", outputPath,
+		"--output-path", outputPath,
 		"--workers", "2",
 		"--url", server.URL + "/image?filename=ignored.jpeg",
 	})
@@ -191,8 +191,8 @@ func TestShortDramaDownloadResult(t *testing.T) {
 	}
 
 	got := decodeJSON(t, stdout.Bytes())
-	if got["output_dir"] != outputPath {
-		t.Fatalf("output_dir = %v, want %s", got["output_dir"], outputPath)
+	if got["output_path"] != outputPath {
+		t.Fatalf("output_path = %v, want %s", got["output_path"], outputPath)
 	}
 	if got["total"] != float64(1) {
 		t.Fatalf("total = %v, want 1", got["total"])
@@ -233,7 +233,7 @@ func TestShortDramaDownloadResultSkipsExistingFile(t *testing.T) {
 	root := NewRootCommand(&stdout, &stderr)
 	root.SetArgs([]string{
 		"short-drama", "+download-result",
-		"--output-dir", outputPath,
+		"--output-path", outputPath,
 		"--url", server.URL + "/image",
 	})
 
@@ -245,8 +245,8 @@ func TestShortDramaDownloadResultSkipsExistingFile(t *testing.T) {
 	}
 
 	got := decodeJSON(t, stdout.Bytes())
-	if got["output_dir"] != outputPath {
-		t.Fatalf("output_dir = %v, want %s", got["output_dir"], outputPath)
+	if got["output_path"] != outputPath {
+		t.Fatalf("output_path = %v, want %s", got["output_path"], outputPath)
 	}
 	if got["total"] != float64(0) {
 		t.Fatalf("total = %v, want 0", got["total"])
@@ -258,7 +258,7 @@ func TestShortDramaDownloadResultSkipsExistingFile(t *testing.T) {
 	assertFileContent(t, outputPath, "existing-data")
 }
 
-func TestShortDramaDownloadResultRequiresOutputDir(t *testing.T) {
+func TestShortDramaDownloadResultRequiresOutputPath(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"})
@@ -267,15 +267,33 @@ func TestShortDramaDownloadResultRequiresOutputDir(t *testing.T) {
 	if err == nil {
 		t.Fatal("Execute() error = nil, want validation error")
 	}
-	if !strings.Contains(err.Error(), "--output-dir is required") {
-		t.Fatalf("error = %q, want output-dir validation", err)
+	if !strings.Contains(err.Error(), "--output-path is required") {
+		t.Fatalf("error = %q, want output-path validation", err)
+	}
+}
+
+func TestShortDramaDownloadResultRejectsOutputDirFlag(t *testing.T) {
+	var stdout, stderr bytes.Buffer
+	root := NewRootCommand(&stdout, &stderr)
+	root.SetArgs([]string{
+		"short-drama", "+download-result",
+		"--output-dir", filepath.Join("results", "image.png"),
+		"--url", "https://example.com/image.png",
+	})
+
+	err := root.Execute()
+	if err == nil {
+		t.Fatal("Execute() error = nil, want unknown flag error")
+	}
+	if !strings.Contains(err.Error(), "unknown flag: --output-dir") {
+		t.Fatalf("error = %q, want output-dir rejection", err)
 	}
 }
 
 func TestShortDramaDownloadResultRequiresURL(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
-	root.SetArgs([]string{"short-drama", "+download-result", "--output-dir", filepath.Join("results", "image.png")})
+	root.SetArgs([]string{"short-drama", "+download-result", "--output-path", filepath.Join("results", "image.png")})
 
 	err := root.Execute()
 	if err == nil {
@@ -289,7 +307,7 @@ func TestShortDramaDownloadResultRequiresURL(t *testing.T) {
 func TestShortDramaDownloadResultRejectsInvalidScheme(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	root := NewRootCommand(&stdout, &stderr)
-	root.SetArgs([]string{"short-drama", "+download-result", "--output-dir", filepath.Join("results", "image.png"), "--url", "file:///etc/passwd"})
+	root.SetArgs([]string{"short-drama", "+download-result", "--output-path", filepath.Join("results", "image.png"), "--url", "file:///etc/passwd"})
 
 	err := root.Execute()
 	if err == nil {
@@ -311,7 +329,7 @@ func TestShortDramaDownloadResultAllFailed(t *testing.T) {
 	root := NewRootCommand(&stdout, &stderr)
 	root.SetArgs([]string{
 		"short-drama", "+download-result",
-		"--output-dir", filepath.Join("results", "missing.png"),
+		"--output-path", filepath.Join("results", "missing.png"),
 		"--url", server.URL + "/notfound",
 	})
 
@@ -337,7 +355,7 @@ func TestShortDramaDownloadResultOutputPath(t *testing.T) {
 	outputPath := filepath.Join("custom", "nested", "cover.png")
 	root.SetArgs([]string{
 		"short-drama", "+download-result",
-		"--output-dir", outputPath,
+		"--output-path", outputPath,
 		"--url", server.URL + "/image.png",
 	})
 
@@ -346,8 +364,8 @@ func TestShortDramaDownloadResultOutputPath(t *testing.T) {
 	}
 
 	got := decodeJSON(t, stdout.Bytes())
-	if got["output_dir"] != outputPath {
-		t.Fatalf("output_dir = %v, want %s", got["output_dir"], outputPath)
+	if got["output_path"] != outputPath {
+		t.Fatalf("output_path = %v, want %s", got["output_path"], outputPath)
 	}
 	assertFileContent(t, outputPath, "image-data")
 }

package/internal/common/download_results.go

@@ -21,9 +21,9 @@ import (
 
 // DownloadResultOptions is the command-facing shape for downloading one result URL.
 type DownloadResultOptions struct {
-	URL       string `json:"url"`
-	OutputDir string `json:"output_dir"`
-	Workers   int    `json:"workers"`
+	URL        string `json:"url"`
+	OutputPath string `json:"output_path"`
+	Workers    int    `json:"workers"`
 }
 
 type DownloadResultError struct {
@@ -33,7 +33,7 @@ type DownloadResultError struct {
 
 // DownloadResultResponse is the JSON envelope printed by `pippit-cli short-drama +download-result`.
 type DownloadResultResponse struct {
-	OutputDir    string                 `json:"output_dir"`
+	OutputPath   string                 `json:"output_path"`
 	Downloaded   []string               `json:"downloaded"`
 	AlreadyExist []string               `json:"already_exist,omitempty"`
 	Total        int                    `json:"total"`
@@ -64,9 +64,9 @@ func DownloadResult(ctx context.Context, opts DownloadResultOptions, runner *Run
 		return nil, fmt.Errorf("download url is required")
 	}
 
-	outputPath := strings.TrimSpace(opts.OutputDir)
+	outputPath := strings.TrimSpace(opts.OutputPath)
 	if outputPath == "" {
-		return nil, fmt.Errorf("output_dir is required")
+		return nil, fmt.Errorf("output_path is required")
 	}
 	// check if the output path exists and is a file
 	if info, err := os.Stat(outputPath); err == nil {
@@ -74,7 +74,7 @@ func DownloadResult(ctx context.Context, opts DownloadResultOptions, runner *Run
 			return nil, fmt.Errorf("output path %q is a directory", outputPath)
 		}
 		return &DownloadResultResponse{
-			OutputDir:    outputPath,
+			OutputPath:   outputPath,
 			AlreadyExist: []string{outputPath},
 			Total:        0,
 		}, nil
@@ -159,7 +159,7 @@ func DownloadResult(ctx context.Context, opts DownloadResultOptions, runner *Run
 	})
 
 	result := &DownloadResultResponse{
-		OutputDir:  outputPath,
+		OutputPath: outputPath,
 		Downloaded: downloaded,
 		Total:      len(downloaded),
 		Errors:     errorList,

package/package.json

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

package/skills/short-drama/SKILL.md

@@ -27,7 +27,7 @@ metadata:
 2. **查询会话进展** - 根据 `thread_id`、`run_id`、`after_seq` 拉取短剧任务消息列表。
 3. **上传文件** - 上传短剧相关参考文件，得到文件 ID，供后续任务引用。
 4. **获取会话文件** - 根据 `thread_id` 拉取会话文件列表，得到 `file_path`、`file_name`、`download_url`。
-5. **下载文件资源** - 使用文件列表中的 `download_url` 下载资源，并按 `file_path` 写入用户本地目录。
+5. **下载文件资源** - 使用文件列表中的 `download_url` 下载资源，并按 `file_path` 写入用户本地目标文件路径。
 
 ## 前置要求
 
@@ -90,11 +90,11 @@ pippit-cli short-drama +list-thread-file --thread-id THREAD_ID --page-num 1 --pa
 ### 5. 下载文件资源
 
 ```bash
-# 下载文件资源到指定目录
-pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-dir FILE_PATH 
+# 下载文件资源到指定文件路径
+pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PATH
 ```
 
-`+download-result` 负责把会话产生的文件，通过URL下载到 `file_path`目录中，如果目标文件已存在，跳过下载。
+`FILE_PATH` 必须直接使用 `+list-thread-file` 返回的完整 `file_path`，包含文件名，不要取父目录。`+download-result` 负责把会话产生的文件通过 URL 下载到该目标文件路径；如果目标文件已存在，跳过下载。
 
 ## 典型工作流
 
@@ -112,11 +112,11 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-dir FILE_PAT
    - 如果后端 Agent 提出问题：展示问题，等待用户回复
 5. 解析 `list-thread-file` 返回的 files，只获取文件元信息：
    - 对每个文件取 file_path、file_name、download_url
-   - 按 file_path 在用户本地环境构建目录
-   - 如果在工作目录中 file_name 已存在：跳过下载
+   - 将 file_path 作为本地目标文件路径，包含文件名
+   - 如果 file_path 已存在：跳过下载
 6. 对缺失的本地文件，调用 +download-result 并行下载资源：
    - 使用第 5 步获取的 download_url 作为 --url
-   - 使用第 5 步获取的 file_path 作为 --output-dir
+   - 使用第 5 步获取的完整 file_path 作为 --output-path
 7. 如用户继续追加需求，使用同一 thread_id 再次 submit-run
 ```
 
@@ -212,8 +212,8 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-dir FILE_PAT
 
 ```json
 {
-  "output_dir": "/path/to/output-dir",
-  "downloaded": ["/path/to/output-dir/01.md"],
+  "output_path": "./{thread-id}/{file_path}/{file_name}",
+  "downloaded": ["./{thread-id}/{file_path}/{file_name}"],
   "total": 1
 }
 ```
@@ -231,14 +231,14 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-dir FILE_PAT
    → 跳过
 2. file_path在本地不存在
    → 记录该file_path和URL
-   → 多个待下载的文件 → 使用 +download-result 工具并行下载文件到本地
+   → 使用 +download-result 将URL资源下载到该file_path
 ```
 
 ### 并行下载文件资源
 
 只对目标路径不存在的文件调用下载工具，可并行：
 
-1. 调用 `pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-dir FILE_PATH`。
+1. 调用 `pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PATH`。
 2. 下载完成后，向用户展示本地文件路径；如果某个文件下载失败，只报告该文件错误，不阻塞已成功落盘的文件展示。
 
 ## 向用户展示内容
@@ -276,5 +276,5 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-dir FILE_PAT
 - `--after-seq` 用于增量拉取消息，首次查询可设置为 `0`。
 - `+upload-file` 当前用于短剧场景文件上传链路，上传后将返回可传给 `+submit-run` 的文件 ID。
 - `+list-thread-file` 只需要 `thread_id`；分页参数默认 `--page-num 1 --page-size 100`。
-- `+list-thread-file` 和 `+download-result` 是两个不同的 CLI 指令：前者获取会话文件元信息，后者下载 URL 资源并写入到本地目录。
-- `+download-result` 接收 `--url`、`--output-dir`、`--workers`。
+- `+list-thread-file` 和 `+download-result` 是两个不同的 CLI 指令：前者获取会话文件元信息，后者下载 URL 资源并写入到本地目标文件路径。
+- `+download-result` 接收 `--url`、`--output-path`、`--workers`；`--output-path` 必须是包含文件名的目标文件路径。