msfluid-bot
commented
1 week ago ⯅ @fluid-example/bundle-size-tests: +245 Bytes
| Metric Name | Baseline Size | Compare Size | Size Diff |
|---|---|---|---|
| aqueduct.js | 461.15 KB | 461.18 KB | ⯅ +35 Bytes |
| azureClient.js | 559.19 KB | 559.24 KB | ⯅ +49 Bytes |
| connectionState.js | 680 Bytes | 680 Bytes | ■ No change |
| containerRuntime.js | 261.99 KB | 262 KB | ⯅ +14 Bytes |
| fluidFramework.js | 399.76 KB | 399.77 KB | ⯅ +14 Bytes |
| loader.js | 134.26 KB | 134.28 KB | ⯅ +14 Bytes |
| map.js | 42.39 KB | 42.39 KB | ⯅ +7 Bytes |
| matrix.js | 146.56 KB | 146.56 KB | ⯅ +7 Bytes |
| odspClient.js | 526.47 KB | 526.52 KB | ⯅ +49 Bytes |
| odspDriver.js | 97.72 KB | 97.74 KB | ⯅ +21 Bytes |
| odspPrefetchSnapshot.js | 42.78 KB | 42.79 KB | ⯅ +14 Bytes |
| sharedString.js | 163.26 KB | 163.26 KB | ⯅ +7 Bytes |
| sharedTree.js | 390.27 KB | 390.28 KB | ⯅ +7 Bytes |
| Total Size | 3.3 MB | 3.3 MB | ⯅ +245 Bytes |
Baseline commit: 5c38a8f3f1d248d3ecb32b88b363288f1697c2e7
Generated by :no_entry_sign: dangerJS against 18f0cbeaaefd8ced301ff7ff540c3fafafacbd5a
This change does some internal organization within the build-tools package to better isolate fluid-build-only functionality from shared functionality. Everything that is fluid-build only - with the exception of the config and the "getConfig" function - should be within the fluidBuild folder and only referenced from other files within that folder.
This change also:
resolveNodeModulefrom common/utils.tsThere is no intended behavior change. It's only meant to reorganize existing code.
There are still some circular relationships that need to be rationalized. For example, the
MonoRepoandPackageclasses expect the fluid-build properties to be included in package.json, but that config should really be specific to fluid-build. That's not addressed in this change, but it and similar issues will be addressed over time, starting with the introduction of a shared build infrastructure package in #22347.