Consider enhancements to MVC for a grow up story from minimal actions

[pranavkm]

• IResult support in MVC #40639
• Analyzer: Warn that IResult should not be returned from Controller Actions #39692
• Add support for implicit inference of FromServices for types that appear in DI #39667
• Inferring FromBody.AllowEmptyBehavior = Allow based nullability information #39754
• Inferring FromServices optionality based on nullability information #39757
• Consider adding support for TryParse as a way to bind primitives #39682
• Consider adding a EnableMinimalAPICompatibilityMode to ApiBehaviorOptions so that controller actions behave like a minimal action.

We'd likely have an option specifically for FromServices behavior rather than a global EnableMinimalAPICompatibilityMode

[pranavkm]

FYI @davidfowl / @halter73

[ghost]

Thanks for contacting us.

We're moving this issue to the Next sprint planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

[davidfowl]

@pranavkm I moved the implicit services for MVC below the cutline. I think we should do the analyzer here that warns if you return an IResult from an MVC controller.

[mkArtakMSFT]

@rafikiassumani-msft do you want to handle this in 6.0 or backlog? Up to you :)

[brunolins16]

@pranavkm for this item "Inferring optionality from nullability information" there is recent PR that updated the DataAnnotationsProvider to use the NullabilityContext #39219.

Is that enough? If not, can you just give me a briefly explanation about this item?

[pranavkm]

@brunolins16 it's about supporting https://docs.microsoft.com/en-us/aspnet/core/fundamentals/minimal-apis?view=aspnetcore-6.0#optional-parameters. Essentially make `

#nullable enable
public IActionResult Post(string? value);

the same as

#nullable disable
public IActionResult Post(string value = null);

[brunolins16]

@pranavkm I thought it was already supported. Let me test it. Thanks.