From 7f265b3cb21c0148d62e5b85d2e3398ebf6946f9 Mon Sep 17 00:00:00 2001 From: Abdessamad Derraz <3028866+Abdess@users.noreply.github.com> Date: Fri, 27 Mar 2026 20:37:48 +0100 Subject: [PATCH] refactor: split pack check into baseline and cores with clear counts --- scripts/generate_pack.py | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/scripts/generate_pack.py b/scripts/generate_pack.py index a2b64685..5c84957a 100644 --- a/scripts/generate_pack.py +++ b/scripts/generate_pack.py @@ -1262,7 +1262,8 @@ def verify_pack_against_platform( checked = baseline_checked + core_checked present = baseline_present + core_present - return len(errors) == 0, checked, present, errors + return (len(errors) == 0, checked, present, errors, + baseline_checked, baseline_present, core_checked, core_present) def verify_and_finalize_packs(output_dir: str, db: dict, @@ -1308,13 +1309,15 @@ def verify_and_finalize_packs(output_dir: str, db: dict, # Stage 2: platform conformance (extract + verify) platforms = pack_to_platform.get(name, []) for pname in platforms: - p_ok, total, matched, p_errors = verify_pack_against_platform( - zip_path, pname, platforms_dir, db=db, - ) - if p_ok: - print(f" platform {pname}: {matched}/{total} OK") - else: - print(f" platform {pname}: {matched}/{total} FAILED") + (p_ok, total, matched, p_errors, + bl_checked, bl_present, core_checked, core_present) = \ + verify_pack_against_platform( + zip_path, pname, platforms_dir, db=db, + ) + status = "OK" if p_ok else "FAILED" + print(f" platform {pname}: {bl_present}/{bl_checked} baseline, " + f"{core_present}/{core_checked} cores, {status}") + if not p_ok: for err in p_errors: print(f" {err}") all_ok = False