@fluidframework/tree 2.0.0-rc.1.0.6 → 2.0.0-rc.2.0.1

package/.dependency-cruiser-known-violations.json

@@ -1,35 +1,4 @@
 [
-  {
-    "type": "cycle",
-    "from": "src/feature-libraries/flex-tree/context.ts",
-    "to": "src/feature-libraries/flex-tree/lazyField.ts",
-    "rule": {
-      "severity": "error",
-      "name": "no-circular"
-    },
-    "cycle": [
-      "src/feature-libraries/flex-tree/lazyField.ts",
-      "src/feature-libraries/node-key/index.ts",
-      "src/feature-libraries/node-key/nodeKeyIndex.ts",
-      "src/feature-libraries/flex-tree/index.ts",
-      "src/feature-libraries/flex-tree/context.ts"
-    ]
-  },
-  {
-    "type": "cycle",
-    "from": "src/feature-libraries/flex-tree/context.ts",
-    "to": "src/feature-libraries/node-key/index.ts",
-    "rule": {
-      "severity": "error",
-      "name": "no-circular"
-    },
-    "cycle": [
-      "src/feature-libraries/node-key/index.ts",
-      "src/feature-libraries/node-key/nodeKeyIndex.ts",
-      "src/feature-libraries/flex-tree/index.ts",
-      "src/feature-libraries/flex-tree/context.ts"
-    ]
-  },
   {
     "type": "cycle",
     "from": "src/feature-libraries/flex-tree/lazyField.ts",
@@ -68,44 +37,5 @@
       "src/feature-libraries/flex-tree/unboxed.ts",
       "src/feature-libraries/flex-tree/lazyNode.ts"
     ]
-  },
-  {
-    "type": "cycle",
-    "from": "src/feature-libraries/modular-schema/comparison.ts",
-    "to": "src/feature-libraries/modular-schema/fieldKind.ts",
-    "rule": {
-      "severity": "error",
-      "name": "no-circular"
-    },
-    "cycle": [
-      "src/feature-libraries/modular-schema/fieldKind.ts",
-      "src/feature-libraries/modular-schema/comparison.ts"
-    ]
-  },
-  {
-    "type": "cycle",
-    "from": "src/feature-libraries/sequence-field/markListFactory.ts",
-    "to": "src/feature-libraries/sequence-field/utils.ts",
-    "rule": {
-      "severity": "error",
-      "name": "no-circular"
-    },
-    "cycle": [
-      "src/feature-libraries/sequence-field/utils.ts",
-      "src/feature-libraries/sequence-field/markListFactory.ts"
-    ]
-  },
-  {
-    "type": "cycle",
-    "from": "src/feature-libraries/sequence-field/moveEffectTable.ts",
-    "to": "src/feature-libraries/sequence-field/utils.ts",
-    "rule": {
-      "severity": "error",
-      "name": "no-circular"
-    },
-    "cycle": [
-      "src/feature-libraries/sequence-field/utils.ts",
-      "src/feature-libraries/sequence-field/moveEffectTable.ts"
-    ]
   }
-]
+]

package/{.eslintrc.js → .eslintrc.cjs}

@@ -4,9 +4,12 @@
  */
 
 module.exports = {
-	extends: [require.resolve("@fluidframework/eslint-config-fluid/minimal"), "prettier"],
+	extends: [
+		require.resolve("@fluidframework/eslint-config-fluid/minimal-deprecated"),
+		"prettier",
+	],
 	parserOptions: {
-		project: ["./tsconfig.json", "./src/test/tsconfig.json"],
+		project: ["./tsconfig.json"],
 	},
 	rules: {
 		"@typescript-eslint/no-namespace": "off",
@@ -26,6 +29,9 @@ module.exports = {
 	overrides: [
 		{
 			files: ["src/test/**/*"],
+			parserOptions: {
+				project: ["./src/test/tsconfig.json"],
+			},
 			rules: {
 				"@typescript-eslint/no-unused-vars": ["off"],
 			},

package/{.mocharc.js → .mocharc.cjs}

@@ -5,9 +5,9 @@
 
 "use strict";
 
-const getFluidTestMochaConfig = require("@fluidframework/mocha-test-setup/mocharc-common");
+const getFluidTestMochaConfig = require("@fluid-internal/mocha-test-setup/mocharc-common");
 
 const packageDir = __dirname;
 const config = getFluidTestMochaConfig(packageDir);
-config.spec = process.env.MOCHA_SPEC ?? "dist/test";
+config.spec = process.env.MOCHA_SPEC ?? "lib/test";
 module.exports = config;

package/.vscode/settings.json

@@ -1,5 +1,5 @@
 {
-	// File path is already set in .mocharc.js, so this should not be required,
+	// File path is already set in .mocharc.cjs, so this should not be required,
 	// but something seems to be overriding that path, and this is needed to make it work.
 	// Also for unknown (but likely related reasons) this cannot be just "dist/test" which is used in the config.
 	// This is not limited to "spec.js" so that benchmarks are included (which will be run in correctness mode).
@@ -7,10 +7,10 @@
 	// "pruneFiles" is disabled because it causesd "source-map-support/register" to give `Unknown file extension ".ts"` errors.
 	// "mochaExplorer.pruneFiles": true,
 	"mochaExplorer.require": [
-		"node_modules/@fluidframework/mocha-test-setup",
+		"node_modules/@fluid-internal/mocha-test-setup",
 		"source-map-support/register",
 	],
-	"mochaExplorer.configFile": ".mocharc.js",
+	"mochaExplorer.configFile": ".mocharc.cjs",
 	"mochaExplorer.timeout": 999999,
-	"cSpell.words": ["deprioritized", "endregion", "insertable", "unhydrated"],
+	"cSpell.words": ["deprioritized", "endregion", "insertable", "reentrantly", "unhydrated"],
 }

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @fluidframework/tree
 
+## 2.0.0-rc.2.0.0
+
+### Minor Changes
+
+-   map, tree: DDS classes are no longer publicly exported ([#19717](https://github.com/microsoft/FluidFramework/issues/19717)) [ae1d0be26d](https://github.com/microsoft/FluidFramework/commits/ae1d0be26d61453cff316b3f622a9f3647149167)
+
+    SharedMap and SharedTree now only export their factories and the interface types.
+    The actual concrete classes which leak implementation details are no longer exported.
+    Users of the `SharedMap` type should use `ISharedMap`.
+    Users of the `SharedTree` type should use `ISharedTree`.
+
+-   tree: Minor API fixes for "@fluidframework/tree" package. ([#19057](https://github.com/microsoft/FluidFramework/issues/19057)) [3e0f218832](https://github.com/microsoft/FluidFramework/commits/3e0f21883255317f8bb1f7c420543650502a5b66)
+
+    Rename `IterableTreeListContent` to `IterableTreeArrayContent`, inline `TreeMapNodeBase` into `TreeMapNode`, rename `TreeArrayNode.spread` to `TreeArrayNode.spread` and remove `create` which was not supposed to be public (use `TreeArrayNode.spread` instead).
+
 ## 2.0.0-rc.1.0.0
 
 ### Major Changes

package/api-extractor-cjs.json

@@ -0,0 +1,18 @@
+{
+	"$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
+	"extends": "../../../common/build/build-common/api-extractor-base.cjs.primary.json",
+	"compiler": {
+		// api-extractor has trouble with Node16 moduleResolution, but Bundler looks okay.
+		"overrideTsconfig": {
+			"$schema": "http://json.schemastore.org/tsconfig",
+			"extends": "./tsconfig.json",
+			"compilerOptions": {
+				"moduleResolution": "Bundler"
+			}
+		}
+	},
+	// CJS is actually secondary; so, no report.
+	"apiReport": {
+		"enabled": false
+	}
+}

package/api-extractor-esm.json

@@ -0,0 +1,14 @@
+{
+	"$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
+	"extends": "../../../common/build/build-common/api-extractor-base.esm.primary.json",
+	"compiler": {
+		// api-extractor has trouble with Node16 moduleResolution, but Bundler looks okay.
+		"overrideTsconfig": {
+			"$schema": "http://json.schemastore.org/tsconfig",
+			"extends": "./tsconfig.json",
+			"compilerOptions": {
+				"moduleResolution": "Bundler"
+			}
+		}
+	}
+}

package/api-extractor-lint.json

@@ -1,4 +1,4 @@
 {
 	"$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
-	"extends": "../../../common/build/build-common/api-extractor-lint.json"
+	"extends": "../../../common/build/build-common/api-extractor-lint.esm.primary.json"
 }

package/api-extractor.json

@@ -1,4 +1,15 @@
 {
 	"$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
-	"extends": "../../../common/build/build-common/api-extractor-base.json"
+	"extends": "../../../common/build/build-common/api-extractor-base.json",
+	"mainEntryPointFilePath": "./lib/index.d.ts",
+	"compiler": {
+		// api-extractor has trouble with Node16 moduleResolution, but Bundler looks okay
+		"overrideTsconfig": {
+			"$schema": "http://json.schemastore.org/tsconfig",
+			"extends": "./tsconfig.json",
+			"compilerOptions": {
+				"moduleResolution": "Bundler"
+			}
+		}
+	}
 }