@cometchat/skills 4.2.1 → 4.4.0

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026 CometChat
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/README.md

@@ -74,6 +74,21 @@ Then in your IDE:
 /cometchat add chat to my app
 ```
 
+Or add voice & video calling (new in v4.2):
+
+```
+/cometchat add voice and video calls to my app
+```
+
+When integrating chat, the dispatcher now asks **how** you want to customize (new in v4.3):
+
+```
+◉ Visually — drag-and-drop in browser
+○ In code — code-driven defaults
+```
+
+Pick **Visually** and the dispatcher opens CometChat's Visual Builder in your browser, waits while you customize colors / layout / features, then fetches the config and emits the integration into your existing app — no ZIP downloads, no manual file copying. Works for React, React Native, iOS, Android v6, and Flutter v6. Angular customers auto-route to the In-code path (Visual Builder doesn't ship Angular code yet).
+
 ## What happens
 
 1. **Detects** your framework (React / Next.js / React Router / Astro / Expo / bare React Native / Angular / Android / Flutter / iOS), router, env prefix, existing auth system
@@ -87,18 +102,24 @@ No templates, no experiences to pick — the agent writes real code that fits yo
 
 ## Supported frameworks
 
-| Framework | Status |
-|---|---|
-| React.js / Vite / CRA | ✅ (`@cometchat/skills`) |
-| Next.js (App Router + Pages Router) | ✅ (`@cometchat/skills`) |
-| React Router v6 / v7 | ✅ (`@cometchat/skills`) |
-| Astro (React islands) | ✅ (`@cometchat/skills`) |
-| Expo (managed + Expo Router) | ✅ (`@cometchat/skills`) |
-| Bare React Native (CLI) | ✅ (`@cometchat/skills`) |
-| Angular 12-15 | ✅ (`@cometchat/skills`) |
-| Android (V5 stable + V6 beta) | ✅ (`@cometchat/skills`) |
-| Flutter (V5 stable + V6 beta) | ✅ (`@cometchat/skills`) |
-| iOS (V5 stable) | ✅ (`@cometchat/skills`) |
+| Framework | Chat | Voice & Video Calls | Visual Builder |
+|---|---|---|---|
+| React.js / Vite / CRA | ✅ | ✅ Ringing + Session | ✅ (v4.3) |
+| Next.js (App Router + Pages Router) | ✅ | ✅ Ringing + Session | ✅ (v4.3) |
+| React Router v6 / v7 | ✅ | ✅ Ringing + Session | ✅ (v4.3) |
+| Astro (React islands) | ✅ | ✅ Ringing + Session | ✅ (v4.3) |
+| Expo (managed + Expo Router) | ✅ | ✅ Ringing + Session | ✅ (v4.3) |
+| Bare React Native (CLI) | ✅ | ✅ Ringing + Session | ✅ (v4.3) |
+| Angular 12-15 | ✅ | ✅ Ringing + Session | ❌ (auto-falls back to In code) |
+| Android (V5 stable + V6 beta) | ✅ | ✅ Ringing + Session | ✅ V6 (v4.3) |
+| Flutter (V5 stable + V6 beta) | ✅ | ✅ Ringing + Session | ✅ V6 chat-only (v4.3) |
+| iOS (V5 stable) | ✅ | ✅ Ringing + Session | ⏸ Pending Xcode 26 vendor fix |
+
+**Calling modes:**
+- **Ringing** — kit-driven incoming/outgoing call surfaces, system-level VoIP push (CallKit on iOS, ConnectionService on Android, web push fallback on browsers). Production-grade for 1:1 + group calls.
+- **Session** — both peers join a shared `/meet/:sessionId` URL. No ringing. For embedded meetings, scheduled calls, support flows, broadcast use-cases.
+
+The dispatcher asks Ringing vs Session up front in Step 3.0.
 
 ## After the first integration
 

package/bin/install.js

@@ -652,11 +652,39 @@ async function delegateToSkillsCli({ skills, families, isGlobal }) {
   ];
   if (isGlobal) npxArgs.push("-g");
 
+  // F42 (2026-05-22) + F79 (2026-05-28): on Windows, `npx` is `npx.cmd` /
+  // `npx.ps1` and Node's spawn can't resolve the shim cleanly. The original
+  // F42 fix used `shell: true`, but that still fails in some Windows envs
+  // (nested npx, PowerShell execution-policy restrictions) with
+  // `spawn npx ENOENT` — reported by a customer on 2026-05-28.
+  //
+  // F79: on Windows, resolve npm's bundled `npx-cli.js` next to the running
+  // Node binary and invoke it directly via `process.execPath`. This bypasses
+  // the .cmd/.ps1 shim entirely. Falls back to the F42 `shell: true` path if
+  // npx-cli.js can't be located (pnpm / yarn / custom Node installs without
+  // a bundled npm). macOS/Linux keep the plain `spawn("npx")` path unchanged.
+  const isWindows = process.platform === "win32";
+
+  let spawnCmd = "npx";
+  let spawnArgs = npxArgs;
+  let spawnOpts = { stdio: "inherit" };
+
+  if (isWindows) {
+    // npm ships npx-cli.js at <node-dir>/node_modules/npm/bin/npx-cli.js
+    const nodeDir = path.dirname(process.execPath);
+    const npxCli = path.join(nodeDir, "node_modules", "npm", "bin", "npx-cli.js");
+    if (fs.existsSync(npxCli)) {
+      spawnCmd = process.execPath;          // the current node binary
+      spawnArgs = [npxCli, ...npxArgs];
+      spawnOpts = { stdio: "inherit" };     // no shell needed — direct node invocation
+    } else {
+      // Fallback: F42 path (shim via shell). Covers pnpm/yarn/custom installs.
+      spawnOpts = { stdio: "inherit", shell: true };
+    }
+  }
+
   return new Promise((resolve) => {
-    const child = spawn("npx", npxArgs, {
-      stdio: "inherit",
-      shell: false,
-    });
+    const child = spawn(spawnCmd, spawnArgs, spawnOpts);
     child.on("close", (code) => {
       if (code === 0) {
         // Post-spawn success: celebrate + concrete next steps
@@ -895,6 +923,33 @@ async function main() {
     console.log(`\n  ${c.bold(c.cyan("CometChat Skills"))}  —  ${ide}  —  ${c.bold(familyLabel)} family\n`);
     console.log(`  Scope: ${scopeLabel}\n`);
 
+    // ENG-35720: warn if cometchat-* skills already exist at a DIFFERENT IDE target.
+    // Reading skills from two paths (e.g. `.agents/skills/` + `.kiro/skills/`) works
+    // for agents that scan both (Kiro), but is fragile and confusing. Surface the
+    // mismatch so the integrator can clean up the older path.
+    if (!isGlobal) {
+      const conflictingPaths = Object.entries(IDE_TARGETS)
+        .filter(([otherIde, otherTarget]) =>
+          otherIde !== ide &&
+          otherTarget.mode === "multi" &&
+          otherTarget.local !== target.local
+        )
+        .map(([otherIde, otherTarget]) => ({ ide: otherIde, path: path.join(process.cwd(), otherTarget.local) }))
+        .filter(({ path: p }) => {
+          try {
+            if (!fs.existsSync(p)) return false;
+            return fs.readdirSync(p).some(d => d.startsWith("cometchat-"));
+          } catch { return false; }
+        });
+      if (conflictingPaths.length > 0) {
+        console.log(`  ${c.yellow("⚠")} Existing CometChat skills found at:`);
+        for (const { ide: oIde, path: p } of conflictingPaths) {
+          console.log(`     ${c.dim("•")} ${p}/  (from --ide ${oIde})`);
+        }
+        console.log(`  ${c.dim("Multi-path skill installs work for IDEs that scan all locations (e.g. Kiro), but are fragile.")}\n  ${c.dim("Consider removing the older path or staying on a single --ide flag across re-runs.\n")}`);
+      }
+    }
+
     ensureDir(baseDir);
 
     // --clean: wipe existing cometchat-* skill dirs in baseDir before install.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cometchat/skills",
-  "version": "4.2.1",
+  "version": "4.4.0",
   "publishConfig": {
     "access": "public"
   },
@@ -48,6 +48,19 @@
   },
   "scripts": {
     "setup": "node bin/install.js add",
+    "catalog:extract": "node test-suite/scripts/extract-catalog.mjs",
+    "test:skill-content": "node --test test-suite/tests/skill-content.test.ts",
+    "verify:fences:web": "npm --prefix test-suite/typecheck/web ci --no-audit --no-fund && node test-suite/scripts/typecheck-fences.mjs --project test-suite/typecheck/web --tsconfig tsconfig.json --discover web",
+    "verify:fences:rn": "npm --prefix test-suite/typecheck/rn ci --no-audit --no-fund && node test-suite/scripts/typecheck-fences.mjs --project test-suite/typecheck/rn --tsconfig tsconfig.json --discover rn",
+    "verify:fences:angular": "npm --prefix test-suite/typecheck/angular ci --no-audit --no-fund && node test-suite/scripts/typecheck-fences.mjs --project test-suite/typecheck/angular --tsconfig tsconfig.json --discover angular",
+    "verify:fences:flutter-v6": "(cd test-suite/typecheck/flutter-v6 && flutter pub get) && node test-suite/scripts/typecheck-fences-dart.mjs --project test-suite/typecheck/flutter-v6 --discover flutter-v6",
+    "verify:fences:flutter-v5": "(cd test-suite/typecheck/flutter-v5 && flutter pub get) && node test-suite/scripts/typecheck-fences-dart.mjs --project test-suite/typecheck/flutter-v5 --discover flutter-v5",
+    "verify:fences:ios": "node test-suite/scripts/typecheck-fences-swift.mjs --discover ios",
+    "verify:fences:android-v5": "node test-suite/scripts/typecheck-fences-kotlin.mjs --project test-suite/typecheck/android --module v5 --discover android-v5",
+    "verify:fences:android-v6": "node test-suite/scripts/typecheck-fences-kotlin.mjs --project test-suite/typecheck/android --module v6 --discover android-v6",
+    "verify:fences:android-v6-compose": "node test-suite/scripts/typecheck-fences-kotlin.mjs --project test-suite/typecheck/android --module v6compose --discover android-v6-compose",
+    "verify:fences:android": "npm run verify:fences:android-v5 && npm run verify:fences:android-v6 && npm run verify:fences:android-v6-compose",
+    "verify:fences": "npm run verify:fences:web && npm run verify:fences:rn && npm run verify:fences:angular && npm run verify:fences:flutter-v6 && npm run verify:fences:flutter-v5 && npm run verify:fences:ios && npm run verify:fences:android",
     "test": "bash test-suite/scripts/run-matrix.sh --models claude-sonnet-4-6 --experiences 1 --projects fresh --ide cli",
     "test:quick": "bash test-suite/scripts/run-test.sh --framework reactjs --experience 1 --model claude-sonnet-4-6 --project fresh --keep",
     "test:matrix": "bash test-suite/scripts/run-matrix.sh",