diff --git a/lib/enhance/skill-patterns.js b/lib/enhance/skill-patterns.js index 830469a..448e04c 100644 --- a/lib/enhance/skill-patterns.js +++ b/lib/enhance/skill-patterns.js @@ -76,7 +76,11 @@ const skillPatterns = { (content && p.test(content)); }); - if (hasSideEffects && frontmatter['disable-model-invocation'] !== true) { + // Accept both the YAML boolean `true` and the quoted string "true" - + // users commonly write `disable-model-invocation: "true"`, which YAML + // parses as a string; a strict `!== true` would wrongly re-flag it. + const dmi = frontmatter['disable-model-invocation']; + if (hasSideEffects && dmi !== true && dmi !== 'true') { return { issue: 'Skill with side effects should have disable-model-invocation: true', fix: 'Add "disable-model-invocation: true" to frontmatter for manual-only invocation' diff --git a/lib/index.js b/lib/index.js index db4fd58..dc54db4 100644 --- a/lib/index.js +++ b/lib/index.js @@ -25,6 +25,7 @@ const customHandler = require('./sources/custom-handler'); const policyQuestions = require('./sources/policy-questions'); const crossPlatform = require('./cross-platform'); const enhance = require('./enhance'); +const repoIntel = require('./repo-intel'); const repoMap = require('./repo-map'); const perf = require('./perf'); const collectors = require('./collectors'); @@ -250,6 +251,7 @@ module.exports = { sources, xplat, enhance, + repoIntel, repoMap, perf, collectors,