Skip to content

Fixing issues with github actions when generating docs and toolkit changes detectiong#780

Open
jottakka wants to merge 28 commits intomainfrom
francisco/gendocsfixversion
Open

Fixing issues with github actions when generating docs and toolkit changes detectiong#780
jottakka wants to merge 28 commits intomainfrom
francisco/gendocsfixversion

Conversation

@jottakka
Copy link
Contributor

@jottakka jottakka commented Feb 17, 2026

Note

Medium Risk
Touches CI automation and the toolkit docs generator’s diff/verification logic; mistakes could lead to missed regenerations or unexpected docs/route changes, but scope is limited to docs tooling (not runtime auth/data paths).

Overview
Toolkit docs automation is hardened and made more deterministic. The generate-toolkit-docs GitHub Action now runs nightly on a schedule, invokes the generator via pnpm dlx tsx against the Engine tool-metadata API with --require-complete/--verbose, avoids pruning empty integration sections by default, and disables Husky when auto-opening PRs (also updating the auto PR title/commit message).

Change detection and output verification are made resilient to schema drift. The generator/verification pipeline can now parse “legacy” toolkit JSON via a new fallback parser (parsePreviousToolkitForDiff) and optionally allow that fallback during diffing/verification, while tool change signatures are normalized to ignore source-unstable fields (descriptions, oauth provider IDs, empty enums vs null, and unknown output types).

Design system loading and route/category resolution are stabilized. Design-system imports are switched to a require.resolve-based loader with Next webpack aliases, listToolkitRoutes now treats generated toolkit JSON as the source of truth for category/docsLink/isHidden (catalog as fallback), and sync-toolkit-sidebar adds --remove-empty-sections[=true|false] (with --prune as an alias) so category removal is opt-in.

Misc reliability tweaks. Pre-commit uses pnpm exec and avoids stash/pop conflicts with partially staged files, external URL tests add retries for transient failures, and the design system dependency is bumped.

Written by Cursor Bugbot for commit 44108b6. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Feb 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Feb 18, 2026 11:50pm

Request Review

@jottakka jottakka changed the title Fix npm version in gen tool docs Fixing issues with github actions for Feb 18, 2026
@jottakka jottakka changed the title Fixing issues with github actions for Fixing issues with github actions when generating docs and toolkit changes detectiong Feb 18, 2026
jottakka and others added 3 commits February 18, 2026 16:57
Remove databases _meta.tsx and restore integrations and development _meta.tsx files to match main.

Co-authored-by: Cursor <cursoragent@cursor.com>
Restore all toolkit JSON data files to match main.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: jottakka <203343514+jottakka@users.noreply.github.com>
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

"!node_modules",
"!public",
"!toolkit-docs-generator/data/toolkits",
"!toolkit-docs-generator/data/toolkits",
Copy link

Choose a reason for hiding this comment

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

Duplicate ignore pattern in biome config

Low Severity

The "!toolkit-docs-generator/data/toolkits" ignore pattern appears twice in the files.includes array. One of the duplicate entries can be removed.

Fix in Cursor Fix in Web

Copy link
Contributor

Choose a reason for hiding this comment

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

This is true

import { withLlmsTxt } from "./lib/next-plugin-llmstxt";
import { remarkGlossary } from "./lib/remark-glossary";

const require = createRequire(import.meta.url);
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure why we are adding all of these.

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.

2 participants

Comments