Develop by ptr727 · Pull Request #279 · ptr727/Utilities

ptr727 · 2026-01-04T03:21:14Z

No description provided.

Bumps csharpier from 1.1.1 to 1.1.2 Bumps xunit.analyzers from 1.23.0 to 1.24.0 Bumps xunit.runner.visualstudio from 3.1.3 to 3.1.4 --- updated-dependencies: - dependency-name: csharpier dependency-version: 1.1.2 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: nuget-deps - dependency-name: xunit.analyzers dependency-version: 1.24.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-deps - dependency-name: xunit.runner.visualstudio dependency-version: 3.1.4 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: nuget-deps ... Signed-off-by: dependabot[bot] <support@github.com>

…op/nuget-deps-0259308462 Bump the nuget-deps group with 3 updates

Bumps xunit.runner.visualstudio from 3.1.4 to 3.1.5 --- updated-dependencies: - dependency-name: xunit.runner.visualstudio dependency-version: 3.1.5 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: nuget-deps ... Signed-off-by: dependabot[bot] <support@github.com>

…evelop/nuget-deps-6aadeae1b1 Bump the nuget-deps group with 1 update

--- updated-dependencies: - dependency-name: csharpier dependency-version: 1.2.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-deps ... Signed-off-by: dependabot[bot] <support@github.com>

…op/nuget-deps-39349b7924 Bump csharpier from 1.1.2 to 1.2.1

--- updated-dependencies: - dependency-name: Serilog.Sinks.Console dependency-version: 6.1.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-deps ... Signed-off-by: dependabot[bot] <support@github.com>

…nuget-deps-602f419b37 Bump Serilog.Sinks.Console from 6.0.0 to 6.1.1

Bumps csharpier from 1.2.1 to 1.2.3 --- updated-dependencies: - dependency-name: csharpier dependency-version: 1.2.3 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: nuget-deps ... Signed-off-by: dependabot[bot] <support@github.com>

…op/nuget-deps-1689e99d0a Bump the nuget-deps group with 1 update

Bumps the actions-deps group with 2 updates in the / directory: [actions/setup-dotnet](https://github.com/actions/setup-dotnet) and [actions/checkout](https://github.com/actions/checkout). Updates `actions/setup-dotnet` from 4 to 5 - [Release notes](https://github.com/actions/setup-dotnet/releases) - [Commits](actions/setup-dotnet@v4...v5) Updates `actions/checkout` from 5 to 6 - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v5...v6) --- updated-dependencies: - dependency-name: actions/setup-dotnet dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions-deps - dependency-name: actions/checkout dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions-deps ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps csharpier from 1.2.3 to 1.2.4 --- updated-dependencies: - dependency-name: csharpier dependency-version: 1.2.4 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: nuget-deps ... Signed-off-by: dependabot[bot] <support@github.com>

…op/nuget-deps-133b2bc611 Bump the nuget-deps group with 1 update

Bumps csharpier from 1.2.4 to 1.2.5 --- updated-dependencies: - dependency-name: csharpier dependency-version: 1.2.5 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: nuget-deps ... Signed-off-by: dependabot[bot] <support@github.com>

…op/nuget-deps-262ca43251 Bump the nuget-deps group with 1 update

Copilot

Pull request overview

This PR upgrades the utilities library to .NET 10 with AOT support and removes the ProcessEx class. The changes include extensive async method additions, comprehensive XML documentation, and updated test coverage. However, the PR contains critical compilation errors in Extensions.cs that must be fixed before merging.

Key Changes:

Upgraded target framework from .NET 9 to .NET 10 with AOT support
Removed ProcessEx class (users should migrate to CliWrap)
Added async versions of file operations, downloads, and compression methods with proper cancellation token support
Added comprehensive XML documentation to all public APIs
Replaced old solution file format with new .slnx format

Reviewed changes

Copilot reviewed 28 out of 29 changed files in this pull request and generated 26 comments.

Show a summary per file

File	Description
version.json	Version bumped from 3.3 to 3.4
UtilitiesTests/UtilitiesTests.csproj	Updated to .NET 10, added analysis settings, upgraded package versions
UtilitiesTests/*.cs	Added comprehensive test coverage for new functionality (StringHistory, async compression, async file operations, downloads, console utilities)
Utilities/Utilities.csproj	Upgraded to .NET 10 with AOT support, enabled nullable reference types, added documentation generation, Husky integration
Utilities/StringHistory.cs	Added XML documentation and null validation
Utilities/StringCompression.cs	Added async methods, compression level support, and comprehensive documentation
Utilities/ProcessEx.cs	Removed entirely (breaking change)
Utilities/LogOptions.cs	Added XML documentation
Utilities/Format.cs	Added XML documentation, improved BytesToKilo implementation with switch expression
Utilities/FileExOptions.cs	Added XML documentation
Utilities/FileEx.cs	Added async versions of all file operations, removed MethodBase parameter from logging, added comprehensive documentation
Utilities/Extensions.cs	CRITICAL: Contains invalid syntax - uses non-existent `extension` keyword
Utilities/Download.cs	Refactored to use Lazy, added async methods, increased timeout to 180s, added comprehensive documentation
Utilities/ConsoleEx.cs	Added XML documentation, improved null handling
Utilities/CommandLineEx.cs	Added XML documentation, improved argument parsing
Utilities.slnx	New solution file format replacing Utilities.sln
Utilities.sln	Removed (replaced by .slnx)
Sandbox/Sandbox.csproj	Updated to .NET 10 with AOT support
Sandbox/Program.cs	Fixed nullable reference handling
README.md	Updated version history
.husky/task-runner.json	Removed IDE0055 diagnostic exclusion
.github/workflows/BuildPublishPipeline.yml	Updated actions versions, changed to .NET 10
.github/copilot-instructions.md	New comprehensive development guidelines (contains incorrect information about extension keyword)
.config/dotnet-tools.json	Updated CSharpier to 1.2.5 and Husky to 0.8.0

Comments suppressed due to low confidence (1)

UtilitiesTests/UtilitiesTests.csproj:26

The package version "18.0.1" for Microsoft.NET.Test.Sdk may not exist. As of January 2025, the latest stable version is 17.x. Please verify this version exists on NuGet before using it.

    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="18.0.1" />

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Utilities/Download.cs

Utilities/Extensions.cs

Utilities/FileEx.cs

Copilot · 2026-01-04T03:25:38Z

UtilitiesTests/ConsoleTests.cs

+        ConsoleEx.WriteLineColor(color, "Test");
+
+        // Reset console
+        StreamWriter standardOutput = new(Console.OpenStandardOutput()) { AutoFlush = true };


Disposable 'StreamWriter' is created but not disposed.

@copilot suggest a fix

UtilitiesTests/FileExAsyncTests.cs

UtilitiesTests/DownloadAsyncTests.cs

Copilot · 2026-01-04T03:31:52Z

@ptr727 I've opened a new pull request, #280, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-04T03:32:09Z

@ptr727 I've opened a new pull request, #281, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-04T03:33:59Z

@ptr727 I've opened a new pull request, #282, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-04T03:34:00Z

@ptr727 I've opened a new pull request, #283, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-04T03:34:07Z

@ptr727 I've opened a new pull request, #284, to work on those changes. Once the pull request is ready, I'll request review from you.

* Initial plan * Add using statement and simplify CallerMemberName attributes --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Initial plan * Make HttpClient timeout configurable via TimeoutSeconds property Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Initial plan * Replace redundant ToString() with string interpolation in FileExAsyncTests --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Initial plan * Replace explicit ToString() calls with string interpolation Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 29 out of 31 changed files in this pull request and generated 4 comments.

Comments suppressed due to low confidence (1)

Utilities/Download.cs:227

The HttpClient timeout was changed from 30 seconds to 180 seconds (3 minutes). This is a significant change that could impact application behavior. Consider whether this change is intentional and if it should be configurable rather than hardcoded, especially for a library where different consumers may have different timeout requirements.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Utilities/Extensions.cs

Sandbox/Program.cs

UtilitiesTests/DownloadAsyncTests.cs

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

Copilot · 2026-01-04T03:43:59Z

@ptr727 I've opened a new pull request, #285, to work on those changes. Once the pull request is ready, I'll request review from you.

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

dependabot bot and others added 19 commits August 19, 2025 20:51

Merge pull request #255 from ptr727/dependabot/nuget/dot-config/devel…

a518228

…op/nuget-deps-0259308462 Bump the nuget-deps group with 3 updates

Merge pull request #259 from ptr727/dependabot/nuget/UtilitiesTests/d…

280c3ab

…evelop/nuget-deps-6aadeae1b1 Bump the nuget-deps group with 1 update

Bump csharpier from 1.1.2 to 1.2.1

862f25c

--- updated-dependencies: - dependency-name: csharpier dependency-version: 1.2.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-deps ... Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #264 from ptr727/dependabot/nuget/dot-config/devel…

74381e3

…op/nuget-deps-39349b7924 Bump csharpier from 1.1.2 to 1.2.1

Merge pull request #265 from ptr727/dependabot/nuget/Sandbox/develop/…

9a06f0d

…nuget-deps-602f419b37 Bump Serilog.Sinks.Console from 6.0.0 to 6.1.1

Update dependencies

7db1156

Merge branch 'main' into develop

4a98e46

Merge pull request #273 from ptr727/dependabot/nuget/dot-config/devel…

907d6cc

…op/nuget-deps-1689e99d0a Bump the nuget-deps group with 1 update

Merge pull request #275 from ptr727/dependabot/nuget/dot-config/devel…

8b17948

…op/nuget-deps-133b2bc611 Bump the nuget-deps group with 1 update

Merge pull request #278 from ptr727/dependabot/nuget/dot-config/devel…

b366e92

…op/nuget-deps-262ca43251 Bump the nuget-deps group with 1 update

.NET 10 and AOT support

52d6b71

Build pipeline also needs to be 10.0

6527023

Copilot AI review requested due to automatic review settings January 4, 2026 03:21

Copilot started reviewing on behalf of ptr727 January 4, 2026 03:21 View session

ptr727 added 2 commits January 3, 2026 19:22

gitignore

c4a686c

Merge branch 'main' into develop

78b8bce

Copilot AI reviewed Jan 4, 2026

View reviewed changes

rider

2554056

Copilot AI mentioned this pull request Jan 4, 2026

[WIP] Update changes based on feedback from pull request #279 #280

Merged

7 tasks

Copilot AI mentioned this pull request Jan 4, 2026

[WIP] Fix issues based on review feedback for pull request 279 #281

Merged

5 tasks

Copilot AI mentioned this pull request Jan 4, 2026

[WIP] Address feedback from review on pull request #279 #282

Merged

5 tasks

Copilot AI mentioned this pull request Jan 4, 2026

[WIP] WIP address feedback from review on 'Develop' pull request #283

Merged

7 tasks

Copilot AI mentioned this pull request Jan 4, 2026

[WIP] Address feedback from review on previous pull request #284

Merged

5 tasks

[WIP] Update changes based on feedback from pull request #279 (#280)

36266b7

* Initial plan * Add using statement and simplify CallerMemberName attributes --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 4, 2026 03:37

Copilot started reviewing on behalf of ptr727 January 4, 2026 03:38 View session

Copilot AI and others added 4 commits January 3, 2026 19:38

Update UtilitiesTests/DownloadAsyncTests.cs

991ce1c

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

[WIP] WIP address feedback from review on 'Develop' pull request (#283)

b1be6c5

* Initial plan * Replace redundant ToString() with string interpolation in FileExAsyncTests --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

Copilot AI reviewed Jan 4, 2026

View reviewed changes

Utilities/Extensions.cs Show resolved Hide resolved

Utilities/Extensions.cs Show resolved Hide resolved

Sandbox/Program.cs Show resolved Hide resolved

UtilitiesTests/DownloadAsyncTests.cs Show resolved Hide resolved

Initial plan (#284)

1cadc44

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

Copilot AI mentioned this pull request Jan 4, 2026

[WIP] Fix issues raised in develop pull request #285

Merged

5 tasks

Initial plan (#285)

42eae42

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

ptr727 merged commit df86e7c into main Jan 4, 2026
5 checks passed

Conversation

ptr727 commented Jan 4, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Key Changes:

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Jan 4, 2026

Choose a reason for hiding this comment

Uh oh!

ptr727 Jan 4, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI commented Jan 4, 2026

Uh oh!

Copilot AI commented Jan 4, 2026

Uh oh!

Copilot AI commented Jan 4, 2026

Uh oh!

Copilot AI commented Jan 4, 2026

Uh oh!

Copilot AI commented Jan 4, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI commented Jan 4, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments