Skip to content

[upstream #16246] #16246: Authenticated startup config drops mcpAppSandbox.operationLimits #153

Description

@berry-13

Raised by the local cross-family review of LibreChat-AI#16246 (finding local:be4a585:L-001), on code that predates that PR on canary.

buildPostLoginPayload in api/server/routes/config.js calls resolveMCPAppsPolicy with every mcpAppSandbox argument up to maxActionPreviewChars, but leaves out the last parameter, operationLimits. When a deployment sets mcpAppSandbox.operationLimits in librechat.yaml, the authenticated startup config therefore publishes the default operation limits, and the client enforces those instead of the configured ones.

Expected: pass appConfig?.mcpAppSandbox?.operationLimits as the final argument, and cover it in api/server/routes/__tests__/config.spec.js with a configured and a default case.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    area: apireviewctl followups statebugSomething isn't workingfixedreviewctl followups state

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions