`helm-framework`: ModifyPlan refactor 2

[BBBmau]

Description

- [ ] refactor plan.Lint.ValueBool() to be it's own PlanModifier https://github.com/hashicorp/terraform-provider-helm/pull/1514/commits/974fdeb0150ca7c7a205003f39958efd66019272 - [for later] Perhaps we can decompose the ModifyPlan function into more discrete planModifiers on a per resource basis, i.e for version, manifest <- This is up for discussion. Since the initial implementation just seems like a lot more code than what it was previously.

• [x] add diagnostic when we hit the “cluster unreachable” logic 833280a14d6e058b200e58bc9a19a61a85f0ce11
• [x] set plan.Version to the state value, rather than setting plan.Version to null (this causes a computed action the attribute) c388c75e4ccf4ccbf3013a5bc81685a85409e47a

Acceptance tests

• [ ] Have you added an acceptance test for the functionality being added?

Release Note

Release note for CHANGELOG:

...

References

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

[JaylonmcShan03]

• The error handling is more descriptive, and uses diagnostics which is better!

• I agree, because it aligns with maintaining the correct state, preventing unnecessary recomputation (I think thats the correct wording haha) by using the previous state version if available