jiebi/CodeConvo huggingface dataset:
Change:
<!-- Please provide a clear and concise description of what the bug is. Include screenshots if needed. Please test using the latest version of the relevant React packages to make sure your issue has not already been fixed. --React version: 18.x.x up to latest Node version: 16.x.x up to latest the React repository. : either: - Link to code example: Relevant code: (line 72) Relevant function: The current behavior: When running the commands or , a is thrown: . This is evident in the provided screenshot. The error originates from the function in . The problem stems from how the value is handled. In these specific commands, is passed as an array (e.g., ), but the code incorrectly wraps it in another array: The function then attempts to apply the method to each element of , expecting them to be strings. However, due to the erroneous wrapping, it encounters an array instead, leading to the TypeError. The expected behavior: The or commands should run without errors. The value should be handled correctly even when passed as an array, and bundle type filtering should work as expected. Additional information: The or commands work without issues because the value is passed as a string or undefined. The function generates and arrays based on the provided names ( or ). These arrays are used for bundle filtering in the function.

Discussion:
return result; } <ins> const argvType = Array.isArray(argv.type) ? argv.type : [argv.type]; </ins> const requestedBundleTypes = argv.type <del> ? parseRequestedNames([argv.type], 'uppercase') </del> <ins> ? parseRequestedNames(argvType, 'uppercase') </ins> : []; const requestedBundleNames = parseRequestedNames(argv._, 'lowercase'); const forcePrettyOutput = argv.pretty; const isWatchMode = argv.watch;

return true; } if (requestedBundleTypes.length > 0) { <del> const isAskingForDifferentType = requestedBundleTypes.every( requestedType => bundleType.indexOf(requestedType) === -1 </del> <ins> const isAskingForDifferentType = requestedBundleTypes.some( requestedType => !bundleType.includes(requestedType) </ins> ); if (isAskingForDifferentType) { return true;
Change:
React version: 18.x.x up to latest the React repository. : either: - the output of the function logs within the script. Relevant code: Relevant function: The function, responsible for determining whether to include a bundle in the build output, contains incorrect logic for filtering when multiple bundle types are specified. Specifically, the following line causes the issue: This code uses with to check if all requested bundle types are missing in the current bundle type. However, when multiple types are specified, it should skip a bundle only if none of the requested types are present. Here's an example of the incorrect behavior observed during debugging: In this case, even though the is and doesn't include , is incorrectly evaluated as , leading to the bundle not being skipped. The function should correctly filter bundles based on multiple requested types. In the example above, the expected behavior is that should be for the bundle type, as it does not include the type. In general, the function should return (skip the bundle) only if none of the requested types are included in the current bundle type.

Discussion:
return result; } <ins> const argvType = Array.isArray(argv.type) ? argv.type : [argv.type]; </ins> const requestedBundleTypes = argv.type <del> ? parseRequestedNames([argv.type], 'uppercase') </del> <ins> ? parseRequestedNames(argvType, 'uppercase') </ins> : []; const requestedBundleNames = parseRequestedNames(argv._, 'lowercase'); const forcePrettyOutput = argv.pretty; const isWatchMode = argv.watch;

return true; } if (requestedBundleTypes.length > 0) { <del> const isAskingForDifferentType = requestedBundleTypes.every( requestedType => bundleType.indexOf(requestedType) === -1 </del> <ins> const isAskingForDifferentType = requestedBundleTypes.some( requestedType => !bundleType.includes(requestedType) </ins> ); if (isAskingForDifferentType) { return true;
Change:
development website Uncaught Error: getCommitTree(): Invalid commit "1" for root "1". There are only "1" commits. Every time react-devtools-extensions 1- getCommitTree(): Invalid commit "2" for root "1". There are only "1" commits.
Hi thank you for reporting. should fix this. It's scheduled to be included in DevTools 6.0.0. Let us know if it's still happening in 6.0.0

Discussion:
}; onProfilingStatus: (isProfiling: boolean) => void = isProfiling => { <ins> if (this._isProfiling === isProfiling) { return; } </ins> if (isProfiling) { this._dataBackends.splice(0); this._dataFrontend = null;

}); } <del> if (this._isProfiling !== isProfiling) { this._isProfiling = isProfiling; </del> <ins> this._isProfiling = isProfiling; </ins> <del> // Invalidate suspense cache if profiling data is being (re-)recorded. // Note that we clear again, in case any views read from the cache while profiling. // (That would have resolved a now-stale value without any profiling data.) this._cache.invalidate(); </del> <ins> // Invalidate suspense cache if profiling data is being (re-)recorded. // Note that we clear again, in case any views read from the cache while profiling. // (That would have resolved a now-stale value without any profiling data.) this._cache.invalidate(); </ins> <del> this.emit('isProfiling'); </del> <ins> this.emit('isProfiling'); </ins> <del> // If we've just finished a profiling session, we need to fetch data stored in each renderer interface // and re-assemble it on the front-end into a format (ProfilingDataFrontend) that can power the Profiler UI. // During this time, DevTools UI should probably not be interactive. if (!isProfiling) { this._dataBackends.splice(0); this._rendererQueue.clear(); </del> <ins> // If we've just finished a profiling session, we need to fetch data stored in each renderer interface // and re-assemble it on the front-end into a format (ProfilingDataFrontend) that can power the Profiler UI. // During this time, DevTools UI should probably not be interactive. if (!isProfiling) { this._dataBackends.splice(0); this._rendererQueue.clear(); </ins> <del> // Only request data from renderers that actually logged it. // This avoids unnecessary bridge requests and also avoids edge case mixed renderer bugs. // (e.g. when v15 and v16 are both present) this._rendererIDsThatReportedProfilingData.forEach(rendererID => { if (!this._rendererQueue.has(rendererID)) { this._rendererQueue.add(rendererID); </del> <ins> // Only request data from renderers that actually logged it. // This avoids unnecessary bridge requests and also avoids edge case mixed renderer bugs. // (e.g. when v15 and v16 are both present) this._rendererIDsThatReportedProfilingData.forEach(rendererID => { if (!this._rendererQueue.has(rendererID)) { this._rendererQueue.add(rendererID); </ins> <del> this._bridge.send('getProfilingData', {rendererID}); } }); </del> <ins> this._bridge.send('getProfilingData', {rendererID}); } }); </ins> <del> this.emit('isProcessingData'); } </del> <ins> this.emit('isProcessingData'); </ins> } }; }