[Port] Adding fabric warning for dacpac dialog and small fix to styles (#20604)

* Follow up fixes after PR merge (#20519)

- Added when clause to DacPac commands in command palette to respect mssql.experimentalFeaturesEnabled flag #20512 
- Renamed DacPac commands to grouped naming convention (mssql.dacpacDialog.*) #20513 
- Added OS-specific localization for "Reveal in Explorer" button (Windows/macOS/Linux) #20511 
- Extracted Data-tier Applications documentation URL to constants
- Updated application version validation to accept 2-part versions (e.g., 1.0) #20514 
- Changed validation state to display warnings

* Adding fabric warning for dacpac dialog and small fix to styles (#20599)

* adding fabric warning for dacpac dialog and small fix to styles

* lint fixes

* localization changes

Author: allancascante

mssql/localization/l10n/bundle.l10n.json

@@ -854,6 +854,8 @@
     "A database with this name already exists on the server": "A database with this name already exists on the server",
     "Application version must be in format n.n.n or n.n.n.n where n is a number (e.g., 1.0.0.0)": "Application version must be in format n.n.n or n.n.n.n where n is a number (e.g., 1.0.0.0)",
     "Learn More": "Learn More",
+    "Fabric targets are currently not supported in this preview, and we are working to improve the experience.": "Fabric targets are currently not supported in this preview, and we are working to improve the experience.",
+    "Learn more about this issue.": "Learn more about this issue.",
     "Save Changes": "Save Changes",
     "Add Row": "Add Row",
     "Show Script": "Show Script",

mssql/localization/xliff/vscode-mssql.xlf

@@ -1570,6 +1570,9 @@
       <note>{0} is the error code
 {1} is the error message</note>
     </trans-unit>
+    <trans-unit id="++CODE++c85fb96bbfbe33bde4a67a8e9223c63dc3a3f817428e22ff8357a1ae30f6f4eb">
+      <source xml:lang="en">Fabric targets are currently not supported in this preview, and we are working to improve the experience.</source>
+    </trans-unit>
     <trans-unit id="++CODE++031a8f0f659df890dfd53c92e45295b0f14c997185bae46e168831e403b273f7">
       <source xml:lang="en">Failed</source>
     </trans-unit>
@@ -2129,6 +2132,9 @@
     <trans-unit id="++CODE++51da53a813a79b97741fb115a94a72fe1629a0b0e9018767157ca28d863c53e3">
       <source xml:lang="en">Learn more about SQL Server 2025 features</source>
     </trans-unit>
+    <trans-unit id="++CODE++11c56d1016670369a74f4b7d2d2ec8959e8098b44e778fd4e3a3548bd846826b">
+      <source xml:lang="en">Learn more about this issue.</source>
+    </trans-unit>
     <trans-unit id="++CODE++adc95605a1b30c73959fbaf21e60f9723efe855c482fa336a957924d4ea0a08b">
       <source xml:lang="en">Length</source>
     </trans-unit>

mssql/src/controllers/dacpacDialogWebviewController.ts

@@ -21,7 +21,7 @@ import * as dacpacDialog from "../sharedInterfaces/dacpacDialog";
 import { TaskExecutionMode } from "../sharedInterfaces/schemaCompare";
 import { ListDatabasesRequest } from "../models/contracts/connection";
 import { IConnectionDialogProfile } from "../sharedInterfaces/connectionDialog";
-import { getConnectionDisplayName } from "../models/connectionInfo";
+import { getConnectionDisplayName, getServerTypes, ServerType } from "../models/connectionInfo";
 import {
     validateDatabaseNameFormat,
     DatabaseNameValidationError,
@@ -715,6 +715,19 @@ export class DacpacDialogWebviewController extends ReactWebviewPanelController<
         }
     }
 
+    /**
+     * Checks if a connection profile is connected to a Fabric server
+     */
+    private isFabricConnection(profile: IConnectionDialogProfile): boolean {
+        try {
+            const serverTypes = getServerTypes(profile as vscodeMssql.IConnectionInfo);
+            return serverTypes.includes(ServerType.Fabric);
+        } catch (error) {
+            this.logger.error(`Failed to determine server type: ${error}`);
+            return false;
+        }
+    }
+
     /**
      * Initializes connection based on initial state from Object Explorer or previous session
      * Handles auto-matching and auto-connecting to provide seamless user experience
@@ -730,6 +743,7 @@ export class DacpacDialogWebviewController extends ReactWebviewPanelController<
         ownerUri?: string;
         autoConnected: boolean;
         errorMessage?: string;
+        isFabric?: boolean;
     }> {
         try {
             // Get all connections
@@ -755,18 +769,21 @@ export class DacpacDialogWebviewController extends ReactWebviewPanelController<
                 };
             }
 
+            // Check if this is a Fabric connection
+            const isFabric = this.isFabricConnection(matchingConnection.profile);
+
             // Handle existing connection from Object Explorer
             if (params.initialOwnerUri) {
                 const existingConnResult = this.useExistingConnection(
                     matchingConnection.profile,
                     params.initialOwnerUri,
                 );
-                return { ...existingConnResult, connections };
+                return { ...existingConnResult, connections, isFabric };
             }
 
             // Attempt to connect to the matched profile
             const connectResult = await this.connectToMatchedProfile(matchingConnection.profile);
-            return { ...connectResult, connections };
+            return { ...connectResult, connections, isFabric };
         } catch (error) {
             this.logger.error(`Failed to initialize connection: ${error}`);
             // Fallback: return empty state
@@ -842,9 +859,12 @@ export class DacpacDialogWebviewController extends ReactWebviewPanelController<
     /**
      * Connects to a server using the specified profile ID
      */
-    private async connectToServer(
-        profileId: string,
-    ): Promise<{ ownerUri: string; isConnected: boolean; errorMessage?: string }> {
+    private async connectToServer(profileId: string): Promise<{
+        ownerUri: string;
+        isConnected: boolean;
+        errorMessage?: string;
+        isFabric?: boolean;
+    }> {
         try {
             // Find the profile in saved connections
             const savedConnections =
@@ -863,13 +883,17 @@ export class DacpacDialogWebviewController extends ReactWebviewPanelController<
                 };
             }
 
+            // Check if this is a Fabric connection
+            const isFabric = this.isFabricConnection(profile as IConnectionDialogProfile);
+
             // Check if already connected and the connection is valid
             let ownerUri = this.connectionManager.getUriForConnection(profile);
             if (ownerUri && this.connectionManager.isConnected(ownerUri)) {
                 // Connection is active and valid
                 return {
                     ownerUri,
                     isConnected: true,
+                    isFabric,
                 };
             }
 
@@ -884,6 +908,7 @@ export class DacpacDialogWebviewController extends ReactWebviewPanelController<
                 return {
                     ownerUri,
                     isConnected: true,
+                    isFabric,
                 };
             } else {
                 // Check if connection failed due to error or if it was never initiated
@@ -897,6 +922,7 @@ export class DacpacDialogWebviewController extends ReactWebviewPanelController<
                     ownerUri: "",
                     isConnected: false,
                     errorMessage,
+                    isFabric,
                 };
             }
         } catch (error) {

mssql/src/reactviews/common/locConstants.ts

@@ -1276,6 +1276,10 @@ export class LocConstants {
                 "Application version must be in format n.n.n or n.n.n.n where n is a number (e.g., 1.0.0.0)",
             ),
             learnMore: l10n.t("Learn More"),
+            fabricWarning: l10n.t(
+                "Fabric targets are currently not supported in this preview, and we are working to improve the experience.",
+            ),
+            fabricWarningLearnMore: l10n.t("Learn more about this issue."),
         };
     }
 

mssql/src/reactviews/pages/DacpacDialog/ServerSelectionSection.tsx

@@ -3,7 +3,7 @@
  *  Licensed under the MIT License. See License.txt in the project root for license information.
  *--------------------------------------------------------------------------------------------*/
 
-import { Dropdown, Field, makeStyles, Option, Spinner } from "@fluentui/react-components";
+import { Dropdown, Field, Link, makeStyles, Option, Spinner } from "@fluentui/react-components";
 import { IConnectionDialogProfile } from "../../../sharedInterfaces/connectionDialog";
 import { locConstants } from "../../common/locConstants";
 
@@ -22,6 +22,7 @@ interface ServerSelectionSectionProps {
     isOperationInProgress: boolean;
     validationMessages: Record<string, ValidationMessage>;
     onServerChange: (profileId: string) => void;
+    isFabric?: boolean;
 }
 
 const useStyles = makeStyles({
@@ -30,6 +31,11 @@ const useStyles = makeStyles({
         flexDirection: "column",
         gap: "12px",
     },
+    fabricWarning: {
+        marginTop: "4px",
+        fontSize: "12px",
+        color: "var(--vscode-descriptionForeground)",
+    },
 });
 
 export const ServerSelectionSection = ({
@@ -39,6 +45,7 @@ export const ServerSelectionSection = ({
     isOperationInProgress,
     validationMessages,
     onServerChange,
+    isFabric = false,
 }: ServerSelectionSectionProps) => {
     const classes = useStyles();
 
@@ -47,9 +54,11 @@ export const ServerSelectionSection = ({
             <Field
                 label={locConstants.dacpacDialog.serverLabel}
                 required
-                validationMessage={validationMessages.connection?.message}
+                validationMessage={isFabric ? undefined : validationMessages.connection?.message}
                 validationState={
-                    validationMessages.connection?.severity === "error" ? "error" : "none"
+                    !isFabric && validationMessages.connection?.severity === "error"
+                        ? "error"
+                        : "none"
                 }>
                 {isConnecting ? (
                     <Spinner size="tiny" label={locConstants.dacpacDialog.connectingToServer} />
@@ -89,6 +98,16 @@ export const ServerSelectionSection = ({
                     </Dropdown>
                 )}
             </Field>
+            {isFabric && (
+                <div className={classes.fabricWarning}>
+                    {locConstants.dacpacDialog.fabricWarning}{" "}
+                    <Link
+                        href="https://github.com/microsoft/vscode-mssql/issues/20568"
+                        target="_blank">
+                        {locConstants.dacpacDialog.fabricWarningLearnMore}
+                    </Link>
+                </div>
+            )}
         </div>
     );
 };

mssql/src/reactviews/pages/DacpacDialog/SourceDatabaseSection.tsx

@@ -23,6 +23,7 @@ interface SourceDatabaseSectionProps {
     validationMessages: Record<string, ValidationMessage>;
     showDatabaseSource: boolean;
     showNewDatabase: boolean;
+    isFabric?: boolean;
 }
 
 const useStyles = makeStyles({
@@ -42,6 +43,7 @@ export const SourceDatabaseSection = ({
     validationMessages,
     showDatabaseSource,
     showNewDatabase,
+    isFabric = false,
 }: SourceDatabaseSectionProps) => {
     const classes = useStyles();
 
@@ -66,7 +68,7 @@ export const SourceDatabaseSection = ({
                         value={databaseName}
                         selectedOptions={[databaseName]}
                         onOptionSelect={(_, data) => setDatabaseName(data.optionText || "")}
-                        disabled={isOperationInProgress || !ownerUri}
+                        disabled={isOperationInProgress || !ownerUri || isFabric}
                         aria-label={locConstants.dacpacDialog.sourceDatabaseLabel}>
                         {availableDatabases.map((db) => (
                             <Option key={db} value={db}>
@@ -88,7 +90,7 @@ export const SourceDatabaseSection = ({
                             value={databaseName}
                             onChange={(_, data) => setDatabaseName(data.value)}
                             placeholder={locConstants.dacpacDialog.enterDatabaseName}
-                            disabled={isOperationInProgress}
+                            disabled={isOperationInProgress || isFabric}
                             aria-label={locConstants.dacpacDialog.databaseNameLabel}
                         />
                     </Field>

mssql/src/reactviews/pages/DacpacDialog/TargetDatabaseSection.tsx

@@ -32,6 +32,7 @@ interface TargetDatabaseSectionProps {
     isOperationInProgress: boolean;
     ownerUri: string;
     validationMessages: Record<string, ValidationMessage>;
+    isFabric?: boolean;
 }
 
 const useStyles = makeStyles({
@@ -56,6 +57,7 @@ export const TargetDatabaseSection = ({
     isOperationInProgress,
     ownerUri,
     validationMessages,
+    isFabric = false,
 }: TargetDatabaseSectionProps) => {
     const classes = useStyles();
 
@@ -70,7 +72,7 @@ export const TargetDatabaseSection = ({
                 <Radio
                     value="new"
                     label={locConstants.dacpacDialog.newDatabase}
-                    disabled={isOperationInProgress}
+                    disabled={isOperationInProgress || isFabric}
                     aria-label={locConstants.dacpacDialog.newDatabase}
                 />
                 <Radio