Live data from Hacker News

Show HN: I canceled my AI code reviewer and wrote a free local one

github.com

1–10 of 39 posts

Re: Show HN: I canceled my AI code reviewer and wrote a free local one

#8
I've never seen this idiom before:

  try:
      AVOUCH_VERSION = importlib.metadata.version("avouch")
  except importlib.metadata.PackageNotFoundError:
      AVOUCH_VERSION = "0.3.2"
What's the intended purpose? It seems to be protecting against a scenario where the package is not installed but the CLI is somehow running...

...But that seems like a very odd thing to be concerned about. Plus, unless I'm missing something, the program will crash with an an ImportError before that block can ever run.

Re: Show HN: I canceled my AI code reviewer and wrote a free local one

#9

I've never seen this idiom before: try: AVOUCH_VERSION = importlib.metadata.version("avouch") except importlib.metadata.PackageNotFoundError: AVOUCH_VERSION = "0.3.2" What's the intended purpose? It seems to be protecting against a scenario where the package is not installed but the CLI is somehow running... ...But that seems like a very odd thing to be concerned about. Plus, unless I'm missing something, the program…

Could be for unit testing, maybe it's something that makes sense for the developer, or it's AI slop. In all cases it's a minor nitpick in my opinion.

Re: Show HN: I canceled my AI code reviewer and wrote a free local one

#10
how is tjis better than (OpenCode|Pi|CC|Codex) that had a Reviewer agent? the whole idea of e.g. CodeRabbit is that it allows to review PRs autonomously. making it local defeats the purpose.

it doesnt help that these projects have AI generated README's. Not AI-assisted, or AI-curated, but fully generated. it's a bunch of sentences that say nothing. "the diff is whats reviewed". okay, what about per-commit reviews? does it keep context of that? just diffs isnt enough to review a PR, it just sounds efficient but youre leaving a lot of context out, so the reviewer will have trouble confidently reviewing your changes.

Post reply on HN