Skip to content

Conversation

@0xced
Copy link
Contributor

@0xced 0xced commented Oct 6, 2025

Fixes #701

  • I have read the Contribution Guidelines
  • I have commented on the issue above and discussed the intended changes
  • A maintainer has signed off on the changes and the issue was assigned to me
  • All newly added code is adequately covered by tests
  • All existing tests are still running without errors
  • The documentation was modified to reflect the changes OR no documentation changes are required.

Opened as draft because the documentation still needs to be updated.

Changes

A new CancellationToken parameter has been added to the Run and RunAsync methods of ICommandApp and everywhere else needed.

This is a breaking change. Both Command and AsyncCommand subclasses will need to be updated to handle the new CancellationToken.

Before:

public sealed class MyCommand : AsyncCommand
{
    public override Task<int> ExecuteAsync(CommandContext context)
    {
        return Task.FromResult(0);
    }
}

After:

public sealed class MyCommand : AsyncCommand
{
    public override async Task<int> ExecuteAsync(CommandContext context, CancellationToken cancellationToken)
    {
        await Task.Delay(0, cancellationToken);
        return 0;
    }
}

When the application terminates because the cancellation token was triggered, the default exit code is 130. This value can be configured through the new CancellationExitCode() fluent method.

var app = new CommandApp();
app.Configure(config => config.CancellationExitCode(123));

Please upvote 👍 this pull request if you are interested in it.

@0xced 0xced force-pushed the RunAsync-CancellationToken branch from 0fa40e8 to 230d3b6 Compare October 6, 2025 16:52
Also raise CA2016 (forward the CancellationToken parameter to methods that take one) to warning

Fixes spectreconsole#701
@0xced 0xced force-pushed the RunAsync-CancellationToken branch from 230d3b6 to a9563b6 Compare October 6, 2025 17:23
@0xced 0xced marked this pull request as ready for review October 11, 2025 18:33
@0xced
Copy link
Contributor Author

0xced commented Oct 11, 2025

I will update the documentation in a follow-up pull request. Let's merge this to get it into a pre-release.

Copy link
Contributor

@patriksvensson patriksvensson left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@patriksvensson patriksvensson merged commit f5f61ca into spectreconsole:main Oct 11, 2025
3 checks passed
@0xced 0xced deleted the RunAsync-CancellationToken branch October 15, 2025 16:14
0xced added a commit to 0xced/0xced.Templates that referenced this pull request Oct 15, 2025
Use top-level cancellation support introduced in spectreconsole/spectre.console#1911
0xced added a commit to 0xced/spectre.console that referenced this pull request Oct 16, 2025
Now that spectreconsole#1911 is merged, all `Execute` methods of commands have a new CancellationToken parameter. Update the documentation to reflect that.

New documentation shall still be written with an example on how to pass the top-level CancellationToken to the app.Run(Async) method.
patriksvensson pushed a commit that referenced this pull request Oct 16, 2025
Now that #1911 is merged, all `Execute` methods of commands have a new CancellationToken parameter. Update the documentation to reflect that.

New documentation shall still be written with an example on how to pass the top-level CancellationToken to the app.Run(Async) method.
0xced added a commit to 0xced/examples that referenced this pull request Oct 27, 2025
All the Cli examples have been updated with the new CancellationToken introduced in spectreconsole/spectre.console#1911
microsoft-github-policy-service bot pushed a commit to Azure/bicep that referenced this pull request Feb 1, 2026
Updated
[Spectre.Console](https://github.com/spectreconsole/spectre.console)
from 0.51.1 to 0.54.0.

<details>
<summary>Release notes</summary>

_Sourced from [Spectre.Console's
releases](https://github.com/spectreconsole/spectre.console/releases)._

## 0.54.0

Version `0.54.0` of Spectre.Console has been released!

## Spectre.Console.Cli has a new home!

We've decided to move `Spectre.Console.Cli` to its own repository, where
we will prepare it for a _1.0_ release. This means that the
_Spectre.Console.Cli_ NuGet packages will no longer be versioned
together with _Spectre.Console_. They will now have a preview version
such as `1.0.0-alpha-0.x`.

There should be no issues staying on version _0.53.0_ of
_Spectre.Console.Cli_ until we release a stable version if you prefer
not to use a pre-release dependency.

## New unit testing package for Spectre.Console.Cli

There is now a new testing package for _Spectre.Console.Cli_ called
[Spectre.Console.Cli.Testing](https://www.nuget.org/packages/Spectre.Console.Cli.Testing).
This is where you will find the `CommandAppTester` from now on.

You can find more information about unit testing in the
[documentation](https://spectreconsole.net/cli/unit-testing).

## What's Changed

* Normalizes paths when writing exceptions to the console for tests. by
[@​phil-scott-78](https://github.com/phil-scott-78) in
[#​1758](spectreconsole/spectre.console#1758)
* Fixes issue with Panel not applying overflow to children by
[@​phil-scott-78](https://github.com/phil-scott-78) in
[#​1942](spectreconsole/spectre.console#1942)
* Remove Spectre.Console.Cli from repository by
[@​patriksvensson](https://github.com/patriksvensson) in
[#​1928](spectreconsole/spectre.console#1928)

**Full Changelog**:
spectreconsole/spectre.console@0.53.0...0.54.0

## 0.53.1

This is a hotfix release for 0.53.0, which accidentally included
`OpenCli.Sources` as a dependency, instead of being included in the
package.

As a bonus, a fix for panels was also included in this release 😄 

## What's Changed
* Ensure OpenCli.Sources are not a dependency of the CLI package. by
@​patriksvensson in d61788d6c681745c8d1158ceeb4545ba37a99522
* Normalizes paths when writing exceptions to the console for tests. by
@​phil-scott-78 in
spectreconsole/spectre.console#1758
* Fixes issue with Panel not applying overflow to children by
@​phil-scott-78 in
spectreconsole/spectre.console#1942

**Full Changelog**:
spectreconsole/spectre.console@0.53.0...0.53.1

## 0.53.0

## What's Changed

* Add top-level CancellationToken support to Spectre.Console.Cli by
[@​0xced](https://github.com/0xced) in
[#​1911](spectreconsole/spectre.console#1911)
* Update the Spectre.Console.Cli documentation with CancellationToken by
[@​0xced](https://github.com/0xced) in
[#​1920](spectreconsole/spectre.console#1920)

**Full Changelog**:
spectreconsole/spectre.console@0.52.0...0.53.0

## 0.52.0

## What's Changed

* Add OpenCLI integration to Spectre.Console.Cli by
[@​patriksvensson](https://github.com/patriksvensson) in
[#​1909](spectreconsole/spectre.console#1909)
* Fix OPENCLI_VISIBILITY_INTERNAL to DefineConstants concat by
[@​devlead](https://github.com/devlead) in
[#​1912](spectreconsole/spectre.console#1912)

**Full Changelog**:
spectreconsole/spectre.console@0.51.1...0.52.0

Commits viewable in [compare
view](spectreconsole/spectre.console@0.51.1...0.54.0).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=Spectre.Console&package-manager=nuget&previous-version=0.51.1&new-version=0.54.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>
###### Microsoft Reviewers: [Open in
CodeFlow](https://microsoft.github.io/open-pr/?codeflow=https://github.com/Azure/bicep/pull/18859)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLI Command RunAsync support passing in/propagating a CancelationToken

2 participants