opencode-1password-auth 1.0.5 → 1.0.6

package/index.ts

@@ -252,17 +252,20 @@ export const OnePasswordAuthPlugin: Plugin = async ({ client, $ }) => {
       
       try {
         // Authenticate with OpenCode runtime
-        debugLog(`Calling client.auth.set for ${providerId}`);
-        await client.auth.set({
+        debugLog(`Calling client.auth.set for ${providerId} with key length ${apiKey.length}`);
+        const result = await client.auth.set({
           path: { id: providerId },
           body: { type: "api", key: apiKey },
         });
-        debugLog(`✓ ${providerId} authenticated (from ${variableName})`);
+        debugLog(`✓ ${providerId} authenticated (from ${variableName}) - Result: ${JSON.stringify(result)}`);
       } catch (err) {
         debugLog(`Failed to authenticate ${providerId}: ${err}`);
         if (err instanceof Error) {
           debugLog(`Error message: ${err.message}`);
+          debugLog(`Error stack: ${err.stack}`);
         }
+        // Also log to console for immediate visibility
+        console.error(`[1Password Plugin] Failed to authenticate ${providerId}: ${err}`);
       }
     }
   };
@@ -311,52 +314,85 @@ export const OnePasswordAuthPlugin: Plugin = async ({ client, $ }) => {
     return toInject;
   };
 
-  return {
-    async event({ event }: { event: { type: string } }) {
-      debugLog(`Received event: ${event.type}`);
-      if (event.type === "server.connected") {
-        debugLog("Initializing...");
+  // Initialize as soon as plugin loads (config hook runs early)
+  const initializePlugin = async () => {
+    debugLog("Initializing plugin...");
 
-        opClient = await initClient();
-        if (!opClient) {
-          debugLog("No client available");
-          return;
-        }
+    opClient = await initClient();
+    if (!opClient) {
+      debugLog("No client available");
+      return;
+    }
 
-        const configEnvId = process.env.OP_CONFIG_ENV_ID;
-        if (!configEnvId) {
-          debugLog("Missing OP_CONFIG_ENV_ID");
-          return;
-        }
+    const configEnvId = process.env.OP_CONFIG_ENV_ID;
+    if (!configEnvId) {
+      debugLog("Missing OP_CONFIG_ENV_ID");
+      return;
+    }
 
-        debugLog(`Reading config from environment ${configEnvId}`);
-        const vars = await readEnvironmentVariables(configEnvId);
-        const configEnvIds: Record<string, string> = {};
-        for (const v of vars) {
-          if (v.name.endsWith("_ENV_ID") && v.value) {
-            configEnvIds[v.name] = v.value;
-            debugLog(`Found ${v.name} = ${v.value.substring(0, 10)}...`);
-          }
-        }
+    debugLog(`Reading config from environment ${configEnvId}`);
+    const vars = await readEnvironmentVariables(configEnvId);
+    const configEnvIds: Record<string, string> = {};
+    for (const v of vars) {
+      if (v.name.endsWith("_ENV_ID") && v.value) {
+        configEnvIds[v.name] = v.value;
+        debugLog(`Found ${v.name} = ${v.value.substring(0, 10)}...`);
+      }
+    }
 
-        const providersEnvId = configEnvIds.OPENCODE_PROVIDERS_ENV_ID;
-        if (providersEnvId) {
-          // First update the config files to use env var references
-          await updateAuthJson(providersEnvId);
-          await authenticateProviders(providersEnvId);
-        }
+    const providersEnvId = configEnvIds.OPENCODE_PROVIDERS_ENV_ID;
+    if (providersEnvId) {
+      // First update the config files to use env var references
+      await updateAuthJson(providersEnvId);
+      await authenticateProviders(providersEnvId);
+    }
 
-        const mcpEnvIdFromConfig = configEnvIds.OPENCODE_MCPS_ENV_ID;
-        if (mcpEnvIdFromConfig) {
-          mcpsEnvId = mcpEnvIdFromConfig;
-          await updateOpenCodeJsonMCP(mcpEnvIdFromConfig);
-          const toInject = await injectMCPSecrets(mcpEnvIdFromConfig);
-          if (Object.keys(toInject).length > 0) {
-            debugLog(`Injected ${Object.keys(toInject).join(", ")} for MCP`);
+    const mcpEnvIdFromConfig = configEnvIds.OPENCODE_MCPS_ENV_ID;
+    if (mcpEnvIdFromConfig) {
+      mcpsEnvId = mcpEnvIdFromConfig;
+      await updateOpenCodeJsonMCP(mcpEnvIdFromConfig);
+      const toInject = await injectMCPSecrets(mcpEnvIdFromConfig);
+      if (Object.keys(toInject).length > 0) {
+        debugLog(`Injected ${Object.keys(toInject).join(", ")} for MCP`);
+      }
+    }
+
+    debugLog("Plugin initialization complete");
+  };
+
+  // Start initialization immediately
+  initializePlugin().catch(err => {
+    debugLog(`Failed to initialize plugin: ${err}`);
+  });
+
+  return {
+    async config(config: any) {
+      debugLog("Config hook called");
+      // Config hook runs early, we could also initialize here
+      // but we're already initializing above
+    },
+
+    async event({ event }: { event: { type: string } }) {
+      debugLog(`Received event: ${event.type}`);
+      if (event.type === "server.connected") {
+        debugLog("Server connected - rechecking authentication");
+        // Re-authenticate providers in case they weren't set earlier
+        if (opClient) {
+          const configEnvId = process.env.OP_CONFIG_ENV_ID;
+          if (configEnvId) {
+            const vars = await readEnvironmentVariables(configEnvId);
+            const configEnvIds: Record<string, string> = {};
+            for (const v of vars) {
+              if (v.name.endsWith("_ENV_ID") && v.value) {
+                configEnvIds[v.name] = v.value;
+              }
+            }
+            const providersEnvId = configEnvIds.OPENCODE_PROVIDERS_ENV_ID;
+            if (providersEnvId) {
+              await authenticateProviders(providersEnvId);
+            }
           }
         }
-
-        debugLog("Initialization complete");
       }
     },
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-1password-auth",
-  "version": "1.0.5",
+  "version": "1.0.6",
   "description": "1Password integration for OpenCode - authenticate providers and inject MCP secrets from 1Password environments",
   "type": "module",
   "main": "index.ts",