@@ -318,7 +319,7 @@ function CertificationsBranch({
) : null}
{instances.length === 0 && findings.length === 0 ? (
- {props.protocolSection?.message || "Nessun protocollo riconosciuto."}
+ {protocolEmptyMessage(props.protocolSection?.message)}
) : null}
@@ -329,9 +330,9 @@ function CertificationsBranch({
export function FindingsTreeEmpty() {
return (
-
Nessun finding visibile
+
No visible findings
- I filtri nascondono l’elenco. Togli un filtro, oppure apri i findings già rivisti sotto.
+ Filters are hiding the list. Clear a filter, or open the reviewed findings below.
FAIL in testa — non sotto Warning.
+ FAIL is listed first — not under Warning.
) : null}
@@ -57,7 +58,7 @@ export function ProtocolInstanceBody({ instance }: { instance: ProtocolTreeInsta
{instance.chainExample ? (
- Catena: {instance.chainExample}
+ Chain: {instance.chainExample}
) : null}
{checks
.filter((c) => c.result === "FAIL" || c.skip_visible)
@@ -113,12 +114,12 @@ export function ProtocolSection({
section?: ProtocolCertificationSection | null;
}) {
const instances = protocolInstancesFromSection(section);
- const message = section?.message || "Nessun protocollo riconosciuto.";
+ const message = protocolEmptyMessage(section?.message);
const maxLevel = section?.max_level_reached || "—";
return (
- Livello massimo {maxLevel} (L0–L3). HDMI 1.4 TMDS resta UNKNOWN — niente Zdiff inventato.
+ Max level {maxLevel} (L0–L3). HDMI 1.4 TMDS stays UNKNOWN — no invented Zdiff.
{instances.length === 0 ? (
{message}
diff --git a/periscope/src/frontend/src/components/report/report-summary.tsx b/periscope/src/frontend/src/components/report/report-summary.tsx
index e1683e6..2c340aa 100644
--- a/periscope/src/frontend/src/components/report/report-summary.tsx
+++ b/periscope/src/frontend/src/components/report/report-summary.tsx
@@ -20,18 +20,18 @@ export function ReportSummary({
const infos = summary.INFO ?? 0;
const total = summary.total || errors + warnings + infos;
const outcome =
- errors > 0 ? "Error" : warnings > 0 ? "Attenzione" : total > 0 ? "Completato" : "Senza findings";
+ errors > 0 ? "Error" : warnings > 0 ? "Warning" : total > 0 ? "Complete" : "No findings";
const bits = [
`${errors} Error`,
`${warnings} Warning`,
`${infos} Info`,
- `${reviewedCount} rivisti`,
+ `${reviewedCount} reviewed`,
];
if (typeof totalCostUsd === "number" && totalCostUsd > 0) {
bits.push(`$${totalCostUsd.toFixed(2)}`);
}
if (typeof creditsSpent === "number" && creditsSpent > 0) {
- bits.push(`${creditsSpent.toFixed(2)} crediti`);
+ bits.push(`${creditsSpent.toFixed(2)} credits`);
}
return (
diff --git a/periscope/src/frontend/src/lib/findings-forest.test.ts b/periscope/src/frontend/src/lib/findings-forest.test.ts
index 93c3c76..e870cf0 100644
--- a/periscope/src/frontend/src/lib/findings-forest.test.ts
+++ b/periscope/src/frontend/src/lib/findings-forest.test.ts
@@ -1,6 +1,13 @@
import assert from "node:assert/strict";
import { test } from "node:test";
-import { groupByWorstStatus, groupDomains, isProtocolFinding, worstStatus } from "./findings-forest";
+import {
+ groupByWorstStatus,
+ groupDomains,
+ isProtocolFinding,
+ protocolEmptyMessage,
+ ruleDomain,
+ worstStatus,
+} from "./findings-forest";
import { isPcbExamFinding } from "./layout-finding";
import type { Finding } from "./types";
@@ -72,6 +79,16 @@ test("Error folder lists domains and rules after status, protocol stays out of t
assert.equal(groupDomains(groups[0].findings.filter((x) => !isProtocolFinding(x))).length, 1);
});
+test("protocol folder label is Protocols, including a stored Italian empty message", () => {
+ const domain = ruleDomain(
+ f({ designator: "J2", status: "ERROR", rule_id: "PE-PRT-L0-001", source: "protocol_l0" }),
+ );
+ assert.equal(domain.label, "Protocols");
+ assert.equal(protocolEmptyMessage("nessun protocollo riconosciuto"), "No protocol recognized.");
+ assert.equal(protocolEmptyMessage(""), "No protocol recognized.");
+ assert.equal(protocolEmptyMessage("USB2 recognized"), "USB2 recognized");
+});
+
test("PE-PLC stays in the PCB exam bucket with PE-BOM", () => {
assert.equal(
isPcbExamFinding({ source: "placement_check", rule_id: "PE-PLC-002" }),
diff --git a/periscope/src/frontend/src/lib/findings-forest.ts b/periscope/src/frontend/src/lib/findings-forest.ts
index 36d7037..4826a68 100644
--- a/periscope/src/frontend/src/lib/findings-forest.ts
+++ b/periscope/src/frontend/src/lib/findings-forest.ts
@@ -1,4 +1,4 @@
-/** Group report findings for the tree: status folders, then domains/rules, then Certificazioni. */
+/** Group report findings for the tree: status folders, then domains/rules, then Certifications. */
import type { Finding, FindingStatus, ProtocolCertificationSection } from "./types";
import {
@@ -45,7 +45,7 @@ const DOMAIN_LABEL: Record = {
PDN: "PDN",
PD: "USB-PD",
AF: "AF Board + AI",
- PRT: "Protocolli",
+ PRT: "Protocols",
MUX: "Pin mux",
NC: "NC pins",
DEC: "Decoupling",
@@ -112,6 +112,17 @@ export function groupByDesignator(findings: Finding[]): DesignatorGroup[] {
.map(([designator, items]) => ({ designator, findings: items }));
}
+export const PROTOCOL_EMPTY_MESSAGE = "No protocol recognized.";
+
+/** Tree copy. Stored 2.83 reports still carry the Italian empty phrase. */
+export function protocolEmptyMessage(message?: string | null): string {
+ const text = (message || "").trim();
+ if (!text || text.toLowerCase() === "nessun protocollo riconosciuto") {
+ return PROTOCOL_EMPTY_MESSAGE;
+ }
+ return text;
+}
+
export function isProtocolFinding(f: {
source?: string | null;
rule_id?: string | null;
@@ -122,7 +133,7 @@ export function isProtocolFinding(f: {
}
export function ruleDomain(f: Finding): { id: string; label: string } {
- if (isProtocolFinding(f)) return { id: "PRT", label: "Protocolli" };
+ if (isProtocolFinding(f)) return { id: "PRT", label: "Protocols" };
const rid = f.rule_id || "";
const m = rid.match(/^PE-([A-Z]+)/);
if (m) {
@@ -173,7 +184,7 @@ export function groupDomains(findings: Finding[]): DomainGroup[] {
/**
* Error / Warning / Info folders. A ref lives in the folder of its worst child.
* U18 with PE-PLC-002 ERROR + five WARNING → Error, badge ERROR, count 6.
- * Protocol findings (PE-PRT) are excluded from domain folders — they sit under Certificazioni.
+ * Protocol findings (PE-PRT) are excluded from domain folders — they sit under Certifications.
*/
export function groupByWorstStatus(findings: Finding[]): StatusGroup[] {
const buckets: Record = {
diff --git a/periscope/src/frontend/src/lib/version.ts b/periscope/src/frontend/src/lib/version.ts
index 101fa68..94820d8 100644
--- a/periscope/src/frontend/src/lib/version.ts
+++ b/periscope/src/frontend/src/lib/version.ts
@@ -1,3 +1,3 @@
/** Stamped from content/changelog.md by scripts/sync-version.mjs. */
-export const APP_VERSION = "2.83.0";
+export const APP_VERSION = "2.84.0";
export const APP_VERSION_DATE = "2026-09-22";
diff --git a/tests/pcb/test_protocol_catalog_m0.py b/tests/pcb/test_protocol_catalog_m0.py
index bb33f42..d7c01b2 100644
--- a/tests/pcb/test_protocol_catalog_m0.py
+++ b/tests/pcb/test_protocol_catalog_m0.py
@@ -223,7 +223,7 @@ def test_reject_same_logical_and_physical_id():
def test_empty_report_section_has_no_z():
section = empty_protocol_section()
dumped = section.model_dump()
- assert section.message == "nessun protocollo riconosciuto"
+ assert section.message == "No protocol recognized."
assert section.recognized_instances == []
assert dumped.get("max_level_reached") is None
blob = json.dumps(dumped)
@@ -238,7 +238,7 @@ def test_empty_report_section_has_no_z():
summary={"ERROR": 0, "WARNING": 0, "INFO": 0},
protocol_certification=dumped,
)
- assert report.protocol_certification["message"] == "nessun protocollo riconosciuto"
+ assert report.protocol_certification["message"] == "No protocol recognized."
def test_pcie_cxl_physical_packs_are_empty_numbers():
diff --git a/tests/pcb/test_protocol_recognize_m1.py b/tests/pcb/test_protocol_recognize_m1.py
index a0409c4..85fbd0d 100644
--- a/tests/pcb/test_protocol_recognize_m1.py
+++ b/tests/pcb/test_protocol_recognize_m1.py
@@ -138,9 +138,9 @@ def test_axi4_internal_does_not_create_pcb_nets():
assert "AXI_WDATA" not in one.nets
-def test_empty_graph_keeps_nessun_protocollo():
+def test_empty_graph_keeps_no_protocol_message():
sec = protocol_section_for_graph(DesignGraph())
- assert sec.message == "nessun protocollo riconosciuto"
+ assert sec.message == "No protocol recognized."
assert sec.recognized_instances == []
blob = json.dumps(sec.model_dump())
assert "ohm" not in blob.lower()