The duplicate code analysis (#8552) flagged CompileToolResponseFilter and CompileToolResponseFilterWithVars in internal/middleware/jqschema.go for sharing an identical ~20-line cache → parse → compile → store → return body.

Resolution

The private generic helper compileToolResponseFilterInternal[K comparable] has been extracted and both public functions now delegate to it:

// Simple case: cache key is the filter string itself
func CompileToolResponseFilter(filter string) (*gojq.Code, error) {
    return compileToolResponseFilterInternal(filter, filter, secureCompileOpts, ...)
}

// Parameterized case: cache key is a composite struct
func CompileToolResponseFilterWithVars(filter string, varNames []string) (*gojq.Code, error) {
    cacheKey := toolResponseFilterVarsCacheKey{filter, buildVarNamesCacheKey(varNames)}
    return compileToolResponseFilterInternal(cacheKey, filter, compileOptsWithVariables(varNames), ...)
}

The generic K comparable type parameter lets both call sites use distinct cache key types (string vs toolResponseFilterVarsCacheKey) while sharing a single implementation of the caching, parsing, and compilation logic.