Skip to content

Increase default repo-memory MaxFileSize from 10KB to 100KB#32826

Merged
pelikhan merged 1 commit into
mainfrom
copilot/increase-size-limit-to-100kb
May 17, 2026
Merged

Increase default repo-memory MaxFileSize from 10KB to 100KB#32826
pelikhan merged 1 commit into
mainfrom
copilot/increase-size-limit-to-100kb

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 17, 2026

push_repo_memory was failing for the sergo workflow because sergo-stats.json (11744 bytes) exceeded the default MAX_FILE_SIZE of 10240 bytes.

Changes

  • pkg/workflow/repo_memory.go: Added defaultRepoMemoryMaxFileSize = 102400 constant; replaced all hardcoded 10240 defaults for MaxFileSize with the new constant
  • pkg/workflow/repo_memory_test.go: Updated default value assertion to 102400
  • pkg/parser/schemas/main_workflow_schema.json: Updated max-file-size field descriptions to reflect 100KB default

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan marked this pull request as ready for review May 17, 2026 14:19
Copilot AI review requested due to automatic review settings May 17, 2026 14:19
@pelikhan pelikhan merged commit 176fada into main May 17, 2026
@pelikhan pelikhan deleted the copilot/increase-size-limit-to-100kb branch May 17, 2026 14:19
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Increases the default repo-memory maximum file size from 10KB to 100KB and updates related schema/test expectations.

Changes:

  • Added a shared defaultRepoMemoryMaxFileSize constant.
  • Replaced repo-memory default max-file-size values with 102400 bytes.
  • Updated schema descriptions and one default-value assertion.
Show a summary per file
File Description
pkg/workflow/repo_memory.go Defines and applies the new 100KB repo-memory max-file-size default.
pkg/workflow/repo_memory_test.go Updates the boolean-default test expectation to 102400.
pkg/parser/schemas/main_workflow_schema.json Updates schema descriptions for max-file-size default text.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comments suppressed due to low confidence (1)

pkg/workflow/repo_memory.go:342

  • The object-style repo-memory default also changed here, but the object test currently supplies an explicit max-file-size, so this default path is not exercised. Please add coverage for object notation without max-file-size to confirm it now uses 102400.
			MaxFileSize:  defaultRepoMemoryMaxFileSize,         // 100KB default
  • Files reviewed: 3/3 changed files
  • Comments generated: 2

"minimum": 1,
"maximum": 104857600,
"description": "Maximum size per file in bytes (default: 10240 = 10KB)"
"description": "Maximum size per file in bytes (default: 102400 = 100KB)"
if memoryMap, ok := item.(map[string]any); ok {
entry := RepoMemoryEntry{
MaxFileSize: 10240, // 10KB default
MaxFileSize: defaultRepoMemoryMaxFileSize, // 100KB default
github-actions Bot added a commit that referenced this pull request May 18, 2026
- Document new checkout.clean-git-credentials field (#32819)
- Fix max-file-size default in repo-memory reference (10KB -> 100KB, #32826)
- Add AgentDB to shared MCP servers list (#32771)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants