refactor: Get the file filter from the invocation context

[danskmt]

Description

What does this PR do?

Takes the file filter from the invocation context instead of building it from separately passed dependencies.

getFilesForPath now calls invocationCtx.GetFileFilter(...), so the filter arrives already wired to the invocation's configuration and logger. Any behaviour the framework gates on configuration — such as the ignore-rule metacharacter fix and the tracked-file handling from CLI-1411 — is picked up centrally rather than needing to be plumbed through this repo.

To reach that far down the call chain, the analysis seam now carries the invocation context rather than its decomposed parts:

    // before
    type OptionalAnalysisFunctions func(context.Context, string, func() *http.Client,
        *zerolog.Logger, configuration.Configuration, ui.UserInterface, analytics.Analytics) (...)

    // after
    type OptionalAnalysisFunctions func(workflow.InvocationContext, string) (...)

defaultAnalyzeFunction derives those six dependencies from the context at the top, and determineAnalyzeInput and getFilesForPath take it in place of config plus logger. EntryPointNative was already handing over values it had pulled off the invocation context, so the values reaching the analysis are the same ones as before — only the plumbing changed. net/http and pkg/analytics are no longer needed in the file.

No behaviour change and no public API change: OptionalAnalysisFunctions lives under internal/, so it is not importable outside this module.

Where should the reviewer start?

• internal/commands/code_workflow/native_workflow.go — the seam, and the three functions that now take the invocation context
• internal/commands/code_workflow/native_workflow_test.go — new testInvocationContext helper; its GetFileFilter builds a real FileFilter the way the framework does, so filtering is exercised rather than stubbed
• pkg/code/code_test.go — the five injected analysis closures now take (workflow.InvocationContext, string)

One test change is worth a look because the compiler cannot catch it: with a custom analysis function injected, EntryPointNative no longer calls GetUserInterface() or GetAnalytics() — those are the analysis function's business now. The affected expectations were relaxed to .AnyTimes() rather than removed, since the default path still needs them.

What are the relevant tickets?

https://snyksec.atlassian.net/browse/CLI-1717

Checklist

• Tests added and all succeed
• Linted
• README.md updated, if user-facing

🚨After having merged, please update the snyk-ls and CLI go.mod to pull in latest client.

[snyk-io]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scan Engine	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues
✅	Licenses	0	0	0	0	0 issues
✅	Code Security	0	0	0	0	0 issues
✅	Secrets	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[snyk-io]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scan Engine	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues
✅	Licenses	0	0	0	0	0 issues
✅	Code Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[robertolopezlopez]

This injects analysisFunc, so GetUserInterface, Context, and GetAnalytics are never called. Can we remove the stubs and keep exact expectations for the context methods EntryPointNative still uses? So we can get rid of the weak AnyTimes() check

[danskmt]

Thanks @robertolopezlopez, fixed it

[snyk-pr-review-bot]

PR Reviewer Guide 🔍

🧪 PR contains tests

🔒 No security concerns identified

⚡ No major issues detected

📚 Repository Context Analyzed This review considered 15 relevant code sections from 10 files (average relevance: 0.98) scan.go (2 segments, lines 1-250) README.md (lines 1-152) pkg/code/code.go (2 segments, lines 1-189) internal/deepcode/mocks/client.go (lines 1-84) internal/analysis/analysis.go (2 segments, lines 1-239) internal/deepcode/client.go (lines 1-269) internal/commands/code_workflow/code_client_helper.go (lines 1-83) ... and 3 more file(s)

🤖 Repository instructions applied (from AGENTS.md)