Skip to content

Commit 4f50108

Browse files
committed
tons of updates
1 parent e950bcd commit 4f50108

3 files changed

Lines changed: 34 additions & 16 deletions

File tree

commands/coldbox/ai/uninstall.cfc

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -48,12 +48,8 @@ component extends="coldbox-cli.models.BaseAICommand" {
4848

4949
try {
5050
printInfo( "🗑️ Removing AI integration..." )
51-
52-
// Remove the .ai directory
5351
directoryDelete( aiDirectory, true )
54-
5552
printSuccess( "✓ AI integration uninstalled successfully!" )
56-
print.line()
5753
printTip( "To reinstall AI integration, run: coldbox ai install" )
5854
} catch ( any e ) {
5955
printError( "Failed to uninstall AI integration: #e.message#" )

models/GuidelineManager.cfc

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -316,8 +316,20 @@ component singleton {
316316
"recommendations" : []
317317
};
318318

319-
// Check for missing core guidelines
319+
// Check for missing core guidelines (respecting language mode)
320+
var language = manifest.language ?: "boxlang"
320321
static.CORE_GUIDELINES.each( ( name ) => {
322+
323+
// Skip cfml guideline for boxlang-only projects
324+
if ( name == "cfml" && language == "boxlang" ) {
325+
return;
326+
}
327+
328+
// Skip boxlang guideline for cfml-only projects
329+
if ( name == "boxlang" && language == "cfml" ) {
330+
return;
331+
}
332+
321333
var found = manifest.guidelines.filter( ( g ) => {
322334
return g.name == name
323335
} )

models/SkillManager.cfc

Lines changed: 21 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -175,8 +175,8 @@ component singleton {
175175

176176
toRemove.each( ( name ) => {
177177
variables.print.yellowLine( " 🗑️ Removing orphaned module skill: #name#" ).toConsole()
178-
deleteSkillDir( arguments.directory, name )
179-
arguments.manifest.skills = arguments.manifest.skills.filter( ( s ) => s.name != name )
178+
deleteSkillDir( directory, name )
179+
manifest.skills = manifest.skills.filter( ( s ) => s.name != name )
180180
changes.removed.append( name )
181181
} )
182182

@@ -193,10 +193,14 @@ component singleton {
193193
remoteSkills.each( ( s ) => {
194194
var owner = s.owner ?: ""
195195
var repo = s.repo ?: ""
196-
if ( !owner.len() || !repo.len() ) return
197-
var key = "#owner#/#repo#"
198-
if ( !repoMap.keyExists( key ) ) repoMap[ key ] = []
199-
repoMap[ key ].append( s )
196+
if ( !owner.len() || !repo.len() ){
197+
return;
198+
}
199+
var targetKey = "#owner#/#repo#"
200+
if ( !repoMap.keyExists( targetKey ) ) {
201+
repoMap[ targetKey ] = []
202+
}
203+
repoMap[ targetKey ].append( s )
200204
} )
201205

202206
// For each repo, fetch the skill list and compare SHAs
@@ -212,7 +216,9 @@ component singleton {
212216
var entrySlug = manifestEntry.slug ?: ""
213217
var entryPath = manifestEntry.path ?: ""
214218
var remote = remoteList.filter( ( r ) => r.slug == entrySlug || r.path == entryPath )
215-
if ( !remote.len() ) return
219+
if ( !remote.len() ){
220+
return;
221+
}
216222

217223
var currentSha = remote.first().sha ?: ""
218224
var storedSha = manifestEntry.sha ?: ""
@@ -287,18 +293,22 @@ component singleton {
287293
if ( directoryExists( skillsDir ) ) {
288294
directoryList( skillsDir, false, "name" ).each( ( dirName ) => {
289295
var skillFilePath = "#skillsDir#/#dirName#/SKILL.md"
290-
if ( !fileExists( skillFilePath ) ) return
296+
if ( !fileExists( skillFilePath ) ) {
297+
return;
298+
}
291299

292-
var alreadyInManifest = arguments.manifest.skills.filter( ( s ) => s.name == dirName ).len() > 0
293-
if ( alreadyInManifest ) return
300+
var alreadyInManifest = manifest.skills.filter( ( s ) => s.name == dirName ).len() > 0
301+
if ( alreadyInManifest ) {
302+
return;
303+
}
294304

295305
variables.print.greenLine( " ✨ Found new custom skill: #dirName#" ).toConsole()
296306

297307
var content = fileRead( skillFilePath )
298308
var parsed = variables.utility.parseFrontmatter( content )
299309
var description = parsed.frontmatter.description ?: ""
300310

301-
arguments.manifest.skills.append( {
311+
manifest.skills.append( {
302312
"name" : dirName,
303313
"owner" : "",
304314
"repo" : "",

0 commit comments

Comments
 (0)