[MDiff] Hide discard actions for imported comparison (#1384)

[MDiff] Enhance metadata diff functionality

- ✨ Do not show discard actions for imported comparison
- ✅ Extend tests to verify behavior for imported metadata diff items

-Priyanshu

Author: priyanshu92

package.json

@@ -1508,6 +1508,11 @@
           "when": "view == microsoft.powerplatform.pages.actionsHub && viewItem == metadataDiffFile && microsoft.powerplatform.pages.metadataDiffEnabled",
           "group": "inline@1"
         },
+        {
+          "command": "microsoft.powerplatform.pages.actionsHub.metadataDiff.openFile",
+          "when": "view == microsoft.powerplatform.pages.actionsHub && viewItem == metadataDiffFileImported && microsoft.powerplatform.pages.metadataDiffEnabled",
+          "group": "inline@1"
+        },
         {
           "command": "microsoft.powerplatform.pages.actionsHub.metadataDiff.discardFile",
           "when": "view == microsoft.powerplatform.pages.actionsHub && viewItem == metadataDiffFile && microsoft.powerplatform.pages.metadataDiffEnabled",
@@ -1528,6 +1533,11 @@
           "when": "view == microsoft.powerplatform.pages.actionsHub && viewItem == metadataDiffFile && microsoft.powerplatform.pages.metadataDiffEnabled",
           "group": "fileAction@1"
         },
+        {
+          "command": "microsoft.powerplatform.pages.actionsHub.metadataDiff.openFile",
+          "when": "view == microsoft.powerplatform.pages.actionsHub && viewItem == metadataDiffFileImported && microsoft.powerplatform.pages.metadataDiffEnabled",
+          "group": "fileAction@1"
+        },
         {
           "command": "microsoft.powerplatform.pages.actionsHub.metadataDiff.discardFile",
           "when": "view == microsoft.powerplatform.pages.actionsHub && viewItem == metadataDiffFile && microsoft.powerplatform.pages.metadataDiffEnabled",

src/client/power-pages/actions-hub/Constants.ts

@@ -24,7 +24,9 @@ export const Constants = {
         METADATA_DIFF_SITE: "metadataDiffSite",
         METADATA_DIFF_SITE_IMPORTED: "metadataDiffSiteImported",
         METADATA_DIFF_FOLDER: "metadataDiffFolder",
+        METADATA_DIFF_FOLDER_IMPORTED: "metadataDiffFolderImported",
         METADATA_DIFF_FILE: "metadataDiffFile",
+        METADATA_DIFF_FILE_IMPORTED: "metadataDiffFileImported",
     },
     Commands: {
         METADATA_DIFF_OPEN_FILE: "microsoft.powerplatform.pages.actionsHub.metadataDiff.openFile",

src/client/power-pages/actions-hub/tree-items/metadata-diff/MetadataDiffFileTreeItem.ts

@@ -34,12 +34,17 @@ export class MetadataDiffFileTreeItem extends ActionsHubTreeItem {
         // Build description based on view mode
         const description = MetadataDiffFileTreeItem.buildDescription(comparisonResult);
 
+        // Use different context value for imported items to hide discard option
+        const contextValue = isImported
+            ? Constants.ContextValues.METADATA_DIFF_FILE_IMPORTED
+            : Constants.ContextValues.METADATA_DIFF_FILE;
+
         super(
             fileName,
             vscode.TreeItemCollapsibleState.None,
             // Pass empty ThemeIcon - it will be overridden by resourceUri
             new vscode.ThemeIcon("file"),
-            Constants.ContextValues.METADATA_DIFF_FILE,
+            contextValue,
             description
         );
         this.comparisonResult = comparisonResult;

src/client/power-pages/actions-hub/tree-items/metadata-diff/MetadataDiffFolderTreeItem.ts

@@ -15,17 +15,24 @@ export class MetadataDiffFolderTreeItem extends ActionsHubTreeItem {
     public readonly childrenMap: Map<string, MetadataDiffFolderTreeItem | MetadataDiffFileTreeItem>;
     private readonly _siteName: string;
     private readonly _folderPath: string;
+    private readonly _isImported: boolean;
+
+    constructor(folderName: string, siteName: string, folderPath: string, isImported: boolean = false) {
+        // Use different context value for imported items to hide discard option
+        const contextValue = isImported
+            ? Constants.ContextValues.METADATA_DIFF_FOLDER_IMPORTED
+            : Constants.ContextValues.METADATA_DIFF_FOLDER;
 
-    constructor(folderName: string, siteName: string, folderPath: string) {
         super(
             folderName,
             vscode.TreeItemCollapsibleState.Expanded,
             Constants.Icons.METADATA_DIFF_FOLDER,
-            Constants.ContextValues.METADATA_DIFF_FOLDER
+            contextValue
         );
         this.childrenMap = new Map();
         this._siteName = siteName;
         this._folderPath = folderPath;
+        this._isImported = isImported;
     }
 
     public getChildren(): ActionsHubTreeItem[] {
@@ -61,6 +68,10 @@ export class MetadataDiffFolderTreeItem extends ActionsHubTreeItem {
         return this._folderPath;
     }
 
+    public get isImported(): boolean {
+        return this._isImported;
+    }
+
     /**
      * Recursively collects all file tree items under this folder
      */

src/client/power-pages/actions-hub/tree-items/metadata-diff/MetadataDiffSiteTreeItem.ts

@@ -138,15 +138,15 @@ export class MetadataDiffSiteTreeItem extends ActionsHubTreeItem {
                     // Look in root children
                     let folder = rootChildren.get(folderName) as MetadataDiffFolderTreeItem | undefined;
                     if (!folder) {
-                        folder = new MetadataDiffFolderTreeItem(folderName, this._siteName, currentPath);
+                        folder = new MetadataDiffFolderTreeItem(folderName, this._siteName, currentPath, this._isImported);
                         rootChildren.set(folderName, folder);
                     }
                     currentFolder = folder;
                 } else if (currentFolder) {
                     // Look in current folder's children
                     let folder = currentFolder.childrenMap.get(folderName) as MetadataDiffFolderTreeItem | undefined;
                     if (!folder) {
-                        folder = new MetadataDiffFolderTreeItem(folderName, this._siteName, currentPath);
+                        folder = new MetadataDiffFolderTreeItem(folderName, this._siteName, currentPath, this._isImported);
                         currentFolder.childrenMap.set(folderName, folder);
                     }
                     currentFolder = folder;

src/client/test/Integration/power-pages/actions-hub/tree-items/metadata-diff/MetadataDiffFileTreeItem.test.ts

@@ -56,6 +56,12 @@ describe("MetadataDiffFileTreeItem", () => {
 
             expect(treeItem.contextValue).to.equal(Constants.ContextValues.METADATA_DIFF_FILE);
         });
+
+        it("should have imported context value when isImported is true", () => {
+            const treeItem = new MetadataDiffFileTreeItem(mockComparisonResult, "Test Site", true);
+
+            expect(treeItem.contextValue).to.equal(Constants.ContextValues.METADATA_DIFF_FILE_IMPORTED);
+        });
     });
 
     describe("resourceUri", () => {

src/client/test/Integration/power-pages/actions-hub/tree-items/metadata-diff/MetadataDiffFolderTreeItem.test.ts

@@ -43,6 +43,12 @@ describe("MetadataDiffFolderTreeItem", () => {
             expect(treeItem.contextValue).to.equal(Constants.ContextValues.METADATA_DIFF_FOLDER);
         });
 
+        it("should have imported context value when isImported is true", () => {
+            const treeItem = new MetadataDiffFolderTreeItem("folder", "Test Site", "folder", true);
+
+            expect(treeItem.contextValue).to.equal(Constants.ContextValues.METADATA_DIFF_FOLDER_IMPORTED);
+        });
+
         it("should have empty children map initially", () => {
             const treeItem = new MetadataDiffFolderTreeItem("folder", "Test Site", "folder");
 
@@ -55,6 +61,18 @@ describe("MetadataDiffFolderTreeItem", () => {
             expect(treeItem.siteName).to.equal("Test Site");
             expect(treeItem.folderPath).to.equal("root/folder");
         });
+
+        it("should have isImported default to false", () => {
+            const treeItem = new MetadataDiffFolderTreeItem("folder", "Test Site", "folder");
+
+            expect(treeItem.isImported).to.be.false;
+        });
+
+        it("should store isImported when provided", () => {
+            const treeItem = new MetadataDiffFolderTreeItem("folder", "Test Site", "folder", true);
+
+            expect(treeItem.isImported).to.be.true;
+        });
     });
 
     describe("childrenMap", () => {