@hubspot/ui-extensions-dev-server 0.8.40 → 0.8.41

package/dist/lib/ast.d.ts

@@ -1,5 +1,6 @@
 import { SourceCodeChecks } from './types';
 import { Program } from 'estree';
-export declare function traverseAbstractSyntaxTree(ast: Program, checks: SourceCodeChecks): {
+export declare function traverseAbstractSyntaxTree(ast: Program, checks: SourceCodeChecks, extensionPath: string): {
     functions: {};
+    badImports: never[];
 };

package/dist/lib/ast.js

@@ -1,7 +1,11 @@
 "use strict";
 /* eslint-disable hubspot-dev/no-unsupported-ts-syntax */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.traverseAbstractSyntaxTree = void 0;
+const path_1 = __importDefault(require("path"));
 // @ts-expect-error no type defs
 const estraverse_1 = require("estraverse");
 function _isVariableImported(node, variableName) {
@@ -47,16 +51,35 @@ function _checkForFunctionMetadata(node, parent, output, functionName) {
         output.functions[functionName].defined = true;
     }
 }
+/**
+ * We only support imports that are within the extension directory.
+ * This function will check if an import is out of bounds and collect any that are out of bounds, so we can warn the user before they run into build issues.
+ */
+function _checkForOutOfBoundsImports(node, output, extensionPath) {
+    if (!node) {
+        return;
+    }
+    if (node.type === 'ImportDeclaration' &&
+        typeof node.source.value === 'string') {
+        const importPath = path_1.default.join(extensionPath, node.source.value);
+        const { dir } = path_1.default.parse(importPath);
+        if (!dir.includes(extensionPath)) {
+            output.badImports.push(importPath);
+        }
+    }
+}
 // Traverses an ESTree as defined by the EsTree spec https://github.com/estree/estree
 // Uses the checks array to search the source code for matches
-function traverseAbstractSyntaxTree(ast, checks) {
+function traverseAbstractSyntaxTree(ast, checks, extensionPath) {
     const state = {
         functions: {},
+        badImports: [],
     };
     (0, estraverse_1.traverse)(ast, {
         enter(node, parent) {
             checks.forEach((check) => {
                 _checkForFunctionMetadata(node, parent, state, check.functionName);
+                _checkForOutOfBoundsImports(node, state, extensionPath);
             });
         },
     });

package/dist/lib/build.js

@@ -42,7 +42,7 @@ logLevel = 'info', }) {
                 rollupOptions: Object.assign(Object.assign({}, constants_1.ROLLUP_OPTIONS), { plugins: [
                         (0, manifestPlugin_1.default)({ output, extensionPath: root, logger: console }),
                         (0, friendlyLoggingPlugin_1.default)({ logger: console }),
-                        (0, codeBlockingPlugin_1.default)({ logger: console }),
+                        (0, codeBlockingPlugin_1.default)({ logger: console, extensionPath: root }),
                     ] }),
                 outDir: outputDir,
                 emptyOutDir,

package/dist/lib/plugins/codeBlockingPlugin.d.ts

@@ -1,7 +1,8 @@
 import { Rollup } from 'vite';
 import { Logger } from '../types';
-export type CodeBlockingPlugin = ({ logger, }: {
+export type CodeBlockingPlugin = ({ logger, extensionPath, }: {
     logger: Logger;
+    extensionPath: string;
 }) => Rollup.Plugin;
 declare const codeBlockingPlugin: CodeBlockingPlugin;
 export default codeBlockingPlugin;

package/dist/lib/plugins/codeBlockingPlugin.js

@@ -3,7 +3,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const utils_1 = require("../utils");
 const ast_1 = require("../ast");
-const codeBlockingPlugin = ({ logger }) => {
+const codeBlockingPlugin = ({ logger, extensionPath }) => {
     return {
         name: 'ui-extensions-code-blocking-plugin',
         enforce: 'post',
@@ -11,16 +11,22 @@ const codeBlockingPlugin = ({ logger }) => {
             if ((0, utils_1.isNodeModule)(filename)) {
                 return { code, map: null }; // We don't want to parse node modules
             }
-            let sourceCodeMetadata = { functions: {} };
+            let sourceCodeMetadata = {
+                functions: {},
+                badImports: [],
+            };
             const requireFunctionName = 'require';
             try {
                 //  Not sure why the types don't match for this.parse and the Rollup docs, but
                 // the docs over on rollup's site specify ESTree.Program as the return type,
                 // and the underlying data matches that https://rollupjs.org/plugin-development/#this-parse
                 const abstractSyntaxTree = this.parse(code);
-                sourceCodeMetadata = (0, ast_1.traverseAbstractSyntaxTree)(abstractSyntaxTree, [
-                    { functionName: requireFunctionName },
-                ]);
+                sourceCodeMetadata = (0, ast_1.traverseAbstractSyntaxTree)(abstractSyntaxTree, [{ functionName: requireFunctionName }], extensionPath);
+                if (sourceCodeMetadata.badImports) {
+                    for (const badImport of sourceCodeMetadata.badImports) {
+                        logger.warn(`Importing files from outside of the extension directory is not supported. Please move the import ${badImport} into the extension directory.`);
+                    }
+                }
             }
             catch (e) {
                 logger.debug('Unable to parse and traverse source code');

package/dist/lib/plugins/devBuildPlugin.js

@@ -71,6 +71,7 @@ const devBuildPlugin = (options) => {
     const devBuild = (server, extensionMetadata, emptyOutDir = false) => __awaiter(void 0, void 0, void 0, function* () {
         try {
             const { config: extensionConfig } = extensionMetadata;
+            const { extensionPath } = extensionConfig;
             yield (0, vite_1.build)({
                 logLevel: 'warn',
                 mode: 'development',
@@ -95,7 +96,7 @@ const devBuildPlugin = (options) => {
                             (0, manifestPlugin_1.default)({
                                 minify: false,
                                 output: extensionConfig.output,
-                                extensionPath: extensionConfig.extensionPath,
+                                extensionPath,
                                 logger,
                             }),
                             (0, codeCheckingPlugin_1.default)({
@@ -107,7 +108,7 @@ const devBuildPlugin = (options) => {
                                 output: extensionConfig.output,
                                 logger,
                             }),
-                            (0, codeBlockingPlugin_1.default)({ logger }),
+                            (0, codeBlockingPlugin_1.default)({ logger, extensionPath }),
                         ], output: Object.assign(Object.assign({}, constants_1.ROLLUP_OPTIONS.output), { sourcemap: 'inline' }) }),
                     outDir: devServerState.outputDir,
                     emptyOutDir,

package/dist/lib/types.d.ts

@@ -119,6 +119,7 @@ export interface SourceCodeMetadata {
     functions: {
         [functionName: string]: FunctionMetadata;
     };
+    badImports: string[];
 }
 export interface FunctionInvocationCheck {
     functionName: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hubspot/ui-extensions-dev-server",
-  "version": "0.8.40",
+  "version": "0.8.41",
   "description": "",
   "bin": {
     "uie": "./dist/lib/bin/cli.js"
@@ -27,7 +27,7 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@hubspot/app-functions-dev-server": "0.8.40",
+    "@hubspot/app-functions-dev-server": "0.8.41",
     "chalk": "^5.4.1",
     "commander": "^13.0.0",
     "cors": "^2.8.5",
@@ -69,5 +69,5 @@
       "optional": true
     }
   },
-  "gitHead": "1f4550a901eeba1e56943ef50c6ea7af30f95bc1"
+  "gitHead": "ce6d6332d471d5c5ac127de88a8fa51dd7506c3e"
 }