Code Action: Allow enabling/disabling and improve ranking of explain

[umpox]

Description

• Group the fix and explain actions
• Allow enabling/disabling the actions all together

Enabled:

Disabled:

Interface:

Test plan

Tested locally, enabling and disabling through interface and manually in VS Code settings

[umpox]

Doesn't affect our ranking, just means users can't trigger this automatically with a keyboard shortcut (wasn't happening anyway AFAICT)

[umpox]

Means that the code action will appear right alongside the fix action, no other impact

[abeatrix]

Currently Ask Cody to Explain does nothing when inline chat is disabled. Should we show a pop-up or fall back to sidebar chat when inline chat is not enabled?

pull.1211.mov

When inline chat is enabled again:

[toolmantim]

Suggested change

	'Enable Cody to fix or explain errors and warnings, directly in your code.',
	'Enable Cody fix and explain options in the Quick Fix menu',

or:

Suggested change

	'Enable Cody to fix or explain errors and warnings, directly in your code.',
	'Enable Cody in the Quick Fix menu for fixing and explaining code errors and warnings',

[umpox]

Thanks! I like the first suggestion, will update

[abeatrix]

awesome!
Not a block: is this a good place to register code action to listen for inline cofig change?

cody/vscode/src/services/InlineController.ts

Lines 76 to 89 in 0176b8c

	vscode.workspace.onDidChangeConfiguration(e => {
	const config = vscode.workspace.getConfiguration('cody')
	if (e.affectsConfiguration('cody')) {
	// Inline Chat
	const enableInlineChat = config.get('inlineChat.enabled') as boolean
	if (enableInlineChat) {
	this.commentController = this.init()
	return
	}
	this.commentController?.dispose()
	this.commentController = null
	this.dispose()
	}
	})

[umpox]

@abeatrix Thought about it but I think it's OK to just keep the basic implementation for now. Doesn't feel like users will often enable/disable inline chat, and the extra overhead of having to register/deregister the action to ensure we don't have duplicate code actions isn't worth it IMO. We can adjust if we get feedback from users that we need this