Skip to content

fix(mobile): Order native-stack patch hunks correctly - #82

Draft
mwolson wants to merge 1 commit into
contrib/mobile-thread-context-idfrom
contrib/native-stack-patch-hunk-order
Draft

mwolson wants to merge 1 commit into
contrib/mobile-thread-context-idfrom
contrib/native-stack-patch-hunk-order

Conversation

@mwolson

@mwolson mwolson commented Oct 10, 2026

Copy link
Copy Markdown
Owner

What Changed

Carries pingdotgg#17839 native-stack-hunk-order. The @react-navigation/native-stack@7.17.6 patch now lists its hunks in line order: in lib/typescript/src/types.d.ts and src/types.tsx, the axisBehavior hunk (lines 864 and 920) comes before the identifier hunk (lines 1091 and 1156), and the lockfile picks up the new patch hash.

Why

pnpm 12 and aube reject the out-of-order patch ("hunks not in order or overlap"), and pnpm 11 accepts it but places axisBehavior seven lines low, after identifier. This replaces the earlier personal layer #78 native-stack-patch-hunk-order. Drop this layer at the refresh after pingdotgg#17839 merges upstream.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

@mwolson
mwolson added this pull request to stack #84 October 10, 2026 16:45
Carries pingdotgg#17839 native-stack-hunk-order: the
@react-navigation/native-stack 7.17.6 patch lists its hunks in line order,
so pnpm 12 can install it and pnpm 11 places the axisBehavior and
identifier properties in the right types. Drop this layer at the refresh
after pingdotgg#17839 native-stack-hunk-order merges upstream.

Co-authored-by: Mike Olson <1510193+mwolson@users.noreply.github.com>
@mwolson
mwolson removed this pull request from stack #84 October 15, 2026 01:56
@mwolson
mwolson force-pushed the contrib/native-stack-patch-hunk-order branch from 7cc3dc0 to 4d5f552 Compare October 15, 2026 01:56
@mwolson
mwolson added this pull request to stack #85 October 15, 2026 01:56

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant