feat: improve analyze command

[rakshitgondwal]

Closes #332

📑 Description

Made the following changes-

• Added \n between yellow sections and the top of the yellow GPT response for better readability.
• Made the index start from 1 instead 0 for the results.
• Added index for the errors too.
• Parentheses would be omitted if the value of result.ParentObject would be empty.

Here's a reference image for these issues-

✅ Checks

• My pull request adheres to the code style of this project
• My code requires changes to the documentation
• I have updated the documentation as required
• All the tests have passed

[rakshitgondwal]

There were two more changes suggested-

• Hide the server could not find the requested resource errors by default (enable in verbose or details mode)
• Hide internal cache related errors

but I am not being able to get around these. Can I get some help here?

[matthisholleville]

I pushed a commit quickly, I think it can inspire you for the rework of the logs concerning the cache : main...matthisholleville:k8sgpt:feat/rework-cache-log

[AlexsJones]

Made the index start from 1 instead 0.

👀

[rakshitgondwal]

Made the index start from 1 instead 0.

eyes

For the results*

[rakshitgondwal]

Hey @matthisholleville
From what I can understand, after going through your commit, we are now handling the cache errors in the ai package itself, rather than handling it in file_based.go but how can we hide the internal system errors?
I was thinking of just returning false from the file_based.go but that won't be a good idea, or would it be?

[matthisholleville]

I think we should add a global argument to handle the verbosity of these logs

[rakshitgondwal]

I think we should add a global argument to handle the verbosity of these logs

Ohh something like if a user provides --versbose then we print the logs else we just return something like error while testing if cache key exists, use --verbose to get more details

[AlexsJones]

Is this ready for review?

[rakshitgondwal]

Is this ready for review?

Nope, still have this question that do we introduce a --verbose flag to print the logs? I think Matthis is busy so can you help me out here?