fix: add refType parameter to configuration_get tool so version tags resolve correctly

src/apis/backendClient.test.ts

@@ -323,6 +323,20 @@ suite('Backend Internal Client', () => {
     t.assert.deepStrictEqual(environmentResult, mockedResult)
   })
 
+  test('get configuration by version', async (t: TestContext) => {
+    const mockedResult = {
+      projectId,
+    }
+
+    agent.get(internalEndpoint).intercept({
+      path: `/api/backend/projects/${projectId}/versions/${refId}/configuration`,
+      method: 'GET',
+    }).reply(200, mockedResult)
+
+    const versionResult = await client.getRevisionBasedConfiguration(projectId, refId, 'versions')
+    t.assert.deepStrictEqual(versionResult, mockedResult)
+  })
+
   test('get configuration must thrown if the API call fails', async (t: TestContext) => {
     agent.get(internalEndpoint).intercept({
       path: `/api/backend/projects/${projectId}/revisions/${refId}/configuration`,
@@ -571,6 +585,20 @@ suite('Backend Client', () => {
     t.assert.deepStrictEqual(environmentResult, mockedResult)
   })
 
+  test('get configuration by version', async (t: TestContext) => {
+    const mockedResult = {
+      projectId,
+    }
+
+    agent.get(mockedEndpoint).intercept({
+      path: `/api/backend/projects/${projectId}/versions/${refId}/configuration`,
+      method: 'GET',
+    }).reply(200, mockedResult)
+
+    const versionResult = await client.getRevisionBasedConfiguration(projectId, refId, 'versions')
+    t.assert.deepStrictEqual(versionResult, mockedResult)
+  })
+
   test('save configuration', async (t: TestContext) => {
     const data = {
       title: 'title',

src/apis/backendClient.ts

@@ -20,7 +20,7 @@ import {
 } from '@mia-platform/console-types'
 
 import { HTTPClient } from './http-client'
-import { ConfigToSave, RetrievedConfiguration, SaveResponse } from './types/configuration'
+import { ConfigToSave, ConfigurationRefType, RetrievedConfiguration, SaveResponse } from './types/configuration'
 import { PostProject, ProjectDraft, Template } from './types/governance'
 
 export const internalEndpoint = process.env.BACKEND_INTERNAL_ENDPOINT || 'http://internal.local:3000'
@@ -92,8 +92,12 @@ export class BackendClient {
     )
   }
 
-  getRevisionBasedConfiguration (prjID: string, refID: string): Promise<RetrievedConfiguration> {
-    return this.#client.get<RetrievedConfiguration>(this.#revisionConfigurationPath(prjID, refID))
+  getRevisionBasedConfiguration (
+    prjID: string,
+    refID: string,
+    refType: ConfigurationRefType = 'revisions',
+  ): Promise<RetrievedConfiguration> {
+    return this.#client.get<RetrievedConfiguration>(this.#revisionConfigurationPath(prjID, refID, refType))
   }
 
   getEnvironmentBasedConfiguration (prjID: string, refID: string): Promise<RetrievedConfiguration> {
@@ -150,8 +154,8 @@ export class BackendClient {
     return this.#client.get<Record<string, unknown>>(this.#companyRulesPath(tenantID), new URLSearchParams({}))
   }
 
-  #revisionConfigurationPath (prjID: string, refID: string): string {
-    return `/api/backend/projects/${prjID}/revisions/${encodeURIComponent(refID)}/configuration`
+  #revisionConfigurationPath (prjID: string, refID: string, refType: ConfigurationRefType = 'revisions'): string {
+    return `/api/backend/projects/${prjID}/${refType}/${encodeURIComponent(refID)}/configuration`
   }
 
   #environmentConfigurationPath (prjID: string, refID: string): string {

src/apis/client.ts

@@ -41,6 +41,7 @@ import { CompareForDeployResponse, PipelineStatus, TriggerDeployResponse } from
 import {
   Config,
   ConfigToSave,
+  ConfigurationRefType,
   DockerSuggestionPrefix,
   ResourcesToCreate,
   RetrievedConfiguration,
@@ -89,7 +90,7 @@ export interface IAPIClient {
 
   // #region Configuration Methods
   getConfigurationRevisions(projectId: string): Promise<Record<string, unknown>>
-  getConfiguration(projectId: string, refId: string): Promise<RetrievedConfiguration>
+  getConfiguration(projectId: string, refId: string, refType?: ConfigurationRefType): Promise<RetrievedConfiguration>
   saveConfiguration(
     projectId: string,
     refId: string,
@@ -294,13 +295,17 @@ export class APIClient implements IAPIClient {
     return await this.saveConfiguration(projectID, refID, resourcesToCreate)
   }
 
-  async getConfiguration (prjID: string, refID: string): Promise<RetrievedConfiguration> {
+  async getConfiguration (
+    prjID: string,
+    refID: string,
+    refType: ConfigurationRefType = 'revisions',
+  ): Promise<RetrievedConfiguration> {
     const ft = await this.#featureFlagsClient.getToggles(prjID, [ ENABLE_ENVIRONMENT_BASED_CONFIGURATION_MANAGEMENT ])
     if (ft[ENABLE_ENVIRONMENT_BASED_CONFIGURATION_MANAGEMENT] || false) {
       return this.#backendClient.getEnvironmentBasedConfiguration(prjID, refID)
     }
 
-    return this.#backendClient.getRevisionBasedConfiguration(prjID, refID)
+    return this.#backendClient.getRevisionBasedConfiguration(prjID, refID, refType)
   }
 
   async saveConfiguration (
@@ -808,7 +813,7 @@ export interface APIClientMockFunctions {
 
   // #region Configuration Methods
   getConfigurationRevisionsMockFn?: (projectId: string) => Promise<Record<string, unknown>>
-  getConfigurationMockFn?: (projectId: string, refId: string) => Promise<RetrievedConfiguration>
+  getConfigurationMockFn?: (projectId: string, refId: string, refType?: ConfigurationRefType) => Promise<RetrievedConfiguration>
   saveConfigurationMockFn?: (projectId: string) => Promise<SaveResponse>
   createServiceFromMarketplaceItemMockFn?: (projectID: string) => Promise<SaveResponse>
   createEndpointsMockFn?: (projectID: string) => Promise<SaveResponse>
@@ -945,12 +950,12 @@ export class APIClientMock implements IAPIClient {
   }
 
 
-  async getConfiguration (projectId: string, refId: string): Promise<RetrievedConfiguration> {
+  async getConfiguration (projectId: string, refId: string, refType?: ConfigurationRefType): Promise<RetrievedConfiguration> {
     if (!this.mocks.getConfigurationMockFn) {
       throw new Error('getConfigurationMockFn not mocked')
     }
 
-    return this.mocks.getConfigurationMockFn(projectId, refId)
+    return this.mocks.getConfigurationMockFn(projectId, refId, refType)
   }
 
   async saveConfiguration (

src/apis/types/configuration.ts

@@ -26,6 +26,8 @@ import {
 
 export type Config = ConfigType
 
+export type ConfigurationRefType = 'revisions' | 'versions'
+
 export type RetrievedConfiguration = ConfigType & {
   fastDataConfig: unknown
   microfrontendPluginsConfig: unknown

src/tools/configuration/index.test.ts

@@ -385,6 +385,81 @@ suite('get configuration tool', () => {
     ])
   })
 
+  it('should default refType to "revisions" when not provided', async (t: it.TestContext) => {
+    const testTenantId = 'tenant123'
+    const testProjectId = 'project123'
+    const refId = 'main'
+
+    const getProjectInfoMockFn = mock.fn(async (projectId: string) => {
+      return {
+        id: projectId,
+        tenantId: testTenantId,
+      } as unknown as IProject
+    })
+    const aiFeaturesMockFn = mock.fn(async () => true)
+    const refTypeAwareGetConfigurationMockFn =
+      mock.fn(async (_projectId: string, _refId: string, _refType?: string): Promise<RetrievedConfiguration> => {
+        return mockConfiguration as unknown as RetrievedConfiguration
+      })
+
+    const client = await getTestMCPServerClient({
+      getConfigurationMockFn: refTypeAwareGetConfigurationMockFn,
+      getProjectInfoMockFn,
+      isAiFeaturesEnabledForTenantMockFn: aiFeaturesMockFn,
+    })
+    await client.request({
+      method: 'tools/call',
+      params: {
+        name: 'configuration_get',
+        arguments: {
+          projectId: testProjectId,
+          refId,
+        },
+      },
+    }, CallToolResultSchema)
+
+    t.assert.equal(refTypeAwareGetConfigurationMockFn.mock.callCount(), 1)
+    t.assert.deepEqual(refTypeAwareGetConfigurationMockFn.mock.calls[0].arguments, [ testProjectId, refId, 'revisions' ])
+  })
+
+  it('should map refType "version" to "versions" when retrieving configuration from a version', async (t: it.TestContext) => {
+    const testTenantId = 'tenant123'
+    const testProjectId = 'project123'
+    const refId = 'v1.0.0'
+
+    const getProjectInfoMockFn = mock.fn(async (projectId: string) => {
+      return {
+        id: projectId,
+        tenantId: testTenantId,
+      } as unknown as IProject
+    })
+    const aiFeaturesMockFn = mock.fn(async () => true)
+    const refTypeAwareGetConfigurationMockFn =
+      mock.fn(async (_projectId: string, _refId: string, _refType?: string): Promise<RetrievedConfiguration> => {
+        return mockConfiguration as unknown as RetrievedConfiguration
+      })
+
+    const client = await getTestMCPServerClient({
+      getConfigurationMockFn: refTypeAwareGetConfigurationMockFn,
+      getProjectInfoMockFn,
+      isAiFeaturesEnabledForTenantMockFn: aiFeaturesMockFn,
+    })
+    await client.request({
+      method: 'tools/call',
+      params: {
+        name: 'configuration_get',
+        arguments: {
+          projectId: testProjectId,
+          refId,
+          refType: 'version',
+        },
+      },
+    }, CallToolResultSchema)
+
+    t.assert.equal(refTypeAwareGetConfigurationMockFn.mock.callCount(), 1)
+    t.assert.deepEqual(refTypeAwareGetConfigurationMockFn.mock.calls[0].arguments, [ testProjectId, refId, 'versions' ])
+  })
+
   it('should return error message if API request fails', async (t: it.TestContext) => {
     const testTenantId = 'tenant123'
     const testProjectId = 'error-project'

src/tools/configuration/index.ts

@@ -67,13 +67,17 @@ export function addConfigurationCapabilities (server: McpServer, client: IAPICli
     {
       projectId: z.string().describe(paramsDescriptions.PROJECT_ID),
       refId: z.string().describe(paramsDescriptions.REF_ID),
+      refType: z.enum([ 'revision', 'version' ]).optional().default('revision').describe(paramsDescriptions.REF_TYPE),
     },
-    async ({ projectId, refId }): Promise<CallToolResult> => {
+    async ({ projectId, refId, refType }): Promise<CallToolResult> => {
       try {
         const project = await client.projectInfo(projectId)
         await assertAiFeaturesEnabledForProject(client, project)
 
-        const config = await client.getConfiguration(projectId, refId)
+        const backendRefType = refType === 'version'
+          ? 'versions'
+          : 'revisions'
+        const config = await client.getConfiguration(projectId, refId, backendRefType)
         return {
           structuredContent: config,
           content: [