@pippit-dev/cli 0.0.27 → 0.0.29

package/checksums.txt

@@ -1,6 +1,6 @@
-3e95e8bbd667579fa1feddd4c58e583d70003f0f179d7249135b883a612165e5  pippit-tool-cli-0.0.27-darwin-amd64.tar.gz
-15671a5880feff8a67bd0c8042ab7e2a77d1ea7c81ed6625427bdd472d925c6a  pippit-tool-cli-0.0.27-darwin-arm64.tar.gz
-aef2fc4896d2c8966f681a82e5bd624b9ec9f0382e23dd05776f014cbb65473d  pippit-tool-cli-0.0.27-linux-amd64.tar.gz
-5b01a760298c22a6fd42661f212f9f1d1e822e84d1feb87307c36590f795515a  pippit-tool-cli-0.0.27-linux-arm64.tar.gz
-18315d247bb409f594d11e64cb2292ffbf48504d73290011f5c5a2ba319f93df  pippit-tool-cli-0.0.27-windows-amd64.zip
-31284208bc244d9d48066c8e7086bf40e167209fff106ea6462c9638cbd6efef  pippit-tool-cli-0.0.27-windows-arm64.zip
+c444ddc4fab86a19832f60148c8ce6d51e269796fa15f72a54465b7d763ee70e  pippit-tool-cli-0.0.29-darwin-amd64.tar.gz
+64c24941c80b7ce592ca0875fae671fa704e0798416d1f13a2168cd6d37b28b2  pippit-tool-cli-0.0.29-darwin-arm64.tar.gz
+551e089b4a533a9ca6fcdb8c3978b48442a2b16ed4d91ed10e789454910f402d  pippit-tool-cli-0.0.29-linux-amd64.tar.gz
+b968b76e2486a04b82bf4fed90859428e076f827a61f692522cf6023772830a6  pippit-tool-cli-0.0.29-linux-arm64.tar.gz
+661f9bd514d04ff1a38143f2b0196fa5dffbd7a8634fac27072d0f9ce71026c1  pippit-tool-cli-0.0.29-windows-amd64.zip
+26bc973e153cf781244cc1cc6175af2d6650563083e9ac1a2061f28932ff6560  pippit-tool-cli-0.0.29-windows-arm64.zip

package/cmd/short_drama/short_drama.go

@@ -143,7 +143,7 @@ func newShortDramaDownloadResultCommand(stdout, stderr io.Writer, runner *common
 }
 
 func newShortDramaGetThreadCommand(stdout, stderr io.Writer, runner *common.Runner) *cobra.Command {
-	var opts short_drama.GetThreadOptions
+	var opts common.GetThreadOptions
 
 	cmd := &cobra.Command{
 		Use:   "+get-thread",
@@ -161,7 +161,7 @@ func newShortDramaGetThreadCommand(stdout, stderr io.Writer, runner *common.Runn
 			}
 			opts.RunID = strings.TrimSpace(opts.RunID)
 
-			result, err := short_drama.GetThread(cmd.Context(), &opts, runner)
+			result, err := common.GetThread(cmd.Context(), &opts, runner)
 			if err != nil {
 				return err
 			}

package/cmd/update/update.go

@@ -11,6 +11,7 @@ import (
 	"path/filepath"
 	"runtime"
 	"strings"
+	"sync"
 	"time"
 
 	"github.com/Pippit-dev/pippit-cli/internal/version"
@@ -23,6 +24,7 @@ const (
 	defaultTelemetryBaseURL = "https://xyq.jianying.com"
 	telemetryPath           = "/api/biz/v1/skill/report_telemetry"
 	telemetryAuthHeader     = "Bearer pippit-cli-skill-telemetry"
+	telemetryWaitTimeout    = time.Second
 )
 
 var legacyGlobalSkills = []string{
@@ -35,7 +37,7 @@ var telemetrySkillNames = []string{
 	"xyq-short-drama-skill",
 }
 
-var telemetryHTTPClient = &http.Client{Timeout: 2 * time.Second}
+var telemetryHTTPClient = &http.Client{Timeout: telemetryWaitTimeout}
 
 // NewCommand builds the update command.
 func NewCommand(stdout, stderr io.Writer) *cobra.Command {
@@ -137,6 +139,7 @@ func reportBundledSkillTelemetry(event string, source string, stderr io.Writer)
 	if os.Getenv("PIPPIT_CLI_DISABLE_TELEMETRY") == "1" {
 		return
 	}
+	var wg sync.WaitGroup
 	for _, skillName := range telemetrySkillNames {
 		payload := telemetryPayload{
 			Event:      event,
@@ -146,12 +149,23 @@ func reportBundledSkillTelemetry(event string, source string, stderr io.Writer)
 			Platform:   runtime.GOOS,
 			Arch:       runtime.GOARCH,
 		}
+		wg.Add(1)
 		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)
 			}
 		}(payload)
 	}
+	done := make(chan struct{})
+	go func() {
+		wg.Wait()
+		close(done)
+	}()
+	select {
+	case <-done:
+	case <-time.After(telemetryWaitTimeout):
+	}
 }
 
 func reportSkillTelemetry(payload telemetryPayload) error {
@@ -166,8 +180,6 @@ func reportSkillTelemetry(payload telemetryPayload) error {
 	}
 	req.Header.Set("Content-Type", "application/json")
 	req.Header.Set("Authorization", telemetryAuthHeader)
-	req.Header.Set("x-use-ppe", "1")
-	req.Header.Set("x-tt-env", "ppe_harness_novel_v2")
 
 	resp, err := telemetryHTTPClient.Do(req)
 	if err != nil {

package/cmd/update/update_test.go

@@ -10,6 +10,7 @@ import (
 	"runtime"
 	"strings"
 	"testing"
+	"time"
 )
 
 func TestInstallSkillsInstallsAllBundledSkills(t *testing.T) {
@@ -76,14 +77,12 @@ func TestCleanupLegacyGlobalSkills(t *testing.T) {
 
 func TestReportSkillTelemetry(t *testing.T) {
 	var gotAuth string
-	var gotHeaders http.Header
 	var gotPayload telemetryPayload
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		if r.URL.Path != telemetryPath {
 			t.Fatalf("path = %q, want %q", r.URL.Path, telemetryPath)
 		}
 		gotAuth = r.Header.Get("Authorization")
-		gotHeaders = r.Header.Clone()
 		if err := json.NewDecoder(r.Body).Decode(&gotPayload); err != nil {
 			t.Fatalf("decode request body: %v", err)
 		}
@@ -106,17 +105,52 @@ func TestReportSkillTelemetry(t *testing.T) {
 	if gotAuth != telemetryAuthHeader {
 		t.Fatalf("Authorization = %q, want %q", gotAuth, telemetryAuthHeader)
 	}
-	if got := gotHeaders.Get("x-use-ppe"); got != "1" {
-		t.Fatalf("x-use-ppe = %q, want 1", got)
-	}
-	if got := gotHeaders.Get("x-tt-env"); got != "ppe_harness_novel_v2" {
-		t.Fatalf("x-tt-env = %q, want ppe_harness_novel_v2", got)
-	}
 	if gotPayload.Event != "update" || gotPayload.SkillName != "xyq-skill" || gotPayload.Source != "cli_update" {
 		t.Fatalf("payload = %#v", gotPayload)
 	}
 }
 
+func TestReportBundledSkillTelemetryWaitsBriefly(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		time.Sleep(2 * time.Second)
+		_, _ = w.Write([]byte(`{"ret":"0","errmsg":""}`))
+	}))
+	defer server.Close()
+
+	t.Setenv("PIPPIT_CLI_TELEMETRY_BASE_URL", server.URL)
+	start := time.Now()
+	reportBundledSkillTelemetry("update", "cli_update", &bytes.Buffer{})
+	if elapsed := time.Since(start); elapsed > 1500*time.Millisecond {
+		t.Fatalf("reportBundledSkillTelemetry() blocked for %v, want <= 1.5s", elapsed)
+	}
+}
+
+func TestReportBundledSkillTelemetryReportsBothSkills(t *testing.T) {
+	skillNames := make(chan string, len(telemetrySkillNames))
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		var payload telemetryPayload
+		if err := json.NewDecoder(r.Body).Decode(&payload); err != nil {
+			t.Fatalf("decode request body: %v", err)
+		}
+		skillNames <- payload.SkillName
+		_, _ = w.Write([]byte(`{"ret":"0","errmsg":""}`))
+	}))
+	defer server.Close()
+
+	t.Setenv("PIPPIT_CLI_TELEMETRY_BASE_URL", server.URL)
+	reportBundledSkillTelemetry("update", "cli_update", &bytes.Buffer{})
+
+	got := map[string]bool{}
+	for i := 0; i < len(telemetrySkillNames); i++ {
+		got[<-skillNames] = true
+	}
+	for _, skillName := range telemetrySkillNames {
+		if !got[skillName] {
+			t.Fatalf("missing telemetry for %s, got %#v", skillName, got)
+		}
+	}
+}
+
 func TestTelemetryBaseURL(t *testing.T) {
 	t.Setenv("PIPPIT_CLI_TELEMETRY_BASE_URL", "   ")
 	t.Setenv("XYQ_OPENAPI_BASE", "https://example.com///")

package/internal/common/client.go

@@ -202,8 +202,6 @@ func (c *httpClient) injectHeaders(req *http.Request) {
 		}
 	}
 	req.Header.Set("User-Agent", "Pippit-CLI/1.0")
-	req.Header.Set("x-use-ppe", "1")
-	req.Header.Set("x-tt-env", "ppe_harness_novel_v2")
 }
 
 func (c *httpClient) do(req *http.Request, out any) error {

package/internal/{short_drama → common}/get_thread.go

@@ -1,10 +1,9 @@
-package short_drama
+package common
 
 import (
 	"context"
 	"fmt"
 
-	"github.com/Pippit-dev/pippit-cli/internal/common"
 	"github.com/Pippit-dev/pippit-cli/internal/config"
 )
 
@@ -29,7 +28,7 @@ type getThreadResponse struct {
 	} `json:"data"`
 }
 
-func GetThread(ctx context.Context, opts *GetThreadOptions, runner *common.Runner) (*GetThreadResult, error) {
+func GetThread(ctx context.Context, opts *GetThreadOptions, runner *Runner) (*GetThreadResult, error) {
 	if runner == nil || runner.Client == nil {
 		return nil, fmt.Errorf("get_thread runner client is required")
 	}
@@ -61,7 +60,7 @@ func GetThread(ctx context.Context, opts *GetThreadOptions, runner *common.Runne
 	}, nil
 }
 
-func getThreadPath(runner *common.Runner) string {
+func getThreadPath(runner *Runner) string {
 	if runner != nil && runner.Config != nil && runner.Config.Paths != nil && runner.Config.Paths.GetThread != "" {
 		return runner.Config.Paths.GetThread
 	}

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@pippit-dev/cli",
-  "version": "0.0.27",
+  "version": "0.0.29",
   "description": "Pippit CLI",
   "bin": {
     "pippit-tool-cli": "scripts/run.js"
   },
   "scripts": {
     "postinstall": "node scripts/install.js",
-    "test": "node scripts/version-check.test.js && node scripts/skills.test.js && go test ./... && go vet ./..."
+    "test": "node scripts/version-check.test.js && node scripts/skills.test.js && node scripts/install-wizard.test.js && go test ./... && go vet ./..."
   },
   "os": [
     "darwin",

package/scripts/install-wizard.js

@@ -6,7 +6,15 @@ const { isWindows, run, runSilent } = require("./platform");
 const { DEFAULT_PKG, installGlobalPackageSkills } = require("./skills");
 const { reportBundledSkillTelemetry } = require("./telemetry");
 
-const PKG = process.env.PIPPIT_CLI_INSTALL_PACKAGE || DEFAULT_PKG;
+const VERSION = require("../package.json").version.replace(/-.*$/, "");
+
+function defaultInstallPackage() {
+  return `${DEFAULT_PKG}@${VERSION}`;
+}
+
+function installPackage() {
+  return process.env.PIPPIT_CLI_INSTALL_PACKAGE || defaultInstallPackage();
+}
 
 function getGloballyInstalledVersion() {
   try {
@@ -38,13 +46,14 @@ function whichPippitToolCli() {
 }
 
 function main() {
+  const pkg = installPackage();
   const installed = getGloballyInstalledVersion();
   if (installed) {
-    console.log(`Updating global pippit-tool-cli (${installed}) via ${PKG}...`);
+    console.log(`Updating global pippit-tool-cli (${installed}) via ${pkg}...`);
   } else {
-    console.log(`Installing ${PKG} globally...`);
+    console.log(`Installing ${pkg} globally...`);
   }
-  run("npm", ["install", "-g", PKG], {
+  run("npm", ["install", "-g", pkg], {
     timeout: 120000,
     env: { ...process.env, PIPPIT_CLI_SKIP_SKILLS: "1" },
   });
@@ -57,7 +66,7 @@ function main() {
       throw err;
     }
     console.log("Existing global package does not contain skills; reinstalling...");
-    run("npm", ["install", "-g", PKG], { timeout: 120000 });
+    run("npm", ["install", "-g", pkg], { timeout: 120000 });
     installGlobalPackageSkills(DEFAULT_PKG);
   }
 
@@ -73,4 +82,12 @@ function main() {
   console.log("Try: pippit-tool-cli short-drama +submit-run --message \"写一个短剧开头\"");
 }
 
-main();
+if (require.main === module) {
+  main();
+}
+
+module.exports = {
+  defaultInstallPackage,
+  installPackage,
+  main,
+};

package/scripts/run.js

@@ -38,7 +38,7 @@ if (process.platform === "win32" && fs.existsSync(oldBin)) {
 // Match the lark-cli install entry: `npx @pippit-dev/cli@latest install`
 // should run the JS setup flow before the native binary exists.
 if (args[0] === "install") {
-  require("./install-wizard.js");
+  require("./install-wizard.js").main();
 } else {
   maybeWarnNewVersion(args);
 

package/scripts/telemetry.js

@@ -48,8 +48,6 @@ function reportSkillTelemetry(payload) {
         "Content-Type": "application/json",
         "Content-Length": Buffer.byteLength(body),
         "Authorization": AUTH_HEADER,
-        "x-use-ppe": "1",
-        "x-tt-env": "ppe_harness_novel_v2",
       },
     }, (res) => {
       res.resume();