Enable --interactive by default for sessions that are local.

[baronfel]

This change unifies the --interactive infrastructure for all dotnet commands. With that common infra, it also enables interactivity for non-CI scenarios by default. We reuse CI detection from telemetry and from output redirection to determine whether to enable interactivity.

Fixes #35165

[Copilot]

PR Overview

This PR unifies the handling of the --interactive option across dotnet CLI commands to enable interactivity by default for local sessions while still allowing it to be explicitly disabled. Key changes include refactoring interactive option definitions to use a common implementation, updating forwarding functions to conditionally include interactive arguments, and adjusting tests to expect the interactive flag in the command arguments.

Reviewed Changes

File	Description
src/Cli/dotnet/CommonOptions.cs	Introduces helper IsCIEnvironmentOrRedirected() and redefines InteractiveOption to use the new common infra.
test/dotnet-run.Tests/GivenDotnetRunBuildsCsProj.cs	Updates tests to reflect that interactivity is enabled by default and adds tests for when it is explicitly disabled.
src/BuiltInTools/dotnet-watch/CommandLineOptions.cs	Adjusts forwarding logic to skip the interactive token when --non-interactive is specified.
src/Cli/dotnet/*	Updates various command parsers to use the common InteractiveOption implementation.
src/Cli/dotnet/OptionForwardingExtensions.cs	Refactors forwarding extension methods to work with nullable values and use a concise array literal syntax.
test/dotnet-watch.Tests/CommandLineOptionsTests.cs	Updates test expectations so that interactive defaults are correctly injected into command and build arguments.

Copilot reviewed 34 out of 34 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

src/Cli/dotnet/OptionForwardingExtensions.cs:14

• [nitpick] The use of array literal syntax like [option.Name] may reduce clarity for readers; consider using an explicit array initializer (e.g. new string[] { option.Name }) for improved readability.

public static ForwardedOption<T> Forward<T>(this ForwardedOption<T> option) => option.SetForwardingFunction((T? o) => [option.Name]);

[Copilot]

The comment in the ItDoesNotShowImportantLevelMessageByDefaultWhenInteractivityDisabled test is misleading because the test name and assertion expect that important level messages are not shown when interactivity is disabled. Please update the comment so it accurately reflects the expected behavior.