Skip to content

fix: enhance Cursor and Qoder detection in SSH and DevContainer environments#2

Closed
yyuuttaaoo wants to merge 1 commit into
ai-coding-workshop:masterfrom
yyuuttaaoo:feature/cursor-qoder-detection
Closed

fix: enhance Cursor and Qoder detection in SSH and DevContainer environments#2
yyuuttaaoo wants to merge 1 commit into
ai-coding-workshop:masterfrom
yyuuttaaoo:feature/cursor-qoder-detection

Conversation

@yyuuttaaoo

Copy link
Copy Markdown
Contributor

This commit improves the detection logic for Cursor and Qoder IDEs by adding
pattern matching for environment variables VSCODE_GIT_ASKPASS_MAIN and BROWSER
that contain '.cursor-server' or '.qoder-server' substrings. This fixes issues
where the IDE detection was not working properly in SSH and DevContainer environments.

The changes include:

  1. Added new environment variable patterns in envConfigs for Cursor and Qoder detection
  2. Enhanced getCoDevelopedBy function to support substring pattern matching
  3. Added comprehensive tests to verify the new detection logic

This ensures that users working in remote development environments (SSH, DevContainer)
will have proper Co-developed-by trailers added to their commits when using Cursor or Qoder.

Change-Id: Ieb98755b1c0aa178b49484b2cdc613c11f90df39
Co-developed-by: Qoder noreply@qoder.com

…onments

This commit improves the detection logic for Cursor and Qoder IDEs by adding
pattern matching for environment variables VSCODE_GIT_ASKPASS_MAIN and BROWSER
that contain '.cursor-server' or '.qoder-server' substrings. This fixes issues
where the IDE detection was not working properly in SSH and DevContainer environments.

The changes include:
1. Added new environment variable patterns in envConfigs for Cursor and Qoder detection
2. Enhanced getCoDevelopedBy function to support substring pattern matching
3. Added comprehensive tests to verify the new detection logic

This ensures that users working in remote development environments (SSH, DevContainer)
will have proper Co-developed-by trailers added to their commits when using Cursor or Qoder.

Change-Id: Ieb98755b1c0aa178b49484b2cdc613c11f90df39
Co-developed-by: Qoder <noreply@qoder.com>
@jiangxin

Copy link
Copy Markdown
Contributor

Hi @yyuuttaaoo

我修改了你的提交说明,做了折行,避免提交说明单行过长。
还有我补充了一个提交使用 minimatch 提供 glob 字符串匹配,这样代码中不用针对星号进行特殊编码了。

@jiangxin

Copy link
Copy Markdown
Contributor

我先关闭这个 PR,请到 PR #3 查看你和我各一个提交。

@jiangxin jiangxin closed this Sep 25, 2025
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