Improve docName and mcpServerName

src/api/models.ts

@@ -42,6 +42,7 @@ export interface VersionRequest {
 }
 
 export interface VersionResponse {
+  doc_name: string
   doc_public_url?: string
   id: string
 }
@@ -65,6 +66,7 @@ export interface DiffRequest {
 export interface DiffResponse {
   breaking?: boolean
   details?: DiffItem[]
+  doc_name?: string
   html?: string
   id: string
   markdown?: string
@@ -92,4 +94,5 @@ export interface WorkflowVersionRequest {
 export interface WorkflowVersionResponse {
   id: string
   mcp_server_id: string
+  mcp_server_name: string
 }

src/commands/deploy.ts

@@ -76,7 +76,7 @@
     token: flagsBuilder.token(),
   }
 
   protected async deployDirectory(
     dir: string,
     dryRun: boolean,
     token: string,
@@ -148,7 +148,7 @@
     }
   }
 
   protected async deploySingleFile(
     api: API,
     dryRun: boolean,
     documentation: string,
@@ -174,17 +174,18 @@
       overlay,
       temporary,
     )
+    const docName = response?.doc_name || documentation
 
     if (dryRun) {
       ux.stdout(ux.colorize('green', 'Definition is valid'))
     } else if (response) {
-      process.stdout.write(ux.colorize('green', `Your ${documentation} documentation...`))
+      process.stdout.write(ux.colorize('green', `Your ${docName} documentation...`))
       ux.stdout(
         ux.colorize('green', `has received a new ${temporary ? 'preview' : 'deployment'} which will soon be ready at:`),
       )
       ux.stdout(ux.colorize('underline', response.doc_public_url!))
     } else {
-      ux.warn(`Your ${documentation} documentation has not changed`)
+      ux.warn(`Your ${docName} documentation has not changed`)
     }
   }
 
@@ -196,9 +197,10 @@
       mcpServer,
       token,
     )
+    const mcpServerName = response?.mcp_server_name || mcpServer
 
     if (response) {
-      process.stdout.write(ux.colorize('green', `Your ${mcpServer} MCP server...`))
+      process.stdout.write(ux.colorize('green', `Your ${mcpServerName} MCP server...`))
       ux.stdout(ux.colorize('green', `has received a new workflow definition which will soon be ready.`))
     } else {
       ux.warn(`Your ${mcpServer} MCP server has not changed.`)

src/core/diff.ts

@@ -29,7 +29,7 @@
     return process.env.BUMP_POLLING_PERIOD ? Number(process.env.BUMP_POLLING_PERIOD) : 1000
   }
 
   async createDiff(
     file1: string,
     file2: string,
     expires: string | undefined,
@@ -64,7 +64,7 @@
     }
   }
 
   async createVersion(
     file: string,
     documentation: string,
     token: string,
@@ -109,10 +109,11 @@
   }
 
   extractDiff(versionWithDiff: VersionResponse & WithDiff): DiffResponse {
     // TODO: return a real diff_id in the GET /version API
     return {
       breaking: versionWithDiff.diff_breaking,
       details: versionWithDiff.diff_details,
+      doc_name: versionWithDiff.doc_name,
       id: versionWithDiff.id,
       markdown: versionWithDiff.diff_markdown,
       public_url: versionWithDiff.diff_public_url,
@@ -133,7 +134,7 @@
     await this.delay(this.pollingPeriod)
   }
 
   public async run(
     file1: string,
     file2: string | undefined,
     documentation: string | undefined,

test/commands/deploy.test.ts

@@ -5,7 +5,7 @@
 
 import {API} from '../../src/definition'
 
 nock.disableNetConnect()
 
 process.env.BUMP_TOKEN = process.env.BUMP_TOKEN || 'BAR'
 
@@ -41,6 +41,20 @@
       expect(stdout).to.contain('http://localhost/doc/1')
     })
 
+    it('sends new version to Bump, and use docName from response', async () => {
+      nock('https://bump.sh')
+        .post('/api/v1/versions', (body) => body.documentation === '42' && !body.branch_name)
+        .reply(201, {doc_name: 'Cou Cou', doc_public_url: 'http://localhost/doc/1'})
+
+      const {stderr, stdout} = await runCommand(['deploy', 'examples/valid/openapi.v3.json', '--doc', '42'].join(' '))
+
+      expect(stderr).to.contain("Let's deploy on Bump.sh... done\n")
+      expect(stdout).to.contain(
+        'Your Cou Cou documentation...has received a new deployment which will soon be ready at:',
+      )
+      expect(stdout).to.contain('http://localhost/doc/1')
+    })
+
     it('sends new version to Bump on given branch', async () => {
       nock('https://bump.sh')
         .post('/api/v1/versions', (body) => body.documentation === 'coucou' && body.branch_name === 'next')
@@ -92,7 +106,7 @@
 
       it("doesn't try to auto create a documentation", async () => {
         nock('https://bump.sh')
           .post('/api/v1/validations', (body) => !body.auto_create_documentation)
           .reply(200)
 
         await runCommand(
@@ -125,6 +139,19 @@
       )
     })
 
+    it('sends new workflow definition (flower) to Bump, and display doc name', async () => {
+      nock('https://bump.sh').post('/api/v1/mcp-servers/crab/deploy').reply(201, {mcp_server_name: 'Flower Power'})
+
+      const {stderr, stdout} = await runCommand(
+        ['deploy', 'examples/valid/flower/parking.yml', '--mcp-server', 'crab'].join(' '),
+      )
+
+      expect(stderr).to.contain("Let's deploy on Bump.sh... done\n")
+      expect(stdout).to.contain(
+        'Your Flower Power MCP server...has received a new workflow definition which will soon be ready.',
+      )
+    })
+
     it('sends new workflow definition with openapi sources (arazzo) to Bump', async () => {
       const [definition, references] = await (
         await API.load('examples/valid/arazzo/wikimedia.json')