packages/provider_devtools_extension/lib/src/provider_screen.dart (1)

6-6:Optional: prefer explicit import prefix overhide for long‑term clarity.

Using an alias for DevTools UI reduces the chance of future symbol clashes and improves readability (e.g.,dt.Split). Non-blocking suggestion.

For example:

// importsimport'package:devtools_app_shared/ui.dart'as dt;import'package:flutter/material.dart';// no hide needed// usagesfinal splitAxis= dt.Split.axisFor(context,0.85);return dt.Split(  axis: splitAxis,  initialFractions:const [0.33,0.67],  children: [/* ... */ ],);

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

packages/provider/test/null_safe/multi_provider_test.dart (1)

14-14:1500 → 500 reduces stress coverage; extract to a constant and optionally keep a gated stress test.

If the reduction was to deflake/accelerate CI, fine—please document the rationale. To preserve intent, use a named constant and consider an opt‑in stress test at 1500 (skipped by default or CI‑flagged).

Apply within this hunk:

-            for (var i = 0; i < 500; i++) Provider<int>.value(value: i),+            for (var i = 0; i < _kLargeProviderCount; i++) Provider<int>.value(value: i),

Add near the imports (outside this hunk):

// Large but CI-friendly default; adjust as needed.const _kLargeProviderCount=500;

Optionally add a separate, skipped-by-default stress test (outside this hunk) to run with 1500 providers under a CI flag.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro