Skip to content

Commit bc159de

Browse files
chrfalchclaude
andcommitted
fix(ios-prebuild): flow strict-local + prettier cleanup for the compose guard test
new Set() needed an explicit type argument under strict-local, and the exec-in-while-condition loop lost Flow's null refinement — matchAll avoids both. headers-rules.md reflowed by prettier. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 0929a23 commit bc159de

2 files changed

Lines changed: 5 additions & 6 deletions

File tree

packages/react-native/scripts/ios-prebuild/__docs__/headers-rules.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -361,9 +361,9 @@ the other third-party deps namespaces, with a single physical home in the
361361
ReactNativeDependenciesHeaders sidecar: relocating a second textual copy into
362362
ReactNativeHeaders collided with the real pod's own headers under
363363
`use_frameworks` (the duplicate-`@interface` / poisoned-module-graph Expo
364-
regression, 2026-07-03), so the set-equality gate asserts the declared
365-
namespace set — `DEPS_NAMESPACES` — matches the deps artifact's namespaces
366-
exactly, in both directions.
364+
regression, 2026-07-03), so the set-equality gate asserts the declared namespace
365+
set — `DEPS_NAMESPACES` — matches the deps artifact's namespaces exactly, in
366+
both directions.
367367

368368
### D. Remaining silent gaps — allowlist maintenance (by design)
369369

packages/react-native/scripts/ios-prebuild/__tests__/headers-compose-test.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,8 @@ describe('COMPOSE_TOOLING_FILES stays in sync with headers-compose.js requires',
2424
'utf8',
2525
);
2626
const requireRe = /require\('\.\/([\w-]+)'\)/g;
27-
const required = new Set();
28-
let match;
29-
while ((match = requireRe.exec(source)) != null) {
27+
const required = new Set<string>();
28+
for (const match of source.matchAll(requireRe)) {
3029
required.add(`${match[1]}.js`);
3130
}
3231
for (const name of required) {

0 commit comments

Comments
 (0)