vinzenz/hyperhive
vinzenz/hyperhive
Watch
0
0
Fork
You've already forked hyperhive
 0
Code Activity

review classifier: the certainty column's header is just certainty, so the column stays narrow

Browse source
This commit is contained in:
atlas 2026-10-15 01:02:48 +02:00 • committed by mara
commit e7ed28b382
2 changed files with 8 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

12
docs/swarm/review-classifier.md
View file

@ -339,12 +339,12 @@ all.
The service account posts the verdict as a forge review, approve or request
changes, and the table is the review's body. An illustrative table:
| result | check | certainty / threshold | reason |
| ----------- | ------------ | --------------------- | -------------------------------------------- |
| ❌ no match | datetime | 92 / 80 | Two new timestamps use naive datetimes. |
| ✅ no match | no-ads | 99 / 90 | No commit message carries an AI attribution. |
| ❔ unsure | bao-first | 95 / 70 | The PR adds no secret. |
| ❔ match | docs-updated | 41 / 60 | Below threshold. |
| result | check | certainty | reason |
| ----------- | ------------ | --------- | -------------------------------------------- |
| ❌ no match | datetime | 92 / 80 | Two new timestamps use naive datetimes. |
| ✅ no match | no-ads | 99 / 90 | No commit message carries an AI attribution. |
| ❔ unsure | bao-first | 95 / 70 | The PR adds no secret. |
| ❔ match | docs-updated | 41 / 60 | Below threshold. |
**Overall: ❌ request changes**

4
swarm-controller/src/review_classifier/review.rs
View file

@ -711,7 +711,7 @@ fn no_verdict(reason: &str, rows: &[Row]) -> String {
/// The result table, one row per classification.
fn table(rows: &[Row]) -> String {
let mut out = String::from(
"| result | check | certainty / threshold | reason |\n\
"| result | check | certainty | reason |\n\
| --- | --- | --- | --- |\n",
);
for r in rows {
@ -920,7 +920,7 @@ mod tests {
assert_eq!(
body,
format!(
"| result | check | certainty / threshold | reason |\n\
"| result | check | certainty | reason |\n\
| --- | --- | --- | --- |\n{expected}\n"
),
"{case}"