Skip to content

[codex] Preserve route topology declaration order#47

Merged
ScriptedAlchemy merged 2 commits into
perf/bundling-performancefrom
codex/order-sensitive-route-topology
Jun 22, 2026
Merged

[codex] Preserve route topology declaration order#47
ScriptedAlchemy merged 2 commits into
perf/bundling-performancefrom
codex/order-sensitive-route-topology

Conversation

@ScriptedAlchemy

Copy link
Copy Markdown
Collaborator

Summary

  • make route topology snapshots preserve route-manifest insertion order
  • encode route ordinal positions so sibling reorder triggers topology changes
  • update route-watch tests for React Router sibling declaration-order semantics

Context

This follows up on the Slack design note in /home/zack/Downloads/react_router_request.md. Most of that note describes a future in-place route topology design or items already covered by PR #46; this PR addresses the remaining small current-baseline correction that route order is semantic.

Verification

  • pnpm test:core -- tests/route-watch.test.ts
  • pnpm build
  • pnpm format:check
  • git diff --check

@ScriptedAlchemy ScriptedAlchemy marked this pull request as ready for review June 22, 2026 22:46
@ScriptedAlchemy ScriptedAlchemy force-pushed the codex/order-sensitive-route-topology branch from b95b6f9 to a1444de Compare June 22, 2026 23:05
@ScriptedAlchemy ScriptedAlchemy force-pushed the codex/order-sensitive-route-topology branch from a1444de to 0f5a76e Compare June 22, 2026 23:17
Base automatically changed from codex/harden-route-transforms-dev-lifecycle to perf/bundling-performance June 22, 2026 23:19
@ScriptedAlchemy ScriptedAlchemy merged commit 0f5a76e into perf/bundling-performance Jun 22, 2026
@ScriptedAlchemy ScriptedAlchemy deleted the codex/order-sensitive-route-topology branch June 22, 2026 23:19
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