starred/orca
1
0
Fork 0
mirror of https://github.com/stablyai/orca.git synced 2026-09-22 00:02:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(config): count the haptics notify module in the session route's page closure (#21908)

Browse Source 
#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
This commit is contained in:
Jinwoo Hong
2026-09-21 00:56:57 -04:00
committed by GitHub
parent 27a0889dcf
commit 2d697a4012
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
config/scripts/mobile-web-app-session-terminal-closure.test.mjs
+7 -2
View File
@@ -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))