From 0b79b16715e6c9f07f190285f5dee79d9e05fd8a Mon Sep 17 00:00:00 2001 From: Jon Chery Date: Thu, 6 Aug 2026 15:47:25 +0000 Subject: [PATCH] =?UTF-8?q?fix(P2):=20add=20metrics=20domain=20to=20sync?= =?UTF-8?q?=5Fto=5Fnova.sh=20=E2=80=94=20consumer=20export=20views=20(REQ-?= =?UTF-8?q?229)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The metrics/ export views (README.md, TRUST_SNAPSHOT.md, powerbi/) are consumer-facing but fell outside the original 13 domains, so the first nova release left them untracked. Adds a 14th domain 'metrics' between docs and workflows. Updates TestSyncToNovaScript domain-order assertion to 14. ---ci--- project: acdl phase: 2 milestone: v1.19 status: complete phase_role: final ---/ci--- --- scripts/sync_to_nova.sh | 4 +++- tests/test_untested_scripts.py | 4 ++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/scripts/sync_to_nova.sh b/scripts/sync_to_nova.sh index 1274c42..536cc3b 100755 --- a/scripts/sync_to_nova.sh +++ b/scripts/sync_to_nova.sh @@ -41,7 +41,8 @@ # 10 scripts scripts/** (consumer runbooks only; internal scripts excluded) # 11 tests tests/** # 12 docs docs/** -# 13 workflows .github/**, workflows-src/** +# 13 metrics metrics/README.md, metrics/TRUST_SNAPSHOT.md, metrics/powerbi/** +# 14 workflows .github/**, workflows-src/** set -euo pipefail SRC="${SRC:-$HOME/acdl}" @@ -91,6 +92,7 @@ DOMAINS=( "scripts|scripts" "tests|tests" "docs|docs" + "metrics|metrics/README.md metrics/TRUST_SNAPSHOT.md metrics/powerbi" "workflows|.github workflows-src" ) diff --git a/tests/test_untested_scripts.py b/tests/test_untested_scripts.py index f855a38..fbc5430 100644 --- a/tests/test_untested_scripts.py +++ b/tests/test_untested_scripts.py @@ -96,11 +96,11 @@ class TestSyncToNovaScript: assert result.returncode == 0 lines = [l for l in result.stdout.splitlines() if l and not l.startswith("DOMAIN")] names = [l.split()[0] for l in lines] - # The 13 consumer domains, in commit order. + # The 14 consumer domains, in commit order. assert names == [ "config", "core", "adapters", "modules", "contracts", "schemas", "pipelines", "mcp", "skills", "scripts", - "tests", "docs", "workflows", + "tests", "docs", "metrics", "workflows", ] def test_internal_scripts_are_excluded(self):