Skip to content

feat: rich download after timeout event in ios - #367

Merged
balaganesh-juspay merged 2 commits into
mainfrom
feat-rich-dowload-after-timeout-event
Jul 15, 2026
Merged

balaganesh-juspay merged 2 commits into
mainfrom
feat-rich-dowload-after-timeout-event

Conversation

@yuvrajjsingh0

@yuvrajjsingh0 yuvrajjsingh0 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Bug Fixes
    • Improved logger event tracking to prevent relevant events from being silently omitted.
    • Enhanced timeout event details with the newly downloaded and currently installed package versions.
    • Clarified that staged packages are applied during the next initialization rather than the current boot.

@semanticdiff-com

semanticdiff-com Bot commented Jul 10, 2026

Copy link
Copy Markdown

@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown

Review Change Stack

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: b001423d-94d8-40f8-806c-9f7751d558c4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

The PR updates Swift package imports for tracker compilation and adds downloaded and installed package version metadata to timeout completion tracking events.

Changes

Tracker observability updates

Layer / File(s) Summary
Swift tracker import ordering
airborne_sdk_iOS/hyper-ota/Airborne/AirborneObjC/ApplicationManager/Tracker/AJPApplicationTracker.m
The Swift package import path adds AirborneSwiftCore and imports Swift modules before AJPApplicationTracker.h.
Timeout tracking metadata
airborne_sdk_iOS/hyper-ota/Airborne/AirborneSwift/AJPApplicationManager.swift
Timeout completion events now include downloaded and currently installed package versions, with staging behavior documented.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested reviewers: balaganesh-juspay, JamesGeorg

Poem

I’m a rabbit tracking packages tonight,
With versions recorded neat and bright.
Swift modules hop in before the header,
Timeout logs grow more precise together.
Staged for tomorrow, not this boot—
I twitch my nose and call it cute!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is clearly related to the timeout-event logging changes and the iOS-specific tracker fix, though it is a bit broad.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat-rich-dowload-after-timeout-event

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

Comment @coderabbitai help to get the list of available commands.

@balaganesh-juspay
balaganesh-juspay merged commit 89dea2c into main Jul 15, 2026
12 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants