@pippit-dev/cli 0.0.11 → 0.0.13

package/README.md

@@ -1,4 +1,4 @@
-# pippit-cli
+# pippit-tool-cli
 
 Minimal demo CLI for Pippit workflows.
 
@@ -10,11 +10,11 @@ matching prebuilt binary for macOS, Linux, or Windows:
 ```bash
 npx @pippit-dev/cli@latest install
 export XYQ_ACCESS_KEY="<access-key>"
-pippit-cli --version
-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-path ./thread_123/results/result.mp4 --url URL
+pippit-tool-cli --version
+pippit-tool-cli short-drama +submit-run --message "写一个赛博朋克短剧开头"
+pippit-tool-cli short-drama +upload-file --path ./reference.doc
+pippit-tool-cli short-drama +get-thread --thread-id thread_123 --run-id run_456 --after-seq 0
+pippit-tool-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 @@ Submit a Run task for the short drama scene:
 export XYQ_ACCESS_KEY="<access-key>"
 go run . --version
 go run . short-drama +submit-run --message "写一个赛博朋克短剧开头"
-go run . short-drama +upload-file --path ./story.md
+go run . short-drama +upload-file --path ./reference.doc
 go run . short-drama +get-thread --thread-id thread_123 --run-id run_456 --after-seq 0
 go run . short-drama +download-result --output-path ./thread_123/results/result.mp4 --url URL
 ```
@@ -35,9 +35,11 @@ go run . short-drama +download-result --output-path ./thread_123/results/result.
 `run_id`, and `web_thread_link`; `--message` is required.
 `+get-thread` calls
 `/api/biz/v1/skill/get_thread` and prints extracted `messages`.
+`+upload-file` calls `/api/biz/v1/skill/upload_file` with
+`multipart/form-data` and prints the returned `asset_id`.
+Only `.doc` and `.txt` file extensions are supported.
 `+download-result` downloads the result URL to
-the `--output-path` file path. `+upload-file` is still mocked while
-its real service contract is wired.
+the `--output-path` file path.
 
 ## HTTP Client
 
@@ -47,6 +49,6 @@ settings such as base URL, HTTP timeout, and API paths are loaded by
 
 ## Auth
 
-`short-drama +submit-run` and `short-drama +get-thread` authenticate with
+`short-drama +submit-run`, `short-drama +get-thread`, and `short-drama +upload-file` authenticate with
 `Authorization: Bearer <XYQ_ACCESS_KEY>`. OAuth command code remains available,
 but runtime short drama requests do not use it.

package/checksums.txt

@@ -1,6 +1,6 @@
-ebff380937bb2269a116272d4bb0c5aaef0a176c2ffcfb6f337733411db34ddc  pippit-cli-0.0.11-darwin-amd64.tar.gz
-de27b844d0df2b112a2a78a24184d023cdf72b145d969497dd351924348161e4  pippit-cli-0.0.11-darwin-arm64.tar.gz
-65da00716f772d3bbb823e10f36e87b73810c8d6f76d5f0c648274d43657f634  pippit-cli-0.0.11-linux-amd64.tar.gz
-420dc63f6ea642c20b0a22f666c652d633d4ddad274b85096b51f74602e85a81  pippit-cli-0.0.11-linux-arm64.tar.gz
-06350f1d74f4130bb420c237e5f804b4df1f358dfc195b659a70ddfd2d48832b  pippit-cli-0.0.11-windows-amd64.zip
-6919d1de3bec32499379bbcc820ccf3bafb3e843fdfa4cb9e826f90fae00feb2  pippit-cli-0.0.11-windows-arm64.zip
+c02c37229ea76195c04881880a9d5a913ab95846958ab4e4ab41bc2b82649e49  pippit-tool-cli-0.0.13-darwin-amd64.tar.gz
+4937231063eeaad1e494be4f2b6218860313abbf529d868568d9d6e48bf62e3e  pippit-tool-cli-0.0.13-darwin-arm64.tar.gz
+da2bb0c5472bdd02a2cc861a17564254abb86fa5c63c44b58c7fbb64211550f1  pippit-tool-cli-0.0.13-linux-amd64.tar.gz
+476e0907a6983f9360324732411edcd9c2c80f9d903a7974949ef2d09b1f35ba  pippit-tool-cli-0.0.13-linux-arm64.tar.gz
+cfe8e5b92b8b6abbf5ed3abc9ce31c1371efcc34698edfa5ee2d8d675d4f7886  pippit-tool-cli-0.0.13-windows-amd64.zip
+edff7af4414b2b080338f96eab5ce9cca2a14d4bd8fc3099a1af7a2dc43109d2  pippit-tool-cli-0.0.13-windows-arm64.zip

package/cmd/root.go

@@ -13,7 +13,7 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// Execute runs the pippit-cli command tree.
+// Execute runs the pippit-tool-cli command tree.
 func Execute() error {
 	return NewRootCommand(os.Stdout, os.Stderr).Execute()
 }
@@ -27,7 +27,7 @@ func NewRootCommand(stdout, stderr io.Writer) *cobra.Command {
 
 func newRootCommand(stdout, stderr io.Writer, runner *common.Runner) *cobra.Command {
 	root := &cobra.Command{
-		Use:           "pippit-cli",
+		Use:           "pippit-tool-cli",
 		Short:         "Pippit CLI",
 		Long:          "Pippit CLI submits short-drama workflows, downloads generated assets, and updates the installed CLI package.",
 		Version:       version.Current(),

package/cmd/short_drama/short_drama.go

@@ -3,7 +3,6 @@ package short_drama
 import (
 	"fmt"
 	"io"
-	"path/filepath"
 	"strings"
 
 	"github.com/Pippit-dev/pippit-cli/internal/common"
@@ -72,8 +71,6 @@ func newShortDramaUploadFileCommand(stdout, stderr io.Writer, runner *common.Run
 			if opts.Path == "" {
 				return fmt.Errorf("--path is required")
 			}
-			opts.FileName = filepath.Base(opts.Path)
-			opts.Mock = true
 
 			result, err := common.UploadFile(cmd.Context(), opts, runner)
 			if err != nil {

package/cmd/short_drama_test.go

@@ -176,23 +176,65 @@ func TestShortDramaSubmitRunRequiresAccessKey(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)
+		}
+		if r.URL.Path != "/api/biz/v1/skill/upload_file" {
+			t.Fatalf("path = %s, want upload_file path", r.URL.Path)
+		}
+		if r.Header.Get("Authorization") != "Bearer test-token" {
+			t.Fatalf("Authorization = %q, want test bearer token", r.Header.Get("Authorization"))
+		}
+		if err := r.ParseMultipartForm(1 << 20); err != nil {
+			t.Fatalf("ParseMultipartForm(): %v", err)
+		}
+		if got := r.FormValue("accessKey"); got != "" {
+			t.Fatalf("accessKey = %q, want empty (auth via header only)", got)
+		}
+		files := r.MultipartForm.File["file"]
+		if len(files) != 1 {
+			t.Fatalf("file parts = %d, want 1", len(files))
+		}
+		if files[0].Filename != "story.txt" {
+			t.Fatalf("filename = %q, want story.txt", files[0].Filename)
+		}
+		if got := files[0].Header.Get("Content-Type"); got != "text/plain; charset=utf-8" {
+			t.Fatalf("Content-Type = %q, want text/plain; charset=utf-8", got)
+		}
+		file, err := files[0].Open()
+		if err != nil {
+			t.Fatalf("Open multipart file: %v", err)
+		}
+		defer file.Close()
+		data, err := io.ReadAll(file)
+		if err != nil {
+			t.Fatalf("ReadAll multipart file: %v", err)
+		}
+		if string(data) != "txt-data" {
+			t.Fatalf("file content = %q, want txt-data", string(data))
+		}
+		_, _ = w.Write([]byte(`{"ret":"0","errmsg":"","data":{"pippit_asset_id":"asset_123"}}`))
+	}))
+	defer server.Close()
+
+	cwd := chdirTemp(t)
+	path := filepath.Join(cwd, "story.txt")
+	if err := os.WriteFile(path, []byte("txt-data"), 0o644); err != nil {
+		t.Fatalf("WriteFile(): %v", err)
+	}
+
 	var stdout, stderr bytes.Buffer
-	root := NewRootCommand(&stdout, &stderr)
-	root.SetArgs([]string{"short-drama", "+upload-file", "--path", "/tmp/story.md"})
+	root := newTestRootCommand(t, &stdout, &stderr, server.URL)
+	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())
 	}
 
 	got := decodeJSON(t, stdout.Bytes())
-	if got["scene"] != "short-drama" {
-		t.Fatalf("scene = %v, want short-drama", got["scene"])
-	}
-	if got["status"] != "uploaded" {
-		t.Fatalf("status = %v, want uploaded", got["status"])
-	}
-	if !strings.HasPrefix(got["file_id"].(string), "file_mock_") {
-		t.Fatalf("file_id = %v, want mock file id", got["file_id"])
+	if got["asset_id"] != "asset_123" {
+		t.Fatalf("asset_id = %v, want asset_123", got["asset_id"])
 	}
 }
 
@@ -210,6 +252,51 @@ func TestShortDramaUploadFileRequiresPath(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")
+	}))
+	defer server.Close()
+
+	cwd := chdirTemp(t)
+	path := filepath.Join(cwd, "story.txt")
+	if err := os.WriteFile(path, []byte("txt-data"), 0o644); err != nil {
+		t.Fatalf("WriteFile(): %v", err)
+	}
+
+	var stdout, stderr bytes.Buffer
+	root := newTestRootCommandWithAccessKey(t, &stdout, &stderr, server.URL, "")
+	root.SetArgs([]string{"short-drama", "+upload-file", "--path", path})
+
+	err := root.Execute()
+	if err == nil {
+		t.Fatal("Execute() error = nil, want access key error")
+	}
+	if !strings.Contains(err.Error(), "XYQ_ACCESS_KEY is required") {
+		t.Fatalf("error = %q, want access key guidance", err)
+	}
+}
+
+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 {
+		t.Fatalf("WriteFile(): %v", err)
+	}
+
+	var stdout, stderr bytes.Buffer
+	root := NewRootCommand(&stdout, &stderr)
+	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 and .txt uploads are supported") {
+		t.Fatalf("error = %q, want unsupported type validation", err)
+	}
+}
+
 func TestShortDramaDownloadResult(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" {

package/cmd/update/update.go

@@ -18,7 +18,7 @@ const defaultPackage = "@pippit-dev/cli"
 func NewCommand(stdout, stderr io.Writer) *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "update",
-		Short: "Update pippit-cli and bundled skills",
+		Short: "Update pippit-tool-cli and bundled skills",
 		RunE: func(cmd *cobra.Command, _ []string) error {
 			return runUpdate(stdout, stderr)
 		},
@@ -32,14 +32,14 @@ func runUpdate(stdout, stderr io.Writer) error {
 		pkg = defaultPackage + "@latest"
 	}
 
-	fmt.Fprintf(stderr, "Updating pippit-cli via npm: %s\n", pkg)
+	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)
 	}
 	if err := runInheritEnv(stderr, []string{"PIPPIT_CLI_SKIP_SKILLS=1"}, "npm", "install", "-g", pkg); err != nil {
 		restore()
-		return fmt.Errorf("update pippit-cli: %w", err)
+		return fmt.Errorf("update pippit-tool-cli: %w", err)
 	}
 
 	root, err := globalPackageRoot(defaultPackage)
@@ -47,12 +47,12 @@ func runUpdate(stdout, stderr io.Writer) error {
 		return fmt.Errorf("locate global package: %w", err)
 	}
 
-	fmt.Fprintln(stderr, "Updating pippit-cli skills...")
+	fmt.Fprintln(stderr, "Updating pippit-tool-cli skills...")
 	if err := installSkills(root, stderr); err != nil {
-		return fmt.Errorf("update pippit-cli skills: %w", err)
+		return fmt.Errorf("update pippit-tool-cli skills: %w", err)
 	}
 
-	fmt.Fprintln(stdout, "pippit-cli and skills updated")
+	fmt.Fprintln(stdout, "pippit-tool-cli and skills updated")
 	return nil
 }
 
@@ -107,7 +107,7 @@ func prepareSelfReplace() (func(), error) {
 	if err != nil {
 		return nil, err
 	}
-	if filepath.Base(exe) != "pippit-cli.exe" || !strings.Contains(exe, "node_modules") {
+	if filepath.Base(exe) != "pippit-tool-cli.exe" || !strings.Contains(exe, "node_modules") {
 		return func() {}, nil
 	}
 	old := exe + ".old"

package/internal/common/client.go

@@ -5,8 +5,12 @@ import (
 	"context"
 	"fmt"
 	"io"
+	"mime/multipart"
 	"net/http"
+	"net/textproto"
 	"net/url"
+	"os"
+	"path/filepath"
 	"reflect"
 	"strings"
 	"time"
@@ -16,12 +20,20 @@ import (
 
 type Client interface {
 	SendRequest(ctx context.Context, path string, body any, out any) error
+	SendMultipartRequest(ctx context.Context, path string, fields map[string]string, file MultipartFile, out any) error
 }
 
 type RequestAuthorizer interface {
 	Inject(ctx context.Context, req *http.Request) error
 }
 
+type MultipartFile struct {
+	FieldName   string
+	Path        string
+	FileName    string
+	ContentType string
+}
+
 type httpClient struct {
 	baseURL    string
 	httpClient *http.Client
@@ -100,6 +112,89 @@ func (c *httpClient) SendRequest(ctx context.Context, path string, body any, out
 	return c.do(req, out)
 }
 
+func (c *httpClient) SendMultipartRequest(ctx context.Context, path string, fields map[string]string, file MultipartFile, out any) error {
+	reqURL, err := c.resolveURL(path, nil)
+	if err != nil {
+		return err
+	}
+	if file.FieldName == "" {
+		return fmt.Errorf("multipart file field name is required")
+	}
+	if file.FileName == "" {
+		file.FileName = filepath.Base(file.Path)
+	}
+	if file.ContentType == "" {
+		file.ContentType = "application/octet-stream"
+	}
+
+	pr, pw := io.Pipe()
+	writer := multipart.NewWriter(pw)
+	req, err := http.NewRequestWithContext(ctx, http.MethodPost, reqURL, pr)
+	if err != nil {
+		_ = pr.Close()
+		_ = pw.Close()
+		return fmt.Errorf("build POST request: %w", err)
+	}
+	req.Header.Set("Content-Type", writer.FormDataContentType())
+	req.Header.Set("Accept", "application/json")
+
+	if c.authorizer == nil {
+		_ = pr.Close()
+		_ = pw.Close()
+		return fmt.Errorf("authorized request requires authorizer")
+	}
+	if err := c.authorizer.Inject(ctx, req); err != nil {
+		_ = pr.Close()
+		_ = pw.Close()
+		return fmt.Errorf("inject auth headers: %w", err)
+	}
+	c.injectHeaders(req)
+
+	go func() {
+		err := writeMultipartBody(writer, fields, file)
+		closeErr := writer.Close()
+		if err != nil {
+			_ = pw.CloseWithError(err)
+			return
+		}
+		_ = pw.CloseWithError(closeErr)
+	}()
+
+	return c.do(req, out)
+}
+
+func writeMultipartBody(writer *multipart.Writer, fields map[string]string, file MultipartFile) error {
+	for key, value := range fields {
+		if err := writer.WriteField(key, value); err != nil {
+			return err
+		}
+	}
+
+	f, err := os.Open(file.Path)
+	if err != nil {
+		return fmt.Errorf("open upload file: %w", err)
+	}
+	defer f.Close()
+
+	header := make(textproto.MIMEHeader)
+	header.Set("Content-Disposition", fmt.Sprintf(`form-data; name="%s"; filename="%s"`, escapeQuotes(file.FieldName), escapeQuotes(file.FileName)))
+	header.Set("Content-Type", file.ContentType)
+	part, err := writer.CreatePart(header)
+	if err != nil {
+		return err
+	}
+	if _, err := io.Copy(part, f); err != nil {
+		return fmt.Errorf("write upload file: %w", err)
+	}
+	return nil
+}
+
+var quotesReplacer = strings.NewReplacer("\\", "\\\\", `"`, `\"`)
+
+func escapeQuotes(s string) string {
+	return quotesReplacer.Replace(s)
+}
+
 func (c *httpClient) injectHeaders(req *http.Request) {
 	for k, values := range c.headers {
 		for _, v := range values {

package/internal/common/download_results.go

@@ -31,7 +31,7 @@ type DownloadResultError struct {
 	Error string `json:"error"`
 }
 
-// DownloadResultResponse is the JSON envelope printed by `pippit-cli short-drama +download-result`.
+// DownloadResultResponse is the JSON envelope printed by `pippit-tool-cli short-drama +download-result`.
 type DownloadResultResponse struct {
 	OutputPath   string                 `json:"output_path"`
 	Downloaded   []string               `json:"downloaded"`

package/internal/common/list_thread_file.go

@@ -24,7 +24,7 @@ type ThreadFile struct {
 	DownloadURL string `json:"download_url"`
 }
 
-// ListThreadFileResult is the JSON envelope printed by `pippit-cli short-drama +list-thread-file`.
+// ListThreadFileResult is the JSON envelope printed by `pippit-tool-cli short-drama +list-thread-file`.
 type ListThreadFileResult struct {
 	Files   []*ThreadFile `json:"files"`
 	Total   int64         `json:"total"`

package/internal/common/upload_file.go

@@ -2,48 +2,103 @@ package common
 
 import (
 	"context"
-	"crypto/sha1"
-	"encoding/hex"
-	"time"
+	"fmt"
+	"mime"
+	"os"
+	"path/filepath"
+	"strings"
+
+	"github.com/Pippit-dev/pippit-cli/internal/config"
 )
 
 // UploadFileOptions is the stable command-facing request shape for file upload.
 type UploadFileOptions struct {
 	Path     string `json:"path"`
 	FileName string `json:"file_name"`
-	Purpose  string `json:"purpose"`
-	Mock     bool   `json:"mock"`
 }
 
-// UploadFileResult is the JSON envelope printed by `pippit-cli short-drama +upload-file`.
+// UploadFileResult is the JSON envelope printed by `pippit-tool-cli short-drama +upload-file`.
 type UploadFileResult struct {
-	Scene    string            `json:"scene"`
-	FileID   string            `json:"file_id"`
-	Status   string            `json:"status"`
-	Uploaded string            `json:"uploaded_at"`
-	Request  UploadFileOptions `json:"request"`
+	AssetID string `json:"asset_id"`
+}
+
+type uploadFileResponse struct {
+	Ret     string `json:"ret"`
+	Errmsg  string `json:"errmsg"`
+	SvrTime int64  `json:"svr_time"`
+	LogID   string `json:"log_id"`
+	Data    struct {
+		PippitAssetID string `json:"pippit_asset_id"`
+		AssetID       string `json:"asset_id"`
+	} `json:"data"`
+}
+
+const uploadFileFieldName = "file"
+
+var allowedUploadExtensions = map[string]bool{
+	".doc": true,
+	".txt": true,
 }
 
-func UploadFile(ctx context.Context, opts UploadFileOptions, _ *Runner) (*UploadFileResult, error) {
+func UploadFile(ctx context.Context, opts UploadFileOptions, runner *Runner) (*UploadFileResult, error) {
 	if err := ctx.Err(); err != nil {
 		return nil, err
 	}
-	key := opts.Purpose + "\x00" + opts.FileName + "\x00" + opts.Path
-	id := stableID(key, 10)
+	if runner == nil || runner.Client == nil {
+		return nil, fmt.Errorf("upload_file runner client is required")
+	}
+
+	path := strings.TrimSpace(opts.Path)
+	if path == "" {
+		return nil, fmt.Errorf("upload file path is required")
+	}
+	info, err := os.Stat(path)
+	if err != nil {
+		return nil, fmt.Errorf("stat upload file: %w", err)
+	}
+	if info.IsDir() {
+		return nil, fmt.Errorf("upload path %q is a directory", path)
+	}
+
+	ext := strings.ToLower(filepath.Ext(path))
+	if !allowedUploadExtensions[ext] {
+		return nil, fmt.Errorf("unsupported file extension %q; only .doc and .txt uploads are supported", ext)
+	}
+	fileName := filepath.Base(path)
+	contentType := mime.TypeByExtension(ext)
+
+	var resp uploadFileResponse
+	if err := runner.Client.SendMultipartRequest(ctx, uploadFilePath(runner), nil, MultipartFile{
+		FieldName:   uploadFileFieldName,
+		Path:        path,
+		FileName:    fileName,
+		ContentType: contentType,
+	}, &resp); err != nil {
+		return nil, fmt.Errorf("upload_file request failed: %w", err)
+	}
+	if resp.Ret != "0" {
+		if resp.Errmsg == "" {
+			resp.Errmsg = "unknown error"
+		}
+		return nil, fmt.Errorf("upload_file failed: ret=%s errmsg=%s", resp.Ret, resp.Errmsg)
+	}
+
+	assetID := strings.TrimSpace(resp.Data.PippitAssetID)
+	if assetID == "" {
+		assetID = strings.TrimSpace(resp.Data.AssetID)
+	}
+	if assetID == "" {
+		return nil, fmt.Errorf("upload_file response missing pippit_asset_id")
+	}
+
 	return &UploadFileResult{
-		Scene:    "short-drama",
-		FileID:   "file_mock_" + id,
-		Status:   "uploaded",
-		Uploaded: time.Now().UTC().Format(time.RFC3339),
-		Request:  opts,
+		AssetID: assetID,
 	}, nil
 }
 
-func stableID(key string, n int) string {
-	sum := sha1.Sum([]byte(key))
-	id := hex.EncodeToString(sum[:])
-	if n > len(id) {
-		return id
+func uploadFilePath(runner *Runner) string {
+	if runner != nil && runner.Config != nil && runner.Config.Paths != nil && runner.Config.Paths.UploadFile != "" {
+		return runner.Config.Paths.UploadFile
 	}
-	return id[:n]
+	return config.UploadFilePath
 }

package/internal/short_drama/get_thread.go

@@ -22,7 +22,7 @@ type ThreadEntry struct {
 	Content []any  `json:"content"`
 }
 
-// GetThreadResult is the JSON envelope printed by `pippit-cli short-drama +get-thread`.
+// GetThreadResult is the JSON envelope printed by `pippit-tool-cli short-drama +get-thread`.
 type GetThreadResult struct {
 	Messages []*ThreadEntry `json:"messages"`
 }

package/internal/short_drama/submit_run.go

@@ -15,7 +15,7 @@ type SubmitRunOptions struct {
 	AssetIDs []string `json:"asset_ids,omitempty"`
 }
 
-// SubmitRunResult is the JSON envelope printed by `pippit-cli short-drama +submit-run`.
+// SubmitRunResult is the JSON envelope printed by `pippit-tool-cli short-drama +submit-run`.
 type SubmitRunResult struct {
 	ThreadID      string `json:"thread_id"`
 	RunID         string `json:"run_id"`

package/package.json

@@ -1,9 +1,9 @@
 {
   "name": "@pippit-dev/cli",
-  "version": "0.0.11",
+  "version": "0.0.13",
   "description": "Pippit CLI",
   "bin": {
-    "pippit-cli": "scripts/run.js"
+    "pippit-tool-cli": "scripts/run.js"
   },
   "scripts": {
     "postinstall": "node scripts/install.js",

package/scripts/install-wizard.js

@@ -17,12 +17,12 @@ function getGloballyInstalledVersion() {
   }
 }
 
-function whichPippitCli() {
+function whichPippitToolCli() {
   try {
     const prefix = runSilent("npm", ["prefix", "-g"], { timeout: 15000 }).toString().trim();
     const bin = isWindows
-      ? path.join(prefix, "pippit-cli.cmd")
-      : path.join(prefix, "bin", "pippit-cli");
+      ? path.join(prefix, "pippit-tool-cli.cmd")
+      : path.join(prefix, "bin", "pippit-tool-cli");
     if (fs.existsSync(bin)) return bin;
   } catch (_) {
     // Fall back to PATH lookup.
@@ -30,7 +30,7 @@ function whichPippitCli() {
 
   try {
     const cmd = isWindows ? "where" : "which";
-    return runSilent(cmd, ["pippit-cli"]).toString().split("\n")[0].trim();
+    return runSilent(cmd, ["pippit-tool-cli"]).toString().split("\n")[0].trim();
   } catch (_) {
     return null;
   }
@@ -39,7 +39,7 @@ function whichPippitCli() {
 function main() {
   const installed = getGloballyInstalledVersion();
   if (installed) {
-    console.log(`Updating global pippit-cli (${installed}) via ${PKG}...`);
+    console.log(`Updating global pippit-tool-cli (${installed}) via ${PKG}...`);
   } else {
     console.log(`Installing ${PKG} globally...`);
   }
@@ -48,7 +48,7 @@ function main() {
     env: { ...process.env, PIPPIT_CLI_SKIP_SKILLS: "1" },
   });
 
-  console.log("Installing pippit-cli skills...");
+  console.log("Installing pippit-tool-cli skills...");
   try {
     installGlobalPackageSkills(DEFAULT_PKG);
   } catch (err) {
@@ -60,15 +60,15 @@ function main() {
     installGlobalPackageSkills(DEFAULT_PKG);
   }
 
-  const bin = whichPippitCli();
+  const bin = whichPippitToolCli();
   if (!bin) {
-    console.error("pippit-cli was installed, but no global command was found in npm prefix.");
+    console.error("pippit-tool-cli was installed, but no global command was found in npm prefix.");
     console.error("Check that npm's global bin directory is in PATH.");
     process.exit(1);
   }
 
-  console.log(`pippit-cli is ready: ${bin}`);
-  console.log("Try: pippit-cli short-drama +submit-run --message \"写一个短剧开头\"");
+  console.log(`pippit-tool-cli is ready: ${bin}`);
+  console.log("Try: pippit-tool-cli short-drama +submit-run --message \"写一个短剧开头\"");
 }
 
 main();

package/scripts/install.js

@@ -9,7 +9,7 @@ const { installSkillsFromRoot } = require("./skills");
 
 const VERSION = require("../package.json").version.replace(/-.*$/, "");
 const REPO = "Pippit-dev/cli";
-const NAME = "pippit-cli";
+const NAME = "pippit-tool-cli";
 const ROOT = path.join(__dirname, "..");
 const BIN_DIR = path.join(ROOT, "bin");
 
@@ -121,7 +121,7 @@ function install() {
 
   fs.mkdirSync(BIN_DIR, { recursive: true });
 
-  const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "pippit-cli-"));
+  const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "pippit-tool-cli-"));
   const archivePath = path.join(tmpDir, archiveName);
   try {
     download(releaseURL, archivePath);

package/scripts/run.js

@@ -6,7 +6,7 @@ const path = require("path");
 const { maybeWarnNewVersion } = require("./version-check");
 
 const ext = process.platform === "win32" ? ".exe" : "";
-const bin = path.join(__dirname, "..", "bin", "pippit-cli" + ext);
+const bin = path.join(__dirname, "..", "bin", "pippit-tool-cli" + ext);
 const args = process.argv.slice(2);
 
 const oldBin = bin + ".old";
@@ -50,7 +50,7 @@ if (args[0] === "install") {
       });
     } catch (_) {
       console.error(
-        "\nFailed to prepare pippit-cli binary.\n" +
+        "\nFailed to prepare pippit-tool-cli binary.\n" +
         "Make sure Go is installed and available in PATH, then retry.\n"
       );
       process.exit(1);

package/scripts/version-check.js

@@ -85,7 +85,7 @@ function maybeWarnNewVersion(args = [], opts = {}) {
   if (compareSemver(latest, current) <= 0) return;
 
   const warn = opts.warn || console.error;
-  warn(`[pippit-cli] New version available: ${current} -> ${latest}. Run: pippit-cli update`);
+  warn(`[pippit-tool-cli] New version available: ${current} -> ${latest}. Run: pippit-tool-cli update`);
 }
 
 module.exports = {

package/skills/short-drama/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: pippit-short-drama-skill
-description: 使用 pippit-cli 的短剧场景能力提交和查询短剧创作任务。覆盖短剧生成、续写、改写、剧情扩展、人物设定、分集草稿、世界观设定、会话文件获取、文件资源下载等创作场景。当用户要求创作短剧、写短剧剧本、续写故事、修改剧情、补充角色设定、查询短剧任务进展、获取短剧会话文件或下载短剧文件资源，或提到 pippit-cli short-drama / 小云雀短剧时触发。
+description: 使用 pippit-tool-cli 的短剧场景能力提交和查询短剧创作任务。覆盖短剧生成、续写、改写、剧情扩展、人物设定、分集草稿、世界观设定、会话文件获取、文件资源下载等创作场景。当用户要求创作短剧、写短剧剧本、续写故事、修改剧情、补充角色设定、查询短剧任务进展、获取短剧会话文件或下载短剧文件资源，或提到 pippit-tool-cli short-drama / 小云雀短剧时触发。
 user-invocable: true
 metadata:
   {
@@ -9,7 +9,7 @@ metadata:
         "emoji": "📖",
         "requires":
           {
-            "bins": ["pippit-cli"]
+            "bins": ["pippit-tool-cli"]
           }
       }
   }
@@ -17,7 +17,7 @@ metadata:
 
 # 小云雀短剧创作
 
-通过 `pippit-cli short-drama` 命令提交短剧创作任务、上传参考文件，并行查询任务进展和会话产物文件，及时把重要资产下载到用户本地。
+通过 `pippit-tool-cli short-drama` 命令提交短剧创作任务、上传参考文件，并行查询任务进展和会话产物文件，及时把重要资产下载到用户本地。
 
 短剧场景面向剧情、人物、分集与画面化叙事创作，用户的原始需求通过 `--message` 发送给后端 Agent。后端 Agent 负责理解任务、编排流程和生成内容；用户侧 Agent 负责提交任务、并行查询进展与产物、主动下载重要资产并展示结果。
 
@@ -33,7 +33,7 @@ metadata:
 
 ## 前置要求
 
-需要已安装 `pippit-cli`：
+需要已安装 `pippit-tool-cli`：
 
 ```bash
 npx @pippit-dev/cli@latest install
@@ -45,20 +45,20 @@ npx @pippit-dev/cli@latest install
 
 ```bash
 # 创建新会话并提交短剧创作需求
-pippit-cli short-drama +submit-run --message "创作一个赛博朋克短剧开头"
+pippit-tool-cli short-drama +submit-run --message "创作一个赛博朋克短剧开头"
 
 # 向已有会话追加新的短剧需求
-pippit-cli short-drama +submit-run --message "继续写下一集，重点描写主角的逃亡" --thread-id THREAD_ID
+pippit-tool-cli short-drama +submit-run --message "继续写下一集，重点描写主角的逃亡" --thread-id THREAD_ID
 
 # 携带已上传文件 ID 提交任务
-pippit-cli short-drama +submit-run --message "参考这个大纲写第一集" --asset-ids ASSET_ID
+pippit-tool-cli short-drama +submit-run --message "参考这个大纲写第一集" --asset-ids ASSET_ID
 ```
 
 ### 2. 查询短剧任务进展
 
 ```bash
 # 查询会话消息列表
-pippit-cli short-drama +get-thread --thread-id THREAD_ID --run-id RUN_ID --after-seq 0
+pippit-tool-cli short-drama +get-thread --thread-id THREAD_ID --run-id RUN_ID --after-seq 0
 ```
 
 > `thread_id` 和 `run_id` 由 `+submit-run` 返回。`after-seq` 用于增量拉取消息，首次查询可使用 `0`。
@@ -68,14 +68,14 @@ pippit-cli short-drama +get-thread --thread-id THREAD_ID --run-id RUN_ID --after
 当用户提供短剧大纲、人物设定、世界观设定、已有分集或剧本等本地文件路径时，可先上传文件。
 
 ```bash
-pippit-cli short-drama +upload-file --path /path/to/outline.md
+pippit-tool-cli short-drama +upload-file --path /path/to/outline.md
 ```
 
 ### 4. 获取会话文件
 
 ```bash
 # 获取会话文件列表
-pippit-cli short-drama +list-thread-file --thread-id THREAD_ID --page-num 1 --page-size 200
+pippit-tool-cli short-drama +list-thread-file --thread-id THREAD_ID --page-num 1 --page-size 200
 ```
 
 `+list-thread-file` 返回的每个文件对象包含：
@@ -93,7 +93,7 @@ pippit-cli short-drama +list-thread-file --thread-id THREAD_ID --page-num 1 --pa
 
 ```bash
 # 下载文件资源到指定文件路径
-pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PATH
+pippit-tool-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PATH
 ```
 
 `FILE_PATH` 必须直接使用 `+list-thread-file` 返回的完整 `file_path`，包含文件名，不要取父目录。`+download-result` 负责把会话产生的文件通过 URL 下载到该目标文件路径；如果目标文件已存在，跳过下载。
@@ -103,12 +103,12 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PA
 ### 场景 1：用户要求生成短剧内容
 
 ```
-1. pippit-cli short-drama +submit-run --message "用户的原始短剧需求"
+1. pippit-tool-cli short-drama +submit-run --message "用户的原始短剧需求"
    → 拿到 thread_id、run_id 和 web_thread_link
 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 200
+   a. pippit-tool-cli short-drama +get-thread --thread-id THREAD_ID --run-id RUN_ID --after-seq SEQUENCE
+   b. pippit-tool-cli short-drama +list-thread-file --thread-id THREAD_ID --page-num PAGE_NUM --page-size 200
 4. 检查 `get-thread` 返回的 messages：
    - 如果任务仍在进行中：展示过程消息，继续查询
    - 如果后端 Agent 提出问题：展示问题，等待用户回复
@@ -130,9 +130,9 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PA
 ### 场景 2：用户提供参考文件要求创作
 
 ```
-1. pippit-cli short-drama +upload-file --path /path/to/file
+1. pippit-tool-cli short-drama +upload-file --path /path/to/file
    → 拿到 file_id
-2. pippit-cli short-drama +submit-run --message "用户的原始短剧需求" --asset-ids file_id
+2. pippit-tool-cli short-drama +submit-run --message "用户的原始短剧需求" --asset-ids file_id
    → 拿到 thread_id、run_id 和 web_thread_link
 3. 后续同场景 1 的并行查询、重要资产发现和文件下载流程
 ```
@@ -140,7 +140,7 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PA
 ### 场景 3：在已有短剧会话中续写或修改
 
 ```
-1. pippit-cli short-drama +submit-run --message "用户的新需求" --thread-id THREAD_ID
+1. pippit-tool-cli short-drama +submit-run --message "用户的新需求" --thread-id THREAD_ID
    → 拿到新的 run_id 和 web_thread_link
 2. 继续按场景 1 展示进展、处理用户补充问题、获取新增会话文件列表，并及时下载新增重要资产
 ```
@@ -264,7 +264,7 @@ pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PA
 
 对带 `download_url` 的重要资产调用下载工具，可并行。重要资产必须主动下载，不要等用户再次要求，也不要在调用下载工具前先检查本地文件是否存在。
 
-1. 调用 `pippit-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PATH`。
+1. 调用 `pippit-tool-cli short-drama +download-result --url DOWNLOAD_URL --output-path FILE_PATH`。
 2. 下载完成后，向用户展示本地文件路径；如果某个文件下载失败，记录失败项并在后续轮询中重试，不阻塞已成功落盘的文件展示。
 
 ## 向用户展示内容