@zhijiewang/openharness 2.3.0 → 2.3.1

package/dist/main.js

@@ -19,6 +19,8 @@ import { readOhConfig } from "./harness/config.js";
 import { emitHook } from "./harness/hooks.js";
 import { detectProject, projectContextToPrompt } from "./harness/onboarding.js";
 import { createRulesFile, loadRules, loadRulesAsPrompt } from "./harness/rules.js";
+import { loadActiveMemories, memoriesToPrompt, userProfileToPrompt } from "./harness/memory.js";
+import { discoverSkills, skillsToPrompt } from "./harness/plugins.js";
 import { listSessions } from "./harness/session.js";
 import { connectedMcpServers, disconnectMcpClients, getMcpInstructions, loadMcpTools } from "./mcp/loader.js";
 import { getAllTools } from "./tools.js";
@@ -78,6 +80,20 @@ function buildSystemPrompt(model) {
     const rulesPrompt = loadRulesAsPrompt();
     if (rulesPrompt)
         parts.push(rulesPrompt);
+    // User profile (highest priority personal context)
+    const userProfile = userProfileToPrompt();
+    if (userProfile)
+        parts.push(userProfile);
+    // Remembered context from past sessions
+    const memories = loadActiveMemories();
+    const memoriesPrompt = memoriesToPrompt(memories);
+    if (memoriesPrompt)
+        parts.push(memoriesPrompt);
+    // Available skills (Level 0 — names + descriptions only)
+    const skills = discoverSkills();
+    const skillsPrompt = skillsToPrompt(skills);
+    if (skillsPrompt)
+        parts.push(skillsPrompt);
     // MCP server instructions (sandboxed — treat as untrusted)
     const mcpInstructions = getMcpInstructions();
     if (mcpInstructions.length > 0) {

package/dist/query/index.js

@@ -21,7 +21,7 @@ const DEFAULT_MAX_TURNS = 50;
 export async function* query(userMessage, config, existingMessages = []) {
     const maxTurns = config.maxTurns ?? DEFAULT_MAX_TURNS;
     const toolContext = {
-        workingDir: process.cwd(),
+        workingDir: config.workingDir ?? process.cwd(),
         abortSignal: config.abortSignal,
         provider: config.provider,
         model: config.model,
@@ -44,9 +44,19 @@ export async function* query(userMessage, config, existingMessages = []) {
             toolPrompts += `\n\n[${deferredCount} additional tools available — use ToolSearch to discover them]`;
         }
     }
-    const fullSystemPrompt = toolPrompts
+    let fullSystemPrompt = toolPrompts
         ? `${config.systemPrompt}\n\n# Available Tools\n\n${toolPrompts}`
         : config.systemPrompt;
+    // Auto-trigger skills matching user message
+    try {
+        const { findTriggeredSkills } = await import("../harness/plugins.js");
+        const triggered = findTriggeredSkills(userMessage);
+        if (triggered.length > 0) {
+            const hints = triggered.map((s) => `- **${s.name}**: ${s.description}`).join("\n");
+            fullSystemPrompt += `\n\n# Suggested Skills\nThese skills match your request. Use Skill tool to load them:\n${hints}`;
+        }
+    }
+    catch { /* skills optional */ }
     const state = {
         messages: [...existingMessages, createUserMessage(userMessage)],
         turn: 0,

package/dist/query/types.d.ts

@@ -16,6 +16,8 @@ export type QueryConfig = {
     maxCost?: number;
     model?: string;
     abortSignal?: AbortSignal;
+    /** Working directory for tool execution (defaults to process.cwd()) */
+    workingDir?: string;
 };
 export type TransitionReason = "next_turn" | "retry_network" | "retry_prompt_too_long" | "retry_max_output_tokens";
 export type QueryLoopState = {

package/dist/repl.js

@@ -674,17 +674,28 @@ export async function startREPL(config) {
                 if (extracted.length > 0) {
                     console.log(`[learn] Extracted ${extracted.length} skill(s) from this session.`);
                 }
-                // User profile update
+                // User profile update with LLM consolidation
                 if (messages.length >= 6) {
                     const detected = await detectMemories(config.provider, messages, currentModel);
                     const profileUpdates = detected.filter((d) => d.type === "user");
                     if (profileUpdates.length > 0) {
                         const currentProfile = loadUserProfile();
                         const newObservations = profileUpdates.map((d) => d.content).join("\n");
-                        const merged = currentProfile
-                            ? `${currentProfile}\n\n## Recent Observations\n${newObservations}`
-                            : newObservations;
-                        updateUserProfile(merged);
+                        if (currentProfile) {
+                            // LLM-assisted merge: consolidate instead of blind append
+                            const { createUserMessage: makeMsg } = await import("./types/message.js");
+                            try {
+                                const consolidated = await config.provider.complete([makeMsg(`Merge this user profile with new observations into a single cohesive profile. Keep the most important and recent information. Remove duplicates. Stay under 2000 characters. Return ONLY the merged profile text.\n\nCurrent profile:\n${currentProfile}\n\nNew observations:\n${newObservations}`)], "You are a profile curator. Return ONLY the merged profile, no commentary.", undefined, currentModel);
+                                updateUserProfile(consolidated.content);
+                            }
+                            catch {
+                                // Fallback to simple append if LLM fails
+                                updateUserProfile(`${currentProfile}\n\n${newObservations}`);
+                            }
+                        }
+                        else {
+                            updateUserProfile(newObservations);
+                        }
                     }
                 }
             }

package/dist/services/SkillExtractor.js

@@ -113,6 +113,23 @@ ${candidate.verification}
     writeFileSync(filePath, content, "utf-8");
     return filePath;
 }
+/** Quick LLM quality check — is this skill worth keeping? */
+async function isSkillWorthy(provider, candidate, model) {
+    try {
+        const prompt = `Is this extracted skill worth saving for future reuse? Answer YES or NO (one word only).
+
+Name: ${candidate.name}
+Description: ${candidate.description}
+Procedure: ${candidate.procedure}
+
+Criteria: Is it reusable (not a one-off)? Is the procedure clear and complete? Would it save time in future sessions?`;
+        const response = await provider.complete([createUserMessage(prompt)], "Answer YES or NO only.", undefined, model);
+        return response.content.trim().toUpperCase().startsWith("YES");
+    }
+    catch {
+        return true; // On error, allow the skill through
+    }
+}
 /**
  * Orchestrate the full extraction pipeline:
  * 1. Check if extraction is warranted
@@ -132,9 +149,12 @@ export async function runExtraction(provider, messages, sessionId, model) {
     const written = [];
     for (const candidate of candidates) {
         const similar = findSimilarSkill(candidate.name, candidate.description, existingSkills);
-        // Skip if a very similar skill already exists (avoid duplicates)
         if (similar)
             continue;
+        // Quality gate: quick LLM check before persisting
+        const worthy = await isSkillWorthy(provider, candidate, model);
+        if (!worthy)
+            continue;
         const filePath = persistSkill(candidate, sessionId);
         written.push(filePath);
     }

package/dist/tools/AgentTool/index.js

@@ -87,6 +87,7 @@ export const AgentTool = {
             model: agentModel,
             maxTurns: 20,
             abortSignal: context.abortSignal,
+            workingDir: agentWorkingDir,
         };
         const agentId = Date.now().toString(36) + Math.random().toString(36).slice(2, 6);
         emitHook("subagentStart", { agentId, toolName: input.subagent_type ?? "general" });
@@ -97,30 +98,13 @@ export const AgentTool = {
             bus.registerBackgroundAgent(bgId, input.subagent_type ?? "general");
             const runAgent = async () => {
                 let finalText = "";
-                const originalCwd = process.cwd();
                 try {
-                    if (worktreePath) {
-                        try {
-                            process.chdir(agentWorkingDir);
-                        }
-                        catch {
-                            /* ignore */
-                        }
-                    }
                     for await (const event of query(input.prompt, config)) {
                         if (event.type === "text_delta")
                             finalText += event.content;
                     }
                 }
                 finally {
-                    if (worktreePath) {
-                        try {
-                            process.chdir(originalCwd);
-                        }
-                        catch {
-                            /* ignore */
-                        }
-                    }
                     // Clean up worktree only if no changes were made
                     if (worktreePath) {
                         const hasChanges = hasWorktreeChanges(worktreePath);
@@ -152,16 +136,6 @@ export const AgentTool = {
         const outputChunks = [];
         let finalText = "";
         try {
-            // Override process.cwd for the sub-agent by setting workingDir in tool context
-            const originalCwd = process.cwd();
-            if (worktreePath) {
-                try {
-                    process.chdir(agentWorkingDir);
-                }
-                catch {
-                    /* ignore */
-                }
-            }
             try {
                 for await (const event of query(input.prompt, config)) {
                     if (event.type === "text_delta") {
@@ -184,15 +158,7 @@ export const AgentTool = {
                 }
             }
             finally {
-                // Restore original working directory
-                if (worktreePath) {
-                    try {
-                        process.chdir(originalCwd);
-                    }
-                    catch {
-                        /* ignore */
-                    }
-                }
+                /* workingDir passed via config — no process.chdir cleanup needed */
             }
         }
         catch (err) {

package/dist/tools/CronTool/index.d.ts

@@ -7,13 +7,13 @@ declare const createSchema: z.ZodObject<{
     prompt: z.ZodString;
 }, "strip", z.ZodTypeAny, {
     action: "create";
-    prompt: string;
     name: string;
+    prompt: string;
     schedule: string;
 }, {
     action: "create";
-    prompt: string;
     name: string;
+    prompt: string;
     schedule: string;
 }>;
 declare const deleteSchema: z.ZodObject<{

package/dist/tools/MemoryTool/index.d.ts

@@ -12,18 +12,18 @@ declare const inputSchema: z.ZodObject<{
     action: "search" | "list" | "save";
     content?: string | undefined;
     type?: "user" | "convention" | "preference" | "project" | "debugging" | "feedback" | "reference" | undefined;
-    query?: string | undefined;
-    description?: string | undefined;
-    name?: string | undefined;
     global?: boolean | undefined;
+    name?: string | undefined;
+    description?: string | undefined;
+    query?: string | undefined;
 }, {
     action: "search" | "list" | "save";
     content?: string | undefined;
     type?: "user" | "convention" | "preference" | "project" | "debugging" | "feedback" | "reference" | undefined;
-    query?: string | undefined;
-    description?: string | undefined;
-    name?: string | undefined;
     global?: boolean | undefined;
+    name?: string | undefined;
+    description?: string | undefined;
+    query?: string | undefined;
 }>;
 export declare const MemoryTool: Tool<typeof inputSchema>;
 export {};

package/dist/tools/PipelineTool/index.d.ts

@@ -8,29 +8,29 @@ declare const inputSchema: z.ZodObject<{
         dependsOn: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
     }, "strip", z.ZodTypeAny, {
         tool: string;
-        id: string;
         args: Record<string, unknown>;
+        id: string;
         dependsOn?: string[] | undefined;
     }, {
         tool: string;
-        id: string;
         args: Record<string, unknown>;
+        id: string;
         dependsOn?: string[] | undefined;
     }>, "many">;
     description: z.ZodOptional<z.ZodString>;
 }, "strip", z.ZodTypeAny, {
     steps: {
         tool: string;
-        id: string;
         args: Record<string, unknown>;
+        id: string;
         dependsOn?: string[] | undefined;
     }[];
     description?: string | undefined;
 }, {
     steps: {
         tool: string;
-        id: string;
         args: Record<string, unknown>;
+        id: string;
         dependsOn?: string[] | undefined;
     }[];
     description?: string | undefined;

package/dist/tools/TaskUpdateTool/index.d.ts

@@ -13,8 +13,8 @@ declare const inputSchema: z.ZodObject<{
 }, "strip", z.ZodTypeAny, {
     taskId: number;
     status?: "completed" | "pending" | "cancelled" | "in_progress" | "deleted" | undefined;
-    metadata?: Record<string, unknown> | undefined;
     description?: string | undefined;
+    metadata?: Record<string, unknown> | undefined;
     subject?: string | undefined;
     activeForm?: string | undefined;
     owner?: string | undefined;
@@ -23,8 +23,8 @@ declare const inputSchema: z.ZodObject<{
 }, {
     taskId: number;
     status?: "completed" | "pending" | "cancelled" | "in_progress" | "deleted" | undefined;
-    metadata?: Record<string, unknown> | undefined;
     description?: string | undefined;
+    metadata?: Record<string, unknown> | undefined;
     subject?: string | undefined;
     activeForm?: string | undefined;
     owner?: string | undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhijiewang/openharness",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "description": "Open-source terminal coding agent. Works with any LLM.",
   "type": "module",
   "bin": {