mirror of
https://github.com/mattermost/mattermost.git
synced 2026-04-13 13:08:56 -04:00
* Add shared AI bridge seam
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Add AI bridge test helper API
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Add AI bridge seam test coverage
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Add Playwright AI bridge recap helpers
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Fix recap channel persistence test
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Restore bridge client compatibility shim
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Expand recap card in Playwright spec
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Recaps e2e test coverage (#35543)
* Add Recaps Playwright page object
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Expand AI recap Playwright coverage
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Format recap Playwright coverage
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Fix recap regeneration test flows
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
---------
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
* Fix AI bridge lint and OpenAPI docs
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Fix recap lint shadowing
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Stabilize failed recap regeneration spec
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Fill AI bridge i18n strings
Co-authored-by: Nick Misasi <nick13misasi@gmail.com>
* Fix i18n
* Add service completion bridge path and operation tracking fields
Extend AgentsBridge with CompleteService for service-based completions,
add ClientOperation/OperationSubType tracking to BridgeCompletionRequest,
and propagate operation metadata through to the bridge client.
Made-with: Cursor
* Fill empty i18n translation strings for enterprise keys
The previous "Fix i18n" commit added 145 i18n entries with empty
translation strings, causing the i18n check to fail in CI. Fill in
all translations based on the corresponding error messages in the
enterprise and server source code.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
* Fix i18n
* Fix i18n again
* Rename Complete/CompleteService to AgentCompletion/ServiceCompletion
Align the AgentsBridge interface method names with the underlying
bridge client methods they delegate to (AgentCompletion, ServiceCompletion).
Made-with: Cursor
* Refactor
* Add e2eAgentsBridge implementation
The new file was missed from the prior refactor commit.
Made-with: Cursor
* Address CodeRabbit review feedback
- Add 400 BadRequest response to AI bridge PUT endpoint OpenAPI spec
- Add missing client_operation, operation_sub_type, service_id fields to
AIBridgeTestHelperRecordedRequest schema
- Deep-clone nested JSON schema values in cloneJSONOutputFormat
- Populate ChannelID on recap summary bridge requests
- Fix msg_count assertion to mention_count for mark-as-read verification
- Make AgentCompletion/ServiceCompletion mutex usage atomic
Made-with: Cursor
* fix(playwright): align recaps page object with placeholder and channel menu
Made-with: Cursor
* fix(playwright): update recaps expectEmptyState to match RecapsList empty state
After the master merge, the recaps page now renders RecapsList's
"You're all caught up" empty state instead of the old placeholder.
Made-with: Cursor
* chore(playwright): update package-lock.json after npm install
Made-with: Cursor
* Revert "chore(playwright): update package-lock.json after npm install"
This reverts commit 95c670863a.
* style(playwright): fix prettier formatting in recaps page object
Made-with: Cursor
* fix(playwright): handle both recaps empty states correctly
The recaps page has two distinct empty states:
- Setup placeholder ("Set up your recap") when allRecaps is empty
- RecapsList caught-up state ("You're all caught up") when the
filtered tab list is empty
Split expectEmptyState into expectSetupPlaceholder and
expectCaughtUpEmptyState, used by the delete and bridge-unavailable
tests respectively.
Made-with: Cursor
---------
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
247 lines
7.2 KiB
Go
247 lines
7.2 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package app
|
|
|
|
import (
|
|
"github.com/blang/semver/v4"
|
|
|
|
agentclient "github.com/mattermost/mattermost-plugin-ai/public/bridgeclient"
|
|
"github.com/mattermost/mattermost/server/public/model"
|
|
"github.com/mattermost/mattermost/server/public/shared/mlog"
|
|
"github.com/mattermost/mattermost/server/public/shared/request"
|
|
)
|
|
|
|
const (
|
|
aiPluginID = "mattermost-ai"
|
|
minAIPluginVersionForBridge = "1.5.0"
|
|
)
|
|
|
|
type BridgeOperation string
|
|
|
|
const (
|
|
BridgeOperationAutoTranslate BridgeOperation = "auto_translate"
|
|
BridgeOperationRecapSummary BridgeOperation = "recap_summary"
|
|
BridgeOperationRewrite BridgeOperation = "rewrite"
|
|
)
|
|
|
|
type BridgeMessage struct {
|
|
Role string
|
|
Message string
|
|
FileIDs []string
|
|
}
|
|
|
|
type BridgeCompletionRequest struct {
|
|
Operation BridgeOperation
|
|
ClientOperation string
|
|
OperationSubType string
|
|
Messages []BridgeMessage
|
|
JSONOutputFormat map[string]any
|
|
UserID string
|
|
ChannelID string
|
|
}
|
|
|
|
type AgentsBridge interface {
|
|
Status(rctx request.CTX) (bool, string)
|
|
GetAgents(sessionUserID, userID string) ([]model.BridgeAgentInfo, error)
|
|
GetServices(sessionUserID, userID string) ([]model.BridgeServiceInfo, error)
|
|
AgentCompletion(sessionUserID, agentID string, req BridgeCompletionRequest) (string, error)
|
|
ServiceCompletion(sessionUserID, serviceID string, req BridgeCompletionRequest) (string, error)
|
|
}
|
|
|
|
type liveAgentsBridge struct {
|
|
ch *Channels
|
|
}
|
|
|
|
func newLiveAgentsBridge(ch *Channels) AgentsBridge {
|
|
return &liveAgentsBridge{ch: ch}
|
|
}
|
|
|
|
func (b *liveAgentsBridge) Status(rctx request.CTX) (bool, string) {
|
|
return b.getLiveStatus(rctx)
|
|
}
|
|
|
|
func (b *liveAgentsBridge) GetAgents(sessionUserID, userID string) ([]model.BridgeAgentInfo, error) {
|
|
if available, _ := b.getLiveStatus(request.EmptyContext(b.ch.srv.Log())); !available {
|
|
return []model.BridgeAgentInfo{}, nil
|
|
}
|
|
|
|
client := agentclient.NewClientFromApp(New(ServerConnector(b.ch)), sessionUserID)
|
|
agents, err := client.GetAgents(userID)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
return toModelBridgeAgents(agents), nil
|
|
}
|
|
|
|
func (b *liveAgentsBridge) GetServices(sessionUserID, userID string) ([]model.BridgeServiceInfo, error) {
|
|
if available, _ := b.getLiveStatus(request.EmptyContext(b.ch.srv.Log())); !available {
|
|
return []model.BridgeServiceInfo{}, nil
|
|
}
|
|
|
|
client := agentclient.NewClientFromApp(New(ServerConnector(b.ch)), sessionUserID)
|
|
services, err := client.GetServices(userID)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
return toModelBridgeServices(services), nil
|
|
}
|
|
|
|
func (b *liveAgentsBridge) AgentCompletion(sessionUserID, agentID string, req BridgeCompletionRequest) (string, error) {
|
|
client := agentclient.NewClientFromApp(New(ServerConnector(b.ch)), sessionUserID)
|
|
return client.AgentCompletion(agentID, toClientCompletionRequest(req))
|
|
}
|
|
|
|
func (b *liveAgentsBridge) ServiceCompletion(sessionUserID, serviceID string, req BridgeCompletionRequest) (string, error) {
|
|
client := agentclient.NewClientFromApp(New(ServerConnector(b.ch)), sessionUserID)
|
|
return client.ServiceCompletion(serviceID, toClientCompletionRequest(req))
|
|
}
|
|
|
|
func (b *liveAgentsBridge) getLiveStatus(rctx request.CTX) (bool, string) {
|
|
pluginsEnvironment := b.ch.GetPluginsEnvironment()
|
|
if pluginsEnvironment == nil {
|
|
rctx.Logger().Debug("AI plugin bridge not available - plugin environment not initialized")
|
|
return false, "app.agents.bridge.not_available.plugin_env_not_initialized"
|
|
}
|
|
|
|
if !pluginsEnvironment.IsActive(aiPluginID) {
|
|
rctx.Logger().Debug("AI plugin bridge not available - plugin is not active or not installed",
|
|
mlog.String("plugin_id", aiPluginID),
|
|
)
|
|
return false, "app.agents.bridge.not_available.plugin_not_active"
|
|
}
|
|
|
|
plugins := pluginsEnvironment.Active()
|
|
for _, plugin := range plugins {
|
|
if plugin.Manifest == nil || plugin.Manifest.Id != aiPluginID {
|
|
continue
|
|
}
|
|
|
|
pluginVersion, err := semver.Parse(plugin.Manifest.Version)
|
|
if err != nil {
|
|
rctx.Logger().Debug("AI plugin bridge not available - failed to parse plugin version",
|
|
mlog.String("plugin_id", aiPluginID),
|
|
mlog.String("version", plugin.Manifest.Version),
|
|
mlog.Err(err),
|
|
)
|
|
return false, "app.agents.bridge.not_available.plugin_version_parse_failed"
|
|
}
|
|
|
|
minVersion, err := semver.Parse(minAIPluginVersionForBridge)
|
|
if err != nil {
|
|
return false, "app.agents.bridge.not_available.min_version_parse_failed"
|
|
}
|
|
|
|
if pluginVersion.LT(minVersion) {
|
|
rctx.Logger().Debug("AI plugin bridge not available - plugin version is too old",
|
|
mlog.String("plugin_id", aiPluginID),
|
|
mlog.String("current_version", plugin.Manifest.Version),
|
|
mlog.String("minimum_version", minAIPluginVersionForBridge),
|
|
)
|
|
return false, "app.agents.bridge.not_available.plugin_version_too_old"
|
|
}
|
|
|
|
return true, ""
|
|
}
|
|
|
|
return false, "app.agents.bridge.not_available.plugin_not_registered"
|
|
}
|
|
|
|
func toModelBridgeAgents(agents []agentclient.BridgeAgentInfo) []model.BridgeAgentInfo {
|
|
if len(agents) == 0 {
|
|
return []model.BridgeAgentInfo{}
|
|
}
|
|
|
|
converted := make([]model.BridgeAgentInfo, 0, len(agents))
|
|
for _, agent := range agents {
|
|
converted = append(converted, model.BridgeAgentInfo{
|
|
ID: agent.ID,
|
|
DisplayName: agent.DisplayName,
|
|
Username: agent.Username,
|
|
ServiceID: agent.ServiceID,
|
|
ServiceType: agent.ServiceType,
|
|
IsDefault: agent.IsDefault,
|
|
})
|
|
}
|
|
|
|
return converted
|
|
}
|
|
|
|
func toModelBridgeServices(services []agentclient.BridgeServiceInfo) []model.BridgeServiceInfo {
|
|
if len(services) == 0 {
|
|
return []model.BridgeServiceInfo{}
|
|
}
|
|
|
|
converted := make([]model.BridgeServiceInfo, 0, len(services))
|
|
for _, service := range services {
|
|
converted = append(converted, model.BridgeServiceInfo{
|
|
ID: service.ID,
|
|
Name: service.Name,
|
|
Type: service.Type,
|
|
})
|
|
}
|
|
|
|
return converted
|
|
}
|
|
|
|
func toBridgeClientPosts(messages []BridgeMessage) []agentclient.Post {
|
|
posts := make([]agentclient.Post, 0, len(messages))
|
|
for _, message := range messages {
|
|
posts = append(posts, agentclient.Post{
|
|
Role: message.Role,
|
|
Message: message.Message,
|
|
FileIDs: append([]string(nil), message.FileIDs...),
|
|
})
|
|
}
|
|
|
|
return posts
|
|
}
|
|
|
|
func toClientCompletionRequest(req BridgeCompletionRequest) agentclient.CompletionRequest {
|
|
return agentclient.CompletionRequest{
|
|
Posts: toBridgeClientPosts(req.Messages),
|
|
JSONOutputFormat: cloneJSONOutputFormat(req.JSONOutputFormat),
|
|
UserID: req.UserID,
|
|
ChannelID: req.ChannelID,
|
|
Operation: req.ClientOperation,
|
|
OperationSubType: req.OperationSubType,
|
|
}
|
|
}
|
|
|
|
func cloneJSONOutputFormat(jsonOutputFormat map[string]any) map[string]any {
|
|
if jsonOutputFormat == nil {
|
|
return nil
|
|
}
|
|
|
|
cloned := make(map[string]any, len(jsonOutputFormat))
|
|
for key, value := range jsonOutputFormat {
|
|
cloned[key] = cloneJSONValue(value)
|
|
}
|
|
|
|
return cloned
|
|
}
|
|
|
|
func cloneJSONValue(value any) any {
|
|
switch v := value.(type) {
|
|
case map[string]any:
|
|
cloned := make(map[string]any, len(v))
|
|
for key, child := range v {
|
|
cloned[key] = cloneJSONValue(child)
|
|
}
|
|
return cloned
|
|
case []any:
|
|
cloned := make([]any, len(v))
|
|
for i, child := range v {
|
|
cloned[i] = cloneJSONValue(child)
|
|
}
|
|
return cloned
|
|
case []string:
|
|
return append([]string(nil), v...)
|
|
default:
|
|
return v
|
|
}
|
|
}
|
|
|
|
var _ AgentsBridge = (*liveAgentsBridge)(nil)
|