Improve cascading values provider demo (#12483)

[yasmoradi]

closes #12483

Summary by CodeRabbit

• Documentation

  • Updated demo examples for CascadingValueProvider component to improve clarity and consistency.

• Refactor

  • Reorganized demo consumer components for better maintainability and presentation.

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 1af43289-053d-4a2e-9e9b-21eb0d1a24bb

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

Walkthrough

Replaces the C#-only CascadingValueConsumer class (deleted) with a new Razor component CascadingValueDemoConsumer.razor. All three demo examples (Basic, Values, ValueList) are updated to render the new component, and embedded code snippet strings are trimmed to remove the previously inlined consumer implementation.

Changes

Consumer Component Replacement and Demo Updates

Layer / File(s)	Summary
New CascadingValueDemoConsumer Razor component ...CascadingValueProvider/CascadingValueDemoConsumer.razor, ...CascadingValueProvider/CascadingValueConsumer.cs	Deletes the old CascadingValueConsumer C# class and adds CascadingValueDemoConsumer.razor with Title, Theme, NotificationCount, IsAuthenticated, NamedUser, and TypedUser parameters, null-safe markup rendering, and a FormatUser helper.
Demo page rendered markup and embedded snippets ...CascadingValueProvider/BitCascadingValueProviderDemo.razor, ...CascadingValueProvider/BitCascadingValueProviderDemo.razor.cs	Swaps all three rendered demo usages to CascadingValueDemoConsumer, removes the Basic example's CsharpCode attribute, updates example1/2/3RazorCode strings to reference the new component with HTML comment source links, and trims example2CsharpCode and example3CsharpCode by removing the previously inlined consumer class.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐇 Hop hop, the consumer's reborn,
A .razor file now proudly worn!
No more C# strings stuffed inside,
The Agentic AI hops with pride.
Clean components, neat and bright —
The rabbit cheers: "That feels right!" 🌟

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The PR improves the cascading values provider demo as part of making the boilerplate template more AI-friendly by using clearer component names and better-organized example code.
Out of Scope Changes check	✅ Passed	All changes are directly related to refactoring the cascading values provider demo and improving code organization, which aligns with the stated objective of making the boilerplate more AI-friendly.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title 'Improve cascading values provider demo' clearly and specifically summarizes the main change across the PR files, which refactors and reorganizes the cascading values demo implementation.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

This PR refactors the BitCascadingValueProvider demo to make the showcased “consumer” component easier to understand and to reduce duplicated inline code snippets, aligning with the goal of making templates/demos more agent-friendly (#12478).

Changes:

• Replaced the old ComponentBase/BuildRenderTree demo consumer with a .razor component (CascadingValueDemoConsumer).
• Updated the demo page to use the new consumer component and trimmed the embedded C# snippet(s) in the demo code-behind.
• Added inline comments in the demo snippet strings pointing readers/agents to the relevant source files in GitHub.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File	Description
src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/CascadingValueDemoConsumer.razor	New Razor-based demo consumer component that displays cascading parameter values.
src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/CascadingValueConsumer.cs	Removes the previous render-tree-based consumer component.
src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/BitCascadingValueProviderDemo.razor.cs	Simplifies embedded demo snippets and adds source links for agents/readers.
src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/BitCascadingValueProviderDemo.razor	Switches demo usage from CascadingValueConsumer to CascadingValueDemoConsumer and adjusts DemoExample parameters.

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/BitCascadingValueProviderDemo.razor (1)

55-65: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Named cascade key mismatch breaks NamedUser binding in ValueList demo.

CascadingValueDemoConsumer expects the named user cascade key "NamedUser", but this block provides "UserInfo" (Line 61), so the named user always renders as null.

💡 Proposed fix

-                            { nullableNamedUser, "UserInfo" },
+                            { nullableNamedUser, "NamedUser" },

Also apply the same key update in BitCascadingValueProviderDemo.razor.cs inside example3RazorCode so rendered demo and shown snippet stay consistent.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/BitCascadingValueProviderDemo.razor`
around lines 55 - 65, The cascade key mismatch causes the named user binding to
fail because BitCascadingValueProvider in the ValueList is providing the
nullableNamedUser value with the key "UserInfo", but CascadingValueDemoConsumer
expects the key "NamedUser". Change the cascade key from "UserInfo" to
"NamedUser" for the nullableNamedUser entry in the BitCascadingValueProvider
ValueList, and also update the same cascade key in the example3RazorCode string
within the code-behind file (BitCascadingValueProviderDemo.razor.cs) to keep the
rendered demo and displayed code snippet consistent.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In
`@src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/BitCascadingValueProviderDemo.razor`:
- Around line 55-65: The cascade key mismatch causes the named user binding to
fail because BitCascadingValueProvider in the ValueList is providing the
nullableNamedUser value with the key "UserInfo", but CascadingValueDemoConsumer
expects the key "NamedUser". Change the cascade key from "UserInfo" to
"NamedUser" for the nullableNamedUser entry in the BitCascadingValueProvider
ValueList, and also update the same cascade key in the example3RazorCode string
within the code-behind file (BitCascadingValueProviderDemo.razor.cs) to keep the
rendered demo and displayed code snippet consistent.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: b07acf2f-8b22-4211-9ac1-89c1a2c000ff

📥 Commits

Reviewing files that changed from the base of the PR and between c202507 and e34a39c.

📒 Files selected for processing (4)

• src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/BitCascadingValueProviderDemo.razor
• src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/BitCascadingValueProviderDemo.razor.cs
• src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/CascadingValueConsumer.cs
• src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/CascadingValueDemoConsumer.razor

💤 Files with no reviewable changes (1)

• src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Utilities/CascadingValueProvider/CascadingValueConsumer.cs