@hubspot/project-parsing-lib 0.1.0 → 0.1.1-beta.0

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@hubspot/project-parsing-lib",
-    "version": "0.1.0",
+    "version": "0.1.1-beta.0",
     "description": "Parsing library for converting projects directory structures to their intermediate representation",
     "license": "Apache-2.0",
     "main": "src/index.js",

package/src/index.js

@@ -21,7 +21,7 @@ async function translate(translationContext, translationOptions = defaultOptions
         throw new Error(copy_1.errorMessages.project.noHsMetaFiles);
     }
     const transformation = (0, transform_1.transform)(metafileContents);
-    const intermediateRepresentation = (0, transform_1.getIntermediateRepresentation)(transformation);
+    const intermediateRepresentation = (0, transform_1.getIntermediateRepresentation)(transformation, skipValidation);
     // Remove once extensions and serverless functions are supported
     if (!skipValidation) {
         await (0, validation_1.validateIntermediateRepresentation)(intermediateRepresentation, transformation, translationContext);

package/src/lib/errors.js

@@ -31,11 +31,13 @@ class TranslationError extends Error {
     toString() {
         const listOfErrors = this.errors.map(({ file, errors }) => {
             if (errors.length === 0) {
-                return '';
+                return null;
             }
             return copy_1.errorMessages.validation.errorWithFileHeader(path_1.default.join(this.translationContext.projectSourceDir, file), errors);
         });
-        return `${this.message}: ${listOfErrors}`;
+        return `${this.message}: ${listOfErrors
+            .filter(error => error !== null)
+            .join('')}`;
     }
 }
 exports.TranslationError = TranslationError;

package/src/lib/transform.d.ts

@@ -1,4 +1,4 @@
 import { FileParseResult, IntermediateRepresentation, Transformation } from './types';
 export declare function mapToUserFacingType(type: string): string;
 export declare function transform(fileParseResults: FileParseResult[]): Transformation[];
-export declare function getIntermediateRepresentation(transformations: Transformation[]): IntermediateRepresentation;
+export declare function getIntermediateRepresentation(transformations: Transformation[], skipValidation: boolean | undefined): IntermediateRepresentation;

package/src/lib/transform.js

@@ -65,18 +65,27 @@ function transform(fileParseResults) {
         };
     });
 }
-function getIntermediateRepresentation(transformations) {
+function getIntermediateRepresentation(transformations, skipValidation) {
     const nodes = transformations.reduce((acc, current) => {
         if (!current.intermediateRepresentation) {
             return acc;
         }
         const { uid } = current.intermediateRepresentation;
-        if (acc[uid]) {
+        if (uid && acc[uid]) {
             const duplicates = transformations
                 .filter(t => t.intermediateRepresentation?.uid === uid)
                 .map(t => t.fileParseResult.file);
             throw new Error(copy_1.errorMessages.project.duplicateUid(uid, duplicates));
         }
+        if (!skipValidation) {
+            return {
+                ...acc,
+                // If the uid is not defined just make one up for the sake of indexing.
+                // It will still fail validation, but this prevents collisions so we validate all files.
+                [uid ||
+                    `missing-${current.fileParseResult.file}`]: current.intermediateRepresentation,
+            };
+        }
         return {
             ...acc,
             [uid]: current.intermediateRepresentation,