@tyvm/knowhow 0.0.82 → 0.0.84

package/src/plugins/github.ts

@@ -35,10 +35,7 @@ export class GitHubPlugin extends PluginBase {
       }
       return false;
     } catch (error) {
-      console.error(
-        "GITHUB PLUGIN: Failed to initialize Octokit client",
-        error
-      );
+      this.log(`Failed to initialize Octokit client: ${error}`, "error");
       return false;
     }
   }
@@ -76,9 +73,7 @@ export class GitHubPlugin extends PluginBase {
   async getDiff(url: string) {
     try {
       const { owner, repo, pullNumber } = this.parseUrl(url);
-      console.log(
-        `GITHUB PLUGIN: Loading diff for ${owner}/${repo}#${pullNumber}`
-      );
+      this.log(`Loading diff for ${owner}/${repo}#${pullNumber}`);
       const { data: diff } = await this.octokit.rest.pulls.get({
         owner,
         repo,
@@ -90,9 +85,9 @@ export class GitHubPlugin extends PluginBase {
 
       return diff;
     } catch (error) {
-      console.error(`GITHUB PLUGIN: Failed to get diff for ${url}:`, error.message);
+      this.log(`Failed to get diff for ${url}: ${error.message}`, "error");
       if (error.status === 401) {
-        console.error("GITHUB PLUGIN: Authentication failed. Please check your GITHUB_TOKEN.");
+        this.log("Authentication failed. Please check your GITHUB_TOKEN.", "error");
       }
       return null;
     }
@@ -107,9 +102,9 @@ export class GitHubPlugin extends PluginBase {
         pull_number: parseInt(pullNumber, 10),
       });
     } catch (error) {
-      console.error(`GITHUB PLUGIN: Failed to get PR for ${url}:`, error.message);
+      this.log(`Failed to get PR for ${url}: ${error.message}`, "error");
       if (error.status === 401) {
-        console.error("GITHUB PLUGIN: Authentication failed. Please check your GITHUB_TOKEN.");
+        this.log("Authentication failed. Please check your GITHUB_TOKEN.", "error");
       }
       return null;
     }
@@ -119,7 +114,7 @@ export class GitHubPlugin extends PluginBase {
     const length = hunks
       .flatMap((hunk) => [...hunk.additions, ...hunk.subtractions])
       .reduce((acc, line) => acc + line.length, 0);
-    console.log(`GITHUB PLUGIN: Length of hunks: ${length}`);
+    this.log(`Length of hunks: ${length}`);
     return length;
   }
 
@@ -131,13 +126,13 @@ export class GitHubPlugin extends PluginBase {
           
           // If getDiff returned null (auth error), skip this URL
           if (!diff) {
-            console.log(`GITHUB PLUGIN: Skipping ${url} due to error`);
+            this.log(`Skipping ${url} due to error`);
             return null;
           }
           
           let parsed = parseHunks(diff.toString());
 
-          console.log(`GITHUB PLUGIN: Parsed ${parsed.length} hunks`);
+          this.log(`Parsed ${parsed.length} hunks`);
 
           const averageHunkSize =
             parsed.reduce((acc, hunk) => acc + hunk.lines.length, 0) /
@@ -147,9 +142,7 @@ export class GitHubPlugin extends PluginBase {
             .flatMap((hunk) => [...hunk.additions, ...hunk.subtractions])
             .reduce((acc, line) => acc + line.length, 0);
 
-          console.log(
-            `GITHUB PLUGIN: Average hunk size: ${averageHunkSize}, total characters: ${totalCharacters}`
-          );
+          this.log(`Average hunk size: ${averageHunkSize}, total characters: ${totalCharacters}`);
 
           const MAX_CHARACTERS = 10000;
           const average = MAX_CHARACTERS / averageHunkSize;
@@ -159,14 +152,10 @@ export class GitHubPlugin extends PluginBase {
             return this.getLengthOfHunks([hunk]) <= PER_HUNK_LIMIT;
           });
 
-          console.log(
-            `GITHUB PLUGIN: Filtered to ${
-              parsed.length
-            } hunks. ${this.getLengthOfHunks(parsed)} characters`
-          );
+          this.log(`Filtered to ${parsed.length} hunks. ${this.getLengthOfHunks(parsed)} characters`);
           return parsed;
         } catch (error) {
-          console.error(`GITHUB PLUGIN: Error parsing diff for ${url}:`, error.message);
+          this.log(`Error parsing diff for ${url}: ${error.message}`, "error");
           return null;
         }
       })
@@ -188,7 +177,7 @@ export class GitHubPlugin extends PluginBase {
           
           // Skip this PR if we couldn't get its data
           if (!prResponse) {
-            console.log(`GITHUB PLUGIN: Skipping ${url} - could not fetch PR data`);
+            this.log(`Skipping ${url} - could not fetch PR data`);
             continue;
           }
           
@@ -210,21 +199,21 @@ export class GitHubPlugin extends PluginBase {
         }
 
         if (prs.length === 0) {
-          return "GITHUB PLUGIN: Could not fetch any pull request data. Please check your GITHUB_TOKEN and permissions.";
+          return "Could not fetch any pull request data. Please check your GITHUB_TOKEN and permissions.";
         }
 
-        const context = `GITHUB PLUGIN: These ${urls} have automatically been expanded to include the changes:\n\n${JSON.stringify(
+        const context = `These ${urls} have automatically been expanded to include the changes:\n\n${JSON.stringify(
           prs,
           null,
           2
         )}`;
-        console.log(context);
+        this.log(context);
         return context;
       } catch (error) {
-        return `GITHUB PLUGIN: Error fetching pull request data: ${error.message}`;
+        return `Error fetching pull request data: ${error.message}`;
       }
     }
 
-    return "GITHUB PLUGIN: No pull request URLs detected.";
+    return "No pull request URLs detected.";
   }
 }

package/src/plugins/jira.ts

@@ -53,7 +53,7 @@ export class JiraPlugin extends PluginBase {
   async getTaskFromUrl(url: string) {
     const issueId = this.extractIdFromUrl(url);
     if (issueId) {
-      console.log(`Fetching Jira issue ${issueId}`);
+      this.log(`Fetching Jira issue ${issueId}`);
       return await this.getIssueData(issueId);
     }
     return null;
@@ -64,7 +64,7 @@ export class JiraPlugin extends PluginBase {
       const issue = await this.jiraClient.findIssue(issueId);
       return issue;
     } catch (error) {
-      console.error("Error fetching Jira issue:", error);
+      this.log(`Error fetching Jira issue: ${error}`, "error");
       return null;
     }
   }

package/src/plugins/language.ts

@@ -67,7 +67,7 @@ export class LanguagePlugin extends PluginBase implements Plugin {
         });
       });
     } catch (error) {
-      console.error("LANGUAGE PLUGIN: Error setting up event handlers:", error);
+      this.log("LANGUAGE PLUGIN: Error setting up event handlers: " + error, "error");
     }
   }
 
@@ -184,7 +184,7 @@ export class LanguagePlugin extends PluginBase implements Plugin {
         );
       }
     } catch (error) {
-      console.error("LANGUAGE PLUGIN: Error handling file event:", error);
+      this.log("LANGUAGE PLUGIN: Error handling file event: " + error, "error");
     }
   }
 
@@ -245,7 +245,7 @@ export class LanguagePlugin extends PluginBase implements Plugin {
         );
       }
     } catch (error) {
-      console.error("LANGUAGE PLUGIN: Error handling agent event:", error);
+      this.log("LANGUAGE PLUGIN: Error handling agent event: " + error, "error");
     }
   }
 
@@ -282,7 +282,7 @@ export class LanguagePlugin extends PluginBase implements Plugin {
     // Use the extracted resolveSources method
     const contexts = await this.resolveSources(matchingTerms);
 
-    console.log("LANGUAGE PLUGIN: Matching terms found:", matchingTerms);
+    this.log("LANGUAGE PLUGIN: Matching terms found: " + matchingTerms);
 
     const output = contexts.every((c) => typeof c === "string")
       ? contexts.join("")

package/src/plugins/linear.ts

@@ -79,7 +79,7 @@ export class LinearPlugin extends PluginBase {
       const issue = await this.linearClient.issue(issueId);
       return issue;
     } catch (error) {
-      console.error("Error fetching Linear issue:", error);
+      this.log(`Error fetching Linear issue: ${error}`, "error");
       return null;
     }
   }
@@ -87,7 +87,7 @@ export class LinearPlugin extends PluginBase {
   async getTaskFromUrl(url: string) {
     const issueId = this.getIdFromUrl(url);
     if (issueId) {
-      console.log(`Fetching Linear issue ${issueId}`);
+      this.log(`Fetching Linear issue ${issueId}`);
       return await this.getIssueData(issueId);
     }
     return null;
@@ -135,7 +135,7 @@ export class LinearPlugin extends PluginBase {
   }
 
   async getTasksForProject(projectId: string) {
-    console.log({ projectId });
+    this.log(`Project ID: ${projectId}`);
     let tasks = await this.linearClient.issues({
       filter: {
         project: { slugId: { eq: projectId } },
@@ -161,7 +161,7 @@ export class LinearPlugin extends PluginBase {
   }
 
   async getTasksForTeam(teamId: string) {
-    console.log({ teamId });
+    this.log(`Team ID: ${teamId}`);
     let tasks = await this.linearClient.issues({
       filter: {
         team: { key: { eq: teamId } },

package/src/plugins/notion.ts

@@ -58,9 +58,7 @@ export class NotionPlugin extends PluginBase {
     if (processed[pageId] || currentDepth > maxDepth) {
       return { results: [] };
     }
-    console.log(
-      `Notion Plugin: Fetching all blocks for page ${pageId} at depth ${currentDepth}`
-    );
+    this.log(`Fetching all blocks for page ${pageId} at depth ${currentDepth}`);
     processed[pageId] = true;
     const response = await this.notionClient.blocks.children.list({
       block_id: pageId,
@@ -68,7 +66,7 @@ export class NotionPlugin extends PluginBase {
 
     let cursor = response.next_cursor;
     while (cursor) {
-      console.log("Fetching more blocks");
+      this.log("Fetching more blocks");
       const childResponse = await this.notionClient.blocks.children.list({
         block_id: pageId,
         start_cursor: cursor,
@@ -109,7 +107,7 @@ export class NotionPlugin extends PluginBase {
         (b) => "has_children" in b && b.has_children
       );
 
-      console.log(JSON.stringify(results, null, 2));
+      this.log(JSON.stringify(results, null, 2));
 
       const childBlocks = await Promise.all(
         childPages.map(async (childPage) => {
@@ -133,7 +131,7 @@ export class NotionPlugin extends PluginBase {
       }
     }
 
-    console.log(JSON.stringify(embeddings, null, 2));
+    this.log(JSON.stringify(embeddings, null, 2));
     return embeddings;
   }
 
@@ -147,7 +145,7 @@ export class NotionPlugin extends PluginBase {
   async getPageFromUrl(url: string) {
     const pageId = url.split("-").pop();
     if (pageId) {
-      console.log(`Fetching Notion page ${pageId}`);
+      this.log(`Fetching Notion page ${pageId}`);
       const page = await this.notionClient.pages.retrieve({ page_id: pageId });
       const blocks = await this.getAllChildBlocks(page.id);
       return { page, blocks };
@@ -175,7 +173,7 @@ export class NotionPlugin extends PluginBase {
     const markdownPages = pagesDataFiltered
       .map((page) => `### Page: ${JSON.stringify(page, null, 2)}\n-`)
       .join("\n\n");
-    console.log(markdownPages);
+    this.log(markdownPages);
     return `NOTION PLUGIN: The following pages were loaded:\n\n${markdownPages}`;
   }
 }

package/src/plugins/plugins.ts

@@ -1,4 +1,5 @@
 import { Plugin, PluginContext } from "./types";
+import { Config } from "../types";
 import { VimPlugin } from "./vim";
 import { LinterPlugin } from "./LinterPlugin";
 import { LanguagePlugin } from "./language";
@@ -17,11 +18,15 @@ import { AgentsMdPlugin } from "./AgentsMdPlugin";
 import { ExecPlugin } from "./exec";
 import { getConfig } from "../config";
 import { getDisabledPlugins } from "../types";
+import { SkillsPlugin } from "./SkillsPlugin";
+import { EventService } from "../services/EventService";
 
 export class PluginService {
   private pluginMap = new Map<string, Plugin>();
+  private events?: EventService;
 
   constructor(context: PluginContext) {
+    this.events = context.Events;
     context.Plugins = this;
 
     // Register migrated PluginBase plugins
@@ -41,6 +46,7 @@ export class PluginService {
     this.pluginMap.set("tmux", new TmuxPlugin(context));
     this.pluginMap.set("agents-md", new AgentsMdPlugin(context));
     this.pluginMap.set("exec", new ExecPlugin(context));
+    this.pluginMap.set("skills", new SkillsPlugin(context));
   }
 
   /* -------- lifecycle helpers ------------------------------------ */
@@ -57,6 +63,26 @@ export class PluginService {
     return instance.meta.key;
   }
 
+  /**
+   * Load plugins from config's pluginPackages map.
+   * Each entry maps a plugin key to an npm package name or file path.
+   * Errors are caught and logged as warnings without crashing.
+   */
+  async loadPluginsFromConfig(config: Config): Promise<void> {
+    const pluginPackages = config.pluginPackages || {};
+    for (const [key, spec] of Object.entries(pluginPackages)) {
+      try {
+        await this.loadPlugin(spec);
+      } catch (error) {
+        this.events?.log(
+          "PluginService",
+          `Failed to load plugin "${key}" from "${spec}": ${error instanceof Error ? error.message : error}`,
+          "warn"
+        );
+      }
+    }
+  }
+
   /** Disable a plugin by its key; returns `true` if found. */
   disablePlugin(key: string): boolean {
     const p = this.pluginMap.get(key);
@@ -101,7 +127,7 @@ export class PluginService {
       return "";
     }
     const calls = plugins.map(async (p) => {
-      return this.callManyForPlugin(p, userInput).catch();
+      return this.callManyForPlugin(p, userInput).catch(() => "");
     });
 
     const results = await Promise.all(calls);
@@ -118,7 +144,7 @@ export class PluginService {
 
     const enabled = await newPlugin.isEnabled();
     if (!enabled) {
-      console.log(`Plugin ${kind} is disabled, skipping`);
+      this.events?.log("PluginService", `Plugin ${kind} is disabled, skipping`);
       return "";
     }
     return newPlugin.call(userInput);
@@ -134,7 +160,7 @@ export class PluginService {
 
     const enabled = await newPlugin.isEnabled();
     if (!enabled) {
-      console.log(`Plugin ${kind} is disabled, skipping`);
+      this.events?.log("PluginService", `Plugin ${kind} is disabled, skipping`);
       return "";
     }
     return newPlugin.callMany(userInput);
@@ -149,7 +175,7 @@ export class PluginService {
 
     const enabled = await newPlugin.isEnabled();
     if (!enabled) {
-      console.log(`Plugin ${kind} is disabled, skipping`);
+      this.events?.log("PluginService", `Plugin ${kind} is disabled, skipping`);
       return [];
     }
     return newPlugin.embed ? newPlugin.embed(userInput) : [];

package/src/plugins/url.ts

@@ -35,7 +35,7 @@ export class UrlPlugin extends PluginBase implements Plugin {
     try {
       const text = await loadWebpage(url);
 
-      console.log(`URL PLUGIN: Fetched content from ${url}:`, text);
+      this.log(`URL PLUGIN: Fetched content from ${url}: ${text}`);
 
       return {
         id: url + "-url",
@@ -43,7 +43,7 @@ export class UrlPlugin extends PluginBase implements Plugin {
         metadata: { url },
       };
     } catch (error) {
-      console.error(`Error fetching or parsing URL ${url}:`, error);
+      this.log(`Error fetching or parsing URL ${url}: ${error}`, "error");
       return null;
     }
   }

package/src/plugins/vim.ts

@@ -59,13 +59,14 @@ export class VimPlugin extends PluginBase {
       return { filePath, content: "DIRECTORY" };
     }
     if (stat.size > 32000) {
-      console.error(
-        `VIM PLUGIN: File ${filePath} is too large with size ${stat.size}`
+      this.log(
+        `VIM PLUGIN: File ${filePath} is too large with size ${stat.size}`,
+        "error"
       );
       return { filePath, content: "FILE TOO LARGE" };
     }
 
-    console.log(`VIM PLUGIN: Reading file ${filePath}`);
+    this.log(`VIM PLUGIN: Reading file ${filePath}`);
     const content = await readFile(filePath, "utf8");
     return { filePath, content };
   }

package/src/services/AgentService.ts

@@ -7,6 +7,7 @@ import { AgentContext } from "../agents/base/base";
 
 export class AgentService {
   private agents: Map<string, IAgent> = new Map();
+  private agentContext: AgentContext | null = null;
 
   constructor(private tools: ToolsService, private events: EventService) {
     this.wireUp();
@@ -57,6 +58,22 @@ export class AgentService {
     this.agents.set(name, agent);
   }
 
+  /**
+   * Set the AgentContext that will be used when creating new agent instances.
+   * Should be called from cli.ts after all services are wired up (including LazyToolsService).
+   */
+  public setAgentContext(context: AgentContext): void {
+    this.agentContext = context;
+  }
+
+  /**
+   * Get the current AgentContext. Falls back to a minimal context using this service's
+   * own tools/events if none has been explicitly set.
+   */
+  public getAgentContext(): AgentContext {
+    return this.agentContext ?? { Tools: this.tools, Events: this.events };
+  }
+
   public getAgent(name: string): IAgent {
     const agent = this.agents.get(name);
     if (!agent) {

package/src/services/AgentSyncFs.ts

@@ -10,6 +10,7 @@ import { watch } from "fs";
 export interface FsSyncOptions {
   taskId: string;
   prompt: string;
+  agentName?: string;
 }
 
 /**
@@ -51,6 +52,7 @@ export class AgentSyncFs {
       await this.writeMetadata({
         taskId: this.taskId,
         prompt: options.prompt,
+        agentName: options.agentName || "unknown",
         startTime: new Date().toISOString(),
         status: "running",
       });
@@ -125,6 +127,7 @@ export class AgentSyncFs {
 
       metadata.threads = agent.getThreads();
       metadata.totalCostUsd = agent.getTotalCostUsd();
+    metadata.agentName = agent.name;
       metadata.inProgress = inProgress;
       metadata.lastUpdate = new Date().toISOString();