clementine-agent 1.0.52 → 1.0.54

package/dist/agent/assistant.js

@@ -1093,7 +1093,9 @@ The **only source of truth for tool availability is your function schema**. Do n
 
 **Never** say the tool "isn't loaded in this session," "doesn't carry over from Claude Desktop," "the tools array is empty," or "MCP server still connecting." If any of those phrasings come to mind, call the tool directly and report what actually happens instead.
 
-\`list_allowed_tools\` / \`disallow_tool\` manage the whitelist. \`integration_status\` is for env-var (API key) integrations — not for claude_ai_* connectors, which are schema-driven.
+\`list_allowed_tools\` / \`disallow_tool\` manage the whitelist. \`integration_status\` is for env-var (API key) integrations — **not** for claude_ai_* connectors, which are schema-driven. Don't use \`integration_status\` as a proxy for "can I call Drive / Gmail / etc." — those are always tried by direct tool call, not status lookup.
+
+**Critical rule: if the user asks you to use a claude_ai_* connector, you call the connector tool. Full stop.** Do not report "I tried and it failed" unless there was an actual tool call that returned an actual error — your audit log records every tool call, so narrating a failed attempt when the audit shows no call will be spotted.
 
 ## Context Window Management
 
@@ -1332,7 +1334,8 @@ You have a cost budget per message — not a hard turn limit. Work until the tas
         if (!isAutonomous) {
             try {
                 const { summarizeIntegrationStatus } = require('../config/integrations-registry.js');
-                const summary = summarizeIntegrationStatus(process.env);
+                const { envSnapshot } = require('../config.js');
+                const summary = summarizeIntegrationStatus(envSnapshot());
                 if (summary)
                     parts.push(`## Integration Status\n\n${summary}\n\nCall \`integration_status\`, \`list_integrations\`, or \`setup_integration\` for details.`);
             }
@@ -1613,13 +1616,16 @@ You have a cost budget per message — not a hard turn limit. Work until the tas
                     type: 'stdio',
                     command: 'node',
                     args: [MCP_SERVER_SCRIPT],
+                    // Spread process.env so the MCP subprocess sees the full environment
+                    // the daemon is running with — API keys hydrated from .env/Keychain,
+                    // PATH, HOME, etc. Without this, tools that inspect env vars
+                    // (integration_status, Outlook/Graph, Salesforce) see only the
+                    // handful we pass and report everything as "missing." Our explicit
+                    // keys come after the spread so we always win on overlaps.
                     env: {
+                        ...process.env,
                         CLEMENTINE_HOME: BASE_DIR,
                         CLEMENTINE_TEAM_AGENT: profile?.slug ?? 'clementine',
-                        // Propagate interaction-source so the MCP subprocess can gate
-                        // owner-only tools. Without this, getInteractionSource() inside
-                        // the subprocess returns the module-default 'autonomous' and
-                        // every owner-DM-gated tool (env_set, allow_tool, etc.) refuses.
                         CLEMENTINE_INTERACTION_SOURCE: sourceOverride ?? inferInteractionSource(sessionKey),
                     },
                 },
@@ -3071,6 +3077,7 @@ You have a cost budget per message — not a hard turn limit. Work until the tas
                             command: 'node',
                             args: [MCP_SERVER_SCRIPT],
                             env: {
+                                ...process.env,
                                 CLEMENTINE_HOME: BASE_DIR,
                                 CLEMENTINE_TEAM_AGENT: profile?.slug ?? 'clementine',
                                 // Auto-memory extractor runs autonomously.

package/dist/config.d.ts

@@ -10,6 +10,8 @@ import type { Models } from './types.js';
 export declare const PKG_DIR: string;
 /** Data home — user data, vault, .env, logs, sessions. */
 export declare const BASE_DIR: string;
+/** Merged view of process.env overlaid with .env. Use for classifyIntegrations / summarizeIntegrationStatus. */
+export declare function envSnapshot(): Record<string, string | undefined>;
 export declare const VAULT_DIR: string;
 export declare const SYSTEM_DIR: string;
 export declare const DAILY_NOTES_DIR: string;

package/dist/config.js

@@ -46,6 +46,10 @@ const env = readEnvFile();
 function getEnv(key, fallback = '') {
     return env[key] ?? process.env[key] ?? fallback;
 }
+/** Merged view of process.env overlaid with .env. Use for classifyIntegrations / summarizeIntegrationStatus. */
+export function envSnapshot() {
+    return { ...process.env, ...env };
+}
 // ── Paths ────────────────────────────────────────────────────────────
 export const VAULT_DIR = path.join(BASE_DIR, 'vault');
 export const SYSTEM_DIR = path.join(VAULT_DIR, '00-System');

package/dist/tools/admin-tools.js

@@ -217,7 +217,14 @@ export function registerAdminTools(server) {
         slug: z.string().optional().describe('Optional: specific integration slug to check (e.g. "slack"). If omitted, returns all.'),
     }, async ({ slug }) => {
         const slugs = slug ? [slug] : undefined;
-        const reports = classifyIntegrations(process.env, slugs);
+        // `env` from shared.ts is the parsed .env file — the authoritative source
+        // for configured credentials in this project. Falls through to process.env
+        // for anything overridden at runtime (keychain-hydrated secrets, shell
+        // env, etc.). Do NOT read process.env alone: config.ts deliberately
+        // keeps .env values out of process.env, so classifying from process.env
+        // shows everything "missing."
+        const merged = { ...process.env, ...env };
+        const reports = classifyIntegrations(merged, slugs);
         if (reports.length === 0)
             return textResult(`Unknown integration slug: ${slug}. Use list_integrations to see available.`);
         const icon = (s) => s === 'configured' ? '✓' : s === 'partial' ? '~' : '✗';
@@ -248,7 +255,8 @@ export function registerAdminTools(server) {
         if (!integration) {
             return textResult(`Unknown integration slug: ${slug}. Run list_integrations to see what's available.`);
         }
-        const [status] = classifyIntegrations(process.env, [integration.slug]);
+        const merged = { ...process.env, ...env };
+        const [status] = classifyIntegrations(merged, [integration.slug]);
         const lines = [];
         lines.push(`## ${integration.label} (${integration.slug})`);
         lines.push('');
@@ -262,7 +270,7 @@ export function registerAdminTools(server) {
         lines.push('');
         lines.push('**Credentials:**');
         for (const req of integration.requirements) {
-            const present = !!process.env[req.envVar];
+            const present = !!merged[req.envVar];
             const badge = present ? '✓ set' : (req.required ? '✗ REQUIRED' : '○ optional');
             const line = `- \`${req.envVar}\` — ${req.label} [${badge}]`;
             lines.push(line);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clementine-agent",
-  "version": "1.0.52",
+  "version": "1.0.54",
   "description": "Clementine — Personal AI Assistant (TypeScript)",
   "type": "module",
   "main": "dist/index.js",