neo-go icon indicating copy to clipboard operation
neo-go copied to clipboard

Divide large code files into smaller ones

Open lock9 opened this issue 1 year ago • 2 comments

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.

  • Enhancement
  • Low priority

lock9 avatar Oct 14 '24 16:10 lock9

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.

roman-khimov avatar Oct 14 '24 16:10 roman-khimov

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

  • dao.go
  • rpcsrv/server.go
  • blockchain.go

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

lock9 avatar Oct 14 '24 16:10 lock9