Remove experimental LLM backend (#298)

* Remove experimental LLM backend * Fix
mattermost · Feb 17, 2025 · ab826dc · ab826dc
1 parent 00e0c95
commit ab826dc
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 376 deletions.
diff --git a/server/asksage/asksage.go b/server/asksage/asksage.go
diff --git a/server/asksage/client.go b/server/asksage/client.go
diff --git a/server/llm/configuration.go b/server/llm/configuration.go
@@ -44,7 +44,6 @@ const (
 	ServiceTypeOpenAI           = "openai"
 	ServiceTypeOpenAICompatible = "openaicompatible"
 	ServiceTypeAzure            = "azure"
-	ServiceTypeAskSage          = "asksage"
 	ServiceTypeAnthropic        = "anthropic"
 )
 
@@ -88,8 +87,6 @@ func (c *BotConfig) IsValid() bool {
 		return c.Service.APIKey != "" && c.Service.APIURL != ""
 	case ServiceTypeAnthropic:
 		return c.Service.APIKey != ""
-	case ServiceTypeAskSage:
-		return c.Service.Username != "" && c.Service.Password != ""
 	default:
 		return false
 	}

diff --git a/server/llm/configuration_test.go b/server/llm/configuration_test.go
@@ -263,48 +263,6 @@ func TestBotConfig_IsValid(t *testing.T) {
 			},
 			want: true,
 		},
-		{
-			name: "Ask Sage service requires username to be set",
-			fields: fields{
-				ID:                 "xxx",
-				Name:               "xxx",
-				DisplayName:        "xxx",
-				CustomInstructions: "",
-				Service: ServiceConfig{
-					Name:                    "Copilot",
-					Type:                    "asksage",
-					Username:                "", // bad
-					Password:                "topsecret",
-					DefaultModel:            "xxx",
-					InputTokenLimit:         100,
-					StreamingTimeoutSeconds: 60,
-				},
-				ChannelAccessLevel: ChannelAccessLevelAll,
-				UserAccessLevel:    UserAccessLevelAll,
-			},
-			want: false,
-		},
-		{
-			name: "Ask Sage service requires password to be set",
-			fields: fields{
-				ID:                 "xxx",
-				Name:               "xxx",
-				DisplayName:        "xxx",
-				CustomInstructions: "",
-				Service: ServiceConfig{
-					Name:                    "Copilot",
-					Type:                    "asksage",
-					Username:                "myuser",
-					Password:                "", // bad
-					DefaultModel:            "xxx",
-					InputTokenLimit:         100,
-					StreamingTimeoutSeconds: 60,
-				},
-				ChannelAccessLevel: ChannelAccessLevelAll,
-				UserAccessLevel:    UserAccessLevelAll,
-			},
-			want: false,
-		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {

diff --git a/server/plugin.go b/server/plugin.go
@@ -15,7 +15,6 @@ import (
 	sq "github.com/Masterminds/squirrel"
 	"github.com/jmoiron/sqlx"
 	"github.com/mattermost/mattermost-plugin-ai/server/anthropic"
-	"github.com/mattermost/mattermost-plugin-ai/server/asksage"
 	"github.com/mattermost/mattermost-plugin-ai/server/enterprise"
 	"github.com/mattermost/mattermost-plugin-ai/server/llm"
 	"github.com/mattermost/mattermost-plugin-ai/server/metrics"
@@ -147,8 +146,6 @@ func (p *Plugin) getLLM(llmBotConfig llm.BotConfig) llm.LanguageModel {
 		result = openai.NewAzure(llmBotConfig.Service, p.llmUpstreamHTTPClient, llmMetrics)
 	case llm.ServiceTypeAnthropic:
 		result = anthropic.New(llmBotConfig.Service, p.llmUpstreamHTTPClient, llmMetrics)
-	case llm.ServiceTypeAskSage:
-		result = asksage.New(llmBotConfig.Service, p.llmUpstreamHTTPClient, llmMetrics)
 	}
 
 	cfg := p.getConfiguration()