Releases/published official releases/official release 2025.4#4348
Merged
Maheshkale447 merged 146 commits intoOct 18, 2025
Merged
Conversation
Official Rlease branch update from master
…LogToCentralReport Send Execution Log to Central Report
…reenshot force cli screenshot bug fix for CLI JSON
License Updated
document refresh issue fix
D52125_D50125 Ginger Play Config and POM Related fix
D52125 Execution Logger Fix
…rFix Runner name added in Source Path
OCR related defect fix
…AppNameChange Target Ap Name change
…onFixForRemoteExecHistory old sol backward compatibility
…nRefresh Local and Remote execution defect fix
…nRefresh coder rabbit comments handled
Updated IOs screenshot fixed
…neFix Execution Log Related Changes
Contributor
|
Caution Review failedThe pull request is closed. WalkthroughThis PR introduces mobile device location capabilities (latitude/longitude/altitude), a new HTTP log appender for centralized log batching, extensive Ginger Play integration enhancements, UI restructuring across external configuration pages with two-column layouts, and miscellaneous improvements to execution history loading, element naming, and application propagation logic. Changes
Sequence Diagram(s)sequenceDiagram
actor CLI as CLI Process
participant GLog as GingerLog
participant HttpAppender as HttpLogAppender
participant Queue as Queue<LogEvent>
participant Batch as Batch Processor
participant API as AccountReportApiHandler
participant CentralDB as Central DB
CLI->>GLog: SetHTTPLogAppenderAPIUrl()
GLog->>HttpAppender: Configure ApiUrl
CLI->>HttpAppender: Append(LoggingEvent)
HttpAppender->>Queue: Enqueue LogEvent
Note over Batch: Background Task<br/>ProcessQueue()
Batch->>Queue: Dequeue batch (size=20)
Batch->>Batch: ProcessBatch()
Batch->>API: SendExecutionLogToCentralDBAsync()
API->>CentralDB: POST execution log
CentralDB-->>API: Response
opt Retry on Failure
API->>API: Exponential backoff
API->>CentralDB: Retry POST
end
CLI->>HttpAppender: Dispose()
HttpAppender->>Queue: Mark complete
Batch->>Queue: Process remaining
HttpAppender-->>CLI: Resources released
sequenceDiagram
participant User as User
participant Page as GingerPlayConfigurationpage
participant GPC as GingerPlayConfiguration
participant EndPoint as GingerPlayEndPointManager
participant ServiceAPI as External Service
User->>Page: Interact with config
Page->>GPC: IsGingerPlayGateWayConfigured()
GPC->>GPC: Check enabled + URL
alt Gateway Configured
GPC-->>Page: true
Page->>EndPoint: GetAccountReportServiceUrlByGateWay()
EndPoint->>EndPoint: Validate gateway URL
EndPoint-->>Page: Service URL
Page->>ServiceAPI: Connect to service
else Not Configured
GPC-->>Page: false
Page->>Page: Show unconfigured UI
end
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Reasoning: This PR spans multiple heterogeneous domains (mobile device features, new HTTP logging infrastructure, Ginger Play integration, UI layout restructuring, execution history async refactoring, CLI configuration, and POM/OCR improvements). While individual cohorts follow consistent patterns, the sheer breadth requires separate reasoning for major features: HttpLogAppender complexity with async queue/batch/retry logic, GingerPlayEndPointManager refactoring affecting multiple callers, AccountReportApiHandler enhancements, async GraphQL initialization changes, and UI layout restructuring across 6+ external configuration pages. Multiple package version updates and enum additions add integration risk. The variety and logic density demand careful cross-subsystem verification. Possibly related PRs
Suggested reviewers
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: ASSERTIVE Plan: Pro ⛔ Files ignored due to path filters (4)
📒 Files selected for processing (84)
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. Comment |
15 tasks
15 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Thank you for your contribution.
Before submitting this PR, please make sure:
Summary by CodeRabbit
Release Notes
New Features
Bug Fixes
UI/UX Improvements