Skip to content

Fix project route param crashes#285

Open
ramarlina wants to merge 1 commit into
mainfrom
fix/eso-710-project-route-crash
Open

Fix project route param crashes#285
ramarlina wants to merge 1 commit into
mainfrom
fix/eso-710-project-route-crash

Conversation

@ramarlina

Copy link
Copy Markdown
Owner

Summary

  • Split the shared project route layout into a server wrapper and client shell so params are resolved before reaching client code.
  • Applied the same server-wrapper pattern to the crash repro project pages: overview, automations, teams, and folders.
  • Added a focused regression test to keep those repro routes from unwrapping params in client components.

Test plan

  • npm test -- tests/app/project-dynamic-route-params.test.ts
  • HTTP smoke on port 41742: /projects, /api/projects, /projects/agx, /projects/agx/automations, /projects/agx/teams, /projects/agx/folders, /projects/alma-dev-team, /projects/mura
  • Extra HTTP smoke on port 41742: /projects/agx/env-vars, /projects/agx/thread, /projects/agx/objectives, /projects/agx/tracking, /projects/agx/notifications
  • npm run build currently fails while prerendering the existing /agents page with Cannot read properties of null (reading 'useState') after compile and TypeScript pass

Fixes ESO-710

Resolve project route params in server wrappers before handing plain slugs to client route shells, avoiding the Next runtime crash on the shared project surface.

Fixes ESO-710
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.

2 participants