logoalt Hacker News

Insensitivity • today at 9:29 AM • 5 replies • view on HN

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

Replies

Overpower0416 • today at 10:12 AM

  export function extractSearchToken(completionToken: {
    token: string;
    isQuoted?: boolean;
  }): string {
    if (completionToken.isQuoted) {
      // Remove @" prefix and optional closing "
      return completionToken.token.slice(2).replace(/"$/, '');
    } else if (completionToken.token.startsWith('@')) {
      return completionToken.token.substring(1);
    } else {
      return completionToken.token;
    }
  }
Why even use else if with return...
➕ show 2 replies
duckmysick • today at 11:27 AM

I'm not that familiar with TypeScript/JavaScript - what would be a proper way of handling complex logic? Switch statements? Decision tables?

➕ show 2 replies
luc_ • today at 9:34 AM

Fits with the origin story of Claude Code...

➕ show 1 reply
loevborg • today at 9:47 AM

useCanUseTool.tsx looks special, maybe it'scodegen'ed or copy 'n pasted? `_c` as an import name, no comments, use of promises instead of async function. Or maybe it's just bad vibing...

➕ show 1 reply
matltc • today at 10:17 AM

Lol even the name is crazy