Commit addf247
fix: correct 5 non-3-level tool names in server-card.json (Smithery quality 34→35)
Smithery "Tool names" 5pt was scoring 0/5 because 5 tools had 2-level
names: crypto.global, finnhub.company_profile, geo.reverse_geocode,
github.user, usrealestate.property_detail.
Root cause: The regex r"toolId:.*?mcpName:" with re.DOTALL used in the
server-card regeneration script fails when a tool's prefetch rule
(elsewhere in tool-definitions.ts) contains an earlier toolId: match.
The non-greedy .*? swallows the wrong content.
Fix: Manually corrected the 5 names using the proper mcpName values from
tool-definitions.ts. Also updated both smithery and onboard-provider
skill SKILL.md files to use a backward-search parser instead.
Also added flywheel rule to prevent recurrence.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>1 parent d7d5ebb commit addf247
1 file changed
+5
-5
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
16849 | 16849 | | |
16850 | 16850 | | |
16851 | 16851 | | |
16852 | | - | |
| 16852 | + | |
16853 | 16853 | | |
16854 | 16854 | | |
16855 | 16855 | | |
| |||
16869 | 16869 | | |
16870 | 16870 | | |
16871 | 16871 | | |
16872 | | - | |
| 16872 | + | |
16873 | 16873 | | |
16874 | 16874 | | |
16875 | 16875 | | |
| |||
16894 | 16894 | | |
16895 | 16895 | | |
16896 | 16896 | | |
16897 | | - | |
| 16897 | + | |
16898 | 16898 | | |
16899 | 16899 | | |
16900 | 16900 | | |
| |||
16932 | 16932 | | |
16933 | 16933 | | |
16934 | 16934 | | |
16935 | | - | |
| 16935 | + | |
16936 | 16936 | | |
16937 | 16937 | | |
16938 | 16938 | | |
| |||
16955 | 16955 | | |
16956 | 16956 | | |
16957 | 16957 | | |
16958 | | - | |
| 16958 | + | |
16959 | 16959 | | |
16960 | 16960 | | |
16961 | 16961 | | |
| |||
0 commit comments