helloadhavan/github_issues huggingface dataset:
Repository: vercel/next.js
Pull request: Turbopack: respect PURE comments for minification

Previously, when Turbopack invoked the swc minifier, comments were ignored, so code explicitly marked as side-effect free was still not removed

```js
const unused = /*@__PURE__*/ (() => {
  state++
})()
```
(same for unused JSX as well)

This makes the output smaller

Closes PACK-4910

Changed files:
filename: turbopack/crates/turbopack-ecmascript/src/minify.rs
patch: @@ -43,11 +43,14 @@ pub fn minify(code: Code, source_maps: bool, mangle: Option<MangleType>) -> Resu
     let (src, mut src_map_buf) = {
         let fm = cm.new_source_file(FileName::Anon.into(), source_code);
 
+        // Collect all comments and pass to the minifier so that `PURE` comments are respected.
+        let comments = SingleThreadedComments::default();
+
         let lexer = Lexer::new(
             Syntax::default(),
             EsVersion::latest(),
             StringInput::from(&*fm),
-            None,
+            Some(&comments),
         );
         let mut parser = Parser::new_from(lexer);
 
@@ -60,7 +63,6 @@ pub fn minify(code: Code, source_maps: bool, mangle: Option<MangleType>) -> Resu
                         bail!("failed to parse source code\n{}", fm.src)
                     }
                 };
-                let comments = SingleThreadedComments::default();
                 let unresolved_mark = Mark::new();
                 let top_level_mark = Mark::new();
 

filename: turbopack/crates/turbopack-tests/tests/execution.rs
patch: @@ -30,7 +30,7 @@ use turbopack::{
     module_options::{EcmascriptOptionsContext, ModuleOptionsContext, TypescriptTransformOptions},
 };
 use turbopack_core::{
-    chunk::ChunkingConfig,
+    chunk::{ChunkingConfig, MangleType, MinifyType},
     compile_time_defines,
     compile_time_info::CompileTimeInfo,
     condition::ContextCondition,
@@ -243,6 +243,8 @@ struct TestOptions {
     tree_shaking_mode: Option<TreeShakingMode>,
     remove_unused_exports: Option<bool>,
     scope_hoisting: Option<bool>,
+    #[serde(default)]
+    minify: bool,
 }
 
 #[turbo_tasks::value]
@@ -437,6 +439,13 @@ async fn run_test_operation(prepared_test: ResolvedVc<PreparedTest>) -> Result<V
         },
     )
     .module_merging(options.scope_hoisting.unwrap_or(true))
+    .minify_type(if options.minify {
+        MinifyType::Minify {
+            mangle: Some(MangleType::OptimalSize),
+        }
+    } else {
+        MinifyType::NoMinify
+    })
     .build();
 
     let jest_entry_source = FileSource::new(jest_entry_path);

filename: turbopack/crates/turbopack-tests/tests/execution/turbopack/minification/pure/input/index.js
patch: @@ -0,0 +1,9 @@
+let state = 0
+
+const unused = /*@__PURE__*/ (() => {
+  state++
+})()
+
+it('should remove unused PURE statements', () => {
+  expect(state).toBe(0)
+})

filename: turbopack/crates/turbopack-tests/tests/execution/turbopack/minification/pure/options.json
patch: @@ -0,0 +1,3 @@
+{
+  "minify": true
+}
Repository: denoland/deno
Pull request: fix(unstable): lint plugin `!==` wrongly parsed as `!=`

That's an emberassing typo. The `!==` operator was wrongly converted to `!=`.

Fixes https://github.com/denoland/deno/issues/28397

Changed files:
filename: cli/tools/lint/ast_buffer/swc.rs
patch: @@ -716,7 +716,7 @@ fn serialize_expr(ctx: &mut TsEsTreeBuilder, expr: &Expr) -> NodeRef {
         BinaryOp::EqEq => (AstNode::BinaryExpression, "=="),
         BinaryOp::NotEq => (AstNode::BinaryExpression, "!="),
         BinaryOp::EqEqEq => (AstNode::BinaryExpression, "==="),
-        BinaryOp::NotEqEq => (AstNode::BinaryExpression, "!="),
+        BinaryOp::NotEqEq => (AstNode::BinaryExpression, "!=="),
         BinaryOp::Lt => (AstNode::BinaryExpression, "<"),
         BinaryOp::LtEq => (AstNode::BinaryExpression, "<="),
         BinaryOp::Gt => (AstNode::BinaryExpression, ">"),

filename: tests/unit/__snapshots__/lint_plugin_test.ts.snap
patch: @@ -2647,7 +2647,7 @@ snapshot[`Plugin - BinaryExpression 8`] = `
     type: "Identifier",
     typeAnnotation: undefined,
   },
-  operator: "!=",
+  operator: "!==",
   range: [
     0,
     7,
Repository: facebook/react
Pull request: export dynamic disableSchedulerTimeoutInWorkLoop flag in www

## Summary

This was not exposed as a dynamic flag in the build for facebook www. By adding it, we'll be able to roll this out incrementally before cleaning up this code altogether.

## How did you test this change?

`yarn build`

Before changes, `disableSchedulerTimeoutInWorkLoop` flag is not included in ReactDOM-* build output for facebook www. Afterwards, it is included.


Changed files:
filename: packages/shared/forks/ReactFeatureFlags.www.js
patch: @@ -31,6 +31,7 @@ export const {
   enableAsyncActions,
   alwaysThrottleRetries,
   enableDO_NOT_USE_disableStrictPassiveEffect,
+  disableSchedulerTimeoutInWorkLoop,
 } = dynamicFeatureFlags;
 
 // On WWW, __EXPERIMENTAL__ is used for a new modern build.
@@ -48,7 +49,6 @@ export const createRootStrictEffectsByDefault = false;
 export const enableSuspenseAvoidThisFallback = true;
 export const enableSuspenseAvoidThisFallbackFizz = false;
 
-export const disableSchedulerTimeoutInWorkLoop = false;
 export const enableCPUSuspense = true;
 export const enableFloat = true;
 export const enableUseMemoCacheHook = true;
Repository: vercel/next.js
Pull request: Update Turbopack production test manifest

This auto-generated PR updates the production integration test manifest used when testing Turbopack.

Changed files:
filename: test/turbopack-build-tests-manifest.json
patch: @@ -4032,6 +4032,7 @@
       "parallel-routes-and-interception route intercepting with dynamic catch-all routes should render intercepted route",
       "parallel-routes-and-interception route intercepting with dynamic optional catch-all routes should render intercepted route",
       "parallel-routes-and-interception route intercepting with dynamic routes should render intercepted route",
+      "parallel-routes-and-interception route intercepting with prerendered dynamic routes  should render intercepted route",
       "parallel-routes-and-interception-conflicting-pages should gracefully handle when two page segments match the `children` parallel slot"
     ],
     "failed": [],
@@ -5484,6 +5485,7 @@
       "segment cache (basic tests) navigate to page with lazily-generated (not at build time) static param",
       "segment cache (basic tests) navigate with prefetched data",
       "segment cache (basic tests) prefetch interception route",
+      "segment cache (basic tests) prefetch interception route with params",
       "segment cache (basic tests) refreshes page segments when navigating to the exact same URL as the current location",
       "segment cache (basic tests) skips dynamic request if prefetched data is fully static",
       "segment cache (basic tests) skips static layouts during partially static navigation"
@@ -5497,7 +5499,7 @@
     "passed": [
       "segment cache (CDN cache busting) perform fully prefetched navigation when a third-party proxy performs a redirect",
       "segment cache (CDN cache busting) perform fully prefetched navigation with a CDN that doesn't respect the Vary header",
-      "segment cache (CDN cache busting) prevent cache poisoning attacks by responding with an error if a custom header is sent during a prefetch without a corresponding cache-busting search param"
+      "segment cache (CDN cache busting) prevent cache poisoning attacks by responding with a redirect to correct cache busting query param if a custom header is sent during a prefetch without a corresponding cache-busting search param"
     ],
     "failed": [],
     "pending": [],
Repository: huggingface/transformers
Pull request: [fix] Add FastSpeech2ConformerWithHifiGan

Implements fix in this [comment](https://github.com/huggingface/transformers/pull/37377#discussion_r2070820456) to enable `FastSpeech2ConformerWithHiFiGan`

Changed files:
filename: src/transformers/models/auto/configuration_auto.py
patch: @@ -130,6 +130,7 @@
         ("falcon_h1", "FalconH1Config"),
         ("falcon_mamba", "FalconMambaConfig"),
         ("fastspeech2_conformer", "FastSpeech2ConformerConfig"),
+        ("fastspeech2_conformer_with_hifigan", "FastSpeech2ConformerWithHifiGanConfig"),
         ("flaubert", "FlaubertConfig"),
         ("flava", "FlavaConfig"),
         ("fnet", "FNetConfig"),
@@ -511,6 +512,7 @@
         ("falcon_h1", "FalconH1"),
         ("falcon_mamba", "FalconMamba"),
         ("fastspeech2_conformer", "FastSpeech2Conformer"),
+        ("fastspeech2_conformer_with_hifigan", "FastSpeech2ConformerWithHifiGan"),
         ("flan-t5", "FLAN-T5"),
         ("flan-ul2", "FLAN-UL2"),
         ("flaubert", "FlauBERT"),
@@ -866,6 +868,7 @@
         ("sam_hq_vision_model", "sam_hq"),
         ("llama4_text", "llama4"),
         ("blip_2_qformer", "blip_2"),
+        ("fastspeech2_conformer_with_hifigan", "fastspeech2_conformer"),
     ]
 )
 
@@ -1178,7 +1181,8 @@ def from_pretrained(cls, pretrained_model_name_or_path: Union[str, os.PathLike[s
 
         >>> unused_kwargs
         {'foo': False}
-        ```"""
+        ```
+        """
         use_auth_token = kwargs.pop("use_auth_token", None)
         if use_auth_token is not None:
             warnings.warn(

filename: src/transformers/models/auto/modeling_auto.py
patch: @@ -121,6 +121,7 @@
         ("falcon_h1", "FalconH1Model"),
         ("falcon_mamba", "FalconMambaModel"),
         ("fastspeech2_conformer", "FastSpeech2ConformerModel"),
+        ("fastspeech2_conformer_with_hifigan", "FastSpeech2ConformerWithHifiGan"),
         ("flaubert", "FlaubertModel"),
         ("flava", "FlavaModel"),
         ("fnet", "FNetModel"),
@@ -1512,6 +1513,7 @@
         ("bark", "BarkModel"),
         ("csm", "CsmForConditionalGeneration"),
         ("fastspeech2_conformer", "FastSpeech2ConformerWithHifiGan"),
+        ("fastspeech2_conformer_with_hifigan", "FastSpeech2ConformerWithHifiGan"),
         ("musicgen", "MusicgenForConditionalGeneration"),
         ("musicgen_melody", "MusicgenMelodyForConditionalGeneration"),
         ("qwen2_5_omni", "Qwen2_5OmniForConditionalGeneration"),
Repository: ggml-org/llama.cpp
Pull request: ci: Enable GGML_CPU_ALL_VARIANTS for ARM

I now have an Orion O6 which has an ARMv9.2 CPU. On this host, I booted up QEMU VMs with `-cpu host` and in it, built a Debug build with `GGML_BACKEND_DL=ON` and `GGML_CPU_ALL_VARIANTS=ON`.

I then booted up the VM repeatedly with various feature flags disabled, and observed the correct backend being loaded.

I only couldn't test SME because it was not supported by the CPU.

Changed files:
filename: .devops/cpu.Dockerfile
patch: @@ -4,19 +4,15 @@ FROM ubuntu:$UBUNTU_VERSION AS build
 
 ARG TARGETARCH
 
-ARG GGML_CPU_ARM_ARCH=armv8-a
-
 RUN apt-get update && \
     apt-get install -y build-essential git cmake libcurl4-openssl-dev
 
 WORKDIR /app
 
 COPY . .
 
-RUN if [ "$TARGETARCH" = "amd64" ]; then \
+RUN if [ "$TARGETARCH" = "amd64" ] || [ "$TARGETARCH" = "arm64" ]; then \
         cmake -S . -B build -DCMAKE_BUILD_TYPE=Release -DGGML_NATIVE=OFF -DLLAMA_BUILD_TESTS=OFF -DGGML_BACKEND_DL=ON -DGGML_CPU_ALL_VARIANTS=ON; \
-    elif [ "$TARGETARCH" = "arm64" ]; then \
-        cmake -S . -B build -DCMAKE_BUILD_TYPE=Release -DGGML_NATIVE=OFF -DLLAMA_BUILD_TESTS=OFF -DGGML_CPU_ARM_ARCH=${GGML_CPU_ARM_ARCH}; \
     else \
         echo "Unsupported architecture"; \
         exit 1; \
Repository: vercel/next.js
Pull request: Update Turbopack development test manifest

This auto-generated PR updates the development integration test manifest used when testing Turbopack.

Changed files:
filename: test/turbopack-dev-tests-manifest.json
patch: @@ -7885,6 +7885,7 @@
       "parallel-routes-and-interception route intercepting with dynamic catch-all routes should render intercepted route",
       "parallel-routes-and-interception route intercepting with dynamic optional catch-all routes should render intercepted route",
       "parallel-routes-and-interception route intercepting with dynamic routes should render intercepted route",
+      "parallel-routes-and-interception route intercepting with prerendered dynamic routes  should render intercepted route",
       "parallel-routes-and-interception-conflicting-pages should gracefully handle when two page segments match the `children` parallel slot"
     ],
     "failed": [],
@@ -8818,18 +8819,16 @@
     "runtimeError": false
   },
   "test/e2e/app-dir/scss/invalid-global-with-app/invalid-global-with-app.test.ts": {
-    "passed": [],
+    "passed": ["Invalid Global CSS with Custom App should fail to build"],
     "failed": [],
-    "pending": [
-      "Invalid Global CSS with Custom App production only should fail to build"
-    ],
+    "pending": [],
     "flakey": [],
     "runtimeError": false
   },
   "test/e2e/app-dir/scss/invalid-global/invalid-global.test.ts": {
-    "passed": [],
+    "passed": ["Invalid Global CSS should fail to build"],
     "failed": [],
-    "pending": ["Invalid Global CSS production only should fail to build"],
+    "pending": [],
     "flakey": [],
     "runtimeError": false
   },
@@ -9052,11 +9051,11 @@
     "runtimeError": false
   },
   "test/e2e/app-dir/scss/valid-and-invalid-global/valid-and-invalid-global.test.ts": {
-    "passed": [],
-    "failed": [],
-    "pending": [
-      "Valid and Invalid Global CSS with Custom App production only should fail to build"
+    "passed": [
+      "Valid and Invalid Global CSS with Custom App should fail to build"
     ],
+    "failed": [],
+    "pending": [],
     "flakey": [],
     "runtimeError": false
   },
Repository: denoland/deno
Pull request: fix(run): skip the cjs suggestion for mjs/mts modules 

This PR is for solving #26557.
1. A check statement was added to verify if the file ends with .mjs or .mts.
2. If this condition is true, the CJS suggestions will be skipped.


Changed files:
filename: runtime/fmt_errors.rs
patch: @@ -309,6 +309,13 @@ fn get_suggestions_for_terminal_errors(e: &JsError) -> Vec<FixSuggestion> {
       || msg.contains("exports is not defined")
       || msg.contains("require is not defined")
     {
+      if let Some(file_name) =
+        e.frames.first().and_then(|f| f.file_name.as_ref())
+      {
+        if file_name.ends_with(".mjs") || file_name.ends_with(".mts") {
+          return vec![];
+        }
+      }
       return vec![
         FixSuggestion::info_multiline(&[
           cstr!("Deno supports CommonJS modules in <u>.cjs</> files, or when the closest"),

filename: tests/specs/run/no_cjs_suggestion_mts_file/__test__.jsonc
patch: @@ -0,0 +1,5 @@
+{
+  "args": "run --allow-read main.mts",
+  "exitCode": 1,
+  "output": "main.out"
+}

filename: tests/specs/run/no_cjs_suggestion_mts_file/a.cts
patch: @@ -0,0 +1,6 @@
+function add(num1, num2) {
+  const result = num1 + num2;
+  return result;
+}
+
+module.exports = { add };

filename: tests/specs/run/no_cjs_suggestion_mts_file/main.mts
patch: @@ -0,0 +1,5 @@
+import * as a from "./a.cts";
+
+console.log(a.add(1, 2));
+
+module.isPreloading = true;

filename: tests/specs/run/no_cjs_suggestion_mts_file/main.out
patch: @@ -0,0 +1,5 @@
+3
+error: Uncaught (in promise) ReferenceError: module is not defined
+module.isPreloading = true;
+^
+    at file:///[WILDLINE]/main.mts:5:1
Repository: facebook/react
Pull request: Cleanup unused unstable_startTransition, unstable_useTransition, unstable_useDeferredValue exports from fb packages

## Summary

came across these TODOs – an internal grep indicated that remaining callsites have been cleaned up, so these can now be removed.

## How did you test this change?

```
yarn flow dom-browser
yarn test
```


Changed files:
filename: fixtures/concurrent/time-slicing/src/index.js
patch: @@ -1,4 +1,4 @@
-import React, {PureComponent, unstable_startTransition} from 'react';
+import React, {PureComponent, startTransition} from 'react';
 import {createRoot} from 'react-dom/client';
 import _ from 'lodash';
 import Charts from './Charts';
@@ -64,7 +64,7 @@ class App extends PureComponent {
     }
     this._ignoreClick = true;
 
-    unstable_startTransition(() => {
+    startTransition(() => {
       this.setState({showDemo: true}, () => {
         this._ignoreClick = false;
       });
@@ -103,7 +103,7 @@ class App extends PureComponent {
         break;
       case 'async':
         // TODO: useTransition hook instead.
-        unstable_startTransition(() => {
+        startTransition(() => {
           this.setState({value});
         });
         break;

filename: packages/react/index.classic.fb.js
patch: @@ -32,7 +32,6 @@ export {
   memo,
   cache,
   startTransition,
-  startTransition as unstable_startTransition, // TODO: Remove once call sights updated to startTransition
   unstable_Cache,
   unstable_TracingMarker,
   unstable_DebugTracingMode,
@@ -48,7 +47,6 @@ export {
   useContext,
   useDebugValue,
   useDeferredValue,
-  useDeferredValue as unstable_useDeferredValue, // TODO: Remove once call sights updated to useDeferredValue
   useEffect,
   experimental_useEffectEvent,
   useImperativeHandle,
@@ -61,7 +59,6 @@ export {
   useState,
   useSyncExternalStore,
   useTransition,
-  useTransition as unstable_useTransition, // TODO: Remove once call sights updated to useTransition
   version,
 } from './src/React';
 export {jsx, jsxs, jsxDEV} from './src/jsx/ReactJSX';

filename: packages/react/index.modern.fb.js
patch: @@ -31,7 +31,6 @@ export {
   memo,
   cache,
   startTransition,
-  startTransition as unstable_startTransition, // TODO: Remove once call sights updated to startTransition
   unstable_Cache,
   unstable_DebugTracingMode,
   unstable_LegacyHidden,
@@ -46,7 +45,6 @@ export {
   useContext,
   useDebugValue,
   useDeferredValue,
-  useDeferredValue as unstable_useDeferredValue, // TODO: Remove once call sights updated to useDeferredValue
   useEffect,
   experimental_useEffectEvent,
   useImperativeHandle,
@@ -59,7 +57,6 @@ export {
   useState,
   useSyncExternalStore,
   useTransition,
-  useTransition as unstable_useTransition, // TODO: Remove once call sights updated to useTransition
   version,
 } from './src/React';
 export {jsx, jsxs, jsxDEV} from './src/jsx/ReactJSX';
Repository: huggingface/transformers
Pull request: Fix: unprotected import of tp plugin

Fixes #39077. We only use `TensorParallelPlugin` [here](https://github.com/huggingface/transformers/blob/9c8d3a70b8bf359150c960c4281aaa853498fe8c/src/transformers/trainer.py#L5204) and that is version checked, the matching import that is also version checked is [here](https://github.com/huggingface/transformers/blob/9c8d3a70b8bf359150c960c4281aaa853498fe8c/src/transformers/trainer.py#L242). The import I deleted is non-version checked and not needed.


Changed files:
filename: src/transformers/trainer.py
patch: @@ -230,7 +230,6 @@
         AutocastKwargs,
         DistributedDataParallelKwargs,
         DistributedType,
-        TorchTensorParallelPlugin,
         load_fsdp_model,
         load_fsdp_optimizer,
         save_fsdp_model,
Repository: ggml-org/llama.cpp
Pull request: readme : update hot topics

Add link to discussion: https://github.com/ggml-org/llama.cpp/discussions/15313

Changed files:
filename: README.md
patch: @@ -17,6 +17,7 @@ LLM inference in C/C++
 
 ## Hot topics
 
+- **[[FEEDBACK] Better packaging for llama.cpp to support downstream consumers 🤗](https://github.com/ggml-org/llama.cpp/discussions/15313)**
 - Support for the `gpt-oss` model with native MXFP4 format has been added | [PR](https://github.com/ggml-org/llama.cpp/pull/15091) | [Collaboration with NVIDIA](https://blogs.nvidia.com/blog/rtx-ai-garage-openai-oss) | [Comment](https://github.com/ggml-org/llama.cpp/discussions/15095)
 - Hot PRs: [All](https://github.com/ggml-org/llama.cpp/pulls?q=is%3Apr+label%3Ahot+) | [Open](https://github.com/ggml-org/llama.cpp/pulls?q=is%3Apr+label%3Ahot+is%3Aopen)
 - Multimodal support arrived in `llama-server`: [#12898](https://github.com/ggml-org/llama.cpp/pull/12898) | [documentation](./docs/multimodal.md)
Repository: denoland/deno
Pull request: fix(cli): warn when an otel env var has an invalid value

<!--
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: cli/args/flags.rs
patch: @@ -930,7 +930,10 @@ impl Flags {
     let otel_var = |name| match std::env::var(name) {
       Ok(s) if s.to_lowercase() == "true" => Some(true),
       Ok(s) if s.to_lowercase() == "false" => Some(false),
-      _ => None,
+      _ => {
+        log::warn!("'{name}' env var value not recognized, only 'true' and 'false' are accepted");
+        None
+      }
     };
 
     let disabled =
@@ -947,7 +950,10 @@ impl Flags {
         Ok("ignore") => OtelConsoleConfig::Ignore,
         Ok("capture") => OtelConsoleConfig::Capture,
         Ok("replace") => OtelConsoleConfig::Replace,
-        _ => {
+        res => {
+          if res.is_ok() {
+            log::warn!("'OTEL_DENO_CONSOLE' env var value not recognized, only 'ignore', 'capture', or 'replace' are accepted");
+          }
           if default {
             OtelConsoleConfig::Capture
           } else {
Repository: facebook/react
Pull request: fix[devtools/updateFiberRecursively]: mount suspense fallback set in timed out case

Fixes https://github.com/facebook/react/issues/26793.

I have received a constantly reproducible example of the error, that is mentioned in the issue above.
When starting `Reload and Profile` in DevTools, React reports an unmount of a functional component inside Suspense's fallback via [`onCommitFiberUnmount`](https://github.com/facebook/react/blob/3ff846d106de9273f59d1e4457793a5fcf625aef/packages/react-devtools-shared/src/hook.js#L408-L413) in [`commitDeletionEffectsOnFiber`](https://github.com/facebook/react/blob/main/packages/react-reconciler/src/ReactFiberCommitWork.js#L2025), but this fiber was never registered as mounted in DevTools.

While debugging, I've noticed that in timed-out case for Suspense trees we only check if both previous fallback child set and next fiber fallback child set are non-null, but in these recursive calls there is also a case when previous fallback child set is null and next set is non-null, so we were skipping the branch.

<img width="1746" alt="Screenshot 2023-07-25 at 15 26 07" src="https://github.com/facebook/react/assets/28902667/da21a682-9973-43ec-9653-254ba98a0a3f">

After these changes, the issue is no longer reproducible, but I am not sure if this is the right solution, since I don't know if this case is correct from reconciler perspective.

Changed files:
filename: packages/react-devtools-shared/src/backend/renderer.js
patch: @@ -2345,6 +2345,18 @@ export function attach(
       const prevFallbackChildSet = prevFiberChild
         ? prevFiberChild.sibling
         : null;
+
+      if (prevFallbackChildSet == null && nextFallbackChildSet != null) {
+        mountFiberRecursively(
+          nextFallbackChildSet,
+          shouldIncludeInTree ? nextFiber : parentFiber,
+          true,
+          traceNearestHostComponentUpdate,
+        );
+
+        shouldResetChildren = true;
+      }
+
       if (
         nextFallbackChildSet != null &&
         prevFallbackChildSet != null &&

filename: packages/react-devtools-shared/src/hook.js
patch: @@ -320,7 +320,7 @@ export function installHook(target: any): DevToolsHook | null {
 
   let uidCounter = 0;
 
-  function inject(renderer: ReactRenderer) {
+  function inject(renderer: ReactRenderer): number {
     const id = ++uidCounter;
     renderers.set(id, renderer);