fix: remove product list from search wrapper

[vmourac-vtex]

What problem is this solving?

This commit has the intention to fix the duplication an array of structured data Product List, that usually occurs on PLP pages.
The ProductList is a component exported by vtex.structured-data. It aims provide necessary information to google to render Rich Results in Google.
We found out that there are two ocurences of ProductList in the same PLP.
The SearchWrapper element from vtex-apps/store is responsible for the first ProductList and vtex.product-summary is responsible for the other ProductList

This PR simply removes the ProductList from the vtex.store project, keeping the vtex.product-summary as is.

How to test it?

Workspace with error

Steps to reproduce:

• Open the link above
• Open DevTools
• Go to Elements tab
• Search (cmd+f) for the following string: {"@context":"https://schema.org","@type":"ItemList","itemListElement"

Expected Behavior:

• You should find two occurrences of this string
• The content of the Structured JSON that starts with the searched string must be the same (duplicates)

Workspace with fix

Steps to reproduce:

• Open the link above
• Open DevTools
• Go to Elements tab
• Search (cmd+f) for the following string: {"@context":"https://schema.org","@type":"ItemList","itemListElement"

Expected Behavior:

• You should find only one occurrence of this string.
  • The first occurrence in the HTML, from the SearchWrapper component, has been removed

Screenshots or example usage:

Before Fix:

First Occurrence:

Second Occurrence:

After Fix:

Describe alternatives you've considered, if any.

We tried approaching the problem by adding a conditional in vtex.product-summary that uses useRuntime().page to check if the user is in a search page (evaluates to search.page#<context>).
It was considered a safer approach that would cover possible scenarios where a search page can exist without a PLP.

Unfortunately, It did not work because when useRuntime().page is evaluated, it returnsstore.home (in sportline). Also, when checking for __RUNTIME__.page in the browser console, the page correctly evaluates to a search.page#<context>. I did not understand why this happened, so I am submitting this PR instead.

Related to / Depends on

N/A

How does this PR make you feel?

[vtex-io-ci-cd]

Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖

Please select which version do you want to release:

• Patch (backwards-compatible bug fixes)

• Minor (backwards-compatible functionality)

• Major (incompatible API changes)

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

• No thanks, I would rather do it manually 😞

[vtex-io-docs-bot]

Beep boop 🤖

I noticed you didn't make any changes at the docs/ folder

• There's nothing new to document 🤔
• I'll do it later 😞

In order to keep track, I'll create an issue if you decide now is not a good time

• I just updated 🎉🎉

[vmourac-vtex]

Closing PR.
Issue fixed on search-result