fix: enhance SQLite error handling and ensure proper resource cleanup

[frontegg-david]

Summary by CodeRabbit

• Documentation
  • Updated skill documentation across the catalog to provide consistent navigation and routing guidance, clarifying references to platform-specific configuration, deployment, development, testing, and best practices resources.
  • Standardized terminology and improved step-by-step guidance for users navigating between related documentation pages.

[coderabbitai]

📝 Walkthrough

Walkthrough

Five skill router documentation files (config, deployment, development, testing, and guides) are updated to consistently reference markdown documentation under references/ directories rather than skill resources. Entry points, decision guidance, routing tables, and supporting text use "reference" terminology throughout.

Changes

Router and Guide Documentation Terminology Refactoring

Layer / File(s)	Summary
Config, Deployment, Development Router Updates libs/skills/catalog/frontmcp-config/SKILL.md, libs/skills/catalog/frontmcp-deployment/SKILL.md, libs/skills/catalog/frontmcp-development/SKILL.md	Config, deployment, and development router entry-point descriptions, recommended bullets, and decision text reworded to route users to references under references/ instead of skills. Scenario routing tables updated from "Skill" to "Reference" column headers, and steps sections clarified to navigate to matching reference documents.
Testing Router Update libs/skills/catalog/frontmcp-testing/SKILL.md	Testing router decision callout and steps section updated to direct readers to the setup-testing reference under references/, with scenario routing table header adjusted to "Reference / Section" terminology and ordered routing flow clarified.
Guide Documentation and References Section libs/skills/catalog/frontmcp-guides/SKILL.md	"Skip When" bullets refined to point to component references under frontmcp-development/references/, "Best Practices" table column headers changed from "Skill Reference" to "Reference", and "Reference" list core-item bullet renamed from "Core skills" to "Core references" with explicit location clarification.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related issues

• agentfront/frontmcp#406: The terminology shifts from "skill" to "reference" in router documentation and scenario routing table headers directly address the documentation/terminology mismatch described in this issue.

Poem

🐰 Hops through the docs with glee,
Reframing skills as references, you see,
From chaos to clarity in organized rows,
Where each router reference gracefully flows. 📚✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The PR title claims to address SQLite error handling and resource cleanup, but the actual changes are exclusively documentation updates to skill routing references in SKILL.md files across multiple frontend MCP catalog directories.	Update the PR title to accurately reflect the actual changes, such as 'docs: update skill documentation to reference canonical references instead of skills' or similar.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/406-router-references

---

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

[coderabbitai]

Caution

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

⚠️ Outside diff range comments (1)

libs/skills/catalog/frontmcp-guides/SKILL.md (1)

1-1: ⚠️ Potential issue | 🔴 Critical | ⚡ Quick win

Critical: PR title and description do not match the actual changes.

The PR is titled "fix: enhance SQLite error handling and ensure proper resource cleanup" but all visible changes are documentation terminology updates to router SKILL.md files (changing "skill" references to "reference" terminology). There are no SQLite-related changes, error handling improvements, or resource cleanup modifications in any of the provided files.

This mismatch suggests one of the following:

• Incorrect PR title/description
• Missing files that contain the actual SQLite/cleanup changes
• Wrong branch or files included in the review

Please verify that:

1. The PR title and description accurately reflect the changes
2. All relevant files are included in this review
3. If this is a documentation-only PR, update the title to reflect that (e.g., "docs: update router terminology to reference-based navigation")

🤖 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 `@libs/skills/catalog/frontmcp-guides/SKILL.md` at line 1, PR metadata and
contents are inconsistent: the branch/commits only include documentation edits
(SKILL.md router terminology changes) but the PR title/description claim
SQLite/error-handling/resource cleanup fixes. Update the PR so it accurately
reflects the change set: either (A) amend the title/description to indicate a
docs-only change (e.g., "docs: update router terminology to reference-based
navigation") and call out files like SKILL.md, or (B) add the missing
implementation commits that touch the SQLite/error-handling/resource cleanup
code if those fixes were intended to be included; also verify the correct branch
is selected and that all relevant files are present in the PR before requesting
review.

🤖 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 `@libs/skills/catalog/frontmcp-guides/SKILL.md`:
- Line 1: PR metadata and contents are inconsistent: the branch/commits only
include documentation edits (SKILL.md router terminology changes) but the PR
title/description claim SQLite/error-handling/resource cleanup fixes. Update the
PR so it accurately reflects the change set: either (A) amend the
title/description to indicate a docs-only change (e.g., "docs: update router
terminology to reference-based navigation") and call out files like SKILL.md, or
(B) add the missing implementation commits that touch the
SQLite/error-handling/resource cleanup code if those fixes were intended to be
included; also verify the correct branch is selected and that all relevant files
are present in the PR before requesting review.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2dfa00c6-13bb-4cf1-9768-d8cb23ebdace

📥 Commits

Reviewing files that changed from the base of the PR and between c291d22 and 842b4cf.

📒 Files selected for processing (5)

• libs/skills/catalog/frontmcp-config/SKILL.md
• libs/skills/catalog/frontmcp-deployment/SKILL.md
• libs/skills/catalog/frontmcp-development/SKILL.md
• libs/skills/catalog/frontmcp-guides/SKILL.md
• libs/skills/catalog/frontmcp-testing/SKILL.md

[github-actions]

Performance Test Results

Status: ✅ All tests passed

Summary

Project	Tests	Passed	Warnings	Failed	Leaks
✅ demo-e2e-agents	4	4	0	0	0
✅ demo-e2e-cache	11	11	0	0	0
✅ demo-e2e-codecall	4	4	0	0	0
✅ demo-e2e-config	4	4	0	0	0
✅ demo-e2e-direct	3	3	0	0	0
✅ demo-e2e-elicitation	1	1	0	0	0
✅ demo-e2e-errors	4	4	0	0	0
✅ demo-e2e-hooks	3	3	0	0	0
✅ demo-e2e-multiapp	4	4	0	0	0
✅ demo-e2e-notifications	3	3	0	0	0
✅ demo-e2e-openapi	2	2	0	0	0
✅ demo-e2e-providers	4	4	0	0	0
✅ demo-e2e-public	4	4	0	0	0
✅ demo-e2e-redis	15	15	0	0	0
✅ demo-e2e-remember	4	4	0	0	0
✅ demo-e2e-remote	5	5	0	0	0
✅ demo-e2e-serverless	2	2	0	0	0
✅ demo-e2e-skills	15	15	0	0	0
✅ demo-e2e-standalone	2	2	0	0	0
✅ demo-e2e-transport-recreation	3	3	0	0	0
✅ demo-e2e-ui	4	4	0	0	0

Total: 101 tests across 21 projects

📊 View full report in workflow run

---

Generated at: 2026-05-25T13:30:36.422Z
Commit: 39d4ccc8