slas:client:get render redirect & callback URIs as lists

The SLAS API returns redirectUri and callbackUri as pipe-delimited
strings. printClientDetails now splits them via parseRedirectUris and
renders one URI per line, matching the formatting used for scopes.

Author: clavery

.changeset/slas-client-get-redirect-uris-list.md

@@ -0,0 +1,5 @@
+---
+'@salesforce/b2c-cli': patch
+---
+
+`b2c slas:client:get` now displays redirect URIs and callback URIs as lists (one per line) instead of the raw pipe-delimited strings returned by the SLAS API, matching the formatting used for scopes.

packages/b2c-cli/src/utils/slas/client.ts

@@ -66,18 +66,27 @@ export function normalizeClientResponse(client: Client): ClientOutput {
  */
 export function printClientDetails(output: ClientOutput, showSecret = true): void {
   const ui = cliui({width: process.stdout.columns || 80});
-  const labelWidth = 14;
+  const labelWidth = 16;
 
   ui.div('');
   ui.div({text: 'Client ID:', width: labelWidth}, {text: output.clientId});
   ui.div({text: 'Name:', width: labelWidth}, {text: output.name});
   ui.div({text: 'Private:', width: labelWidth}, {text: String(output.isPrivateClient)});
   ui.div({text: 'Channels:', width: labelWidth}, {text: output.channels.join(', ')});
   ui.div({text: 'Scopes:', width: labelWidth}, {text: output.scopes.join('\n' + ' '.repeat(labelWidth))});
-  ui.div({text: 'Redirect URI:', width: labelWidth}, {text: output.redirectUri});
+
+  const redirectUris = parseRedirectUris(output.redirectUri);
+  ui.div(
+    {text: 'Redirect URIs:', width: labelWidth},
+    {text: redirectUris.length > 0 ? redirectUris.join('\n' + ' '.repeat(labelWidth)) : ''},
+  );
 
   if (output.callbackUri) {
-    ui.div({text: 'Callback URI:', width: labelWidth}, {text: output.callbackUri});
+    const callbackUris = parseRedirectUris(output.callbackUri);
+    ui.div(
+      {text: 'Callback URIs:', width: labelWidth},
+      {text: callbackUris.length > 0 ? callbackUris.join('\n' + ' '.repeat(labelWidth)) : output.callbackUri},
+    );
   }
 
   if (showSecret && output.secret) {

packages/b2c-cli/test/utils/slas/client.test.ts

@@ -203,6 +203,32 @@ describe('utils/slas/client', () => {
       expect(text).not.to.include('my-secret');
     });
 
+    it('renders multiple redirect URIs one per line', () => {
+      const stdoutStub = sinon.stub(ux, 'stdout');
+      const output = {
+        clientId: 'client-1',
+        name: 'Test',
+        scopes: [],
+        channels: [],
+        redirectUri: 'http://a.example.com|http://b.example.com|http://c.example.com',
+        isPrivateClient: true,
+      };
+
+      printClientDetails(output, false);
+      const text = stdoutStub.firstCall.args[0] as string;
+      expect(text).to.include('Redirect URIs:');
+      expect(text).to.not.include('|');
+      const lines = text.split('\n');
+      const aLine = lines.find((l) => l.includes('http://a.example.com'));
+      const bLine = lines.find((l) => l.includes('http://b.example.com'));
+      const cLine = lines.find((l) => l.includes('http://c.example.com'));
+      expect(aLine, 'a on its own line').to.exist;
+      expect(bLine, 'b on its own line').to.exist;
+      expect(cLine, 'c on its own line').to.exist;
+      expect(aLine).to.not.include('http://b.example.com');
+      expect(bLine).to.not.include('http://c.example.com');
+    });
+
     it('prints callbackUri when present', () => {
       const stdoutStub = sinon.stub(ux, 'stdout');
       const output = {
@@ -220,5 +246,27 @@ describe('utils/slas/client', () => {
       const text = stdoutStub.firstCall.args[0];
       expect(text).to.include('https://example.com/cb');
     });
+
+    it('renders multiple callback URIs one per line', () => {
+      const stdoutStub = sinon.stub(ux, 'stdout');
+      const output = {
+        clientId: 'client-1',
+        name: 'Test',
+        scopes: [],
+        channels: [],
+        redirectUri: '',
+        isPrivateClient: true,
+        callbackUri: 'https://a.example.com/cb|https://b.example.com/cb|https://c.example.com/cb',
+      };
+
+      printClientDetails(output, false);
+      const text = stdoutStub.firstCall.args[0] as string;
+      expect(text).to.include('Callback URIs:');
+      expect(text).to.not.include('|');
+      const lines = text.split('\n');
+      expect(lines.find((l) => l.includes('https://a.example.com/cb'))).to.exist;
+      expect(lines.find((l) => l.includes('https://b.example.com/cb'))).to.exist;
+      expect(lines.find((l) => l.includes('https://c.example.com/cb'))).to.exist;
+    });
   });
 });