Skip to content

fix(editor): make fold all reliable for large files - #2912

Merged
bajrangCoder merged 4 commits into
mainfrom
fix/fold-all-large-files
Sep 18, 2026
Merged

bajrangCoder merged 4 commits into
mainfrom
fix/fold-all-large-files

Conversation

@bajrangCoder

Copy link
Copy Markdown
Member

No description provided.

@greptile-apps

greptile-apps Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge, with the previously requested progressive-path and cancellation coverage now present.

Findings

  1. P2 Progressive Path Lacks Coverage

Summary

This PR makes “Fold all” complete large-file parsing incrementally during idle periods before applying all discovered folds.

  • Adds cancellable, progressive parsing with bounded idle-time budgets.
  • Cancels pending folding when the document changes or “Unfold all” runs.
  • Collects syntax and fold-service ranges into one ordered transaction.
  • Adds unit coverage for multi-slice completion and cancellation behavior.

Reviews (2) · Last reviewed commit: "Merge branch 'fix/fold-all-large-files' ..."

Comment thread src/cm/foldingCommands.js
@bajrangCoder

This comment was marked as outdated.

@bajrangCoder
bajrangCoder added this pull request to the merge queue Sep 18, 2026
Merged via the queue into main with commit 2803dda Sep 18, 2026
11 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in The Code Board - Acode Sep 18, 2026
@bajrangCoder
bajrangCoder deleted the fix/fold-all-large-files branch September 18, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant