New command: search. Closes #5344

[MartinM85]

Closes #5344

[MartinM85]

A few notes:

• without option resultsOnly, the output to csv and markdown is empty, I think it's caused by the complex structure of searchResponse
• with option resultsOnly, the output to csv, markdown and text is not useful at all for users, but it seems to me to be out of the scope of #5344

The class has name SearchSearchCommand, because of some conventions.

[milanholemans]

Thank you, we'll try to have a look at it ASAP!

[Adam-it]

@MartinM85 awesome work 👏🤩 We are getting a lot closer. Let's do another round before we move forward.

One thing I noticed is the additional scopes we will need to add for this command

Were the first is not that bad but the second one Sites.ReadWrite.All seems like a huge overkill 😮. Why would a search endpoint need write permissions anyway 🤔. I need to research this a bit. @waldekmastykarz what do you think?

I just checked and it seems only Sites.Read.All is enough 🤔

[Adam-it]

Ready to merge 🚀 Along the way, I should do a small fixup

[Adam-it]

@waldekmastykarz I double-checked over my own app registration and it seems we need to add MS Graph Sites.Read.All scope to our PnP Management Shell app

[Adam-it]

Merged manually. You Rock 🤩👏