nspcc-dev / neo-go

Go Node and SDK for the Neo blockchain
MIT License
123 stars 79 forks source link

Divide large code files into smaller ones #3614

Open lock9 opened 4 days ago

lock9 commented 4 days ago

Is your feature request related to a problem? Please describe.

Some files in the project, like blockchain.go, are too large for most AI models. With over 3,000 lines, it can't be directly 'pasted' into ChatGPT. I had to debug neo-go to get some things working, and locating what I needed was challenging.

Note: I don't intend to change any structure, just to 'split source code files into smaller chunks.'

Describe the solution you'd like

Split blockchain.go into smaller files (and possibly other large files as well).

Describe alternatives you've considered

Additional context

This is primarily a structural improvement to the code. It shouldn't affect the expected behavior. My tool already splits the file into smaller parts before sending it to ChatGPT, but this consumes far more API credits than necessary, and the responses are often inaccurate or incomplete. Having a 'human' divide the code will help the AI avoid mistakes.

roman-khimov commented 4 days ago

Likely these tools will be upgraded before we do this and it's hard for me to accept the idea of changing something in the code for tools that are supposed to be eating petabytes of data for breakfast. But I admit blockchain.go and RPC's server.go are not easy to navigate through for humans as well, so some refactoring can still be beneficial.

lock9 commented 4 days ago

These were the files that I 'struggled' the most:

Note: Having several files with the same name also made me get lost a few times (now that I know which one to use) Screenshot 2024-10-14 at 14 08 03

Screenshot 2024-10-14 at 14 08 50