@floomhq/signaldash 0.22.0 → 0.28.0

package/lib/cli.js

@@ -290,7 +290,7 @@ export async function connectCommand(channel, options, dependencies = {}) {
 }
 
 function registrationLine() {
-  return "Register: claude mcp add signaldash -- npx -y @floomhq/signaldash mcp | Cursor command: npx -y @floomhq/signaldash mcp";
+  return "Register: claude mcp add signaldash -s user -- npx -y @floomhq/signaldash mcp | Cursor command: npx -y @floomhq/signaldash mcp";
 }
 
 export async function mcpCommand(options, dependencies = {}) {

package/lib/mcp.js

@@ -127,7 +127,7 @@ function cleanMessage(message) {
     id: message.id ?? message.provider_id ?? null,
     text: message.text ?? null,
     timestamp: message.timestamp ?? null,
-    is_sender: Boolean(message.is_sender),
+    is_sender: providerIsOwn(message.is_sender),
     sender_id: message.sender_id ?? null,
     attachments: (message.attachments || []).map((attachment) => ({
       id: attachment.id ?? null,
@@ -140,6 +140,16 @@ function cleanMessage(message) {
   };
 }
 
+function providerIsOwn(value) {
+  if (typeof value === "boolean") return value;
+  if (typeof value === "number") return value !== 0;
+  if (typeof value === "string") {
+    const normalized = value.trim().toLowerCase();
+    return ["1", "true", "yes", "y"].includes(normalized);
+  }
+  return false;
+}
+
 function itemList(payload) {
   return Array.isArray(payload?.items)
     ? payload.items
@@ -207,7 +217,7 @@ async function executeTool({ name, args, config, client, guard }) {
         const recent = itemList(await client.listMessages(chatId, 20));
         const duplicate = recent.find(
           (message) =>
-            message.is_sender &&
+            providerIsOwn(message.is_sender) &&
             typeof message.text === "string" &&
             message.text.trim() === text,
         );

package/lib/skill-template.cjs

@@ -0,0 +1,32 @@
+'use strict';
+
+// Single source of truth for substituting the skill template's version
+// placeholder. Three call sites independently need this exact substitution:
+// `cmdSkill` in bin/sd.mjs (installs the skill into ~/.claude/skills), the
+// public signaldash.dev page renderer (ops/render-skill-page.mjs and
+// ops/render-skill-markdown.mjs), and the server's own /skill + /skill.md
+// routes (server/server.cjs). Before this file existed the substitution was
+// hand-duplicated in bin/sd.mjs and ops/render-skill-page.mjs, and the
+// server routes and ops/sync-public-skill.sh did not substitute at all --
+// which is how the publicly served skill ended up telling agents to run
+// `npx -y @floomhq/signaldash@{{PACKAGE_VERSION}} <invite-code>`, a command
+// that does not exist.
+//
+// .cjs so both ESM (bin/sd.mjs, ops/*.mjs, via Node's CJS/ESM interop) and
+// CommonJS (server/server.cjs) call sites can require/import it without a
+// build step or a second copy.
+
+const PACKAGE_VERSION_PLACEHOLDER = '{{PACKAGE_VERSION}}';
+
+function renderSkillTemplate(rawMarkdown, packageVersion) {
+  if (typeof rawMarkdown !== 'string') {
+    throw new TypeError('renderSkillTemplate: rawMarkdown must be a string');
+  }
+  if (typeof packageVersion !== 'string' || !packageVersion) {
+    throw new TypeError('renderSkillTemplate: packageVersion must be a non-empty string');
+  }
+  return rawMarkdown.split(PACKAGE_VERSION_PLACEHOLDER).join(packageVersion);
+}
+
+exports.renderSkillTemplate = renderSkillTemplate;
+exports.PACKAGE_VERSION_PLACEHOLDER = PACKAGE_VERSION_PLACEHOLDER;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@floomhq/signaldash",
-  "version": "0.22.0",
+  "version": "0.28.0",
   "description": "Secure LinkedIn, WhatsApp, and email access for AI agents",
   "type": "module",
   "bin": {
@@ -18,7 +18,7 @@
   },
   "scripts": {
     "test": "node --test",
-    "check": "node --check bin/sd.mjs && node --check server/server.cjs && node --check server/write-control.cjs && node --check bin/signaldash.js && node --check lib/cli.js && node --check lib/config-file.js && node --check lib/mcp.js && node --check lib/rate-guard.js && node --check lib/secrets.js && node --check lib/unipile.js"
+    "check": "node --check bin/sd.mjs && node --check server/server.cjs && node --check server/composio-email.cjs && node --check server/attachments.cjs && node --check server/write-control.cjs && node --check server/voice-profile.cjs && node --check server/campaign-store.cjs && node --check server/campaign-runner.cjs && node --check server/withdrawal-store.cjs && node --check server/post-scheduler.cjs && node --check server/message-scheduler.cjs && node --check bin/signaldash.js && node --check lib/cli.js && node --check lib/config-file.js && node --check lib/mcp.js && node --check lib/rate-guard.js && node --check lib/secrets.js && node --check lib/unipile.js"
   },
   "keywords": [
     "mcp",
@@ -34,4 +34,4 @@
     "ora": "8.0.1",
     "open": "10.1.0"
   }
-}
+}