docs: add further disposition description

CLAUDE.md

@@ -192,7 +192,6 @@ gh label list --repo electron/electron --search target/ --json name,color --jq '
 ```bash
 npm run lint              # Run all linters
 npm run lint:clang-format # C++ formatting
-npm run lint:api-history  # Validate API history YAML blocks in docs
 ```
 
 ## Key Files

build/args/all.gn

@@ -2,7 +2,7 @@ is_electron_build = true
 root_extra_deps = [ "//electron" ]
 
 # Registry of NMVs --> https://github.com/nodejs/node/blob/main/doc/abi_version_registry.json
-node_module_version = 146
+node_module_version = 145
 
 v8_promise_internal_field_count = 1
 v8_embedder_string = "-electron.0"

docs/CLAUDE.md

@@ -1,58 +0,0 @@
-# Electron Documentation Guide
-
-## API History Migration
-
-Guide: `docs/development/api-history-migration-guide.md`
-Style rules: `docs/development/style-guide.md` (see "API History" section)
-Schema: `docs/api-history.schema.json`
-Lint: `npm run lint:api-history`
-
-### Format
-
-Place YAML history block directly after the Markdown header, before parameters:
-
-````md
-### `module.method(args)`
-
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/XXXXX
-```
--->
-
-* `arg` type - Description.
-````
-
-### Finding when an API was added
-
-- `git log --all --reverse --oneline -S "methodName" -- docs/api/file.md` — find first commit adding a method name
-- `git log --reverse -L :FunctionName:path/to/source.cc` — trace C++ implementation history
-- `git log --grep="keyword" --oneline` — find merge commits referencing PRs
-- `gh pr view <number> --repo electron/electron --json baseRefName` — verify PR targets main (not a backport)
-- Always use the main-branch PR URL in history blocks, not backport PRs
-
-### Cross-referencing breaking changes
-
-- Search `docs/breaking-changes.md` for the API name to find deprecations/removals
-- Use `git blame` on the breaking-changes entry to find the associated PR
-- Add `breaking-changes-header` field using the heading ID from breaking-changes.md
-
-### Placement rules
-
-- Only add blocks to actual API entries (methods, events, properties with backtick signatures)
-- Do NOT add blocks to section headers like `## Methods`, `### Instance Methods`, `## Events`
-- Module-level blocks go after the `# moduleName` heading, before the module description quote
-- For changes affecting multiple APIs, add a block under each affected top-level heading (see style guide "Change affecting multiple APIs")
-
-### Key details
-
-- `added` and `deprecated` arrays have `maxItems: 1`; `changes` can have multiple items
-- `changes` entries require a `description` field; `added`/`deprecated` do not
-- Wrap descriptions in double quotes to avoid YAML parsing issues with special characters
-- Early Electron APIs (pre-2015) use merge-commit PRs (e.g., `Merge pull request #534`)
-- Very early APIs (2013-2014, e.g., `ipcMain.on`, `ipcRenderer.send`) predate GitHub PRs — skip history blocks for these
-- When multiple APIs were added in the same PR, they all reference the same PR URL
-- Promisification PRs (e.g., #17355) count as `changes` entries with a description
-  - These PRs are breaking changes and should have their notes as "This method now returns a Promise instead of using a callback function."
-- APIs that were deprecated and then removed from docs don't need history blocks (the removal is in `breaking-changes.md`)

docs/api/dialog.md

@@ -28,10 +28,7 @@ added:
 * `window` [BaseWindow](base-window.md) (optional)
 * `options` Object
   * `title` string (optional)
-  * `defaultPath` string (optional) - Absolute directory path, absolute file
-    path, or file name to use by default. If not provided, the dialog will
-    default to the user's Downloads folder, or their home directory if Downloads
-    doesn't exist.
+  * `defaultPath` string (optional)
   * `buttonLabel` string (optional) - Custom label for the confirmation button, when
     left empty the default label will be used.
   * `filters` [FileFilter[]](structures/file-filter.md) (optional)
@@ -112,10 +109,7 @@ changes:
 * `window` [BaseWindow](base-window.md) (optional)
 * `options` Object
   * `title` string (optional)
-  * `defaultPath` string (optional) - Absolute directory path, absolute file
-    path, or file name to use by default. If not provided, the dialog will
-    default to the user's Downloads folder, or their home directory if Downloads
-    doesn't exist.
+  * `defaultPath` string (optional)
   * `buttonLabel` string (optional) - Custom label for the confirmation button, when
     left empty the default label will be used.
   * `filters` [FileFilter[]](structures/file-filter.md) (optional)
@@ -204,9 +198,7 @@ added:
 * `options` Object
   * `title` string (optional) - The dialog title. Cannot be displayed on some _Linux_ desktop environments.
   * `defaultPath` string (optional) - Absolute directory path, absolute file
-    path, or file name to use by default. If not provided, the dialog will
-    default to the user's Downloads folder, or their home directory if Downloads
-    doesn't exist.
+    path, or file name to use by default.
   * `buttonLabel` string (optional) - Custom label for the confirmation button, when
     left empty the default label will be used.
   * `filters` [FileFilter[]](structures/file-filter.md) (optional)
@@ -246,9 +238,7 @@ changes:
 * `options` Object
   * `title` string (optional) - The dialog title. Cannot be displayed on some _Linux_ desktop environments.
   * `defaultPath` string (optional) - Absolute directory path, absolute file
-    path, or file name to use by default. If not provided, the dialog will
-    default to the user's Downloads folder, or their home directory if Downloads
-    doesn't exist.
+    path, or file name to use by default.
   * `buttonLabel` string (optional) - Custom label for the confirmation button, when
     left empty the default label will be used.
   * `filters` [FileFilter[]](structures/file-filter.md) (optional)

docs/api/global-shortcut.md

@@ -55,13 +55,6 @@ The `globalShortcut` module has the following methods:
 
 ### `globalShortcut.register(accelerator, callback)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/534
-```
--->
-
 * `accelerator` string - An [accelerator](../tutorial/keyboard-shortcuts.md#accelerators) shortcut.
 * `callback` Function
 
@@ -84,13 +77,6 @@ the app has been authorized as a [trusted accessibility client](https://develope
 
 ### `globalShortcut.registerAll(accelerators, callback)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/15542
-```
--->
-
 * `accelerators` string[] - An array of [accelerator](../tutorial/keyboard-shortcuts.md#accelerators) shortcuts.
 * `callback` Function
 
@@ -110,13 +96,6 @@ the app has been authorized as a [trusted accessibility client](https://develope
 
 ### `globalShortcut.isRegistered(accelerator)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/534
-```
--->
-
 * `accelerator` string - An [accelerator](../tutorial/keyboard-shortcuts.md#accelerators) shortcut.
 
 Returns `boolean` - Whether this application has registered `accelerator`.
@@ -127,24 +106,10 @@ don't want applications to fight for global shortcuts.
 
 ### `globalShortcut.unregister(accelerator)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/534
-```
--->
-
 * `accelerator` string - An [accelerator](../tutorial/keyboard-shortcuts.md#accelerators) shortcut.
 
 Unregisters the global shortcut of `accelerator`.
 
 ### `globalShortcut.unregisterAll()`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/534
-```
--->
-
 Unregisters all of the global shortcuts.

docs/api/image-view.md

@@ -35,13 +35,6 @@ webContentsView.webContents.loadURL('https://electronjs.org')
 
 ## Class: ImageView extends `View`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/46760
-```
--->
-
 > A View that displays an image.
 
 Process: [Main](../glossary.md#main-process)
@@ -56,13 +49,6 @@ Process: [Main](../glossary.md#main-process)
 
 ### `new ImageView()` _Experimental_
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/46760
-```
--->
-
 Creates an ImageView.
 
 ### Instance Methods
@@ -72,13 +58,6 @@ addition to those inherited from [View](view.md):
 
 #### `image.setImage(image)` _Experimental_
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/46760
-```
--->
-
 * `image` NativeImage
 
 Sets the image for this `ImageView`. Note that only image formats supported by

docs/api/in-app-purchase.md

@@ -1,12 +1,5 @@
 # inAppPurchase
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/11292
-```
--->
-
 > In-app purchases on Mac App Store.
 
 Process: [Main](../glossary.md#main-process)
@@ -17,13 +10,6 @@ The `inAppPurchase` module emits the following events:
 
 ### Event: 'transactions-updated'
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/11292
-```
--->
-
 Returns:
 
 * `event` Event
@@ -37,19 +23,6 @@ The `inAppPurchase` module has the following methods:
 
 ### `inAppPurchase.purchaseProduct(productID[, opts])`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/11292
-changes:
-  - pr-url: https://github.com/electron/electron/pull/17355
-    description: "This method now returns a Promise instead of using a callback function."
-    breaking-changes-header: api-changed-callback-based-versions-of-promisified-apis
-  - pr-url: https://github.com/electron/electron/pull/35902
-    description: "Added `username` option to `opts` parameter."
-```
--->
-
 * `productID` string
 * `opts` Integer | Object (optional) - If specified as an integer, defines the quantity.
   * `quantity` Integer (optional) - The number of items the user wants to purchase.
@@ -61,17 +34,6 @@ You should listen for the `transactions-updated` event as soon as possible and c
 
 ### `inAppPurchase.getProducts(productIDs)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/12464
-changes:
-  - pr-url: https://github.com/electron/electron/pull/17355
-    description: "This method now returns a Promise instead of using a callback function."
-    breaking-changes-header: api-changed-callback-based-versions-of-promisified-apis
-```
--->
-
 * `productIDs` string[] - The identifiers of the products to get.
 
 Returns `Promise<Product[]>` - Resolves with an array of [`Product`](structures/product.md) objects.
@@ -80,59 +42,24 @@ Retrieves the product descriptions.
 
 ### `inAppPurchase.canMakePayments()`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/11292
-```
--->
-
 Returns `boolean` - whether a user can make a payment.
 
 ### `inAppPurchase.restoreCompletedTransactions()`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/21461
-```
--->
-
 Restores finished transactions. This method can be called either to install purchases on additional devices, or to restore purchases for an application that the user deleted and reinstalled.
 
 [The payment queue](https://developer.apple.com/documentation/storekit/skpaymentqueue?language=objc) delivers a new transaction for each previously completed transaction that can be restored. Each transaction includes a copy of the original transaction.
 
 ### `inAppPurchase.getReceiptURL()`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/11292
-```
--->
-
 Returns `string` - the path to the receipt.
 
 ### `inAppPurchase.finishAllTransactions()`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/12464
-```
--->
-
 Completes all pending transactions.
 
 ### `inAppPurchase.finishTransactionByDate(date)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/12464
-```
--->
-
 * `date` string - The ISO formatted date of the transaction to finish.
 
 Completes the pending transactions corresponding to the date.

docs/api/ipc-main.md

@@ -46,13 +46,6 @@ Listens to `channel`, when a new message arrives `listener` would be called with
 
 ### `ipcMain.off(channel, listener)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/44651
-```
--->
-
 * `channel` string
 * `listener` Function
   * `event` [IpcMainEvent][ipc-main-event]
@@ -96,13 +89,6 @@ Removes all listeners from the specified `channel`. Removes all listeners from a
 
 ### `ipcMain.handle(channel, listener)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/18449
-```
--->
-
 * `channel` string
 * `listener` Function\<Promise\<any\> | any\>
   * `event` [IpcMainInvokeEvent][ipc-main-invoke-event]
@@ -140,13 +126,6 @@ provided to the renderer process. Please refer to
 
 ### `ipcMain.handleOnce(channel, listener)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/18449
-```
--->
-
 * `channel` string
 * `listener` Function\<Promise\<any\> | any\>
   * `event` [IpcMainInvokeEvent][ipc-main-invoke-event]
@@ -157,13 +136,6 @@ Handles a single `invoke`able IPC message, then removes the listener. See
 
 ### `ipcMain.removeHandler(channel)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/18449
-```
--->
-
 * `channel` string
 
 Removes any handler for `channel`, if present.

docs/api/ipc-renderer.md

@@ -59,13 +59,6 @@ for more info.
 
 ### `ipcRenderer.off(channel, listener)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/39816
-```
--->
-
 * `channel` string
 * `listener` Function
   * `event` [IpcRendererEvent][ipc-renderer-event]
@@ -86,13 +79,6 @@ only the next time a message is sent to `channel`, after which it is removed.
 
 ### `ipcRenderer.addListener(channel, listener)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/39816
-```
--->
-
 * `channel` string
 * `listener` Function
   * `event` [IpcRendererEvent][ipc-renderer-event]
@@ -143,13 +129,6 @@ If you want to receive a single response from the main process, like the result
 
 ### `ipcRenderer.invoke(channel, ...args)`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/18449
-```
--->
-
 * `channel` string
 * `...args` any[]
 
@@ -230,13 +209,6 @@ and replies by setting `event.returnValue`.
 
 ### `ipcRenderer.postMessage(channel, message, [transfer])`
 
-<!--
-```YAML history
-added:
-  - pr-url: https://github.com/electron/electron/pull/22404
-```
--->
-
 * `channel` string
 * `message` any
 * `transfer` MessagePort[] (optional)

docs/api/web-contents.md

@@ -226,7 +226,8 @@ Returns:
     Only defined when the window is being created by a form that set
     `target=_blank`.
   * `disposition` string - Can be `default`, `foreground-tab`,
-    `background-tab`, `new-window` or `other`.
+    `background-tab`, `new-window` or `other`. Corresponds to the manner an associated link was clicked. See Chromium's
+    [WindowOpenDisposition](https://source.chromium.org/chromium/chromium/src/+/main:ui/base/window_open_disposition.h).
 
 Emitted _after_ successful creation of a window via `window.open` in the renderer.
 Not emitted if the creation of the window is canceled from
@@ -1450,7 +1451,8 @@ Ignore application menu shortcuts while this web contents is focused.
     * `frameName` string - Name of the window provided in `window.open()`
     * `features` string - Comma separated list of window features provided to `window.open()`.
     * `disposition` string - Can be `default`, `foreground-tab`, `background-tab`,
-      `new-window` or `other`.
+      `new-window` or `other`. Corresponds to the manner an associated link was clicked. See Chromium's
+      [WindowOpenDisposition](https://source.chromium.org/chromium/chromium/src/+/main:ui/base/window_open_disposition.h).
     * `referrer` [Referrer](structures/referrer.md) - The referrer that will be
       passed to the new window. May or may not result in the `Referer` header being
       sent, depending on the referrer policy.

docs/breaking-changes.md

@@ -12,34 +12,6 @@ This document uses the following convention to categorize breaking changes:
 * **Deprecated:** An API was marked as deprecated. The API will continue to function, but will emit a deprecation warning, and will be removed in a future release.
 * **Removed:** An API or feature was removed, and is no longer supported by Electron.
 
-## Planned Breaking API Changes (43.0)
-
-### Behavior Changed: Dialog methods default to Downloads directory
-
-The `defaultPath` option for the following methods now defaults to the user's Downloads folder (or their home directory if Downloads doesn't exist) when not explicitly provided:
-
-* `dialog.showOpenDialog`
-* `dialog.showOpenDialogSync`
-* `dialog.showSaveDialog`
-* `dialog.showSaveDialogSync`
-
-Previously, when no `defaultPath` was provided, the underlying OS file dialog would determine the initial directory — typically remembering the last directory the user navigated to, or falling back to an OS-specific default. Now, Electron explicitly sets the initial directory to Downloads, which also means the OS will no longer track and restore the last-used directory between dialog invocations.
-
-To preserve the old behavior, you can track the last-used directory yourself and pass it as `defaultPath`:
-
-```js
-const path = require('node:path')
-
-let lastUsedPath
-const result = await dialog.showOpenDialog({
-  defaultPath: lastUsedPath
-})
-
-if (!result.canceled && result.filePaths.length > 0) {
-  lastUsedPath = path.dirname(result.filePaths[0])
-}
-```
-
 ## Planned Breaking API Changes (42.0)
 
 ### Behavior Changed: macOS notifications now use `UNNotification` API
@@ -87,17 +59,6 @@ npm install electron --save-dev --ignore-scripts
 npx install-electron --no
 ```
 
-If you need to test changes across platforms or architectures, you should now use the
-`ELECTRON_INSTALL_ARCH` and `ELECTRON_INSTALL_PLATFORM` environment variables.
-
-```sh
-# before: pass npm config flag on install command
-npm install --platform=mas electron --save-dev
-# after: add env var when you first run the Electron command
-npm install electron --save-dev
-ELECTRON_INSTALL_PLATFORM=mas npx electron . --no
-```
-
 ### Removed: `quotas` object from `Session.clearStorageData(options)`
 
 When calling `Session.clearStorageData(options)`, the `options.quotas` object is no longer supported because it has been

docs/tutorial/installation.md

@@ -51,40 +51,20 @@ any dependencies in your app will not be installed.
 ## Customization
 
 If you want to change the architecture that is downloaded (e.g., `x64` on an
-`arm64` machine), you can set the `ELECTRON_INSTALL_ARCH` environment variable:
+`arm64` machine), you can use the `--arch` flag with npm install or set the
+`npm_config_arch` environment variable:
 
-```sh
-# Inside an npm script or with npx
-ELECTRON_INSTALL_ARCH=x64 electron .
+```shell
+npm install --arch=x64 electron
 ```
 
-Supported architectures are a subset of Node.js [`process.arch`](https://nodejs.org/api/process.html#processarch)
-values, and include:
-
-* `x64` (Intel Mac and 64-bit Windows)
-* `ia32` (32-bit Windows)
-* `arm64` (Apple silicon, Windows on ARM, ARM64 Linux)
-* `arm` (32-bit ARM)
-
 In addition to changing the architecture, you can also specify the platform
 (e.g., `win32`, `linux`, etc.) using the `--platform` flag:
 
-```sh
-# Inside an npm script or with npx
-ELECTRON_INSTALL_PLATFORM=mas electron .
+```shell
+npm install --platform=win32 electron
 ```
 
-Supported platforms are Node-like [platform strings](https://nodejs.org/api/process.html#processplatform):
-
-* `darwin`
-* `mas` ([Mac App Store](./mac-app-store-submission-guide.md))
-* `win32`
-* `linux`
-
-> [!TIP]
-> To see all available platform/architecture combinations for a particular release, see the artifacts
-> on [Electron's GitHub Releases](https://github.com/electron/electron/releases).
-
 ## Proxies
 
 If you need to use an HTTP proxy, you need to set the `ELECTRON_GET_USE_PROXY` variable to any

filenames.auto.gni

@@ -78,7 +78,6 @@ auto_filenames = {
     "docs/api/web-utils.md",
     "docs/api/webview-tag.md",
     "docs/api/window-open.md",
-    "docs/api/structures/activation-arguments.md",
     "docs/api/structures/base-window-options.md",
     "docs/api/structures/bluetooth-device.md",
     "docs/api/structures/browser-window-options.md",

npm/install.js

@@ -21,11 +21,8 @@ if (isInstalled()) {
   process.exit(0);
 }
 
-const platform = process.env.ELECTRON_INSTALL_PLATFORM || process.env.npm_config_platform || process.platform;
-let arch =
-  process.env.ELECTRON_INSTALL_ARCH ||
-  process.env.npm_config_arch ||
-  process.arch;
+const platform = process.env.npm_config_platform || process.platform;
+let arch = process.env.npm_config_arch || process.arch;
 
 if (platform === 'darwin' && process.platform === 'darwin' && arch === 'x64' &&
     process.env.npm_config_arch === undefined) {

patches/squirrel.mac/.patches

@@ -9,5 +9,5 @@ refactor_use_non-deprecated_nskeyedarchiver_apis.patch
 chore_turn_off_launchapplicationaturl_deprecation_errors_in_squirrel.patch
 fix_crash_when_process_to_extract_zip_cannot_be_launched.patch
 use_uttype_class_instead_of_deprecated_uttypeconformsto.patch
-fix_clean_up_orphaned_staged_updates_before_downloading_new_update.patch
+fix_clean_up_old_staged_updates_before_downloading_new_update.patch
 fix_add_explicit_json_property_mappings_for_shipit_request_model.patch

patches/squirrel.mac/fix_clean_up_old_staged_updates_before_downloading_new_update.patch

@@ -0,0 +1,64 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Andy Locascio <loc@anthropic.com>
+Date: Tue, 6 Jan 2026 08:23:03 -0800
+Subject: fix: clean up old staged updates before downloading new update
+
+When checkForUpdates() is called while an update is already staged,
+Squirrel creates a new temporary directory for the download without
+cleaning up the old one. This can lead to significant disk usage if
+the app keeps checking for updates without restarting.
+
+This change adds a force parameter to pruneUpdateDirectories that
+bypasses the AwaitingRelaunch state check. This is called before
+creating a new temp directory, ensuring old staged updates are
+cleaned up when a new download starts.
+
+diff --git a/Squirrel/SQRLUpdater.m b/Squirrel/SQRLUpdater.m
+index d156616e81e6f25a3bded30e6216b8fc311f31bc..6cd4346bf43b191147aff819cb93387e71275a46 100644
+--- a/Squirrel/SQRLUpdater.m
++++ b/Squirrel/SQRLUpdater.m
+@@ -543,11 +543,17 @@ - (RACSignal *)downloadBundleForUpdate:(SQRLUpdate *)update intoDirectory:(NSURL
+ #pragma mark File Management
+ 
+ - (RACSignal *)uniqueTemporaryDirectoryForUpdate {
+-	return [[[RACSignal
++	// Clean up any old staged update directories before creating a new one.
++	// This prevents disk usage from growing when checkForUpdates() is called
++	// multiple times without the app restarting.
++	return [[[[[self
++		pruneUpdateDirectoriesWithForce:YES]
++		ignoreValues]
++		concat:[RACSignal
+ 		defer:^{
+ 			SQRLDirectoryManager *directoryManager = [[SQRLDirectoryManager alloc] initWithApplicationIdentifier:SQRLShipItLauncher.shipItJobLabel];
+ 			return [directoryManager storageURL];
+-		}]
++		}]]
+ 		flattenMap:^(NSURL *storageURL) {
+ 			NSURL *updateDirectoryTemplate = [storageURL URLByAppendingPathComponent:[SQRLUpdaterUniqueTemporaryDirectoryPrefix stringByAppendingString:@"XXXXXXX"]];
+ 			char *updateDirectoryCString = strdup(updateDirectoryTemplate.path.fileSystemRepresentation);
+@@ -643,7 +649,7 @@ - (BOOL)isRunningOnReadOnlyVolume {
+ 
+ - (RACSignal *)performHousekeeping {
+ 	return [[RACSignal
+-		merge:@[ [self pruneUpdateDirectories], [self truncateLogs] ]]
++		merge:@[ [self pruneUpdateDirectoriesWithForce:NO], [self truncateLogs] ]]
+ 		catch:^(NSError *error) {
+ 			NSLog(@"Error doing housekeeping: %@", error);
+ 			return [RACSignal empty];
+@@ -658,11 +664,12 @@ - (RACSignal *)performHousekeeping {
+ ///
+ /// Sends each removed directory then completes, or errors, on an unspecified
+ /// thread.
+-- (RACSignal *)pruneUpdateDirectories {
++- (RACSignal *)pruneUpdateDirectoriesWithForce:(BOOL)force {
+ 	return [[[RACSignal
+ 		defer:^{
+-			// If we already have updates downloaded we don't wanna prune them.
+-			if (self.state == SQRLUpdaterStateAwaitingRelaunch) return [RACSignal empty];
++			// If we already have updates downloaded we don't wanna prune them,
++			// unless force is YES (used when starting a new download).
++			if (!force && self.state == SQRLUpdaterStateAwaitingRelaunch) return [RACSignal empty];
+ 
+ 			SQRLDirectoryManager *directoryManager = [[SQRLDirectoryManager alloc] initWithApplicationIdentifier:SQRLShipItLauncher.shipItJobLabel];
+ 			return [directoryManager storageURL];

patches/squirrel.mac/fix_clean_up_orphaned_staged_updates_before_downloading_new_update.patch

@@ -1,130 +0,0 @@
-From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
-From: Andy Locascio <loc@anthropic.com>
-Date: Tue, 6 Jan 2026 08:23:03 -0800
-Subject: fix: clean up orphaned staged updates before downloading new update
-
-When checkForUpdates() is called while an update is already staged,
-Squirrel creates a new temporary directory for the download without
-cleaning up the old one. This can lead to significant disk usage if
-the app keeps checking for updates without restarting.
-
-This change adds a pruneOrphanedUpdateDirectories step before creating
-a new temp directory. Unlike a blanket prune, this reads the current
-ShipItState.plist and preserves the directory it references, deleting
-only truly orphaned update directories. This keeps the on-disk
-footprint bounded (at most 2 dirs) while ensuring quitAndInstall
-remains safe to call even when a new check is in progress.
-
-Refs https://github.com/electron/electron/issues/50200
-
-diff --git a/Squirrel/SQRLUpdater.m b/Squirrel/SQRLUpdater.m
-index d156616e81e6f25a3bded30e6216b8fc311f31bc..41856e5754228d33982db72f97f2ff241615a357 100644
---- a/Squirrel/SQRLUpdater.m
-+++ b/Squirrel/SQRLUpdater.m
-@@ -543,11 +543,19 @@ - (RACSignal *)downloadBundleForUpdate:(SQRLUpdate *)update intoDirectory:(NSURL
- #pragma mark File Management
- 
- - (RACSignal *)uniqueTemporaryDirectoryForUpdate {
--	return [[[RACSignal
-+	// Clean up any orphaned update directories before creating a new one.
-+	// This prevents disk usage from growing when checkForUpdates() is called
-+	// multiple times without the app restarting. The currently staged update
-+	// (referenced by ShipItState.plist) is always preserved so quitAndInstall
-+	// remains safe to call while a new check is in progress.
-+	return [[[[[self
-+		pruneOrphanedUpdateDirectories]
-+		ignoreValues]
-+		concat:[RACSignal
- 		defer:^{
- 			SQRLDirectoryManager *directoryManager = [[SQRLDirectoryManager alloc] initWithApplicationIdentifier:SQRLShipItLauncher.shipItJobLabel];
- 			return [directoryManager storageURL];
--		}]
-+		}]]
- 		flattenMap:^(NSURL *storageURL) {
- 			NSURL *updateDirectoryTemplate = [storageURL URLByAppendingPathComponent:[SQRLUpdaterUniqueTemporaryDirectoryPrefix stringByAppendingString:@"XXXXXXX"]];
- 			char *updateDirectoryCString = strdup(updateDirectoryTemplate.path.fileSystemRepresentation);
-@@ -668,25 +676,68 @@ - (RACSignal *)pruneUpdateDirectories {
- 			return [directoryManager storageURL];
- 		}]
- 		flattenMap:^(NSURL *storageURL) {
--			NSFileManager *manager = [[NSFileManager alloc] init];
--			NSDirectoryEnumerator *enumerator = [manager enumeratorAtURL:storageURL includingPropertiesForKeys:nil options:NSDirectoryEnumerationSkipsSubdirectoryDescendants errorHandler:^(NSURL *URL, NSError *error) {
--				NSLog(@"Error enumerating item %@ within directory %@: %@", URL, storageURL, error);
--				return YES;
--			}];
-+			return [self removeUpdateDirectoriesInStorageURL:storageURL excludingURL:nil];
-+		}]
-+		setNameWithFormat:@"%@ -prunedUpdateDirectories", self];
-+}
- 
--			return [[enumerator.rac_sequence.signal
--				filter:^(NSURL *enumeratedURL) {
--					NSString *name = enumeratedURL.lastPathComponent;
--					return [name hasPrefix:SQRLUpdaterUniqueTemporaryDirectoryPrefix];
--				}]
--				doNext:^(NSURL *directoryURL) {
--					NSError *error = nil;
--					if (![manager removeItemAtURL:directoryURL error:&error]) {
--						NSLog(@"Error removing old update directory at %@: %@", directoryURL, error.sqrl_verboseDescription);
--					}
-+/// Lazily removes orphaned temporary directories upon subscription, always
-+/// preserving the directory currently referenced by ShipItState.plist so that
-+/// quitAndInstall remains safe to call mid-check.
-+///
-+/// Safe to call in any state. Sends each removed directory then completes on
-+/// an unspecified thread. Errors reading the staged request are swallowed
-+/// (treated as "nothing staged").
-+- (RACSignal *)pruneOrphanedUpdateDirectories {
-+	return [[[[[SQRLShipItRequest
-+		readUsingURL:self.shipItStateURL]
-+		map:^(SQRLShipItRequest *request) {
-+			// The request holds the URL to the staged .app bundle; its parent
-+			// is the update.XXXXXXX directory we must preserve.
-+			return [request.updateBundleURL URLByDeletingLastPathComponent];
-+		}]
-+		catch:^(NSError *error) {
-+			// No staged request (or unreadable) — nothing to preserve.
-+			return [RACSignal return:nil];
-+		}]
-+		flattenMap:^(NSURL *stagedDirectoryURL) {
-+			SQRLDirectoryManager *directoryManager = [[SQRLDirectoryManager alloc] initWithApplicationIdentifier:SQRLShipItLauncher.shipItJobLabel];
-+			return [[directoryManager storageURL]
-+				flattenMap:^(NSURL *storageURL) {
-+					return [self removeUpdateDirectoriesInStorageURL:storageURL excludingURL:stagedDirectoryURL];
- 				}];
- 		}]
--		setNameWithFormat:@"%@ -prunedUpdateDirectories", self];
-+		setNameWithFormat:@"%@ -pruneOrphanedUpdateDirectories", self];
-+}
-+
-+/// Shared enumerate-and-delete logic for update temp directories.
-+///
-+/// storageURL  - The Squirrel storage root to enumerate. Must not be nil.
-+/// excludedURL - Directory to skip (compared by standardized path). May be nil.
-+- (RACSignal *)removeUpdateDirectoriesInStorageURL:(NSURL *)storageURL excludingURL:(NSURL *)excludedURL {
-+	NSParameterAssert(storageURL != nil);
-+
-+	NSFileManager *manager = [[NSFileManager alloc] init];
-+	NSDirectoryEnumerator *enumerator = [manager enumeratorAtURL:storageURL includingPropertiesForKeys:nil options:NSDirectoryEnumerationSkipsSubdirectoryDescendants errorHandler:^(NSURL *URL, NSError *error) {
-+		NSLog(@"Error enumerating item %@ within directory %@: %@", URL, storageURL, error);
-+		return YES;
-+	}];
-+
-+	NSString *excludedPath = excludedURL.URLByStandardizingPath.path;
-+
-+	return [[enumerator.rac_sequence.signal
-+		filter:^(NSURL *enumeratedURL) {
-+			NSString *name = enumeratedURL.lastPathComponent;
-+			if (![name hasPrefix:SQRLUpdaterUniqueTemporaryDirectoryPrefix]) return NO;
-+			if (excludedPath != nil && [enumeratedURL.URLByStandardizingPath.path isEqualToString:excludedPath]) return NO;
-+			return YES;
-+		}]
-+		doNext:^(NSURL *directoryURL) {
-+			NSError *error = nil;
-+			if (![manager removeItemAtURL:directoryURL error:&error]) {
-+				NSLog(@"Error removing old update directory at %@: %@", directoryURL, error.sqrl_verboseDescription);
-+			}
-+		}];
- }
- 
- 

shell/browser/ui/file_dialog_linux.cc

@@ -13,7 +13,6 @@
 #include "shell/browser/javascript_environment.h"
 #include "shell/browser/native_window_views.h"
 #include "shell/browser/ui/file_dialog.h"
-#include "shell/common/electron_paths.h"
 #include "shell/common/gin_converters/callback_converter.h"
 #include "shell/common/gin_converters/file_path_converter.h"
 #include "shell/common/gin_helper/dictionary.h"
@@ -82,18 +81,14 @@ class FileChooserDialog : public ui::SelectFileDialog::Listener {
     ui::SelectFileDialog::FileTypeInfo file_info =
         GetFilterInfo(settings.filters);
     ApplySettings(settings);
-    base::FilePath default_path = settings.default_path.empty()
-                                      ? electron::GetDefaultPath()
-                                      : settings.default_path;
-
-    dialog_->SelectFile(ui::SelectFileDialog::SELECT_SAVEAS_FILE,
-                        base::UTF8ToUTF16(settings.title), default_path,
-                        &file_info /* file_types */, 0 /* file_type_index */,
-                        base::FilePath::StringType() /* default_extension */,
-                        settings.parent_window
-                            ? settings.parent_window->GetNativeWindow()
-                            : nullptr,
-                        nullptr);
+    dialog_->SelectFile(
+        ui::SelectFileDialog::SELECT_SAVEAS_FILE,
+        base::UTF8ToUTF16(settings.title), settings.default_path,
+        &file_info /* file_types */, 0 /* file_type_index */,
+        base::FilePath::StringType() /* default_extension */,
+        settings.parent_window ? settings.parent_window->GetNativeWindow()
+                               : nullptr,
+        nullptr);
   }
 
   void RunSaveDialog(gin_helper::Promise<gin_helper::Dictionary> promise,
@@ -113,13 +108,9 @@ class FileChooserDialog : public ui::SelectFileDialog::Listener {
     ui::SelectFileDialog::FileTypeInfo file_info =
         GetFilterInfo(settings.filters);
     ApplySettings(settings);
-    base::FilePath default_path = settings.default_path.empty()
-                                      ? electron::GetDefaultPath()
-                                      : settings.default_path;
-
     dialog_->SelectFile(
         GetDialogType(settings.properties), base::UTF8ToUTF16(settings.title),
-        default_path, &file_info, 0 /* file_type_index */,
+        settings.default_path, &file_info, 0 /* file_type_index */,
         base::FilePath::StringType() /* default_extension */,
         settings.parent_window ? settings.parent_window->GetNativeWindow()
                                : nullptr,

shell/browser/ui/file_dialog_mac.mm

@@ -21,7 +21,6 @@
 #include "content/public/browser/browser_thread.h"
 #include "electron/mas.h"
 #include "shell/browser/native_window.h"
-#include "shell/common/electron_paths.h"
 #include "shell/common/gin_converters/file_path_converter.h"
 #include "shell/common/gin_helper/dictionary.h"
 #include "shell/common/gin_helper/promise.h"
@@ -187,18 +186,14 @@ void SetupDialog(NSSavePanel* dialog, const DialogSettings& settings) {
 
   [dialog setShowsTagField:settings.shows_tag_field];
 
-  base::FilePath default_path = settings.default_path.empty()
-                                    ? electron::GetDefaultPath()
-                                    : settings.default_path;
-
   NSString* default_dir = nil;
   NSString* default_filename = nil;
-  if (!default_path.empty()) {
+  if (!settings.default_path.empty()) {
     electron::ScopedAllowBlockingForElectron allow_blocking;
-    if (base::DirectoryExists(default_path)) {
-      default_dir = base::SysUTF8ToNSString(default_path.value());
+    if (base::DirectoryExists(settings.default_path)) {
+      default_dir = base::SysUTF8ToNSString(settings.default_path.value());
     } else {
-      if (default_path.IsAbsolute()) {
+      if (settings.default_path.IsAbsolute()) {
         default_dir =
             base::SysUTF8ToNSString(settings.default_path.DirName().value());
       }

shell/browser/ui/file_dialog_win.cc

@@ -21,7 +21,6 @@
 #include "base/win/registry.h"
 #include "shell/browser/native_window_views.h"
 #include "shell/browser/ui/win/dialog_thread.h"
-#include "shell/common/electron_paths.h"
 #include "shell/common/gin_converters/file_path_converter.h"
 #include "shell/common/gin_helper/dictionary.h"
 #include "shell/common/gin_helper/promise.h"
@@ -107,12 +106,8 @@ static HRESULT ShowFileDialog(IFileDialog* dialog,
 static void ApplySettings(IFileDialog* dialog, const DialogSettings& settings) {
   std::wstring file_part;
 
-  base::FilePath default_path = settings.default_path.empty()
-                                    ? electron::GetDefaultPath()
-                                    : settings.default_path;
-
-  if (!IsDirectory(default_path))
-    file_part = default_path.BaseName().value();
+  if (!IsDirectory(settings.default_path))
+    file_part = settings.default_path.BaseName().value();
 
   dialog->SetFileName(file_part.c_str());
 
@@ -154,8 +149,8 @@ static void ApplySettings(IFileDialog* dialog, const DialogSettings& settings) {
     }
   }
 
-  if (default_path.IsAbsolute()) {
-    SetDefaultFolder(dialog, default_path);
+  if (settings.default_path.IsAbsolute()) {
+    SetDefaultFolder(dialog, settings.default_path);
   }
 }
 

shell/common/electron_paths.cc

@@ -114,18 +114,4 @@ void RegisterPathProvider() {
                                       PATH_END);
 }
 
-base::FilePath GetDefaultPath() {
-  base::FilePath path;
-
-  ScopedAllowBlockingForElectron allow_blocking;
-  if (base::PathService::Get(chrome::DIR_DEFAULT_DOWNLOADS, &path) &&
-      base::DirectoryExists(path))
-    return path;
-
-  if (base::PathService::Get(base::DIR_HOME, &path))
-    return path;
-
-  return base::FilePath();
-}
-
 }  // namespace electron

shell/common/electron_paths.h

@@ -51,10 +51,6 @@ static_assert(PATH_START < PATH_END, "invalid PATH boundaries");
 // Register the path provider with the base::PathService.
 void RegisterPathProvider();
 
-// Returns a default directory for file dialogs when no default path is
-// provided.
-base::FilePath GetDefaultPath();
-
 }  // namespace electron
 
 #endif  // ELECTRON_SHELL_COMMON_ELECTRON_PATHS_H_

spec/api-autoupdater-darwin-spec.ts

@@ -403,7 +403,7 @@ ifdescribe(shouldRunCodesignTests)('autoUpdater behavior', function () {
       });
     });
 
-    it('should preserve the staged update directory and prune orphaned ones when a new update is downloaded', async () => {
+    it('should clean up old staged update directories when a new update is downloaded', async () => {
       // Clean up any existing update directories before the test
       await cleanSquirrelCache();
 
@@ -419,23 +419,16 @@ ifdescribe(shouldRunCodesignTests)('autoUpdater behavior', function () {
         }, async (_, updateZipPath3) => {
           let updateCount = 0;
           let downloadCount = 0;
-          let dirsDuringFirstDownload: string[] = [];
-          let dirsDuringSecondDownload: string[] = [];
+          let directoriesDuringSecondDownload: string[] = [];
 
           server.get('/update-file', async (req, res) => {
             downloadCount++;
-            // Snapshot update directories at the moment each download begins.
-            // By this point uniqueTemporaryDirectoryForUpdate has already run
-            // (prune + mkdtemp). We want to verify:
-            //   1st download: 1 dir (nothing to preserve, nothing to prune)
-            //   2nd download: 2 dirs (staged dir from 1st check is preserved
-            //                 so quitAndInstall stays safe, + new temp dir)
-            // The count never exceeds 2 across repeated checks — orphaned dirs
-            // (no longer referenced by ShipItState.plist) get pruned.
-            if (downloadCount === 1) {
-              dirsDuringFirstDownload = await getUpdateDirectoriesInCache();
-            } else if (downloadCount === 2) {
-              dirsDuringSecondDownload = await getUpdateDirectoriesInCache();
+            // When the second download request arrives, Squirrel has already
+            // called uniqueTemporaryDirectoryForUpdate which (with our patch)
+            // cleans up old directories before creating the new one.
+            // Without the patch, both directories would exist at this point.
+            if (downloadCount === 2) {
+              directoriesDuringSecondDownload = await getUpdateDirectoriesInCache();
             }
             res.download(updateCount > 1 ? updateZipPath3 : updateZipPath2);
           });
@@ -462,181 +455,15 @@ ifdescribe(shouldRunCodesignTests)('autoUpdater behavior', function () {
 
           await relaunchPromise;
 
-          // First download: exactly one temp dir (the first update).
-          expect(dirsDuringFirstDownload).to.have.lengthOf(1,
-            `Expected 1 update directory during first download but found ${dirsDuringFirstDownload.length}: ${dirsDuringFirstDownload.join(', ')}`);
-
-          // Second download: exactly two — the staged one preserved + the new
-          // one. Crucially the first download's directory must still be present,
-          // otherwise a mid-download quitAndInstall would find a dangling
-          // ShipItState.plist.
-          expect(dirsDuringSecondDownload).to.have.lengthOf(2,
-            `Expected 2 update directories during second download (staged + new) but found ${dirsDuringSecondDownload.length}: ${dirsDuringSecondDownload.join(', ')}`);
-          expect(dirsDuringSecondDownload).to.include(dirsDuringFirstDownload[0],
-            'The staged update directory from the first download must be preserved during the second download');
+          // During the second download, the old staged update directory should
+          // have been cleaned up. With our patch, there should be exactly 1
+          // directory (the new one). Without the patch, there would be 2.
+          expect(directoriesDuringSecondDownload).to.have.lengthOf(1,
+            `Expected 1 update directory during second download but found ${directoriesDuringSecondDownload.length}: ${directoriesDuringSecondDownload.join(', ')}`);
         });
       });
     });
 
-    it('should keep the update directory count bounded across repeated checks', async () => {
-      // Verifies the orphan prune actually fires: after a second download
-      // completes and rewrites ShipItState.plist, the first directory is no
-      // longer referenced and must be removed when a third check begins.
-      // Without this, directories would accumulate forever.
-      await cleanSquirrelCache();
-
-      await withUpdatableApp({
-        nextVersion: '2.0.0',
-        startFixture: 'update-triple-stack',
-        endFixture: 'update-triple-stack'
-      }, async (appPath, updateZipPath2) => {
-        await withUpdatableApp({
-          nextVersion: '3.0.0',
-          startFixture: 'update-triple-stack',
-          endFixture: 'update-triple-stack'
-        }, async (_, updateZipPath3) => {
-          await withUpdatableApp({
-            nextVersion: '4.0.0',
-            startFixture: 'update-triple-stack',
-            endFixture: 'update-triple-stack'
-          }, async (__, updateZipPath4) => {
-            let downloadCount = 0;
-            const dirsPerDownload: string[][] = [];
-
-            server.get('/update-file', async (req, res) => {
-              downloadCount++;
-              // Snapshot after prune+mkdtemp but before the payload transfers.
-              dirsPerDownload.push(await getUpdateDirectoriesInCache());
-              const zips = [updateZipPath2, updateZipPath3, updateZipPath4];
-              res.download(zips[Math.min(downloadCount, zips.length) - 1]);
-            });
-            server.get('/update-check', (req, res) => {
-              res.json({
-                url: `http://localhost:${port}/update-file`,
-                name: 'My Release Name',
-                notes: 'Theses are some release notes innit',
-                pub_date: (new Date()).toString()
-              });
-            });
-            const relaunchPromise = new Promise<void>((resolve) => {
-              server.get('/update-check/updated/:version', (req, res) => {
-                res.status(204).send();
-                resolve();
-              });
-            });
-
-            const launchResult = await launchApp(appPath, [`http://localhost:${port}/update-check`]);
-            logOnError(launchResult, () => {
-              expect(launchResult).to.have.property('code', 0);
-              expect(launchResult.out).to.include('Update Downloaded');
-            });
-
-            await relaunchPromise;
-            expect(requests[requests.length - 1].url).to.equal('/update-check/updated/4.0.0');
-
-            expect(dirsPerDownload).to.have.lengthOf(3);
-
-            // 1st: fresh cache, 1 dir.
-            expect(dirsPerDownload[0]).to.have.lengthOf(1,
-              `1st download: ${dirsPerDownload[0].join(', ')}`);
-
-            // 2nd: staged (1st) preserved + new = 2 dirs.
-            expect(dirsPerDownload[1]).to.have.lengthOf(2,
-              `2nd download: ${dirsPerDownload[1].join(', ')}`);
-            expect(dirsPerDownload[1]).to.include(dirsPerDownload[0][0]);
-
-            // 3rd: 1st is now orphaned (plist points to 2nd) — must be pruned.
-            // Staged (2nd) preserved + new = still 2 dirs. Bounded.
-            expect(dirsPerDownload[2]).to.have.lengthOf(2,
-              `3rd download: ${dirsPerDownload[2].join(', ')}`);
-            expect(dirsPerDownload[2]).to.not.include(dirsPerDownload[0][0],
-              'The first (now orphaned) update directory must be pruned on the third check');
-            const secondDir = dirsPerDownload[1].find(d => d !== dirsPerDownload[0][0]);
-            expect(dirsPerDownload[2]).to.include(secondDir,
-              'The second (currently staged) update directory must be preserved on the third check');
-          });
-        });
-      });
-    });
-
-    // Regression test for https://github.com/electron/electron/issues/50200
-    //
-    // When checkForUpdates() is called again after an update has been staged,
-    // Squirrel creates a new temporary directory and prunes old ones. If the
-    // prune removes the directory that ShipItState.plist references while the
-    // second download is still in flight, a subsequent quitAndInstall() will
-    // fail with ENOENT and the app will never relaunch.
-    it('should install the staged update when quitAndInstall is called while a second check is in flight', async () => {
-      await cleanSquirrelCache();
-
-      await withUpdatableApp({
-        nextVersion: '2.0.0',
-        startFixture: 'update-race',
-        endFixture: 'update-race'
-      }, async (appPath, updateZipPath) => {
-        let downloadCount = 0;
-        let stalledResponse: express.Response | null = null;
-
-        server.get('/update-file', (req, res) => {
-          downloadCount++;
-          if (downloadCount === 1) {
-            // First download completes normally and stages the update.
-            res.download(updateZipPath);
-          } else {
-            // Second download: stall indefinitely to simulate a slow
-            // network. This keeps the second check "in progress" when
-            // quitAndInstall() fires. Hold onto the response so we can
-            // clean it up later.
-            stalledResponse = res;
-          }
-        });
-        server.get('/update-check', (req, res) => {
-          res.json({
-            url: `http://localhost:${port}/update-file`,
-            name: 'My Release Name',
-            notes: 'Theses are some release notes innit',
-            pub_date: (new Date()).toString()
-          });
-        });
-        const relaunchPromise = new Promise<void>((resolve) => {
-          server.get('/update-check/updated/:version', (req, res) => {
-            res.status(204).send();
-            resolve();
-          });
-        });
-
-        const launchResult = await launchApp(appPath, [`http://localhost:${port}/update-check`]);
-        logOnError(launchResult, () => {
-          expect(launchResult).to.have.property('code', 0);
-          expect(launchResult.out).to.include('Update Downloaded');
-          expect(launchResult.out).to.include('Calling quitAndInstall mid-download');
-          // First check + first download + second check + stalled second download.
-          expect(requests).to.have.lengthOf(4);
-          expect(requests[0]).to.have.property('url', '/update-check');
-          expect(requests[1]).to.have.property('url', '/update-file');
-          expect(requests[2]).to.have.property('url', '/update-check');
-          expect(requests[3]).to.have.property('url', '/update-file');
-          // The second download must have been in flight (never completed)
-          // when quitAndInstall was called.
-          expect(launchResult.out).to.not.include('Unexpected second download completion');
-        });
-
-        // Unblock the stalled response now that the initial app has exited
-        // so the express server can shut down cleanly.
-        if (stalledResponse) {
-          (stalledResponse as express.Response).status(500).end();
-        }
-
-        // The originally staged update (2.0.0) must have been applied and
-        // the app must relaunch, proving the staged update directory was
-        // not pruned out from under ShipItState.plist.
-        await relaunchPromise;
-        expect(requests).to.have.lengthOf(5);
-        expect(requests[4].url).to.equal('/update-check/updated/2.0.0');
-        expect(requests[4].header('user-agent')).to.include('Electron/');
-      });
-    });
-
     it('should update to lower version numbers', async () => {
       await withUpdatableApp({
         nextVersion: '0.0.1',

spec/fixtures/auto-update/update-race/index.js

@@ -1,82 +0,0 @@
-const { app, autoUpdater } = require('electron');
-
-const fs = require('node:fs');
-const path = require('node:path');
-
-process.on('uncaughtException', (err) => {
-  console.error(err);
-  process.exit(1);
-});
-
-let installInvoked = false;
-
-autoUpdater.on('error', (err) => {
-  // Once quitAndInstall() has been invoked the second in-flight check may
-  // surface a cancellation/network error as the process tears down; ignore
-  // errors after that point so we test the actual install race, not teardown.
-  if (installInvoked) {
-    console.log('Ignoring post-install error:', err && err.message);
-    return;
-  }
-  console.error(err);
-  process.exit(1);
-});
-
-const urlPath = path.resolve(__dirname, '../../../../url.txt');
-let feedUrl = process.argv[1];
-
-if (feedUrl === 'remain-open') {
-  // Hold the event loop
-  setInterval(() => {});
-} else {
-  if (!feedUrl || !feedUrl.startsWith('http')) {
-    feedUrl = `${fs.readFileSync(urlPath, 'utf8')}/${app.getVersion()}`;
-  } else {
-    fs.writeFileSync(urlPath, `${feedUrl}/updated`);
-  }
-
-  autoUpdater.setFeedURL({
-    url: feedUrl
-  });
-
-  autoUpdater.checkForUpdates();
-
-  autoUpdater.on('update-available', () => {
-    console.log('Update Available');
-  });
-
-  let downloadedOnce = false;
-
-  autoUpdater.on('update-downloaded', () => {
-    console.log('Update Downloaded');
-    if (!downloadedOnce) {
-      downloadedOnce = true;
-      // Simulate a periodic update check firing after an update was already
-      // staged. The test server is expected to stall this second download so
-      // that it remains in flight while we call quitAndInstall().
-      // The short delay lets checkForUpdatesCommand's RACCommand executing
-      // state settle; calling immediately would hit the command's "disabled"
-      // guard since RACCommand disallows concurrent execution.
-      setTimeout(() => {
-        autoUpdater.checkForUpdates();
-        // Give Squirrel enough time to enter the second check (creating a new
-        // temporary directory, which with the regression prunes the directory
-        // that the staged update lives in) before invoking the install.
-        setTimeout(() => {
-          console.log('Calling quitAndInstall mid-download');
-          installInvoked = true;
-          autoUpdater.quitAndInstall();
-        }, 3000);
-      }, 1000);
-    } else {
-      // Should not reach here — the second download is stalled on purpose.
-      console.log('Unexpected second download completion');
-      autoUpdater.quitAndInstall();
-    }
-  });
-
-  autoUpdater.on('update-not-available', () => {
-    console.error('No update available');
-    process.exit(1);
-  });
-}

spec/fixtures/auto-update/update-race/package.json

@@ -1,5 +0,0 @@
-{
-  "name": "electron-test-update-race",
-  "version": "1.0.0",
-  "main": "./index.js"
-}

spec/fixtures/auto-update/update-triple-stack/index.js

@@ -1,57 +0,0 @@
-const { app, autoUpdater } = require('electron');
-
-const fs = require('node:fs');
-const path = require('node:path');
-
-process.on('uncaughtException', (err) => {
-  console.error(err);
-  process.exit(1);
-});
-
-autoUpdater.on('error', (err) => {
-  console.error(err);
-  process.exit(1);
-});
-
-const urlPath = path.resolve(__dirname, '../../../../url.txt');
-let feedUrl = process.argv[1];
-
-if (feedUrl === 'remain-open') {
-  // Hold the event loop
-  setInterval(() => {});
-} else {
-  if (!feedUrl || !feedUrl.startsWith('http')) {
-    feedUrl = `${fs.readFileSync(urlPath, 'utf8')}/${app.getVersion()}`;
-  } else {
-    fs.writeFileSync(urlPath, `${feedUrl}/updated`);
-  }
-
-  autoUpdater.setFeedURL({
-    url: feedUrl
-  });
-
-  autoUpdater.checkForUpdates();
-
-  autoUpdater.on('update-available', () => {
-    console.log('Update Available');
-  });
-
-  let updateStackCount = 0;
-
-  autoUpdater.on('update-downloaded', () => {
-    updateStackCount++;
-    console.log('Update Downloaded');
-    if (updateStackCount > 2) {
-      autoUpdater.quitAndInstall();
-    } else {
-      setTimeout(() => {
-        autoUpdater.checkForUpdates();
-      }, 1000);
-    }
-  });
-
-  autoUpdater.on('update-not-available', () => {
-    console.error('No update available');
-    process.exit(1);
-  });
-}

spec/fixtures/auto-update/update-triple-stack/package.json

@@ -1,5 +0,0 @@
-{
-  "name": "electron-test-update-triple-stack",
-  "version": "1.0.0",
-  "main": "./index.js"
-}