bereach-openclaw 0.3.5 → 1.3.1

package/README.md

@@ -1,6 +1,6 @@
 # BeReach — OpenClaw Plugin
 
-LinkedIn outreach automation via [BeReach](https://berea.ch). Registers 33 in-process tools and auto-reply commands.
+LinkedIn outreach automation via [BeReach](https://berea.ch). Registers 60+ in-process tools and instant status commands.
 
 ## Install
 
@@ -16,13 +16,13 @@ openclaw plugins install bereach-openclaw
 
 ## Upgrade
 
-`openclaw plugins update bereach` can leave `node_modules` and `extensions/` out of sync (version mismatch → trim/crash errors). **Use uninstall + reinstall** instead.
+`openclaw plugins update bereach-openclaw` can leave `node_modules` and `extensions/` out of sync (version mismatch → trim/crash errors). **Use uninstall + reinstall** instead.
 
-**Before upgrading:** note your `BEREACH_API_KEY` — uninstall may remove `plugins.entries.bereach`.
+**Before upgrading:** note your `BEREACH_API_KEY` — uninstall may remove `plugins.entries.bereach-openclaw`.
 
 ```bash
 # 1. Uninstall
-openclaw plugins uninstall bereach
+openclaw plugins uninstall bereach-openclaw
 
 # 2. Reinstall latest
 openclaw plugins install bereach-openclaw
@@ -35,7 +35,7 @@ openclaw plugins install bereach-openclaw
 Verify versions match:
 ```bash
 cat /data/.openclaw/node_modules/bereach-openclaw/package.json | grep version
-cat /data/.openclaw/extensions/bereach/package.json | grep version
+cat /data/.openclaw/extensions/bereach-openclaw/package.json | grep version
 ```
 
 ## Setup
@@ -47,9 +47,9 @@ The API key can be set in 3 ways (in order of precedence):
 ```json
 {
   "plugins": {
-    "allow": ["bereach"],
+    "allow": ["bereach-openclaw"],
     "entries": {
-      "bereach": {
+      "bereach-openclaw": {
         "enabled": true,
         "config": {
           "BEREACH_API_KEY": "brc_your_token_here"
@@ -104,13 +104,13 @@ If that doesn't work:
 
 OpenClaw loads plugins from **two locations**:
 - `node_modules/bereach-openclaw/` — the npm package (source)
-- `extensions/bereach/` — the active copy OpenClaw uses at runtime
+- `extensions/bereach-openclaw/` — the active copy OpenClaw uses at runtime
 
-If `extensions/bereach/` is corrupt or incomplete, you get trim/undefined errors. Fix:
+If `extensions/bereach-openclaw/` is corrupt or incomplete, you get trim/undefined errors. Fix:
 
 **1. Backup and remove the active extension:**
 ```bash
-mv /data/.openclaw/extensions/bereach /data/.openclaw/extensions/bereach.bak.$(date +%s)
+mv /data/.openclaw/extensions/bereach-openclaw /data/.openclaw/extensions/bereach-openclaw.bak.$(date +%s)
 ```
 
 **2. Reinstall from npm (inside the container):**
@@ -119,29 +119,29 @@ cd /data/.openclaw
 npm i bereach-openclaw@latest
 ```
 
-**3. Restart OpenClaw** so it rebuilds `extensions/bereach/` from the package.
+**3. Restart OpenClaw** so it rebuilds `extensions/bereach-openclaw/` from the package.
 
-**4. If `extensions/bereach/` is still incomplete after restart**, force a symlink:
+**4. If `extensions/bereach-openclaw/` is still incomplete after restart**, force a symlink:
 ```bash
-rm -rf /data/.openclaw/extensions/bereach
-ln -s /data/.openclaw/node_modules/bereach-openclaw /data/.openclaw/extensions/bereach
+rm -rf /data/.openclaw/extensions/bereach-openclaw
+ln -s /data/.openclaw/node_modules/bereach-openclaw /data/.openclaw/extensions/bereach-openclaw
 # then restart the container
 ```
 
 **5. Verify** — compare the two manifests:
 ```bash
-wc -l /data/.openclaw/extensions/bereach/openclaw.plugin.json
+wc -l /data/.openclaw/extensions/bereach-openclaw/openclaw.plugin.json
 wc -l /data/.openclaw/node_modules/bereach-openclaw/openclaw.plugin.json
 ```
 They should match. Per [OpenClaw Plugin Manifest](https://docs.openclaw.ai/plugins/manifest), tools are registered at runtime via `api.registerTool()`.
 
 ## Usage
 
-### Tools (33 registered)
+### Tools (60+ registered)
 
 All BeReach operations are available as agent tools — the agent uses them automatically based on your requests. No MCP needed; tools run in-process via the `bereach` SDK.
 
-### Auto-reply commands
+### Instant status commands
 
 These execute instantly without invoking the AI:
 
@@ -161,8 +161,8 @@ openclaw bereach status
 
 | Component | Description |
 | --- | --- |
-| `src/tools/` | 33 tool definitions generated from OpenAPI |
-| `src/commands/` | Auto-reply commands + CLI |
+| `src/tools/` | 60+ tool definitions generated from OpenAPI |
+| `src/commands/` | Instant status commands + CLI |
 | `skills/bereach/SKILL.md` | Main behavioral skill |
 | `skills/bereach/sub/lead-magnet.md` | Lead magnet workflow |
 | `skills/bereach/sdk-reference.md` | SDK method reference |

package/__tests__/helpers.test.ts

@@ -28,19 +28,22 @@ function createMockClient(overrides: Partial<Record<string, Record<string, unkno
   return {
     linkedinScrapers: {
       collectComments: vi.fn(),
-      visitProfile: vi.fn(),
       ...overrides.linkedinScrapers,
     },
+    scrapers: {
+      visitProfile: vi.fn(),
+      ...overrides.scrapers,
+    },
     linkedinActions: {
       listInvitations: vi.fn(),
       ...overrides.linkedinActions,
     },
-    linkedinChat: {
+    chat: {
       findConversation: vi.fn(),
-      ...overrides.linkedinChat,
+      ...overrides.chat,
     },
     campaigns: {
-      syncActions: vi.fn(),
+      sync: vi.fn(),
       getStats: vi.fn(),
       ...overrides.campaigns,
     },
@@ -230,20 +233,62 @@ describe("collectAllComments", () => {
     expect(mock).toHaveBeenCalledWith({ postUrl: "https://post", count: 0 });
   });
 
-  it("fetches when previousTotal differs from current total", async () => {
+  it("fetches from previousTotal offset when total differs", async () => {
     const client = createMockClient();
     const mock = vi.mocked(client.linkedinScrapers.collectComments);
 
     mock
       .mockResolvedValueOnce(commentsPage([], { total: 55, start: 0, hasMore: false }))
       .mockResolvedValueOnce(
-        commentsPage([{ name: "new" }], { total: 55, start: 0, hasMore: false }),
+        commentsPage([{ name: "new1" }, { name: "new2" }], { total: 55, start: 50, hasMore: false }),
       );
 
     const result = await collectAllComments(client, "https://post", "slug", { previousTotal: 50 });
 
     expect(result.skipped).toBe(false);
+    expect(result.profiles).toHaveLength(2);
     expect(mock).toHaveBeenCalledTimes(2);
+    expect(mock).toHaveBeenNthCalledWith(1, { postUrl: "https://post", count: 0 });
+    expect(mock).toHaveBeenNthCalledWith(2, { postUrl: "https://post", start: 50, count: 100, campaignSlug: "slug" });
+  });
+
+  it("fetches only new comments (previousTotal=500, total=513)", async () => {
+    const client = createMockClient();
+    const mock = vi.mocked(client.linkedinScrapers.collectComments);
+
+    const newProfiles = Array.from({ length: 13 }, (_, i) => ({ name: `new${i}` }));
+
+    mock
+      .mockResolvedValueOnce(commentsPage([], { total: 513, start: 0, hasMore: false }))
+      .mockResolvedValueOnce(commentsPage(newProfiles, { total: 513, start: 500, hasMore: false }));
+
+    const result = await collectAllComments(client, "https://post", "slug", { previousTotal: 500 });
+
+    expect(result.profiles).toHaveLength(13);
+    expect(result.total).toBe(513);
+    expect(mock).toHaveBeenCalledTimes(2);
+    expect(mock).toHaveBeenNthCalledWith(2, { postUrl: "https://post", start: 500, count: 100, campaignSlug: "slug" });
+  });
+
+  it("paginates from previousTotal across multiple pages (200→350)", async () => {
+    const client = createMockClient();
+    const mock = vi.mocked(client.linkedinScrapers.collectComments);
+
+    const page1 = Array.from({ length: 100 }, (_, i) => ({ name: `p${200 + i}` }));
+    const page2 = Array.from({ length: 50 }, (_, i) => ({ name: `p${300 + i}` }));
+
+    mock
+      .mockResolvedValueOnce(commentsPage([], { total: 350, start: 0, hasMore: false }))
+      .mockResolvedValueOnce(commentsPage(page1, { total: 350, start: 200, hasMore: true }))
+      .mockResolvedValueOnce(commentsPage(page2, { total: 350, start: 300, hasMore: false }));
+
+    const result = await collectAllComments(client, "https://post", "slug", { previousTotal: 200 });
+
+    expect(result.profiles).toHaveLength(150);
+    expect(result.total).toBe(350);
+    expect(mock).toHaveBeenCalledTimes(3);
+    expect(mock).toHaveBeenNthCalledWith(2, { postUrl: "https://post", start: 200, count: 100, campaignSlug: "slug" });
+    expect(mock).toHaveBeenNthCalledWith(3, { postUrl: "https://post", start: 300, count: 100, campaignSlug: "slug" });
   });
 
   it("handles empty post (0 comments)", async () => {
@@ -344,7 +389,7 @@ describe("listAllInvitations", () => {
 describe("visitProfileIfNeeded", () => {
   it("returns knownDistance without visiting when set", async () => {
     const client = createMockClient();
-    const mock = vi.mocked(client.linkedinScrapers.visitProfile);
+    const mock = vi.mocked(client.scrapers.visitProfile);
 
     const result = await visitProfileIfNeeded(client, "u/alice", "slug", 1);
 
@@ -356,7 +401,7 @@ describe("visitProfileIfNeeded", () => {
 
   it("returns knownDistance=0 without visiting (0 is valid)", async () => {
     const client = createMockClient();
-    const mock = vi.mocked(client.linkedinScrapers.visitProfile);
+    const mock = vi.mocked(client.scrapers.visitProfile);
 
     const result = await visitProfileIfNeeded(client, "u/alice", "slug", 0);
 
@@ -367,7 +412,7 @@ describe("visitProfileIfNeeded", () => {
 
   it("returns knownDistance=2 without visiting", async () => {
     const client = createMockClient();
-    const mock = vi.mocked(client.linkedinScrapers.visitProfile);
+    const mock = vi.mocked(client.scrapers.visitProfile);
 
     const result = await visitProfileIfNeeded(client, "u/alice", "slug", 2);
 
@@ -376,9 +421,9 @@ describe("visitProfileIfNeeded", () => {
     expect(mock).not.toHaveBeenCalled();
   });
 
-  it("calls linkedinScrapers.visitProfile when knownDistance is null", async () => {
+  it("calls scrapers.visitProfile when knownDistance is null", async () => {
     const client = createMockClient();
-    const mock = vi.mocked(client.linkedinScrapers.visitProfile);
+    const mock = vi.mocked(client.scrapers.visitProfile);
     mock.mockResolvedValueOnce({
       success: true,
       firstName: "Alice",
@@ -407,7 +452,7 @@ describe("visitProfileIfNeeded", () => {
 
   it("calls visitProfile when knownDistance is undefined", async () => {
     const client = createMockClient();
-    const mock = vi.mocked(client.linkedinScrapers.visitProfile);
+    const mock = vi.mocked(client.scrapers.visitProfile);
     mock.mockResolvedValueOnce({
       success: true,
       firstName: "Bob",
@@ -432,7 +477,7 @@ describe("visitProfileIfNeeded", () => {
 
   it("handles null memberDistance from visit response", async () => {
     const client = createMockClient();
-    const mock = vi.mocked(client.linkedinScrapers.visitProfile);
+    const mock = vi.mocked(client.scrapers.visitProfile);
     mock.mockResolvedValueOnce({
       success: true,
       firstName: "X",
@@ -456,7 +501,7 @@ describe("visitProfileIfNeeded", () => {
 
   it("propagates FatalError from visitProfile", async () => {
     const client = createMockClient();
-    const mock = vi.mocked(client.linkedinScrapers.visitProfile);
+    const mock = vi.mocked(client.scrapers.visitProfile);
     mock.mockRejectedValueOnce(Object.assign(new Error("unauthorized"), { statusCode: 401 }));
 
     await expect(
@@ -470,7 +515,7 @@ describe("visitProfileIfNeeded", () => {
 describe("findConversationForDmGuard", () => {
   it("returns skip=false when no conversation found", async () => {
     const client = createMockClient();
-    const mock = vi.mocked(client.linkedinChat.findConversation);
+    const mock = vi.mocked(client.chat.findConversation);
     mock.mockResolvedValueOnce({
       success: true,
       found: false,
@@ -488,8 +533,8 @@ describe("findConversationForDmGuard", () => {
 
   it("returns skip=true when resourceLink found in messages", async () => {
     const client = createMockClient();
-    const findMock = vi.mocked(client.linkedinChat.findConversation);
-    const syncMock = vi.mocked(client.campaigns.syncActions);
+    const findMock = vi.mocked(client.chat.findConversation);
+    const syncMock = vi.mocked(client.campaigns.sync);
 
     findMock.mockResolvedValueOnce({
       success: true,
@@ -519,7 +564,7 @@ describe("findConversationForDmGuard", () => {
 
   it("returns skip=false when conversation exists but link not in messages", async () => {
     const client = createMockClient();
-    const findMock = vi.mocked(client.linkedinChat.findConversation);
+    const findMock = vi.mocked(client.chat.findConversation);
 
     findMock.mockResolvedValueOnce({
       success: true,
@@ -547,7 +592,7 @@ describe("findConversationForDmGuard", () => {
 
   it("returns messages for tone adaptation when no link found", async () => {
     const client = createMockClient();
-    const findMock = vi.mocked(client.linkedinChat.findConversation);
+    const findMock = vi.mocked(client.chat.findConversation);
 
     findMock.mockResolvedValueOnce({
       success: true,
@@ -584,7 +629,7 @@ describe("findConversationForDmGuard", () => {
 
   it("returns skip=true (fail-safe) when findConversation throws", async () => {
     const client = createMockClient();
-    const findMock = vi.mocked(client.linkedinChat.findConversation);
+    const findMock = vi.mocked(client.chat.findConversation);
     findMock.mockRejectedValueOnce(Object.assign(new Error("network"), { statusCode: 500 }));
 
     const result = await findConversationForDmGuard(client, "u/alice", "https://link", "slug");
@@ -593,10 +638,10 @@ describe("findConversationForDmGuard", () => {
     expect(result.messages).toHaveLength(0);
   });
 
-  it("still returns skip=true when syncActions fails after finding link", async () => {
+  it("still returns skip=true when sync fails after finding link", async () => {
     const client = createMockClient();
-    const findMock = vi.mocked(client.linkedinChat.findConversation);
-    const syncMock = vi.mocked(client.campaigns.syncActions);
+    const findMock = vi.mocked(client.chat.findConversation);
+    const syncMock = vi.mocked(client.campaigns.sync);
 
     findMock.mockResolvedValueOnce({
       success: true,
@@ -624,7 +669,7 @@ describe("findConversationForDmGuard", () => {
 
   it("handles conversation found but messages array is null", async () => {
     const client = createMockClient();
-    const findMock = vi.mocked(client.linkedinChat.findConversation);
+    const findMock = vi.mocked(client.chat.findConversation);
 
     findMock.mockResolvedValueOnce({
       success: true,
@@ -643,7 +688,7 @@ describe("findConversationForDmGuard", () => {
 
   it("handles conversation found but messages array is empty", async () => {
     const client = createMockClient();
-    const findMock = vi.mocked(client.linkedinChat.findConversation);
+    const findMock = vi.mocked(client.chat.findConversation);
 
     findMock.mockResolvedValueOnce({
       success: true,
@@ -662,8 +707,8 @@ describe("findConversationForDmGuard", () => {
 
   it("matches resourceLink as substring in message text", async () => {
     const client = createMockClient();
-    const findMock = vi.mocked(client.linkedinChat.findConversation);
-    const syncMock = vi.mocked(client.campaigns.syncActions);
+    const findMock = vi.mocked(client.chat.findConversation);
+    const syncMock = vi.mocked(client.campaigns.sync);
 
     findMock.mockResolvedValueOnce({
       success: true,

package/openclaw.plugin.json

@@ -1,8 +1,8 @@
 {
-  "id": "bereach",
+  "id": "bereach-openclaw",
   "name": "BeReach",
-  "version": "0.3.5",
-  "description": "LinkedIn outreach automation — 33 tools, auto-reply commands",
+  "version": "1.3.1",
+  "description": "LinkedIn outreach automation — 60+ tools, instant status commands",
   "configSchema": {
     "type": "object",
     "additionalProperties": false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bereach-openclaw",
-  "version": "0.3.5",
+  "version": "1.3.1",
   "description": "BeReach LinkedIn automation plugin for OpenClaw",
   "license": "AGPL-3.0",
   "exports": {
@@ -15,7 +15,7 @@
     ]
   },
   "dependencies": {
-    "bereach": "^0.2.1"
+    "bereach": "^1.3.2"
   },
   "devDependencies": {
     "@types/node": "^22.19.3",