github/claude-code-best
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: remove duplicate review skill registration

Browse Source 
Remove registerBundledSkill for /review and /security-review since
extract-to-disk + skill scanner already registers them. Only register
/strict:review and /strict:security-review via bundled skill.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
kingboung 2026-05-12 17:03:38 +08:00
parent 09bbfb77e1
commit ca5114dfcf
2 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/costrict/skills/reviewSkills.ts → src/costrict/skills/strictReview.ts
View File

@ -23,7 +23,7 @@ const ALLOWED_TOOLS = [
'Agent', 'Agent',
] ]
function registerReviewSkill( function registerStrictReviewSkill(
name: string, name: string,
skillKey: string, skillKey: string,
files: Record<string, string>, files: Record<string, string>,
@ -47,7 +47,7 @@ function registerReviewSkill(
}) })
} }
export function registerReviewSkills(): void { export function registerStrictReviewSkills(): void {
const locale = getLocale() const locale = getLocale()
const localeFiles = SKILL_FILES[locale] const localeFiles = SKILL_FILES[locale]
const localeMetadata = SKILL_METADATA[locale] const localeMetadata = SKILL_METADATA[locale]
@ -57,7 +57,6 @@ export function registerReviewSkills(): void {
const meta = localeMetadata[skillKey] const meta = localeMetadata[skillKey]
if (!meta || !files) continue if (!meta || !files) continue
registerReviewSkill(meta.name, skillKey, files, meta.description) registerStrictReviewSkill(`strict:${skillKey}`, skillKey, files, meta.description)
registerReviewSkill(`strict:${skillKey}`, skillKey, files, meta.description)
} }
} }

4
src/skills/bundled/index.ts
View File

@ -17,7 +17,7 @@ import { registerUpdateConfigSkill } from './updateConfig.js'
import { registerVerifySkill } from './verify.js' import { registerVerifySkill } from './verify.js'
import { registerStrictPlanSkill } from 'src/costrict/skills/strictPlan.js' import { registerStrictPlanSkill } from 'src/costrict/skills/strictPlan.js'
import { registerStrictSpecSkill } from 'src/costrict/skills/strictSpec.js' import { registerStrictSpecSkill } from 'src/costrict/skills/strictSpec.js'
import { registerReviewSkills } from 'src/costrict/skills/reviewSkills.js' import { registerStrictReviewSkills } from 'src/costrict/skills/strictReview.js'
import { registerProjectWikiSkill } from 'src/costrict/skills/projectWiki.js' import { registerProjectWikiSkill } from 'src/costrict/skills/projectWiki.js'
import { registerTddSkill } from 'src/costrict/skills/tdd.js' import { registerTddSkill } from 'src/costrict/skills/tdd.js'
@ -39,7 +39,7 @@ export function initBundledSkills(): void {
registerTddSkill() registerTddSkill()
registerStrictPlanSkill() registerStrictPlanSkill()
registerStrictSpecSkill() registerStrictSpecSkill()
registerReviewSkills() registerStrictReviewSkills()
registerKeybindingsSkill() registerKeybindingsSkill()
registerVerifySkill() registerVerifySkill()
registerDebugSkill() registerDebugSkill()