Skip to content

fix(docs): update mcp db with full set of documents - #1767

Merged
kdinev merged 4 commits into
masterfrom
dkalinov/mcp-db-fix
Aug 5, 2026
Merged

fix(docs): update mcp db with full set of documents#1767
kdinev merged 4 commits into
masterfrom
dkalinov/mcp-db-fix

Conversation

@dkalinovInfra

Copy link
Copy Markdown
Contributor

Description

Related Issue

Closes #

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring / code cleanup
  • Build / CI configuration change

Affected Packages

  • igniteui-cli (packages/cli)
  • @igniteui/cli-core (packages/core)
  • @igniteui/angular-templates (packages/igx-templates)
  • @igniteui/angular-schematics (packages/ng-schematics)
  • @igniteui/mcp-server (packages/igniteui-mcp)

Checklist

  • I have tested my changes locally (npm run test)
  • I have built the project successfully (npm run build)
  • I have run the linter (npm run lint)
  • I have added/updated tests as needed
  • My changes do not introduce new warnings or errors

Additional Context

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Ignite UI MCP docs toolchain by adjusting the default OpenAI model used by the doc-compression scripts, improving resilience in the DB build “optional copy” step, and refreshing Angular docs baseline snippets to remove outdated component factory usage.

Changes:

  • Switch default --model in all compress-*-docs.ts scripts from gpt-5.4-mini to gpt-5.6-luna.
  • Make the “copy DB to docs-backend” step non-fatal by wrapping it in a try/catch with a warning.
  • Update Angular Excel Library baseline docs to drop ComponentFactoryResolver/Injector usage and use ViewContainerRef.createComponent(...) directly.

Reviewed changes

Copilot reviewed 8 out of 10 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
packages/igniteui-mcp/igniteui-doc-mcp/scripts/compress-wc-docs.ts Changes default OpenAI model used for Web Components docs compression.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/compress-react-docs.ts Changes default OpenAI model used for React docs compression.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/compress-blazor-docs.ts Changes default OpenAI model used for Blazor docs compression.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/compress-angular-docs.ts Changes default OpenAI model used for Angular docs compression.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/build-db.ts Makes the optional backend DB copy step tolerant to failures.
packages/igniteui-mcp/igniteui-doc-mcp/docs_baseline/angular/excel-library.md Updates Angular sample code to remove unused DI imports/constructor params.
packages/igniteui-mcp/igniteui-doc-mcp/docs_baseline/angular/excel-library-using-workbooks.md Updates sample to use createComponent(IgxGridComponent) and removes unused DI imports/constructor params.
packages/igniteui-mcp/igniteui-doc-mcp/docs_baseline/angular/excel-library-using-cells.md Updates Angular sample code to remove unused DI imports/constructor params.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/igniteui-mcp/igniteui-doc-mcp/scripts/compress-wc-docs.ts
Comment thread packages/igniteui-mcp/igniteui-doc-mcp/scripts/build-db.ts
@coveralls

coveralls commented Aug 5, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 87.816%. remained the same — dkalinov/mcp-db-fix into master

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: dkalinovInfra <130121354+dkalinovInfra@users.noreply.github.com>
Co-authored-by: dkalinovInfra <130121354+dkalinovInfra@users.noreply.github.com>
@kdinev kdinev added the mcp label Aug 5, 2026
@kdinev
kdinev merged commit cea4330 into master Aug 5, 2026
4 checks passed
@kdinev
kdinev deleted the dkalinov/mcp-db-fix branch August 5, 2026 14:36
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.

5 participants