Add support for local Ollama models and unify commit message generation

[tsdevau]

Resolves issue #37

• Added dual provider support with a new diffCommit.provider configuration option
• Implemented Ollama integration with model selection and server configuration
• Added new commands for Ollama setup and model switching:
  • DiffCommit: Configure Ollama Model
  • DiffCommit: Change Ollama Model
• Added new configuration settings:
  • diffCommit.ollamaHostname for server connection
  • diffCommit.ollamaModel for model selection
• Updated documentation with Ollama requirements and setup instructions
• Enhanced error handling for Ollama-specific scenarios
• Updated workflow documentation to include provider selection

[Copilot]

Pull Request Overview

This PR adds support for using local Ollama models alongside Anthropic, unifies commit message generation logic, and updates tests and docs accordingly.

• Introduces diffCommit.provider for selecting Anthropic or Ollama
• Implements OllamaManager with setup/change commands and error handling
• Refactors CommitMessageGenerator to handle both providers and updates related tests/docs

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
test/withProgressAPI.test.ts	Updated progress message from API key validation to configuration
test/ollamaManager.test.ts	Added tests for Ollama model config and error scenarios
test/messageHandling.test.ts	Mocked default Ollama config in message handling tests
test/gitIntegration.test.ts	Aligned default config values (provider/hostname/model)
test/gitAndCommands.test.ts	Updated command registration tests for new Ollama commands
test/errorHandling.test.ts	Mocked Ollama API errors and added handling tests
test/configurationHandling.test.ts	Included provider and Ollama config in configuration tests
test/anthropicResponseHandling.test.ts	Adjusted console log assertions for separated token logs
src/ollamaManager.ts	New OllamaManager implementation for model setup and error handling
src/extension.ts	Integrated Ollama commands and extended commit flow for providers
src/configManager.ts	Extended config manager to include provider, hostname, and model
src/commitMessageGenerator.ts	Unified generator class for Anthropic and Ollama with prompt builder
package.json	Added Ollama dependency, provider config, and updated extension desc
README.md	Documented Ollama support, commands, and updated usage instructions

Comments suppressed due to low confidence (2)

src/ollamaManager.ts:2

• Importing console is unnecessary since it's a global in Node/VSCode; remove this import and use the global console directly.

import console from "console"

src/commitMessageGenerator.ts:20

• The overload signatures declare parameters but the implementation signature takes none. Update the implementation to constructor(...args: any[]) so it matches the overloads.

constructor() {

[tsdevau]

OK to merge for feature addition of a new alternate provider being local Ollama models for an offline and cost free option to resolve issue #37.

[tsdevau]

Resolves issue #37

• Added dual provider support with a new diffCommit.provider configuration option
• Implemented Ollama integration with model selection and server configuration
• Added new commands for Ollama setup and model switching:
  • DiffCommit: Configure Ollama Model
  • DiffCommit: Change Ollama Model
• Added new configuration settings:
  • diffCommit.ollamaHostname for server connection
  • diffCommit.ollamaModel for model selection
• Updated documentation with Ollama requirements and setup instructions
• Enhanced error handling for Ollama-specific scenarios
• Updated workflow documentation to include provider selection

[Copilot]

Pull Request Overview

This PR adds support for using local Ollama models alongside Anthropic, unifies commit message generation logic, and updates tests and docs accordingly.

• Introduces diffCommit.provider for selecting Anthropic or Ollama
• Implements OllamaManager with setup/change commands and error handling
• Refactors CommitMessageGenerator to handle both providers and updates related tests/docs

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
test/withProgressAPI.test.ts	Updated progress message from API key validation to configuration
test/ollamaManager.test.ts	Added tests for Ollama model config and error scenarios
test/messageHandling.test.ts	Mocked default Ollama config in message handling tests
test/gitIntegration.test.ts	Aligned default config values (provider/hostname/model)
test/gitAndCommands.test.ts	Updated command registration tests for new Ollama commands
test/errorHandling.test.ts	Mocked Ollama API errors and added handling tests
test/configurationHandling.test.ts	Included provider and Ollama config in configuration tests
test/anthropicResponseHandling.test.ts	Adjusted console log assertions for separated token logs
src/ollamaManager.ts	New OllamaManager implementation for model setup and error handling
src/extension.ts	Integrated Ollama commands and extended commit flow for providers
src/configManager.ts	Extended config manager to include provider, hostname, and model
src/commitMessageGenerator.ts	Unified generator class for Anthropic and Ollama with prompt builder
package.json	Added Ollama dependency, provider config, and updated extension desc
README.md	Documented Ollama support, commands, and updated usage instructions

Comments suppressed due to low confidence (2)

src/ollamaManager.ts:2

• Importing console is unnecessary since it's a global in Node/VSCode; remove this import and use the global console directly.

import console from "console"

src/commitMessageGenerator.ts:20

• The overload signatures declare parameters but the implementation signature takes none. Update the implementation to constructor(...args: any[]) so it matches the overloads.

constructor() {

[tsdevau]

OK to merge for feature addition of a new alternate provider being local Ollama models for an offline and cost free option to resolve issue #37.

[tsdevau]

Resolves issue #37

• Added dual provider support with a new diffCommit.provider configuration option
• Implemented Ollama integration with model selection and server configuration
• Added new commands for Ollama setup and model switching:
  • DiffCommit: Configure Ollama Model
  • DiffCommit: Change Ollama Model
• Added new configuration settings:
  • diffCommit.ollamaHostname for server connection
  • diffCommit.ollamaModel for model selection
• Updated documentation with Ollama requirements and setup instructions
• Enhanced error handling for Ollama-specific scenarios
• Updated workflow documentation to include provider selection

[Copilot]

Pull Request Overview

This PR adds support for using local Ollama models alongside Anthropic, unifies commit message generation logic, and updates tests and docs accordingly.

• Introduces diffCommit.provider for selecting Anthropic or Ollama
• Implements OllamaManager with setup/change commands and error handling
• Refactors CommitMessageGenerator to handle both providers and updates related tests/docs

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
test/withProgressAPI.test.ts	Updated progress message from API key validation to configuration
test/ollamaManager.test.ts	Added tests for Ollama model config and error scenarios
test/messageHandling.test.ts	Mocked default Ollama config in message handling tests
test/gitIntegration.test.ts	Aligned default config values (provider/hostname/model)
test/gitAndCommands.test.ts	Updated command registration tests for new Ollama commands
test/errorHandling.test.ts	Mocked Ollama API errors and added handling tests
test/configurationHandling.test.ts	Included provider and Ollama config in configuration tests
test/anthropicResponseHandling.test.ts	Adjusted console log assertions for separated token logs
src/ollamaManager.ts	New OllamaManager implementation for model setup and error handling
src/extension.ts	Integrated Ollama commands and extended commit flow for providers
src/configManager.ts	Extended config manager to include provider, hostname, and model
src/commitMessageGenerator.ts	Unified generator class for Anthropic and Ollama with prompt builder
package.json	Added Ollama dependency, provider config, and updated extension desc
README.md	Documented Ollama support, commands, and updated usage instructions

Comments suppressed due to low confidence (2)

src/ollamaManager.ts:2

• Importing console is unnecessary since it's a global in Node/VSCode; remove this import and use the global console directly.

import console from "console"

src/commitMessageGenerator.ts:20

• The overload signatures declare parameters but the implementation signature takes none. Update the implementation to constructor(...args: any[]) so it matches the overloads.

constructor() {

[tsdevau]

OK to merge for feature addition of a new alternate provider being local Ollama models for an offline and cost free option to resolve issue #37.

[tsdevau]

Resolves issue #37

• Added dual provider support with a new diffCommit.provider configuration option
• Implemented Ollama integration with model selection and server configuration
• Added new commands for Ollama setup and model switching:
  • DiffCommit: Configure Ollama Model
  • DiffCommit: Change Ollama Model
• Added new configuration settings:
  • diffCommit.ollamaHostname for server connection
  • diffCommit.ollamaModel for model selection
• Updated documentation with Ollama requirements and setup instructions
• Enhanced error handling for Ollama-specific scenarios
• Updated workflow documentation to include provider selection

[Copilot]

Pull Request Overview

This PR adds support for using local Ollama models alongside Anthropic, unifies commit message generation logic, and updates tests and docs accordingly.

• Introduces diffCommit.provider for selecting Anthropic or Ollama
• Implements OllamaManager with setup/change commands and error handling
• Refactors CommitMessageGenerator to handle both providers and updates related tests/docs

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
test/withProgressAPI.test.ts	Updated progress message from API key validation to configuration
test/ollamaManager.test.ts	Added tests for Ollama model config and error scenarios
test/messageHandling.test.ts	Mocked default Ollama config in message handling tests
test/gitIntegration.test.ts	Aligned default config values (provider/hostname/model)
test/gitAndCommands.test.ts	Updated command registration tests for new Ollama commands
test/errorHandling.test.ts	Mocked Ollama API errors and added handling tests
test/configurationHandling.test.ts	Included provider and Ollama config in configuration tests
test/anthropicResponseHandling.test.ts	Adjusted console log assertions for separated token logs
src/ollamaManager.ts	New OllamaManager implementation for model setup and error handling
src/extension.ts	Integrated Ollama commands and extended commit flow for providers
src/configManager.ts	Extended config manager to include provider, hostname, and model
src/commitMessageGenerator.ts	Unified generator class for Anthropic and Ollama with prompt builder
package.json	Added Ollama dependency, provider config, and updated extension desc
README.md	Documented Ollama support, commands, and updated usage instructions

Comments suppressed due to low confidence (2)

src/ollamaManager.ts:2

• Importing console is unnecessary since it's a global in Node/VSCode; remove this import and use the global console directly.

import console from "console"

src/commitMessageGenerator.ts:20

• The overload signatures declare parameters but the implementation signature takes none. Update the implementation to constructor(...args: any[]) so it matches the overloads.

constructor() {

[tsdevau]

OK to merge for feature addition of a new alternate provider being local Ollama models for an offline and cost free option to resolve issue #37.

[tsdevau]

Resolves issue #37

• Added dual provider support with a new diffCommit.provider configuration option
• Implemented Ollama integration with model selection and server configuration
• Added new commands for Ollama setup and model switching:
  • DiffCommit: Configure Ollama Model
  • DiffCommit: Change Ollama Model
• Added new configuration settings:
  • diffCommit.ollamaHostname for server connection
  • diffCommit.ollamaModel for model selection
• Updated documentation with Ollama requirements and setup instructions
• Enhanced error handling for Ollama-specific scenarios
• Updated workflow documentation to include provider selection

[Copilot]

Pull Request Overview

This PR adds support for using local Ollama models alongside Anthropic, unifies commit message generation logic, and updates tests and docs accordingly.

• Introduces diffCommit.provider for selecting Anthropic or Ollama
• Implements OllamaManager with setup/change commands and error handling
• Refactors CommitMessageGenerator to handle both providers and updates related tests/docs

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
test/withProgressAPI.test.ts	Updated progress message from API key validation to configuration
test/ollamaManager.test.ts	Added tests for Ollama model config and error scenarios
test/messageHandling.test.ts	Mocked default Ollama config in message handling tests
test/gitIntegration.test.ts	Aligned default config values (provider/hostname/model)
test/gitAndCommands.test.ts	Updated command registration tests for new Ollama commands
test/errorHandling.test.ts	Mocked Ollama API errors and added handling tests
test/configurationHandling.test.ts	Included provider and Ollama config in configuration tests
test/anthropicResponseHandling.test.ts	Adjusted console log assertions for separated token logs
src/ollamaManager.ts	New OllamaManager implementation for model setup and error handling
src/extension.ts	Integrated Ollama commands and extended commit flow for providers
src/configManager.ts	Extended config manager to include provider, hostname, and model
src/commitMessageGenerator.ts	Unified generator class for Anthropic and Ollama with prompt builder
package.json	Added Ollama dependency, provider config, and updated extension desc
README.md	Documented Ollama support, commands, and updated usage instructions

Comments suppressed due to low confidence (2)

src/ollamaManager.ts:2

• Importing console is unnecessary since it's a global in Node/VSCode; remove this import and use the global console directly.

import console from "console"

src/commitMessageGenerator.ts:20

• The overload signatures declare parameters but the implementation signature takes none. Update the implementation to constructor(...args: any[]) so it matches the overloads.

constructor() {

[tsdevau]

OK to merge for feature addition of a new alternate provider being local Ollama models for an offline and cost free option to resolve issue #37.

[tsdevau]

Resolves issue #37

• Added dual provider support with a new diffCommit.provider configuration option
• Implemented Ollama integration with model selection and server configuration
• Added new commands for Ollama setup and model switching:
  • DiffCommit: Configure Ollama Model
  • DiffCommit: Change Ollama Model
• Added new configuration settings:
  • diffCommit.ollamaHostname for server connection
  • diffCommit.ollamaModel for model selection
• Updated documentation with Ollama requirements and setup instructions
• Enhanced error handling for Ollama-specific scenarios
• Updated workflow documentation to include provider selection

[Copilot]

Pull Request Overview

This PR adds support for using local Ollama models alongside Anthropic, unifies commit message generation logic, and updates tests and docs accordingly.

• Introduces diffCommit.provider for selecting Anthropic or Ollama
• Implements OllamaManager with setup/change commands and error handling
• Refactors CommitMessageGenerator to handle both providers and updates related tests/docs

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
test/withProgressAPI.test.ts	Updated progress message from API key validation to configuration
test/ollamaManager.test.ts	Added tests for Ollama model config and error scenarios
test/messageHandling.test.ts	Mocked default Ollama config in message handling tests
test/gitIntegration.test.ts	Aligned default config values (provider/hostname/model)
test/gitAndCommands.test.ts	Updated command registration tests for new Ollama commands
test/errorHandling.test.ts	Mocked Ollama API errors and added handling tests
test/configurationHandling.test.ts	Included provider and Ollama config in configuration tests
test/anthropicResponseHandling.test.ts	Adjusted console log assertions for separated token logs
src/ollamaManager.ts	New OllamaManager implementation for model setup and error handling
src/extension.ts	Integrated Ollama commands and extended commit flow for providers
src/configManager.ts	Extended config manager to include provider, hostname, and model
src/commitMessageGenerator.ts	Unified generator class for Anthropic and Ollama with prompt builder
package.json	Added Ollama dependency, provider config, and updated extension desc
README.md	Documented Ollama support, commands, and updated usage instructions

Comments suppressed due to low confidence (2)

src/ollamaManager.ts:2

• Importing console is unnecessary since it's a global in Node/VSCode; remove this import and use the global console directly.

import console from "console"

src/commitMessageGenerator.ts:20

• The overload signatures declare parameters but the implementation signature takes none. Update the implementation to constructor(...args: any[]) so it matches the overloads.

constructor() {

[tsdevau]

OK to merge for feature addition of a new alternate provider being local Ollama models for an offline and cost free option to resolve issue #37.

[tsdevau]

Resolves issue #37

• Added dual provider support with a new diffCommit.provider configuration option
• Implemented Ollama integration with model selection and server configuration
• Added new commands for Ollama setup and model switching:
  • DiffCommit: Configure Ollama Model
  • DiffCommit: Change Ollama Model
• Added new configuration settings:
  • diffCommit.ollamaHostname for server connection
  • diffCommit.ollamaModel for model selection
• Updated documentation with Ollama requirements and setup instructions
• Enhanced error handling for Ollama-specific scenarios
• Updated workflow documentation to include provider selection

[Copilot]

Pull Request Overview

This PR adds support for using local Ollama models alongside Anthropic, unifies commit message generation logic, and updates tests and docs accordingly.

• Introduces diffCommit.provider for selecting Anthropic or Ollama
• Implements OllamaManager with setup/change commands and error handling
• Refactors CommitMessageGenerator to handle both providers and updates related tests/docs

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
test/withProgressAPI.test.ts	Updated progress message from API key validation to configuration
test/ollamaManager.test.ts	Added tests for Ollama model config and error scenarios
test/messageHandling.test.ts	Mocked default Ollama config in message handling tests
test/gitIntegration.test.ts	Aligned default config values (provider/hostname/model)
test/gitAndCommands.test.ts	Updated command registration tests for new Ollama commands
test/errorHandling.test.ts	Mocked Ollama API errors and added handling tests
test/configurationHandling.test.ts	Included provider and Ollama config in configuration tests
test/anthropicResponseHandling.test.ts	Adjusted console log assertions for separated token logs
src/ollamaManager.ts	New OllamaManager implementation for model setup and error handling
src/extension.ts	Integrated Ollama commands and extended commit flow for providers
src/configManager.ts	Extended config manager to include provider, hostname, and model
src/commitMessageGenerator.ts	Unified generator class for Anthropic and Ollama with prompt builder
package.json	Added Ollama dependency, provider config, and updated extension desc
README.md	Documented Ollama support, commands, and updated usage instructions

Comments suppressed due to low confidence (2)

src/ollamaManager.ts:2

• Importing console is unnecessary since it's a global in Node/VSCode; remove this import and use the global console directly.

import console from "console"

src/commitMessageGenerator.ts:20

• The overload signatures declare parameters but the implementation signature takes none. Update the implementation to constructor(...args: any[]) so it matches the overloads.

constructor() {

[tsdevau]

OK to merge for feature addition of a new alternate provider being local Ollama models for an offline and cost free option to resolve issue #37.

[tsdevau]

Resolves issue #37

• Added dual provider support with a new diffCommit.provider configuration option
• Implemented Ollama integration with model selection and server configuration
• Added new commands for Ollama setup and model switching:
  • DiffCommit: Configure Ollama Model
  • DiffCommit: Change Ollama Model
• Added new configuration settings:
  • diffCommit.ollamaHostname for server connection
  • diffCommit.ollamaModel for model selection
• Updated documentation with Ollama requirements and setup instructions
• Enhanced error handling for Ollama-specific scenarios
• Updated workflow documentation to include provider selection

[Copilot]

Pull Request Overview

This PR adds support for using local Ollama models alongside Anthropic, unifies commit message generation logic, and updates tests and docs accordingly.

• Introduces diffCommit.provider for selecting Anthropic or Ollama
• Implements OllamaManager with setup/change commands and error handling
• Refactors CommitMessageGenerator to handle both providers and updates related tests/docs

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
test/withProgressAPI.test.ts	Updated progress message from API key validation to configuration
test/ollamaManager.test.ts	Added tests for Ollama model config and error scenarios
test/messageHandling.test.ts	Mocked default Ollama config in message handling tests
test/gitIntegration.test.ts	Aligned default config values (provider/hostname/model)
test/gitAndCommands.test.ts	Updated command registration tests for new Ollama commands
test/errorHandling.test.ts	Mocked Ollama API errors and added handling tests
test/configurationHandling.test.ts	Included provider and Ollama config in configuration tests
test/anthropicResponseHandling.test.ts	Adjusted console log assertions for separated token logs
src/ollamaManager.ts	New OllamaManager implementation for model setup and error handling
src/extension.ts	Integrated Ollama commands and extended commit flow for providers
src/configManager.ts	Extended config manager to include provider, hostname, and model
src/commitMessageGenerator.ts	Unified generator class for Anthropic and Ollama with prompt builder
package.json	Added Ollama dependency, provider config, and updated extension desc
README.md	Documented Ollama support, commands, and updated usage instructions

Comments suppressed due to low confidence (2)

src/ollamaManager.ts:2

• Importing console is unnecessary since it's a global in Node/VSCode; remove this import and use the global console directly.

import console from "console"

src/commitMessageGenerator.ts:20

• The overload signatures declare parameters but the implementation signature takes none. Update the implementation to constructor(...args: any[]) so it matches the overloads.

constructor() {

[tsdevau]

OK to merge for feature addition of a new alternate provider being local Ollama models for an offline and cost free option to resolve issue #37.