Skip to content
This repository was archived by the owner on Oct 27, 2026. It is now read-only.

Some tweaks in tool docs - #847

Merged
jottakka merged 15 commits into
mainfrom
francisco/some-tweaks-tool-docs
Mar 5, 2026
Merged

jottakka merged 15 commits into
mainfrom
francisco/some-tweaks-tool-docs

Conversation

@jottakka

@jottakka jottakka commented Mar 3, 2026 •

Copy link
Copy Markdown
Contributor

Note

Medium Risk
Moderate risk because the docs generator now auto-deletes previously generated toolkit JSON when a toolkit is detected as removed, so any change-detection bug could remove output unexpectedly; other changes are mainly CI/test refactors.

Overview
Docs site: Refactors integration category _meta.tsx files to use a new createCategoryMeta() helper that auto-inserts Optimized/Starter separators based on each entry’s type, avoiding duplicated separator boilerplate.

CI/tests: Adds a new External URL check GitHub Action running Vitest with vitest.external-urls.config.ts, and updates the external URL test to emit warnings (not failures) while excluding it from the default vitest.config.ts run.

Toolkit docs generator: Removes stale toolkit JSON entries (ComplexTools, Deepwiki, Test2) from generated data, and extends generate --skip-unchanged to collect removed toolkits via collectRemovedToolkitIds() and pass them through the existing cleanup step so stale <id>.json files are deleted and index.json is rebuilt (covered by new unit + scenario tests).

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

@vercel

vercel Bot commented Mar 3, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Mar 5, 2026 9:15pm

Request Review

@jottakka jottakka self-assigned this Mar 3, 2026

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

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.

Autofix Details

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Unnecessary separator added to single-type categories
    • Modified createCategoryMeta to only emit separators when both Optimized and Starter groups are non-empty, matching the documented behavior.

Create PR

Or push these changes by commenting:

@cursor push 8f1eb8ded9
Preview (8f1eb8ded9)
diff --git a/app/en/resources/integrations/create-category-meta.ts b/app/en/resources/integrations/create-category-meta.ts
--- a/app/en/resources/integrations/create-category-meta.ts
+++ b/app/en/resources/integrations/create-category-meta.ts
@@ -29,16 +29,21 @@
   const starter = entries.filter((e) => e.type === "arcade_starter");
 
   const result: MetaRecord = {};
+  const showSeparators = optimized.length > 0 && starter.length > 0;
 
   if (optimized.length > 0) {
-    result["-- Optimized"] = { type: "separator", title: "Optimized" };
+    if (showSeparators) {
+      result["-- Optimized"] = { type: "separator", title: "Optimized" };
+    }
     for (const entry of optimized) {
       result[entry.slug] = { title: entry.title, href: entry.href };
     }
   }
 
   if (starter.length > 0) {
-    result["-- Starter"] = { type: "separator", title: "Starter" };
+    if (showSeparators) {
+      result["-- Starter"] = { type: "separator", title: "Starter" };
+    }
     for (const entry of starter) {
       result[entry.slug] = { title: entry.title, href: entry.href };
     }
This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

Comment thread tests/external-url-check.test.ts Resolved
jottakka added 2 commits March 4, 2026 17:36
The previous URL (build-sophisticated-apps-for-your-pagerduty-environment-using-oauth-2-0-and-api-scopes) returns 404. Replaced with updating-your-tools-for-api-scopes which covers the same topic: OAuth 2.0, API scopes, app setup, and token management.

Made-with: Cursor
Next.js 16 deprecated the middleware file convention in favour of proxy.
Rename the file and its export accordingly. The API surface (NextRequest,
NextResponse, config matcher) is identical, no behaviour changes.

Fixes Vercel build warning:
  ⚠ The "middleware" file convention is deprecated. Please use "proxy" instead.

Made-with: Cursor
@@ -1,6 +1,4 @@
{

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@EricGustin removing the toolkits that shouldnt be here

);
}

// Auto-delete toolkit JSON files for toolkits that no longer exist in the API.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@EricGustin removing goes here adding to the id list to be deleted latter

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

part of the logic to delete was already in place, so we just have to add those with state removed to the list for the ones that will be removed.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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.

@jottakka
jottakka requested a review from EricGustin March 5, 2026 21:02
@jottakka
jottakka merged commit 2222707 into main Mar 5, 2026
7 checks passed
@jottakka
jottakka deleted the francisco/some-tweaks-tool-docs branch March 5, 2026 22:57

This branch was successfully deployed

1 active deployment
Preview — b6d08139 Deployed Mar 5, 2026 by vercel[bot]
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants