Skip to content

feat: To add support for Cline#548

Closed
shenjian-8 wants to merge 2 commits intolaravel:mainfrom
shenjian-8:feature-cline
Closed

feat: To add support for Cline#548
shenjian-8 wants to merge 2 commits intolaravel:mainfrom
shenjian-8:feature-cline

Conversation

@shenjian-8
Copy link

Support for Cline

Comment on lines +93 to +95
Platform::Darwin => $home.'/Library/Application Support/Code/User/globalStorage/saoudrizwan.claude-dev/settings/cline_mcp_settings.json',
Platform::Linux => $home.'/.config/Code/User/globalStorage/saoudrizwan.claude-dev/settings/cline_mcp_settings.json',
Platform::Windows => $home.'\AppData\Roaming\Code\User\globalStorage\saoudrizwan.claude-dev\settings\cline_mcp_settings.json',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this paths should be global not your paths.

Comment on lines +24 to +40
public function systemDetectionConfig(Platform $platform): array
{
return match ($platform) {
Platform::Darwin => [
'paths' => ['/Applications/Visual Studio Code.app'],
],
Platform::Linux => [
'command' => 'command -v code',
],
Platform::Windows => [
'paths' => [
'%ProgramFiles%\\Microsoft VS Code',
'%LOCALAPPDATA%\\Programs\\Microsoft VS Code',
],
],
};
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is for vscode and not cline if user has installed vscode that does not mean user has cline we need specific detection for cline. there seems to be cline cli which could work in this case to detect automatically.

Comment on lines +66 to +69
'autoApprove' => [],
'timeout' => 300,
'type' => 'stdio',
'disabled' => false,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how many of this fields are necessary fields ? if not we can just add necessary fields so server gets activated automatically.

@pushpak1300
Copy link
Member

Thanks for your pull request!

Unfortunately, i just check cline still does not support project level mcp so it doesn't make sense to add that at this point. cline/cline#2418

If applicable, please consider releasing your code as an additional package so that the community can still take advantage of your contributions!

@pushpak1300 pushpak1300 closed this Feb 9, 2026
@shenjian-8
Copy link
Author

Thanks for the suggestion! I'll extract this functionality into a standalone Laravel package and publish it on Packagist. Once ready, I'll share the link here for anyone interested. This approach keeps the core project lean while still providing the feature to users who need it.

@shenjian-8
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants