Skip to content

Increase file_args limit to 5 MB #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
shivashish-suman-glean merged 2 commits into from
Aug 10, 2026
+3 −3
Merged

Increase file_args limit to 5 MB #11

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0a088b5
Increase file_args limit to 5 MB
shivashish-suman-glean Aug 10, 2026
1fb3935
minor
shivashish-suman-glean Aug 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion sources/glean-vnext/skills/glean_run/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ run_tool({
Constraints:
- Paths must be absolute.
- A key in `file_args` must not also appear in `arguments`.
- Each file must be ≤ 1 MB (override via `GLEAN_FILE_ARG_MAX_BYTES`).
- Each file must be ≤ 5 MB (override via `GLEAN_FILE_ARG_MAX_BYTES`).

## Rules

Expand Down
2 changes: 1 addition & 1 deletion sources/glean-vnext/src/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ const RUN_TOOL_TOOL: Tool = {
"string. Use this to keep large values out of the inline call — " +
"long-form text (Slack message bodies, Confluence pages, doc " +
"contents) or a large structured argument (e.g. an agent spec). " +
"Paths must be absolute. Each file must be ≤ 1 MB (override " +
"Paths must be absolute. Each file must be ≤ 5 MB (override " +
"via GLEAN_FILE_ARG_MAX_BYTES). A key in `file_args` must not " +
"also appear in `arguments`.",
additionalProperties: { type: "string" },
Expand Down
2 changes: 1 addition & 1 deletion sources/glean-vnext/src/tools/run-tool.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import { callRemoteTool } from "../remote-client.js";
import { buildCompactArgs, writeApprovalArgsFile } from "./approval-args.js";
import { resolveSessionId } from "../session-id.js";

const DEFAULT_FILE_ARG_MAX_BYTES = 1 * 1024 * 1024;
const DEFAULT_FILE_ARG_MAX_BYTES = 5 * 1024 * 1024;

// How long a user has to respond to an approval prompt. The MCP SDK's own
// request timeout is 60s and, on expiry, elicitInput REJECTS — so unless we
Expand Down