NThe Prayer Network
  • new
  • past
  • show
  • ask
  • show
  • jobs
  • submit
Show HN: Stage – Putting humans back in control of code review (stagereview.app)
embedding-shape 1 minutes ago [-]
[delayed]
phyzix5761 2 minutes ago [-]
This is a really cool idea but where's the moat? What's stopping someone from replicating the functionality?
tasuki 17 minutes ago [-]
> Stage automatically analyzes the diff, clusters related changes, and generates chapters.

Isn't that what commits are for? I see no reason for adding this as an after-thought. If the committers (whether human or LLM) are well-behaved, this info is already available in the PR.

dean_stratakos 4 minutes ago [-]
In our experience, it's difficult to create well-mannered commits as you code and new ideas pop into your head or you iterate on different designs (even for LLMs). One concept we toyed around with was telling an LLM to re-do a branch using "perfect commits" right before putting up a PR. But even then you might discover new edge cases and have to tack them on as additional commits.

We thought git wasn't the right level of abstraction and decided to tackle things at the PR level instead. Curious to hear your experiences!

kvdveer 3 minutes ago [-]
I feest that grouping related changes in commits can be challenging, as git really presents commits as grouping in time, not topic.

It is certainly possible to do topic-grouping in commits, but it requires significant effort to het that consistent on a team level.

electrum 32 minutes ago [-]
Does Stage work for PRs that have multiple commits? These could be considered "stacked diffs", but in the same PR.
dean_stratakos 16 minutes ago [-]
Chapters are regenerated every time a new commit is pushed to a PR. Our thinking is that the chapters should serve as "auto stacked diffs" since they should follow a logical order.

Do you or your team use stacking in your workflows?

whywhywhywhy 19 minutes ago [-]
The idea of a workplace where people can’t be bothered to read what the ai is coding but someone else is expected to read and understand if it’s good or slop just doesn’t really add up.

I personally see the value of code review but I promise you the most vocal vibe coders I work with don’t at all and really it feels like something that could be just automated to even me.

The age of someone gatekeeping the codebase and pushing their personal coding style foibles on the rest of the team via reviews doesn’t feels like something that will exist anymore if your ceo is big on vibe coding.

gracealwan 17 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.
dean_stratakos 17 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.
t0mas88 7 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.

ryanjso 21 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?

tasuki 15 minutes ago [-]
> a lot of PRs I review should really be like 5 prs

Can't you push back on that? I feel like this tool is trying to fix misbehaved colleagues...

dean_stratakos 21 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

sscarduzio 17 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 17 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 9 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

21 hours ago [-]
sebakubisz 4 hours ago [-]
Can reviewers adjust the chapter splits manually if they disagree with how it grouped the PR, or are the chapters fixed once generated?
dean_stratakos 23 minutes ago [-]
We don't support that currently, but would love to see examples where you disagree with the chapters so we can figure out the best interface

You can regenerate the chapters anytime, but it might lead to similar results as the first time

Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact
Rendered at 14:35:45 GMT+0000 (Coordinated Universal Time) with Vercel.