@fluidframework/driver-utils 2.0.0-dev.1.4.6.106135 → 2.0.0-dev.2.2.0.111723

package/src/messageRecognition.ts

@@ -4,37 +4,6 @@
  */
 import { IDocumentMessage, ISequencedDocumentMessage, MessageType } from "@fluidframework/protocol-definitions";
 
-/**
- * Determines whether or not the message type is one of the following:
- *
- * - {@link @fluidframework/protocol-definitions#MessageType.Operation}
- *
- * - {@link @fluidframework/protocol-definitions#MessageType.Summarize}
- *
- * - {@link @fluidframework/protocol-definitions#MessageType.Propose}
- *
- * - {@link @fluidframework/protocol-definitions#MessageType.Reject}
- *
- * - {@link @fluidframework/protocol-definitions#MessageType2.Accept}
- *
- * - {@link @fluidframework/protocol-definitions#MessageType.NoOp}
- */
-export function isClientMessage(message: ISequencedDocumentMessage | IDocumentMessage): boolean {
-    if (isRuntimeMessage(message)) {
-        return true;
-    }
-    switch (message.type) {
-        case MessageType.Propose:
-        case MessageType.Reject:
-        case MessageType.NoOp:
-        case MessageType2.Accept:
-        case MessageType.Summarize:
-            return true;
-        default:
-            return false;
-    }
-}
-
 /**
  * Tells if message was sent by container runtime
  * @privateRemarks ADO #1385: To be moved to container-definitions

package/src/network.ts

@@ -38,8 +38,11 @@ export function isOnline(): OnlineStatus {
  * "Any" in the interface name is a nod to the fact that errorType has lost its type constraint.
  * It will be either DriverErrorType or the specific driver's specialized error type enum,
  * but we can't reference a specific driver's error type enum in this code.
- */
- export interface IAnyDriverError extends Omit<IDriverErrorBase, "errorType"> {
+ *
+ * @deprecated - In favour of {@link @fluidframework/driver-definitions#IAnyDriverError} so that
+ * it can used from the base to upper layers.
+*/
+export interface IAnyDriverError extends Omit<IDriverErrorBase, "errorType"> {
     readonly errorType: string;
 }
 
@@ -61,16 +64,24 @@ export class GenericNetworkError extends LoggingError implements IDriverErrorBas
     }
 }
 
-// Todo GH #6214: Remove after next drive def bump. This is necessary as there is no
-// compatible way to augment an enum, as it can't be optional. So for now
-// we need to duplicate the value here. We likely need to rethink our
-// DriverErrorType strategy so that it supports extension with optional
-// value.
-const deltaStreamConnectionForbiddenStr = "deltaStreamConnectionForbidden";
-export class DeltaStreamConnectionForbiddenError extends LoggingError implements IFluidErrorBase {
-    static readonly errorType: string =
-        DriverErrorType[deltaStreamConnectionForbiddenStr] ?? deltaStreamConnectionForbiddenStr;
-    readonly errorType: string = DeltaStreamConnectionForbiddenError.errorType;
+/**
+ * FluidInvalidSchema error class.
+ */
+ export class FluidInvalidSchemaError extends LoggingError implements IDriverErrorBase, IFluidErrorBase {
+    readonly errorType = DriverErrorType.fluidInvalidSchema;
+    readonly canRetry = false;
+
+    constructor(
+        message: string,
+        props: DriverErrorTelemetryProps,
+    ) {
+        super(message, props);
+    }
+}
+
+export class DeltaStreamConnectionForbiddenError extends LoggingError implements IDriverErrorBase, IFluidErrorBase {
+    static readonly errorType = DriverErrorType.deltaStreamConnectionForbidden;
+    readonly errorType = DeltaStreamConnectionForbiddenError.errorType;
     readonly canRetry = false;
 
     constructor(message: string, props: DriverErrorTelemetryProps) {

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/driver-utils";
-export const pkgVersion = "2.0.0-dev.1.4.6.106135";
+export const pkgVersion = "2.0.0-dev.2.2.0.111723";