@geoly-ai/social-hub-cli 0.3.24 → 0.3.26

package/dist/register-subreddit-assets.test.js

@@ -0,0 +1,124 @@
+import { Command } from "commander";
+import { describe, expect, it } from "vitest";
+import { parseDeclaredPermissions, registerSubredditAssetCommands, } from "./register-subreddit-assets.js";
+import { collectCommandPaths } from "./cmd-manifest.js";
+describe("parseDeclaredPermissions", () => {
+    it("逗号分隔 → 数组，去空白", () => {
+        expect(parseDeclaredPermissions("all, posts ,flair")).toEqual([
+            "all",
+            "posts",
+            "flair",
+        ]);
+    });
+    /**
+     * 🔴 「没给这个选项」与「显式给了空」必须可区分。
+     * 前者 → `undefined`（不写这个字段，服务端保留既有声明）；
+     * 后者 → `[]`（显式声明"没有任何权限位"）。
+     * 混成一个值的话，一次漏传选项就会把人工声明的权限位清空 —— 而 declaredPermissions
+     * 是「我们应该拥有什么权限」的人工记录，被静默清掉就再也对不出来了。
+     */
+    it("未给选项 → undefined（不是空数组）", () => {
+        expect(parseDeclaredPermissions(undefined)).toBeUndefined();
+    });
+    it("给了空串 / 全是分隔符 → 空数组（显式声明为空）", () => {
+        expect(parseDeclaredPermissions("")).toEqual([]);
+        expect(parseDeclaredPermissions(" , , ")).toEqual([]);
+    });
+});
+describe("命令树", () => {
+    it("命令树完整（读 2 / 身份写 1 / 资产写 2 / 绑定写 3 / 档位 2 + 两个分组）", () => {
+        const program = new Command();
+        registerSubredditAssetCommands(program);
+        const paths = collectCommandPaths(program).map((entry) => entry.path);
+        expect(paths).toEqual(expect.arrayContaining([
+            "subreddit-asset",
+            "subreddit-asset list",
+            "subreddit-asset get",
+            "subreddit-asset register-subreddit",
+            "subreddit-asset register",
+            "subreddit-asset set-status",
+            "subreddit-asset moderator",
+            "subreddit-asset moderator add",
+            "subreddit-asset moderator revoke",
+            "subreddit-asset moderator replace",
+            "subreddit-asset protection",
+            "subreddit-asset protection get",
+            "subreddit-asset protection set",
+        ]));
+    });
+    /**
+     * 🔴 每一个**写**命令都必须有 `--dry-run` / `--apply` 双闸。
+     * 建一条绑定 = enforce 档下把一个账号的全部自动化永久停掉（无个体 override）；
+     * `protection set --mode enforce` 更是一次影响全部 20 个账号。
+     * 少一个闸就意味着有条路径能不经确认做出这种事。
+     */
+    it("所有写命令都带 --dry-run / --apply", () => {
+        const program = new Command();
+        registerSubredditAssetCommands(program);
+        const asset = program.commands.find((cmd) => cmd.name() === "subreddit-asset");
+        const writeLeaves = [];
+        const walk = (cmd, prefix) => {
+            for (const sub of cmd.commands) {
+                const next = [...prefix, sub.name()];
+                if (sub.commands.length > 0)
+                    walk(sub, next);
+                else
+                    writeLeaves.push(next.join(" "));
+            }
+        };
+        walk(asset, []);
+        const writes = [
+            "register-subreddit",
+            "register",
+            "set-status",
+            "moderator add",
+            "moderator revoke",
+            "moderator replace",
+            "protection set",
+        ];
+        for (const path of writes) {
+            expect(writeLeaves).toContain(path);
+            const parts = path.split(" ");
+            let cursor = asset;
+            for (const part of parts) {
+                cursor = cursor.commands.find((cmd) => cmd.name() === part);
+                expect(cursor).toBeDefined();
+            }
+            const flags = cursor.options.map((opt) => opt.long);
+            expect(flags).toContain("--dry-run");
+            expect(flags).toContain("--apply");
+        }
+    });
+    /** CAS 是这套接口的硬要求：改状态/撤销/更替/改档位都必须带 rowVersion。 */
+    it("所有 CAS 命令都把 --row-version 设为必填", () => {
+        const program = new Command();
+        registerSubredditAssetCommands(program);
+        const asset = program.commands.find((cmd) => cmd.name() === "subreddit-asset");
+        const casPaths = [
+            ["set-status"],
+            ["moderator", "revoke"],
+            ["moderator", "replace"],
+            ["protection", "set"],
+        ];
+        for (const parts of casPaths) {
+            let cursor = asset;
+            for (const part of parts) {
+                cursor = cursor.commands.find((cmd) => cmd.name() === part);
+            }
+            const rowVersion = cursor.options.find((opt) => opt.long === "--row-version");
+            expect(rowVersion, parts.join(" ")).toBeDefined();
+            expect(rowVersion.required || rowVersion.mandatory).toBe(true);
+        }
+    });
+    /** 读命令不该有写闸（否则 `list --apply` 这种误用会读起来像在写）。 */
+    it("读命令不带 --apply", () => {
+        const program = new Command();
+        registerSubredditAssetCommands(program);
+        const asset = program.commands.find((cmd) => cmd.name() === "subreddit-asset");
+        for (const name of ["list", "get"]) {
+            const cmd = asset.commands.find((c) => c.name() === name);
+            expect(cmd.options.map((opt) => opt.long)).not.toContain("--apply");
+        }
+    });
+});
+//# sourceMappingURL=register-subreddit-assets.test.js.map

package/dist/register-subreddit-assets.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"register-subreddit-assets.test.js","sourceRoot":"","sources":["../src/register-subreddit-assets.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AACpC,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EACL,wBAAwB,EACxB,8BAA8B,GAC/B,MAAM,gCAAgC,CAAC;AACxC,OAAO,EAAE,mBAAmB,EAAE,MAAM,mBAAmB,CAAC;AAExD,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;IACxC,EAAE,CAAC,eAAe,EAAE,GAAG,EAAE;QACvB,MAAM,CAAC,wBAAwB,CAAC,mBAAmB,CAAC,CAAC,CAAC,OAAO,CAAC;YAC5D,KAAK;YACL,OAAO;YACP,OAAO;SACR,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH;;;;;;OAMG;IACH,EAAE,CAAC,yBAAyB,EAAE,GAAG,EAAE;QACjC,MAAM,CAAC,wBAAwB,CAAC,SAAS,CAAC,CAAC,CAAC,aAAa,EAAE,CAAC;IAC9D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4BAA4B,EAAE,GAAG,EAAE;QACpC,MAAM,CAAC,wBAAwB,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QACjD,MAAM,CAAC,wBAAwB,CAAC,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,KAAK,EAAE,GAAG,EAAE;IACnB,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,MAAM,OAAO,GAAG,IAAI,OAAO,EAAE,CAAC;QAC9B,8BAA8B,CAAC,OAAO,CAAC,CAAC;QACxC,MAAM,KAAK,GAAG,mBAAmB,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QACtE,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CACnB,MAAM,CAAC,eAAe,CAAC;YACrB,iBAAiB;YACjB,sBAAsB;YACtB,qBAAqB;YACrB,oCAAoC;YACpC,0BAA0B;YAC1B,4BAA4B;YAC5B,2BAA2B;YAC3B,+BAA+B;YAC/B,kCAAkC;YAClC,mCAAmC;YACnC,4BAA4B;YAC5B,gCAAgC;YAChC,gCAAgC;SACjC,CAAC,CACH,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH;;;;;OAKG;IACH,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,OAAO,GAAG,IAAI,OAAO,EAAE,CAAC;QAC9B,8BAA8B,CAAC,OAAO,CAAC,CAAC;QACxC,MAAM,KAAK,GAAG,OAAO,CAAC,QAAQ,CAAC,IAAI,CACjC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,EAAE,KAAK,iBAAiB,CACzC,CAAC;QACH,MAAM,WAAW,GAAa,EAAE,CAAC;QACjC,MAAM,IAAI,GAAG,CAAC,GAAY,EAAE,MAAgB,EAAE,EAAE;YAC9C,KAAK,MAAM,GAAG,IAAI,GAAG,CAAC,QAAQ,EAAE,CAAC;gBAC/B,MAAM,IAAI,GAAG,CAAC,GAAG,MAAM,EAAE,GAAG,CAAC,IAAI,EAAE,CAAC,CAAC;gBACrC,IAAI,GAAG,CAAC,QAAQ,CAAC,MAAM,GAAG,CAAC;oBAAE,IAAI,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;;oBACxC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;YACxC,CAAC;QACH,CAAC,CAAC;QACF,IAAI,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;QAEhB,MAAM,MAAM,GAAG;YACb,oBAAoB;YACpB,UAAU;YACV,YAAY;YACZ,eAAe;YACf,kBAAkB;YAClB,mBAAmB;YACnB,gBAAgB;SACjB,CAAC;QACF,KAAK,MAAM,IAAI,IAAI,MAAM,EAAE,CAAC;YAC1B,MAAM,CAAC,WAAW,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;YACpC,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;YAC9B,IAAI,MAAM,GAAY,KAAK,CAAC;YAC5B,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;gBACzB,MAAM,GAAG,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,EAAE,KAAK,IAAI,CAAE,CAAC;gBAC7D,MAAM,CAAC,MAAM,CAAC,CAAC,WAAW,EAAE,CAAC;YAC/B,CAAC;YACD,MAAM,KAAK,GAAG,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YACpD,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC;YACrC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;QACrC,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,kDAAkD;IAClD,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;QACxC,MAAM,OAAO,GAAG,IAAI,OAAO,EAAE,CAAC;QAC9B,8BAA8B,CAAC,OAAO,CAAC,CAAC;QACxC,MAAM,KAAK,GAAG,OAAO,CAAC,QAAQ,CAAC,IAAI,CACjC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,EAAE,KAAK,iBAAiB,CACzC,CAAC;QACH,MAAM,QAAQ,GAAG;YACf,CAAC,YAAY,CAAC;YACd,CAAC,WAAW,EAAE,QAAQ,CAAC;YACvB,CAAC,WAAW,EAAE,SAAS,CAAC;YACxB,CAAC,YAAY,EAAE,KAAK,CAAC;SACtB,CAAC;QACF,KAAK,MAAM,KAAK,IAAI,QAAQ,EAAE,CAAC;YAC7B,IAAI,MAAM,GAAY,KAAK,CAAC;YAC5B,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;gBACzB,MAAM,GAAG,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,EAAE,KAAK,IAAI,CAAE,CAAC;YAC/D,CAAC;YACD,MAAM,UAAU,GAAG,MAAM,CAAC,OAAO,CAAC,IAAI,CACpC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,KAAK,eAAe,CACtC,CAAC;YACF,MAAM,CAAC,UAAU,EAAE,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;YAClD,MAAM,CAAC,UAAW,CAAC,QAAQ,IAAI,UAAW,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACnE,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,+CAA+C;IAC/C,EAAE,CAAC,eAAe,EAAE,GAAG,EAAE;QACvB,MAAM,OAAO,GAAG,IAAI,OAAO,EAAE,CAAC;QAC9B,8BAA8B,CAAC,OAAO,CAAC,CAAC;QACxC,MAAM,KAAK,GAAG,OAAO,CAAC,QAAQ,CAAC,IAAI,CACjC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,EAAE,KAAK,iBAAiB,CACzC,CAAC;QACH,KAAK,MAAM,IAAI,IAAI,CAAC,MAAM,EAAE,KAAK,CAAC,EAAE,CAAC;YACnC,MAAM,GAAG,GAAG,KAAK,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,KAAK,IAAI,CAAE,CAAC;YAC3D,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;QACtE,CAAC;IACH,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geoly-ai/social-hub-cli",
-  "version": "0.3.24",
+  "version": "0.3.26",
   "type": "module",
   "description": "social-hub CLI for Social Ops Hub",
   "repository": {
@@ -22,8 +22,8 @@
   },
   "dependencies": {
     "commander": "^12.1.0",
-    "@geoly-ai/social-hub-authz": "0.0.18",
-    "@geoly-ai/social-hub-sdk": "0.1.1"
+    "@geoly-ai/social-hub-sdk": "0.1.2",
+    "@geoly-ai/social-hub-authz": "0.0.19"
   },
   "devDependencies": {
     "@types/node": "^22.10.2",

package/skills/manifest.json

@@ -1,7 +1,7 @@
 {
   "version": 1,
   "package": "social-ops-hub",
-  "cliVersion": "0.3.24",
+  "cliVersion": "0.3.26",
   "repository": "social-ops-hub",
   "publish": {
     "registry": "https://skill.sh",

package/skills/social-hub-content-review/SKILL.md

@@ -176,6 +176,9 @@ social-hub comment-drafts get    -t <team-id> --draft <uuid>
 # 排期前 threadPermalink 必填（可 create 就带，或后续 update 补）：
 social-hub comment-drafts create -t <team-id> \
   -j '{ "brandId":"<uuid>", "body":"<评论>", "subreddit":"buildapc", "threadPermalink":"https://reddit.com/r/buildapc/comments/xxx" }'
+# 无品牌评论:省略 brandId 必须配显式 brandMention（"忘了传" ≠ "明确没有品牌意图"）
+social-hub comment-drafts create -t <team-id> \
+  -j '{ "brandMention":"none", "body":"<评论>", "subreddit":"buildapc" }'
 social-hub comment-drafts update -t <team-id> --draft <uuid> -j '{ "body":"...", "expectedVersion": 2 }'
 social-hub comment-drafts delete -t <team-id> --draft <uuid>
 social-hub comment-drafts schedule -t <team-id> --draft <uuid> \
@@ -190,6 +193,13 @@ social-hub comment-drafts schedule -t <team-id> --draft <uuid> \
   complete 系统驱动,不可客户端直设(直设会撞 DB CHECK 或与真实 job 漂移)。
 - **排期前 `threadPermalink` 必填**:草稿没选帖(无 threadPermalink)直接 `schedule` 会被拒——
   先在 `create` 带上或用 `update` 补 `threadPermalink` 再排期。
+- **无品牌评论草稿**(与帖子侧同一套语义):省略 `brandId` **必须**同时传 `brandMention: "none"`,
+  两者同现(或与 `campaignId` 同现)一律 400 ——「忘了传」不等于「明确没有品牌意图」。
+  服务端据此落 `brand_intent = 'declared_none'`,把它与**存量 / 历史 bug 落成的 NULL** 区分开;
+  **存量 `brand_intent IS NULL` 的行对品牌受限主体一律不可见**。品牌受限主体读/建/改/排期
+  这类草稿分别要 `noBrandContent.read/create/update` 与 `noBrandContentSchedule.create`
+  四格(删除**没有**对应格子,照旧 404);不受限主体行为不变。声明了无品牌的正文还会在
+  **创建、改 body、排期**三处过一次「正文里不得出现已登记品牌」的确定性核验。
 - **`comment-drafts update` 的 `expectedVersion`** 开乐观并发:版本不匹配 → 409;version 只在
   内容字段编辑时 +1——`body/subreddit/threadPermalink/parentCommentId/threadContext` 任一变更都
   bump,**只有纯 `status` 更新不 bump**。