docs: make review classifier page generic over swarms
Drop the swarm-specific reviewer agent name and the hyperhive-repo approval count from the architecture page and its index line.
This commit is contained in:
parent
c0af8d1fd3
commit
dcd005623e
2 changed files with 9 additions and 7 deletions
|
|
@ -87,7 +87,7 @@ declarations.
|
||||||
- **How do I set an agent's model or effort from the swarm, and what happens
|
- **How do I set an agent's model or effort from the swarm, and what happens
|
||||||
to its own pick?** → [`swarm/inference.md`](swarm/inference.md).
|
to its own pick?** → [`swarm/inference.md`](swarm/inference.md).
|
||||||
- **How will a PR get checked against its repo's own yes/no rules before
|
- **How will a PR get checked against its repo's own yes/no rules before
|
||||||
argus reviews it?** → [`swarm/review-classifier.md`](swarm/review-classifier.md)
|
a full review?** → [`swarm/review-classifier.md`](swarm/review-classifier.md)
|
||||||
(planned, not implemented yet).
|
(planned, not implemented yet).
|
||||||
|
|
||||||
## Scheduler, CI, observability
|
## Scheduler, CI, observability
|
||||||
|
|
|
||||||
|
|
@ -11,9 +11,10 @@ rules that the repo itself defines, and posts the result as a forge review.
|
||||||
A small model answers one rule at a time, so mundane rule checks come out
|
A small model answers one rule at a time, so mundane rule checks come out
|
||||||
the same way on every PR.
|
the same way on every PR.
|
||||||
|
|
||||||
It doesn't replace argus's review. argus reviews a PR only once the
|
It doesn't replace a full review by a human or agent reviewer. A swarm can
|
||||||
classifier has passed it, and argus runs on a bigger model. The classifier
|
request that review only once the classifier has passed the PR, and use a
|
||||||
takes the cheap, repetitive checks off argus.
|
larger model there. The classifier takes the cheap, repetitive checks off
|
||||||
|
that reviewer.
|
||||||
|
|
||||||
## Repo configuration
|
## Repo configuration
|
||||||
|
|
||||||
|
|
@ -189,7 +190,8 @@ changes, and the table is the review's body. An illustrative table:
|
||||||
|
|
||||||
The approval is a real one: it counts toward a repo's required approvals
|
The approval is a real one: it counts toward a repo's required approvals
|
||||||
like any other ([`pr-review-gate.md`](../process/pr-review-gate.md)). The
|
like any other ([`pr-review-gate.md`](../process/pr-review-gate.md)). The
|
||||||
hyperhive repo requires more approvals to account for it.
|
Repos that rely on it typically raise their required approval count to
|
||||||
|
account for it.
|
||||||
|
|
||||||
## Runtimes
|
## Runtimes
|
||||||
|
|
||||||
|
|
@ -284,8 +286,8 @@ These get settled, and filled in here, as the work reaches them:
|
||||||
separate from the agents' on the existing dashboards.
|
separate from the agents' on the existing dashboards.
|
||||||
- **Async message medium.** Which queue subjects or streams carry the
|
- **Async message medium.** Which queue subjects or streams carry the
|
||||||
messages between `swarm-controller` and the classifier.
|
messages between `swarm-controller` and the classifier.
|
||||||
- **argus handoff.** How argus learns the classifier passed a PR before it
|
- **Reviewer handoff.** How a later reviewer, human or agent, learns the
|
||||||
starts its own review.
|
classifier passed a PR before it starts its own review.
|
||||||
- **Opt-in details.** Beyond the config file: the service account's access
|
- **Opt-in details.** Beyond the config file: the service account's access
|
||||||
to a repo, so someone can request it as reviewer and it can push try-out
|
to a repo, so someone can request it as reviewer and it can push try-out
|
||||||
branches.
|
branches.
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue