feat: add support for Github Copilot as a provider

[caezium]

You can now authenticate with Github to use Github Copilot as a model provider.

• Authenticate with one button in the bottom of the Api keys setting
• Add model with Github Copilot selected as the provider
• and use

#506

does not support streaming as it already bypasses CORS
does not support auto-getting available models

(why does typing a with 4.1 give me sometimes this language or chinese)

[logancyang]

bugbot run

[caezium]

bugbot run :(

[logancyang]

bugbot run

[caezium]

should be fixed

[logancyang]

bugbot run

[cursor]

Bug: Refactoring Breaks Provider Validation

The createModelInstance method was refactored, introducing a call to the undefined this.getProviderConstructor(model), which causes a runtime error for all non-GitHub Copilot providers. This refactoring also removed critical validation checks for model existence and API key presence, leading to less informative errors or failures when API keys are missing for these providers.

src/LLMProviders/chatModelManager.ts#L363-L378

obsidian-copilot/src/LLMProviders/chatModelManager.ts

Lines 363 to 378 in ba51103

	new Notice(`Failed to set chat model: ${e.message}`);
	ChatModelManager.chatModel = null;
	}
	}
	async createModelInstance(model: CustomModel): Promise<BaseChatModel> {
	if (model.provider === ChatModelProviders.GITHUB_COPILOT) {
	const provider = new GitHubCopilotProvider();
	return new CopilotChatModel({ provider, modelName: model.name });
	}
	const AIConstructor = this.getProviderConstructor(model);
	const config = await this.getModelConfig(model);
	return new AIConstructor(config);
	}

Fix in Cursor

---

Bug: Constructor Ignores Config, Provider Uninitialized

The ChatGitHubCopilot class's constructor ignores its config parameter, creating an unconfigured GitHubCopilotProvider. This suggests the class is either unused or incorrectly implemented.

src/LLMProviders/githubCopilotChatModel.ts#L84-L108

obsidian-copilot/src/LLMProviders/githubCopilotChatModel.ts

Lines 84 to 108 in ba51103

	export class ChatGitHubCopilot {
	private provider: GitHubCopilotProvider;
	constructor(config: any) {
	this.provider = new GitHubCopilotProvider();
	}
	async send(messages: { role: string; content: string }[], model = "gpt-4") {
	return this.provider.sendChatMessage(messages, model);
	}
	getAuthState() {
	return this.provider.getAuthState();
	}
	async startAuth() {
	return this.provider.startDeviceCodeFlow();
	}
	async pollForAccessToken() {
	return this.provider.pollForAccessToken();
	}
	async fetchCopilotToken() {
	return this.provider.fetchCopilotToken();
	}
	resetAuth() {
	this.provider.resetAuth();
	}
	}

Fix in Cursor

---

Was this report helpful? Give feedback by reacting with 👍 or 👎

[caezium]

omg mb

[caezium]

fixed. i tested the other models too. do i have to provide api keys for the integration test that keeps failing?

[Defiect]

Can't wait for this to get merged

[Emt-lin]

sorry, i will review the code later

[caezium]

ok ty, sorry if my code isn't perfect

[z4m1]

I've been really looking forward to this feature. What's the current status?