feat: Explicit OAuth Configuration for Remote MCP Servers (#2248) #4898
dgageotAnnotations
12 errors and 1 warning
|
lint:
pkg/audit/audit.go#L378
paramTypeCombine: func(ctx context.Context, sess *session.Session, agentName string, input string, prompt string, confirmed bool) (*AuditRecord, error) could be replaced with func(ctx context.Context, sess *session.Session, agentName, input, prompt string, confirmed bool) (*AuditRecord, error) (gocritic)
|
|
lint:
pkg/audit/audit.go#L365
paramTypeCombine: func(ctx context.Context, sess *session.Session, agentName string, reason string) (*AuditRecord, error) could be replaced with func(ctx context.Context, sess *session.Session, agentName, reason string) (*AuditRecord, error) (gocritic)
|
|
lint:
pkg/audit/audit.go#L329
paramTypeCombine: func(ctx context.Context, sess *session.Session, agentName string, command string, args []string, env map[string]string, workDir string, output string, exitCode int, duration time.Duration) (*AuditRecord, error) could be replaced with func(ctx context.Context, sess *session.Session, agentName, command string, args []string, env map[string]string, workDir, output string, exitCode int, duration time.Duration) (*AuditRecord, error) (gocritic)
|
|
lint:
pkg/audit/audit.go#L306
paramTypeCombine: func(ctx context.Context, sess *session.Session, agentName string, method, url string, headers map[string]string, body string, statusCode int, response string, duration time.Duration) (*AuditRecord, error) could be replaced with func(ctx context.Context, sess *session.Session, agentName, method, url string, headers map[string]string, body string, statusCode int, response string, duration time.Duration) (*AuditRecord, error) (gocritic)
|
|
lint:
pkg/audit/audit.go#L291
paramTypeCombine: func(ctx context.Context, sess *session.Session, agentName string, actionType ActionType, path string, content string, mode string) (*AuditRecord, error) could be replaced with func(ctx context.Context, sess *session.Session, agentName string, actionType ActionType, path, content, mode string) (*AuditRecord, error) (gocritic)
|
|
lint:
pkg/runtime/runtime.go#L18
File is not properly formatted (gci)
|
|
lint:
pkg/audit/audit_test.go#L197
use of `context.Background` forbidden because "do not use context.Background() in tests, use t.Context()" (forbidigo)
|
|
lint:
pkg/audit/audit_test.go#L159
use of `context.Background` forbidden because "do not use context.Background() in tests, use t.Context()" (forbidigo)
|
|
lint:
pkg/audit/audit_test.go#L98
use of `context.Background` forbidden because "do not use context.Background() in tests, use t.Context()" (forbidigo)
|
|
lint:
pkg/tools/mcp/oauth_config_test.go#L110
Error return value of `server.Shutdown` is not checked (errcheck)
|
|
build-and-test
Process completed with exit code 201.
|
|
Task 'test' failed
exit status 1
|
|
build-and-test
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: go-task/setup-task@0ab1b2a65bc55236a3bc64cde78f80e20e8885c2. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|