pi-harness-runtime 1.1.414 → 1.1.416

package/packages/webmcp-tester/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/webmcp-tester",
-  "version": "1.1.414",
+  "version": "1.1.416",
   "description": "WebMCP-powered E2E testing for pi-harness - natural language test scenarios",
   "type": "module",
   "main": "./dist/index.js",

package/packages/wiki/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/wiki",
-  "version": "1.1.414",
+  "version": "1.1.416",
   "description": "LLM Wiki integration for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/workflow-events/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/workflow-events",
-  "version": "1.1.414",
+  "version": "1.1.416",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/workspace-scanner/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/workspace-scanner",
-  "version": "1.1.414",
+  "version": "1.1.416",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/worktree/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/worktree",
-  "version": "1.1.414",
+  "version": "1.1.416",
   "description": "pi-harness-runtime — worktree",
   "type": "module",
   "main": "./dist/index.js",

package/packages/write-review/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moocoding/write-review",
-  "version": "1.1.414",
+  "version": "1.1.416",
   "description": "Two-agent write-review loop with blackboard coordination",
   "type": "module",
   "main": "src/index.ts",

package/scripts/sync-to-qdrant.ts

@@ -12,7 +12,7 @@
  */
 
 import { existsSync, readdirSync, readFileSync, statSync } from "node:fs";
-import { join, basename } from "node:path";
+import { join } from "node:path";
 import { QdrantClient } from "@qdrant/js-client-rest";
 // The runtime and this script must write into the SAME collection with the
 // SAME vectors: a collection holds vectors of exactly one size, and a vector
@@ -31,59 +31,11 @@ import {
   skillPointPayload,
   skillQdrantPoint,
 } from "../packages/skills/dist/vector-schema.js";
-
-// =============================================================================
-// SANITIZATION: Replace client-specific names with placeholders
-// =============================================================================
-
-interface SanitizationRule {
-  pattern: RegExp;
-  replacement: string;
-}
-
-const SANITIZATION_RULES: SanitizationRule[] = [
-  // Client company names (case-insensitive)
-  { pattern: /\binpac\b/gi, replacement: "[Client A]" },
-  { pattern: /\bm-capital\b/gi, replacement: "[Client A]" },
-  { pattern: /\bmcapital\b/gi, replacement: "[Client A]" },
-  { pattern: /\bdigisoft\b/gi, replacement: "[Client B]" },
-
-  // Project/app specific names
-  { pattern: /\bgse-insurance\b/gi, replacement: "[Insurance Project]" },
-  { pattern: /\btbs-import\b/gi, replacement: "[Import Project]" },
-  { pattern: /\bdigisoft_erp\b/gi, replacement: "[Client B Erp]" },
-  { pattern: /\bm_capital\b/gi, replacement: "[Client A]" },
-  { pattern: /\binpac_\b/gi, replacement: "[Client A]_" },
-  { pattern: /\bpaperclip_\b/gi, replacement: "[Client D]_" },
-  { pattern: /\bopenclaw_\b/gi, replacement: "[Client F]_" },
-
-  // Generic replacements
-  { pattern: /Company Name/gi, replacement: "[Client Name]" },
-  { pattern: /your company/gi, replacement: "[Client]" },
-  { pattern: /our client/gi, replacement: "[Client]" },
-  { pattern: /client-specific/gi, replacement: "[Project-specific]" },
-];
-
-// Skill name sanitization (for skill directory name -> sanitized display name)
-const SKILL_NAME_MAP: Record<string, string> = {
-  'gse-insurance-step5-slot-filter': 'insurance-project-step-filter',
-  'inpac-pe-approval-workflow': 'client-a-pe-approval-workflow',
-  'inpac-po-approval-workflow': 'client-a-po-approval-workflow',
-  'inpac-pr-approval-workflow': 'client-a-pr-approval-workflow',
-  'tbs-import-clearance-lcv': 'import-project-clearance',
-  'whispertool': 'whisper-tool',
-};
-
-/**
- * Sanitize text by replacing client-specific patterns
- */
-function sanitize(text: string): string {
-  let result = text;
-  for (const rule of SANITIZATION_RULES) {
-    result = result.replace(rule.pattern, rule.replacement);
-  }
-  return result;
-}
+// The parser and the sanitization live in the shared package. The runtime
+// registry writes into the same collection and needs the same rules, or it
+// would leave the raw name and the raw body of a client skill in a shared
+// cluster - and a parser of its own would name the same skill differently.
+import { parseSkillContent } from "../packages/skills/dist/parser.js";
 
 // =============================================================================
 // Qdrant Config
@@ -131,44 +83,27 @@ interface SkillDoc {
   text: string;
 }
 
-function parseFrontmatter(content: string): Record<string, string> | null {
-  const match = content.match(/^---\n([\s\S]*?)\n---/);
-  if (!match) return null;
-  
-  const fm: Record<string, string> = {};
-  for (const line of match[1].split("\n")) {
-    const idx = line.indexOf(":");
-    if (idx === -1) continue;
-    fm[line.slice(0, idx).trim()] = line.slice(idx + 1).trim();
-  }
-  return fm;
-}
-
 function readSkill(dirPath: string): SkillDoc | null {
   const skillPath = join(dirPath, "SKILL.md");
   if (!existsSync(skillPath)) return null;
 
   try {
     const rawContent = readFileSync(skillPath, "utf-8");
-    const fm = parseFrontmatter(rawContent);
-    if (!fm) return null;
-
-    // Get original skill name from directory
-    const originalName = fm.name || basename(dirPath);
-    
-    // Sanitize content
-    const sanitizedContent = sanitize(rawContent);
-    
-    // Use name map for known sensitive skill names, otherwise sanitize
-    const name = SKILL_NAME_MAP[originalName] || sanitize(originalName);
-    
-    const description = sanitize(fm.description || "");
-    const body = sanitizedContent.replace(/^---[\s\S]*?---\n/, "").trim();
+    // The SAME parser as the runtime registry. A parser of its own would
+    // read a quoted name with the quotes and write a point that the
+    // runtime never addresses - the two writers would name the same skill
+    // differently.
+    const skill = parseSkillContent(rawContent, skillPath);
 
     // The payload and the text come from the shared schema, so this script
     // and the runtime store the SAME point for the same skill: the point
-    // id is the hash of the name the payload carries.
-    const payload = skillPointPayload({ name, description, body });
+    // id is the hash of the name the payload carries, and the payload
+    // carries no client name.
+    const payload = skillPointPayload({
+      name: skill.id,
+      description: skill.frontmatter.description ?? "",
+      body: skill.body,
+    });
 
     return {
       name: payload.name,