[dotnet] Support `UnhandledPromptBehavior` option as string and map #16557

nvborisenko · 2025-11-06T18:21:24Z

User description

🔗 Related Issues

💥 What does this PR do?

This PR changes UnhandledPromptBehavior property return type to custom type, supporting both string and map representation at serialization phase.

🔧 Implementation Notes

It is not a breaking change at compilation level! One file change to emphasize that users are not affected.

💡 Additional Considerations

I intentionally didn't introduce new property in Options class, it would lead to a mess in my opinion.

🔄 Types of changes

New feature (non-breaking change which adds functionality and tests!)
Breaking change (fix or feature that would cause existing functionality to change)

PR Type

Enhancement

Description

Support UnhandledPromptBehavior as both string and dictionary representations
Introduce abstract record hierarchy for flexible prompt behavior configuration
Enable per-prompt-type behavior specification via UnhandledPromptBehaviorMultiOption
Maintain backward compatibility with existing enum-based API

Diagram Walkthrough

flowchart LR
  A["UnhandledPromptBehavior enum"] -->|implicit conversion| B["UnhandledPromptBehaviorOption"]
  B -->|Single| C["UnhandledPromptBehaviorSingleOption"]
  B -->|Multi| D["UnhandledPromptBehaviorMultiOption"]
  C -->|serializes to| E["string capability"]
  D -->|serializes to| F["dictionary capability"]

File Walkthrough

Relevant files

Enhancement

DriverOptions.cs `Add flexible prompt behavior option types with dual serialization` dotnet/src/webdriver/DriverOptions.cs Introduced abstract record `UnhandledPromptBehaviorOption` with two sealed implementations: `UnhandledPromptBehaviorSingleOption` (for string representation) and `UnhandledPromptBehaviorMultiOption` (for dictionary representation with per-prompt-type settings) Added implicit operator and factory methods (`Single()`, `Multi()`) for convenient instantiation Changed `UnhandledPromptBehavior` property type from enum to nullable `UnhandledPromptBehaviorOption` Refactored serialization logic in `GenerateDesiredCapabilities()` to handle both single and multi-option cases, converting to appropriate capability format Updated merge conflict detection to use null checks instead of enum comparison Extracted enum-to-string conversion into local helper function `UnhandledPromptBehaviorToString()`	+79/-18

qodo-code-review · 2025-11-06T18:21:54Z

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance

🟢

No security concerns identified

No security vulnerabilities detected by AI analysis. Human verification advised for critical code.

Ticket Compliance

🟡

🎫 #16102

🟢	Allow setting unhandledPromptBehavior as a dictionary/map per spec for BiDi/Chrome 137+.
	Preserve ability to set unhandledPromptBehavior as a single value (legacy enum) for backward compatibility.
	Ensure options serialization outputs correct wire values (strings or map) for capabilities.
	Avoid requiring users to use AdditionalOption workaround or internal DesiredCapabilities.
	Behavior should work in .NET bindings (C#) without breaking compilation for existing users.
⚪	Validate in real Chrome 137+ with BiDi that alerts behave per provided map (runtime/browser check).

Codebase Duplication Compliance

⚪

Codebase context is not defined

Follow the guide to enable codebase context checks.

Custom Compliance

🟢

Generic: Meaningful Naming and Self-Documenting Code

Objective: Ensure all identifiers clearly express their purpose and intent, making code
self-documenting

Status: Passed

Generic: Secure Error Handling

Objective: To prevent the leakage of sensitive system information through error messages while
providing sufficient detail for internal debugging.

Status: Passed

Generic: Secure Logging Practices

Objective: To ensure logs are useful for debugging and auditing without exposing sensitive
information like PII, PHI, or cardholder data.

Status: Passed

⚪

Generic: Comprehensive Audit Trails

Objective: To create a detailed and reliable record of critical system actions for security analysis
and compliance.

Status:
No auditing: The new serialization logic for UnhandledPromptBehavior does not include any audit logging
of critical actions, but this may be out of scope for a client options class.

Referred Code

[return: NotNullIfNotNull(nameof(behavior))]
static string? UnhandledPromptBehaviorToString(UnhandledPromptBehavior? behavior) => behavior switch
{
    Selenium.UnhandledPromptBehavior.Ignore => "ignore",
    Selenium.UnhandledPromptBehavior.Accept => "accept",
    Selenium.UnhandledPromptBehavior.Dismiss => "dismiss",
    Selenium.UnhandledPromptBehavior.AcceptAndNotify => "accept and notify",
    Selenium.UnhandledPromptBehavior.DismissAndNotify => "dismiss and notify",
    _ => null
};

if (this.UnhandledPromptBehavior is UnhandledPromptBehaviorSingleOption singleOption)
{
    var stringValue = UnhandledPromptBehaviorToString(singleOption.Value);

    if (stringValue is not null)
    {
        capabilities.SetCapability(CapabilityType.UnhandledPromptBehavior, stringValue);
    }
}
else if (this.UnhandledPromptBehavior is UnhandledPromptBehaviorMultiOption multiOption)


 ... (clipped 33 lines)

Generic: Robust Error Handling and Edge Case Management

Objective: Ensure comprehensive error handling that provides meaningful context and graceful
degradation

Status:
Missing validation: Mapping functions accept nullable behaviors and silently skip invalid/unknown values
without error or logging, which could hide misconfigurations.

Referred Code

[return: NotNullIfNotNull(nameof(behavior))]
static string? UnhandledPromptBehaviorToString(UnhandledPromptBehavior? behavior) => behavior switch
{
    Selenium.UnhandledPromptBehavior.Ignore => "ignore",
    Selenium.UnhandledPromptBehavior.Accept => "accept",
    Selenium.UnhandledPromptBehavior.Dismiss => "dismiss",
    Selenium.UnhandledPromptBehavior.AcceptAndNotify => "accept and notify",
    Selenium.UnhandledPromptBehavior.DismissAndNotify => "dismiss and notify",
    _ => null
};

if (this.UnhandledPromptBehavior is UnhandledPromptBehaviorSingleOption singleOption)
{
    var stringValue = UnhandledPromptBehaviorToString(singleOption.Value);

    if (stringValue is not null)
    {
        capabilities.SetCapability(CapabilityType.UnhandledPromptBehavior, stringValue);
    }
}
else if (this.UnhandledPromptBehavior is UnhandledPromptBehaviorMultiOption multiOption)


 ... (clipped 33 lines)

Generic: Security-First Input Validation and Data Handling

Objective: Ensure all data inputs are validated, sanitized, and handled securely to prevent
vulnerabilities

Status:
Input validation: The new API accepts nullable option values and constructs a dictionary without explicit
validation of keys/values, potentially allowing unsupported configurations to pass
silently.

Referred Code

{
    Dictionary<string, string> multiOptionDictionary = [];

    if (multiOption.Alert is not null)
    {
        multiOptionDictionary["alert"] = UnhandledPromptBehaviorToString(multiOption.Alert);
    }

    if (multiOption.Confirm is not null)
    {
        multiOptionDictionary["confirm"] = UnhandledPromptBehaviorToString(multiOption.Confirm);
    }

    if (multiOption.Prompt is not null)
    {
        multiOptionDictionary["prompt"] = UnhandledPromptBehaviorToString(multiOption.Prompt);
    }

    if (multiOption.BeforeUnload is not null)
    {
        multiOptionDictionary["beforeUnload"] = UnhandledPromptBehaviorToString(multiOption.BeforeUnload);


 ... (clipped 12 lines)

Update

Compliance status legend

🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

dotnet/src/webdriver/DriverOptions.cs

qodo-code-review · 2025-11-06T18:23:10Z

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
High-level	The solution introduces a breaking change The suggestion highlights that changing the `UnhandledPromptBehavior` property's type is a breaking change for users who read its value. It recommends adding a new property for the dictionary-based functionality to ensure backward compatibility. Examples: dotnet/src/webdriver/DriverOptions.cs [198] public UnhandledPromptBehaviorOption? UnhandledPromptBehavior { get; set; } Solution Walkthrough: Before: // In DriverOptions.cs public class DriverOptions { public UnhandledPromptBehavior UnhandledPromptBehavior { get; set; } = UnhandledPromptBehavior.Default; // ... } // Example user code (compiles) public void ConfigureDriver(DriverOptions options) { // Reading the property is straightforward UnhandledPromptBehavior behavior = options.UnhandledPromptBehavior; if (behavior == UnhandledPromptBehavior.Ignore) { // ... } } After: // In DriverOptions.cs public class DriverOptions { public UnhandledPromptBehaviorOption? UnhandledPromptBehavior { get; set; } // ... } // Example user code (breaks compilation) public void ConfigureDriver(DriverOptions options) { // This line no longer compiles due to type mismatch // UnhandledPromptBehavior behavior = options.UnhandledPromptBehavior; // User must now add type checking and casting if (options.UnhandledPromptBehavior is UnhandledPromptBehaviorSingleOption single) { UnhandledPromptBehavior? behavior = single.Value; // ... } } Suggestion importance[1-10]: 9 __ Why: The suggestion correctly identifies a critical source-breaking change in the public API by altering the type of the `UnhandledPromptBehavior` property, which contradicts the PR's claim of being non-breaking.	High
Learned best practice	✅ ~~Validate mapped capability values~~ Suggestion Impact: The commit changed UnhandledPromptBehavior handling to be non-nullable with defaults and made the mapping function throw on unknown values, and it only adds entries when not Default. This achieves the validation intent, though it didn't introduce the exact TryAdd helper. code diff: - [return: NotNullIfNotNull(nameof(behavior))] - static string? UnhandledPromptBehaviorToString(UnhandledPromptBehavior? behavior) => behavior switch + static string UnhandledPromptBehaviorToString(UnhandledPromptBehavior behavior) => behavior switch { Selenium.UnhandledPromptBehavior.Ignore => "ignore", Selenium.UnhandledPromptBehavior.Accept => "accept", Selenium.UnhandledPromptBehavior.Dismiss => "dismiss", Selenium.UnhandledPromptBehavior.AcceptAndNotify => "accept and notify", Selenium.UnhandledPromptBehavior.DismissAndNotify => "dismiss and notify", - _ => null + _ => throw new ArgumentOutOfRangeException(nameof(behavior), $"UnhandledPromptBehavior value '{behavior}' is not recognized."), }; - if (this.UnhandledPromptBehavior is UnhandledPromptBehaviorSingleOption singleOption) + if (this.UnhandledPromptBehavior is UnhandledPromptBehaviorSingleOption singleOption && singleOption.Value != Selenium.UnhandledPromptBehavior.Default) { var stringValue = UnhandledPromptBehaviorToString(singleOption.Value); - if (stringValue is not null) - { - capabilities.SetCapability(CapabilityType.UnhandledPromptBehavior, stringValue); - } + capabilities.SetCapability(CapabilityType.UnhandledPromptBehavior, stringValue); } else if (this.UnhandledPromptBehavior is UnhandledPromptBehaviorMultiOption multiOption) { Dictionary<string, string> multiOptionDictionary = []; - if (multiOption.Alert is not null) + if (multiOption.Alert is not Selenium.UnhandledPromptBehavior.Default) { multiOptionDictionary["alert"] = UnhandledPromptBehaviorToString(multiOption.Alert); } - if (multiOption.Confirm is not null) + if (multiOption.Confirm is not Selenium.UnhandledPromptBehavior.Default) { multiOptionDictionary["confirm"] = UnhandledPromptBehaviorToString(multiOption.Confirm); } - if (multiOption.Prompt is not null) + if (multiOption.Prompt is not Selenium.UnhandledPromptBehavior.Default) { multiOptionDictionary["prompt"] = UnhandledPromptBehaviorToString(multiOption.Prompt); } - if (multiOption.BeforeUnload is not null) + if (multiOption.BeforeUnload is not Selenium.UnhandledPromptBehavior.Default) { multiOptionDictionary["beforeUnload"] = UnhandledPromptBehaviorToString(multiOption.BeforeUnload); } - if (multiOption.Default is not null) + if (multiOption.Default is not Selenium.UnhandledPromptBehavior.Default) { multiOptionDictionary["default"] = UnhandledPromptBehaviorToString(multiOption.Default); } Validate that mapped strings are non-null before adding to the dictionary to prevent invalid capability values. Optionally throw a clear error for unknown enum values. dotnet/src/webdriver/DriverOptions.cs [566-589] -if (multiOption.Alert is not null) +void TryAdd(IDictionary<string, string> dict, string key, UnhandledPromptBehavior? value) { - multiOptionDictionary["alert"] = UnhandledPromptBehaviorToString(multiOption.Alert); + var str = UnhandledPromptBehaviorToString(value); + if (str is null && value is not null) + { + throw new ArgumentOutOfRangeException(nameof(value), $"Unsupported UnhandledPromptBehavior value '{value}'."); + } + if (str is not null) + { + dict[key] = str; + } } -if (multiOption.Confirm is not null) -{ - multiOptionDictionary["confirm"] = UnhandledPromptBehaviorToString(multiOption.Confirm); -} +TryAdd(multiOptionDictionary, "alert", multiOption.Alert); +TryAdd(multiOptionDictionary, "confirm", multiOption.Confirm); +TryAdd(multiOptionDictionary, "prompt", multiOption.Prompt); +TryAdd(multiOptionDictionary, "beforeUnload", multiOption.BeforeUnload); +TryAdd(multiOptionDictionary, "default", multiOption.Default); -if (multiOption.Prompt is not null) -{ - multiOptionDictionary["prompt"] = UnhandledPromptBehaviorToString(multiOption.Prompt); -} - -if (multiOption.BeforeUnload is not null) -{ - multiOptionDictionary["beforeUnload"] = UnhandledPromptBehaviorToString(multiOption.BeforeUnload); -} - -if (multiOption.Default is not null) -{ - multiOptionDictionary["default"] = UnhandledPromptBehaviorToString(multiOption.Default); -} - `[Suggestion processed]` Suggestion importance[1-10]: 6 __ Why: Relevant best practice - Guard external API-facing option mapping with validation to avoid null or invalid values leaking into capabilities.	Low
General	Use init-only setters for record Replace the `set` accessors with `init` accessors in the `UnhandledPromptBehaviorMultiOption` record to make its properties immutable after initialization. dotnet/src/webdriver/DriverOptions.cs [47-58] public sealed record UnhandledPromptBehaviorMultiOption : UnhandledPromptBehaviorOption { - public UnhandledPromptBehavior? Alert { get; set; } + public UnhandledPromptBehavior? Alert { get; init; } - public UnhandledPromptBehavior? Confirm { get; set; } + public UnhandledPromptBehavior? Confirm { get; init; } - public UnhandledPromptBehavior? Prompt { get; set; } + public UnhandledPromptBehavior? Prompt { get; init; } - public UnhandledPromptBehavior? BeforeUnload { get; set; } + public UnhandledPromptBehavior? BeforeUnload { get; init; } - public UnhandledPromptBehavior? Default { get; set; } + public UnhandledPromptBehavior? Default { get; init; } } Apply / Chat Suggestion importance[1-10]: 5 __ Why: The suggestion correctly points out that using `init` setters for a record's properties enhances immutability, which is idiomatic for records and improves code quality and maintainability.	Low
Update

dotnet/src/webdriver/DriverOptions.cs

This reverts commit fc98a78.

dotnet/src/webdriver/DriverOptions.cs

nvborisenko · 2025-11-06T22:22:46Z

dotnet/src/webdriver/DriverOptions.cs

        }

-        if (this.UnhandledPromptBehavior != UnhandledPromptBehavior.Default && other.UnhandledPromptBehavior != UnhandledPromptBehavior.Default)
+        if (this.UnhandledPromptBehavior is not null && other.UnhandledPromptBehavior is not null)


Hm, != operator for records?

It does feel like this method should tolerate it when both values are the same.

However, we should investigate the history of this method before making those decisions.

I understand it as don't send anything over wire if default.

However, we should investigate the history of this method before making those decisions.

Oh yes, we should understand the purpose of this method, not obvious to me at glance.

@jimevans do you know?

RenderMichael

A breaking change like this should be well-tested. Could we add tests here?

nvborisenko · 2025-11-06T22:39:56Z

A breaking change like this should be well-tested. Could we add tests here?

I have no capacity to test the functionality which was not indented to be tested. You can "request changes", please propose.

RenderMichael · 2025-11-06T22:42:09Z

A breaking change like this should be well-tested. Could we add tests here?

I have no capacity to test the functionality which was not indented to be tested. You can "request changes", please propose.

I propose new tests for this functionality.

nvborisenko · 2025-11-06T22:51:14Z

Existing tests don't satisfy you? You ask me to increase the coverage. But what is coverage?

nvborisenko · 2025-11-06T22:58:32Z

@RenderMichael welcome to #15536

UPD: is it only one concern from your side?

YevgeniyShunevych · 2025-11-07T09:42:44Z

Generally, looks good, @nvborisenko. What I don't like is a nullability of properties and parameters (UnhandledPromptBehaviorOption?, UnhandledPromptBehavior?). If I understand correctly, for UnhandledPromptBehavior? null and Default values are equivalent. So 2 states basically indicate the same state. So why not to make it non-nullable and use Default as a default. I would also rename UnhandledPromptBehavior.Default to None, as it is not clear what "Default" means, but "None" means no behavior is specified.

dotnet/src/webdriver/DriverOptions.cs

RenderMichael · 2025-11-26T21:41:50Z

dotnet/src/webdriver/UnhandledPromptBehaviorOption.cs

+    /// <summary>
+    /// Gets or sets the behavior to use when an unexpected alert is encountered during automation.
+    /// </summary>
+    public UnhandledPromptBehavior Alert { get; set; } = UnhandledPromptBehavior.Default;


Is it correct that

All of these are set to Default

They have setters?

Please help me if you see hidden stones.

I have no idea if these work because there are no tests yet, but this looks wrong.

True, it looks strange because historically we have Default in enum. It should be nullable, but it is as is. Please read comments in this PR.

How to move on...

RenderMichael · 2025-12-01T08:57:36Z

dotnet/src/webdriver/DriverOptions.cs

+
+            capabilities.SetCapability(CapabilityType.UnhandledPromptBehavior, stringValue);
+        }
+        else if (this.UnhandledPromptBehavior is UnhandledPromptBehaviorMultiOption multiOption)


We should follow our usual pattern here to have a ToDictionary() method directly on the UnhandledPromptBehaviorMultiOption type:

public Dictionary<string, string> ToDictionary() { Dictionary<string, string> multiOptionDictionary = []; if (Alert is not Selenium.UnhandledPromptBehavior.Default) { multiOptionDictionary["alert"] = UnhandledPromptBehaviorToString(Alert); } if (Confirm is not Selenium.UnhandledPromptBehavior.Default) { multiOptionDictionary["confirm"] = UnhandledPromptBehaviorToString(Confirm); } if (Prompt is not Selenium.UnhandledPromptBehavior.Default) { multiOptionDictionary["prompt"] = UnhandledPromptBehaviorToString(Prompt); } if (BeforeUnload is not Selenium.UnhandledPromptBehavior.Default) { multiOptionDictionary["beforeUnload"] = UnhandledPromptBehaviorToString(BeforeUnload); } if (Default is not Selenium.UnhandledPromptBehavior.Default) { multiOptionDictionary["default"] = UnhandledPromptBehaviorToString(Default); } return multiOptionDictionary; }

and then to call that method here

else if (this.UnhandledPromptBehavior is UnhandledPromptBehaviorMultiOption multiOption) { multiOptionDictionary = multiOption.ToDictionary(); if (multiOptionDictionary.Count != 0) { capabilities.SetCapability(CapabilityType.UnhandledPromptBehavior, multiOptionDictionary); } }

This also allows us to move UnhandledPromptBehaviorToString to the UnhandledPromptBehaviorOption, cleaning up this 150-line method.

RenderMichael · 2025-12-01T08:58:20Z

dotnet/src/webdriver/DriverOptions.cs

+                multiOptionDictionary["default"] = UnhandledPromptBehaviorToString(multiOption.Default);
+            }
+
+            if (multiOptionDictionary.Count != 0)


Do we need to guard against {}? It feels like we should transparently sent this to the server.

What happens if we do this, anyway?

RenderMichael · 2025-12-01T09:16:43Z

dotnet/src/webdriver/UnhandledPromptBehaviorOption.cs

+    /// Creates an <see cref="UnhandledPromptBehaviorOption"/> allowing individual <see cref="UnhandledPromptBehavior"/> values per prompt type.
+    /// </summary>
+    /// <returns>An <see cref="UnhandledPromptBehaviorOption"/> with per-prompt configurable behaviors.</returns>
+    public static UnhandledPromptBehaviorOption Multi()


These semantics are a little unwieldy:

Maybe we could have a nested class here?

Since we are changing the type name, I think it would align better with the spec to call it UnhandledPromptHandlers

nvborisenko added 3 commits November 6, 2025 21:00

First implementation

cb357bf

Don't change tests to emphasize backward compatibility

5a2cdd8

Static factory?

5c53619

selenium-ci added the C-dotnet .NET Bindings label Nov 6, 2025

qodo-code-review bot added the Review effort 2/5 label Nov 6, 2025

nvborisenko commented Nov 6, 2025

View reviewed changes

dotnet/src/webdriver/DriverOptions.cs Outdated Show resolved Hide resolved

nvborisenko commented Nov 6, 2025

View reviewed changes

dotnet/src/webdriver/DriverOptions.cs Outdated Show resolved Hide resolved

nvborisenko commented Nov 6, 2025

View reviewed changes

dotnet/src/webdriver/DriverOptions.cs Outdated Show resolved Hide resolved

nvborisenko added 2 commits November 6, 2025 21:53

Throw exception if enum is not branched

fc98a78

Revert "Throw exception if enum is not branched"

75fc654

This reverts commit fc98a78.

nvborisenko commented Nov 6, 2025

View reviewed changes

dotnet/src/webdriver/DriverOptions.cs Outdated Show resolved Hide resolved

nvborisenko requested a review from RenderMichael November 6, 2025 19:25

Throw exception if enum is not branched

50275d2

nvborisenko commented Nov 6, 2025

View reviewed changes

RenderMichael requested changes Nov 6, 2025

View reviewed changes

nvborisenko commented Nov 7, 2025

View reviewed changes

dotnet/src/webdriver/DriverOptions.cs Outdated Show resolved Hide resolved

nvborisenko commented Nov 7, 2025

View reviewed changes

dotnet/src/webdriver/DriverOptions.cs Outdated Show resolved Hide resolved

nvborisenko added 6 commits November 7, 2025 22:00

Non nullable enum values

9accc2b

Merge branch 'trunk' into dotnet-prompt-behavior-dict

1acc7de

Move to new UnhandledPromptBehaviorOption file

727a5b3

Docs

96cd5cd

Docs for possible types

83b5218

Clear as options

f5e5845

RenderMichael reviewed Nov 26, 2025

View reviewed changes

RenderMichael reviewed Dec 1, 2025

View reviewed changes

Uh oh!

[dotnet] Support UnhandledPromptBehavior option as string and map #16557

Are you sure you want to change the base?

[dotnet] Support UnhandledPromptBehavior option as string and map #16557

Conversation

nvborisenko commented Nov 6, 2025 • edited by qodo-code-review bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

User description

🔗 Related Issues

💥 What does this PR do?

🔧 Implementation Notes

💡 Additional Considerations

🔄 Types of changes

PR Type

Description

Diagram Walkthrough

File Walkthrough

Uh oh!

qodo-code-review bot commented Nov 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Compliance Guide 🔍

Uh oh!

Uh oh!

qodo-code-review bot commented Nov 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Code Suggestions ✨

Examples:

Solution Walkthrough:

Before:

After:

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

RenderMichael left a comment

Choose a reason for hiding this comment

Uh oh!

nvborisenko commented Nov 6, 2025

Uh oh!

RenderMichael commented Nov 6, 2025

Uh oh!

nvborisenko commented Nov 6, 2025

Uh oh!

nvborisenko commented Nov 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

YevgeniyShunevych commented Nov 7, 2025

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

[dotnet] Support `UnhandledPromptBehavior` option as string and map #16557

[dotnet] Support `UnhandledPromptBehavior` option as string and map #16557

nvborisenko commented Nov 6, 2025 •

edited by qodo-code-review bot

Loading

qodo-code-review bot commented Nov 6, 2025 •

edited

Loading

qodo-code-review bot commented Nov 6, 2025 •

edited

Loading

nvborisenko commented Nov 6, 2025 •

edited

Loading