Merge master to official release

[rathimayur]

Description

Type of Change

• Bug fix - [ ] New feature - [ ] Breaking change - [ ] Plugin update

Checklist

• PR description clearly describes the changes
• Target branch is correct (master for features, Releases/* for fixes)
• Latest code from target branch merged
• No commented/junk code included
• No new build warnings or errors
• All existing unit tests pass
• New unit tests added for new functionality
• Cross-platform compatibility verified (Windows/Linux/macOS)
• CI/CD pipeline passes
• Code follows project conventions (Act{Platform}{Type}, {Platform}Driver)
• Repository objects use [IsSerializedForLocalRepository] where needed
• Error handling uses Reporter.ToLog() pattern
• Documentation updated for user-facing changes
• Self-review completed and code review comments addressed

Summary by CodeRabbit

• New Features
  • Added expression editing dialogs for table column headers and “Where” clause filters.
• Bug Fixes
  • Improved PostgreSQL connection string building for more reliable connectivity.
  • Updated table edit layout and control arrangement for clearer column/row/where input.
  • Ensured all running plugin processes are closed after standalone and runset execution.
• Chores
  • Updated Magick.NET to 14.13.1 and bumped GingerCoreCommon version metadata.

[coderabbitai]

Walkthrough

Adds ValueExpressionEditorPage support to ActTableEditPage via new XAML docked VE buttons and corresponding click handlers. Calls CloseAllRunningPluginProcesses() at the end of standalone and runset executions. Fixes PostgreSQL connection-string building to branch on ConnectionStringCalculated. Bumps Magick.NET-Q16-AnyCPU to 14.13.1 across three project files, updates GingerCoreCommon assembly versioning to 26.6.0.0, and applies whitespace fixes to UFTCredentialsDialog.xaml.

Changes

ActTableEditPage Value Expression Editor Support

Layer / File(s)	Summary
Column/Row/Where XAML layout with VE buttons Ginger/Ginger/Actions/ActionEditPages/ActTableEditPage.xaml	Grid column widths updated; Column, Row, and Where value areas restructured into DockPanel layouts with right-docked VE buttons (ColumnValueVE, RowSelectorValueVE, WhereColumnTitleVE) alongside existing editable controls.
VE click handlers in code-behind Ginger/Ginger/Actions/ActionEditPages/ActTableEditPage.xaml.cs	Two new click handlers open ValueExpressionEditorPage for LocateColTitle and WhereColumnTitle, update cmbColumnValue/WhereColumnTitle from mAct, call SetDescriptionDetails(), and call UpdateRelatedActions() when in WindowExplorer mode.

Plugin Process Cleanup on Execution End

Layer / File(s)	Summary
Plugin shutdown in execution engines Ginger/GingerCoreNET/Run/GingerExecutionEngine.cs, Ginger/GingerCoreNET/Run/RunsetExecutor.cs	CloseAllRunningPluginProcesses() called inside the standaloneExecution block in RunBusinessFlow and after CloseAllEnvironments() in RunRunset.

PostgreSQL Conditional Connection String Fix

Layer / File(s)	Summary
Conditional NpgsqlConnectionStringBuilder construction Ginger/GingerCoreNET/Database/DatabaseOperations.cs	Connect now branches: when ConnectionStringCalculated is empty, parses TNSCalculated for host/port with decrypted password; otherwise sets the builder's ConnectionString from GetConnectionString().

Magick.NET Bump, Version Metadata, and Whitespace Cleanup

Layer / File(s)	Summary
Magick.NET 14.13.1 version bump Ginger/Ginger/Ginger.csproj, Ginger/GingerCoreNET/GingerCoreNET.csproj, Ginger/GingerTest/GingerTest.csproj	Magick.NET-Q16-AnyCPU PackageReference bumped from 14.13.0 to 14.13.1 across all three project files.
GingerCoreCommon assembly and package version bump Ginger/GingerCoreCommon/GingerCoreCommon.csproj	AssemblyVersion and FileVersion bumped from 26.5.0.0 to 26.6.0.0; package Version updated from 2026.5.0.0 to 2026.6.0.0.
UFTCredentialsDialog whitespace fixes Ginger/Ginger/Drivers/DriversConfigsEditPages/UFTCredentialsDialog.xaml	Indentation-only adjustments to closing </StackPanel>, </Border>, and </ScrollViewer> tags in the Apps tab layout.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Possibly related PRs

• Ginger-Automation/Ginger#4498: Directly overlaps with this PR's ActTableEditPage.xaml/.xaml.cs restructuring and addition of ColumnValueVE_Click and WhereColumnTitleVE_Click handlers.
• Ginger-Automation/Ginger#4494: Modifies the same CloseAllRunningPluginProcesses() usage in GingerExecutionEngine.RunBusinessFlow and RunsetExecutor.RunRunset.
• Ginger-Automation/Ginger#4500: Directly overlaps with the conditional NpgsqlConnectionStringBuilder construction change in DatabaseOperations.Connect.

Suggested reviewers

• Maheshkale447
• ravirk91

🐇 A bunny hops through the code so neat,
With VE buttons added — what a treat!
Plugins now close when the run is done,
Postgres connects with a smarter run. 🎉
Magick bumped up, versions aligned right,
Everything tidied on this fine night! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description contains only the template checklist with no substantive details about the specific changes being merged or which checklist items have been completed.	Provide a detailed description of the changes being merged, explain the purpose of this merge, mark completed checklist items, and clarify the specific enhancements and fixes included.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Merge master to official release' accurately describes the primary purpose of this merge pull request, clearly indicating the consolidation of changes from master into the official release branch.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch Merge-Master-to-Official-Release

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codacy-production]

Not up to standards ⛔

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.}

[coderabbitai]

Actionable comments posted: 3

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@Ginger/Ginger/Actions/ActionEditPages/ActTableEditPage.xaml`:
- Line 78: The Description DockPanel element in ActTableEditPage.xaml has
Grid.Row set to 1, but the Grid only defines a single RowDefinition with Height
70, meaning only row index 0 exists. Change the Grid.Row attribute on the
Description DockPanel from 1 to 0 to correctly place it in the only available
row of the Grid.

In `@Ginger/GingerCoreNET/Run/GingerExecutionEngine.cs`:
- Around line 4518-4520: Wrap the CloseAllRunningPluginProcesses() call in a
try-catch block to prevent exceptions from blocking the state finalization code.
In the catch block, log the error using Reporter.ToLog(eLogLevel.ERROR, message)
with details about the plugin shutdown failure. Ensure that the IsRunning =
false and Status = RunsetStatus assignments either execute after the try-catch
block or are placed in a finally block so they always execute regardless of
whether plugin shutdown succeeds or throws an exception.

In `@Ginger/GingerCoreNET/Run/RunsetExecutor.cs`:
- Line 733: The CloseAllRunningPluginProcesses call is currently only executed
on the success path, meaning if RunRunset throws an exception before reaching
line 733, plugin processes will not be cleaned up. Wrap the RunRunset invocation
in a try-finally block, keeping the CloseAllRunningPluginProcesses call in the
finally block to guarantee it executes regardless of whether RunRunset succeeds
or throws an exception, ensuring plugins are always properly shut down.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: e891e1e9-785c-47a1-aead-e8e532f4f28a

📥 Commits

Reviewing files that changed from the base of the PR and between 9d673bb and 4f98abe.

⛔ Files ignored due to path filters (1)

• .github/workflows/Test.yml is excluded by !**/*.yml

📒 Files selected for processing (9)

• Ginger/Ginger/Actions/ActionEditPages/ActTableEditPage.xaml
• Ginger/Ginger/Actions/ActionEditPages/ActTableEditPage.xaml.cs
• Ginger/Ginger/Drivers/DriversConfigsEditPages/UFTCredentialsDialog.xaml
• Ginger/Ginger/Ginger.csproj
• Ginger/GingerCoreNET/Database/DatabaseOperations.cs
• Ginger/GingerCoreNET/GingerCoreNET.csproj
• Ginger/GingerCoreNET/Run/GingerExecutionEngine.cs
• Ginger/GingerCoreNET/Run/RunsetExecutor.cs
• Ginger/GingerTest/GingerTest.csproj

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@Ginger/GingerCoreCommon/GingerCoreCommon.csproj`:
- Line 11: The TargetFramework element in the GingerCoreCommon.csproj project
file is set to net10.0, but coding guidelines require .NET 8.0 with Windows
10.0.17763.0 compatibility. Update the TargetFramework element from net10.0 to
net8.0 and add a Windows compatibility target configuration (such as a
SupportedOSPlatformVersion or RuntimeIdentifier property) to specify Windows
10.0.17763.0 compatibility. Apply the same changes to all other occurrences
mentioned in the file.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9215f477-3097-4f4e-8f6e-673516df5599

📥 Commits

Reviewing files that changed from the base of the PR and between 4f98abe and f889186.

📒 Files selected for processing (1)

• Ginger/GingerCoreCommon/GingerCoreCommon.csproj