Address code review feedback

Author: Saurabh7019

.devproxy/api-specs/sharepoint.yaml

@@ -82,7 +82,7 @@ paths:
           description: list GUID
           schema:
             type: string
-          example: "'b2307a39-e878-458b-bc90-03bc578531d6'"
+          example: b2307a39-e878-458b-bc90-03bc578531d6
         - name: itemId
           in: path
           required: true

docs/docs/cmd/spo/file/file-unarchive.mdx

@@ -65,4 +65,42 @@ m365 spo file unarchive --webUrl https://contoso.sharepoint.com/sites/Marketing
 
 ## Response
 
-The command won't return a response on success.
+<Tabs>
+  <TabItem value="JSON">
+
+  ```json
+  {
+    "value": "reactivating"
+  }
+  ```
+
+  </TabItem>
+  <TabItem value="Text">
+
+  ```text
+  value: reactivating
+  ```
+
+  </TabItem>
+  <TabItem value="CSV">
+
+  ```csv
+  value
+  reactivating
+  ```
+
+  </TabItem>
+  <TabItem value="Markdown">
+
+  ```md
+  # spo file unarchive --debug "false" --verbose "false" --webUrl "https://contoso.sharepoint.com/sites/Marketing" --url "/sites/Marketing/shared documents/General/document.docx" --force "true"
+
+  Date: 5/8/2026
+
+  Property | Value
+  ---------|-------
+  value | reactivating
+  ```
+
+  </TabItem>
+</Tabs>

docs/src/config/sidebars.ts

@@ -2756,6 +2756,11 @@ const sidebars: SidebarsConfig = {
               label: 'file rename',
               id: 'cmd/spo/file/file-rename'
             },
+            {
+              type: 'doc',
+              label: 'file unarchive',
+              id: 'cmd/spo/file/file-unarchive'
+            },
             {
               type: 'doc',
               label: 'file retentionlabel ensure',
@@ -2821,11 +2826,6 @@ const sidebars: SidebarsConfig = {
               label: 'file sharinglink set',
               id: 'cmd/spo/file/file-sharinglink-set'
             },
-            {
-              type: 'doc',
-              label: 'file unarchive',
-              id: 'cmd/spo/file/file-unarchive'
-            },
             {
               type: 'doc',
               label: 'file version clear',

src/m365/spo/commands/file/file-unarchive.spec.ts

@@ -23,6 +23,10 @@ describe(commands.FILE_UNARCHIVE, () => {
   let confirmationPromptStub: sinon.SinonStub;
   let loggerLogSpy: sinon.SinonSpy;
 
+  const unarchiveResponse = {
+    value: 'reactivating'
+  };
+
   before(() => {
     sinon.stub(auth, 'restoreAuth').resolves();
     sinon.stub(telemetry, 'trackEvent').resolves();
@@ -134,7 +138,7 @@ describe(commands.FILE_UNARCHIVE, () => {
 
   it('prompts before unarchiving file when confirmation argument not passed', async () => {
     sinon.stub(request, 'get').resolves({ ListId: 'b2307a39-e878-458b-bc90-03bc578531d6', ListItemAllFields: { Id: 1 } });
-    sinon.stub(request, 'post').resolves();
+    sinon.stub(request, 'post').resolves(unarchiveResponse);
 
     await command.action(logger, {
       options: {
@@ -147,7 +151,7 @@ describe(commands.FILE_UNARCHIVE, () => {
 
   it('aborts unarchiving file when prompt not confirmed', async () => {
     const getStub = sinon.stub(request, 'get').resolves({ ListId: 'b2307a39-e878-458b-bc90-03bc578531d6', ListItemAllFields: { Id: 1 } });
-    const postStub = sinon.stub(request, 'post').resolves();
+    const postStub = sinon.stub(request, 'post').resolves(unarchiveResponse);
 
     await command.action(logger, {
       options: {
@@ -176,7 +180,7 @@ describe(commands.FILE_UNARCHIVE, () => {
 
     const postStub = sinon.stub(request, 'post').callsFake(async (opts) => {
       if (opts.url === `https://contoso.sharepoint.com/sites/test/_api/Lists(guid'b2307a39-e878-458b-bc90-03bc578531d6')/items(1)/UnArchive`) {
-        return;
+        return unarchiveResponse;
       }
 
       throw 'Invalid request';
@@ -191,6 +195,7 @@ describe(commands.FILE_UNARCHIVE, () => {
     });
 
     assert(postStub.calledOnce);
+    assert(loggerLogSpy.calledWith(unarchiveResponse));
   });
 
   it('unarchives file by id', async () => {
@@ -210,7 +215,7 @@ describe(commands.FILE_UNARCHIVE, () => {
 
     const postStub = sinon.stub(request, 'post').callsFake(async (opts) => {
       if (opts.url === `https://contoso.sharepoint.com/sites/test/_api/Lists(guid'b2307a39-e878-458b-bc90-03bc578531d6')/items(1)/UnArchive`) {
-        return;
+        return unarchiveResponse;
       }
 
       throw 'Invalid request';
@@ -226,6 +231,7 @@ describe(commands.FILE_UNARCHIVE, () => {
     });
 
     assert(postStub.calledOnce);
+    assert(loggerLogSpy.calledWith(unarchiveResponse));
   });
 
   it('unarchives file using site-relative url', async () => {
@@ -244,7 +250,7 @@ describe(commands.FILE_UNARCHIVE, () => {
 
     const postStub = sinon.stub(request, 'post').callsFake(async (opts) => {
       if (opts.url === `https://contoso.sharepoint.com/sites/test/_api/Lists(guid'b2307a39-e878-458b-bc90-03bc578531d6')/items(1)/UnArchive`) {
-        return;
+        return unarchiveResponse;
       }
 
       throw 'Invalid request';
@@ -259,11 +265,12 @@ describe(commands.FILE_UNARCHIVE, () => {
     });
 
     assert(postStub.calledOnce);
+    assert(loggerLogSpy.calledWith(unarchiveResponse));
   });
 
-  it('outputs no result when unarchiving a file', async () => {
+  it('outputs the API response when unarchiving a file', async () => {
     sinon.stub(request, 'get').resolves({ ListId: 'b2307a39-e878-458b-bc90-03bc578531d6', ListItemAllFields: { Id: 1 } });
-    sinon.stub(request, 'post').resolves();
+    sinon.stub(request, 'post').resolves(unarchiveResponse);
 
     await command.action(logger, {
       options: {
@@ -273,7 +280,7 @@ describe(commands.FILE_UNARCHIVE, () => {
       }
     });
 
-    assert(loggerLogSpy.notCalled);
+    assert(loggerLogSpy.calledOnceWith(unarchiveResponse));
   });
 
   it('handles error correctly', async () => {

src/m365/spo/commands/file/file-unarchive.ts

@@ -55,7 +55,7 @@ class SpoFileUnarchiveCommand extends SpoCommand {
     const { webUrl, url, id, force, verbose } = args.options;
 
     if (!force) {
-      const result = await cli.promptForConfirmation({ message: `This item is archived. Reactivation could take up to 24 hours. Are you sure you would like to unarchive this item?` });
+      const result = await cli.promptForConfirmation({ message: `Reactivation could take up to 24 hours. Files that are reactivated cannot be archived again for 30 days. Are you sure you would like to unarchive this item?` });
       if (!result) {
         return;
       }
@@ -93,7 +93,8 @@ class SpoFileUnarchiveCommand extends SpoCommand {
         responseType: 'json'
       };
 
-      await request.post(requestOptions);
+      const res = await request.post(requestOptions);
+      await logger.log(res);
     }
     catch (err: any) {
       this.handleRejectedODataJsonPromise(err);