The #Cursor prompt:
Remove the Bookwyrm code for now
removed nearly 300 lines of code and touched every file 😱
Removing "unnecessary" only deleted 6 lines (so the problem maybe wasn't as bad as I initially thought)
Remove all "game" related code surprisingly added on LOC, a comment.
But funny enough it made the code far clearer to understand. Where previously "gamekeys" were used now it was "order_keys". Additionally, the comment that the get_gamekeys() method returns all order keys was added (not sure if this is true)
Sidenote: Switching the variable naming to suddenly use an '_' is a bit strange.
OK let's try one more thing before wrapping up this experiment and start over with smaller steps.
Let's try to add UnitTests!
3/n
I asked #Cursor to generate unit tests for the project.
The result? Does it spark joy (or confidence)?
😬
Again, not really 😕
Nope, that is not a good unit test. This test check the setting of an internal property. This should actually be an implementation detail.