Show HN: Stage – Putting humans back in control of code review

(stagereview.app)

12 points | by cpan22 18 hours ago

5 comments

  • gracealwan 14 hours ago
    Totally different part of the reviewing experience, but I would love to see PR comments (or any revisions really) be automatically synced back to the context coding agents have about a codebase or engineer. There’s no reason nowadays for an engineer or a team of engineers to make the same code quality mistake twice. We manually maintain our agents.md with codebase conventions, etc, but it’d be great not to have to do that.
    • t0mas88 4 hours ago
      I've been working on that as a small open source tool: https://github.com/smithy-ai/smithy-ai

      It keeps a repository with markdown files as the agent context, makes those available (via a simple search and summarise MCP) and when closing a merge request it checks whether the context needs updating based on the review comments. If it needs updating a PR is opened on the context repository with suggested changes/additions.

    • dean_stratakos 14 hours ago
      100%. A big part of code review in my mind is to automate away specific mistakes and anti-patterns across a team. I think there are a lot of interesting things to be done to merge the code writing and code reviewing cycles.
  • sebakubisz 1 hour ago
    Can reviewers adjust the chapter splits manually if they disagree with how it grouped the PR, or are the chapters fixed once generated?
  • sscarduzio 14 hours ago
    We have the same problem, and I came up with this:

    https://sscarduzio.github.io/pr-war-stories/

    Basically it’s distilling knowledge from pr reviews back into Bugbot fine tuning and CLAUDE.md

    So the automatic review catches more, and code assistant produces more aligned code.

    • cpan22 14 hours ago
      This is really cool and we definitely have this problem as well. I really like the flowchart deciding on where to put each learning. Will have to try it out!

      Do you find that this list of learnings that end up BUGBOT.md or LESSONS.md ever gets too long? Or does it do a good job of deduplicating redundant learnings?

      • sscarduzio 6 hours ago
        Thanks! We have ~1000PRs/year. Seniors are way less than juniors and a lot of knowledge is transferred via pr messages.

        The deduplication and generalisation steps really help, and the extra bugbot context ends up in just about 2000 tok.

        Global LESSONS.md has less than 20 “pearls” with brief examples

  • ryanjso 18 hours ago
    I like the chapters thing, a lot of PRs I review should really be like 5 prs so its nice to have it auto split like that.

    Do you see a world where it splits them up on the git level?

    • dean_stratakos 18 hours ago
      Yeah that could be useful, especially with the increased popularity of stacked PRs

      But I see it working together with chapters, not instead of bc it's still good to see the granularity within a PR