Skip to content

feat(mcp): Some enhancements to automated mcp db build - #1777

Open
dkalinovInfra wants to merge 18 commits into
masterfrom
dkalinov/mcp-db-git-workflow
Open

feat(mcp): Some enhancements to automated mcp db build #1777
dkalinovInfra wants to merge 18 commits into
masterfrom
dkalinov/mcp-db-git-workflow

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 lite review requested due to automatic review settings August 10, 2026 05:53

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 adds automation and guardrails around building the Ignite UI MCP documentation SQLite database, aiming to prevent partial/invalid publishes and reduce metadata drift from LLM compression.

Changes:

  • Adds a manual GitHub Actions workflow + composite action to rebuild framework docs, assemble a single DB, and open a publishing PR.
  • Introduces scripts to restore dist/docs_final from the committed DB and to deterministically derive component frontmatter post-compression.
  • Hardens compression and DB build behavior (model override via env, batch invalid-response retry, conditional VACUUM, build summary reporting, DB health checks).

Reviewed changes

Copilot reviewed 14 out of 16 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
spec/unit/docs-db-counts-spec.ts New Jasmine spec to sanity-check DB completeness/quality before publishing.
packages/igniteui-mcp/igniteui-doc-mcp/switch-submodules.sh Utility to move doc-source submodules to a requested branch.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/restore-docs-final.ts Restores dist/docs_final (and optional toc stubs) from the committed SQLite DB for incremental runs.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/report-build-summary.ts Emits a Markdown build summary for CI step summaries.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/derive-components.ts Deterministically derives component frontmatter using API index + document heuristics.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/compress-angular-docs.ts Allows model override via env; adds richer invalid logging + inline retry for batch results.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/compress-react-docs.ts Same as above for React.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/compress-blazor-docs.ts Same as above for Blazor.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/compress-wc-docs.ts Same as above for Web Components + tighter component prompt rules.
packages/igniteui-mcp/igniteui-doc-mcp/scripts/build-db.ts Tracks pre-open DB existence and conditionally VACUUMs to avoid growing committed DB size.
packages/igniteui-mcp/igniteui-doc-mcp/package.json Wires derive-components:* into pipelines and adds script entries.
packages/igniteui-mcp/igniteui-doc-mcp/docs/knowledgebase.md Documents new failure mode + mitigation for component drift.
.github/workflows/build-docs-db.yml New workflow to build per-framework artifacts, assemble one DB, verify counts, and open a publish PR.
.github/actions/build-framework-docs/action.yml New composite action implementing per-framework build + artifact upload + validation.
Suppressed comments (1)

spec/unit/docs-db-counts-spec.ts:67

  • toBeGreaterThanOrEqual doesn’t accept a failure-message argument in Jasmine; this will cause a TypeScript compile error in CI. Use withContext(...) to attach the diagnostic message.
			expect(counts[fw] || 0)
				.toBeGreaterThanOrEqual(MIN_DOCS[fw], `${fw} has ${counts[fw] || 0} docs, expected >= ${MIN_DOCS[fw]}`);

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread spec/unit/docs-db-counts-spec.ts
Comment thread .github/actions/build-framework-docs/action.yml
dkalinovInfra and others added 3 commits August 10, 2026 09:00
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@dkalinovInfra
dkalinovInfra requested a review from kdinev August 10, 2026 06:25
@coveralls

coveralls commented Aug 10, 2026

Copy link
Copy Markdown

Coverage Status

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants