bug: zip.go silently truncates large repos at maxTotalSize with no warning to the user

[claude]

In internal/zip/zip.go lines 94-96, when the accumulated file size hits maxTotalSize (10 MB), the walk is stopped by returning io.EOF. The caller treats io.EOF as normal success. No flag or error is returned to indicate the zip is incomplete. The run command in cmd/run.go never learns files were silently dropped, so it submits a partial repo to the API without any warning. For large repos this means graph analysis is based on a subset of files with no visible indication to the user.

Fix: Change RepoZip signature to return a truncated bool: func RepoZip(root string) ([]byte, bool, error). In cmd/run.go caller, log a warning if truncated is true. All callers of RepoZip in cmd/run.go and dryRunHandler in cmd/status.go need updating.

Affected files:

• internal/zip/zip.go:94-96
• cmd/run.go:118
• cmd/status.go dryRunHandler

@claude please implement this

[coderabbitai]

🔗 Related PRs

#18 - feat: show circular dependency warning in context bomb [closed]

---

📝 Issue Planner

_{Check the box below or use the @coderabbitai plan command to generate an implementation plan and prompts that you can use with your favorite coding assistant.}

• Create Plan

---

🧪 Issue enrichment is currently in open beta.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

💬 Have feedback or questions? Drop into our discord!

[greynewell]

@claude please implement this

[claude]

Claude finished @greynewell's task in 3m 8s —— View job • claude/issue-34-20260227-0311 • [Create PR ➔](https://github.com/supermodeltools/Uncompact/compare/main...claude/issue-34-20260227-0311?quick_pull=1&title=Issue%20%2334%3A%20Changes%20from%20Claude&body=This%20PR%20addresses%20issue%20%2334%0A%0AGenerated%20with%20%5BClaude%20Code%5D(https%3A%2F%2Fclaude.ai%2Fcode)

---

Implementation Complete

• Gather context and understand the request
• Read affected files (internal/zip/zip.go, cmd/run.go, cmd/status.go)
• Modify RepoZip to return truncated bool
• Update callers in cmd/run.go (runHandler and runWithoutCache)
• Update callers in cmd/pregen.go (also called RepoZip — discovered during build)
• Verify build passes (go build ./... and go vet ./...)
• Commit and push changes
• Create PR

Changes Made

internal/zip/zip.go

• Added var truncated bool to track when the size limit is hit
• Set truncated = true before returning io.EOF at the budget check
• Changed return signature from ([]byte, error) to ([]byte, bool, error)

cmd/run.go

• Updated runHandler (line 118) and runWithoutCache (line 192) to unpack the new truncated return value
• Added logFn("[warn] repo zip truncated at 10 MB limit — ...") when truncated is true

cmd/pregen.go (additional caller discovered during build)

• Updated both pregenFetch and the caching variant with the same pattern

Note: dryRunHandler in cmd/status.go does not call RepoZip (it only reads the cache or reports it would fetch), so no changes were needed there.

PR #38 |