@fluidframework/file-driver 2.0.0-dev.6.4.0.191515 → 2.0.0-dev.7.2.0.203917

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fluidframework/file-driver",
-  "version": "2.0.0-dev.6.4.0.191515",
+  "version": "2.0.0-dev.7.2.0.203917",
   "description": "A driver that reads/write from/to local file storage.",
   "homepage": "https://fluidframework.com",
   "repository": {
@@ -14,27 +14,26 @@
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "dependencies": {
-    "@fluid-internal/client-utils": "2.0.0-dev.6.4.0.191515",
-    "@fluidframework/core-interfaces": "2.0.0-dev.6.4.0.191515",
-    "@fluidframework/core-utils": "2.0.0-dev.6.4.0.191515",
-    "@fluidframework/driver-definitions": "2.0.0-dev.6.4.0.191515",
-    "@fluidframework/driver-utils": "2.0.0-dev.6.4.0.191515",
-    "@fluidframework/protocol-definitions": "^1.1.0",
-    "@fluidframework/replay-driver": "2.0.0-dev.6.4.0.191515"
+    "@fluid-internal/client-utils": "2.0.0-dev.7.2.0.203917",
+    "@fluidframework/core-interfaces": "2.0.0-dev.7.2.0.203917",
+    "@fluidframework/core-utils": "2.0.0-dev.7.2.0.203917",
+    "@fluidframework/driver-definitions": "2.0.0-dev.7.2.0.203917",
+    "@fluidframework/driver-utils": "2.0.0-dev.7.2.0.203917",
+    "@fluidframework/protocol-definitions": "^3.0.0",
+    "@fluidframework/replay-driver": "2.0.0-dev.7.2.0.203917"
   },
   "devDependencies": {
-    "@fluid-tools/build-cli": "^0.22.0",
-    "@fluidframework/build-common": "^2.0.0",
-    "@fluidframework/build-tools": "^0.22.0",
-    "@fluidframework/eslint-config-fluid": "^2.1.0",
-    "@fluidframework/file-driver-previous": "npm:@fluidframework/file-driver@2.0.0-internal.6.3.0",
-    "@microsoft/api-extractor": "^7.34.4",
+    "@fluid-tools/build-cli": "0.26.0-203096",
+    "@fluidframework/build-common": "^2.0.2",
+    "@fluidframework/build-tools": "0.26.0-203096",
+    "@fluidframework/eslint-config-fluid": "^3.0.0",
+    "@fluidframework/file-driver-previous": "npm:@fluidframework/file-driver@2.0.0-internal.7.1.0",
+    "@microsoft/api-extractor": "^7.37.0",
     "@types/node": "^16.18.38",
-    "copyfiles": "^2.4.1",
-    "eslint": "~8.6.0",
-    "prettier": "~2.6.2",
+    "eslint": "~8.50.0",
+    "prettier": "~3.0.3",
     "rimraf": "^4.4.0",
-    "typescript": "~4.5.5"
+    "typescript": "~5.1.6"
   },
   "fluidBuild": {
     "tasks": {
@@ -50,8 +49,8 @@
   "scripts": {
     "build": "fluid-build . --task build",
     "build:compile": "fluid-build . --task compile",
-    "build:docs": "api-extractor run --local --typescript-compiler-folder ../../../node_modules/typescript && copyfiles -u 1 ./_api-extractor-temp/doc-models/* ../../../_api-extractor-temp/",
-    "ci:build:docs": "api-extractor run --typescript-compiler-folder ../../../node_modules/typescript && copyfiles -u 1 ./_api-extractor-temp/* ../../../_api-extractor-temp/",
+    "build:docs": "api-extractor run --local",
+    "ci:build:docs": "api-extractor run",
     "clean": "rimraf --glob 'dist' 'lib' '*.tsbuildinfo' '*.build.log' '_api-extractor-temp'",
     "eslint": "eslint --format stylish src",
     "eslint:fix": "eslint --format stylish src --fix --fix-type problem,suggestion,layout",

package/src/fileDocumentStorageService.ts

@@ -27,7 +27,10 @@ export class FluidFetchReader
 {
 	protected docTree: api.ISnapshotTree | null = null;
 
-	constructor(private readonly path: string, private readonly versionName?: string) {
+	constructor(
+		private readonly path: string,
+		private readonly versionName?: string,
+	) {
 		super();
 	}
 
@@ -35,6 +38,7 @@ export class FluidFetchReader
 	 * Read the file and returns the snapshot tree.
 	 * @param version - The version contains the path of the file which contains the snapshot tree.
 	 */
+	// eslint-disable-next-line @rushstack/no-new-null
 	public async getSnapshotTree(version?: api.IVersion): Promise<api.ISnapshotTree | null> {
 		assert(version !== null, 0x092 /* "version input for reading snapshot tree is null!" */);
 		assert(
@@ -74,9 +78,10 @@ export class FluidFetchReader
 	 * @param versionId - version ID.
 	 * @param count - Number of versions to be returned.
 	 */
+	// eslint-disable-next-line @rushstack/no-new-null
 	public async getVersions(versionId: string | null, count: number): Promise<api.IVersion[]> {
 		if (versionId === FileStorageDocumentName || versionId === null) {
-			if (this.docTree || this.versionName !== undefined) {
+			if (this.docTree !== null || this.versionName !== undefined) {
 				return [{ id: "latest", treeId: FileStorageVersionTreeId }];
 			}
 			// Started with ops - return empty set.
@@ -136,6 +141,7 @@ export const FileSnapshotWriterClassFactory = <TBase extends ReaderConstructor>(
 			return super.readBlob(sha);
 		}
 
+		// eslint-disable-next-line @rushstack/no-new-null
 		public async getVersions(versionId: string | null, count: number): Promise<api.IVersion[]> {
 			// If we already saved document, that means we are getting here because of snapshot generation.
 			// Not returning tree ensures that ContainerRuntime.snapshot() would regenerate subtrees for
@@ -153,6 +159,7 @@ export const FileSnapshotWriterClassFactory = <TBase extends ReaderConstructor>(
 			return super.getVersions(versionId, count);
 		}
 
+		// eslint-disable-next-line @rushstack/no-new-null
 		public async getSnapshotTree(version?: api.IVersion): Promise<api.ISnapshotTree | null> {
 			if (this.latestWriterTree && (!version || version.id === "latest")) {
 				return this.latestWriterTree;