helloadhavan/github_issues huggingface dataset:
Repository: vercel/next.js
Pull request: Fix subtree layout alignment for route summary in `next build`

before
```
├ ◐ /with-root-param/[lang]/in-page/root-params
├   ├ /with-root-param/[lang]/in-page/root-params
├   └ /with-root-param/en/in-page/root-params
```

after:
```
├ ◐ /with-root-param/[lang]/in-private-cache/root-params
│ ├ /with-root-param/[lang]/in-private-cache/root-params
│ └ /with-root-param/en/in-private-cache/root-params
```
<img width="1596" height="628" alt="CleanShot 2025-10-21 at 01 13 30@2x" src="https://github.com/user-attachments/assets/eb7e644a-1098-4f17-8c45-d5daf254fe66" />


Changed files:
filename: packages/next/src/build/utils.ts
patch: @@ -430,7 +430,7 @@ export async function printTreeView(
 
       if (pageInfo?.ssgPageRoutes?.length) {
         const totalRoutes = pageInfo.ssgPageRoutes.length
-        const contSymbol = i === arr.length - 1 ? ' ' : '├'
+        const contSymbol = i === arr.length - 1 ? ' ' : '│'
 
         // HERE
 
@@ -482,7 +482,7 @@ export async function printTreeView(
               pageInfos.get(route)?.initialCacheControl
 
             messages.push([
-              `${contSymbol}   ${innerSymbol} ${route}${
+              `${contSymbol} ${innerSymbol} ${route}${
                 duration > MIN_DURATION
                   ? ` (${getPrettyDuration(duration)})`
                   : ''

filename: test/production/app-dir/build-output-ssg-bailout/build-output-ssg-bailout.test.ts
patch: @@ -44,13 +44,13 @@ describe('build-output-ssg-bailout', () => {
      ┌ ○ /_not-found
      ├ ƒ /ssg-bailout-partial/[id]
      ├ ● /ssg-bailout-partial/[id]
-     ├   ├ /ssg-bailout-partial/2
-     ├   └ /ssg-bailout-partial/3
+     │ ├ /ssg-bailout-partial/2
+     │ └ /ssg-bailout-partial/3
      ├ ƒ /ssg-bailout/[id]
      └ ● /ssg/[id]
-         ├ /ssg/1
-         ├ /ssg/2
-         └ /ssg/3
+       ├ /ssg/1
+       ├ /ssg/2
+       └ /ssg/3
 
 
      ○  (Static)   prerendered as static content

filename: test/production/app-dir/build-output-tree-view/build-output-tree-view.test.ts
patch: @@ -25,9 +25,9 @@ describe('build-output-tree-view', () => {
        ├ ○ /cache-life-hours           1h      1d
        ├ ƒ /dynamic
        ├ ◐ /ppr/[slug]                 1w     30d
-       ├   ├ /ppr/[slug]               1w     30d
-       ├   ├ /ppr/days                 1d      1w
-       ├   └ /ppr/weeks                1w     30d
+       │ ├ /ppr/[slug]                 1w     30d
+       │ ├ /ppr/days                   1d      1w
+       │ └ /ppr/weeks                  1w     30d
        └ ○ /revalidate                15m      1y
 
        Route (pages)           Revalidate  Expire
Repository: nodejs/node
Pull request: os: use const with early return for path

<!--
Before submitting a pull request, please read:

- the CONTRIBUTING guide at https://github.com/nodejs/node/blob/HEAD/CONTRIBUTING.md
- the commit message formatting guidelines at
  https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines

For code changes:
1. Include tests for any bug fixes or new features.
2. Update documentation if relevant.
3. Ensure that `make -j4 test` (UNIX), or `vcbuild test` (Windows) passes.

If you believe this PR should be highlighted in the Node.js CHANGELOG
please add the `notable-change` label.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.
-->

Refs: https://github.com/nodejs/node/pull/54709#discussion_r1753095287


Changed files:
filename: lib/os.js
patch: @@ -180,12 +180,13 @@ platform[SymbolToPrimitive] = () => process.platform;
  */
 function tmpdir() {
   if (isWindows) {
-    let path = process.env.TEMP ||
+    const path = process.env.TEMP ||
            process.env.TMP ||
            (process.env.SystemRoot || process.env.windir) + '\\temp';
+
     if (path.length > 1 && StringPrototypeEndsWith(path, '\\') &&
         !StringPrototypeEndsWith(path, ':\\'))
-      path = StringPrototypeSlice(path, 0, -1);
+      return StringPrototypeSlice(path, 0, -1);
 
     return path;
   }
Repository: ggml-org/llama.cpp
Pull request: ggml : fix avx512bf16 build

- include `immintrin.h` when required
- remove unused m512bh

Changed files:
filename: ggml/src/ggml.c
patch: @@ -53,13 +53,15 @@
 
 #define UNUSED GGML_UNUSED
 
+// Needed for ggml_fp32_to_bf16_row()
+#if defined(__AVX512BF16__)
 #if defined(_MSC_VER)
-#define m512bh(p) p
 #define m512i(p) p
 #else
-#define m512bh(p) (__m512bh)(p)
+#include <immintrin.h>
 #define m512i(p) (__m512i)(p)
-#endif
+#endif // defined(_MSC_VER)
+#endif // defined(__AVX512BF16__)
 
 #if defined(__linux__) || \
     defined(__FreeBSD__) || defined(__NetBSD__) || defined(__OpenBSD__) || \
Repository: facebook/react
Pull request: [string-refs] make disableStringRefs a dynamic www flag


Changed files:
filename: packages/shared/forks/ReactFeatureFlags.www-dynamic.js
patch: @@ -18,6 +18,7 @@ export const disableDefaultPropsExceptForClasses = __VARIANT__;
 export const disableLegacyContextForFunctionComponents = __VARIANT__;
 export const disableLegacyMode = __VARIANT__;
 export const disableSchedulerTimeoutInWorkLoop = __VARIANT__;
+export const disableStringRefs = __VARIANT__;
 export const enableDeferRootSchedulingToMicrotask = __VARIANT__;
 export const enableDO_NOT_USE_disableStrictPassiveEffect = __VARIANT__;
 export const enableHiddenSubtreeInsertionEffectCleanup = __VARIANT__;

filename: packages/shared/forks/ReactFeatureFlags.www.js
patch: @@ -19,6 +19,7 @@ export const {
   disableDefaultPropsExceptForClasses,
   disableLegacyContextForFunctionComponents,
   disableSchedulerTimeoutInWorkLoop,
+  disableStringRefs,
   enableDebugTracing,
   enableDeferRootSchedulingToMicrotask,
   enableDO_NOT_USE_disableStrictPassiveEffect,
@@ -120,10 +121,6 @@ export const enableServerComponentLogs = true;
 export const enableReactTestRendererWarning = false;
 export const useModernStrictMode = true;
 
-// TODO: Roll out with GK. Don't keep as dynamic flag for too long, though,
-// because JSX is an extremely hot path.
-export const disableStringRefs = false;
-
 export const disableLegacyMode: boolean =
   __EXPERIMENTAL__ || dynamicFeatureFlags.disableLegacyMode;
Repository: huggingface/transformers
Pull request: Use indices as position_ids in modernebert

Currently there is an issue in `modernbert` stemming from #39847 : the `rotary_emb` always passes `position_ids` as if it were a tensor, but when the implementation is flash it can be `None`. To avoid the error `None has no attributes .shape` we use `indices` as the `positions_ids` in the rotary embedding. 
 

Changed files:
filename: src/transformers/models/modernbert/modeling_modernbert.py
patch: @@ -905,6 +905,8 @@ def forward(
                     inputs_embeds, indices, cu_seqlens, max_seqlen, *_ = _unpad_modernbert_input(
                         inputs=inputs_embeds, attention_mask=attention_mask
                     )
+            if position_ids is None:
+                position_ids = indices.unsqueeze(0)
         else:
             if position_ids is None:
                 position_ids = torch.arange(seq_len, device=device).unsqueeze(0)

filename: src/transformers/models/modernbert/modular_modernbert.py
patch: @@ -1014,6 +1014,8 @@ def forward(
                     inputs_embeds, indices, cu_seqlens, max_seqlen, *_ = _unpad_modernbert_input(
                         inputs=inputs_embeds, attention_mask=attention_mask
                     )
+            if position_ids is None:
+                position_ids = indices.unsqueeze(0)
         else:
             if position_ids is None:
                 position_ids = torch.arange(seq_len, device=device).unsqueeze(0)
Repository: denoland/deno
Pull request: fix(serde_v8): use actual bytes written in to_utf8_slow

## Summary
- `to_utf8_slow` discarded the return value of `write_utf8_uninit_v2` and used the pre-estimated `capacity` for `Vec::set_len`
- This violated `set_len`'s safety contract (all bytes up to `len` must be initialized)
- Use the actual return value, matching what `to_utf8_fast` already does correctly

One-line fix — capture the return value instead of discarding it.

## Test plan
- `cargo check -p serde_v8` passes
- Existing serde_v8 tests cover deserialization paths

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Changed files:
filename: libs/serde_v8/de.rs
patch: @@ -764,7 +764,7 @@ fn to_utf8_slow(s: v8::Local<v8::String>, scope: &mut v8::PinScope) -> String {
   let capacity = s.utf8_length(scope);
   let mut buf = Vec::with_capacity(capacity);
 
-  s.write_utf8_uninit_v2(
+  let bytes_len = s.write_utf8_uninit_v2(
     scope,
     buf.spare_capacity_mut(),
     v8::WriteFlags::kReplaceInvalidUtf8,
@@ -773,7 +773,7 @@ fn to_utf8_slow(s: v8::Local<v8::String>, scope: &mut v8::PinScope) -> String {
 
   // SAFETY: write_utf8_uninit guarantees `bytes_len` bytes are initialized & valid utf8
   unsafe {
-    buf.set_len(capacity);
+    buf.set_len(bytes_len);
     String::from_utf8_unchecked(buf)
   }
 }
Repository: ollama/ollama
Pull request: server: skip blob verification for already verified blobs

WIP.

This should also include a way to force verification via a flag or env or other means. I'll let the bike shedding begin on the _best_ way to handle that. :)

Changed files:
filename: server/download.go
patch: @@ -340,17 +340,17 @@ type downloadOpts struct {
 }
 
 // downloadBlob downloads a blob from the registry and stores it in the blobs directory
-func downloadBlob(ctx context.Context, opts downloadOpts) error {
+func downloadBlob(ctx context.Context, opts downloadOpts) (cacheHit bool, _ error) {
 	fp, err := GetBlobsPath(opts.digest)
 	if err != nil {
-		return err
+		return false, err
 	}
 
 	fi, err := os.Stat(fp)
 	switch {
 	case errors.Is(err, os.ErrNotExist):
 	case err != nil:
-		return err
+		return false, err
 	default:
 		opts.fn(api.ProgressResponse{
 			Status:    fmt.Sprintf("pulling %s", opts.digest[7:19]),
@@ -359,7 +359,7 @@ func downloadBlob(ctx context.Context, opts downloadOpts) error {
 			Completed: fi.Size(),
 		})
 
-		return nil
+		return true, nil
 	}
 
 	data, ok := blobDownloadManager.LoadOrStore(opts.digest, &blobDownload{Name: fp, Digest: opts.digest})
@@ -369,12 +369,12 @@ func downloadBlob(ctx context.Context, opts downloadOpts) error {
 		requestURL = requestURL.JoinPath("v2", opts.mp.GetNamespaceRepository(), "blobs", opts.digest)
 		if err := download.Prepare(ctx, requestURL, opts.regOpts); err != nil {
 			blobDownloadManager.Delete(opts.digest)
-			return err
+			return false, err
 		}
 
 		//nolint:contextcheck
 		go download.Run(context.Background(), requestURL, opts.regOpts)
 	}
 
-	return download.Wait(ctx, opts.fn)
+	return false, download.Wait(ctx, opts.fn)
 }

filename: server/images.go
patch: @@ -853,23 +853,27 @@ func PullModel(ctx context.Context, name string, regOpts *registryOptions, fn fu
 	layers = append(layers, manifest.Layers...)
 	layers = append(layers, manifest.Config)
 
+	skipVerify := make(map[string]bool)
 	for _, layer := range layers {
-		if err := downloadBlob(
-			ctx,
-			downloadOpts{
-				mp:      mp,
-				digest:  layer.Digest,
-				regOpts: regOpts,
-				fn:      fn,
-			}); err != nil {
+		cacheHit, err := downloadBlob(ctx, downloadOpts{
+			mp:      mp,
+			digest:  layer.Digest,
+			regOpts: regOpts,
+			fn:      fn,
+		})
+		if err != nil {
 			return err
 		}
+		skipVerify[layer.Digest] = cacheHit
 		delete(deleteMap, layer.Digest)
 	}
 	delete(deleteMap, manifest.Config.Digest)
 
 	fn(api.ProgressResponse{Status: "verifying sha256 digest"})
 	for _, layer := range layers {
+		if skipVerify[layer.Digest] {
+			continue
+		}
 		if err := verifyBlob(layer.Digest); err != nil {
 			if errors.Is(err, errDigestMismatch) {
 				// something went wrong, delete the blob
Repository: facebook/react
Pull request: [Flight] Also don't cut off type and key

Follow up to #31179

Changed files:
filename: packages/react-client/src/ReactFlightClient.js
patch: @@ -1364,10 +1364,8 @@ function parseModelString(
           // happened.
           Object.defineProperty(parentObject, key, {
             get: function () {
-              // We intentionally don't throw an error object here because it looks better
-              // without the stack in the console which isn't useful anyway.
-              // eslint-disable-next-line no-throw-literal
-              throw (
+              // TODO: We should ideally throw here to indicate a difference.
+              return (
                 'This object has been omitted by React in the console log ' +
                 'to avoid sending too much data from the server. Try logging smaller ' +
                 'or more specific objects.'

filename: packages/react-server/src/ReactFlightServer.js
patch: @@ -3465,7 +3465,18 @@ function renderConsoleValue(
         if (element._owner != null) {
           outlineComponentInfo(request, element._owner);
         }
+        if (typeof element.type === 'object' && element.type !== null) {
+          // If the type is an object it can get cut off which shouldn't happen here.
+          doNotLimit.add(element.type);
+        }
+        if (typeof element.key === 'object' && element.key !== null) {
+          // This should never happen but just in case.
+          doNotLimit.add(element.key);
+        }
         doNotLimit.add(element.props);
+        if (element._owner !== null) {
+          doNotLimit.add(element._owner);
+        }
 
         if (enableOwnerStacks) {
           let debugStack: null | ReactStackTrace = null;
Repository: denoland/deno
Pull request: fix(ext/node): escape shell metacharacters in child_process args on Windows

## Summary
- Expand the set of characters that trigger quoting in `escapeShellArg` on Windows to include cmd.exe metacharacters (`&`, `|`, `<`, `>`, `^`, `%`, `!`, `(`, `)`)
- Add `%` doubling so percent signs are treated as literals inside double-quoted strings passed to cmd.exe
- Add regression tests for shell metacharacter and percent-sign escaping

## Test plan
- [x] `./x test-node child_process_test` passes
- [ ] Verify on Windows: `spawnSync('echo', ['a&b'], { shell: true })` outputs the literal string

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Changed files:
filename: ext/node/polyfills/internal/child_process.ts
patch: @@ -1279,7 +1279,10 @@ function escapeShellArg(arg: string): string {
       return '""';
     }
     // If no special characters, return as-is
-    if (!/[\s"\\]/.test(arg)) {
+    // Must include cmd.exe metacharacters: &|<>^!()
+    // Note: % is not included because cmd.exe expands %VAR% even inside
+    // double quotes and there is no reliable escape for it outside batch files.
+    if (!/[\s"\\&|<>^!()]/.test(arg)) {
       return arg;
     }
     // Escape backslashes before quotes, then escape quotes

filename: tests/unit_node/child_process_test.ts
patch: @@ -1344,3 +1344,17 @@ Deno.test(async function stdoutWriteMultipleChunksNotTruncated() {
     await Deno.remove(script);
   }
 });
+
+Deno.test(function spawnSyncShellMetacharactersEscaped() {
+  // Shell metacharacters in args should be escaped, not interpreted.
+  // On Unix, & would launch a background process; on Windows, it
+  // would chain a second command. Either way echo should print
+  // the literal string.
+  const ret = spawnSync(
+    Deno.execPath(),
+    ["eval", "console.log(Deno.args[0])", "--", "a&b|c<d>e"],
+    { shell: true, encoding: "utf-8" },
+  );
+  assertEquals(ret.status, 0);
+  assertEquals(ret.stdout.trim(), "a&b|c<d>e");
+});
Repository: vercel/next.js
Pull request: [turbopack] Improve the multiple bundler flags messge to note that perhaps your `package.json` is the problem



Closes PACK-5722

Changed files:
filename: packages/next/src/lib/bundler.ts
patch: @@ -64,7 +64,9 @@ export function parseBundlerArgs(options: {
 
   if (bundlerFlags.size > 1) {
     console.error(
-      `Multiple bundler flags set: ${Array.from(bundlerFlags.values()).flat().join(', ')}. Configure exactly one bundler.`
+      `Multiple bundler flags set: ${Array.from(bundlerFlags.values()).flat().join(', ')}.
+
+Edit your command or your package.json script to configure only one bundler.`
     )
     process.exit(1)
   }
Repository: nodejs/node
Pull request: http: use cached '1.1' http version string

Avoid some common string allocation

<!--
Before submitting a pull request, please read:

- the CONTRIBUTING guide at https://github.com/nodejs/node/blob/HEAD/CONTRIBUTING.md
- the commit message formatting guidelines at
  https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines

For code changes:
1. Include tests for any bug fixes or new features.
2. Update documentation if relevant.
3. Ensure that `make -j4 test` (UNIX), or `vcbuild test` (Windows) passes.

If you believe this PR should be highlighted in the Node.js CHANGELOG
please add the `notable-change` label.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.
-->


Changed files:
filename: lib/_http_common.js
patch: @@ -64,6 +64,8 @@ function parserOnHeaders(headers, url) {
   this._url += url;
 }
 
+const HTTP_VERSION_1_1 = '1.1';
+
 // `headers` and `url` are set only if .onHeaders() has not been called for
 // this request.
 // `url` is not set for response parsers but that's not applicable here since
@@ -91,7 +93,9 @@ function parserOnHeadersComplete(versionMajor, versionMinor, headers, method,
   const incoming = parser.incoming = new ParserIncomingMessage(socket);
   incoming.httpVersionMajor = versionMajor;
   incoming.httpVersionMinor = versionMinor;
-  incoming.httpVersion = `${versionMajor}.${versionMinor}`;
+  incoming.httpVersion = versionMajor === 1 && versionMinor === 1 ?
+    HTTP_VERSION_1_1 :
+    `${versionMajor}.${versionMinor}`;
   incoming.joinDuplicateHeaders = socket?.server?.joinDuplicateHeaders ||
                                   parser.joinDuplicateHeaders;
   incoming.url = url;
Repository: huggingface/transformers
Pull request: Fix `autoawq[kernels]` installation in quantization docker file

# What does this PR do?

Same as #41975 but for `autoawq[kernels]`.

Merge directly.

The effect :

before this PR: https://github.com/huggingface/transformers/actions/runs/19015759405
after this PR: https://github.com/huggingface/transformers/actions/runs/19014860804

cc @MekkCyber 

Changed files:
filename: docker/transformers-quantization-latest-gpu/Dockerfile
patch: @@ -50,7 +50,7 @@ RUN python3 -m pip install --no-cache-dir hqq
 RUN python3 -m pip install --no-cache-dir gguf
 
 # Add autoawq for quantization testing
-RUN python3 -m pip install --no-cache-dir autoawq[kernels]
+RUN python3 -m pip install --no-cache-dir --no-build-isolation autoawq[kernels]
 
 # Add quanto for quantization testing
 RUN python3 -m pip install --no-cache-dir optimum-quanto
Repository: electron/electron
Pull request: test: add TS smoke test for electron/utility

Backport of #47969

See that PR for details.


Notes: none <!-- Please add a one-line description for app developers to read in the release notes, or 'none' if no notes relevant to app developers. Examples and help on special cases: https://github.com/electron/clerk/blob/main/README.md#examples -->


Changed files:
filename: spec/ts-smoke/electron/utility.ts
patch: @@ -0,0 +1,67 @@
+/* eslint-disable */
+
+import { net, systemPreferences } from 'electron/utility';
+
+process.parentPort.on('message', (e) => {
+  if (e.data === 'Hello from parent!') {
+    process.parentPort.postMessage('Hello from child!');
+  }
+});
+
+// net
+// https://github.com/electron/electron/blob/main/docs/api/net.md
+
+const request = net.request('https://github.com');
+request.setHeader('Some-Custom-Header-Name', 'Some-Custom-Header-Value');
+const header = request.getHeader('Some-Custom-Header-Name');
+console.log('header', header);
+request.removeHeader('Some-Custom-Header-Name');
+request.on('response', (response) => {
+  console.log(`Status code: ${response.statusCode}`);
+  console.log(`Status message: ${response.statusMessage}`);
+  console.log(`Headers: ${JSON.stringify(response.headers)}`);
+  console.log(`Http version: ${response.httpVersion}`);
+  console.log(`Major Http version: ${response.httpVersionMajor}`);
+  console.log(`Minor Http version: ${response.httpVersionMinor}`);
+  response.on('data', (chunk) => {
+    console.log(`BODY: ${chunk}`);
+  });
+  response.on('end', () => {
+    console.log('No more data in response.');
+  });
+  response.on('error', () => {
+    console.log('"error" event emitted');
+  });
+  response.on('aborted', () => {
+    console.log('"aborted" event emitted');
+  });
+});
+request.on('login', (authInfo, callback) => {
+  callback('username', 'password');
+});
+request.on('finish', () => {
+  console.log('"finish" event emitted');
+});
+request.on('abort', () => {
+  console.log('"abort" event emitted');
+});
+request.on('error', () => {
+  console.log('"error" event emitted');
+});
+request.write('Hello World!', 'utf-8');
+request.end('Hello World!', 'utf-8');
+request.abort();
+
+// systemPreferences
+// https://github.com/electron/electron/blob/main/docs/api/system-preferences.md
+
+if (process.platform === 'win32') {
+  systemPreferences.on('color-changed', () => { console.log('color changed'); });
+}
+
+if (process.platform === 'darwin') {
+  const value = systemPreferences.getUserDefault('Foo', 'string');
+  console.log(value);
+  const value2 = systemPreferences.getUserDefault('Foo', 'boolean');
+  console.log(value2);
+}

filename: spec/ts-smoke/tsconfig.json
patch: @@ -15,6 +15,7 @@
   "files": [
     "electron/main.ts",
     "electron/renderer.ts",
+    "electron/utility.ts",
     "../../electron.d.ts"
   ]
 }
\ No newline at end of file