Skip to content

Refactor to split up big files #1186

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.

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

Conversation

GrahamTheCoder
Copy link
Member

@GrahamTheCoder GrahamTheCoder commented Jun 16, 2025

Breaking down the files should help tools - especially LLMs - deal with the codebase better and more cheaply since most don't integrate with file outlining tools yet to pull individual members.
So long as the groupings of things are sane, they will be helpful to humans at least sometimes.

Prompt

The solution has several large files in them. First find the top 10 largest .cs files. Then one at a time, assess which methods naturally group together in a semantically coherent nameable chunk, and extract it into a separate file of no more than 250 lines. Always try to have more than one method per file though. If the file is a test file, this should be simple enough, consider using folders to group the files based on their original name. For production code, e.g. the Visitor classes, they could become partial classes instead e.g. DeclaredNodeVisitor.MethodDeclaration.cs

… done so far and provide feedback for Jules to continue.
… done so far and provide feedback for Jules to continue.
@GrahamTheCoder GrahamTheCoder mentioned this pull request Jun 23, 2025
@GrahamTheCoder
Copy link
Member Author

Superseded: #1187

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.

1 participant