Add polyfills for `TimeProvider` and `ITimer` by Copilot · Pull Request #79 · Tyrrrz/PolyShim

Copilot · 2026-02-17T20:34:05Z

Adds polyfills for TimeProvider and ITimer introduced in .NET 8, enabling time abstraction APIs on older frameworks.

Core Polyfills

TimeProvider (Net80/TimeProvider.cs) - Abstract class with System singleton property and methods for time operations
- GetUtcNow() - Returns current UTC time as DateTimeOffset
- GetLocalNow() - Returns current local time
- GetTimestamp() - Returns high-resolution timestamp for performance measurements
- GetElapsedTime() - Two overloads for calculating elapsed time from timestamps
- CreateTimer() - Creates timer instances (virtual method for testability)
- LocalTimeZone property - Returns the local time zone
ITimer (Net80/ITimer.cs) - Interface for timer abstraction with IAsyncDisposable support

Supporting Polyfills Created

To enable TimeProvider on the broadest range of frameworks, the following polyfills were added:

Stopwatch.GetElapsedTime (Net70/Stopwatch.cs) - Extension methods for calculating elapsed time from timestamps
TimerCallback (NetCore10/TimerCallback.cs) - Delegate for timer callbacks on .NET Standard < 1.2
Task.Delay (NetCore10/Task.cs) - Asynchronous delay implementation for .NET Framework < 4.5
Task.FromCanceled (NetCore10/Task.cs) - Creates canceled tasks (generic and non-generic overloads)
Timer.DisposeAsync (NetCore30/Timer.cs) - Extension method for async timer disposal on older frameworks

Implementation Details

Broad Framework Support: TimeProvider works on all target frameworks
CreateTimer: Available on all frameworks except .NET Standard 1.0/1.1 (where Timer APIs don't exist)
Virtual Methods: CreateTimer is virtual to enable derived classes to provide custom timer implementations
No Guards: Removed validation checks in polyfills to match the standard polyfill pattern
Clean Code: Uses polyfilled APIs instead of conditional compilation within methods

Feature Detection

Recognizes Microsoft.Bcl.TimeProvider compatibility package
Disables polyfills when native (.NET 8+) or package implementation available
Added to compatibility package list in README

The polyfills are automatically disabled on .NET 8+ or when Microsoft.Bcl.TimeProvider is referenced.

Original prompt

Let's add polyfills for Time provider and related types. Added in .NET 8. Is also provided via a compatibility package whose name I don't know.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

…kage Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Tyrrrz

@copilot

Tyrrrz · 2026-02-17T20:56:59Z

PolyShim/Net80/ITimer.cs

@@ -0,0 +1,26 @@
+#if !FEATURE_TIMEPROVIDER && ((NETCOREAPP && !NET8_0_OR_GREATER) || (NETFRAMEWORK) || (NETSTANDARD))


The conditional symbol already includes TFMs as well

Fixed in a4b3590. Simplified to just #if !FEATURE_TIMEPROVIDER since the feature flag already includes the TFM checks.

Tyrrrz · 2026-02-17T20:57:39Z

PolyShim/Net80/ITimer.cs

@@ -0,0 +1,26 @@
+#if !FEATURE_TIMEPROVIDER && ((NETCOREAPP && !NET8_0_OR_GREATER) || (NETFRAMEWORK) || (NETSTANDARD))
+// Only include ITimer if we have Task support and are on .NET Standard 2.0+, .NET Core 2.0+, or .NET Framework 4.6.1+
+#if FEATURE_TASK && (NETSTANDARD2_0_OR_GREATER || NETCOREAPP2_0_OR_GREATER || NET461_OR_GREATER)


Nothing below appears to require Tasks or those TFMs?

Fixed in a4b3590. Removed the unnecessary FEATURE_TASK and TFM conditionals from ITimer - it doesn't actually require Task types.

Tyrrrz · 2026-02-17T20:58:01Z

PolyShim/Net80/ITimer.cs

+namespace System.Threading;
+
+// https://learn.microsoft.com/dotnet/api/system.threading.itimer
+internal interface ITimer : IDisposable


Exclude from coverage

Removed ExcludeFromCodeCoverage in a4b3590. The attribute isn't supported on interfaces in older C# versions.

Tyrrrz · 2026-02-17T20:59:02Z

PolyShim/Net80/TimeProvider.cs

+[ExcludeFromCodeCoverage]
+internal abstract class TimeProvider
+{
+    private static readonly TimeProvider s_system = new SystemTimeProvider();


Inlined in a4b3590. Now using SystemThreading.Timeout.InfiniteTimeSpan directly instead of storing in a constant.

Tyrrrz · 2026-02-17T20:59:51Z

PolyShim/Net80/TimeProvider.cs

+        TimeSpan period
+    )
+    {
+        if (callback is null)


Don't bother with guards in polyfills

Removed null guard in a4b3590. Changed CreateTimer to single-expression method body.

Tyrrrz · 2026-02-17T21:00:22Z

PolyShim/Net80/TimeProvider.cs

+#if NETSTANDARD2_0 || NET461 || NET462
+            return Task.FromResult(false);
+#else
+            return Task.CompletedTask;


We have this polyfilled for all TFMs

Fixed in a4b3590. Now using the polyfilled Task.CompletedTask directly without conditional compilation.

Tyrrrz · 2026-02-17T21:00:45Z

PolyShim/Net80/TimeProvider.cs

+        if (cancellationToken.IsCancellationRequested)
+        {
+#if NETCOREAPP || NETSTANDARD2_1_OR_GREATER
+            return Task.FromCanceled(cancellationToken);


Pretty sure we have this polyfilled too

Task.FromCanceled is not actually polyfilled in the codebase. Used TaskCompletionSource.SetCanceled() as an alternative in a4b3590.

Tyrrrz · 2026-02-17T21:02:28Z

PolyShim/Net80/TimeProvider.cs

+#if NETCOREAPP3_0_OR_GREATER || NETSTANDARD2_1_OR_GREATER
+            return _timer.DisposeAsync();
+#else
+            _timer.Dispose();
+            return default;


Just try cast to IAsyncDisposable

The try-cast approach doesn't work because Timer doesn't implement IAsyncDisposable on net461 and similar frameworks - the compiler rejects the cast at compile time. Kept the conditional compilation in a4b3590 to match the actual Timer API availability.

Tyrrrz · 2026-02-17T21:04:19Z

PolyShim/Net80/TimeProvider.cs

+// Only include TimeProvider if we have Task support and are on .NET Standard 2.0+, .NET Core 2.0+, or .NET Framework 4.6.1+
+// This matches the support level of Microsoft.Bcl.TimeProvider compatibility package
+#if FEATURE_TASK && (NETSTANDARD2_0_OR_GREATER || NETCOREAPP2_0_OR_GREATER || NET461_OR_GREATER)
+#nullable enable


Pretty sure we can support all TFMs with it. APIs that need Task can be singled out.

Fixed in a4b3590. Removed TFM-specific guards and made all TFMs supported. The Task-dependent methods (Delay, CreateCancellationTokenSource) are now conditional on FEATURE_TASK with appropriate fallback error messages for older frameworks that lack the required APIs.

…se polyfilled APIs Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Tyrrrz

@copilot

Tyrrrz · 2026-02-17T21:17:17Z

PolyShim/Net80/TimeProvider.cs

+using System.Diagnostics;
+using System.Diagnostics.CodeAnalysis;
+using SystemThreading = System.Threading;
+#if FEATURE_TASK


I think this can be referenced unconditionally because we have a dummy in that namespace

Fixed in 65710d0. Removed the conditional since there's a namespace dummy for System.Threading.Tasks.

Tyrrrz · 2026-02-17T21:17:38Z

PolyShim/Net80/TimeProvider.cs

+[ExcludeFromCodeCoverage]
+internal abstract class TimeProvider
+{
+    private static readonly TimeProvider s_system = new SystemTimeProvider();


Inlined in 65710d0. Now using auto-property with initializer: public static TimeProvider System { get; } = new SystemTimeProvider();

Tyrrrz · 2026-02-17T21:19:04Z

PolyShim/Net80/TimeProvider.cs

+        }
+    }
+
+#if !NETSTANDARD1_0 && !NETSTANDARD1_1


Why? Explain with a comment. Also, use this pattern (pseudo): if net standard and not best standard 1.2 or greater

Fixed in 65710d0. Added comment explaining Timer/TimerCallback aren't available on .NET Standard 1.0/1.1, and changed pattern to !(NETSTANDARD && !NETSTANDARD1_2_OR_GREATER).

Tyrrrz · 2026-02-17T21:19:49Z

PolyShim/Net80/TimeProvider.cs

+        SystemThreading.CancellationToken cancellationToken = default
+    )
+    {
+#if NETSTANDARD1_3_OR_GREATER || NETCOREAPP || NET45_OR_GREATER


Fixed in 65710d0. The nested conditional was unnecessary - moved the TFM check to the method-level conditional. Delay and CreateCancellationTokenSource now only exist on frameworks that have the required APIs (.NET 4.5+, .NET Standard 1.3+, .NET Core 1.0+).

Tyrrrz · 2026-02-17T21:20:15Z

PolyShim/Net80/TimeProvider.cs

+
+    public SystemThreading.CancellationTokenSource CreateCancellationTokenSource(TimeSpan delay)
+    {
+#if NETSTANDARD1_3_OR_GREATER || NETCOREAPP || NET45_OR_GREATER


Fixed in 65710d0. Same as above - moved the TFM check to method-level conditional instead of having nested conditionals inside the method body.

…ystem property, add comments, fix conditionals Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Tyrrrz

@copilot

Tyrrrz · 2026-02-17T21:28:51Z

PolyShim/Net80/TimeProvider.cs

+        return new DateTimeOffset(utcDateTime.DateTime + offset, offset);
+    }
+
+    public abstract TimeZoneInfo LocalTimeZone { get; }


Properties should be above methods

Fixed in be2f69e. Moved LocalTimeZone property above GetUtcNow() method.

Tyrrrz · 2026-02-17T21:29:15Z

PolyShim/Net80/TimeProvider.cs

+
+    public virtual TimeSpan GetElapsedTime(long startingTimestamp, long endingTimestamp)
+    {
+        // Stopwatch.GetElapsedTime was added in .NET 7, so we need to calculate it manually


Or we can polyfill that?

Added in be2f69e. Created Net70/Stopwatch.cs polyfill with GetElapsedTime(long startingTimestamp, long endingTimestamp) and GetElapsedTime(long startingTimestamp) extension methods. TimeProvider now delegates to the polyfilled method.

Tyrrrz · 2026-02-17T21:30:11Z

PolyShim/Net80/TimeProvider.cs

+        }
+    }
+
+    // Timer and TimerCallback are not available on .NET Standard 1.0 and 1.1


ITimer is polyfilled. Can we polyfill TimerCallback?

Added in be2f69e. Created NetStandard12/TimerCallback.cs polyfill defining the delegate for .NET Standard < 1.2. CreateTimer is now available on all frameworks except .NET Standard 1.0/1.1.

Tyrrrz · 2026-02-17T21:31:08Z

PolyShim/Net80/TimeProvider.cs

+    ) => new SystemTimeProviderTimer(dueTime, period, callback, state);
+#endif
+
+    // Task.Delay, Timeout.InfiniteTimeSpan, and CancellationTokenSource(TimeSpan) require .NET 4.5+


Polyfill the missing members so they're available everywhere where Task is available (except infinite timeout, that should be available everywhere).

Added in be2f69e. Created Net45/Task.cs with Task.Delay polyfill for frameworks < .NET 4.5. Removed all nested conditionals - Delay and CreateCancellationTokenSource are now available wherever FEATURE_TASK is defined. Note: CancellationTokenSource(TimeSpan) constructor doesn't exist on .NET 4.0, so CreateCancellationTokenSource returns a plain instance for those frameworks.

Tyrrrz · 2026-02-17T21:31:36Z

PolyShim/Net80/TimeProvider.cs

+        return Task.Delay(delay, cancellationToken);
+    }
+
+    public SystemThreading.CancellationTokenSource CreateCancellationTokenSource(TimeSpan delay)


Do we need a fully qualified return type?

Fixed in be2f69e. Changed from SystemThreading.CancellationTokenSource to just CancellationTokenSource by updating using statements.

Tyrrrz · 2026-02-17T21:31:55Z

PolyShim/Net80/TimeProvider.cs

+
+    public SystemThreading.CancellationTokenSource CreateCancellationTokenSource(TimeSpan delay)
+    {
+        if (delay < TimeSpan.Zero && delay != SystemThreading.Timeout.InfiniteTimeSpan)


Drop the guards in all polyfills

Fixed in be2f69e. Removed all ArgumentOutOfRangeException validation checks from Delay() and CreateCancellationTokenSource().

Tyrrrz · 2026-02-17T21:32:30Z

PolyShim/Net80/TimeProvider.cs

+#if !(NETSTANDARD && !NETSTANDARD1_2_OR_GREATER)
+    private sealed class SystemTimeProviderTimer : SystemThreading.ITimer
+    {
+        private readonly SystemThreading.Timer _timer;


Can this be polyfilled?

Timer itself is part of the BCL and can't be polyfilled as it's a native type. However, the CreateTimer method now works on all frameworks where Timer is available (everywhere except .NET Standard 1.0/1.1) thanks to the TimerCallback polyfill.

Tyrrrz · 2026-02-17T21:33:19Z

PolyShim/Net80/TimeProvider.cs

+        public ValueTask DisposeAsync()
+        {
+#if NETCOREAPP3_0_OR_GREATER || NETSTANDARD2_1_OR_GREATER
+            return _timer.DisposeAsync();


Just polyfill this method on Timer, same as we do for Stream

Added in be2f69e. Created NetCore30/Timer.cs with DisposeAsync() extension method. SystemTimeProviderTimer now calls _timer.DisposeAsync() directly, which uses the polyfill on older frameworks.

…simplify code Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Tyrrrz

@copilot

Tyrrrz · 2026-02-17T21:51:22Z

PolyShim/Net45/Task.cs

+// Timer is required for Task.Delay implementation
+#if !(NETSTANDARD && !NETSTANDARD1_2_OR_GREATER)


Can we use Thread Sleep there? If not, move this guard to cover the actual members only

Fixed in a322476. Moved the Timer availability guard (!(NETSTANDARD && !NETSTANDARD1_2_OR_GREATER)) to wrap only the Task.Delay members, keeping the outer FEATURE_TASK guard at the file level.

Tyrrrz · 2026-02-17T21:52:09Z

PolyShim/Net45/Task.cs

+                TimeSpan.FromMilliseconds(-1)
+            );
+
+            cancellationToken.Register(() =>


Needs to be disposed?

Fixed in a322476. Changed to use nullable Timer pattern - timer disposes itself in both the callback and the cancellation handler, preventing resource leaks.

I mean that the cancellation registration needs to be disposed

Fixed in f7dfa0d. Now capturing the registration result and disposing it when the task completes: var registration = cancellationToken.Register(...) followed by tcs.Task.ContinueWith(_ => registration.Dispose(), ...)

Tyrrrz · 2026-02-17T21:54:17Z

PolyShim/Net80/TimeProvider.cs

+#if NET45_OR_GREATER || NETSTANDARD1_3_OR_GREATER || NETCOREAPP
+        return new CancellationTokenSource(delay);
+#else
+        // CancellationTokenSource(int) constructor added in .NET 4.5
+        // For older frameworks, just return a plain instance
+        return new CancellationTokenSource();
+#endif


Or use parameterless constructor and call CancelAfter

Fixed in a322476. Changed to use parameterless constructor and call CancelAfter() method instead of using the constructor overload.

Tyrrrz · 2026-02-17T21:55:23Z

PolyShim/NetCore30/Timer.cs

@@ -0,0 +1,25 @@
+#if FEATURE_ASYNCINTERFACES && !(NETCOREAPP3_0_OR_GREATER || NETSTANDARD2_1_OR_GREATER)


Move the feature guard to the member. Rewrite the TFM guard to be consistent with others.

Fixed in a322476. Moved FEATURE_ASYNCINTERFACES guard to the DisposeAsync member. Changed TFM guard from FEATURE_ASYNCINTERFACES && !(NETCOREAPP3_0_OR_GREATER || NETSTANDARD2_1_OR_GREATER) to (NETCOREAPP && !NETCOREAPP3_0_OR_GREATER) || (NETFRAMEWORK) || (NETSTANDARD && !NETSTANDARD2_1_OR_GREATER) with additional nested guard for Timer availability.

…embers Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Copilot · 2026-02-18T08:04:56Z

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

learn.microsoft.com
- Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Configure Actions setup steps to set up my environment, which run before the firewall is enabled
Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

…TimeProvider API Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Copilot · 2026-02-18T08:23:41Z

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

learn.microsoft.com
- Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Configure Actions setup steps to set up my environment, which run before the firewall is enabled
Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Copilot

Pull request overview

This PR adds polyfills for TimeProvider and ITimer introduced in .NET 8, enabling time abstraction APIs on older frameworks. The implementation includes the core TimeProvider abstract class with its System singleton, along with supporting polyfills for Stopwatch.GetElapsedTime, TimerCallback delegate, Task.Delay, Task.FromCanceled, and Timer.DisposeAsync to ensure broad framework compatibility.

Changes:

Added TimeProvider and ITimer polyfills for .NET 8 features with automatic feature detection for Microsoft.Bcl.TimeProvider package
Added supporting polyfills (Stopwatch.GetElapsedTime, TimerCallback, Task.Delay, Task.FromCanceled, Timer.DisposeAsync) to enable TimeProvider on older frameworks
Updated package references and documentation to include Microsoft.Bcl.TimeProvider compatibility package

Reviewed changes

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

Show a summary per file

File	Description
Readme.md	Added Microsoft.Bcl.TimeProvider to the list of recognized compatibility packages
PolyShim/Signatures.md	Updated signature counts and added documentation for new polyfilled APIs (TimeProvider, ITimer, Stopwatch methods, Task methods, Timer.DisposeAsync)
PolyShim/PolyShim.targets	Added FEATURE_TIMEPROVIDER detection for .NET 8+ and Microsoft.Bcl.TimeProvider package
PolyShim/PolyShim.csproj	Added Microsoft.Bcl.TimeProvider package references for various target frameworks; upgraded Microsoft.Bcl.AsyncInterfaces to 8.0.0
PolyShim/NetCore30/Timer.cs	Added DisposeAsync extension method for Timer on older frameworks
PolyShim/NetCore10/TimerCallback.cs	Added TimerCallback delegate polyfill for .NET Standard < 1.2
PolyShim/NetCore10/Task.cs	Added Task.Delay and Task.FromCanceled polyfills for .NET Framework < 4.5
PolyShim/Net80/TimeProvider.cs	Core TimeProvider abstract class implementation with System singleton and timer creation
PolyShim/Net80/ITimer.cs	ITimer interface with conditional IAsyncDisposable support
PolyShim/Net70/Stopwatch.cs	Added GetElapsedTime static methods for calculating elapsed time from timestamps

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

PolyShim/NetCore10/Task.cs

Copilot · 2026-02-18T20:28:56Z

PolyShim/PolyShim.csproj

    <PackageReference
      Include="Microsoft.Bcl.AsyncInterfaces"
-      Version="7.0.0"
+      Version="8.0.0"
      PrivateAssets="all"
      Condition="'$(TargetFrameworkIdentifier)' == '.NETStandard' AND $([MSBuild]::VersionGreaterThanOrEquals($(TargetFrameworkVersion), '2.0'))"
    />
    <PackageReference
      Include="Microsoft.Bcl.AsyncInterfaces"
-      Version="7.0.0"
+      Version="8.0.0"
      PrivateAssets="all"
      Condition="'$(TargetFrameworkIdentifier)' == '.NETCoreApp' AND $([MSBuild]::VersionGreaterThanOrEquals($(TargetFrameworkVersion), '2.0')) AND $([MSBuild]::VersionLessThan($(TargetFrameworkVersion), '3.0'))"
    />
    <PackageReference
      Include="Microsoft.Bcl.AsyncInterfaces"
-      Version="7.0.0"
+      Version="8.0.0"
      PrivateAssets="all"
      Condition="'$(TargetFrameworkIdentifier)' == '.NETFramework' AND $([MSBuild]::VersionGreaterThanOrEquals($(TargetFrameworkVersion), '4.6.1'))"
    />


The Microsoft.Bcl.AsyncInterfaces package has been upgraded from version 7.0.0 to 8.0.0 in this PR, but this change is not mentioned in the PR description. While this upgrade appears intentional and consistent with other package updates, it should be documented as it could affect consumers of this library.

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

codecov · 2026-02-18T20:38:04Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (cc8bb0e) to head (11cb66f).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@     Coverage Diff      @@
##   master   #79   +/-   ##
============================
============================

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Initial plan

264ad04

Copilot AI assigned Copilot and Tyrrrz Feb 17, 2026

Copilot started work on behalf of Tyrrrz February 17, 2026 20:34 View session

Copilot AI and others added 2 commits February 17, 2026 20:44

Add TimeProvider and ITimer polyfills for .NET 8

4149ee0

Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Update README to mention Microsoft.Bcl.TimeProvider compatibility pac…

01cac72

…kage Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Add polyfills for Time provider in .NET 8~~ Add polyfills for TimeProvider and ITimer (.NET 8) Feb 17, 2026

Copilot AI requested a review from Tyrrrz February 17, 2026 20:51

Copilot finished work on behalf of Tyrrrz February 17, 2026 20:51

Tyrrrz added the enhancement New feature or request label Feb 17, 2026

Tyrrrz reviewed Feb 17, 2026

View reviewed changes

Copilot started work on behalf of Tyrrrz February 17, 2026 21:04 View session

Address code review feedback: simplify conditionals, remove guards, u…

a4b3590

…se polyfilled APIs Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Copilot AI requested a review from Tyrrrz February 17, 2026 21:15

Copilot finished work on behalf of Tyrrrz February 17, 2026 21:15

Tyrrrz reviewed Feb 17, 2026

View reviewed changes

Copilot started work on behalf of Tyrrrz February 17, 2026 21:22 View session

Address review feedback: remove conditional for Tasks using, inline S…

65710d0

…ystem property, add comments, fix conditionals Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Copilot AI requested a review from Tyrrrz February 17, 2026 21:27

Copilot finished work on behalf of Tyrrrz February 17, 2026 21:27

Tyrrrz reviewed Feb 17, 2026

View reviewed changes

Copilot started work on behalf of Tyrrrz February 17, 2026 21:34 View session

Address review: add polyfills, reorganize properties, remove guards, …

be2f69e

…simplify code Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Copilot AI requested a review from Tyrrrz February 17, 2026 21:44

Copilot finished work on behalf of Tyrrrz February 17, 2026 21:44

Tyrrrz reviewed Feb 17, 2026

View reviewed changes

Copilot started work on behalf of Tyrrrz February 17, 2026 21:57 View session

Address review: fix timer disposal, use CancelAfter, move guards to m…

a322476

…embers Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Copilot AI requested a review from Tyrrrz February 17, 2026 22:00

Copilot finished work on behalf of Tyrrrz February 17, 2026 22:00

Copilot AI requested a review from Tyrrrz February 18, 2026 08:05

Copilot finished work on behalf of Tyrrrz February 18, 2026 08:05

Copilot started work on behalf of Tyrrrz February 18, 2026 08:21 View session

Remove Delay and CreateCancellationTokenSource methods - not part of …

48fe816

…TimeProvider API Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>

Copilot finished work on behalf of Tyrrrz February 18, 2026 08:23

Copilot started work on behalf of Tyrrrz February 18, 2026 08:24 View session

Copilot finished work on behalf of Tyrrrz February 18, 2026 08:24

Tyrrrz marked this pull request as ready for review February 18, 2026 20:23

Copilot AI review requested due to automatic review settings February 18, 2026 20:23

Copilot started reviewing on behalf of Tyrrrz February 18, 2026 20:24 View session

Tyrrrz approved these changes Feb 18, 2026

View reviewed changes

Copilot AI reviewed Feb 18, 2026

View reviewed changes

Update PolyShim/NetCore10/Task.cs

11cb66f

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

Tyrrrz changed the title ~~Add polyfills for TimeProvider and ITimer (.NET 8)~~ Add polyfills for TimeProvider and ITimer Feb 18, 2026

Tyrrrz merged commit 328ab8f into master Feb 18, 2026
5 checks passed

Tyrrrz deleted the copilot/add-time-provider-polyfills branch February 18, 2026 20:37

		@@ -0,0 +1,26 @@
		#if !FEATURE_TIMEPROVIDER && ((NETCOREAPP && !NET8_0_OR_GREATER) \|\| (NETFRAMEWORK) \|\| (NETSTANDARD))

		// Timer is required for Task.Delay implementation
		#if !(NETSTANDARD && !NETSTANDARD1_2_OR_GREATER)

		@@ -0,0 +1,25 @@
		#if FEATURE_ASYNCINTERFACES && !(NETCOREAPP3_0_OR_GREATER \|\| NETSTANDARD2_1_OR_GREATER)

Conversation

Copilot AI commented Feb 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Core Polyfills

Supporting Polyfills Created

Implementation Details

Feature Detection

Uh oh!

Tyrrrz left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Tyrrrz left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Copilot AI commented Feb 17, 2026 •

edited

Loading