Skip to content

Conversation

@jpoly1219
Copy link
Contributor

@jpoly1219 jpoly1219 commented Aug 29, 2025

Description

Closes CON-3748.

Add documentation for next edit codebase.

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-general-review or @continue-detailed-review

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Summary by cubic

Add developer-facing documentation across Next Edit and the VS Code extension to explain architecture, state flow (chains/jumps), and why prefetch/region-finding paths are currently unused. Addresses CON-3748 by making the Next Edit codebase easier to understand and maintain, with no behavior changes.

  • Refactors
    • Removed unused _calculateOptimalEditableRegion and stale comments.
    • Clarified that prefetch queue and deterministic editable-region logic are not used; kept wrappers for potential future use.
    • Added clear entry-point notes, rendering flow, and toggling guidance for Next Edit (including where to enable/disable in tests).

@jpoly1219 jpoly1219 requested a review from a team as a code owner August 29, 2025 23:25
@jpoly1219 jpoly1219 requested review from RomneyDa and removed request for a team August 29, 2025 23:25
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Aug 29, 2025
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Aug 29, 2025
@github-actions
Copy link

⚠️ AI review completed but no review output was generated. Check the action logs for details.


💡 To request a new review, comment @continue-general-review

Copy link
Collaborator

@RomneyDa RomneyDa left a comment

Choose a reason for hiding this comment

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

Helpful comments for next edit!

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Sep 2, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 2, 2025
@RomneyDa RomneyDa merged commit b294bb0 into main Sep 2, 2025
55 checks passed
@RomneyDa RomneyDa deleted the jacob/con-3748 branch September 2, 2025 17:38
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Sep 2, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Sep 2, 2025
@sestinj
Copy link
Contributor

sestinj commented Sep 3, 2025

🎉 This PR is included in version 1.13.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Sep 3, 2025

🎉 This PR is included in version 1.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

lgtm This PR has been approved by a maintainer released size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants