Initialize MCP tools once at start up instead of every time we auth. #3483

scidomino · 2025-07-07T21:31:06Z

TLDR

Initialize MCP tools once at start up instead of every time we auth.

Dive Deeper

Previously we were initializing MCP tools every time we authenticate. This was confusing users with MCP errors because they'd see "Waiting for Auth..." in the start up screen when they had a MCP server that was slow to start up. Even worse, if there was an error with MCP users would assume it was related to Auth.

This change does cause startup to hang while we wait for the MCP servers to start up but I think this is still an improvement. We can add a MCP loading spinner in subsequent PRs.

Reviewer Test Plan

Use the following MCP server in your settings.json to simulate a MCP server that takes a long time to start up.

{
  "mcpServers": {
    "slow": {
      "command": "node",
      "args": [
        "-e",
        "setInterval(() => {}, 1000);"
      ],
      "timeout": 10000
    }
  }
}

Testing Matrix

	🍏	🪟	🐧
npm run	x	❓	❓
npx	❓	❓	❓
Docker	❓	❓	❓
Podman	❓	-	-
Seatbelt	❓	-	-

Linked issues / bugs

Fixes #2926

gemini-code-assist

Summary of Changes

Hello @scidomino, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

I've refactored the application's startup sequence to initialize Managed Code Platform (MCP) tools and other core services only once at the beginning of the application lifecycle, rather than repeatedly during authentication attempts. This change aims to improve the user experience by providing clearer feedback during startup, preventing confusion between authentication issues and slow MCP server initialization, and addressing a specific bug related to this behavior.

Highlights

Centralized Initialization: I've consolidated the initialization of core services like FileDiscovery, Git, and MCP tools into a new config.initialize() method, ensuring they are set up once at application startup.
Improved User Experience: I've decoupled MCP tool initialization from the authentication flow. This change aims to prevent user confusion by clarifying error messages and providing better feedback when MCP servers are slow to start, addressing issue #2926.
Refactored Tool Dependencies: I've updated EditTool and WriteFileTool so they no longer directly hold a GeminiClient instance. Instead, they now retrieve it on demand from the Config object, promoting better dependency management and reducing direct coupling.
Configuration Simplification: I've simplified the createContentGeneratorConfig function by removing its dependency on the Config object for model determination, streamlining its usage.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature	Command	Description
Code Review	`/gemini review`	Performs a code review for the current pull request in its current state.
Pull Request Summary	`/gemini summary`	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in issue comments and review comments.
Help	`/gemini help`	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist is currently in preview and may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments to provide feedback.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

gemini-code-assist

Code Review

This pull request refactors the initialization of MCP tools to occur once at startup, rather than on every authentication. This is a significant architectural improvement that decouples tool initialization from the authentication flow, addressing user confusion with slow or failing MCP servers.

The changes are well-executed, moving initialization logic into a new Config.initialize method and simplifying the refreshAuth process. A notable improvement is that tools now fetch the GeminiClient instance from the Config on-demand, which resolves a latent bug where tools could hold onto a stale client after re-authentication.

I have one high-severity suggestion regarding error handling during git service initialization to ensure that failures in optional features like checkpointing are not silent.

packages/core/src/config/config.ts

github-actions · 2025-07-07T21:47:05Z

Code Coverage Summary

Package	Lines	Statements	Functions	Branches
CLI	71.81%	71.81%	73.5%	76.99%
Core	71.99%	71.99%	74.4%	80.97%

CLI Package - Full Text Report

-------------------|---------|----------|---------|---------|-------------------
File               | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
-------------------|---------|----------|---------|---------|-------------------
All files          |   71.81 |    76.99 |    73.5 |   71.81 |                   
 src               |   42.89 |    56.25 |    37.5 |   42.89 |                   
  gemini.tsx       |   17.04 |    16.66 |   16.66 |   17.04 | ...83-216,221-292 
  ...ractiveCli.ts |   90.16 |    65.38 |     100 |   90.16 | ...27,130,153-154 
 src/config        |      82 |    71.07 |   82.14 |      82 |                   
  auth.ts          |   15.38 |      100 |       0 |   15.38 | 11-39             
  config.ts        |   94.71 |    77.14 |    62.5 |   94.71 | ...06-207,269-273 
  extension.ts     |    73.8 |    70.58 |     100 |    73.8 | ...02-106,115-116 
  sandboxConfig.ts |   51.35 |    16.66 |   66.66 |   51.35 | ...43,53-69,74-91 
  settings.ts      |   90.77 |    78.94 |     100 |   90.77 | ...65-266,302-303 
 src/generated     |     100 |      100 |     100 |     100 |                   
  git-commit.ts    |     100 |      100 |     100 |     100 |                   
 src/services      |     100 |      100 |     100 |     100 |                   
  ...andService.ts |     100 |      100 |     100 |     100 |                   
 src/test-utils    |     100 |      100 |     100 |     100 |                   
  ...andContext.ts |     100 |      100 |     100 |     100 |                   
 src/ui            |   60.47 |    55.26 |   68.18 |   60.47 |                   
  App.tsx          |   57.96 |     45.9 |    62.5 |   57.96 | ...71-783,789-818 
  colors.ts        |   86.04 |      100 |   76.92 |   86.04 | 12-13,18-19,42-43 
  constants.ts     |       0 |        0 |       0 |       0 | 1-15              
  types.ts         |     100 |      100 |     100 |     100 |                   
 src/ui/commands   |   99.13 |    94.73 |   83.33 |   99.13 |                   
  clearCommand.ts  |     100 |      100 |     100 |     100 |                   
  helpCommand.ts   |     100 |      100 |     100 |     100 |                   
  memoryCommand.ts |     100 |      100 |     100 |     100 |                   
  types.ts         |       0 |        0 |       0 |       0 | 1                 
 src/ui/components |      71 |    70.98 |   64.28 |      71 |                   
  AboutBox.tsx     |     100 |       50 |     100 |     100 | 102               
  AsciiArt.ts      |     100 |      100 |     100 |     100 |                   
  AuthDialog.tsx   |   89.74 |      100 |      50 |   89.74 | 44-51             
  ...nProgress.tsx |   15.78 |      100 |       0 |   15.78 | 17-57             
  ...Indicator.tsx |   15.15 |      100 |       0 |   15.15 | 17-47             
  ...lePatcher.tsx |   73.17 |      100 |   66.66 |   73.17 | 34-46             
  ...ryDisplay.tsx |   21.05 |      100 |       0 |   21.05 | 17-35             
  ...ryDisplay.tsx |   97.82 |    95.83 |     100 |   97.82 | 59                
  ...esDisplay.tsx |   10.52 |      100 |       0 |   10.52 | 24-82             
  ...ngsDialog.tsx |    6.76 |      100 |       0 |    6.76 | 26-168            
  Footer.tsx       |   71.42 |    11.11 |     100 |   71.42 | ...,90-97,100-103 
  ...ngSpinner.tsx |      80 |    33.33 |     100 |      80 | 29,31-32          
  Header.tsx       |   96.87 |       60 |     100 |   96.87 | 27                
  Help.tsx         |    3.73 |      100 |       0 |    3.73 | 17-154            
  ...emDisplay.tsx |   68.65 |    57.14 |     100 |   68.65 | ...55-60,79-86,89 
  InputPrompt.tsx  |   79.52 |     62.5 |     100 |   79.52 | ...19,362,389-393 
  ...Indicator.tsx |     100 |      100 |     100 |     100 |                   
  ...geDisplay.tsx |   25.92 |      100 |       0 |   25.92 | 14-36             
  ...tsDisplay.tsx |     100 |      100 |     100 |     100 |                   
  ...ryDisplay.tsx |     100 |      100 |     100 |     100 |                   
  ...Indicator.tsx |   44.44 |      100 |       0 |   44.44 | 12-17             
  ...MoreLines.tsx |      60 |       25 |     100 |      60 | 24-27,33-40       
  StatsDisplay.tsx |   98.39 |    86.66 |     100 |   98.39 | 173-175           
  ...nsDisplay.tsx |   83.05 |    61.53 |     100 |   83.05 | 34-39,42-43,87-89 
  ThemeDialog.tsx  |    82.7 |       20 |      25 |    82.7 | ...,95-99,197-208 
  Tips.tsx         |      16 |      100 |       0 |      16 | 17-45             
  ...tsDisplay.tsx |     100 |     87.5 |     100 |     100 | 30-31             
  ...ification.tsx |   36.36 |      100 |       0 |   36.36 | 15-22             
 ...nents/messages |   62.67 |     80.9 |   57.89 |   62.67 |                   
  ...onMessage.tsx |   18.51 |      100 |       0 |   18.51 | 22-49             
  DiffRenderer.tsx |   96.31 |    82.66 |     100 |   96.31 | ...01-202,206,271 
  ErrorMessage.tsx |   22.22 |      100 |       0 |   22.22 | 16-31             
  ...niMessage.tsx |   18.51 |      100 |       0 |   18.51 | 20-43             
  ...geContent.tsx |   19.04 |      100 |       0 |   19.04 | 25-43             
  InfoMessage.tsx  |     100 |      100 |     100 |     100 |                   
  ...onMessage.tsx |   38.91 |     62.5 |   33.33 |   38.91 | ...32-164,200-225 
  ...upMessage.tsx |     9.3 |      100 |       0 |     9.3 | 26-123            
  ToolMessage.tsx  |   87.76 |       80 |     100 |   87.76 | ...,91-95,169-171 
  UserMessage.tsx  |     100 |      100 |     100 |     100 |                   
  ...llMessage.tsx |   36.36 |      100 |       0 |   36.36 | 17-25             
 ...ponents/shared |    74.7 |    71.95 |   85.18 |    74.7 |                   
  MaxSizedBox.tsx  |   78.51 |    79.43 |   88.88 |   78.51 | ...36-438,537-538 
  ...tonSelect.tsx |   88.05 |    92.85 |      60 |   88.05 | ...,68-71,108-109 
  text-buffer.ts   |   72.44 |    67.24 |    92.3 |   72.44 | ...1235-1238,1243 
 src/ui/contexts   |    92.8 |    86.36 |     100 |    92.8 |                   
  ...owContext.tsx |   91.07 |    81.81 |     100 |   91.07 | 46-47,59-61       
  ...onContext.tsx |     100 |      100 |     100 |     100 |                   
  ...ngContext.tsx |   71.42 |       50 |     100 |   71.42 | 17-20             
 src/ui/editors    |   93.87 |    85.71 |   66.66 |   93.87 |                   
  ...ngsManager.ts |   93.87 |    85.71 |   66.66 |   93.87 | 53,67-68          
 src/ui/hooks      |   71.39 |    80.72 |   65.75 |   71.39 |                   
  ...dProcessor.ts |   82.07 |    82.75 |     100 |   82.07 | ...92-395,406-422 
  ...dProcessor.ts |      82 |    74.35 |      80 |      82 | ...15-324,328-329 
  ...dProcessor.ts |   67.17 |    77.12 |   45.45 |   67.17 | ...1199-1204,1207 
  ...uthCommand.ts |    7.14 |      100 |       0 |    7.14 | 17-76             
  ...tIndicator.ts |     100 |      100 |     100 |     100 |                   
  ...ketedPaste.ts |     100 |      100 |     100 |     100 |                   
  useCompletion.ts |   68.62 |    86.17 |      80 |   68.62 | ...14-420,485-488 
  ...leMessages.ts |   96.96 |    88.88 |     100 |   96.96 | 25-26             
  ...orSettings.ts |     100 |      100 |     100 |     100 |                   
  ...miniStream.ts |   67.49 |    70.33 |     100 |   67.49 | ...36,638,687-777 
  ...BranchName.ts |   91.66 |    84.61 |     100 |   91.66 | 57-63             
  ...oryManager.ts |   98.41 |    93.33 |     100 |   98.41 | 43                
  ...putHistory.ts |    92.5 |    85.71 |     100 |    92.5 | 62-63,71,93-95    
  useKeypress.ts   |    60.6 |       50 |     100 |    60.6 | ...1,75-76,92-101 
  ...gIndicator.ts |     100 |      100 |     100 |     100 |                   
  useLogger.ts     |      25 |      100 |       0 |      25 | 14-32             
  ...raseCycler.ts |    95.5 |       75 |     100 |    95.5 | ...66-167,185-187 
  ...cySettings.ts |     4.5 |      100 |       0 |     4.5 | 19-135            
  ...lScheduler.ts |   79.01 |    94.87 |     100 |   79.01 | ...00-203,293-303 
  ...oryCommand.ts |       0 |        0 |       0 |       0 | 1-7               
  ...ellHistory.ts |   91.95 |    79.16 |   83.33 |   91.95 | 28-30,41-42,87-88 
  ...oryCommand.ts |       0 |        0 |       0 |       0 | 1-75              
  ...tateAndRef.ts |   95.45 |    66.66 |     100 |   95.45 | 25                
  ...rminalSize.ts |   77.27 |      100 |      50 |   77.27 | 19-23             
  ...emeCommand.ts |   64.28 |     87.5 |     100 |   64.28 | ...,92-93,100-106 
  useTimer.ts      |   88.09 |    85.71 |     100 |   88.09 | 44-45,51-53       
 src/ui/privacy    |   13.77 |      100 |       0 |   13.77 |                   
  ...acyNotice.tsx |    9.58 |      100 |       0 |    9.58 | 20-113            
  ...acyNotice.tsx |    12.9 |      100 |       0 |    12.9 | 15-55             
  ...acyNotice.tsx |   10.81 |      100 |       0 |   10.81 | 15-58             
  ...acyNotice.tsx |   30.76 |      100 |       0 |   30.76 | 19-36,39-41       
 src/ui/themes     |   99.42 |    89.47 |     100 |   99.42 |                   
  ansi-light.ts    |     100 |      100 |     100 |     100 |                   
  ansi.ts          |     100 |      100 |     100 |     100 |                   
  atom-one-dark.ts |     100 |      100 |     100 |     100 |                   
  ayu-light.ts     |     100 |      100 |     100 |     100 |                   
  ayu.ts           |     100 |      100 |     100 |     100 |                   
  default-light.ts |     100 |      100 |     100 |     100 |                   
  default.ts       |     100 |      100 |     100 |     100 |                   
  dracula.ts       |     100 |      100 |     100 |     100 |                   
  github-dark.ts   |     100 |      100 |     100 |     100 |                   
  github-light.ts  |     100 |      100 |     100 |     100 |                   
  googlecode.ts    |     100 |      100 |     100 |     100 |                   
  no-color.ts      |     100 |      100 |     100 |     100 |                   
  ...-of-purple.ts |     100 |      100 |     100 |     100 |                   
  theme-manager.ts |   89.77 |    84.21 |     100 |   89.77 | 66,98-103,108-109 
  theme.ts         |   98.44 |    94.73 |     100 |   98.44 | 304-307           
  xcode.ts         |     100 |      100 |     100 |     100 |                   
 src/ui/utils      |   78.47 |     84.1 |   97.14 |   78.47 |                   
  ...Colorizer.tsx |    80.5 |     87.5 |     100 |    80.5 | 88-89,159-183     
  ...nRenderer.tsx |   57.57 |    45.45 |     100 |   57.57 | ...21-129,131-133 
  ...wnDisplay.tsx |   64.24 |    71.69 |      75 |   64.24 | ...94-322,354-378 
  ...eRenderer.tsx |     100 |    96.29 |     100 |     100 | 121               
  commandUtils.ts  |     100 |      100 |     100 |     100 |                   
  computeStats.ts  |     100 |      100 |     100 |     100 |                   
  displayUtils.ts  |     100 |      100 |     100 |     100 |                   
  errorParsing.ts  |     100 |      100 |     100 |     100 |                   
  formatters.ts    |   90.47 |       96 |     100 |   90.47 | 57-60             
  ...nUtilities.ts |   69.84 |    85.71 |     100 |   69.84 | 75-91,100-101     
  textUtils.ts     |   93.93 |    92.85 |     100 |   93.93 | 14-15             
  updateCheck.ts   |     100 |      100 |     100 |     100 |                   
 src/utils         |    8.78 |    77.77 |    62.5 |    8.78 |                   
  cleanup.ts       |   91.66 |       50 |     100 |   91.66 | 18                
  package.ts       |   88.88 |    85.71 |     100 |   88.88 | 33-34             
  readStdin.ts     |    3.44 |      100 |       0 |    3.44 | 7-39              
  sandbox.ts       |       0 |        0 |       0 |       0 | 1-871             
  ...upWarnings.ts |   23.07 |      100 |       0 |   23.07 | 14-40             
  ...upWarnings.ts |     100 |      100 |     100 |     100 |                   
  version.ts       |     100 |       50 |     100 |     100 | 11                
-------------------|---------|----------|---------|---------|-------------------

Core Package - Full Text Report

-------------------|---------|----------|---------|---------|-------------------
File               | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
-------------------|---------|----------|---------|---------|-------------------
All files          |   71.99 |    80.97 |    74.4 |   71.99 |                   
 src               |     100 |      100 |     100 |     100 |                   
  index.ts         |     100 |      100 |     100 |     100 |                   
 src/__mocks__/fs  |     100 |      100 |     100 |     100 |                   
  promises.ts      |     100 |      100 |     100 |     100 |                   
 src/code_assist   |   61.96 |    74.32 |      65 |   61.96 |                   
  codeAssist.ts    |   29.41 |      100 |       0 |   29.41 | 12-24             
  converter.ts     |   89.06 |    96.15 |   81.81 |   89.06 | 176-180,199-209   
  oauth2.ts        |    69.6 |    54.28 |   83.33 |    69.6 | ...94-296,300-302 
  server.ts        |    46.4 |       80 |   53.84 |    46.4 | ...60-201,204-206 
  setup.ts         |   10.34 |      100 |       0 |   10.34 | 19-22,30-85       
  types.ts         |     100 |      100 |     100 |     100 |                   
 src/config        |   68.04 |    90.16 |   58.49 |   68.04 |                   
  config.ts        |   67.68 |    90.16 |   58.49 |   67.68 | ...50-455,472-522 
  models.ts        |     100 |      100 |     100 |     100 |                   
 src/core          |   71.28 |     75.6 |   77.52 |   71.28 |                   
  client.ts        |   70.74 |    73.84 |   85.71 |   70.74 | ...15,527-528,531 
  ...tGenerator.ts |   44.44 |     62.5 |      50 |   44.44 | 50-99,121,132-135 
  ...lScheduler.ts |   72.07 |    69.33 |   81.25 |   72.07 | ...51-657,674-683 
  geminiChat.ts    |   64.97 |    75.67 |   64.28 |   64.97 | ...61,576,580-588 
  geminiRequest.ts |     100 |      100 |     100 |     100 |                   
  logger.ts        |   82.71 |    78.26 |     100 |   82.71 | ...72-276,285-286 
  modelCheck.ts    |    4.65 |      100 |       0 |    4.65 | 20-68             
  ...olExecutor.ts |     100 |    66.66 |     100 |     100 | 63,93             
  prompts.ts       |   81.05 |    68.18 |      50 |   81.05 | ...60-265,281-338 
  tokenLimits.ts   |      15 |      100 |       0 |      15 | 15-31             
  turn.ts          |   80.67 |    81.48 |     100 |   80.67 | ...13-216,229-230 
 src/services      |   77.71 |    86.11 |   70.58 |   77.71 |                   
  ...eryService.ts |   77.33 |       75 |   85.71 |   77.33 | ...1-62,84,91-103 
  gitService.ts    |   78.02 |      100 |      60 |   78.02 | ...10-114,117-121 
 src/telemetry     |   82.05 |    86.77 |   82.22 |   82.05 |                   
  constants.ts     |     100 |      100 |     100 |     100 |                   
  index.ts         |     100 |      100 |     100 |     100 |                   
  loggers.ts       |   82.94 |       85 |    87.5 |   82.94 | 160-199           
  metrics.ts       |   60.36 |    95.65 |    62.5 |   60.36 | ...36-158,161-184 
  sdk.ts           |   82.85 |    28.57 |     100 |   82.85 | ...18,126-127,133 
  types.ts         |   85.79 |     97.5 |   73.33 |   85.79 | ...50-155,158-171 
  uiTelemetry.ts   |   99.18 |    95.65 |     100 |   99.18 | 119               
 ...learcut-logger |   80.75 |    81.25 |   72.22 |   80.75 |                   
  ...cut-logger.ts |   80.51 |    80.64 |   76.47 |   80.51 | ...02-414,417-419 
  ...tadata-key.ts |    82.6 |      100 |       0 |    82.6 | 143-153           
 src/tools         |   63.76 |    81.31 |   69.85 |   63.76 |                   
  diffOptions.ts   |     100 |      100 |     100 |     100 |                   
  edit.ts          |    77.9 |    82.19 |   84.61 |    77.9 | ...53-454,458-493 
  glob.ts          |   87.03 |       80 |   85.71 |   87.03 | ...93-294,302-309 
  grep.ts          |   56.81 |     75.8 |   72.72 |   56.81 | ...31-536,541-545 
  ls.ts            |    8.55 |      100 |    12.5 |    8.55 | ...89-194,202-312 
  mcp-client.ts    |   75.65 |    78.33 |      30 |   75.65 | ...73-374,393-394 
  mcp-tool.ts      |   91.75 |    85.71 |     100 |   91.75 | ...36,142,147-148 
  memoryTool.ts    |   97.43 |    84.84 |     100 |   97.43 | 93,95,97-98       
  ...iable-tool.ts |     100 |    84.61 |     100 |     100 | 90,97             
  read-file.ts     |   98.42 |    96.15 |     100 |   98.42 | 124-125           
  ...many-files.ts |   79.78 |    75.86 |   83.33 |   79.78 | ...67-468,475-476 
  shell.ts         |   31.28 |      100 |   58.33 |   31.28 | ...56-278,281-495 
  tool-registry.ts |   58.24 |    73.46 |   78.57 |   58.24 | ...99-304,310-311 
  tools.ts         |   76.31 |      100 |      40 |   76.31 | ...44-145,154-159 
  web-fetch.ts     |   31.29 |    72.22 |   66.66 |   31.29 | ...89-190,218-356 
  web-search.ts    |   10.92 |      100 |      20 |   10.92 | ...06-107,110-199 
  write-file.ts    |   81.49 |    82.35 |      80 |   81.49 | ...22-327,389-419 
 src/utils         |   81.75 |    82.37 |   87.25 |   81.75 |                   
  LruCache.ts      |   70.96 |     62.5 |     100 |   70.96 | 20-22,28,30-34    
  bfsFileSearch.ts |   92.45 |    86.66 |     100 |   92.45 | 53-54,67-68       
  editCorrector.ts |   77.35 |    61.11 |   91.66 |   77.35 | ...70-682,716,730 
  editor.ts        |   97.33 |    93.75 |     100 |   97.33 | 134,196,199-200   
  ...rReporting.ts |   83.52 |    84.61 |     100 |   83.52 | 81-85,106-114     
  errors.ts        |   41.46 |       60 |      75 |   41.46 | 17-21,37-52,56-62 
  fetch.ts         |   34.04 |      100 |       0 |   34.04 | 22-27,31-57       
  fileUtils.ts     |   94.69 |    91.54 |     100 |   94.69 | ...45-249,318-324 
  ...eUtilities.ts |   96.03 |       96 |     100 |   96.03 | 28-29,57-58       
  ...rStructure.ts |      95 |    93.42 |     100 |      95 | ...66-167,344-346 
  ...noreParser.ts |   96.36 |     91.3 |     100 |   96.36 | 70-71             
  gitUtils.ts      |   46.34 |    66.66 |      50 |   46.34 | 24-25,40-41,50-73 
  ...yDiscovery.ts |    80.7 |    75.43 |      75 |    80.7 | ...12-313,316-317 
  ...tProcessor.ts |   86.86 |    88.46 |     100 |   86.86 | 115-124,131-140   
  ...Inspectors.ts |     100 |      100 |     100 |     100 |                   
  ...kerChecker.ts |   83.14 |    82.35 |     100 |   83.14 | ...,94-99,107-113 
  paths.ts         |   62.79 |    73.33 |   57.14 |   62.79 | ...23-133,139-140 
  retry.ts         |   68.85 |    73.91 |     100 |   68.85 | ...91-211,256-271 
  ...aValidator.ts |     100 |      100 |     100 |     100 |                   
  session.ts       |     100 |      100 |     100 |     100 |                   
  testUtils.ts     |   84.44 |    72.72 |   83.33 |   84.44 | 27-28,34-35,70-72 
  user_id.ts       |   63.63 |    33.33 |      80 |   63.63 | ...57,74-75,78-79 
-------------------|---------|----------|---------|---------|-------------------

For detailed HTML reports, please see the 'coverage-reports-24.x' artifact from the main CI run.

packages/cli/src/gemini.tsx

…oogle-gemini#3483)

Initalize MCP tools once on start up.

123a300

scidomino requested a review from a team as a code owner July 7, 2025 21:31

gemini-code-assist bot reviewed Jul 7, 2025

View reviewed changes

packages/core/src/config/config.ts Show resolved Hide resolved

Merge branch 'main' into tomm_auth_mcp

e20d27a

NTaylorMullen approved these changes Jul 7, 2025

View reviewed changes

packages/cli/src/gemini.tsx Show resolved Hide resolved

scidomino added this pull request to the merge queue Jul 7, 2025

Merged via the queue into main with commit 357546a Jul 7, 2025
13 checks passed

scidomino deleted the tomm_auth_mcp branch July 7, 2025 22:04

yulin0629 pushed a commit to yulin0629/gemini-cli that referenced this pull request Jul 8, 2025

Initialize MCP tools once at start up instead of every time we auth. (g…

576288d

…oogle-gemini#3483)

slhck pushed a commit to slhck/gemini-cli that referenced this pull request Jul 9, 2025

Initialize MCP tools once at start up instead of every time we auth. (g…

f22a88d

…oogle-gemini#3483)

scidomino mentioned this pull request Jul 11, 2025

Don't print MCP server logs. It's too noisy. #3867

Merged

JunYang-tes pushed a commit to JunYang-tes/gemini-cli.nvim that referenced this pull request Jul 12, 2025

Initialize MCP tools once at start up instead of every time we auth. (g…

f338923

…oogle-gemini#3483)

sportsculture mentioned this pull request Jul 17, 2025

Sync with upstream Google gemini-cli (2025-01-16) sportsculture/sport-cli#1

Merged

davenportjw pushed a commit to davenportjw/gemini-cli that referenced this pull request Jul 24, 2025

Initialize MCP tools once at start up instead of every time we auth. (g…

b6192a8

…oogle-gemini#3483)

swissspidy mentioned this pull request Aug 23, 2025

Add MCP loading indicator #3638

Closed

involvex pushed a commit to involvex/gemini-cli that referenced this pull request Sep 11, 2025

Initialize MCP tools once at start up instead of every time we auth. (g…

951f6a3

…oogle-gemini#3483)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Initialize MCP tools once at start up instead of every time we auth. #3483

Initialize MCP tools once at start up instead of every time we auth. #3483

Uh oh!

scidomino commented Jul 7, 2025 •

edited

Loading

Uh oh!

gemini-code-assist bot left a comment

Uh oh!

gemini-code-assist bot left a comment

Uh oh!

Uh oh!

github-actions bot commented Jul 7, 2025 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Initialize MCP tools once at start up instead of every time we auth. #3483

Initialize MCP tools once at start up instead of every time we auth. #3483

Uh oh!

Conversation

scidomino commented Jul 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

TLDR

Dive Deeper

Reviewer Test Plan

Testing Matrix

Linked issues / bugs

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Summary of Changes

Highlights

Footnotes

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

github-actions bot commented Jul 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code Coverage Summary

Uh oh!

Uh oh!

Uh oh!

Uh oh!

scidomino commented Jul 7, 2025 •

edited

Loading

github-actions bot commented Jul 7, 2025 •

edited

Loading