Network Log - ClearExisting log operation by GokulBothe99 · Pull Request #4260 · Ginger-Automation/Ginger

GokulBothe99 · 2025-07-22T10:51:32Z

Thank you for your contribution.
Before submitting this PR, please make sure:

Summary by CodeRabbit

New Features
- Added a new browser control action to clear existing network logs during web automation.
Bug Fixes
- Improved handling of network log file names when saving logs.
Refactor
- Separated the clearing of network logs from the log processing flow for better control and clarity.
- Enhanced network log monitoring management and cleanup in web drivers.
Style
- Adjusted UI margins for radio button panels for improved layout.
- Removed the "Clear Existing Network Log" checkbox from the user interface.

coderabbitai · 2025-07-22T10:51:39Z

"""

Walkthrough

This change removes the "Clear Existing Network Log" checkbox from the UI and all related code, properties, and bindings. Instead, it introduces a new explicit browser control action, ClearExistingNetworkLog, with corresponding handler logic and public methods for both Playwright and Selenium drivers to clear network logs programmatically.

Changes

File(s)	Change Summary
.../Actions/ActionEditPages/ActBrowserElementEditPage.xaml	Removed "Clear Existing Network Log" checkbox panel; adjusted margins for other UI elements.
.../Actions/ActionEditPages/ActBrowserElementEditPage.xaml.cs	Removed all logic and bindings for the "Clear Existing Log" control; updated event handler for network log saving.
.../Actions/Web/ActBrowserElement.cs	Removed `ClearExistingLog` property and field; added `ClearExistingNetworkLog` to control actions enum.
.../Web/ActionHandlers/ActBrowserElementHandler.cs	Added handling for new control action `ClearExistingNetworkLog` with error handling and synchronous invocation.
.../Web/BrowserHelper.cs	Removed extra blank lines; no functional change.
.../Web/Playwright/PlaywrightBrowserTab.cs	Added guard to prevent duplicate network log monitoring start; removed automatic clearing of logs after retrieval; added public method to clear logs explicitly.
.../Web/Selenium/SeleniumDriver.cs	Added async method for clearing network logs; updated handler and removed old conditional clearing logic; improved stop monitoring; added private cleanup method.
.../Platforms/PlatformsInfo/WebPlatformInfo.cs	Added `ClearExistingNetworkLog` to supported browser control operations.

Sequence Diagram(s)

sequenceDiagram
    participant UI
    participant ActionHandler
    participant BrowserTab

    UI->>ActionHandler: Invoke ClearExistingNetworkLog action
    alt Playwright
        ActionHandler->>BrowserTab: ClearExistingNetworkLog()
        BrowserTab-->>ActionHandler: (Logs cleared)
    else Selenium
        ActionHandler->>BrowserTab: ClearExistingNetworkLog()
        BrowserTab-->>ActionHandler: (Logs cleared)
    end
    ActionHandler-->>UI: Completion/Error status

Estimated code review effort

3 (~45 minutes)

Possibly related PRs

Ginger-Automation/Ginger#4253: Removes the "Clear Existing Network Log" UI and related code, which is directly opposite to the additions in this PR.
Ginger-Automation/Ginger#4010: Adds network log handling methods in the same handler class, closely related to the new control action and handler logic introduced here.
Ginger-Automation/Ginger#4035: Improves network log capture robustness in PlaywrightBrowserTab, related to network log handling but without clearing functionality.

Suggested reviewers

Maheshkale447

Poem

A checkbox vanished, logs now clear by code,
No more stray panels cluttering the road.
Actions explicit, with handlers anew,
Logs swept away with a click (or a queue).
🐇 Code hops forward, neat and bright—
Network logs vanish, out of sight!
"""

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c4e9c1a and 138f677.

📒 Files selected for processing (2)

Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/ActionHandlers/ActBrowserElementHandler.cs (2 hunks)
Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs (8 hunks)

🧠 Learnings (3)

📓 Common learnings

Learnt from: prashelke
PR: Ginger-Automation/Ginger#3429
File: Ginger/Ginger/AutomatePageLib/AddActionMenu/WindowExplorer/Common/WindowExplorerPage.xaml.cs:1139-1142
Timestamp: 2024-07-26T22:04:12.930Z
Learning: The user has updated the code by removing or addressing the commented code as suggested in the review.

Learnt from: IamRanjeetSingh
PR: Ginger-Automation/Ginger#3862
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightNonPersistentBrowser.cs:26-38
Timestamp: 2024-08-14T11:51:45.785Z
Learning: The use of `Task.Run` in the `PlaywrightNonPersistentBrowser` constructor is planned to be removed in future changes.

Learnt from: prashelke
PR: Ginger-Automation/Ginger#4017
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs:11096-11096
Timestamp: 2024-12-04T11:45:57.024Z
Learning: In the `OnNetworkRequestSent` method in `SeleniumDriver.cs`, adding a null check for `_BrowserHelper` is not necessary.

Learnt from: GokulBothe99
PR: Ginger-Automation/Ginger#3835
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightDriver.cs:1022-1029
Timestamp: 2024-07-18T09:05:15.264Z
Learning: The user prefers readability over concise code in the context of deactivating locators in the `LearnElementInfoDetails` method of the `PlaywrightDriver` class.

Learnt from: GokulBothe99
PR: Ginger-Automation/Ginger#4095
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs:1330-1333
Timestamp: 2025-02-11T13:45:38.303Z
Learning: The `SwitchToShadowDomAsync` method in PlaywrightBrowserTab class is intentionally implemented as a placeholder that returns true, with the actual shadow DOM handling logic to be implemented in the future.

Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/ActionHandlers/ActBrowserElementHandler.cs (22)

Learnt from: IamRanjeetSingh
PR: #3738
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs:72-77
Timestamp: 2024-06-07T20:51:31.445Z
Learning: Error handling for the GoToURLAsync method in the PlaywrightBrowserTab class is managed in the parent method or surrounding context.

Learnt from: GokulBothe99
PR: #4242
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs:623-656
Timestamp: 2025-06-24T06:08:31.804Z
Learning: In Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs, the CSS selector construction by directly interpolating user input values in the GetElementLocator method is considered acceptable by the team and working as expected for their use cases.

Learnt from: prashelke
PR: #4017
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs:11096-11096
Timestamp: 2024-12-04T11:45:57.024Z
Learning: In the OnNetworkRequestSent method in SeleniumDriver.cs, adding a null check for _BrowserHelper is not necessary.

Learnt from: GokulBothe99
PR: #4107
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/ActionHandlers/ActUIElementHandler.cs:255-297
Timestamp: 2025-02-19T06:06:45.918Z
Learning: In the ActUIElementHandler class, error handling is done at the parent HandleAsync method level, making try-catch blocks in child methods unnecessary if they only rethrow exceptions.

Learnt from: GokulBothe99
PR: #4107
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserElement.cs:20-20
Timestamp: 2025-02-19T06:09:12.815Z
Learning: The InvalidActionConfigurationException from Amdocs.Ginger.CoreNET.Drivers.CoreDrivers.Web.Exceptions namespace is used for handling configuration errors in Playwright browser element operations, particularly in drag-drop and drawing actions.

Learnt from: IamRanjeetSingh
PR: #3753
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs:99-99
Timestamp: 2024-06-12T12:54:44.221Z
Learning: User IamRanjeetSingh prefers exceptions to propagate rather than being caught and handled locally within methods in PlaywrightBrowserTab.cs.

Learnt from: GokulBothe99
PR: #4095
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs:1330-1333
Timestamp: 2025-02-11T13:45:38.303Z
Learning: The SwitchToShadowDomAsync method in PlaywrightBrowserTab class is intentionally implemented as a placeholder that returns true, with the actual shadow DOM handling logic to be implemented in the future.

Learnt from: GokulBothe99
PR: #4107
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserElement.cs:664-705
Timestamp: 2025-02-19T06:07:43.309Z
Learning: In the Ginger automation framework, exceptions in lower-level browser element operations are handled by their parent methods, which have more context about the operation being performed. Therefore, catch-and-rethrow without additional context is acceptable in the element-level methods.

Learnt from: IamRanjeetSingh
PR: #3783
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightDriver.cs:129-129
Timestamp: 2024-06-24T08:39:59.351Z
Learning: User IamRanjeetSingh has indicated that the RunAction method in the PlaywrightDriver class should remain synchronous due to current constraints in the codebase.

Learnt from: IamRanjeetSingh
PR: #3862
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightNonPersistentBrowser.cs:26-38
Timestamp: 2024-08-14T11:51:45.785Z
Learning: The use of Task.Run in the PlaywrightNonPersistentBrowser constructor is planned to be removed in future changes.

Learnt from: prashelke
PR: #4010
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs:773-801
Timestamp: 2024-11-28T14:05:03.513Z
Learning: In PlaywrightBrowserTab.OnNetworkResponseReceived, it's not necessary to call Response.BodyAsync().

Learnt from: prashelke
PR: #4249
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs:4684-4695
Timestamp: 2025-07-10T07:11:28.974Z
Learning: In Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs, attribute values used in CSS selectors (e.g., ByTitle, ByAriaLabel, ByDataTestId, ByPlaceholder) should be escaped using a helper method (such as EscapeCssAttributeValue) to prevent selector breakage due to special characters.

Learnt from: prashelke
PR: #4249
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs:7199-7202
Timestamp: 2025-07-10T07:12:52.786Z
Learning: In Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs, text values used in XPath expressions should be escaped using a helper method (such as EscapeXPathString) to handle single quotes and ensure valid XPath syntax.

Learnt from: prashelke
PR: #3429
File: Ginger/Ginger/AutomatePageLib/AddActionMenu/WindowExplorer/Common/WindowExplorerPage.xaml.cs:1123-1126
Timestamp: 2024-07-26T22:04:12.930Z
Learning: The user has removed an unnecessary empty line at 1123 in WindowExplorerPage.xaml.cs for better code compactness.

Learnt from: prashelke
PR: #3429
File: Ginger/Ginger/AutomatePageLib/AddActionMenu/WindowExplorer/Common/WindowExplorerPage.xaml.cs:1139-1142
Timestamp: 2024-07-26T22:04:12.930Z
Learning: The user has updated the code by removing or addressing the commented code as suggested in the review.

Learnt from: IamRanjeetSingh
PR: #3488
File: Ginger/GingerCoreNET/DataSource/LiteDB.cs:53-60
Timestamp: 2024-07-26T22:04:12.930Z
Learning: User @IamRanjeetSingh has reviewed and accepted the side effects of triggering TryUpgradeDataFile upon setting FileFullPath in GingerLiteDB class.

Learnt from: prashelke
PR: #4254
File: Ginger/GingerCoreNET/GeneralLib/ElementWrapperInfo.cs:89-93
Timestamp: 2025-07-16T14:17:56.429Z
Learning: In Ginger/GingerCoreNET/GeneralLib/ElementWrapperInfo.cs, the user prashelke prefers to keep the current casing for properties insideIframe and insideShadowDOM in the ElementwrapperContext class rather than changing them to PascalCase.

Learnt from: prashelke
PR: #4193
File: Ginger/Ginger/ApplicationModelsLib/POMModels/POMWizardLib/LearnWizard/AIGeneratedPreviewWizardPage.xaml.cs:131-138
Timestamp: 2025-04-30T13:21:34.994Z
Learning: In AIGeneratedPreviewWizardPage.xaml.cs, the current code checks if the response ends with triple backticks and removes them manually. This method is fragile and should be replaced with a more robust regex-based parser. The user plans to address this in a future update.

Learnt from: prashelke
PR: #4222
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Mobile/Appium/GenericAppiumDriver.cs:2645-2656
Timestamp: 2025-06-19T05:08:45.192Z
Learning: In the Ginger mobile automation framework (GenericAppiumDriver.cs), XPath checks for "android" and "XCUIElement" should remain case-sensitive as currently implemented. The team prefers not to make these checks case-insensitive.

Learnt from: IamRanjeetSingh
PR: #3897
File: Ginger/GingerCoreNET/Telemetry/TelemetryQueue.cs:155-186
Timestamp: 2024-09-16T10:13:19.599Z
Learning: In this codebase, methods prefixed with Try (e.g., TryAddToDBAsync, TrySendToCollectorAsync, TryDeleteRecordsFromDBAsync) internally handle exceptions, so additional exception handling in the calling methods is unnecessary.

Learnt from: prashelke
PR: #4228
File: Ginger/Ginger/Actions/ActionEditPages/ActMobileDeviceEditPage.xaml.cs:83-84
Timestamp: 2025-06-16T10:36:01.993Z
Learning: For the UnLockType property in Ginger/Ginger/Actions/ActionEditPages/ActMobileDeviceEditPage.xaml.cs, the current naming convention with capital 'L' in the middle should be maintained per project preferences.

Learnt from: IamRanjeetSingh
PR: #3738
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserWindow.cs:30-67
Timestamp: 2024-06-07T21:51:52.559Z
Learning: User IamRanjeetSingh prefers keeping certain operations synchronous in the context of the PlaywrightBrowserWindow class due to specific requirements or constraints.

Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs (18)

Learnt from: prashelke
PR: #4017
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs:11096-11096
Timestamp: 2024-12-04T11:45:57.024Z
Learning: In the OnNetworkRequestSent method in SeleniumDriver.cs, adding a null check for _BrowserHelper is not necessary.

Learnt from: prashelke
PR: #4249
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs:4684-4695
Timestamp: 2025-07-10T07:11:28.974Z
Learning: In Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs, attribute values used in CSS selectors (e.g., ByTitle, ByAriaLabel, ByDataTestId, ByPlaceholder) should be escaped using a helper method (such as EscapeCssAttributeValue) to prevent selector breakage due to special characters.

Learnt from: prashelke
PR: #4249
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs:7199-7202
Timestamp: 2025-07-10T07:12:52.786Z
Learning: In Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs, text values used in XPath expressions should be escaped using a helper method (such as EscapeXPathString) to handle single quotes and ensure valid XPath syntax.

Learnt from: GokulBothe99
PR: #3835
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightDriver.cs:1022-1029
Timestamp: 2024-07-18T09:05:15.264Z
Learning: The user prefers readability over concise code in the context of deactivating locators in the LearnElementInfoDetails method of the PlaywrightDriver class.

Learnt from: IamRanjeetSingh
PR: #3862
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightNonPersistentBrowser.cs:26-38
Timestamp: 2024-08-14T11:51:45.785Z
Learning: The use of Task.Run in the PlaywrightNonPersistentBrowser constructor is planned to be removed in future changes.

Learnt from: IamRanjeetSingh
PR: #3783
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightDriver.cs:129-129
Timestamp: 2024-06-24T08:39:59.351Z
Learning: User IamRanjeetSingh has indicated that the RunAction method in the PlaywrightDriver class should remain synchronous due to current constraints in the codebase.

Learnt from: AmanPrasad43
PR: #4034
File: Ginger/GingerCoreNET/ALMLib/Azure/AzureDevOpsCore.cs:242-270
Timestamp: 2024-12-12T09:43:09.791Z
Learning: Using .GetAwaiter().GetResult() instead of Task.Run(async () => { ... }).Wait(); in the AddAttachmentsToDefect method can cause deadlocks when executing through the CLI. Retaining Task.Run avoids this issue.

Learnt from: GokulBothe99
PR: #4072
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs:1075-1083
Timestamp: 2025-01-30T09:07:12.356Z
Learning: In Playwright-based element wait operations (like WaitForElementsVisibleAsync), do not short-circuit when elements are initially absent, as the method is designed to keep polling until timeout to handle dynamic element appearance.

Learnt from: GokulBothe99
PR: #4072
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs:1104-1143
Timestamp: 2025-01-31T13:25:31.910Z
Learning: In Playwright-based element wait operations (like WaitForElementsCheckedAsync), do not throw immediately when elements are initially absent, as the method is designed to keep polling until timeout to handle dynamic element appearance.

Learnt from: IamRanjeetSingh
PR: #3971
File: Ginger/GingerCoreNET/External/Katalon/Conversion/KatalonWebElementEntityToHTMLElementInfoConverter.cs:223-225
Timestamp: 2024-10-28T11:17:30.080Z
Learning: In the Ginger/GingerCoreNET/External/Katalon/Conversion/KatalonWebElementEntityToHTMLElementInfoConverter.cs file, when optional web element properties like id or name are not found during conversion, we should log warnings at eLogLevel.WARN level, as per the team's preference.

Learnt from: prashelke
PR: #3429
File: Ginger/Ginger/AutomatePageLib/AddActionMenu/WindowExplorer/Common/WindowExplorerPage.xaml.cs:1123-1126
Timestamp: 2024-07-26T22:04:12.930Z
Learning: The user has removed an unnecessary empty line at 1123 in WindowExplorerPage.xaml.cs for better code compactness.

Learnt from: GokulBothe99
PR: #4242
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs:623-656
Timestamp: 2025-06-24T06:08:31.804Z
Learning: In Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs, the CSS selector construction by directly interpolating user input values in the GetElementLocator method is considered acceptable by the team and working as expected for their use cases.

Learnt from: prashelke
PR: #4193
File: Ginger/Ginger/ApplicationModelsLib/POMModels/POMWizardLib/LearnWizard/AIGeneratedPreviewWizardPage.xaml.cs:131-138
Timestamp: 2025-04-30T13:21:34.994Z
Learning: In AIGeneratedPreviewWizardPage.xaml.cs, the current code checks if the response ends with triple backticks and removes them manually. This method is fragile and should be replaced with a more robust regex-based parser. The user plans to address this in a future update.

Learnt from: prashelke
PR: #4232
File: Ginger/GingerCoreNET/ActionsLib/UI/VisualTesting/VRTAnalyzer.cs:219-226
Timestamp: 2025-06-16T10:13:01.346Z
Learning: In VRTAnalyzer.cs for the GenericAppiumDriver class, the mAct.IsFullPageScreenshot flag is not applicable and should not be used to control screenshot behavior. GenericAppiumDriver always uses CaptureFullPageCroppedScreenshot() method regardless of the IsFullPageScreenshot flag setting.

Learnt from: prashelke
PR: #4222
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Mobile/Appium/GenericAppiumDriver.cs:2645-2656
Timestamp: 2025-06-19T05:08:45.192Z
Learning: In the Ginger mobile automation framework (GenericAppiumDriver.cs), XPath checks for "android" and "XCUIElement" should remain case-sensitive as currently implemented. The team prefers not to make these checks case-insensitive.

Learnt from: IamRanjeetSingh
PR: #3753
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs:99-99
Timestamp: 2024-06-12T12:54:44.221Z
Learning: User IamRanjeetSingh prefers exceptions to propagate rather than being caught and handled locally within methods in PlaywrightBrowserTab.cs.

Learnt from: GokulBothe99
PR: #4107
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserElement.cs:664-705
Timestamp: 2025-02-19T06:07:43.309Z
Learning: In the Ginger automation framework, exceptions in lower-level browser element operations are handled by their parent methods, which have more context about the operation being performed. Therefore, catch-and-rethrow without additional context is acceptable in the element-level methods.

Learnt from: IamRanjeetSingh
PR: #3738
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/GingerWebDriver.cs:39-53
Timestamp: 2024-07-26T22:04:12.930Z
Learning: The GetSupportedBrowserTypes method in GingerWebDriver.cs does not throw exceptions.

🧰 Additional context used

🧠 Learnings (3)

📓 Common learnings

Learnt from: prashelke
PR: Ginger-Automation/Ginger#3429
File: Ginger/Ginger/AutomatePageLib/AddActionMenu/WindowExplorer/Common/WindowExplorerPage.xaml.cs:1139-1142
Timestamp: 2024-07-26T22:04:12.930Z
Learning: The user has updated the code by removing or addressing the commented code as suggested in the review.

Learnt from: IamRanjeetSingh
PR: Ginger-Automation/Ginger#3862
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightNonPersistentBrowser.cs:26-38
Timestamp: 2024-08-14T11:51:45.785Z
Learning: The use of `Task.Run` in the `PlaywrightNonPersistentBrowser` constructor is planned to be removed in future changes.

Learnt from: prashelke
PR: Ginger-Automation/Ginger#4017
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs:11096-11096
Timestamp: 2024-12-04T11:45:57.024Z
Learning: In the `OnNetworkRequestSent` method in `SeleniumDriver.cs`, adding a null check for `_BrowserHelper` is not necessary.

Learnt from: GokulBothe99
PR: Ginger-Automation/Ginger#3835
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightDriver.cs:1022-1029
Timestamp: 2024-07-18T09:05:15.264Z
Learning: The user prefers readability over concise code in the context of deactivating locators in the `LearnElementInfoDetails` method of the `PlaywrightDriver` class.

Learnt from: GokulBothe99
PR: Ginger-Automation/Ginger#4095
File: Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Playwright/PlaywrightBrowserTab.cs:1330-1333
Timestamp: 2025-02-11T13:45:38.303Z
Learning: The `SwitchToShadowDomAsync` method in PlaywrightBrowserTab class is intentionally implemented as a placeholder that returns true, with the actual shadow DOM handling logic to be implemented in the future.