logoalt Hacker News

loevborgtoday at 9:23 AM3 repliesview on HN

Can you give an example? Looks fairly decent to me


Replies

Insensitivitytoday at 9:29 AM

the "useCanUseTool.tsx" hook, is definitely something I would hate seeing in any code base I come across.

It's extremely nested, it's basically an if statement soup

`useTypeahead.tsx` is even worse, extremely nested, a ton of "if else" statements, I doubt you'd look at it and think this is sane code

show 5 replies
wklmtoday at 10:36 AM

have a look at src/bootstrap/state.ts :D

q3ktoday at 9:36 AM

  1. Randomly peeking at process.argv and process.env all around. Other weird layering violations, too.
  2. Tons of repeat code, eg. multiple ad-hoc implementations of hash functions / PRNGs.
  3. Almost no high-level comments about structure - I assume all that lives in some CLAUDE.md instead.
show 3 replies