Skip to content

Log Windows updating the FW Lite MSIX package - #18

Closed
myieye wants to merge 1 commit into
developfrom
claude/fwlite-log-os-package-updates
Closed

Log Windows updating the FW Lite MSIX package#18
myieye wants to merge 1 commit into
developfrom
claude/fwlite-log-os-package-updates

Conversation

@myieye

@myieye myieye commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Diagnostics only. Subscribes to PackageCatalog.PackageUpdating for our own package and logs target version, progress and errors.

I/Claude observed windows killing my app in order to update it. That was only after it tried a few times to update it first and then I guess it got impatient 😉.

I want to log this event, because it could potentially be a hook to let us do some cleanup before being ungracefully killed.

Windows stages an .appinstaller update on its own schedule and registers it at
activation, force-terminating a running instance to do so. None of that reaches
our log today, so a forced shutdown looks like an unexplained restart.

Subscribe to PackageCatalog.PackageUpdating for our own package and log target
version, progress and errors. Diagnostics only, no behaviour change.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto 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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 51f3c1ef-b806-4783-ac1e-0dbbd58644f9

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

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

@deepsource-io

deepsource-io Bot commented Aug 18, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in ba45997...767f640 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
C# Aug 18, 2026 12:33p.m. Review ↗
Docker Aug 18, 2026 12:33p.m. Review ↗
JavaScript Aug 18, 2026 12:33p.m. Review ↗
Shell Aug 18, 2026 12:33p.m. Review ↗
SQL Aug 18, 2026 12:33p.m. Review ↗
Secrets Aug 18, 2026 12:33p.m. Review ↗
PowerShell Aug 18, 2026 12:33p.m. Review ↗
CSS Aug 18, 2026 12:33p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@myieye myieye changed the title [claude] Log Windows updating the FW Lite MSIX package Log Windows updating the FW Lite MSIX package Aug 18, 2026
@myieye

myieye commented Aug 18, 2026

Copy link
Copy Markdown
Owner Author

Promoted to sillsdev#2576

@myieye myieye closed this Aug 18, 2026
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.

1 participant