@teambit/workspace 1.0.173 → 1.0.175

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "@teambit/workspace",
-  "version": "1.0.173",
+  "version": "1.0.175",
   "homepage": "https://bit.cloud/teambit/workspace/workspace",
   "main": "dist/index.js",
   "componentId": {
     "scope": "teambit.workspace",
     "name": "workspace",
-    "version": "1.0.173"
+    "version": "1.0.175"
   },
   "dependencies": {
     "lodash": "4.17.21",
@@ -55,42 +55,42 @@
     "@teambit/explorer.ui.gallery.component-grid": "0.0.496",
     "@teambit/scopes.scope-id": "0.0.7",
     "@teambit/workspace.ui.empty-workspace": "0.0.508",
-    "@teambit/component": "1.0.173",
-    "@teambit/dependency-resolver": "1.0.173",
-    "@teambit/envs": "1.0.173",
+    "@teambit/component": "1.0.175",
+    "@teambit/dependency-resolver": "1.0.175",
+    "@teambit/envs": "1.0.175",
     "@teambit/logger": "0.0.944",
-    "@teambit/scope": "1.0.173",
-    "@teambit/graph": "1.0.173",
+    "@teambit/scope": "1.0.175",
+    "@teambit/graph": "1.0.175",
     "@teambit/cli": "0.0.851",
-    "@teambit/isolator": "1.0.173",
-    "@teambit/component-tree": "1.0.173",
+    "@teambit/isolator": "1.0.175",
+    "@teambit/component-tree": "1.0.175",
     "@teambit/harmony.modules.resolved-component": "0.0.497",
-    "@teambit/watcher": "1.0.173",
-    "@teambit/aspect-loader": "1.0.173",
-    "@teambit/config": "0.0.924",
+    "@teambit/watcher": "1.0.175",
+    "@teambit/aspect-loader": "1.0.175",
+    "@teambit/config": "0.0.926",
     "@teambit/global-config": "0.0.854",
     "@teambit/harmony.modules.requireable-component": "0.0.497",
     "@teambit/toolbox.modules.module-resolver": "0.0.7",
-    "@teambit/workspace.modules.node-modules-linker": "0.0.164",
-    "@teambit/graphql": "1.0.173",
-    "@teambit/bundler": "1.0.173",
-    "@teambit/pubsub": "1.0.173",
-    "@teambit/ui": "1.0.173",
-    "@teambit/variants": "0.0.1017",
+    "@teambit/workspace.modules.node-modules-linker": "0.0.165",
+    "@teambit/graphql": "1.0.175",
+    "@teambit/bundler": "1.0.175",
+    "@teambit/pubsub": "1.0.175",
+    "@teambit/ui": "1.0.175",
+    "@teambit/variants": "0.0.1019",
     "@teambit/component-issues": "0.0.141",
     "@teambit/workspace.modules.match-pattern": "0.0.505",
-    "@teambit/component.ui.component-drawer": "0.0.392",
+    "@teambit/component.ui.component-drawer": "0.0.393",
     "@teambit/lanes.hooks.use-lane-components": "0.0.258",
-    "@teambit/lanes.hooks.use-lanes": "0.0.259",
+    "@teambit/lanes.hooks.use-lanes": "0.0.260",
     "@teambit/lanes.ui.models.lanes-model": "0.0.212",
-    "@teambit/command-bar": "1.0.173",
+    "@teambit/command-bar": "1.0.175",
     "@teambit/component.ui.component-filters.component-filter-context": "0.0.218",
     "@teambit/component.ui.component-filters.deprecate-filter": "0.0.216",
-    "@teambit/component.ui.component-filters.env-filter": "0.0.222",
+    "@teambit/component.ui.component-filters.env-filter": "0.0.223",
     "@teambit/component.ui.component-filters.show-main-filter": "0.0.209",
-    "@teambit/sidebar": "1.0.173",
+    "@teambit/sidebar": "1.0.175",
     "@teambit/component-descriptor": "0.0.415",
-    "@teambit/deprecation": "1.0.173",
+    "@teambit/deprecation": "1.0.175",
     "@teambit/ui-foundation.ui.buttons.collapser": "0.0.217",
     "@teambit/cloud.hooks.use-cloud-scopes": "0.0.3",
     "@teambit/preview.ui.preview-placeholder": "0.0.514",

package/workspace-component/comp-files.ts

@@ -13,14 +13,14 @@ export class CompFiles {
     private repository: Repository,
     private currentFiles: SourceFile[],
     readonly compDir: PathLinux,
-    private headFiles: SourceFileModel[] = []
+    private modelFiles: SourceFileModel[] = []
   ) {}
 
   isModified(): boolean {
-    if (!this.headFiles.length) return false;
-    if (this.currentFiles.length !== this.headFiles.length) return true;
+    if (!this.modelFiles.length) return false;
+    if (this.currentFiles.length !== this.modelFiles.length) return true;
     return this.currentFiles.some((file) => {
-      const headFile = this.headFiles.find((h) => h.relativePath === file.relative);
+      const headFile = this.modelFiles.find((h) => h.relativePath === file.relative);
       if (!headFile) return true;
       return !headFile.file.isEqual(file.toSourceAsLinuxEOL().hash());
     });
@@ -31,12 +31,12 @@ export class CompFiles {
   }
 
   async getHeadFiles(): Promise<SourceFile[]> {
-    return Promise.all(this.headFiles.map((file) => SourceFile.loadFromSourceFileModel(file, this.repository)));
+    return Promise.all(this.modelFiles.map((file) => SourceFile.loadFromSourceFileModel(file, this.repository)));
   }
 
   getFilesStatus(): FilesStatus {
     const result = this.currentFiles.reduce((acc, file) => {
-      const headFile = this.headFiles.find((h) => h.relativePath === file.relative);
+      const headFile = this.modelFiles.find((h) => h.relativePath === file.relative);
       const getType = (): FILE_STATUS => {
         if (!headFile) return 'new';
         if (headFile.file.isEqual(file.toSourceAsLinuxEOL().hash())) return 'unchanged';
@@ -45,7 +45,7 @@ export class CompFiles {
       acc[file.relative] = getType();
       return acc;
     }, {});
-    this.headFiles.forEach((headFile) => {
+    this.modelFiles.forEach((headFile) => {
       const currentFile = this.currentFiles.find((c) => c.relative === headFile.relativePath);
       if (!currentFile) {
         result[headFile.relativePath] = 'deleted';

package/workspace-component/component-status-loader.ts

@@ -54,7 +54,11 @@ export class ComponentStatusLoader {
    */
   async getComponentStatusById(id: ComponentID): Promise<ComponentStatusLegacy> {
     if (!this._componentsStatusCache[id.toString()]) {
-      this._componentsStatusCache[id.toString()] = await this.getStatus(id);
+      // don't do this: `this._componentsStatusCache[id.toString()] = await this.getStatus(id);`
+      // yes, it doesn't make sense right? turns out that "getStatus" can call `linkIfMissingWorkspaceAspects` which
+      // calls `linkToNodeModulesByIds` which deletes this cache. and makes this: `this._componentsStatusCache[id.toString()]` undefined.
+      const result = await this.getStatus(id);
+      this._componentsStatusCache[id.toString()] = result;
     }
     return this._componentsStatusCache[id.toString()];
   }