helloadhavan/github_issues huggingface dataset:
Repository: vercel/next.js
Pull request: Turbopack: improve cell order lint rule for TraitRef::cell

The "cell in async map" pattern
because `cell()` has a task-local counter that is incremented, so it shouldn't happen inside of a `.map(async` block

Changed files:
filename: .config/ast-grep/rules/no-map-async-cell.yml
patch: @@ -13,6 +13,7 @@ rule:
         - pattern: ResolvedVc::cell($_X)
         - pattern: Vc::cell($_X)
         - pattern: ReadRef::cell($_X)
+        - pattern: TraitRef::cell($_X)
     - inside:
         stopBy: end
         all:

filename: turbopack/crates/turbopack-browser/src/ecmascript/list/version.rs
patch: @@ -31,8 +31,9 @@ impl Version for EcmascriptDevChunkListVersion {
             let mut by_path = self
                 .by_path
                 .iter()
+                .map(|(path, version)| (path, TraitRef::cell(version.clone())))
                 .map(|(path, version)| async move {
-                    let id = TraitRef::cell(version.clone()).id().owned().await?;
+                    let id = version.id().owned().await?;
                     Ok((path, id))
                 })
                 .try_join()
@@ -44,9 +45,7 @@ impl Version for EcmascriptDevChunkListVersion {
             let mut by_merger = self
                 .by_merger
                 .iter()
-                .map(|(_merger, version)| async move {
-                    TraitRef::cell(version.clone()).id().owned().await
-                })
+                .map(|(_merger, version)| TraitRef::cell(version.clone()).id().owned())
                 .try_join()
                 .await?;
             by_merger.sort();
Repository: electron/electron
Pull request: fix: prevent traffic light buttons flashing on deminiaturize

Backport of #50183

See that PR for details.


Notes: Fixed an issue where traffic light buttons would flash at position (0,0) when restoring a window with a custom `trafficLightPosition` from minimization on macOS.

Changed files:
filename: shell/browser/native_window_mac.h
patch: @@ -170,6 +170,12 @@ class NativeWindowMac : public NativeWindow,
   void NotifyWindowDidFailToEnterFullScreen();
   void NotifyWindowWillLeaveFullScreen();
 
+  // Hide/show traffic light buttons around miniaturize/deminiaturize to
+  // prevent them from flashing at the default position during the restore
+  // animation when a custom trafficLightPosition is configured.
+  void HideTrafficLights();
+  void RestoreTrafficLights();
+
   // Cleanup observers when window is getting closed. Note that the destructor
   // can be called much later after window gets closed, so we should not do
   // cleanup in destructor.

filename: shell/browser/native_window_mac.mm
patch: @@ -1498,6 +1498,18 @@ static bool FromV8(v8::Isolate* isolate,
     [buttons_proxy_ redraw];
 }
 
+void NativeWindowMac::HideTrafficLights() {
+  if (buttons_proxy_)
+    [buttons_proxy_ setVisible:NO];
+}
+
+void NativeWindowMac::RestoreTrafficLights() {
+  if (buttons_proxy_ && window_button_visibility_.value_or(true)) {
+    [buttons_proxy_ redraw];
+    [buttons_proxy_ setVisible:YES];
+  }
+}
+
 // In simpleFullScreen mode, update the frame for new bounds.
 void NativeWindowMac::UpdateFrame() {
   NSWindow* window = GetNativeWindow().GetNativeNSWindow();

filename: shell/browser/ui/cocoa/electron_ns_window_delegate.mm
patch: @@ -255,6 +255,10 @@ - (void)windowWillMiniaturize:(NSNotification*)notification {
   shell_->SetWindowLevel(NSNormalWindowLevel);
   shell_->UpdateWindowOriginalFrame();
   shell_->DetachChildren();
+  // Hide the traffic light buttons container before miniaturize so that
+  // when the window is restored, macOS does not render the buttons at
+  // their default position during the deminiaturize animation.
+  shell_->HideTrafficLights();
 }
 
 - (void)windowDidMiniaturize:(NSNotification*)notification {
@@ -272,6 +276,10 @@ - (void)windowDidDeminiaturize:(NSNotification*)notification {
   shell_->set_wants_to_be_visible(true);
   shell_->AttachChildren();
   shell_->SetWindowLevel(level_);
+  // Reposition traffic light buttons and make them visible again.
+  // They were hidden in windowWillMiniaturize to prevent a flash at
+  // the default (0,0) position during the restore animation.
+  shell_->RestoreTrafficLights();
   shell_->NotifyWindowRestore();
 }
Repository: ollama/ollama
Pull request: fix: MacOS file picker extensions broken

On macOS 11+ (especially 15.x), setAllowedFileTypes is deprecated and may not work properly with video files. We need to update to use the modern allowedContentTypes API with UTIs (Uniform Type Identifiers). Without this the changes in https://github.com/ollama/ollama/pull/12962 wont work and also you are unable to select non-image files like .txt or .mp4 (fir video support).

## Patched Version (this branch)
<img width="2256" height="1606" alt="Screenshot 2025-11-05 at 00 16 45" src="https://github.com/user-attachments/assets/82ee1bcb-b4d1-4e52-b7ea-0392ad6dcb4a" />


## Unpatched Version (current release)
<img width="2262" height="1600" alt="Screenshot 2025-11-05 at 00 17 05" src="https://github.com/user-attachments/assets/3b411887-4126-4948-909e-c370cdb2852f" />


Changed files:
filename: app/dialog/cocoa/dlg.m
patch: @@ -3,6 +3,11 @@
 #include <string.h>
 #include <sys/syslimits.h>
 
+// Import UniformTypeIdentifiers for macOS 11+
+#if __MAC_OS_X_VERSION_MAX_ALLOWED >= 110000
+#import <UniformTypeIdentifiers/UniformTypeIdentifiers.h>
+#endif
+
 void* NSStr(void* buf, int len) {
 	return (void*)[[NSString alloc] initWithBytes:buf length:len encoding:NSUTF8StringEncoding];
 }
@@ -109,12 +114,20 @@ - (NSInteger)runPanel:(NSSavePanel*)panel {
 	if(self->params->title != nil) {
 		[panel setTitle:[[NSString alloc] initWithUTF8String:self->params->title]];
 	}
-#pragma clang diagnostic push
-#pragma clang diagnostic ignored "-Wdeprecated-declarations"
+	// Use modern allowedContentTypes API for better file type support (especially video files)
 	if(self->params->numext > 0) {
-		[panel setAllowedFileTypes:[NSArray arrayWithObjects:(NSString**)self->params->exts count:self->params->numext]];
+		NSMutableArray *utTypes = [NSMutableArray arrayWithCapacity:self->params->numext];
+		NSString** exts = (NSString**)self->params->exts;
+		for(int i = 0; i < self->params->numext; i++) {
+			UTType *type = [UTType typeWithFilenameExtension:exts[i]];
+			if(type) {
+				[utTypes addObject:type];
+			}
+		}
+		if([utTypes count] > 0) {
+			[panel setAllowedContentTypes:utTypes];
+		}
 	}
-#pragma clang diagnostic pop
 	if(self->params->relaxext) {
 		[panel setAllowsOtherFileTypes:YES];
 	}

filename: app/dialog/cocoa/dlg_darwin.go
patch: @@ -1,6 +1,6 @@
 package cocoa
 
-// #cgo darwin LDFLAGS: -framework Cocoa
+// #cgo darwin LDFLAGS: -framework Cocoa -framework UniformTypeIdentifiers
 // #include <stdlib.h>
 // #include <sys/syslimits.h>
 // #include "dlg.h"
Repository: rust-lang/rust
Pull request: Use verbose span suggestion for type const

close: https://github.com/rust-lang/rust/issues/154214

r? fmease


Changed files:
filename: compiler/rustc_hir_analysis/src/hir_ty_lowering/mod.rs
patch: @@ -2947,7 +2947,7 @@ impl<'tcx> dyn HirTyLowerer<'tcx> + '_ {
             );
             if def_id.is_local() {
                 let name = tcx.def_path_str(def_id);
-                err.span_suggestion(
+                err.span_suggestion_verbose(
                     tcx.def_span(def_id).shrink_to_lo(),
                     format!("add `type` before `const` for `{name}`"),
                     format!("type "),

filename: tests/ui/const-generics/mgca/assoc-const-without-type_const.stderr
patch: @@ -1,20 +1,24 @@
 error: use of `const` in the type system not defined as `type const`
   --> $DIR/assoc-const-without-type_const.rs:8:35
    |
-LL |     const SIZE: usize;
-   |     - help: add `type` before `const` for `Tr::SIZE`: `type`
-...
 LL | fn mk_array<T: Tr>(_x: T) -> [(); T::SIZE] {
    |                                   ^^^^^^^
+   |
+help: add `type` before `const` for `Tr::SIZE`
+   |
+LL |     type const SIZE: usize;
+   |     ++++
 
 error: use of `const` in the type system not defined as `type const`
   --> $DIR/assoc-const-without-type_const.rs:10:10
    |
-LL |     const SIZE: usize;
-   |     - help: add `type` before `const` for `Tr::SIZE`: `type`
-...
 LL |     [(); T::SIZE]
    |          ^^^^^^^
+   |
+help: add `type` before `const` for `Tr::SIZE`
+   |
+LL |     type const SIZE: usize;
+   |     ++++
 
 error: aborting due to 2 previous errors
 

filename: tests/ui/const-generics/mgca/unmarked-free-const.stderr
patch: @@ -1,11 +1,13 @@
 error: use of `const` in the type system not defined as `type const`
   --> $DIR/unmarked-free-const.rs:9:18
    |
-LL | const N: usize = 4;
-   | - help: add `type` before `const` for `N`: `type`
-...
 LL |     let x = [(); N];
    |                  ^
+   |
+help: add `type` before `const` for `N`
+   |
+LL | type const N: usize = 4;
+   | ++++
 
 error: aborting due to 1 previous error
Repository: ollama/ollama
Pull request: ci: re-enable signing


Changed files:
filename: .github/workflows/release.yaml
patch: @@ -264,20 +264,20 @@ jobs:
       KEY_CONTAINER: ${{ vars.KEY_CONTAINER }}
     steps:
       - uses: actions/checkout@v4
-      # - uses: google-github-actions/auth@v2
-      #   with:
-      #     project_id: ollama
-      #     credentials_json: ${{ secrets.GOOGLE_SIGNING_CREDENTIALS }}
-      # - run: |
-      #     $ErrorActionPreference = "Stop"
-      #     Invoke-WebRequest -Uri "https://go.microsoft.com/fwlink/p/?LinkId=323507" -OutFile "${{ runner.temp }}\sdksetup.exe"
-      #     Start-Process "${{ runner.temp }}\sdksetup.exe" -ArgumentList @("/q") -NoNewWindow -Wait
+      - uses: google-github-actions/auth@v2
+        with:
+          project_id: ollama
+          credentials_json: ${{ secrets.GOOGLE_SIGNING_CREDENTIALS }}
+      - run: |
+          $ErrorActionPreference = "Stop"
+          Invoke-WebRequest -Uri "https://go.microsoft.com/fwlink/p/?LinkId=323507" -OutFile "${{ runner.temp }}\sdksetup.exe"
+          Start-Process "${{ runner.temp }}\sdksetup.exe" -ArgumentList @("/q") -NoNewWindow -Wait
 
-      #     Invoke-WebRequest -Uri "https://github.com/GoogleCloudPlatform/kms-integrations/releases/download/cng-v1.0/kmscng-1.0-windows-amd64.zip" -OutFile "${{ runner.temp }}\plugin.zip"
-      #     Expand-Archive -Path "${{ runner.temp }}\plugin.zip" -DestinationPath "${{ runner.temp }}\plugin\"
-      #     & "${{ runner.temp }}\plugin\*\kmscng.msi" /quiet
+          Invoke-WebRequest -Uri "https://github.com/GoogleCloudPlatform/kms-integrations/releases/download/cng-v1.0/kmscng-1.0-windows-amd64.zip" -OutFile "${{ runner.temp }}\plugin.zip"
+          Expand-Archive -Path "${{ runner.temp }}\plugin.zip" -DestinationPath "${{ runner.temp }}\plugin\"
+          & "${{ runner.temp }}\plugin\*\kmscng.msi" /quiet
 
-      #     echo "${{ vars.OLLAMA_CERT }}" >ollama_inc.crt
+          echo "${{ vars.OLLAMA_CERT }}" >ollama_inc.crt
       - uses: actions/setup-go@v5
         with:
           go-version-file: go.mod
Repository: facebook/react
Pull request: [DevTools] Filter out built-in stack frames

Treat fake eval anonymous stacks as built-in. Hide built-in stack frames unless they're used to call into a non-ignored stack frame.

The two main things to fix here is that 1) we're showing a linkified stack for fake anonymous and 2) we're showing only built-ins when the stack is completely internal. Meaning framework code is all noise.

Changed files:
filename: packages/react-devtools-shared/src/devtools/views/Components/StackTraceView.css
patch: @@ -2,13 +2,17 @@
   padding: 0.25rem;
 }
 
-.CallSite, .IgnoredCallSite {
+.CallSite {
   display: block;
   padding-left: 1rem;
 }
 
-.IgnoredCallSite {
-  opacity: 0.5;
+.IgnoredCallSite, .BuiltInCallSite {
+  display: none;
+}
+
+.CallSite + .BuiltInCallSite {
+  display: block;
 }
 
 .Link {

filename: packages/react-devtools-shared/src/devtools/views/Components/StackTraceView.js
patch: @@ -60,16 +60,22 @@ export function CallSiteView({
     symbolicatedCallSite !== null ? symbolicatedCallSite.location : callSite;
   const ignored =
     symbolicatedCallSite !== null ? symbolicatedCallSite.ignored : false;
-  if (ignored) {
-    // TODO: Make an option to be able to toggle the display of ignore listed rows.
-    // Ideally this UI should be higher than a single Stack Trace so that there's not
-    // multiple buttons in a single inspection taking up space.
-    return null;
-  }
+  // TODO: Make an option to be able to toggle the display of ignore listed rows.
+  // Ideally this UI should be higher than a single Stack Trace so that there's not
+  // multiple buttons in a single inspection taking up space.
+
+  const isBuiltIn = url === '' || url.startsWith('<anonymous>'); // This looks like a fake anonymous through eval.
   return (
-    <div className={ignored ? styles.IgnoredCallSite : styles.CallSite}>
+    <div
+      className={
+        ignored
+          ? styles.IgnoredCallSite
+          : isBuiltIn
+            ? styles.BuiltInCallSite
+            : styles.CallSite
+      }>
       {functionName || virtualFunctionName}
-      {url !== '' && (
+      {!isBuiltIn && (
         <>
           {' @ '}
           <span
Repository: ollama/ollama
Pull request: mac: fix stale VRAM data

The scheduler updates free VRAM based on current loaded models.  This was mutating the persisted list of GPUs, and when coupled with the non-refreshing logic for Metal that lead to stale low VRAM reporting after unload.  The fix is to make sure the GPU discovery always returns a copy so the schedulers GPU list is in fact ephemeral and doesn't leak any temporary adjustments back into the persistent list.

Fixes #12912 

Changed files:
filename: discover/runner.go
patch: @@ -237,7 +237,7 @@ func GPUDevices(ctx context.Context, runners []ml.FilteredRunnerDiscovery) []ml.
 	} else {
 		if runtime.GOOS == "darwin" && runtime.GOARCH == "arm64" {
 			// metal never updates free VRAM
-			return devices
+			return append([]ml.DeviceInfo{}, devices...)
 		}
 
 		slog.Debug("refreshing free memory")
@@ -336,7 +336,7 @@ func GPUDevices(ctx context.Context, runners []ml.FilteredRunnerDiscovery) []ml.
 		}
 	}
 
-	return devices
+	return append([]ml.DeviceInfo{}, devices...)
 }
 
 func filterOverlapByLibrary(supported map[string]map[string]map[string]int, needsDelete []bool) {
Repository: nodejs/node
Pull request: crypto: ensure documented RSA-PSS saltLength default is used

This PR ensures that the documented RSA-PSS saltLength default is used regardless of the OpenSSL version used by the runtime.

Changed files:
filename: lib/internal/crypto/sig.js
patch: @@ -50,6 +50,8 @@ const {
   isArrayBufferView,
 } = require('internal/util/types');
 
+const constants = internalBinding('constants').crypto;
+
 function Sign(algorithm, options) {
   if (!(this instanceof Sign))
     return new Sign(algorithm, options);
@@ -85,7 +87,11 @@ function getPadding(options) {
 }
 
 function getSaltLength(options) {
-  return getIntOption('saltLength', options);
+  let saltLength = getIntOption('saltLength', options);
+  if (options.padding === constants.RSA_PKCS1_PSS_PADDING && saltLength === undefined) {
+    saltLength = constants.RSA_PSS_SALTLEN_MAX_SIGN;
+  }
+  return saltLength;
 }
 
 function getDSASignatureEncoding(options) {

filename: test/parallel/test-crypto-rsa-pss-default-salt-length.js
patch: @@ -0,0 +1,37 @@
+'use strict';
+const common = require('../common');
+if (!common.hasCrypto)
+  common.skip('missing crypto');
+
+const assert = require('assert');
+const crypto = require('crypto');
+
+const fixtures = require('../common/fixtures');
+
+const privateKey = crypto.createPrivateKey(fixtures.readKey('rsa_private.pem', 'ascii'));
+const publicKey = crypto.createPublicKey(fixtures.readKey('rsa_public.pem', 'ascii'));
+
+const data = crypto.randomBytes(32);
+
+for (const digest of ['sha256', 'sha384', 'sha512']) {
+  const hLen = crypto.hash(digest, data, 'buffer').byteLength;
+  const maxSaltLength =
+      privateKey.asymmetricKeyDetails.modulusLength / 8 - hLen - 2;
+
+  const sig = crypto.sign(digest, data, {
+    key: privateKey,
+    padding: crypto.constants.RSA_PKCS1_PSS_PADDING,
+    // No "saltLength" provided, documented default RSA_PSS_SALTLEN_MAX_SIGN expected
+  });
+
+  assert.strictEqual(crypto.verify(
+    digest,
+    data,
+    {
+      key: publicKey,
+      padding: crypto.constants.RSA_PKCS1_PSS_PADDING,
+      saltLength: maxSaltLength,
+    },
+    sig
+  ), true);
+}
Repository: huggingface/transformers
Pull request: [ Dynamic weight loader] fix remote code when format matches

# What does this PR do?
Some renaming should just never be applied when the weight format already matches. (this is actually regardless of remote code). 

This allows us to remove 1 test added in https://github.com/huggingface/transformers/commit/8c1b5d37827a6691fef4b2d926f2d04fb6f5a9e3 ! 

FYI @vasqu we discussed offline with @Cyrilvallez and this makes sense mostly for "errors" or special cases were a model type has 2 checkpoint formats. 



Changed files:
filename: src/transformers/core_model_loading.py
patch: @@ -1128,6 +1128,9 @@ def convert_and_load_state_dict_in_model(
         renamed_key, source_pattern = rename_source_key(
             original_key, renamings, converters, prefix, meta_model_state_dict
         )
+        if renamed_key not in meta_model_state_dict and original_key in meta_model_state_dict:
+            # Key should probably not have been renamed but we might need the `prefix` to be added.`
+            renamed_key, source_pattern = rename_source_key(original_key, [], [], prefix, meta_model_state_dict)
 
         # 2. finally, collect the tensor into the proper converter
         if renamed_key in meta_model_state_dict:

filename: tests/utils/test_modeling_utils.py
patch: @@ -1765,22 +1765,6 @@ def test_model_from_pretrained_from_mlx(self):
             outputs_from_saved = new_model(input_ids)
             torch.testing.assert_close(outputs_from_saved["logits"], outputs["logits"])
 
-    def test_warning_for_beta_gamma_parameters(self):
-        config = PreTrainedConfig()
-        model = TestModelGammaBeta(config)
-
-        with tempfile.TemporaryDirectory() as tmp_dir:
-            model.save_pretrained(tmp_dir)
-            with LoggingLevel(logging.INFO):
-                _, loading_info = TestModelGammaBeta.from_pretrained(tmp_dir, config=config, output_loading_info=True)
-
-        missing_keys = loading_info["missing_keys"]
-        unexpected_keys = loading_info["unexpected_keys"]
-        self.assertIn("LayerNorm.gamma", missing_keys)
-        self.assertIn("LayerNorm.weight", unexpected_keys)
-        self.assertIn("LayerNorm.beta", missing_keys)
-        self.assertIn("LayerNorm.bias", unexpected_keys)
-
     def test_can_generate(self):
         """Tests the behavior of `PreTrainedModel.can_generate` method."""
         logger = logging.get_logger("transformers.modeling_utils")
Repository: vercel/next.js
Pull request: fix(turbopack): use posix.join for client URL manifest paths on Windows

## Summary

Fixes #90381

On Windows, `path.join()` uses backslashes as separators. The manifest paths constructed in `TurbopackManifestLoader` (`buildManifestPath`, `ssgManifestPath`, `clientMiddlewareManifestPath`) are used both as filesystem paths (combined with `this.distDir`) **and** as client-facing URL segments returned in `lowPriorityFiles`.

When these paths contain backslashes, the client router fetches URLs like:
```
/_next/static%5Cdevelopment%5C_buildManifest.js      ← 404
/_next/static%5Cdevelopment%5C_ssgManifest.js        ← 404
/_next/static%5Cdevelopment%5C_clientMiddlewareManifest.js  ← 404
```

## Fix

Replace `join()` with `posix.join()` (already imported from `'path'`) for the three variables that end up as URL segments:
- `buildManifestPath`
- `ssgManifestPath`
- `clientMiddlewareManifestPath`

The `join(this.distDir, ...)` calls used for filesystem writes are left unchanged — `path.join()` handles mixed-separator paths correctly on all platforms.

## Testing

Reproduce by running `next dev` on Windows with Turbopack and checking the Network tab for 404s on manifest files.

Changed files:
filename: packages/next/src/shared/lib/turbopack/manifest-loader.ts
patch: @@ -637,12 +637,12 @@ export class TurbopackManifestLoader {
 
     const sortedPageKeys = getSortedRoutes(pagesKeys)
 
-    let buildManifestPath = join(
+    let buildManifestPath = posix.join(
       CLIENT_STATIC_FILES_PATH,
       this.buildId,
       '_buildManifest.js'
     )
-    let ssgManifestPath = join(
+    let ssgManifestPath = posix.join(
       CLIENT_STATIC_FILES_PATH,
       this.buildId,
       '_ssgManifest.js'
@@ -837,7 +837,7 @@ export class TurbopackManifestLoader {
   private writeMiddlewareManifest(): {
     clientMiddlewareManifestPath: string
   } {
-    let clientMiddlewareManifestPath = join(
+    let clientMiddlewareManifestPath = posix.join(
       CLIENT_STATIC_FILES_PATH,
       this.buildId,
       TURBOPACK_CLIENT_MIDDLEWARE_MANIFEST
Repository: ollama/ollama
Pull request: bugfix: show connection string for interactive cli usage


Changed files:
filename: cmd/cmd.go
patch: @@ -284,6 +284,14 @@ func loadOrUnloadModel(cmd *cobra.Command, opts *runOptions) error {
 		return err
 	} else if info.RemoteHost != "" {
 		// Cloud model, no need to load/unload
+		if opts.ShowConnect {
+			p.StopAndClear()
+			if strings.HasPrefix(info.RemoteHost, "https://ollama.com") {
+				fmt.Fprintf(os.Stderr, "Connecting to '%s' on 'ollama.com' ⚡\n", info.RemoteModel)
+			} else {
+				fmt.Fprintf(os.Stderr, "Connecting to '%s' on '%s'\n", info.RemoteModel, info.RemoteHost)
+			}
+		}
 		return nil
 	}
 
@@ -296,14 +304,6 @@ func loadOrUnloadModel(cmd *cobra.Command, opts *runOptions) error {
 	}
 
 	return client.Generate(cmd.Context(), req, func(r api.GenerateResponse) error {
-		if r.RemoteModel != "" && opts.ShowConnect {
-			p.StopAndClear()
-			if strings.HasPrefix(r.RemoteHost, "https://ollama.com") {
-				fmt.Fprintf(os.Stderr, "Connecting to '%s' on 'ollama.com' ⚡\n", r.RemoteModel)
-			} else {
-				fmt.Fprintf(os.Stderr, "Connecting to '%s' on '%s'\n", r.RemoteModel, r.RemoteHost)
-			}
-		}
 		return nil
 	})
 }
Repository: electron/electron
Pull request: fix: don't call `TaskDialogIndirect` with disabled parent windows

#### Description of Change

Should resolve #50068. This is a naïve fix, and I don't have a Windows dev box to easily test this on.

[The documentation for `TaskDialogIndirect`](https://learn.microsoft.com/en-us/windows/win32/api/commctrl/nf-commctrl-taskdialogindirect) includes the following line:
> The parent window should not be hidden or disabled when this function is called.

This ensures we don't violate that remark.

#### Checklist

- [x] PR description included
- [ ] I have built and tested this PR
- [ ] `npm test` passes
- [x] [PR release notes](https://github.com/electron/clerk/blob/main/README.md) describe the change in a way relevant to app developers, and are [capitalized, punctuated, and past tense](https://github.com/electron/clerk/blob/main/README.md#examples).

#### Release Notes

Notes: Fixed bug where opening a message box immediately upon closing a child window may cause the parent window to freeze on Windows.

Changed files:
filename: shell/browser/ui/message_box_win.cc
patch: @@ -158,7 +158,7 @@ DialogResult ShowTaskDialogWstr(gfx::AcceleratedWidget parent,
   config.hInstance = GetModuleHandle(nullptr);
   config.dwFlags = flags;
 
-  if (parent) {
+  if (parent && ::IsWindowEnabled(parent)) {
     config.hwndParent = parent;
     config.dwFlags |= TDF_POSITION_RELATIVE_TO_WINDOW;
   }