Define API `query` parameter regression

[oliviertassinari]

🐛 bug report

Preflight Checklist

• I have read the Contributing Guidelines for this project.
• I agree to follow the Code of Conduct that this project
  adheres to.
• I have searched the issue tracker for an issue that matches the one I want
  to file, without success.

Description of the problem

The query param described in https://codesandbox.io/docs/learn/sandboxes/cli-api#supported-parameters doesn't seem to work.

How has this issue affected you? What are you trying to accomplish?

I noticed because in https://mui.com/material-ui/react-switch/#color opening a codesandbox does no longer set the file in focus. The DX could be better

Actual:

Expected:

To Reproduce

https://codesandbox.io/p/sandbox/define-api-get-request-forked-v5zl4m?file=%2Findex.js%3A16%2C5

taken from https://codesandbox.io/docs/learn/sandboxes/cli-api#get-request. In practice, we use the POST API but I assume the bug is shared, as it seems to happen during an internal codesandbox redirection.

Your Environment

Software	Name/Version
Сodesandbox
Browser
Operating System

[CompuIves]

This is a regression! Fix is underway.

[CompuIves]

Hmm, I can't reproduce it yet. Does it work for you in an incognito?

[CompuIves]

I'm expecting that this could be a race condition, where the initial file is set too late (only after the editor is initialized), but I cannot repro it yet.

[oliviertassinari]

A recording in incognito with the provided reproduction sandbox:

Screen.Recording.2024-01-29.at.15.41.07.mov

https://codesandbox.io/p/sandbox/define-api-get-request-forked-v5zl4m?file=%2Findex.js%3A16%2C5

[oliviertassinari]

I can reproduce the regression on React's docs as well https://react.dev/learn#components.

[CompuIves]

Ohh, this is really helpful! It looks like the redirect to the v2 editor does not carry over the query params! Taking a look...

[CompuIves]

Just opened a PR with the fix. I was testing after the redirect, and there the query param was used. I didn't realise the redirect was before that. This PR should now pass on the query parameters properly 😄