Skip to content

AST-180812 - Detect VS Code Copilot CLI sessions from GITHUB_COPILOT_… - #1604

Merged
cx-rahul-pidde merged 1 commit into
stagingfrom
feature/AST-180812-vscode-copilot-cli-host
Oct 30, 2026
Merged

cx-rahul-pidde merged 1 commit into
stagingfrom
feature/AST-180812-vscode-copilot-cli-host

Conversation

@cx-rahul-pidde

Copy link
Copy Markdown
Contributor

…INTEGRATION_ID

VS Code's Copilot CLI chat sessions run the cx-devassist hooks without a VS Code PLUGIN_ROOT, so hostenv.PluginHost reported HostUnknown there and cx used the terminal scanner switches and threshold instead of the VS Code extension's cx-onboarding-ide.properties.

When PLUGIN_ROOT is not set, GITHUB_COPILOT_INTEGRATION_ID=vscode-chat (set by VS Code on the Copilot CLI engine it bundles) is HostVSCode again. A PLUGIN_ROOT still decides first, so the terminal Copilot CLI and other hosts are unchanged. Matches the cx-devassist Copilot plugin's _in_vscode.

By submitting this pull request, you agree to the terms within the Checkmarx Code of Conduct. Please review the contributing guidelines for guidance on creating high-quality pull requests.

Description

Please provide a summary of the changes and the related issue. Include relevant motivation and context.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Related Issues

Link any related issues or tickets.

Checklist

  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in downstream modules
  • I have updated the CLI help for new/changed functionality in this PR (if applicable)
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

Screenshots (if applicable)

Add screenshots to help explain your changes.

Additional Notes

Add any other relevant information.

…INTEGRATION_ID

VS Code's Copilot CLI chat sessions run the cx-devassist hooks without a
VS Code PLUGIN_ROOT, so hostenv.PluginHost reported HostUnknown there and
cx used the terminal scanner switches and threshold instead of the VS Code
extension's cx-onboarding-ide.properties.

When PLUGIN_ROOT is not set, GITHUB_COPILOT_INTEGRATION_ID=vscode-chat (set
by VS Code on the Copilot CLI engine it bundles) is HostVSCode again. A
PLUGIN_ROOT still decides first, so the terminal Copilot CLI and other
hosts are unchanged. Matches the cx-devassist Copilot plugin's _in_vscode.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cx-rahul-pidde
cx-rahul-pidde requested a review from a team as a code owner October 30, 2026 14:33

@cx-sumit-morchhale cx-sumit-morchhale left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved

@cx-rahul-pidde
cx-rahul-pidde merged commit f51009f into staging Oct 30, 2026
20 of 24 checks passed
cx-aniket-shinde added a commit that referenced this pull request Nov 5, 2026
* Honour the VS Code Checkmarx Self-Healing setting in Copilot agent hooks (AST-180812) (#1603)

* AST-180812 - Detect VS Code Copilot CLI sessions from GITHUB_COPILOT_INTEGRATION_ID (#1604)

* Feature/ast 180812 vscode copilot cli host (#1607)

* Update brotli dependency to v1.2.7 (AST-189161) (#1610)

* Update github.com/andybalholm/brotli from v1.2.0 to v1.2.7 (AST-189161)

Indirect dependency pulled in via anchore/syft -> anchore/archiver.
No API removals; archiver only uses NewWriterLevel/NewReader, which are unchanged.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* Update Docker base image checkmarx/bash:5.3 to latest digest (AST-189161)

Bumps the SHA256-pinned base image to the 5.3-r14 build pushed on 2026-11-04.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* Replace .trivyignore entries with current binary findings (AST-189161)

Ignore GO-2026-5932 (x/crypto/openpgp via helm provenance), CVE-2026-78661 and
CVE-2026-97045 (x/net/html via syft Maven cataloger). Remove the previous Wolfi
base-image and syft/docker entries, which no longer apply.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* Restore syft and docker entries in .trivyignore (AST-189161)

CVE-2026-33481 (syft temp cleanup) and CVE-2026-33997 (moby plugin privilege
bypass) are still reported against the binary. Neither has an adoptable fix:
syft >= 1.42.3 reintroduces the binary-size regression reverted in AST-188320
and docker/docker has no 29.x release on its module path. Justifications and
reachability analysis are recorded in the file.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* Bump golang.org/x/net to v0.62.0 and trim .trivyignore (AST-189161)

Bump the indirect golang.org/x/net dependency to v0.62.0, which fixes
CVE-2026-78661 and CVE-2026-97045, so their ignore entries are dropped.
Keep only the ignores that cannot be fixed directly: GO-2026-5932
(x/crypto openpgp, via helm), CVE-2026-33481 (syft) and CVE-2026-33997
(docker).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
Co-authored-by: atishj99 <atish.jadhav@checkmarx.com>

* revret exclude git config changes (#1609)

Co-authored-by: Aniket Shinde <aniket.shinde@checkmarx.com>

* Use self-repository syntax for reusable workflow in nightly-parallel (AST-189161)

Replace `uses: ./.github/workflows/ci-tests.yml` with the `$/` form to
satisfy the zizmor `self-repository` audit (added in zizmor v1.30.0).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Anurag Dalke <anurag.dalke@checkmarx.com>
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
Co-authored-by: atishj99 <atish.jadhav@checkmarx.com>
Co-authored-by: Sumit Morchhale <sumit.morchhale@checkmarx.com>
Co-authored-by: Aniket Shinde <aniket.shinde@checkmarx.com>
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