@pippit-dev/cli 1.0.31 → 1.0.32

package/README.md

@@ -28,7 +28,7 @@
 npx skills add Pippit-dev/cli --skill xyq-marketing-skill
 ```
 
-需要 Node.js 16+，使用 Skill 内自包含脚本，无额外 npm 依赖。脚本从当前进程读取 `XYQ_ACCESS_KEY`；API 与 CLI 可使用同一 Access Key，但 CLI 浏览器登录不会设置此环境变量。密钥在本机安全配置，不写入请求 JSON 或命令参数。
+需要 Node.js 16+ 和支持 `marketing` 命令的 CLI。Skill 脚本通过原生 `marketing` 命令复用 CLI 登录态；先运行 `pippit-tool-cli status`，未登录时执行 `pippit-tool-cli login` 完成浏览器授权，无需用户提供 access_token 或 Access Key。生成请求通过 stdin 传入原生 CLI，凭据仅在 CLI 内用于鉴权。
 
 ```bash
 # 从仓库根目录执行；营销请求字段见接口契约，默认只预览

package/checksums.txt

@@ -1,6 +1,6 @@
-0023facaf65dc042d7dd08b285efbd046efd3dd4cebd29f31e3421b919d38bc9  pippit-tool-cli-1.0.31-darwin-amd64.tar.gz
-a6989ce590556924da7a2ceb5d94a8d75f81b459dae5fe016d0d929ae9c31a0f  pippit-tool-cli-1.0.31-darwin-arm64.tar.gz
-2ccc7bfb01fc00a0a7c2948abb18eda6da0769c7c1f21bca3db942a1e7b3a0b3  pippit-tool-cli-1.0.31-linux-amd64.tar.gz
-2a6c10aada4090701a97ef8af69ad91abf4d0fda000634c344157487c7895da7  pippit-tool-cli-1.0.31-linux-arm64.tar.gz
-fcf2e3e988952e046e23a4505630d758e7471d6d7125541eba7d5882588b67cb  pippit-tool-cli-1.0.31-windows-amd64.zip
-5b9ce586c808866404c3d6e006c9ed359d1d8aecee105ac38e0015b79be78d64  pippit-tool-cli-1.0.31-windows-arm64.zip
+2f3c2a486a66e408a6d3841f19ecb909fbaa657636a469ee782a424e59596a68  pippit-tool-cli-1.0.32-darwin-amd64.tar.gz
+f4ccd97a007167a9677ea85be52339a041f5448c4e4afd2ec996d32006ccec83  pippit-tool-cli-1.0.32-darwin-arm64.tar.gz
+91ae74eb2067844be741aa8a5f5f932152214c168d43efae801244d46e3f5cab  pippit-tool-cli-1.0.32-linux-amd64.tar.gz
+1928d81ea0f11fa20ae65a689654a9d2dddc4cacb4e6a5ae41a5cdd35e40eac5  pippit-tool-cli-1.0.32-linux-arm64.tar.gz
+79283336e84c3196454aecf96c367383795636f4e153234a11996124a78dd7a5  pippit-tool-cli-1.0.32-windows-amd64.zip
+aa73d963a3dd7d0eb5dba433fdf14d97510dfbe90515aab5318e55c0147d14d9  pippit-tool-cli-1.0.32-windows-arm64.zip

package/cmd/marketing.go

@@ -0,0 +1,144 @@
+package cmd
+
+import (
+	"context"
+	"encoding/json"
+	"fmt"
+	"io"
+	"mime"
+	"os"
+	"path/filepath"
+	"strings"
+	"time"
+
+	"github.com/Pippit-dev/pippit-cli/internal/common"
+	"github.com/Pippit-dev/pippit-cli/internal/config"
+	"github.com/spf13/cobra"
+)
+
+var marketingPaths = map[string]string{
+	"generate": "/api/biz/v1/agent/submit_marketing_run",
+	"query":    "/api/biz/v1/agent/query_generate_video_result",
+	"upload":   config.UploadFilePath,
+	"balance":  config.GetCreditBalancePath,
+}
+
+// Marketing uses the same AuthManager as login/status and all other commands.
+// Only fixed public endpoints are exposed; credentials never leave the CLI.
+func newMarketingCommand(stdout, stderr io.Writer, runner *common.Runner) *cobra.Command {
+	root := &cobra.Command{Use: "marketing", Short: "Marketing API using the shared CLI login"}
+	root.SetOut(stdout)
+	root.SetErr(stderr)
+	for _, action := range []string{"generate", "query", "upload", "balance"} {
+		root.AddCommand(newMarketingAction(action, stdout, stderr, runner))
+	}
+	return root
+}
+
+func newMarketingAction(action string, stdout, stderr io.Writer, runner *common.Runner) *cobra.Command {
+	var requestFile, file, threadID, runID string
+	var execute bool
+	var timeout time.Duration
+	command := &cobra.Command{Use: action, Args: cobra.NoArgs, Short: "Call marketing " + action}
+	command.SetOut(stdout)
+	command.SetErr(stderr)
+	command.Flags().DurationVar(&timeout, "timeout", 60*time.Second, "request deadline (e.g. 60s)")
+	switch action {
+	case "generate":
+		command.Flags().StringVar(&requestFile, "request", "", "request JSON file, or - for stdin")
+		command.Flags().BoolVar(&execute, "execute", false, "submit generation; otherwise preview only")
+	case "query":
+		command.Flags().StringVar(&threadID, "thread-id", "", "marketing thread ID")
+		command.Flags().StringVar(&runID, "run-id", "", "marketing run ID")
+	case "upload":
+		command.Flags().StringVar(&file, "file", "", "local media file")
+	}
+	command.RunE = func(cmd *cobra.Command, _ []string) error {
+		if timeout <= 0 || timeout > 30*time.Minute {
+			return fmt.Errorf("timeout 必须大于 0 且不超过 30m")
+		}
+		var body any = map[string]any{}
+		switch action {
+		case "generate":
+			if requestFile == "" {
+				return fmt.Errorf("缺少必填参数 --request")
+			}
+			reader := cmd.InOrStdin()
+			if requestFile != "-" {
+				f, err := os.Open(requestFile)
+				if err != nil {
+					return err
+				}
+				defer f.Close()
+				reader = f
+			}
+			var value map[string]json.RawMessage
+			decoder := json.NewDecoder(io.LimitReader(reader, 8*1024*1024+1))
+			if err := decoder.Decode(&value); err != nil {
+				return fmt.Errorf("请求 JSON 无效: %w", err)
+			}
+			var extra any
+			if err := decoder.Decode(&extra); err != io.EOF {
+				return fmt.Errorf("请求只能包含一个 JSON 对象")
+			}
+			for key := range value {
+				if key != "message" && key != "asset_ids" && key != "thread_id" && key != "general_agent_settings" {
+					return fmt.Errorf("未支持的营销请求字段: %s", key)
+				}
+			}
+			var message string
+			var settings struct {
+				VideoModel string `json:"video_model"`
+			}
+			if json.Unmarshal(value["message"], &message) != nil || strings.TrimSpace(message) == "" {
+				return fmt.Errorf("message 必须为非空字符串")
+			}
+			if json.Unmarshal(value["general_agent_settings"], &settings) != nil || strings.TrimSpace(settings.VideoModel) == "" {
+				return fmt.Errorf("general_agent_settings.video_model 必填")
+			}
+			body = value
+			if !execute {
+				return common.WriteJSON(stdout, map[string]any{"dry_run": true, "url": config.DefaultBaseURL + marketingPaths[action], "body": body})
+			}
+		case "query":
+			if strings.TrimSpace(threadID) == "" || strings.TrimSpace(runID) == "" {
+				return fmt.Errorf("query 需要 --thread-id 和 --run-id")
+			}
+			body = map[string]string{"thread_id": threadID, "run_id": runID}
+		case "upload":
+			if err := validateMediaUpload(file); err != nil {
+				return err
+			}
+			info, err := os.Stat(file)
+			if err != nil {
+				return err
+			}
+			if info.Size() == 0 {
+				return fmt.Errorf("上传需要非空文件")
+			}
+		}
+		ctx, cancel := context.WithTimeout(cmd.Context(), timeout)
+		defer cancel()
+		client := common.NewNonRedirectingHTTPClient(runner.Config.BaseURL, timeout, newRunnerAuthorizer(runner))
+		var result map[string]json.RawMessage
+		var err error
+		if action == "upload" {
+			contentType := mime.TypeByExtension(strings.ToLower(filepath.Ext(file)))
+			err = client.SendMultipartRequest(ctx, marketingPaths[action], nil, common.MultipartFile{FieldName: "file", Path: file, ContentType: contentType}, &result)
+		} else {
+			err = client.SendRequest(ctx, marketingPaths[action], body, &result)
+		}
+		if err != nil {
+			return err
+		}
+		ret := strings.TrimSpace(string(result["ret"]))
+		if ret != `"0"` && ret != "0" {
+			var message, logID string
+			_ = json.Unmarshal(result["errmsg"], &message)
+			_ = json.Unmarshal(result["log_id"], &logID)
+			return common.NewLogIDError(fmt.Sprintf("营销 API 失败: ret=%s errmsg=%s", ret, message), logID)
+		}
+		return common.WriteJSON(stdout, result)
+	}
+	return command
+}

package/cmd/marketing_test.go

@@ -0,0 +1,179 @@
+package cmd
+
+import (
+	"bytes"
+	"context"
+	"fmt"
+	"io"
+	"net/http"
+	"net/http/httptest"
+	"os"
+	"path/filepath"
+	"strings"
+	"testing"
+	"time"
+
+	"github.com/Pippit-dev/pippit-cli/internal/auth"
+	"github.com/Pippit-dev/pippit-cli/internal/config"
+)
+
+type marketingCredentialStore struct {
+	auth.CredentialStore
+	credential *auth.Credential
+	loads      int
+}
+
+func (s *marketingCredentialStore) Load(context.Context) (*auth.Credential, error) {
+	s.loads++
+	if s.credential == nil {
+		return nil, auth.ErrCredentialNotFound
+	}
+	return s.credential, nil
+}
+
+const marketingRequest = `{"message":"make an ad","general_agent_settings":{"video_model":"chosen-model","show_subtitle":false}}`
+
+func TestMarketingUsesSharedBrowserAuth(t *testing.T) {
+	for _, action := range []string{"generate", "query", "upload", "balance"} {
+		t.Run(action, func(t *testing.T) {
+			calls := 0
+			server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+				calls++
+				if r.Header.Get("Authorization") != "Bearer browser-secret" {
+					t.Error("browser login not reused")
+				}
+				if r.URL.Path != marketingPaths[action] || r.Method != "POST" {
+					t.Error("wrong endpoint")
+				}
+				data, _ := io.ReadAll(r.Body)
+				if bytes.Contains(data, []byte("browser-secret")) {
+					t.Error("credential leaked into payload")
+				}
+				if action == "upload" && !bytes.Contains(data, []byte(`name="file"`)) {
+					t.Error("missing multipart file")
+				}
+				fmt.Fprint(w, `{"ret":"0","log_id":"log-test","data":{"thread_id":"thread","run_id":"run"}}`)
+			}))
+			defer server.Close()
+			cfg := config.Load()
+			cfg.BaseURL = server.URL
+			cfg.AccessKey = ""
+			store := &marketingCredentialStore{credential: &auth.Credential{AccessKey: "browser-secret", UID: "user", DeviceID: "device", ExpiredAt: time.Now().Add(time.Hour).Unix()}}
+			runner := newRootRunner(cfg)
+			runner.Auth = auth.NewManager(cfg, auth.WithCredentialStore(store))
+			var output bytes.Buffer
+			root := newRootCommand(&output, io.Discard, runner)
+			args := []string{"marketing", action}
+			switch action {
+			case "generate":
+				args = append(args, "--request", "-", "--execute")
+				root.SetIn(strings.NewReader(marketingRequest))
+			case "query":
+				args = append(args, "--thread-id", "thread", "--run-id", "run")
+			case "upload":
+				file := filepath.Join(t.TempDir(), "product.png")
+				if err := os.WriteFile(file, []byte("image"), 0600); err != nil {
+					t.Fatal(err)
+				}
+				args = append(args, "--file", file)
+			}
+			root.SetArgs(args)
+			if err := root.Execute(); err != nil {
+				t.Fatal(err)
+			}
+			if calls != 1 || store.loads != 1 {
+				t.Fatalf("HTTP calls=%d credential loads=%d", calls, store.loads)
+			}
+			if strings.Contains(output.String(), "browser-secret") || !strings.Contains(output.String(), "log-test") {
+				t.Fatal("raw result or credential boundary broken")
+			}
+		})
+	}
+}
+
+func TestMarketingAuthFailureAndEnvironmentPrecedence(t *testing.T) {
+	for _, scenario := range []string{"missing", "expired", "override"} {
+		t.Run(scenario, func(t *testing.T) {
+			calls := 0
+			server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+				calls++
+				if r.Header.Get("Authorization") != "Bearer explicit-key" {
+					t.Error("environment override not preserved")
+				}
+				fmt.Fprint(w, `{"ret":"0","data":{}}`)
+			}))
+			defer server.Close()
+			cfg := config.Load()
+			cfg.BaseURL = server.URL
+			cfg.AccessKey = ""
+			store := &marketingCredentialStore{}
+			if scenario == "expired" {
+				store.credential = &auth.Credential{AccessKey: "expired", ExpiredAt: 1}
+			}
+			if scenario == "override" {
+				cfg.AccessKey = "explicit-key"
+			}
+			runner := newRootRunner(cfg)
+			runner.Auth = auth.NewManager(cfg, auth.WithCredentialStore(store))
+			root := newRootCommand(io.Discard, io.Discard, runner)
+			root.SetArgs([]string{"marketing", "balance"})
+			err := root.Execute()
+			if scenario == "override" {
+				if err != nil || calls != 1 || store.loads != 0 {
+					t.Fatalf("override: err=%v calls=%d loads=%d", err, calls, store.loads)
+				}
+			} else if err == nil || !strings.Contains(err.Error(), "pippit-tool-cli login") || calls != 0 {
+				t.Fatalf("missing/expired: err=%v calls=%d", err, calls)
+			}
+		})
+	}
+}
+
+func TestMarketingPreviewAndHelpDoNotReadCredentials(t *testing.T) {
+	for _, args := range [][]string{{"marketing", "--help"}, {"marketing", "generate", "--request", "-"}, {"marketing", "query"}} {
+		cfg := config.Load()
+		cfg.AccessKey = ""
+		store := &marketingCredentialStore{}
+		runner := newRootRunner(cfg)
+		runner.Auth = auth.NewManager(cfg, auth.WithCredentialStore(store))
+		root := newRootCommand(io.Discard, io.Discard, runner)
+		root.SetIn(strings.NewReader(marketingRequest))
+		root.SetArgs(args)
+		err := root.Execute()
+		if args[1] != "query" && err != nil {
+			t.Fatal(err)
+		}
+		if args[1] == "query" && err == nil {
+			t.Fatal("missing IDs accepted")
+		}
+		if store.loads != 0 {
+			t.Fatal("offline operation accessed credentials")
+		}
+	}
+}
+
+func TestMarketingDoesNotReplaySubmission(t *testing.T) {
+	for _, status := range []int{302, 307, 504} {
+		t.Run(fmt.Sprint(status), func(t *testing.T) {
+			calls := 0
+			server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+				calls++
+				w.Header().Set("Location", "/unexpected")
+				w.WriteHeader(status)
+			}))
+			defer server.Close()
+			cfg := config.Load()
+			cfg.BaseURL = server.URL
+			cfg.AccessKey = "test-key"
+			root := newRootCommand(io.Discard, io.Discard, newRootRunner(cfg))
+			root.SetIn(strings.NewReader(marketingRequest))
+			root.SetArgs([]string{"marketing", "generate", "--request", "-", "--execute"})
+			if err := root.Execute(); err == nil {
+				t.Fatal("failed/redirected request succeeded")
+			}
+			if calls != 1 {
+				t.Fatalf("submission replayed %d times", calls)
+			}
+		})
+	}
+}

package/cmd/root.go

@@ -73,6 +73,7 @@ func newRootCommand(stdout, stderr io.Writer, runner *common.Runner) *cobra.Comm
 	root.AddCommand(newDownloadResultCommand(stdout, stderr, runner))
 	root.AddCommand(newGetCreditBalanceCommand(stdout, stderr, runner))
 	root.AddCommand(newModelCommand(stdout, stderr, runner))
+	root.AddCommand(newMarketingCommand(stdout, stderr, runner))
 	root.AddCommand(newGetThreadCommand(stdout, stderr, runner))
 	root.AddCommand(newSubmitRunCommand(stdout, stderr, runner))
 	root.AddCommand(newUploadFileCommand(stdout, stderr, runner))

package/dist/checksums.txt

@@ -1,6 +1,6 @@
-0023facaf65dc042d7dd08b285efbd046efd3dd4cebd29f31e3421b919d38bc9  pippit-tool-cli-1.0.31-darwin-amd64.tar.gz
-a6989ce590556924da7a2ceb5d94a8d75f81b459dae5fe016d0d929ae9c31a0f  pippit-tool-cli-1.0.31-darwin-arm64.tar.gz
-2ccc7bfb01fc00a0a7c2948abb18eda6da0769c7c1f21bca3db942a1e7b3a0b3  pippit-tool-cli-1.0.31-linux-amd64.tar.gz
-2a6c10aada4090701a97ef8af69ad91abf4d0fda000634c344157487c7895da7  pippit-tool-cli-1.0.31-linux-arm64.tar.gz
-fcf2e3e988952e046e23a4505630d758e7471d6d7125541eba7d5882588b67cb  pippit-tool-cli-1.0.31-windows-amd64.zip
-5b9ce586c808866404c3d6e006c9ed359d1d8aecee105ac38e0015b79be78d64  pippit-tool-cli-1.0.31-windows-arm64.zip
+2f3c2a486a66e408a6d3841f19ecb909fbaa657636a469ee782a424e59596a68  pippit-tool-cli-1.0.32-darwin-amd64.tar.gz
+f4ccd97a007167a9677ea85be52339a041f5448c4e4afd2ec996d32006ccec83  pippit-tool-cli-1.0.32-darwin-arm64.tar.gz
+91ae74eb2067844be741aa8a5f5f932152214c168d43efae801244d46e3f5cab  pippit-tool-cli-1.0.32-linux-amd64.tar.gz
+1928d81ea0f11fa20ae65a689654a9d2dddc4cacb4e6a5ae41a5cdd35e40eac5  pippit-tool-cli-1.0.32-linux-arm64.tar.gz
+79283336e84c3196454aecf96c367383795636f4e153234a11996124a78dd7a5  pippit-tool-cli-1.0.32-windows-amd64.zip
+aa73d963a3dd7d0eb5dba433fdf14d97510dfbe90515aab5318e55c0147d14d9  pippit-tool-cli-1.0.32-windows-arm64.zip

package/internal/common/client.go

@@ -46,7 +46,17 @@ func NewHTTPClient(baseURL string, timeout time.Duration, authorizer RequestAuth
 	return newHTTPClient(baseURL, timeout, authorizer)
 }
 
-func newHTTPClient(baseURL string, timeout time.Duration, authorizer RequestAuthorizer) Client {
+// NewNonRedirectingHTTPClient keeps one-shot submissions from being replayed.
+// Authentication and request handling still use the shared CLI client.
+func NewNonRedirectingHTTPClient(baseURL string, timeout time.Duration, authorizer RequestAuthorizer) Client {
+	client := newHTTPClient(baseURL, timeout, authorizer)
+	client.httpClient.CheckRedirect = func(_ *http.Request, _ []*http.Request) error {
+		return fmt.Errorf("拒绝营销 API 重定向；请求未重试")
+	}
+	return client
+}
+
+func newHTTPClient(baseURL string, timeout time.Duration, authorizer RequestAuthorizer) *httpClient {
 	client := &httpClient{
 		baseURL:    strings.TrimRight(baseURL, "/"),
 		headers:    make(http.Header),

package/package.json

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

package/skills/xyq-marketing-skill/SKILL.md

@@ -3,7 +3,7 @@ name: xyq-marketing-skill
 description: 使用小云雀公开营销 API，根据商品图文生成剧情广告、品牌大片或达人带货营销视频；上传素材、提交营销成片、查询进度、下载交付结果及查询积分。用户要求小云雀营销一键成片或接入营销 API 时使用。
 user-invocable: true
 metadata:
-  {"openclaw": {"emoji": "🛍️", "requires": {"bins": ["node"], "env": ["XYQ_ACCESS_KEY"]}}}
+  {"openclaw": {"emoji": "🛍️", "requires": {"bins": ["node", "pippit-tool-cli"]}}}
 ---
 
 # 小云雀营销成片
@@ -12,9 +12,11 @@ metadata:
 
 ## Setup
 
-本 Skill 自包含，只需要 Node.js 16+，不依赖其它 Skill 或 npm 安装。脚本位置为 `"{baseDir}/scripts/marketing.js"`；下面示例中的相对路径从本 Skill 目录执行。
+需要 Node.js 16+ 和支持 `marketing` 命令的 `@pippit-dev/cli`。脚本位置为 `"{baseDir}/scripts/marketing.js"`；下面示例中的相对路径从本 Skill 目录执行。先检查 `pippit-tool-cli marketing --help`；命令不存在时安装或更新 CLI：`npm install -g @pippit-dev/cli@latest`，再检查一次；仍不支持就报告版本阻塞，不改为向用户索要密钥。
 
-API 和 CLI 可以使用同一个 Access Key，但脚本只读取当前进程的 `XYQ_ACCESS_KEY`。CLI 浏览器登录保存的凭据不会自动变成环境变量，不读取系统钥匙串或 CLI 凭据文件。缺少变量时，引导用户在 [官网 API 页](https://xyq.jianying.com/cli?tab=api) 管理 Access Key 并在本机安全配置，不能让用户在聊天中发送密钥，也不要将密钥写入请求文件、命令参数或日志。
+营销 API 调用复用其它 CLI 命令的登录态，脚本不读取或导出凭据。执行真实请求前运行 `pippit-tool-cli status`，读取 JSON 的 `logged_in`，不能只看退出码。未登录或凭据过期时运行 `pippit-tool-cli login`，让用户在浏览器完成授权；等待 CLI 成功返回后再次检查 status，再继续原任务。已有有效登录态直接复用，不重复登录；离线预览和帮助不要求登录。
+
+不要让用户提供或复制 `access_token`、Access Key，也不要把凭据写入聊天、请求文件、命令参数或日志。CLI 保留原有 `XYQ_ACCESS_KEY` 显式环境覆盖规则，优先于网页登录；这仅用于已配置的自动化环境，不作为普通用户的必填项。覆盖无效时不能静默切换账号。网页登录凭据被拒绝时按 CLI 的 `login --force` 流程处理，不自动重提可能已创建的生成任务。
 
 先按 [接口契约](references/api.md) 准备请求；需要完整执行示例时读 [商品图到营销视频](examples/product-video.md)。
 
@@ -24,7 +26,7 @@ API 和 CLI 可以使用同一个 Access Key，但脚本只读取当前进程的
 2. 有本地商品素材时逐个上传，保留返回的 `data.pippit_asset_id`。远程素材先取得用户授权使用的本地文件；不能把 URL 或路径放进 `asset_ids`。用户已提供有效资产 ID 时直接复用。无素材的纯文字请求无需上传。
 3. 写入 UTF-8 JSON 请求文件。`message` 保留用户指令；`general_agent_settings.video_model` 必填，不能传 `{}`。用户指定模型时原样使用；未指定时询问，或在用户已明确授权“你决定”等选择范围内选定并说明。其它选项按用户给定或已授权的偏好设置，不静默换模型。`thread_id` 仅在继续已有营销会话时传入真实 ID。
 4. 先预览校验，已获生成授权后使用 `--execute` 提交。立即保存响应并展示真实 `data.web_thread_link`，保留 `data.run.thread_id`、`data.run.run_id`。缺少网页链接时只报告实际返回信息，不自行拼接链接。
-5. 用当前 thread/run 查询到结束并下载媒体。遇到确认或问卷时沿用已有授权；缺少必要选择再问用户。确认后继续同一 thread，并取得最新 run_id 再查询：旧 Run 可永久保持等待交互状态。可通过宿主浏览器查看已返回的网页链接；安装了 `pippit-tool-cli` 时，也可用 `get-thread --thread-id THREAD_ID` 读取各 Run 的真实 ID。不要用旧 Run 重复确认或重新生成。提交成功、网页链接或进度链接都不等于交付完成。
+5. 用当前 thread/run 查询到结束并下载媒体。遇到确认或问卷时沿用已有授权；缺少必要选择再问用户。确认后继续同一 thread，并取得最新 run_id 再查询：旧 Run 可永久保持等待交互状态。可通过宿主浏览器查看已返回的网页链接；也可用 `pippit-tool-cli` 的 `get-thread --thread-id THREAD_ID` 读取各 Run 的真实 ID。不要用旧 Run 重复确认或重新生成。提交成功、网页链接或进度链接都不等于交付完成。
 
 ```bash
 node scripts/marketing.js upload --file /path/to/product.png
@@ -34,7 +36,7 @@ node scripts/marketing.js query --thread-id THREAD_ID --run-id RUN_ID --wait --m
 node scripts/marketing.js balance
 ```
 
-生成默认是离线预览；预览不需要密钥。上传、查询、余额是实际 API 请求。`--timeout` 设置单请求总时限（秒，默认 60）。`--wait` 由脚本每 10 秒查询，默认最多 900 秒；不再叠加其它轮询器。没有 `--wait` 时只查询一次。脚本输出逐行 JSON，查询下载前先输出服务端响应，再逐个输出已下载文件，最后输出含 `downloaded_files` 的响应。
+生成默认是离线预览；预览不需要登录。上传、查询、余额是实际 API 请求。`--timeout` 设置单请求总时限（秒，默认 60）。`--wait` 由脚本每 10 秒查询，默认最多 900 秒；不再叠加其它轮询器。没有 `--wait` 时只查询一次。脚本输出逐行 JSON，查询下载前先输出服务端响应，再逐个输出已下载文件，最后输出含 `downloaded_files` 的响应。
 
 ## Completion and Recovery
 
@@ -48,4 +50,4 @@ node scripts/marketing.js balance
 
 ## Scope
 
-本 Skill 仅使用正式公开营销接口；不宣称支持团队空间切换。鉴权范围由用户配置的 Access Key 和服务端决定，公开请求没有 `TeamID` 字段，不自行加入团队字段或跨账号复用资产/任务 ID。沉浸式短片、火山引擎服务和现有 CLI 的来源统计参数不在此脚本范围；不要把 `--source` 等未公开字段传给营销接口。
+本 Skill 仅使用正式公开营销接口；不宣称支持团队空间切换。鉴权范围沿用当前 CLI 登录身份和服务端授权，公开请求没有 `TeamID` 字段，不自行加入团队字段或跨账号复用资产/任务 ID。沉浸式短片、火山引擎服务和现有 CLI 的来源统计参数不在此脚本范围；不要把 `--source` 等未公开字段传给营销接口。

package/skills/xyq-marketing-skill/examples/product-video.md

@@ -2,7 +2,7 @@
 
 用户示例：“用这张保温杯商品图做一个 15 秒竖屏达人带货视频，不要字幕，用 Seedance 2.0 VIP，1080p。”
 
-1. 确认当前进程已配置 `XYQ_ACCESS_KEY`，然后上传用户给出的真实文件：
+1. 先检查 `pippit-tool-cli marketing --help` 与 `pippit-tool-cli status`；未登录时执行 `pippit-tool-cli login` 并等待浏览器授权成功，无需用户提供 token。随后上传用户给出的真实文件：
 
    ```bash
    node scripts/marketing.js upload --file /path/to/cup.png

package/skills/xyq-marketing-skill/references/api.md

@@ -4,11 +4,11 @@
 
 ## Endpoints
 
-Base URL：`https://xyq.jianying.com`。全部使用 POST，认证为 `Authorization: Bearer <Access Key>`，`Accept: application/json`。
+Base URL：`https://xyq.jianying.com`。全部使用 POST，`Accept: application/json`。CLI 的统一认证层从现有登录态注入 Authorization 请求头；营销脚本不接触凭据，不要求用户提供 token。
 
 | 操作 | 路径 | 请求体 |
 | --- | --- | --- |
-| 上传单文件 | `/api/biz/v1/skill/upload_file` | multipart/form-data，字段 `file`，boundary 由脚本生成 |
+| 上传单文件 | `/api/biz/v1/skill/upload_file` | multipart/form-data，字段 `file`，boundary 由 CLI 生成 |
 | 营销成片 | `/api/biz/v1/agent/submit_marketing_run` | JSON，字段见下表 |
 | 查询结果 | `/api/biz/v1/agent/query_generate_video_result` | JSON：真实 `thread_id`、`run_id` |
 | 查询积分 | `/api/biz/v1/skill/get_credit_balance` | JSON：`{}` |

package/skills/xyq-marketing-skill/scripts/marketing.js

@@ -1,10 +1,10 @@
 #!/usr/bin/env node
-// Documented Xiaoyunque marketing API; Node.js >= 16, no dependencies.
+// Marketing orchestration; authenticated API calls stay inside pippit-tool-cli.
 const fs = require('fs');
 const path = require('path');
 const https = require('https');
 const crypto = require('crypto');
-const { Readable } = require('stream');
+const { spawn } = require('child_process');
 const { pipeline } = require('stream/promises');
 
 const BASE = 'https://xyq.jianying.com';
@@ -63,7 +63,64 @@ function checkResponse(action, result, body) {
   return result;
 }
 
-function createClient({ key = process.env.XYQ_ACCESS_KEY, request = https.request, timeout = 60000 } = {}) {
+function resolveCLI({ platform = process.platform, searchPath = process.env.PATH || '', packageRoot = path.resolve(__dirname, '../../..') } = {}) {
+  const binaryName = platform === 'win32' ? 'pippit-tool-cli.exe' : 'pippit-tool-cli';
+  function fromPackage(root) {
+    const manifest = path.join(root, 'package.json');
+    if (!fs.existsSync(manifest) || JSON.parse(fs.readFileSync(manifest, 'utf8')).name !== '@pippit-dev/cli') return null;
+    const command = path.join(root, 'bin', binaryName);
+    requireValue(fs.existsSync(command), 'CLI 原生程序缺失，请重新安装 @pippit-dev/cli；无需提供 access_token');
+    return { command, args: [] };
+  }
+  // Run the native binary directly so deadlines also stop the API process.
+  const bundled = fromPackage(packageRoot);
+  if (bundled) return bundled;
+  for (const dir of searchPath.split(path.delimiter).filter(Boolean)) {
+    const binary = path.join(dir, binaryName);
+    if (fs.existsSync(binary)) {
+      const resolved = fs.realpathSync(binary);
+      const npmPackage = fromPackage(path.resolve(path.dirname(resolved), '..'));
+      return npmPackage || { command: binary, args: [] };
+    }
+    if (platform === 'win32') {
+      const npmPackage = fromPackage(path.join(dir, 'node_modules/@pippit-dev/cli'));
+      if (npmPackage) return npmPackage;
+    }
+  }
+  throw new Error('请先安装或更新 @pippit-dev/cli，并运行 pippit-tool-cli login；无需提供 access_token');
+}
+
+function invokeCLI(args, input, timeout, invocation = resolveCLI()) {
+  return new Promise((resolve, reject) => {
+    const child = spawn(invocation.command, [...invocation.args, ...args], {
+      shell: false, windowsHide: true, stdio: ['pipe', 'pipe', 'pipe'],
+    });
+    let stdout = '', stderr = '', failure;
+    const stop = message => { failure = new Error(message); child.kill(); };
+    const timer = setTimeout(() => stop('CLI 请求超时；提交结果可能不明确，请勿自动重提'), timeout);
+    child.stdout.setEncoding('utf8'); child.stderr.setEncoding('utf8');
+    child.stdout.on('data', chunk => {
+      stdout += chunk;
+      if (Buffer.byteLength(stdout) > 8 * 1024 * 1024) stop('CLI 响应超出 8 MiB 限制');
+    });
+    child.stderr.on('data', chunk => {
+      stderr += chunk;
+      if (Buffer.byteLength(stderr) > 1024 * 1024) stop('CLI 错误输出过大');
+    });
+    child.once('error', () => { clearTimeout(timer); reject(new Error('无法启动 pippit-tool-cli，请检查安装；无需提供 access_token')); });
+    child.stdin.on('error', error => { if (error.code !== 'EPIPE') failure = new Error('无法写入 CLI 请求'); });
+    child.once('close', code => {
+      clearTimeout(timer);
+      if (failure) return reject(failure);
+      if (code !== 0) return reject(new Error(`CLI 调用失败：${stderr.trim() || `退出码 ${code}`}；请求未重试。若不支持 marketing 命令，请更新 CLI。`));
+      try { resolve(JSON.parse(stdout)); }
+      catch (_) { reject(new Error('CLI 未返回有效 JSON；提交结果可能不明确，请勿自动重提')); }
+    });
+    child.stdin.end(input);
+  });
+}
+
+function createClient({ request = https.request, timeout = 60000, invoke = invokeCLI } = {}) {
   function open(url, method, headers, body) {
     return new Promise((resolve, reject) => {
       const target = new URL(url);
@@ -84,42 +141,23 @@ function createClient({ key = process.env.XYQ_ACCESS_KEY, request = https.reques
     });
   }
 
-  async function api(action, body, headers = { 'Content-Type': 'application/json' }) {
-    requireValue(nonempty(key) && !/[\r\n]/.test(key), '请在本机环境设置 XYQ_ACCESS_KEY，不要在聊天中发送密钥；CLI 登录不会设置此变量');
-    const wire = headers['Content-Type'] === 'application/json' ? JSON.stringify(body) : body;
-    const res = await open(BASE + PATHS[action], 'POST', { ...headers, Accept: 'application/json', Authorization: `Bearer ${key}` }, wire);
-    // Never follow API redirects or automatically retry a POST.
-    if (res.statusCode < 200 || res.statusCode >= 300) {
-      res.resume();
-      throw new Error(`HTTP ${res.statusCode}；请求未重试，生成结果可能不明确`);
-    }
-    const chunks = [];
-    let size = 0;
-    for await (const chunk of res) {
-      size += chunk.length;
-      requireValue(size <= 8 * 1024 * 1024, 'API 响应超出 8 MiB 限制');
-      chunks.push(chunk);
+  async function api(action, body) {
+    requireValue(['generate', 'query', 'balance'].includes(action), '无效营销 API 操作');
+    const args = ['marketing', action, '--timeout', `${timeout}ms`];
+    let input;
+    if (action === 'generate') {
+      validate(body);
+      args.push('--request', '-', '--execute');
+      input = JSON.stringify(body);
     }
-    let result;
-    try { result = JSON.parse(Buffer.concat(chunks).toString('utf8')); }
-    catch (_) { throw new Error('API 未返回有效 JSON；提交结果可能不明确，请勿自动重提'); }
-    return checkResponse(action, result, body);
+    if (action === 'query') args.push('--thread-id', body.thread_id, '--run-id', body.run_id);
+    return checkResponse(action, await invoke(args, input, timeout), body);
   }
 
   async function upload(file) {
     const stat = await fs.promises.stat(file);
     requireValue(stat.isFile() && stat.size > 0 && stat.size < 500000000, '上传需要非空文件且小于 500 MB');
-    const boundary = 'xyq-' + crypto.randomBytes(16).toString('hex');
-    const name = path.basename(file).replace(/["\r\n\\]/g, '_');
-    const mime = { '.png': 'image/png', '.jpg': 'image/jpeg', '.jpeg': 'image/jpeg', '.webp': 'image/webp', '.mp4': 'video/mp4', '.mp3': 'audio/mpeg', '.wav': 'audio/wav' }[path.extname(file).toLowerCase()] || 'application/octet-stream';
-    const head = Buffer.from(`--${boundary}\r\nContent-Disposition: form-data; name="file"; filename="${name}"\r\nContent-Type: ${mime}\r\n\r\n`);
-    const tail = Buffer.from(`\r\n--${boundary}--\r\n`);
-    const body = Readable.from((async function* () {
-      yield head;
-      for await (const chunk of fs.createReadStream(file)) yield chunk;
-      yield tail;
-    })());
-    return api('upload', body, { 'Content-Type': `multipart/form-data; boundary=${boundary}`, 'Content-Length': head.length + stat.size + tail.length });
+    return checkResponse('upload', await invoke(['marketing', 'upload', '--file', path.resolve(file), '--timeout', `${timeout}ms`], undefined, timeout));
   }
 
   async function download(url, destination) {
@@ -159,7 +197,7 @@ const HELP = `小云雀营销 Skill（Node.js >= 16）
   node marketing.js upload --file product.png
   node marketing.js query --thread-id ID --run-id ID [--wait] [--max-wait 900] [--output-dir DIR]
   node marketing.js balance
-所有 API 调用从环境读取 XYQ_ACCESS_KEY；generate 默认仅预览。
+所有 API 调用复用 pippit-tool-cli 登录态；未登录先运行 pippit-tool-cli login，无需提供 access_token。generate 默认仅预览。
 --timeout 秒数：单请求总时限，默认 60；--max-wait：轮询总时限，默认 900。
 query 输出 API 原始响应；有 --output-dir 时成功结果附带 downloaded_files。
 退出码：0 成功/单次查询进行中；1 输入或接口错误；2 生成失败/取消/无视频；3 等待超时；4 等待用户交互；5 未知或未指定状态。
@@ -189,7 +227,7 @@ function parseArgs(argv) {
   return { action, options };
 }
 
-async function main(argv, { env = process.env, out = console.log, clientFactory = createClient, now = Date.now, pause = ms => new Promise(resolve => setTimeout(resolve, ms)) } = {}) {
+async function main(argv, { out = console.log, clientFactory = createClient, now = Date.now, pause = ms => new Promise(resolve => setTimeout(resolve, ms)) } = {}) {
   if (!argv.length || argv.includes('--help') || argv.includes('-h')) { out(HELP); return 0; }
   const { action, options } = parseArgs(argv);
   let body = {};
@@ -203,13 +241,13 @@ async function main(argv, { env = process.env, out = console.log, clientFactory
     body = { thread_id: options['thread-id'], run_id: options['run-id'] };
   }
   if (action === 'upload') requireValue(nonempty(options.file), 'upload 需要 --file');
-  const client = clientFactory({ key: env.XYQ_ACCESS_KEY, timeout: options.timeout * 1000 });
+  const client = clientFactory({ timeout: options.timeout * 1000 });
   let result;
   const deadline = now() + options['max-wait'] * 1000;
   do {
     // A poll request cannot overrun the remaining wait budget.
     const pollClient = action === 'query' && options.wait
-      ? clientFactory({ key: env.XYQ_ACCESS_KEY, timeout: Math.min(options.timeout * 1000, Math.max(1, deadline - now())) }) : client;
+      ? clientFactory({ timeout: Math.min(options.timeout * 1000, Math.max(1, deadline - now())) }) : client;
     result = action === 'upload' ? await client.upload(options.file) : await pollClient.api(action, body);
     if (action !== 'query' || !options.wait || !WAIT_STATES.includes(String(result.data.run_state))) break;
     if (now() >= deadline) {
@@ -265,4 +303,4 @@ if (require.main === module) {
     process.exitCode = 1;
   });
 }
-module.exports = { BASE, PATHS, validate, checkResponse, createClient, parseArgs, main };
+module.exports = { BASE, PATHS, validate, checkResponse, resolveCLI, invokeCLI, createClient, parseArgs, main };