chore: Bump code-client-go - get the file filter from the invocation context - #7082
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
This comment has been minimized.
This comment has been minimized.
00988eb to
30d2a18
Compare
PR Reviewer Guide 🔍
|
There was a problem hiding this comment.
Automatically approved — this PR is only a dependency version bump (no source code changes in this repo).
Scope verified: the diff touches exactly 4 files, all mechanical version/checksum updates: cliv2/go.mod, cliv2/go.sum, cliv2-private/go.mod, cliv2-private/go.sum, bumping github.com/snyk/code-client-go from v1.31.1 → v1.31.3 consistently across both modules.
Verification performed (semantic analysis, adversarial review, security scan, and code review, all with no findings):
- Both
go.sumchecksum pairs match the public Go checksum transparency log (sum.golang.org) byte-for-byte — no tampering. v1.31.3is a legitimate, tagged upstream release (commite316c23, PR snyk/code-client-go#171) described upstream as a no-behavior-change refactor; the interveningv1.31.2is a CI-only release.cliv2builds and vets cleanly against the new version (go build ./...,go vet ./...,go mod verifyall pass);go mod tidyproduces zero diff.- No known security advisories for
code-client-goat either version. // indirectscoping differences betweencliv2/go.mod(direct) andcliv2-private/go.mod(indirect) match the pre-existing pattern for every other shared extension dependency — not introduced by this PR.
Per this automation's policy, dependency-bump-only changes are auto-approved; any change that touches source code would not be.
Sent by Cursor Automation: Automatic PR verification


Pull Request Submission Checklist
What does this PR do?
Bumps
code-client-gofrom v1.31.1 to v1.31.3.In v1.31.3,
snyk code test's file listing obtains its file filter from the invocation context (invocationCtx.GetFileFilter(...)) instead of constructing one locally. The filter therefore arrives already wired to the invocation's configuration, which means filtering behaviour the framework gates on configuration now takes effect for Snyk Code.No CLI code changes are needed; this is the
go.mod/go.sumbump only.Where should the reviewer start?
cliv2/go.modandcliv2-private/go.mod— the version bump, identical in both runtimescliv2/go.sum,cliv2-private/go.sum— corresponding checksumsHow should this be manually tested?
Use a repository whose absolute path contains regex metacharacters, containing:
.gitignorewith*.logandnode_modulesnode_modules/lib/index.js(untracked, matched by thenode_modulesrule)src/app.js(not matched by any rule)Then run, from a build of this branch:
INTERNAL_SNYK_FILE_FILTER_METACHARACTER_FIX_ENABLED=false snyk code test "$PWD" -d
INTERNAL_SNYK_FILE_FILTER_METACHARACTER_FIX_ENABLED=true snyk code test "$PWD" -d
Expected, from the
Snyk Code file filteringandCoverage reportlines:.jsfilesnode_modules/lib/index.jsis not excludedsrc/app.jsWhat's the product update that needs to be communicated to CLI users?
None for this bump on its own. The filtering fixes it enables are feature-flagged and announced with their own rollouts.
What are the relevant tickets?
CLI-1717