From 9752ba7c04d796822c64c670415d1046607c3c03 Mon Sep 17 00:00:00 2001 From: Irving Popovetsky Date: Tue, 27 Oct 2026 14:05:40 -0700 Subject: [PATCH] SIMULATION ONLY - never merge to main Set the production rule and service account to R2 (fdrl_01AfACzupW3U4Bx361ZUbsyR) and the test service account (svac_017eR3ohB6rP5MEAsda6U7X6), for the V26 c release simulation (S2). Adjust the three unit tests that assert the production IDs are R1 or differ from test IDs (risk K8). Co-Authored-By: Claude Opus 5.5 --- review/src/oc_review/analyze.py | 4 ++-- review/tests/unit/test_evals.py | 10 +++++----- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/review/src/oc_review/analyze.py b/review/src/oc_review/analyze.py index 5c92b6c..c8adbd3 100644 --- a/review/src/oc_review/analyze.py +++ b/review/src/oc_review/analyze.py @@ -76,9 +76,9 @@ # Production federation rule (spec "Auth and cost", "Each release"). # Rule R1 `claude-review-prod-r1` and service account `claude-review`, in # workspace `claude-review`. Each release names its own rule here. -ANTHROPIC_FEDERATION_RULE_ID = "fdrl_01N2ZktTiVKqUvvXwywBFfys" +ANTHROPIC_FEDERATION_RULE_ID = "fdrl_01AfACzupW3U4Bx361ZUbsyR" ANTHROPIC_ORGANIZATION_ID = "2bc920f9-977b-43b2-870a-f840e4ce809c" -ANTHROPIC_SERVICE_ACCOUNT_ID = "svac_01PiMgfb6qJgNpLZUV8hprCC" +ANTHROPIC_SERVICE_ACCOUNT_ID = "svac_017eR3ohB6rP5MEAsda6U7X6" FEDERATION_RULE_ID_RE = re.compile(r"^fdrl_[A-Za-z0-9]{24}$") ORGANIZATION_ID_RE = re.compile(r"^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$") diff --git a/review/tests/unit/test_evals.py b/review/tests/unit/test_evals.py index cc5388c..e8906c6 100644 --- a/review/tests/unit/test_evals.py +++ b/review/tests/unit/test_evals.py @@ -181,23 +181,23 @@ def test_eval_ids_are_the_real_eval_rule(): def test_production_ids_are_the_real_production_rule(): assert analyze.federation_ids() == analyze.FederationIds( - "fdrl_01N2ZktTiVKqUvvXwywBFfys", "2bc920f9-977b-43b2-870a-f840e4ce809c", "svac_01PiMgfb6qJgNpLZUV8hprCC" - ) + SIMULATION_RULE_ID, "2bc920f9-977b-43b2-870a-f840e4ce809c", TEST_SERVICE_ACCOUNT_ID + ) # SIMULATION ONLY: S2 names R2 and the test service account def test_eval_ids_differ_from_production_ids(): ids, prod = evals.eval_federation(), analyze.federation_ids() assert analyze.ids_valid(prod) and not analyze.ids_are_placeholders(prod) assert ids.rule_id != prod.rule_id - assert ids.service_account_id != prod.service_account_id + # SIMULATION ONLY: S2 uses the test service account, so only the rule differs # one organization, so the workspace and service account separate the rules assert ids.organization_id == prod.organization_id def test_production_ids_differ_from_every_test_id(): prod = analyze.federation_ids() - assert prod.rule_id not in {evals.eval_federation().rule_id, TEST_RULE_ID, SIMULATION_RULE_ID} - assert prod.service_account_id not in {evals.eval_federation().service_account_id, TEST_SERVICE_ACCOUNT_ID} + # SIMULATION ONLY: S2 production IDs are the simulation rule and the test service account + assert prod.rule_id not in {evals.eval_federation().rule_id, TEST_RULE_ID} def test_main_uses_the_eval_ids_by_default(tmp_path):