From 2117cc501692c6351705a617b0a14357a2866b40 Mon Sep 17 00:00:00 2001 From: le king fu Date: Fri, 25 Sep 2026 19:59:17 -0400 Subject: [PATCH] test: clarify deep link round trip scope (expo-router fork not covered) Refs #117 Co-Authored-By: Claude Opus 5.5 --- tests/smoke.test.cjs | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tests/smoke.test.cjs b/tests/smoke.test.cjs index f1ac57b..483e328 100644 --- a/tests/smoke.test.cjs +++ b/tests/smoke.test.cjs @@ -242,6 +242,10 @@ check('every installed metro copy honors the 0.83.x override pin', () => { // @react-navigation/core <= 7.22.0 pulls query-string -> decode-uri-component // <= 0.4.2 (GHSA-vcc3-ghjq-m6fr). 7.22.1 parses queries itself, so the deep // link round trip must still carry query params through. +// Scope: this covers @react-navigation/core's own linking helpers only. The +// app's deep links (simplliste://task/) are resolved by expo-router's fork +// (expo-router/build/fork/getStateFromPath*.js), which still uses +// query-string@7 (see #120) and is not exercised here. check('@react-navigation/core is >= 7.22.1 and drops query-string', () => { const core = require('@react-navigation/core/package.json'); const [major, minor, patch] = core.version.split('.').map(Number);