@teambit/objects 0.0.0-fe8e30c7942d34e4dfb8bda5b2527ab810254a3d → 0.0.0-ff3121b14c6880670d2b2c275634315b73294711

package/objects/repository.ts

@@ -3,37 +3,35 @@ import uidNumber from 'uid-number';
 import { Mutex } from 'async-mutex';
 import { compact, uniqBy, differenceWith, isEqual } from 'lodash';
 import { BitError } from '@teambit/bit-error';
-import { ComponentID } from '@teambit/component-id';
+import type { ComponentID } from '@teambit/component-id';
 import { HASH_SIZE, isSnap } from '@teambit/component-version';
 import * as path from 'path';
 import { pMapPool } from '@teambit/toolbox.promise.map-pool';
 import { OBJECTS_DIR } from '@teambit/legacy.constants';
 import { logger } from '@teambit/legacy.logger';
-import { glob, writeFile, ChownOptions, PathOsBasedAbsolute } from '@teambit/legacy.utils';
+import type { ChownOptions, PathOsBasedAbsolute } from '@teambit/legacy.utils';
+import { writeFile } from '@teambit/legacy.utils';
+import { glob } from 'glob';
 import { removeEmptyDir } from '@teambit/toolbox.fs.remove-empty-dir';
 import { concurrentIOLimit } from '@teambit/harmony.modules.concurrency';
-import {
-  Types,
-  HashNotFound,
-  OutdatedIndexJson,
-  ScopeJson,
-  UnmergedComponents,
-  RemoteLanes,
-} from '@teambit/legacy.scope';
-import { ScopeIndex, IndexType, IndexItem } from './scope-index';
+import type { Types, ScopeJson } from '@teambit/legacy.scope';
+import { HashNotFound, OutdatedIndexJson, UnmergedComponents, RemoteLanes } from '@teambit/legacy.scope';
+import type { IndexType, IndexItem } from './scope-index';
+import { ScopeIndex } from './scope-index';
 import BitObject from './object';
-import { ObjectItem, ObjectList } from './object-list';
+import type { ObjectItem } from './object-list';
+import { ObjectList } from './object-list';
 import BitRawObject from './raw-object';
 import Ref from './ref';
-import { ContentTransformer, onPersist, onRead } from './repository-hooks';
-import { getMaxSizeForObjects, InMemoryCache, createInMemoryCache } from '@teambit/harmony.modules.in-memory-cache';
+import type { InMemoryCache } from '@teambit/harmony.modules.in-memory-cache';
+import { getMaxSizeForObjects, createInMemoryCache } from '@teambit/harmony.modules.in-memory-cache';
 import { ScopeMeta, Lane, ModelComponent } from '../models';
 
+type ContentTransformer = (content: Buffer) => Buffer;
 const OBJECTS_BACKUP_DIR = `${OBJECTS_DIR}.bak`;
 const TRASH_DIR = 'trash';
 
 export default class Repository {
-  // @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
   objects: { [key: string]: BitObject } = {};
   objectsToRemove: Ref[] = [];
   scopeJson: ScopeJson;
@@ -48,8 +46,8 @@ export default class Repository {
   constructor(scopePath: string, scopeJson: ScopeJson) {
     this.scopePath = scopePath;
     this.scopeJson = scopeJson;
-    this.onRead = onRead(scopePath, scopeJson);
-    this.onPersist = onPersist(scopePath, scopeJson);
+    this.onRead = (content: Buffer) => Repository.onPostObjectRead?.(content) || content;
+    this.onPersist = (content: Buffer) => Repository.onPreObjectPersist?.(content) || content;
     this.cache = createInMemoryCache({ maxSize: getMaxSizeForObjects() });
   }
 
@@ -92,6 +90,20 @@ export default class Repository {
 
   static onPostObjectsPersist: () => Promise<void>;
 
+  /**
+   * Hook for transforming content before objects are persisted to the filesystem.
+   * Note: This function cannot be async because it's used by the synchronous `loadSync` method
+   * which needs to maintain sync behavior for compatibility with existing code.
+   */
+  static onPreObjectPersist: (content: Buffer) => Buffer;
+
+  /**
+   * Hook for transforming content after objects are read from the filesystem.
+   * Note: This function cannot be async because it's used by the synchronous `loadSync` method
+   * which needs to maintain sync behavior for compatibility with existing code.
+   */
+  static onPostObjectRead: (content: Buffer) => Buffer;
+
   async reLoadScopeIndex() {
     this.scopeIndex = await this.loadOptionallyCreateScopeIndex();
   }

package/objects/scope-index.ts

@@ -7,8 +7,8 @@ import { logger } from '@teambit/legacy.logger';
 import { InvalidIndexJson } from '@teambit/legacy.scope';
 import { ModelComponent, Symlink } from '../models';
 import Lane from '../models/lane';
-import BitObject from './object';
-import Ref from './ref';
+import type BitObject from './object';
+import type Ref from './ref';
 import { difference } from 'lodash';
 
 const COMPONENTS_INDEX_FILENAME = 'index.json';
@@ -111,7 +111,6 @@ export class ScopeIndex {
   }
 
   getHashes(indexType: IndexType): string[] {
-    // @ts-ignore how to tell TS that all this.index.prop are array?
     return this.index[indexType].map((indexItem: IndexItem) => indexItem.hash);
   }
   getHashesByQuery(indexType: IndexType, filter: Function): string[] {

package/package.json

@@ -1,24 +1,24 @@
 {
   "name": "@teambit/objects",
-  "version": "0.0.0-fe8e30c7942d34e4dfb8bda5b2527ab810254a3d",
+  "version": "0.0.0-ff3121b14c6880670d2b2c275634315b73294711",
   "homepage": "https://bit.cloud/teambit/scope/objects",
   "main": "dist/index.js",
   "componentId": {
     "scope": "teambit.scope",
     "name": "objects",
-    "version": "fe8e30c7942d34e4dfb8bda5b2527ab810254a3d"
+    "version": "ff3121b14c6880670d2b2c275634315b73294711"
   },
   "dependencies": {
-    "@pnpm/dependency-path": "1001.0.1",
-    "@pnpm/lockfile.types": "^1001.0.8",
+    "@pnpm/dependency-path": "1001.1.7",
+    "@pnpm/lockfile.types": "^1002.0.6",
     "semver": "7.7.1",
     "lodash": "4.17.21",
     "uuid": "8.3.2",
     "async-mutex": "0.3.1",
     "p-map-series": "2.1.0",
     "tar-stream": "2.2.0",
-    "is-relative-path": "2.0.0",
     "fs-extra": "10.0.0",
+    "glob": "13.0.0",
     "uid-number": "0.0.6",
     "@teambit/harmony": "0.4.7",
     "@teambit/component-id": "1.2.4",
@@ -26,42 +26,42 @@
     "@teambit/component-version": "1.0.4",
     "@teambit/lane-id": "0.0.312",
     "@teambit/legacy-bit-id": "1.1.3",
-    "@teambit/graph.cleargraph": "0.0.11",
-    "@teambit/cli": "0.0.0-81601363f5d92834876d7fc06c2f2e36696ab1c2",
-    "@teambit/legacy.utils": "0.0.21",
-    "@teambit/harmony.modules.get-basic-log": "0.0.56",
-    "@teambit/component.snap-distance": "0.0.56",
-    "@teambit/config-store": "0.0.0-1899900ee3812e7608cdc89f69d6cd54e5b87309",
-    "@teambit/legacy.cli.error": "0.0.21",
-    "@teambit/legacy.constants": "0.0.12",
-    "@teambit/legacy.logger": "0.0.21",
-    "@teambit/legacy.scope": "0.0.55",
-    "@teambit/toolbox.crypto.sha1": "0.0.7",
-    "@teambit/component.sources": "0.0.107",
-    "@teambit/legacy-component-log": "0.0.407",
-    "@teambit/legacy.consumer-component": "0.0.56",
-    "@teambit/legacy.consumer-config": "0.0.55",
-    "@teambit/legacy.extension-data": "0.0.57",
-    "@teambit/pkg.modules.semver-helper": "0.0.11",
     "@teambit/toolbox.array.duplications-finder": "0.0.3",
-    "@teambit/bit.get-bit-version": "0.0.5",
-    "@teambit/semantics.doc-parser": "0.0.63",
-    "@teambit/graph": "0.0.0-1e9b2605564ee8ea56a8a999248e9355dca3a3ac",
-    "@teambit/harmony.modules.concurrency": "0.0.13",
-    "@teambit/toolbox.promise.map-pool": "0.0.6",
-    "@teambit/harmony.modules.in-memory-cache": "0.0.14",
-    "@teambit/toolbox.fs.remove-empty-dir": "0.0.5"
+    "@teambit/graph.cleargraph": "0.0.11",
+    "@teambit/cli": "0.0.0-69a03b2177c66dececbe9883c1fcd9c630beda4a",
+    "@teambit/legacy.utils": "0.0.0-6295a8192f6b5690d60e820e03b50b49f6e073da",
+    "@teambit/harmony.modules.get-basic-log": "0.0.0-47a2e6f6b3937ad7e7ac6d560fb367ba0ae10cea",
+    "@teambit/component.snap-distance": "0.0.0-2274755b369ae5a0699b1b1f59e27784a89cc53a",
+    "@teambit/config-store": "0.0.0-1ff31af92694d12576e1e3a72baec12f81890d0c",
+    "@teambit/legacy.cli.error": "0.0.0-4b4ca3eae38b41b16b582e88cc71df7486bb9320",
+    "@teambit/legacy.constants": "0.0.0-d75a19d39ec69adb051d384b260cd1035a1cb3f6",
+    "@teambit/legacy.logger": "0.0.0-0cf1108d874f2b5ba9c3fd3a3b422e9bf48e99c5",
+    "@teambit/legacy.scope": "0.0.0-4d81bcba0c98a9aff72479ad04da65c4ca7d4e90",
+    "@teambit/toolbox.crypto.sha1": "0.0.0-1aab01159fc75f42007d28d2b4b7eee55c5f8022",
+    "@teambit/component.sources": "0.0.0-b3da4721a60c5b9c9cb6ea87432561899b4c15ac",
+    "@teambit/legacy-component-log": "0.0.0-de6dd7d5cab27d5a97a9550195a4baaea45bc915",
+    "@teambit/legacy.consumer-component": "0.0.0-ee4ca34eb70b6969a710bdb63d229a4b2281480c",
+    "@teambit/legacy.consumer-config": "0.0.0-83bab93821cb0e23164cdd37894626985d3f06cb",
+    "@teambit/legacy.extension-data": "0.0.0-3990447ef57cae9d7d1d1ca8a0361873d610cc14",
+    "@teambit/pkg.modules.semver-helper": "0.0.0-6390eb2fd66bae6c4e6ae5db48a0a2d5542a827f",
+    "@teambit/bit.get-bit-version": "0.0.0-c0d1877a0377936650ad85593ecd49745504282b",
+    "@teambit/semantics.doc-parser": "0.0.0-9cd4d1f915ff7bb5f83ebcb0160191ace7026628",
+    "@teambit/graph": "0.0.0-a9bb47c7ba50e86f9149a45b5f62b1af653a55de",
+    "@teambit/harmony.modules.concurrency": "0.0.0-43f782bb55ca2196348e9a1e2393ce3a8459377b",
+    "@teambit/toolbox.promise.map-pool": "0.0.0-2c58e80bf30293b35bdd68bb5b9f4ff02907c5d8",
+    "@teambit/harmony.modules.in-memory-cache": "0.0.0-b0882c18906d00edf0ca7bdd784115e84cd40a3e",
+    "@teambit/toolbox.fs.remove-empty-dir": "0.0.0-cd8f2201e578f5c349b56a8333cff10824df6629"
   },
   "devDependencies": {
     "@types/semver": "7.5.8",
     "@types/lodash": "4.14.165",
     "@types/uuid": "8.3.4",
     "@types/fs-extra": "9.0.7",
-    "@teambit/harmony.envs.core-aspect-env": "0.0.69"
+    "@teambit/harmony.envs.core-aspect-env": "0.1.0"
   },
   "peerDependencies": {
-    "chai": "4.3.0",
-    "@types/chai": "4.3.19"
+    "chai": "5.2.1",
+    "@types/chai": "5.2.2"
   },
   "license": "Apache-2.0",
   "optionalDependencies": {},

package/dist/objects/repository-hooks.d.ts

@@ -1,4 +0,0 @@
-import { ScopeJson } from '@teambit/legacy.scope';
-export type ContentTransformer = (content: Buffer) => Buffer;
-export declare function onPersist(scopePath: string, scopeJson: ScopeJson): ContentTransformer;
-export declare function onRead(scopePath: string, scopeJson: ScopeJson): ContentTransformer;

package/dist/objects/repository-hooks.js

@@ -1,56 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.onPersist = onPersist;
-exports.onRead = onRead;
-function _isRelativePath() {
-  const data = _interopRequireDefault(require("is-relative-path"));
-  _isRelativePath = function () {
-    return data;
-  };
-  return data;
-}
-function _path() {
-  const data = _interopRequireDefault(require("path"));
-  _path = function () {
-    return data;
-  };
-  return data;
-}
-function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
-function loadHooks(scopePath, scopeJson) {
-  const hooksPath = scopeJson.hooksPath;
-  if (hooksPath) {
-    const hooksFinalPath = (0, _isRelativePath().default)(hooksPath) ? _path().default.join(scopePath, hooksPath) : hooksPath;
-    // eslint-disable-next-line global-require, import/no-dynamic-require
-    const hooks = require(hooksFinalPath);
-    return hooks;
-  }
-  return undefined;
-}
-function onPersist(scopePath, scopeJson) {
-  const defaultFunc = content => content;
-  const hooks = loadHooks(scopePath, scopeJson);
-  if (hooks) {
-    const onReadFunction = hooks.onPersist;
-    if (onReadFunction) {
-      return onReadFunction;
-    }
-  }
-  return defaultFunc;
-}
-function onRead(scopePath, scopeJson) {
-  const defaultFunc = content => content;
-  const hooks = loadHooks(scopePath, scopeJson);
-  if (hooks) {
-    const onReadFunction = hooks.onRead;
-    if (onReadFunction) {
-      return onReadFunction;
-    }
-  }
-  return defaultFunc;
-}
-
-//# sourceMappingURL=repository-hooks.js.map

package/dist/objects/repository-hooks.js.map

@@ -1 +0,0 @@
-{"version":3,"names":["_isRelativePath","data","_interopRequireDefault","require","_path","e","__esModule","default","loadHooks","scopePath","scopeJson","hooksPath","hooksFinalPath","isRelative","path","join","hooks","undefined","onPersist","defaultFunc","content","onReadFunction","onRead"],"sources":["repository-hooks.ts"],"sourcesContent":["import isRelative from 'is-relative-path';\nimport path from 'path';\n\nimport { ScopeJson } from '@teambit/legacy.scope';\n\nexport type ContentTransformer = (content: Buffer) => Buffer;\n\nfunction loadHooks(scopePath: string, scopeJson: ScopeJson): any | undefined {\n  const hooksPath = scopeJson.hooksPath;\n  if (hooksPath) {\n    const hooksFinalPath = isRelative(hooksPath) ? path.join(scopePath, hooksPath) : hooksPath;\n    // eslint-disable-next-line global-require, import/no-dynamic-require\n    const hooks = require(hooksFinalPath);\n    return hooks;\n  }\n  return undefined;\n}\n\nexport function onPersist(scopePath: string, scopeJson: ScopeJson): ContentTransformer {\n  const defaultFunc = (content) => content;\n  const hooks = loadHooks(scopePath, scopeJson);\n  if (hooks) {\n    const onReadFunction = hooks.onPersist;\n    if (onReadFunction) {\n      return onReadFunction;\n    }\n  }\n  return defaultFunc;\n}\n\nexport function onRead(scopePath: string, scopeJson: ScopeJson): ContentTransformer {\n  const defaultFunc = (content) => content;\n\n  const hooks = loadHooks(scopePath, scopeJson);\n  if (hooks) {\n    const onReadFunction = hooks.onRead;\n    if (onReadFunction) {\n      return onReadFunction;\n    }\n  }\n  return defaultFunc;\n}\n"],"mappings":";;;;;;;AAAA,SAAAA,gBAAA;EAAA,MAAAC,IAAA,GAAAC,sBAAA,CAAAC,OAAA;EAAAH,eAAA,YAAAA,CAAA;IAAA,OAAAC,IAAA;EAAA;EAAA,OAAAA,IAAA;AAAA;AACA,SAAAG,MAAA;EAAA,MAAAH,IAAA,GAAAC,sBAAA,CAAAC,OAAA;EAAAC,KAAA,YAAAA,CAAA;IAAA,OAAAH,IAAA;EAAA;EAAA,OAAAA,IAAA;AAAA;AAAwB,SAAAC,uBAAAG,CAAA,WAAAA,CAAA,IAAAA,CAAA,CAAAC,UAAA,GAAAD,CAAA,KAAAE,OAAA,EAAAF,CAAA;AAMxB,SAASG,SAASA,CAACC,SAAiB,EAAEC,SAAoB,EAAmB;EAC3E,MAAMC,SAAS,GAAGD,SAAS,CAACC,SAAS;EACrC,IAAIA,SAAS,EAAE;IACb,MAAMC,cAAc,GAAG,IAAAC,yBAAU,EAACF,SAAS,CAAC,GAAGG,eAAI,CAACC,IAAI,CAACN,SAAS,EAAEE,SAAS,CAAC,GAAGA,SAAS;IAC1F;IACA,MAAMK,KAAK,GAAGb,OAAO,CAACS,cAAc,CAAC;IACrC,OAAOI,KAAK;EACd;EACA,OAAOC,SAAS;AAClB;AAEO,SAASC,SAASA,CAACT,SAAiB,EAAEC,SAAoB,EAAsB;EACrF,MAAMS,WAAW,GAAIC,OAAO,IAAKA,OAAO;EACxC,MAAMJ,KAAK,GAAGR,SAAS,CAACC,SAAS,EAAEC,SAAS,CAAC;EAC7C,IAAIM,KAAK,EAAE;IACT,MAAMK,cAAc,GAAGL,KAAK,CAACE,SAAS;IACtC,IAAIG,cAAc,EAAE;MAClB,OAAOA,cAAc;IACvB;EACF;EACA,OAAOF,WAAW;AACpB;AAEO,SAASG,MAAMA,CAACb,SAAiB,EAAEC,SAAoB,EAAsB;EAClF,MAAMS,WAAW,GAAIC,OAAO,IAAKA,OAAO;EAExC,MAAMJ,KAAK,GAAGR,SAAS,CAACC,SAAS,EAAEC,SAAS,CAAC;EAC7C,IAAIM,KAAK,EAAE;IACT,MAAMK,cAAc,GAAGL,KAAK,CAACM,MAAM;IACnC,IAAID,cAAc,EAAE;MAClB,OAAOA,cAAc;IACvB;EACF;EACA,OAAOF,WAAW;AACpB","ignoreList":[]}

package/objects/repository-hooks.ts

@@ -1,42 +0,0 @@
-import isRelative from 'is-relative-path';
-import path from 'path';
-
-import { ScopeJson } from '@teambit/legacy.scope';
-
-export type ContentTransformer = (content: Buffer) => Buffer;
-
-function loadHooks(scopePath: string, scopeJson: ScopeJson): any | undefined {
-  const hooksPath = scopeJson.hooksPath;
-  if (hooksPath) {
-    const hooksFinalPath = isRelative(hooksPath) ? path.join(scopePath, hooksPath) : hooksPath;
-    // eslint-disable-next-line global-require, import/no-dynamic-require
-    const hooks = require(hooksFinalPath);
-    return hooks;
-  }
-  return undefined;
-}
-
-export function onPersist(scopePath: string, scopeJson: ScopeJson): ContentTransformer {
-  const defaultFunc = (content) => content;
-  const hooks = loadHooks(scopePath, scopeJson);
-  if (hooks) {
-    const onReadFunction = hooks.onPersist;
-    if (onReadFunction) {
-      return onReadFunction;
-    }
-  }
-  return defaultFunc;
-}
-
-export function onRead(scopePath: string, scopeJson: ScopeJson): ContentTransformer {
-  const defaultFunc = (content) => content;
-
-  const hooks = loadHooks(scopePath, scopeJson);
-  if (hooks) {
-    const onReadFunction = hooks.onRead;
-    if (onReadFunction) {
-      return onReadFunction;
-    }
-  }
-  return defaultFunc;
-}