2 pointsby riggo5 hours ago2 comments
  • anee7693 hours ago
    The hardest part of review tools like this is signal-to-noise — one round of false positives and people start reflex-accepting or muting it. How does Ox decide what's worth flagging on a given PR, and does it dedupe against pre-existing debt so it only surfaces what the change actually introduced? In my company, we use CodeRabbit with Azure DevOps and it flags a lot of things and mostly they were false positives so eventually we stopped relying on it.
    • riggo3 hours ago
      Existing debt is currently handled in a debt queue structure. If you ignore a suggestion once it will remember that and store it in that queue so that it doesn't bring it up again. There is a mechanism to review that queue but I haven't totally thought out how to resurface those in a convenient manner.

      There are obvious solutions like dumping those ignored changes to Jira issues so you have a pipeline of debt to address though I feel like this just moves the issue so I'm trying to think of something more proactive.

      • 3 hours ago
        undefined
    • manu143 hours ago
      I had same issues as such tools provide noise more than something concrete.
      • riggo3 hours ago
        Right it's key that noise is reduced or eliminated(aspirational but I'll try).
  • johnsillings5 hours ago
    Cool project. How is this different from Greptile, Cubic, etc?
    • riggo5 hours ago
      It's very similar to those with two key distinctions. Ox aims to be more simple in it's behavior to not add work to the review process and this approach allows Ox to operate more cheaply while delivering everything competitors do. In sum it is more simple (on purpose).