Skip to content

Conversation

jjonescz
Copy link
Member

Follow up on #48749.

Debug functionality will need to know the target executable path to launch. This API provides that info.

@jjonescz jjonescz added the Area-run-file Items related to the "dotnet run <file>" effort label May 12, 2025
@jjonescz jjonescz marked this pull request as ready for review May 12, 2025 17:30
@jjonescz jjonescz requested review from Copilot and a team May 12, 2025 17:30
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a file-based program API for debuggers by introducing a new run-api command and supporting changes in project building and command handling. Key changes include:

  • A new test (Api_RunCommand) for the run-api command.
  • Modifications to VirtualProjectBuildingCommand and RunCommand to support custom artifact paths and launch profile adjustments.
  • Updates to RunApiCommand, CommandFactoryUsingResolver, and Command to integrate the new API functionality.

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
test/dotnet.Tests/CommandTests/Run/RunFileTests.cs Added test for verifying the run-api command output.
src/Cli/dotnet/Commands/Run/VirtualProjectBuildingCommand.cs Integrated a CustomArtifactsPath property into the artifacts path resolution.
src/Cli/dotnet/Commands/Run/RunCommand.cs Made launch profile and internal methods nullable/internal to support the new API.
src/Cli/dotnet/Commands/Run/Api/RunApiCommand.cs Introduced new RunApiInput/Output types for the file-based API functionality.
src/Cli/dotnet/CommandFactory/CommandFactoryUsingResolver.cs Updated command creation to support custom environment variables.
src/Cli/Microsoft.DotNet.Cli.Utils/Command.cs Enhanced the constructor and environment variable handling to track custom environment values.

Co-authored-by: Copilot <[email protected]>
@jjonescz jjonescz requested review from RikkiGibson and chsienki May 13, 2025 08:22
@jaredpar jaredpar added this to the 10.0.1xx milestone Jun 10, 2025
@jjonescz
Copy link
Member Author

@dotnet/run-file for reviews, thanks

@jjonescz jjonescz merged commit 5e3306a into dotnet:main Jun 12, 2025
30 checks passed
@jjonescz jjonescz deleted the sprint-api-debug branch June 12, 2025 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-run-file Items related to the "dotnet run <file>" effort
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants