fix(hmr): duplicated modules because of query params mismatch (fixes #2255)

[sapphi-red]

Description

This PR fixes #2255 by only appending ?import when it should be.
I passed explicitImportRequired property to do the check on server side, not to duplicate isExplicitImportRequired function to client/client.ts.

fixes #2255

Additional context

?import is used to differentiate import './foo.png' and fetch('./foo.png'). The ideal way is to use Accept header but unfortunately requests by import uses Accept: */* and cannot be used.

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

Before submitting the PR, please make sure you do the following

• Read the Contributing Guidelines.
• Read the Pull Request Guidelines and follow the Commit Convention.
• Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• Ideally, include relevant tests that fail without this PR but pass with it.

[sapphi-red]

I noticed this is not correct.
dep is always acceptedPath, not path.

[sapphi-red]

I considered about including ?import in path property, but I didn't reach a conclusion.
One thing I can say is that introducing explicitImportRequired property has less implication to existing implementations than including ?import in path property.

[patak-dev]

@sapphi-red lets mark the new field as @internal or @experimental so we can merge this fix in 3.1, and we can discuss later with more time in team meeting