total-recall-brain 3.32.3 → 3.32.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "total-recall-brain",
-  "version": "3.32.3",
+  "version": "3.32.4",
   "description": "Personal AI system — database-free, SSSS-powered intelligence engine deployable to any POSIX host.",
   "type": "module",
   "workspaces": [

package/src/cli/command.mjs

@@ -170,7 +170,7 @@ function printHelp() {
 `);
 }
 
-function resolveTargetDir(isGlobal) {
+export function resolveTargetDir(isGlobal) {
   if (isGlobal) {
     try {
       const gBrain = resolveBrainLayer('global');
@@ -235,7 +235,7 @@ function existingTag(file, key) {
  * Composable instructions: the surfaces list every command, so any change to
  * the command set recompiles them (project layer, or every project for global).
  */
-function recompileSurfaces(isGlobal) {
+export function recompileSurfaces(isGlobal) {
   if (process.env.TR_COMMAND_NO_COMPILE === '1' || !process.argv[1]) return;
   try {
     const child = spawn(process.execPath, [process.argv[1], 'compile', isGlobal ? '--global' : '--project'], { detached: true, stdio: 'ignore' });
@@ -246,6 +246,48 @@ function recompileSurfaces(isGlobal) {
   }
 }
 
+/**
+ * Register every `commands` entry of an installed plugin as a composable command.
+ * A command that already exists and was generated from this plugin is refreshed;
+ * one that exists from anywhere else is left alone and reported.
+ */
+export function syncPluginCommands(pluginDir, { global: isGlobal = false } = {}) {
+  const manifest = JSON.parse(fs.readFileSync(path.join(fs.realpathSync(pluginDir), 'plugin.json'), 'utf8'));
+  const commandsDir = resolveTargetDir(isGlobal);
+  const result = { created: [], skipped: [] };
+  for (const entry of manifest.commands || []) {
+    const file = path.join(commandsDir, `${entry.name}.mjs`);
+    if (fs.existsSync(file)) {
+      if (!fs.readFileSync(file, 'utf8').startsWith(`// Generated from plugin ${manifest.id};`)) {
+        result.skipped.push({ name: entry.name, reason: 'a different command with this name exists' });
+        continue;
+      }
+      fs.rmSync(file);
+    }
+    generatePluginCommand(pluginDir, entry.name, commandsDir);
+    result.created.push(entry.name);
+  }
+  if (result.created.length) recompileSurfaces(isGlobal);
+  return result;
+}
+
+/** Remove the commands a plugin generated (only files carrying its generated header). */
+export function removePluginCommands(pluginId, { global: isGlobal = false } = {}) {
+  const commandsDir = resolveTargetDir(isGlobal);
+  const removed = [];
+  if (!fs.existsSync(commandsDir)) return removed;
+  for (const name of fs.readdirSync(commandsDir)) {
+    if (!name.endsWith('.mjs')) continue;
+    const file = path.join(commandsDir, name);
+    if (fs.readFileSync(file, 'utf8').startsWith(`// Generated from plugin ${pluginId};`)) {
+      fs.rmSync(file);
+      removed.push(name.replace(/\.mjs$/, ''));
+    }
+  }
+  if (removed.length) recompileSurfaces(isGlobal);
+  return removed;
+}
+
 export default async function commandCmd(rawArgs = []) {
   const isGlobal = rawArgs.includes('--global') || rawArgs.includes('-g');
   const asJson = rawArgs.includes('--json');

package/src/cli/plugin/install.mjs

@@ -1,4 +1,5 @@
 import { installPlugin as install } from '../../core/plugin-store.mjs';
+import { syncPluginCommands } from '../command.mjs';
 
 export async function installPlugin(args = []) {
   const isGlobal = args.includes('--global') || args.includes('-g');
@@ -16,7 +17,11 @@ export async function installPlugin(args = []) {
     const from = result.source.kind === 'peer' ? `peer ${result.source.peer_hostname}` : result.source.kind;
     console.log(`\n✅ Installed '${result.plugin.name}' (id: ${result.plugin.id}, v${result.plugin.version}) from ${from}`);
     console.log(`   Location: ${result.plugin.dir}`);
-    console.log(`   sha256:   ${result.sha256}\n`);
+    console.log(`   sha256:   ${result.sha256}`);
+    const cmds = syncPluginCommands(result.plugin.dir, { global: isGlobal });
+    if (cmds.created.length) console.log(`   Commands: ${cmds.created.map((c) => `total-recall ${c}`).join(', ')}`);
+    for (const c of cmds.skipped) console.log(`   Skipped command '${c.name}': ${c.reason}`);
+    console.log('');
   } catch (err) {
     console.error(`❌ ${err.message}`);
     process.exit(1);

package/src/cli/plugin/remove.mjs

@@ -1,4 +1,5 @@
 import { uninstallPlugin, setPluginShared } from '../../core/plugin-store.mjs';
+import { removePluginCommands } from '../command.mjs';
 import { publicPluginShareUrl } from '../../core/plugin-public.mjs';
 
 export async function removePlugin(args = []) {
@@ -13,7 +14,10 @@ export async function removePlugin(args = []) {
 
   try {
     const result = await uninstallPlugin(id, { projectRoot: process.cwd(), global: isGlobal ? true : undefined });
-    console.log(`\n✅ Removed plugin '${id}' (${result.scope}) from ${result.dir}\n`);
+    const removed = removePluginCommands(id, { global: result.scope === 'global' });
+    console.log(`\n✅ Removed plugin '${id}' (${result.scope}) from ${result.dir}`);
+    if (removed.length) console.log(`   Removed commands: ${removed.join(', ')}`);
+    console.log('');
   } catch (err) {
     console.error(`❌ ${err.message}`);
     process.exit(1);

package/src/core/env-import.mjs

@@ -10,6 +10,7 @@ import fs from 'node:fs';
 import path from 'node:path';
 import os from 'node:os';
 import { setSecret, listSecretsMeta, updateSecretMeta } from './secrets-store.mjs';
+import { providerForKeyName } from './provider-catalog.mjs';
 
 /**
  * @deprecated Kept as empty/export for older callers. Import does NOT use this
@@ -108,13 +109,9 @@ export function inferProvider(key) {
   if (!key) return null;
   const k = String(key);
 
-  // Lazy catalog match (optional enrichment only)
-  try {
-    // Dynamic import avoided in sync path — inline thin match from catalog module if already loadable
-    // Use sync require-free approach: duplicate-free segment heuristic only here for speed.
-  } catch {
-    /* ignore */
-  }
+  // A known provider anywhere in the key name wins (JSN_SMTP2GO_API_KEY -> smtp2go).
+  const known = providerForKeyName(k);
+  if (known) return known.id;
 
   // Strip common prefixes that are not the vendor
   let base = k