@finos/legend-graph 4.0.3 → 5.0.0

package/src/models/protocols/pure/v1/model/packageableElements/store/relational/connection/V1_RelationalDatabaseConnection.ts

@@ -24,6 +24,8 @@ import type { V1_DatasourceSpecification } from './V1_DatasourceSpecification';
 import type { V1_AuthenticationStrategy } from './V1_AuthenticationStrategy';
 import type { V1_PostProcessor } from './postprocessor/V1_PostProcessor';
 
+// TODO: to be modularized or handled differently
+// See https://github.com/finos/legend-studio/issues/946
 export enum V1_DatabaseType {
   DB2 = 'DB2',
   H2 = 'H2',

package/src/models/protocols/pure/v1/model/packageableElements/store/relational/connection/postprocessor/V1_PostProcessor.ts

@@ -17,7 +17,5 @@
 import type { Hashable } from '@finos/legend-shared';
 
 export abstract class V1_PostProcessor implements Hashable {
-  private readonly _$nominalTypeBrand!: 'V1_PostProcessor';
-
   abstract get hashCode(): string;
 }

package/src/models/protocols/pure/v1/model/packageableElements/store/relational/model/V1_RelationalDataType.ts

@@ -18,8 +18,6 @@ import { type Hashable, hashArray } from '@finos/legend-shared';
 import { CORE_HASH_STRUCTURE } from '../../../../../../../../../MetaModelConst';
 
 export abstract class V1_RelationalDataType implements Hashable {
-  private readonly _$nominalTypeBrand!: 'V1_RelationalDataType';
-
   abstract get hashCode(): string;
 }
 

package/src/models/protocols/pure/v1/model/packageableElements/store/relational/model/V1_RelationalOperationElement.ts

@@ -20,8 +20,6 @@ import type { V1_JoinPointer } from './V1_JoinPointer';
 import type { V1_TablePtr } from './V1_TablePtr';
 
 export abstract class V1_RelationalOperationElement implements Hashable {
-  private readonly _$nominalTypeBrand!: 'V1_RelationalOperationElement';
-
   abstract get hashCode(): string;
 }
 

package/src/models/protocols/pure/v1/model/packageableElements/store/relational/model/milestoning/V1_Milestoning.ts

@@ -15,9 +15,9 @@
  */
 
 import type { Hashable } from '@finos/legend-shared';
-import type { V1_RawInstanceValue } from '../../../../../rawValueSpecification/V1_RawInstanceValue';
+import type { V1_RawPrimitiveInstanceValue } from '../../../../../rawValueSpecification/V1_RawPrimitiveInstanceValue';
 
 export abstract class V1_Milestoning implements Hashable {
-  infinityDate?: V1_RawInstanceValue | undefined;
+  infinityDate?: V1_RawPrimitiveInstanceValue | undefined;
   abstract get hashCode(): string;
 }

package/src/models/protocols/pure/v1/model/rawValueSpecification/{V1_RawInstanceValue.ts → V1_RawPrimitiveInstanceValue.ts}

@@ -19,10 +19,10 @@ import { CORE_HASH_STRUCTURE } from '../../../../../../MetaModelConst';
 import {
   type V1_RawValueSpecificationVisitor,
   V1_RawValueSpecification,
-} from '../../model/rawValueSpecification/V1_RawValueSpecification';
-import type { V1_Multiplicity } from '../../model/packageableElements/domain/V1_Multiplicity';
+} from './V1_RawValueSpecification';
+import type { V1_Multiplicity } from '../packageableElements/domain/V1_Multiplicity';
 
-export class V1_RawInstanceValue
+export class V1_RawPrimitiveInstanceValue
   extends V1_RawValueSpecification
   implements Hashable
 {
@@ -44,6 +44,6 @@ export class V1_RawInstanceValue
   accept_RawValueSpecificationVisitor<T>(
     visitor: V1_RawValueSpecificationVisitor<T>,
   ): T {
-    return visitor.visit_InstanceValue(this);
+    return visitor.visit_PrimitiveInstanceValue(this);
   }
 }

package/src/models/protocols/pure/v1/model/rawValueSpecification/V1_RawValueSpecification.ts

@@ -16,12 +16,14 @@
 
 import type { V1_RawVariable } from '../../model/rawValueSpecification/V1_RawVariable';
 import type { V1_RawLambda } from '../../model/rawValueSpecification/V1_RawLambda';
-import type { V1_RawInstanceValue } from './V1_RawInstanceValue';
+import type { V1_RawPrimitiveInstanceValue } from './V1_RawPrimitiveInstanceValue';
 
 export interface V1_RawValueSpecificationVisitor<T> {
   visit_Lambda(valueSpecification: V1_RawLambda): T;
   visit_Variable(valueSpecification: V1_RawVariable): T;
-  visit_InstanceValue(valueSpecification: V1_RawInstanceValue): T;
+  visit_PrimitiveInstanceValue(
+    valueSpecification: V1_RawPrimitiveInstanceValue,
+  ): T;
 }
 
 export abstract class V1_RawValueSpecification {

package/src/models/protocols/pure/v1/model/valueSpecification/raw/path/V1_PathElement.ts

@@ -14,6 +14,4 @@
  * limitations under the License.
  */
 
-export abstract class V1_PathElement {
-  private readonly _$nominalTypeBrand!: 'V1_PathElement';
-}
+export abstract class V1_PathElement {}

package/src/models/protocols/pure/v1/transformation/pureGraph/from/V1_MappingTransformer.ts

@@ -25,7 +25,6 @@ import type { Mapping } from '../../../../../../metamodels/pure/packageableEleme
 import type {
   SetImplementationVisitor,
   SetImplementation,
-  TEMPORARY__UnresolvedSetImplementation,
 } from '../../../../../../metamodels/pure/packageableElements/mapping/SetImplementation';
 import type {
   PropertyMappingVisitor,
@@ -149,6 +148,7 @@ import type { SubstituteStore } from '../../../../../../metamodels/pure/packagea
 import { V1_BindingTransformer } from '../../../model/packageableElements/externalFormat/store/V1_BindingTransformer';
 import { V1_MergeOperationClassMapping } from '../../../model/packageableElements/mapping/V1_MergeOperationClassMapping';
 import { MergeOperationSetImplementation } from '../../../../../../metamodels/pure/packageableElements/mapping/MergeOperationSetImplementation';
+import type { TEMPORARY__UnresolvedSetImplementation } from '../../../../../../metamodels/pure/packageableElements/mapping/TEMPORARY__UnresolvedSetImplementation';
 
 export const V1_transformPropertyReference = (
   element: PropertyReference,

package/src/models/protocols/pure/v1/transformation/pureGraph/from/V1_MilestoningTransformer.ts

@@ -25,7 +25,7 @@ import { V1_ProcessingMilestoning } from '../../../model/packageableElements/sto
 import { ProcessingMilestoning } from '../../../../../../metamodels/pure/packageableElements/store/relational/model/milestoning/ProcessingMilestoning';
 import type { StoreRelational_PureProtocolProcessorPlugin_Extension } from '../../../../StoreRelational_PureProtocolProcessorPlugin_Extension';
 import type { V1_GraphTransformerContext } from './V1_GraphTransformerContext';
-import { V1_RawInstanceValue } from '../../../model/rawValueSpecification/V1_RawInstanceValue';
+import { V1_RawPrimitiveInstanceValue } from '../../../model/rawValueSpecification/V1_RawPrimitiveInstanceValue';
 import { V1_RawValueSpecificationTransformer } from './V1_RawValueSpecificationTransformer';
 
 const transformBusinessMilesoning = (
@@ -41,7 +41,7 @@ const transformBusinessMilesoning = (
       metamodel.infinityDate.accept_RawValueSpecificationVisitor(
         new V1_RawValueSpecificationTransformer(context),
       ),
-      V1_RawInstanceValue,
+      V1_RawPrimitiveInstanceValue,
     );
   }
   return protocol;
@@ -68,7 +68,7 @@ const transformProcessingMilestoning = (
       metamodel.infinityDate.accept_RawValueSpecificationVisitor(
         new V1_RawValueSpecificationTransformer(context),
       ),
-      V1_RawInstanceValue,
+      V1_RawPrimitiveInstanceValue,
     );
   }
   return protocol;

package/src/models/protocols/pure/v1/transformation/pureGraph/from/V1_PackageableElementTransformer.ts

@@ -14,7 +14,10 @@
  * limitations under the License.
  */
 
-import { UnsupportedOperationError } from '@finos/legend-shared';
+import {
+  assertErrorThrown,
+  UnsupportedOperationError,
+} from '@finos/legend-shared';
 import type {
   PackageableElement,
   PackageableElementVisitor,
@@ -62,7 +65,7 @@ import type {
 } from '../../../../PureProtocolProcessorPlugin';
 import type { V1_GraphTransformerContext } from './V1_GraphTransformerContext';
 
-export class V1_PackageableElementTransformer
+class V1_PackageableElementTransformer
   implements PackageableElementVisitor<V1_PackageableElement>
 {
   context: V1_GraphTransformerContext;
@@ -167,3 +170,20 @@ export class V1_PackageableElementTransformer
     return V1_transformGenerationSpecification(element);
   }
 }
+
+export const V1_transformPackageableElement = (
+  element: PackageableElement,
+  plugins: PureProtocolProcessorPlugin[],
+  context: V1_GraphTransformerContext,
+): V1_PackageableElement => {
+  try {
+    return element.accept_PackageableElementVisitor(
+      new V1_PackageableElementTransformer(plugins, context),
+    );
+  } catch (error) {
+    assertErrorThrown(error);
+    // TODO?: should we wrap this in GraphTransformerError?
+    error.message = `Can't transform element '${element.path}': ${error.message}`;
+    throw error;
+  }
+};

package/src/models/protocols/pure/v1/transformation/pureGraph/from/V1_RawValueSpecificationTransformer.ts

@@ -27,8 +27,8 @@ import { SOURCE_INFORMATION_KEY } from '../../../../../../../MetaModelConst';
 import { recursiveOmit } from '@finos/legend-shared';
 import type { V1_GraphTransformerContext } from './V1_GraphTransformerContext';
 import type { V1_RawValueSpecification } from '../../../model/rawValueSpecification/V1_RawValueSpecification';
-import { V1_RawInstanceValue } from '../../../model/rawValueSpecification/V1_RawInstanceValue';
-import type { RawInstanceValue } from '../../../../../../metamodels/pure/rawValueSpecification/RawInstanceValue';
+import { V1_RawPrimitiveInstanceValue } from '../../../model/rawValueSpecification/V1_RawPrimitiveInstanceValue';
+import type { RawPrimitiveInstanceValue } from '../../../../../../metamodels/pure/rawValueSpecification/RawPrimitiveInstanceValue';
 
 export class V1_RawValueSpecificationTransformer
   implements RawValueSpecificationVisitor<V1_RawValueSpecification>
@@ -78,18 +78,16 @@ export class V1_RawValueSpecificationTransformer
     return rawVariable;
   }
 
-  visit_RawInstanceValue(
-    rawValueSpecification: RawInstanceValue,
+  visit_RawPrimitiveInstanceValue(
+    rawValueSpecification: RawPrimitiveInstanceValue,
   ): V1_RawValueSpecification {
-    const rawInstanceValue = new V1_RawInstanceValue();
-    rawInstanceValue.type = V1_transformElementReference(
-      rawValueSpecification.type,
-    );
-    rawInstanceValue.multiplicity = V1_transformMultiplicity(
+    const protocol = new V1_RawPrimitiveInstanceValue();
+    protocol.type = V1_transformElementReference(rawValueSpecification.type);
+    protocol.multiplicity = V1_transformMultiplicity(
       rawValueSpecification.multiplicity,
     );
-    rawInstanceValue.values = rawValueSpecification.values;
-    return rawInstanceValue;
+    protocol.values = rawValueSpecification.values;
+    return protocol;
   }
 }
 

package/src/models/protocols/pure/v1/transformation/pureGraph/from/V1_ValueSpecificationTransformer.ts

@@ -15,6 +15,7 @@
  */
 
 import {
+  filterByType,
   guaranteeNonNullable,
   guaranteeType,
   UnsupportedOperationError,
@@ -295,10 +296,7 @@ class V1_ValueSpecificationTransformer
     collection.multiplicity.upperBound =
       valueSpecification.multiplicity.upperBound;
     collection.values = valueSpecification.values
-      .filter(
-        (type: unknown): type is ValueSpecification =>
-          type instanceof ValueSpecification,
-      )
+      .filter(filterByType(ValueSpecification))
       .map((e) =>
         e.accept_ValueSpecificationVisitor(
           new V1_ValueSpecificationTransformer(

package/src/models/protocols/pure/v1/transformation/pureGraph/to/V1_ElementBuilder.ts

@@ -18,11 +18,19 @@ import {
   type GenericClazz,
   assertNonEmptyString,
   assertTrue,
+  AssertionError,
 } from '@finos/legend-shared';
-import { _package_addElement } from '../../../../../../../helpers/DomainHelper';
+import {
+  addElementToPackage,
+  getOrCreateGraphPackage,
+} from '../../../../../../../helpers/DomainHelper';
+import type { Package } from '../../../../../../metamodels/pure/packageableElements/domain/Package';
 import type { PackageableElement } from '../../../../../../metamodels/pure/packageableElements/PackageableElement';
 import type { V1_PackageableElement } from '../../../model/packageableElements/V1_PackageableElement';
-import type { V1_GraphBuilderContext } from './V1_GraphBuilderContext';
+import {
+  V1_buildFullPath,
+  type V1_GraphBuilderContext,
+} from './V1_GraphBuilderContext';
 
 export type V1_ElementBuilderPass = (
   elementProtocol: V1_PackageableElement,
@@ -38,6 +46,25 @@ export type V1_ElementThirdPassBuilder = V1_ElementBuilderPass;
 export type V1_ElementFourthPassBuilder = V1_ElementBuilderPass;
 export type V1_ElementFifthPassBuilder = V1_ElementBuilderPass;
 
+export const V1_checkDuplicatedElement = (
+  path: string,
+  context: V1_GraphBuilderContext,
+  cache: Set<string> | undefined,
+): void => {
+  if (cache) {
+    if (cache.has(path)) {
+      throw new AssertionError(`Element '${path}' already exists`);
+    } else {
+      cache.add(path);
+    }
+  } else {
+    assertTrue(
+      !context.graph.getNullableElement(path),
+      `Element '${path}' already exists`,
+    );
+  }
+};
+
 /**
  * Element builder is a mechanism to handling the building process of
  * element.
@@ -55,8 +82,8 @@ export type V1_ElementFifthPassBuilder = V1_ElementBuilderPass;
  *
  * Also note that as of right now we will run build pass 2-5 of each element
  * consecutively so there is no opportunity to `interleave` build passes of
- * different elements. This is definitely more flexible, but also a complexity
- * we don't see the need for right now.
+ * different elements. That definitely seems more flexible, but also incur
+ * a fair amount of complexity we don't see the need for right now.
  */
 export class V1_ElementBuilder<T extends V1_PackageableElement> {
   readonly elementClassName: string;
@@ -104,6 +131,8 @@ export class V1_ElementBuilder<T extends V1_PackageableElement> {
   runFirstPass(
     elementProtocol: T,
     context: V1_GraphBuilderContext,
+    packageCache: Map<string, Package> | undefined,
+    elementPathCache: Set<string> | undefined,
   ): PackageableElement {
     assertNonEmptyString(
       elementProtocol.package,
@@ -113,17 +142,18 @@ export class V1_ElementBuilder<T extends V1_PackageableElement> {
       elementProtocol.name,
       `Element 'name' field is missing or empty`,
     );
-    const path = context.currentSubGraph.buildPath(
+    const path = V1_buildFullPath(
       elementProtocol.package,
       elementProtocol.name,
     );
-    assertTrue(
-      !context.graph.getNullableElement(path),
-      `Element '${path}' already exists`,
-    );
+    V1_checkDuplicatedElement(path, context, elementPathCache);
     const element = this.firstPass(elementProtocol, context);
-    _package_addElement(
-      context.currentSubGraph.getOrCreatePackage(elementProtocol.package),
+    addElementToPackage(
+      getOrCreateGraphPackage(
+        context.currentSubGraph,
+        elementProtocol.package,
+        packageCache,
+      ),
       element,
     );
     return element;

package/src/models/protocols/pure/v1/transformation/pureGraph/to/V1_GraphBuilderContext.ts

@@ -17,6 +17,7 @@
 import {
   PRIMITIVE_TYPE,
   ELEMENT_PATH_DELIMITER,
+  ROOT_PACKAGE_NAME,
 } from '../../../../../../../MetaModelConst';
 import {
   type Log,
@@ -79,6 +80,15 @@ import type { GraphBuilderOptions } from '../../../../../../../graphManager/Abst
 import { DataType } from '../../../../../../metamodels/pure/packageableElements/domain/DataType';
 import { GraphBuilderError } from '../../../../../../../graphManager/GraphManagerUtils';
 
+export const V1_buildFullPath = (
+  packagePath: string | undefined,
+  name: string | undefined,
+): string =>
+  `${guaranteeNonNullable(
+    packagePath,
+    'Package path is required',
+  )}${ELEMENT_PATH_DELIMITER}${guaranteeNonNullable(name, 'Name is required')}`;
+
 interface ResolutionResult<T> {
   /**
    * The resolved element.
@@ -95,24 +105,57 @@ interface ResolutionResult<T> {
 }
 
 export class V1_GraphBuilderContext {
+  private readonly autoImports: Package[];
+  private readonly sectionImports: Package[] = [];
   readonly log: Log;
   readonly currentSubGraph: BasicModel;
   readonly extensions: V1_GraphBuilderExtensions;
   readonly graph: PureModel;
-  readonly imports: Package[] = [];
   readonly section?: Section | undefined;
   readonly options?: GraphBuilderOptions | undefined;
 
   constructor(builder: V1_GraphBuilderContextBuilder) {
     this.log = builder.log;
     this.graph = builder.graph;
+    this.autoImports = this.graph.autoImports;
     this.currentSubGraph = builder.currentSubGraph;
     this.extensions = builder.extensions;
-    this.imports = builder.imports;
+    this.sectionImports = builder.sectionImports;
     this.section = builder.section;
     this.options = builder.options;
   }
 
+  /**
+   * Since we haven't fully supported section index, shortened paths
+   * using imports in the graph might need to be fully resolved.
+   *
+   * To handle this need, we make use of references. References can auto
+   * resolve full paths when the section index is deleted. But, when
+   * building the graph, we leave the value specifications
+   * raw/unprocessed. Hence, we cannot make use of references to do full
+   * path resolution, as such, we make a best effort traversal in the model
+   * of raw value specifications to resolve path automatically.
+   *
+   * We create this flag to control the behavior of lambda auto path-resolution.
+   * This rewriting behavior should not be done for immutable graphs, such as
+   * system, depdendencies, and generation. However, in overall, it would be controlled
+   * also by the `TEMPORARY__preserveSectionIndex` flag.
+   *
+   * NOTE: When we fully support section index, we would certainly need to
+   * revise the usefullness of this flag, perhaps, we don't auto-resolve anymore,
+   * but this mechanism would still be beneficial as we can keep it as an utility
+   * to resolve raw lambdas' paths when the user deliberately delete the section
+   * index, for example.
+   *
+   * https://github.com/finos/legend-studio/issues/1067
+   */
+  get enableRawLambdaAutoPathResolution(): boolean {
+    return (
+      this.graph.root.path === ROOT_PACKAGE_NAME.MAIN &&
+      !this.options?.TEMPORARY__preserveSectionIndex
+    );
+  }
+
   resolve<T>(
     path: string,
     resolverFn: (path: string) => T,
@@ -134,21 +177,37 @@ export class V1_GraphBuilderContext {
     // NOTE: here we make the assumption that we have populated the indices properly so the same element
     // is not referred using 2 different paths in the same element index
     const results = new Map<string, ResolutionResult<T>>();
-    uniq(this.imports).forEach((importPackage) => {
+    this.autoImports.forEach((importPackage) => {
       try {
         const fullPath = importPackage.path + ELEMENT_PATH_DELIMITER + path;
         const element = resolverFn(fullPath);
         if (element) {
           results.set(fullPath, {
             element,
-            resolvedUsingSectionImports:
-              !this.graph.sectionAutoImports.includes(importPackage),
+            resolvedUsingSectionImports: false,
           });
         }
       } catch {
         // do nothing
       }
     });
+    // only resolve section imports if there is a section
+    if (this.section) {
+      this.sectionImports.forEach((importPackage) => {
+        try {
+          const fullPath = importPackage.path + ELEMENT_PATH_DELIMITER + path;
+          const element = resolverFn(fullPath);
+          if (element) {
+            results.set(fullPath, {
+              element,
+              resolvedUsingSectionImports: true,
+            });
+          }
+        } catch {
+          // do nothing
+        }
+      });
+    }
     switch (results.size) {
       /**
        * NOTE: if nothing is found then we will try to find user-defined elements at root package (i.e. no package)
@@ -468,7 +527,7 @@ export class V1_GraphBuilderContextBuilder {
   currentSubGraph: BasicModel;
   extensions: V1_GraphBuilderExtensions;
   graph: PureModel;
-  imports: Package[] = [];
+  sectionImports: Package[] = [];
   section?: Section | undefined;
   options?: GraphBuilderOptions | undefined;
 
@@ -487,19 +546,18 @@ export class V1_GraphBuilderContextBuilder {
   }
 
   withElement(element: V1_PackageableElement): V1_GraphBuilderContextBuilder {
-    const section = this.graph.getOwnSection(element.path);
+    const section = this.graph.getOwnNullableSection(element.path);
     return this.withSection(section);
   }
 
   withSection(section: Section | undefined): V1_GraphBuilderContextBuilder {
     this.section = section;
-    // NOTE: we add auto-imports regardless the type of the section or whether if there is any section at all
-    // so system elements will always be resolved no matter what.
-    this.imports = this.graph.sectionAutoImports;
     if (section instanceof ImportAwareCodeSection) {
-      this.imports = this.imports.concat(section.imports.map((i) => i.value));
+      this.sectionImports = this.sectionImports.concat(
+        section.imports.map((i) => i.value),
+      );
     }
-    this.imports = uniq(this.imports); // remove duplicates
+    this.sectionImports = uniq(this.sectionImports); // remove duplicates
     return this;
   }
 

package/src/models/protocols/pure/v1/transformation/pureGraph/to/V1_ProtocolToMetaModelClassMappingFirstPassBuilder.ts

@@ -44,7 +44,7 @@ import { V1_getInferredClassMappingId } from '../../../transformation/pureGraph/
 import { AggregationAwareSetImplementation } from '../../../../../../metamodels/pure/packageableElements/mapping/aggregationAware/AggregationAwareSetImplementation';
 import type { InstanceSetImplementation } from '../../../../../../metamodels/pure/packageableElements/mapping/InstanceSetImplementation';
 import { V1_buildAggregateContainer } from './helpers/V1_AggregationAwareClassMappingBuilderHelper';
-import { V1_resolvePathsInRawLambda } from './helpers/V1_ValueSpecificationPathResolver';
+import { V1_buildRawLambdaWithResolvedPaths } from './helpers/V1_ValueSpecificationPathResolver';
 import { V1_buildRelationalMappingFilter } from './helpers/V1_RelationalClassMappingBuilderHelper';
 import { toOptionalPackageableElementReference } from '../../../../../../metamodels/pure/packageableElements/PackageableElementReference';
 import type { DSLMapping_PureProtocolProcessorPlugin_Extension } from '../../../../DSLMapping_PureProtocolProcessorPlugin_Extension';
@@ -132,10 +132,10 @@ export class V1_ProtocolToMetaModelClassMappingFirstPassBuilder
       targetClass,
       InferableMappingElementRootExplicitValue.create(classMapping.root),
       getClassMappingOperationType(classMapping.operation),
-      V1_resolvePathsInRawLambda(
-        this.context,
+      V1_buildRawLambdaWithResolvedPaths(
         classMapping.validationFunction.parameters,
         classMapping.validationFunction.body,
+        this.context,
       ),
     );
   }
@@ -163,7 +163,11 @@ export class V1_ProtocolToMetaModelClassMappingFirstPassBuilder
       toOptionalPackageableElementReference(srcClassReference),
     );
     pureInstanceSetImplementation.filter = classMapping.filter
-      ? V1_resolvePathsInRawLambda(this.context, [], classMapping.filter.body)
+      ? V1_buildRawLambdaWithResolvedPaths(
+          [],
+          classMapping.filter.body,
+          this.context,
+        )
       : undefined;
     return pureInstanceSetImplementation;
   }
@@ -191,7 +195,11 @@ export class V1_ProtocolToMetaModelClassMappingFirstPassBuilder
         sourceRootRecordType,
       );
     flatDataInstanceSetImplementation.filter = classMapping.filter
-      ? V1_resolvePathsInRawLambda(this.context, [], classMapping.filter.body)
+      ? V1_buildRawLambdaWithResolvedPaths(
+          [],
+          classMapping.filter.body,
+          this.context,
+        )
       : undefined;
     return flatDataInstanceSetImplementation;
   }

package/src/models/protocols/pure/v1/transformation/pureGraph/to/V1_ProtocolToMetaModelGraphFifthPassBuilder.ts

@@ -15,7 +15,10 @@
  */
 
 import { UnsupportedOperationError } from '@finos/legend-shared';
-import type { V1_GraphBuilderContext } from './V1_GraphBuilderContext';
+import {
+  V1_buildFullPath,
+  type V1_GraphBuilderContext,
+} from './V1_GraphBuilderContext';
 import type {
   V1_PackageableElement,
   V1_PackageableElementVisitor,
@@ -69,8 +72,8 @@ export class V1_ProtocolToMetaModelGraphFifthPassBuilder
   }
 
   visit_Class(element: V1_Class): void {
-    const _class = this.context.graph.getClass(
-      this.context.graph.buildPath(element.package, element.name),
+    const _class = this.context.currentSubGraph.getOwnClass(
+      V1_buildFullPath(element.package, element.name),
     );
     _class.derivedProperties = element.derivedProperties.map(
       (derivedProperty) =>
@@ -92,14 +95,12 @@ export class V1_ProtocolToMetaModelGraphFifthPassBuilder
   }
 
   visit_FlatData(element: V1_FlatData): void {
-    this.context.graph.getFlatDataStore(
-      this.context.graph.buildPath(element.package, element.name),
-    );
+    return;
   }
 
   visit_Database(element: V1_Database): void {
-    const database = this.context.graph.getDatabase(
-      this.context.graph.buildPath(element.package, element.name),
+    const database = this.context.currentSubGraph.getOwnDatabase(
+      V1_buildFullPath(element.package, element.name),
     );
     element.schemas.forEach((schema) =>
       V1_buildDatabaseSchemaViewsSecondPass(schema, this.context, database),