Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

fix(updater): macOS in-place update can quit the app and leave a broken bundle

オープン
#1,587 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
52/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
dart
領域
desktop, release

調査の方向性

Start with buildMacAppReplaceScript in lib/core/updater/installers/update_install_utils.dart and MacosSparkleInstaller.install; trace how the script is launched and how failures are reported on the next start. Add tests for script quoting and failure paths, as the issue requests. Done means pre-flight checks, rollback and cleanup work, and failures are reported without leaving the app unavailable.

索引モデルが issue の本文から書いたものです。

説明

bug P1 stability

Problem

buildMacAppReplaceScript (lib/core/updater/installers/update_install_utils.dart) and MacosSparkleInstaller.install can quit the app without installing anything, or corrupt the bundle.

  • mv "$TARGET" "$TARGET.old" || true swallows failures. When the move fails (no write access to /Applications, read-only DMG volume, App Translocation after running from Downloads), cp -R "$NEW" "$TARGET" copies into the existing bundle: a nested .app, a broken seal, "damaged" from Gatekeeper.
  • With set -e, a failing cp after a successful mv aborts the script and leaves no app at $TARGET. There is no rollback from .old.
  • MacosSparkleInstaller calls exit(0) right after starting the script and never checks its result, so the user just sees the app close.
  • $TARGET.old is not removed on success; the extracted querya-update-* temp directory is never cleaned.

Expected

Check before exiting that the target's parent is writable and the bundle is not translocated / on a read-only volume; if not, tell the user (and offer the download). Replace atomically with rollback from .old, remove .old on success, clean the temp dir, and report a script failure on next start.

Tasks

  • Pre-flight writable / translocation / read-only check before exit(0).
  • Remove || true, add rollback, remove .old.
  • Clean the extract dir.
  • Unit tests for the script builder (quoting, failure paths).
主要言語
Dart
スター
23
フォーク
1
平均マージ
31分
マージ済み PR(30日)
570

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

QueryaHub/Querya-Desktop のほかの issue

QueryaHub/Querya-Desktop の issue をすべて見る

似ている issue

Dart の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。