@pippit-dev/cli 0.0.7 → 0.0.9

package/checksums.txt

@@ -1,6 +1,6 @@
-b2cba39200b8114a21e496bca83d677fe2b478b23e374666a0a8cab2ad2e457e  pippit-cli-0.0.7-darwin-amd64.tar.gz
-44be0c9cb8dc98edec7519a704290c06dcfa966360c48bcad709f1e53d5a80d0  pippit-cli-0.0.7-darwin-arm64.tar.gz
-175816681f3affd9eb20900c6e0cfbc035ec1f24eaf06dc6c41686b6cdea5ec7  pippit-cli-0.0.7-linux-amd64.tar.gz
-ebf0c0e9b4b2bbed872acce3f9cf07dd0b21c52ab7c11c62d6778f820c0479fa  pippit-cli-0.0.7-linux-arm64.tar.gz
-573c22cdbd208e9a9a541ae4f67cd922e63129995716d7b92e665d828cec8f49  pippit-cli-0.0.7-windows-amd64.zip
-996d18e5ca27083ee8e9b200f39571c53ff8829f7fdc85c8ba7631778e7f3178  pippit-cli-0.0.7-windows-arm64.zip
+3f9cf3a1fabbc3d178c89c3d8044c18d86110faf7d26ffa469930319d13d68e6  pippit-cli-0.0.9-darwin-amd64.tar.gz
+5e14f8ce6034564d9646918e49f8f2d7b6178708518b528c3f95a0af67bd3cd5  pippit-cli-0.0.9-darwin-arm64.tar.gz
+91e90ab17b577b402c7676965b701df7d317b90a2e2b717929bdadce96158a71  pippit-cli-0.0.9-linux-amd64.tar.gz
+dad9fe0a930c91c52036290a121d721e7a100e80700b1b90807988a577a79342  pippit-cli-0.0.9-linux-arm64.tar.gz
+62af305df2c5d5b6cfaa615f36cb0f014007ed35df8c4ffdf07b527e09993842  pippit-cli-0.0.9-windows-amd64.zip
+c6d40c79e4bf777ad71caba29191a4fc3a25f34f9643157ce08ec270b8d6eb42  pippit-cli-0.0.9-windows-arm64.zip

package/cmd/short_drama/short_drama.go

@@ -164,8 +164,8 @@ func newShortDramaListThreadFileCommand(stdout, stderr io.Writer, runner *common
 			if opts.ThreadID == "" {
 				return fmt.Errorf("--thread-id is required")
 			}
-			if opts.PageSize <= 0 || opts.PageSize > 1000 {
-				return fmt.Errorf("--page-size must be between 1 and 1000")
+			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")
@@ -182,7 +182,7 @@ func newShortDramaListThreadFileCommand(stdout, stderr io.Writer, runner *common
 	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", 1000, "number of files per page (between 1 and 1000)")
+	cmd.Flags().IntVar(&opts.PageSize, "page-size", common.MaxListThreadFilePageSize, fmt.Sprintf("number of files per page (between 1 and %d)", common.MaxListThreadFilePageSize))
 	return cmd
 }
 

package/cmd/short_drama_test.go

@@ -258,11 +258,12 @@ func TestShortDramaDownloadResultSkipsExistingFile(t *testing.T) {
 	assertFileContent(t, outputPath, "existing-data")
 }
 
-func TestShortDramaDownloadResultSkipsMetaJSON(t *testing.T) {
-	serverCalled := false
+func TestShortDramaDownloadResultDownloadsMetaJSON(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		serverCalled = true
-		t.Fatal("server should not receive request for meta.json")
+		if r.URL.Path != "/meta.json" {
+			t.Fatalf("path = %s, want meta.json", r.URL.Path)
+		}
+		_, _ = w.Write([]byte(`{"ok":true}`))
 	}))
 	defer server.Close()
 
@@ -280,12 +281,6 @@ func TestShortDramaDownloadResultSkipsMetaJSON(t *testing.T) {
 	if err := root.Execute(); err != nil {
 		t.Fatalf("Execute() error = %v, stderr = %s", err, stderr.String())
 	}
-	if serverCalled {
-		t.Fatal("server was called, want meta.json to skip download")
-	}
-	if _, err := os.Stat(outputPath); !os.IsNotExist(err) {
-		t.Fatalf("meta.json stat err = %v, want file to be absent", err)
-	}
 
 	got := decodeJSON(t, stdout.Bytes())
 	if got["output_path"] != outputPath {
@@ -294,10 +289,11 @@ func TestShortDramaDownloadResultSkipsMetaJSON(t *testing.T) {
 	if _, ok := got["total"]; ok {
 		t.Fatalf("total should not be returned: %#v", got)
 	}
-	skipped, ok := got["skipped"].([]any)
-	if !ok || len(skipped) != 1 || skipped[0] != outputPath {
-		t.Fatalf("skipped = %#v, want meta.json output path", got["skipped"])
+	downloaded, ok := got["downloaded"].([]any)
+	if !ok || len(downloaded) != 1 || downloaded[0] != outputPath {
+		t.Fatalf("downloaded = %#v, want meta.json output path", got["downloaded"])
 	}
+	assertFileContent(t, outputPath, `{"ok":true}`)
 }
 
 func TestShortDramaDownloadResultRequiresOutputPath(t *testing.T) {
@@ -557,7 +553,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 1000; continue querying with current --page-num 2\n</system-remind>"
+	wantMessage := "<system-remind>\n- total is below 200; continue querying with current --page-num 2\n</system-remind>"
 	if got["message"] != wantMessage {
 		t.Fatalf("message = %v, want current page hint", got["message"])
 	}
@@ -586,7 +582,7 @@ func TestShortDramaListThreadFileMessageWhenPageFull(t *testing.T) {
 		if !strings.Contains(r.URL.Path, "list_thread_file") {
 			t.Fatalf("path = %s, want list_thread_file path", r.URL.Path)
 		}
-		_, _ = w.Write([]byte(`{"ret":"0","errmsg":"","data":{"total":1000,"files":[]}}`))
+		_, _ = w.Write([]byte(`{"ret":"0","errmsg":"","data":{"total":200,"files":[]}}`))
 	}))
 	defer server.Close()
 
@@ -596,7 +592,7 @@ func TestShortDramaListThreadFileMessageWhenPageFull(t *testing.T) {
 		"short-drama", "+list-thread-file",
 		"--thread-id", "thread_123",
 		"--page-num", "2",
-		"--page-size", "1000",
+		"--page-size", "200",
 	})
 
 	if err := root.Execute(); err != nil {
@@ -604,7 +600,7 @@ func TestShortDramaListThreadFileMessageWhenPageFull(t *testing.T) {
 	}
 
 	got := decodeJSON(t, stdout.Bytes())
-	wantMessage := "<system-remind>\n- total reached 1000; query the next page with --page-num 3\n</system-remind>"
+	wantMessage := "<system-remind>\n- total reached 200; query the next page with --page-num 3\n</system-remind>"
 	if got["message"] != wantMessage {
 		t.Fatalf("message = %v, want next page hint", got["message"])
 	}
@@ -624,6 +620,24 @@ func TestShortDramaListThreadFileRequiresThreadID(t *testing.T) {
 	}
 }
 
+func TestShortDramaListThreadFileRejectsPageSizeAboveMax(t *testing.T) {
+	var stdout, stderr bytes.Buffer
+	root := NewRootCommand(&stdout, &stderr)
+	root.SetArgs([]string{
+		"short-drama", "+list-thread-file",
+		"--thread-id", "thread_123",
+		"--page-size", "201",
+	})
+
+	err := root.Execute()
+	if err == nil {
+		t.Fatal("Execute() error = nil, want validation error")
+	}
+	if !strings.Contains(err.Error(), "--page-size must be between 1 and 200") {
+		t.Fatalf("error = %q, want page-size validation", err)
+	}
+}
+
 func newTestRootCommand(t *testing.T, stdout, stderr io.Writer, baseURL string) *cobra.Command {
 	t.Helper()
 	return newTestRootCommandWithAccessKey(t, stdout, stderr, baseURL, "test-token")

package/internal/common/download_results.go

@@ -36,7 +36,6 @@ type DownloadResultResponse struct {
 	OutputPath   string                 `json:"output_path"`
 	Downloaded   []string               `json:"downloaded"`
 	AlreadyExist []string               `json:"already_exist,omitempty"`
-	Skipped      []string               `json:"skipped,omitempty"`
 	Errors       []*DownloadResultError `json:"errors,omitempty"`
 }
 
@@ -68,12 +67,6 @@ func DownloadResult(ctx context.Context, opts DownloadResultOptions, runner *Run
 	if outputPath == "" {
 		return nil, fmt.Errorf("output_path is required")
 	}
-	if shouldSkipDownload(outputPath) {
-		return &DownloadResultResponse{
-			OutputPath: outputPath,
-			Skipped:    []string{outputPath},
-		}, nil
-	}
 	// check if the output path exists and is a file
 	if info, err := os.Stat(outputPath); err == nil {
 		if info.IsDir() {
@@ -174,10 +167,6 @@ func DownloadResult(ctx context.Context, opts DownloadResultOptions, runner *Run
 	return result, nil
 }
 
-func shouldSkipDownload(outputPath string) bool {
-	return filepath.Base(outputPath) == "meta.json"
-}
-
 func downloadFileWithRetry(ctx context.Context, client Client, rawURL string, targetPath string) error {
 	var lastErr error
 	for attempt := 0; attempt <= maxDownloadRetries; attempt++ {

package/internal/common/list_thread_file.go

@@ -9,6 +9,8 @@ import (
 	"github.com/Pippit-dev/pippit-cli/internal/config"
 )
 
+const MaxListThreadFilePageSize = 200
+
 // ListThreadFileOptions is the command-facing request shape for listing thread files.
 type ListThreadFileOptions struct {
 	ThreadID string `json:"thread_id"`
@@ -95,8 +97,10 @@ func listThreadFileMessage(total int64, pageNum int) string {
 		pageNum = 1
 	}
 	var message string
-	if total >= 1000 {
-		message = fmt.Sprintf("total reached 1000; query the next page with --page-num %d", pageNum+1)
+	if total >= MaxListThreadFilePageSize {
+		message = fmt.Sprintf("total reached %d; query the next page with --page-num %d", MaxListThreadFilePageSize, pageNum+1)
+	} else {
+		message = fmt.Sprintf("total is below %d; continue querying with current --page-num %d", MaxListThreadFilePageSize, pageNum)
 	}
 	return fmt.Sprintf("%s\n- %s\n%s", start, message, end)
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pippit-dev/cli",
-  "version": "0.0.7",
+  "version": "0.0.9",
   "description": "Pippit CLI",
   "bin": {
     "pippit-cli": "scripts/run.js"
@@ -35,6 +35,7 @@
     "scripts/platform.js",
     "scripts/run.js",
     "scripts/skills.js",
+    "scripts/version-check.js",
     "checksums.txt",
     "README.md",
     "LICENSE"

package/scripts/run.js

@@ -3,6 +3,7 @@
 const { execFileSync } = require("child_process");
 const fs = require("fs");
 const path = require("path");
+const { maybeWarnNewVersion } = require("./version-check");
 
 const ext = process.platform === "win32" ? ".exe" : "";
 const bin = path.join(__dirname, "..", "bin", "pippit-cli" + ext);
@@ -39,6 +40,8 @@ if (process.platform === "win32" && fs.existsSync(oldBin)) {
 if (args[0] === "install") {
   require("./install-wizard.js");
 } else {
+  maybeWarnNewVersion(args);
+
   if (!fs.existsSync(bin)) {
     try {
       execFileSync(process.execPath, [path.join(__dirname, "install.js")], {

package/scripts/version-check.js

@@ -0,0 +1,96 @@
+const fs = require("fs");
+const os = require("os");
+const path = require("path");
+const { runSilent } = require("./platform");
+const { DEFAULT_PKG } = require("./skills");
+
+const CHECK_INTERVAL_MS = 24 * 60 * 60 * 1000;
+
+function defaultCacheFile() {
+  return path.join(os.homedir(), ".pippit-cli", "version-check.json");
+}
+
+function currentVersion() {
+  return require("../package.json").version.replace(/-.*$/, "");
+}
+
+function parseSemver(version) {
+  const match = String(version || "").trim().match(/^v?(\d+)\.(\d+)\.(\d+)/);
+  if (!match) return null;
+  return match.slice(1).map(Number);
+}
+
+function compareSemver(a, b) {
+  const parsedA = parseSemver(a);
+  const parsedB = parseSemver(b);
+  if (!parsedA || !parsedB) return 0;
+  for (let i = 0; i < 3; i++) {
+    const diff = parsedA[i] - parsedB[i];
+    if (diff !== 0) return diff;
+  }
+  return 0;
+}
+
+function readCache(cacheFile) {
+  try {
+    return JSON.parse(fs.readFileSync(cacheFile, "utf8"));
+  } catch (_) {
+    return null;
+  }
+}
+
+function writeCache(cacheFile, data) {
+  try {
+    fs.mkdirSync(path.dirname(cacheFile), { recursive: true });
+    fs.writeFileSync(cacheFile, JSON.stringify(data), "utf8");
+  } catch (_) {
+    // Version checks must never block normal CLI commands.
+  }
+}
+
+function fetchLatestVersion(pkg = DEFAULT_PKG) {
+  return runSilent("npm", ["view", pkg, "version"], { timeout: 3000 }).toString().trim();
+}
+
+function shouldSkip(args, env) {
+  const cmd = args[0];
+  return (
+    env.PIPPIT_CLI_DISABLE_UPDATE_CHECK === "1" ||
+    env.CI ||
+    cmd === "install" ||
+    cmd === "update"
+  );
+}
+
+function maybeWarnNewVersion(args = [], opts = {}) {
+  const env = opts.env || process.env;
+  if (shouldSkip(args, env)) return;
+
+  const now = opts.now || Date.now();
+  const cacheFile = opts.cacheFile || defaultCacheFile();
+  const cache = readCache(cacheFile);
+  const cacheFresh = cache && now - cache.checkedAt < CHECK_INTERVAL_MS;
+
+  let latest = cacheFresh ? cache.latest : "";
+  if (!latest) {
+    try {
+      latest = (opts.fetchLatestVersion || fetchLatestVersion)(opts.pkg || DEFAULT_PKG);
+      writeCache(cacheFile, { latest, checkedAt: now });
+    } catch (_) {
+      return;
+    }
+  }
+
+  const current = opts.currentVersion || currentVersion();
+  if (compareSemver(latest, current) <= 0) return;
+
+  const warn = opts.warn || console.error;
+  warn(`[pippit-cli] New version available: ${current} -> ${latest}. Run: pippit-cli update`);
+}
+
+module.exports = {
+  CHECK_INTERVAL_MS,
+  compareSemver,
+  maybeWarnNewVersion,
+  parseSemver,
+};

package/skills/short-drama/SKILL.md

@@ -75,7 +75,7 @@ pippit-cli short-drama +upload-file --path /path/to/outline.md
 
 ```bash
 # 获取会话文件列表
-pippit-cli short-drama +list-thread-file --thread-id THREAD_ID --page-num 1 --page-size 1000
+pippit-cli short-drama +list-thread-file --thread-id THREAD_ID --page-num 1 --page-size 200
 ```
 
 `+list-thread-file` 返回的每个文件对象包含：
@@ -108,7 +108,7 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PA
 2. 立即将 web_thread_link 展示给用户
 3. 并行发起，二者同等重要：
    a. pippit-cli short-drama +get-thread --thread-id THREAD_ID --run-id RUN_ID --after-seq SEQUENCE
-   b. pippit-cli short-drama +list-thread-file --thread-id THREAD_ID --page-num PAGE_NUM --page-size 1000
+   b. pippit-cli short-drama +list-thread-file --thread-id THREAD_ID --page-num PAGE_NUM --page-size 200
 4. 检查 `get-thread` 返回的 messages：
    - 如果任务仍在进行中：展示过程消息，继续查询
    - 如果后端 Agent 提出问题：展示问题，等待用户回复
@@ -117,7 +117,7 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PA
    - 将 file_path 作为本地目标文件路径，包含文件名
    - 有 download_url 的重要资产：加入本轮下载队列
    - 不判断 file_path 在本地是否已存在，是否跳过由 +download-result 内部处理
-   - 如果本轮 total 达到 1000：下一轮将 PAGE_NUM 加 1，继续查询新一页文件
+   - 如果本轮 total 达到 200：下一轮将 PAGE_NUM 加 1，继续查询新一页文件
 6. 对重要资产，立即调用 +download-result 并行下载资源：
    - 使用第 5 步获取的 download_url 作为 --url
    - 使用第 5 步获取的完整 file_path 作为 --output-path
@@ -150,7 +150,7 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PA
 - **间隔**：每 10 秒查询一次。
 - **增量拉取**：首次使用 `--after-seq 0`，后续根据已读消息进度调整 `after-seq`。
 - **并行查询**：每次 `+submit-run` 返回 `thread_id` 后，同时发起 `+get-thread` 和 `+list-thread-file`；二者同等重要，不能只查询会话进展而忽略会话文件。
-- **文件分页**：`+list-thread-file` 使用 `--page-size 1000`。如果本轮返回的 `total` 达到 1000，下一轮使用 `--page-num` 加 1 查询新一页结果；如果未达到 1000，保持当前页继续轮询新增产物。
+- **文件分页**：`+list-thread-file` 使用 `--page-size 200`。如果本轮返回的 `total` 达到 200，下一轮使用 `--page-num` 加 1 查询新一页结果；如果未达到 200，保持当前页继续轮询新增产物。
 - **重要资产识别**：每轮都检查 `+list-thread-file` 返回的文件。剧本设计、场景设计、场景图、人物角色设计、人物图、分集草稿、故事板、最终视频产物都是重要资产。
 - **文件下载**：解析 `+list-thread-file` 的结果后，对带 `download_url` 的重要资产立即调用 `+download-result` 下载资源；不要在 `list-thread-file` 阶段检查文件是否已存在，存在性检查由下载工具内部处理。
 - **下载完成标准**：不要把文件元信息展示当成下载完成；必须拿到本地 `file_path`，或明确记录该文件在重试后仍下载失败。
@@ -163,7 +163,7 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PA
 一次短剧任务不能只以 `+get-thread` 返回完成消息作为结束条件。完成前必须同时检查：
 
 1. 已处理 `+get-thread` 返回的最新会话进展、用户确认问题和最终消息。
-2. 已用 `--page-size 1000` 调用 `+list-thread-file` 获取会话文件列表；如果本轮 `total` 达到 1000，已在后续轮询中递增 `page-num` 查询新一页。
+2. 已用 `--page-size 200` 调用 `+list-thread-file` 获取会话文件列表；如果本轮 `total` 达到 200，已在后续轮询中递增 `page-num` 查询新一页。
 3. 对所有带 `download_url` 的重要资产，已调用 `+download-result` 下载到本地 `file_path`。
 4. 对查询失败或下载失败的资产，已在后续轮询中主动重试，并在最终回复中列出仍失败的文件或命令。
 
@@ -224,11 +224,11 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PA
     }
   ],
   "total": 1,
-  "message": "<system-remind>\n- total reached 1000; query the next page with --page-num {page-num} + 1\n</system-remind>"
+  "message": "<system-remind>\n- total reached 200; query the next page with --page-num {page-num} + 1\n</system-remind>"
 }
 ```
 
-当 `total` 达到 1000 时，`message` 会用 `<system-remind>` 提示下一轮将 `page-num` 加 1 查询新一页。
+当 `total` 达到 200 时，`message` 会用 `<system-remind>` 提示下一轮将 `page-num` 加 1 查询新一页。
 
 **+download-result** 返回：
 
@@ -251,9 +251,9 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PA
 1. 有download_url的重要资产
    → 记录该file_path和URL
    → 使用 +download-result 将URL资源下载到该file_path
-2. 本轮total达到1000
+2. 本轮total达到200
    → 下一轮page-num加1，继续查询新一页结果
-3. 本轮total未达到1000
+3. 本轮total未达到200
    → 后续轮询保持当前页，继续发现新增产物
 4. list-thread-file或download-result失败
    → 记录失败参数和错误
@@ -302,6 +302,6 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PA
 - 查询进展时优先使用 `+submit-run` 返回的 `thread_id` 和 `run_id`。
 - `--after-seq` 用于增量拉取消息，首次查询可设置为 `0`。
 - `+upload-file` 当前用于短剧场景文件上传链路，上传后将返回可传给 `+submit-run` 的文件 ID。
-- `+list-thread-file` 只需要 `thread_id`；分页参数使用 `--page-num 1 --page-size 1000` 起步，`total` 达到 1000 时下一轮递增 `page-num`。
+- `+list-thread-file` 只需要 `thread_id`；分页参数使用 `--page-num 1 --page-size 200` 起步，`total` 达到 200 时下一轮递增 `page-num`。
 - `+list-thread-file` 和 `+download-result` 是两个不同的 CLI 指令：前者获取会话文件元信息，后者下载 URL 资源并写入到本地目标文件路径。
 - `+download-result` 接收 `--url`、`--output-path`、`--workers`；`--output-path` 必须是包含文件名的目标文件路径。