Skip to content

docs: align README, DEVELOPMENT guide, Index and skills with Syncfusion.Blazor.Toolkit 2.0 API - #98

Open
PrinceOliver wants to merge 3 commits into
mainfrom
bold-create/0eea1ef0-update-documents-repository-latest
Open

PrinceOliver wants to merge 3 commits into
mainfrom
bold-create/0eea1ef0-update-documents-repository-latest

Conversation

@PrinceOliver

Copy link
Copy Markdown
Collaborator

📍 Agent Session: View Full Session | 👤 Requested by: @PrinceOliver | 🤖 Agent: Agent

📋 Task Context

Update repository documentation (README, DEVELOPMENT.md, Index.md, and the AI skills under .github/skills) so build commands, package versions, namespaces, enum names, service registration and links match the current 2.0 package structure (single root namespace Syncfusion.Blazor.Toolkit).

📝 Implementation Summary

Core docs

  • README.md: build path -> ./src/Syncfusion.Blazor.Toolkit.slnx; Version="x.x.x" -> 2.0.0; chart sample @using -> Syncfusion.Blazor.Toolkit; sales mailto subject fixed (was "Maui toolkit").
  • .github/DEVELOPMENT.md: build path -> ./src/Syncfusion.Blazor.Toolkit.slnx.
  • .github/Index.md + evidences/performance/: renamed virtualization-startegy.md -> virtualization-strategy.md and fixed the index link.

Skills (.github/skills/**)

  • Replaced legacy sub-namespaces (.Charts, .Buttons, .Inputs, .Calendars, .Popups, .Spinner) with Syncfusion.Blazor.Toolkit in all skill docs (35 files). Syncfusion.Blazor.Toolkit.Data is untouched.
  • Charts skill: package is Syncfusion.Blazor.Toolkit (no .Charts, no Syncfusion.Blazor.Themes); service registration is AddSyncfusionBlazorToolkit(); SelectionMode.* / Syncfusion.Blazor.Toolkit.Charts.SelectionMode.* -> ChartSelectionMode.*, and the API reference enum is renamed to ChartSelectionMode (matches ChartEnumerations.cs).
  • Charts getting-started: removed the nonexistent Syncfusion.Blazor.Core/.../syncfusion-blazor.min.js and Toolkit.Charts/.../sf-chart.min.js script references; now points at _content/Syncfusion.Blazor.Toolkit/scripts/chart.js (exists in src/wwwroot/scripts).

Documentation-only; no source code changed.

✅ Verification Results

  • ✅ dotnet build ./src/Syncfusion.Blazor.Toolkit.slnx (net8.0/net9.0/net10.0): Build succeeded, 0 errors

🔍 Quality Checklist

  • Code follows project conventions
  • All tests pass
  • No security vulnerabilities introduced
  • Documentation updated (if applicable)
  • Breaking changes documented (if any)

⚠️ Human Review Required

Please pay special attention to:

  • Chart script reference in references/getting-started.md now points to _content/Syncfusion.Blazor.Toolkit/scripts/chart.js; please confirm this is the intended way (vs. fully automatic loading via JS interop) to document it.
  • First build attempt failed only because the CycloneDX global tool was not installed in the sandbox (environment issue); after installing it the build passed. Not related to this change.
  • Playwright test-plan markdown files under tests/ still mention Syncfusion.Blazor.Toolkit.Buttons; left untouched as out of scope (not user documentation).

View Full Session • Generated by Agent

Bold Create Agent and others added 2 commits October 9, 2026 18:15
…NT.md, .github/Index.md, .github/evidences/performance/virtualization-strategy.md, .github/skills/syncfusion-blazor-toolkit-charts/SKILL.md, .github/skills/syncfusion-blazor-toolkit-charts/references/getting-started.md

Co-authored-by: Prince Oliver <31838171+PrinceOliver@users.noreply.github.com>
…zor.Toolkit API

- Replace legacy sub-namespaces (.Charts/.Buttons/.Inputs/.Calendars/.Popups/.Spinner) with Syncfusion.Blazor.Toolkit in all skill docs
- Use ChartSelectionMode (actual enum name) in chart skill docs and API reference
- Replace obsolete Syncfusion.Blazor.Core / sf-chart.min.js script guidance with toolkit chart.js script

Co-authored-by: Prince Oliver <31838171+PrinceOliver@users.noreply.github.com>
Co-authored-by: Prince Oliver <31838171+PrinceOliver@users.noreply.github.com>
@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown

CI Summary

Job Status
bUnit (.NET 8 / 9 / 10) ✅ Passed
Playwright ✅ Passed
NuGet vulnerability scan ✅ Passed
ESLint security ✅ Passed
XSS / unsafe markup scan ✅ Passed
Pack (.NET 8/9/10, unsigned) ✅ Passed

Overall: ✅ All checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant