Comment by q3k Comment by q3k 3 days ago 2 replies Copy Link View on Hacker News https://github.com/currentspace/capn-rs/blob/main/capnweb-co...mmm delicious slop may i have more please sir
Copy Link no_circuit 3 days ago Next Collapse Comment - Right, for example, there are no comments on why something there is something called a PromiseIdAllocator that starts with the magic number "1":https://github.com/currentspace/capn-rs/blob/a816bfca5fb6ae5...Yet there is a public interface that allows for initialization with "0":https://github.com/currentspace/capn-rs/blob/a816bfca5fb6ae5...It's like the LLM was able to predict that 1 is needed in the protocol, but wasn't relevant to check in the boilerplate.I don't have a problem with newtype-all-the-things to ensure correctness in some areas, but no comments/constants does not lead to confidence. Reply View | 0 replies
Copy Link scottydelta 3 days ago Prev Collapse Comment - So we are allowing Claude to make commits now? Smh Reply View | 0 replies
Right, for example, there are no comments on why something there is something called a PromiseIdAllocator that starts with the magic number "1":
https://github.com/currentspace/capn-rs/blob/a816bfca5fb6ae5...
Yet there is a public interface that allows for initialization with "0":
https://github.com/currentspace/capn-rs/blob/a816bfca5fb6ae5...
It's like the LLM was able to predict that 1 is needed in the protocol, but wasn't relevant to check in the boilerplate.
I don't have a problem with newtype-all-the-things to ensure correctness in some areas, but no comments/constants does not lead to confidence.