FEAT: Anthropic Claude support about wingman HOT 4 CLOSED

yes! I absolutely plan to support Claude, just not quite there yet. if you get to it before I do, that's totally cool!

from wingman.

I stubbed out support for Claude using the new Provider API, and the OpenAIProvider can serve as an example for how to implement a new provider (if you are interested):

• Provider: https://github.com/nvms/wingman/blob/8817a200a0fa674e066ec2519e96bdb639cd10a2/src/providers/index.ts
• OpenAIProvider: https://github.com/nvms/wingman/blob/8817a200a0fa674e066ec2519e96bdb639cd10a2/src/providers/openai.ts

I still don't have access to Claude yet, so apart from the above, I haven't been able to really dig into this yet

from wingman.

@nvms I've started taking an initial look at this. Full disclosure: I have exactly zero experience with JS/TS or vscode extensions so I'm kind of learning it and the relevant tooling on the go.

The main thing that's blocking me is the fact that the anthropic/claude API has no counterpart to the OpenAI conversation tracking stuff. It's much simpler/more limited. Each request exists on its own and is basically just a single blob of text with alternating User: ... and Assistant: ... blocks. So, for a conversation I'd basically need to manually build the next prompt by keeping the current state somewhere and appending new questions and responses at each step.

It's not clear to me where this state should live, or when/how to clear it.

from wingman.

no counterpart to the OpenAI conversation tracking stuff

ah, I wasn't aware of this. I think your method of preserving state is exactly what you'll want to do, then.

maybe something like this?:

• in create, set state to an empty string (e.g. this.conversationState = "")
• when a complete response is received, concatenate everything onto this string (e.g. this.conversationState = this.conversationState + "\n\nHuman: <last question>\n\nAssistant: <last reply>"
• the next time you send a prompt, prefix it with conversationState.
• the place to clear this state would be the Provider's destroy method, which is called before every new conversation:

  wingman/src/views.ts

  Lines 202 to 204 in b93e22b

  	// Every command run should instantiate a new provider, which is why we call destroy here.
  	getProviderInstance()?.destroy();
  	getProviderInstance()?.create(this as any, command);

from wingman.