From 2d697a40129cdb2bc4a368992e11b3a81bb4143d Mon Sep 17 00:00:00 2001 From: Jinwoo Hong <73622457+Jinwoo-H@users.noreply.github.com> Date: Mon, 21 Sep 2026 00:56:57 -0400 Subject: [PATCH] test(config): count the haptics notify module in the session route's page closure (#21908) #21864 (haptics on the page) and #21871 (mermaid on the page) were each green against a main that lacked the other. Together, `haptics.web.ts` reaches `bridge-haptics-notify.ts` inside the session route's closure, so the module pin recorded by #21871 reads 4324 on main, not 4323. Pin the measured count and name the module. Claude-Session: https://claude.ai/code/session_01JNnE9qzUZMMnqpZWCqM3nb --- .../mobile-web-app-session-terminal-closure.test.mjs | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/config/scripts/mobile-web-app-session-terminal-closure.test.mjs b/config/scripts/mobile-web-app-session-terminal-closure.test.mjs index 08ab772d7e2..26eaeac1ebf 100644 --- a/config/scripts/mobile-web-app-session-terminal-closure.test.mjs +++ b/config/scripts/mobile-web-app-session-terminal-closure.test.mjs @@ -106,8 +106,13 @@ const EXPECTED_OFFENDERS = 0 const MERMAID_PAGE_ENGINE = 'src/components/pr-sidebar/mermaid-page-engine.generated.ts' const MERMAID_PACKAGE = 'node_modules/mermaid/' -/** The module list with mermaid on the page, recorded at the base in the docstring above. */ -const MODULES_WITH_MERMAID = 4323 +/** + * The module list with mermaid on the page, recorded at the base in the docstring above, plus + * one: `src/mobile-web-shell/bridge/bridge-haptics-notify.ts`, which `haptics.web.ts` reaches + * since C7.10 E landed beside this pin (#21864 and #21871 were each green against a main without + * the other). + */ +const MODULES_WITH_MERMAID = 4324 const artifactModules = (inputs) => inputs.filter((input) => input.includes(MERMAID_PAGE_ENGINE)) const packageModules = (inputs) => inputs.filter((input) => input.includes(MERMAID_PACKAGE))