claude-code - 💡(How to fix) Fix VS Code extension: hardcoded 3000ms shell-integration wait causes ~5s delay clicking extension icon [2 comments, 2 participants]

TL;DR

The issue can be resolved by making the 3000ms timeout configurable or short-circuiting the execution of the executeCommand if terminal.shellIntegration is already present.

Guidance

• The current implementation has a hardcoded 3000ms fallback, which is causing a delay of around 5 seconds before "claude" is typed into the terminal.
• To mitigate this, the claudeCode.shellIntegrationTimeoutMs setting could be introduced to make the timeout configurable, allowing users to adjust the delay to their needs.
• Alternatively, the code could be modified to short-circuit the execution of executeCommand if terminal.shellIntegration is already present after creating the terminal, eliminating the need for the timeout.
• Verifying the fix would involve checking the time it takes for "claude" to be typed into the terminal after clicking the Claude Code icon, and ensuring it is significantly less than 5 seconds.

Example

// Potential short-circuit implementation
if (term.shellIntegration) {
  term.shellIntegration.executeCommand("claude");
} else {
  await new Promise((resolve) => {
    onDidChangeTerminalShellIntegration((event) => {
      if (event.terminal === term) {
        onDidStartTerminalShellExecution((exec) => {
          if (exec.terminal === term) resolve();
        });
        event.shellIntegration.executeCommand("claude");
      }
    });
    setTimeout(() => {
      if (!term.shellIntegration) { term.sendText("claude"); resolve(); }
    }, claudeCode.shellIntegrationTimeoutMs);
  });
}

Notes

The provided solution assumes that the claudeCode.shellIntegrationTimeoutMs setting or the short-circuit approach would be implemented in the extension code. The exact implementation details may vary depending on the extension's architecture and requirements.

Recommendation

Apply a workaround by setting claudeCode.useTerminal to false and using the webview path, as this is a known working alternative, until a more permanent fix can be implemented.