helloadhavan/github_issues huggingface dataset:
Repository: denoland/deno
Pull request: fix(ext/process): suppress child process kill errors in onAbort handler



Co-authored-by: Divy Srivastava <dj.srivastava23@gmail.com>

Closes https://github.com/denoland/deno/issues/27112
Closes https://github.com/denoland/deno/pull/27641
Closes https://github.com/denoland/deno/pull/27124

Changed files:
filename: ext/process/40_process.js
patch: @@ -299,9 +299,14 @@ class ChildProcess {
       this.#stderr = readableStreamForRidUnrefable(stderrRid);
     }
 
-    const onAbort = () => this.kill("SIGTERM");
+    const onAbort = () => {
+      try {
+        this.kill("SIGTERM");
+      } catch {
+        // Ignore the error for https://github.com/denoland/deno/issues/27112
+      }
+    };
     signal?.[abortSignal.add](onAbort);
-
     const waitPromise = op_spawn_wait(this.#rid);
     this.#waitPromise = waitPromise;
     this.#status = PromisePrototypeThen(waitPromise, (res) => {

filename: tests/unit/command_test.ts
patch: @@ -1093,3 +1093,21 @@ Deno.test(
     assertEquals(child.success, true);
   },
 );
+
+Deno.test(
+  { ignore: Deno.build.os === "windows" },
+  async function abortChildProcessRightWhenItExitsShouldNotThrow() {
+    const controller = new AbortController();
+    const cb = () => controller.abort();
+    Deno.addSignalListener("SIGCHLD", cb);
+    const output = await new Deno.Command("true", { signal: controller.signal })
+      .output();
+    assertEquals(output.success, true);
+    assertEquals(output.code, 0);
+    assertEquals(output.signal, null);
+    assertEquals(output.stdout, new Uint8Array());
+    assertEquals(output.stderr, new Uint8Array());
+
+    Deno.removeSignalListener("SIGCHLD", cb);
+  },
+);
Repository: vercel/next.js
Pull request: [devtool] dynamically import segment explorer when enabled

Exclude segment explorer bundle when it's not enabled.

There's an issue in Edge browser on Windows platform, the browser check from mui will crash. It happens when the segment explorer is not enabled. More details described in #81467.
Hence we changed to only import the related modules when segment explorer is enabled to avoid mui being bundled. The proper fix would be bump mui to newer version once [their fix](https://github.com/mui/base-ui/pull/2273) is released.

Fixes #81467

Changed files:
filename: packages/next/src/next-devtools/dev-overlay/components/devtools-panel/devtools-panel-tab/segments-explorer-tab.tsx
patch: @@ -1,4 +1,8 @@
-import { PageSegmentTree } from '../../overview/segment-explorer'
+const PageSegmentTree = process.env.__NEXT_DEVTOOL_SEGMENT_EXPLORER
+  ? (
+      require('../../overview/segment-explorer') as typeof import('../../overview/segment-explorer')
+    ).PageSegmentTree
+  : () => null
 
 function SegmentsExplorer({
   routerType,

filename: packages/next/src/next-devtools/dev-overlay/components/errors/dev-tools-indicator/dev-tools-indicator.tsx
patch: @@ -28,7 +28,6 @@ import {
 } from './dev-tools-info/preferences'
 import { Draggable } from './draggable'
 import { useShortcuts } from '../../../hooks/use-shortcuts'
-import { SegmentsExplorer } from './dev-tools-info/segments-explorer'
 
 // TODO: add E2E tests to cover different scenarios
 
@@ -97,6 +96,15 @@ export type Overlays = (typeof OVERLAYS)[keyof typeof OVERLAYS]
 
 const INDICATOR_PADDING = 20
 
+// Dynamic import for SegmentsExplorer which including base-ui that causing Edge browser crash
+// x-ref: https://github.com/vercel/next.js/pull/64602
+// TODO: remove this once the new base-ui version is released
+const SegmentsExplorer = process.env.__NEXT_DEVTOOL_SEGMENT_EXPLORER
+  ? (
+      require('./dev-tools-info/segments-explorer') as typeof import('./dev-tools-info/segments-explorer')
+    ).SegmentsExplorer
+  : () => null
+
 function DevToolsPopover({
   routerType,
   disabled,
Repository: huggingface/transformers
Pull request: Fix L270 - hasattr("moe_args") returning False error

Hi, I've been trying to convert the Llama-4 weights to HF using the `convert_llama4_weights_to_hf.py` and I've been having an error that causes the `num_experts` to be set to 0 due to the problem in L270 as referenced below:

https://github.com/huggingface/transformers/blame/81799d8b556b3c810ed314187674bc439c0582b4/src/transformers/models/llama4/convert_llama4_weights_to_hf.py#L270

The `if hasattr(params, "moe_args")` in L270 always returns false since "moe_args" is a key of the params dict (not an attribute). I changed the line to `if params.get("moe_args", None):` and the error is gone. I do recommend the change. Thank you.

Tag: @ArthurZucker


Changed files:
filename: src/transformers/models/llama4/convert_llama4_weights_to_hf.py
patch: @@ -267,7 +267,7 @@ def write_model(
 
     num_key_value_heads = params["n_kv_heads"]  # for GQA / MQA
 
-    if hasattr(params, "moe_args"):
+    if params.get("moe_args", False):
         num_experts = params["moe_args"]["num_experts"]
         interleave_moe_layer_step = params["moe_args"].get("interleave_moe_layer_step", 1)
     else:
Repository: electron/electron
Pull request: fix: possible timing issue in utility-process spec

Backport of #45690

See that PR for details.


Notes: none

Changed files:
filename: spec/fixtures/api/utility-process/log.js
patch: @@ -1,3 +1,7 @@
-console.log('hello');
-process.stderr.write('world');
-process.exit(0);
+function write (writable, chunk) {
+  return new Promise((resolve) => writable.write(chunk, resolve));
+}
+
+write(process.stdout, 'hello\n')
+  .then(() => write(process.stderr, 'world'))
+  .then(() => process.exit(0));
Repository: nodejs/node
Pull request: doc: move typescript support to active development

`--experimental-strip-types` has been released for almost ~3 months.
There is only one issue open and TypeScript has created a [flag for compatibility](https://devblogs.microsoft.com/typescript/announcing-typescript-5-7-beta/) with our implementation.

Changed files:
filename: doc/api/cli.md
patch: @@ -983,7 +983,7 @@ JavaScript.
 added: v22.7.0
 -->
 
-> Stability: 1.0 - Early development
+> Stability: 1.1 - Active development
 
 Enables the transformation of TypeScript-only syntax into JavaScript code.
 Implies `--experimental-strip-types` and `--enable-source-maps`.
@@ -1121,7 +1121,7 @@ Enable the experimental [`node:sqlite`][] module.
 added: v22.6.0
 -->
 
-> Stability: 1.0 - Early development
+> Stability: 1.1 - Active development
 
 Enable experimental type-stripping for TypeScript files.
 For more information, see the [TypeScript type-stripping][] documentation.

filename: doc/api/module.md
patch: @@ -358,7 +358,7 @@ resolution and loading behavior. See [Customization hooks][].
 added: REPLACEME
 -->
 
-> Stability: 1.0 - Early development
+> Stability: 1.1 - Active development
 
 * `code` {string} The code to strip type annotations from.
 * `options` {Object}

filename: doc/api/process.md
patch: @@ -1997,7 +1997,7 @@ added:
  - v22.10.0
 -->
 
-> Stability: 1.0 - Early development
+> Stability: 1.1 - Active development
 
 * {boolean|string}
 

filename: doc/api/typescript.md
patch: @@ -7,7 +7,7 @@ changes:
     description: Added `--experimental-transform-types` flag.
 -->
 
-> Stability: 1.0 - Early development
+> Stability: 1.1 - Active development
 
 ## Enabling
 
@@ -50,7 +50,7 @@ To use TypeScript with full support for all TypeScript features, including
 added: v22.6.0
 -->
 
-> Stability: 1.0 - Early development
+> Stability: 1.1 - Active development
 
 The flag [`--experimental-strip-types`][] enables Node.js to run TypeScript
 files. By default Node.js will execute only files that contain no
Repository: facebook/react
Pull request: Fix check_error_codes CI job

The diff command was missing a --quiet argument, causing the job not to fail when unminified messages were found.

This commit will cause CI to fail because there's an unminified error in main. The unminified error will be fixed by #24688.

Changed files:
filename: .circleci/config.yml
patch: @@ -329,7 +329,7 @@ jobs:
           name: Search build artifacts for unminified errors
           command: |
             yarn extract-errors
-            git diff || (echo "Found unminified errors. Either update the error codes map or disable error minification for the affected build, if appropriate." && false)
+            git diff --quiet || (echo "Found unminified errors. Either update the error codes map or disable error minification for the affected build, if appropriate." && false)
 
   yarn_test:
     docker: *docker
Repository: huggingface/transformers
Pull request: [chat template] add a testcase for kwargs

# What does this PR do?

Just adds a testcase, I didn't know this was supported and I don't want it to break in the future

Changed files:
filename: tests/test_processing_common.py
patch: @@ -1110,3 +1110,42 @@ def test_chat_template_audio_from_video(self):
         self.assertEqual(len(out_dict["attention_mask"]), 1)  # batch-size=1
         self.assertEqual(len(out_dict[self.audio_input_name]), 1)  # 1 audio in the conversation
         self.assertEqual(len(out_dict[self.videos_input_name]), 1)  # 1 video in the conversation
+
+    def test_chat_template_jinja_kwargs(self):
+        """Tests that users can pass any kwargs and they will be used in jinja templates."""
+        processor = self.get_processor()
+        if processor.chat_template is None:
+            self.skipTest("Processor has no chat template")
+
+        messages = [
+            {
+                "role": "user",
+                "content": [
+                    {"type": "text", "text": "Which of these animals is making the sound?"},
+                ],
+            },
+            {
+                "role": "assistant",
+                "content": [{"type": "text", "text": "It is a cow."}],
+            },
+        ]
+
+        dummy_template = (
+            "{% for message in messages %}"
+            "{% if add_system_prompt %}"
+            "{{'You are a helpful assistant.'}}"
+            "{% endif %}"
+            "{% if (message['role'] != 'assistant') %}"
+            "{{'
' + message['role'] + '\n' + message['content'][0]['text'] + '
' + '\n'}}"
+            "{% elif (message['role'] == 'assistant')%}"
+            "{{'
' + message['role'] + '\n'}}"
+            "{{message['content'][0]['text'] + '
' + '\n'}}"
+            "{% endif %}"
+            "{% endfor %}"
+        )
+
+        formatted_prompt = processor.apply_chat_template(
+            messages, add_system_prompt=True, tokenize=False, chat_template=dummy_template
+        )
+        expected_prompt = "You are a helpful assistant.
user\nWhich of these animals is making the sound?
\nYou are a helpful assistant.
assistant\nIt is a cow.
\n"
+        self.assertEqual(formatted_prompt, expected_prompt)
Repository: denoland/deno
Pull request: fix(ext/node): `getColorDepth` for writable stdio streams

Fixes https://github.com/denoland/deno/issues/28605
Fixes https://github.com/denoland/deno/issues/29175

` ../deno/target/debug/deno run -A npm:create-vue@latest DIR` works now

Changed files:
filename: ext/node/polyfills/_process/streams.mjs
patch: @@ -23,6 +23,7 @@ import {
 import { Duplex, Readable, Writable } from "node:stream";
 import * as io from "ext:deno_io/12_io.js";
 import { guessHandleType } from "ext:deno_node/internal_binding/util.ts";
+import { op_bootstrap_color_depth } from "ext:core/ops";
 
 // https://github.com/nodejs/node/blob/00738314828074243c9a52a228ab4c68b04259ef/lib/internal/bootstrap/switches/is_main_thread.js#L41
 export function createWritableStdioStream(writer, name, warmup = false) {
@@ -103,6 +104,13 @@ export function createWritableStdioStream(writer, name, warmup = false) {
       value: () =>
         writer?.isTerminal() ? ObjectValues(Deno.consoleSize?.()) : undefined,
     },
+    getColorDepth: {
+      __proto__: null,
+      enumerable: true,
+      configurable: true,
+      writable: true,
+      value: () => op_bootstrap_color_depth(),
+    },
   });
 
   // If we're warming up, create a stdout/stderr stream that assumes a terminal (the most likely case).

filename: tests/unit_node/process_test.ts
patch: @@ -657,6 +657,7 @@ Deno.test({
     const consoleSize = isTTY ? Deno.consoleSize() : undefined;
     assertEquals(process.stdout.columns, consoleSize?.columns);
     assertEquals(process.stdout.rows, consoleSize?.rows);
+    assert([1, 4, 8, 24].includes(process.stdout.getColorDepth()));
     assertEquals(
       `${process.stdout.getWindowSize()}`,
       `${consoleSize && [consoleSize.columns, consoleSize.rows]}`,
Repository: denoland/deno
Pull request: fix: terminate control sock event with newline

these should be newline terminated

Changed files:
filename: cli/main.rs
patch: @@ -686,7 +686,8 @@ fn wait_for_start(
         Serving,
       }
 
-      let buf = deno_core::serde_json::to_vec(&Event::Serving).unwrap();
+      let mut buf = deno_core::serde_json::to_vec(&Event::Serving).unwrap();
+      buf.push(b'\n');
       let _ = tx.write_all(&buf).await;
     });
 

filename: tests/specs/run/unconfigured/main.out
patch: @@ -1,4 +1,5 @@
 true
 hello world
 EVENT: "Serving"
+
 [Object: null prototype] { success: true, code: 0, signal: null }
Repository: denoland/deno
Pull request: feat(cli): `deno check` maps to `deno check .`

This PR implements the suggestion given in the comment https://github.com/denoland/deno/issues/28455#issuecomment-2734956368

Now `deno check` without arguments maps to `deno check .`

Changed files:
filename: cli/args/flags.rs
patch: @@ -1939,7 +1939,6 @@ Unless --reload is specified, this command will not re-download already cached d
         .arg(
           Arg::new("file")
             .num_args(1..)
-            .required_unless_present("help")
             .value_hint(ValueHint::FilePath),
         )
         .arg(frozen_lockfile_arg())
@@ -4681,7 +4680,10 @@ fn check_parse(
   compile_args_without_check_parse(flags, matches)?;
   unstable_args_parse(flags, matches, UnstableArgsConfig::ResolutionAndRuntime);
   frozen_lockfile_arg_parse(flags, matches);
-  let files = matches.remove_many::<String>("file").unwrap().collect();
+  let files = match matches.remove_many::<String>("file") {
+    Some(f) => f.collect(),
+    None => vec![".".to_string()], // default
+  };
   if matches.get_flag("all") || matches.get_flag("remote") {
     flags.type_check_mode = TypeCheckMode::All;
   }
@@ -7576,6 +7578,21 @@ mod tests {
       }
     );
 
+    let r = flags_from_vec(svec!["deno", "check"]);
+    assert_eq!(
+      r.unwrap(),
+      Flags {
+        subcommand: DenoSubcommand::Check(CheckFlags {
+          files: svec!["."],
+          doc: false,
+          doc_only: false,
+        }),
+        type_check_mode: TypeCheckMode::Local,
+        code_cache_enabled: true,
+        ..Flags::default()
+      }
+    );
+
     let r = flags_from_vec(svec!["deno", "check", "--doc", "script.ts"]);
     assert_eq!(
       r.unwrap(),
Repository: vercel/next.js
Pull request: Add @sparticuz/chromium-min to default server externals

This package needs to be externalized to have it's wasm dependencies traced properly for node runtime so this adds it to the default list.

Changed files:
filename: docs/01-app/03-api-reference/05-config/01-next-config-js/serverExternalPackages.mdx
patch: @@ -34,6 +34,7 @@ Next.js includes a [short list of popular packages](https://github.com/vercel/ne
 - `@react-pdf/renderer`
 - `@sentry/profiling-node`
 - `@sparticuz/chromium`
+- `@sparticuz/chromium-min`
 - `@swc/core`
 - `@xenova/transformers`
 - `argon2`

filename: docs/02-pages/04-api-reference/04-config/01-next-config-js/serverExternalPackages.mdx
patch: @@ -34,6 +34,7 @@ Next.js includes a [short list of popular packages](https://github.com/vercel/ne
 - `@react-pdf/renderer`
 - `@sentry/profiling-node`
 - `@sparticuz/chromium`
+- `@sparticuz/chromium-min`
 - `@swc/core`
 - `@xenova/transformers`
 - `argon2`

filename: packages/next/src/lib/server-external-packages.json
patch: @@ -15,6 +15,7 @@
   "@react-pdf/renderer",
   "@sentry/profiling-node",
   "@sparticuz/chromium",
+  "@sparticuz/chromium-min",
   "@swc/core",
   "@xenova/transformers",
   "argon2",
Repository: vercel/next.js
Pull request: fix: referencing unavailable __dirname

Patch missing __dirname usage for storybook css injection

Changed files:
filename: packages/next/.storybook/main.ts
patch: @@ -1,5 +1,7 @@
 import type { StorybookConfig } from '@storybook/react-webpack5'
 import { join, dirname, resolve } from 'path'
+import { fileURLToPath } from 'url'
+
 /**
  * This function is used to resolve the absolute path of a package.
  * It is needed in projects that use Yarn PnP or are set up within a monorepo.
@@ -65,7 +67,7 @@ const config: StorybookConfig = {
           options: {
             injectType: 'styleTag',
             insert: resolve(
-              __dirname,
+              dirname(fileURLToPath(import.meta.url)),
               '../src/build/webpack/loaders/devtool/devtool-style-inject.js'
             ),
           },
Repository: denoland/deno
Pull request: fix(ext/node): Fix dns.lookup when promisified with `options.all`

Fixes https://github.com/denoland/deno/issues/29074

<!--
Before submitting a PR, please read https://docs.deno.com/runtime/manual/references/contributing

1. Give the PR a descriptive title.

  Examples of good title:
    - fix(std/http): Fix race condition in server
    - docs(console): Update docstrings
    - feat(doc): Handle nested reexports

  Examples of bad title:
    - fix #7123
    - update docs
    - fix bugs

2. Ensure there is a related issue and it is referenced in the PR text.
3. Ensure there are tests that cover the changes.
4. Ensure `cargo test` passes.
5. Ensure `./tools/format.js` passes without changing files.
6. Ensure `./tools/lint.js` passes.
7. Open as a draft PR if your work is still in progress. The CI won't run
   all steps, but you can add '[ci]' to a commit message to force it to.
8. If you would like to run the benchmarks on the CI, add the 'ci-bench' label.
-->


Changed files:
filename: ext/node/polyfills/dns.ts
patch: @@ -133,7 +133,11 @@ function onlookupall(
     };
   }
 
-  this.callback(null, parsedAddresses, undefined, netPermToken);
+  if (this.callback.length > 1) {
+    this.callback(null, parsedAddresses, undefined, netPermToken);
+  } else {
+    this.callback(null, parsedAddresses);
+  }
 }
 
 type LookupCallback = (

filename: tests/unit_node/net_test.ts
patch: @@ -5,6 +5,7 @@ import { assert, assertEquals } from "@std/assert";
 import * as path from "@std/path";
 import * as http from "node:http";
 import * as dns from "node:dns";
+import util from "node:util";
 import console from "node:console";
 
 Deno.test("[node/net] close event emits after error event - when host is not found", async () => {
@@ -274,3 +275,11 @@ Deno.test("dns.resolve with ttl", async () => {
   assert(ret2.length > 0);
   assert(typeof ret2[0] === "string");
 });
+
+Deno.test("[node/dns] dns.lookup (all=true) util promisify", async () => {
+  const lookup = util.promisify(dns.lookup);
+  const result = await lookup("localhost", { all: true });
+  assert(Array.isArray(result));
+  assert(typeof result[0].address === "string");
+  assert(typeof result[0].family === "number");
+});
Repository: vercel/next.js
Pull request: Revert "[test] Skip flaky cdn-cache-busting test"

Reverts vercel/next.js#81429

The test should™ not be flaky anymore after merging #80967. 

Changed files:
filename: test/e2e/app-dir/segment-cache/cdn-cache-busting/cdn-cache-busting.test.ts
patch: @@ -101,8 +101,7 @@ describe('segment cache (CDN cache busting)', () => {
     }
   )
 
-  // 50% failure rate of this test in CI.
-  it.skip(
+  it(
     'perform fully prefetched navigation when a third-party proxy ' +
       'performs a redirect',
     async () => {
Repository: ggml-org/llama.cpp
Pull request: Document the new max GPU layers default in help

This is a key change, just letting users know.


Changed files:
filename: common/arg.cpp
patch: @@ -2466,7 +2466,7 @@ common_params_context common_params_parser_init(common_params & params, llama_ex
     ).set_examples({LLAMA_EXAMPLE_SPECULATIVE, LLAMA_EXAMPLE_SERVER}).set_env("LLAMA_ARG_N_CPU_MOE_DRAFT"));
     add_opt(common_arg(
         {"-ngl", "--gpu-layers", "--n-gpu-layers"}, "N",
-        "number of layers to store in VRAM",
+        string_format("max. number of layers to store in VRAM (default: %d)", params.n_gpu_layers),
         [](common_params & params, int value) {
             params.n_gpu_layers = value;
             if (!llama_supports_gpu_offload()) {
Repository: denoland/deno
Pull request: fix(fmt): upgrade markup_fmt and malva

Closes https://github.com/denoland/deno/issues/29034

Changed files:
filename: Cargo.lock
patch: @@ -5334,9 +5334,9 @@ dependencies = [
 
 [[package]]
 name = "malva"
-version = "0.11.2"
+version = "0.12.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6a952f521471c6c8302a17fc0c64a512221a9d4ff268af4d43a02bc2be48c712"
+checksum = "5a10a4f69b1fd3c73e65e9b0446904f60019b7a746e10440d89fbb6e1659144b"
 dependencies = [
  "aho-corasick",
  "itertools 0.14.0",
@@ -5347,9 +5347,9 @@ dependencies = [
 
 [[package]]
 name = "markup_fmt"
-version = "0.19.1"
+version = "0.20.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ff37c5cadfdf4d91c0f140d516755c055a184a4e37757ff4a071272635b9f546"
+checksum = "7fbf2254c47336b12f41d91da52d985d6e5242f26e41b7cd48bf658f3446d959"
 dependencies = [
  "aho-corasick",
  "css_dataset",

filename: Cargo.toml
patch: @@ -296,8 +296,8 @@ env_logger = "=0.11.6"
 eszip = "=0.87.0"
 fancy-regex = "=0.14.0"
 libsui = "0.10.0"
-malva = "=0.11.2"
-markup_fmt = "=0.19.1"
+malva = "=0.12.0"
+markup_fmt = "=0.20.0"
 open = "5.0.1"
 pathdiff = "0.2.1"
 pretty_yaml = "=0.5.0"

filename: cli/tools/fmt.rs
patch: @@ -618,6 +618,8 @@ fn format_embedded_css(
       selector_override_comment_directive: "malva-selector-override".into(),
       ignore_comment_directive: "malva-ignore".into(),
       ignore_file_comment_directive: "malva-ignore-file".into(),
+      declaration_order_group_by:
+        config::DeclarationOrderGroupBy::NonDeclarationAndEmptyLine,
     },
   };
   // Wraps the text in a css block of `a { ... }`
@@ -711,6 +713,7 @@ fn format_embedded_html(
       script_formatter: None,
       ignore_comment_directive: "deno-fmt-ignore".into(),
       ignore_file_comment_directive: "deno-fmt-ignore-file".into(),
+      single_attr_same_line: true,
     },
   };
   let Ok(text) = markup_fmt::format_text(
@@ -1463,6 +1466,8 @@ fn get_resolved_malva_config(
     selector_override_comment_directive: "deno-fmt-selector-override".into(),
     ignore_comment_directive: "deno-fmt-ignore".into(),
     ignore_file_comment_directive: "deno-fmt-ignore-file".into(),
+    declaration_order_group_by:
+      DeclarationOrderGroupBy::NonDeclarationAndEmptyLine,
   };
 
   FormatOptions {
@@ -1523,6 +1528,7 @@ fn get_resolved_markup_fmt_config(
     astro_attr_shorthand: Some(true),
     ignore_comment_directive: "deno-fmt-ignore".into(),
     ignore_file_comment_directive: "deno-fmt-ignore-file".into(),
+    single_attr_same_line: true,
   };
 
   FormatOptions {
Repository: nodejs/node
Pull request: dns: restore dns query cache ttl

Fixes: https://github.com/nodejs/node/issues/57636 

When c-ares was [updated](https://github.com/nodejs/node/commit/c5174f5e60) the default caching behavior changed.

> As of c-ares 1.31.0, the query cache is enabled by default with a TTL of 1hr. To disable the query cache, specify this option with a TTL of 0.  The query cache is based on the returned TTL in the DNS message.

This PR restores the caching behavior by setting `qcache_max_ttl` to `0`.

This change in c-ares is reasonable; thus, as a follow up to this _fix_, we should implement a cache management API for DNS queries and then re-enable the new default cache value. Tracking issue for follow up work: https://github.com/nodejs/node/issues/57641

Changed files:
filename: src/cares_wrap.cc
patch: @@ -778,6 +778,7 @@ void ChannelWrap::Setup() {
   options.sock_state_cb_data = this;
   options.timeout = timeout_;
   options.tries = tries_;
+  options.qcache_max_ttl = 0;
 
   int r;
   if (!library_inited_) {