@hubspot/ui-extensions-dev-server 0.8.47 → 0.8.50

package/dist/lib/DevModeParentInterface.js

@@ -44,7 +44,10 @@ class DevModeParentInterface {
                 return constants_1.PLATFORM_VERSION.V20231;
             case constants_1.PLATFORM_VERSION.V20232:
                 return constants_1.PLATFORM_VERSION.V20232;
+            case constants_1.PLATFORM_VERSION.V20251:
+                return constants_1.PLATFORM_VERSION.V20251;
             case constants_1.PLATFORM_VERSION.V20252:
+            case constants_1.PLATFORM_VERSION.UNSTABLE:
                 return constants_1.PLATFORM_VERSION.V20252;
             default:
                 return (0, utils_1.throwUnhandledPlatformVersionError)(platformVersion);

package/dist/lib/ast.d.ts

@@ -8,5 +8,8 @@ export declare function checkForOutOfBoundsImageImports(node: Node, output: Sour
 export declare function traverseAbstractSyntaxTree(ast: Program, checks: SourceCodeChecks, extensionPath: string, logger: Logger): {
     functions: {};
     badImports: never[];
-    dataDependencies: never[];
+    dataDependencies: {
+        importedHooks: {};
+        dependencies: never[];
+    };
 };

package/dist/lib/ast.js

@@ -9,6 +9,7 @@ const path_1 = __importDefault(require("path"));
 // @ts-expect-error no type defs
 const estraverse_1 = require("estraverse");
 const utils_1 = require("./utils");
+const EXPERIMENTAL_HOOKS = ['useCrmProperties'];
 function _isVariableImported(node, variableName) {
     if (!node) {
         return false;
@@ -84,40 +85,93 @@ function _collectDataDependencies(node, output, logger) {
         return;
     }
     try {
-        // For now we only support `useCrmProperties`, but we can generalize this in the future.
-        if (node.type === 'CallExpression' &&
-            node.callee.type === 'Identifier' &&
-            node.callee.name === 'useCrmProperties') {
-            const propertyType = 'CrmRecordProperties';
-            // Get the second argument, the properties array.
-            const propertiesNode = node.arguments[1];
-            const requestedProperties = [];
-            // If the second argument is an array with at least one element, we will collect the properties.
-            if (propertiesNode &&
-                propertiesNode.type === 'ArrayExpression' &&
-                propertiesNode.elements.length > 0) {
-                propertiesNode.elements.forEach((element) => {
-                    /**
-                     * We only support strings for now, and ignore the rest.
-                     * Again, this might be more generalized in the future as we support more hooks.
-                     */
-                    if (element &&
-                        element.type === 'Literal' &&
-                        typeof element.value === 'string') {
-                        requestedProperties.push(element.value);
+        // Check for imports of our hooks.
+        if (node.type === 'ImportDeclaration' &&
+            typeof node.source.value === 'string' &&
+            node.source.value.startsWith('@hubspot/ui-extensions')) {
+            const isExperimental = node.source.value === '@hubspot/ui-extensions/experimental';
+            // If the imports are coming from our own package, loop over them to check for hooks
+            node.specifiers.forEach((specifier) => {
+                // If the specifier is an ImportSpecifier and the imported name is one of our tracked hooks, we will track it.
+                if (specifier.type === 'ImportSpecifier' &&
+                    isExperimental &&
+                    EXPERIMENTAL_HOOKS.includes(specifier.imported.name)) {
+                    // The local name is the name the hook is imported as in the file, and the imported name is the original name of the hook.
+                    output.dataDependencies.importedHooks[`${specifier.local.name}`] =
+                        specifier.imported.name;
+                }
+                else if (
+                // We also have to track namespace level imports
+                specifier.type === 'ImportNamespaceSpecifier') {
+                    // If the specifier is a namespace import, we will track all hooks that are imported from that namespace. Experimental and stable hooks are handled separately.
+                    if (isExperimental) {
+                        EXPERIMENTAL_HOOKS.forEach((hook) => {
+                            output.dataDependencies.importedHooks[`${specifier.local.name}.${hook}`] = hook;
+                        });
                     }
-                });
-                output.dataDependencies.push({
-                    /**
-                     * This refID is a hash of the property type and the requested properties.
-                     * This should allow us to create the same hash at execution time, to find the correct data from BE.
-                     */
-                    referenceId: (0, utils_1.generateHash)(propertyType, requestedProperties),
-                    properties: {
-                        type: propertyType,
-                        recordProperties: requestedProperties,
-                    },
-                });
+                }
+            });
+            // Check for calls to our hooks.
+        }
+        else if (node.type === 'CallExpression') {
+            let hookName = '';
+            // This handles the case where the hook is called directly, eg `useCrmProperties()` (also supports aliased hooks)
+            if (node.callee.type === 'Identifier' &&
+                output.dataDependencies.importedHooks[node.callee.name]) {
+                hookName = output.dataDependencies.importedHooks[node.callee.name];
+            }
+            else if (
+            // This handles namespace import usage, eg `uiExtensions.useCrmProperties()`
+            node.callee.type === 'MemberExpression' &&
+                node.callee.object.type === 'Identifier' &&
+                node.callee.property.type === 'Identifier' &&
+                output.dataDependencies.importedHooks[`${node.callee.object.name}.${node.callee.property.name}`]) {
+                hookName =
+                    output.dataDependencies.importedHooks[`${node.callee.object.name}.${node.callee.property.name}`];
+            }
+            // Then we handle each hook individually, as the usages and tracking format are different.
+            if (hookName === 'useCrmProperties') {
+                const propertyType = 'CrmRecordProperties';
+                // Validate actions parameter
+                if (!node.arguments[0]) {
+                    logger.warn('useCrmProperties called without actions parameter');
+                    return;
+                }
+                // Get the second argument, the properties array
+                const propertiesNode = node.arguments[1];
+                const requestedProperties = [];
+                // If the second argument is an array with at least one element, collect the properties.
+                if (propertiesNode &&
+                    propertiesNode.type === 'ArrayExpression' &&
+                    propertiesNode.elements.length > 0) {
+                    propertiesNode.elements.forEach((element) => {
+                        /**
+                         * We only support strings for now, and ignore the rest.
+                         * This might be more generalized in the future as we support more hooks.
+                         */
+                        if (element &&
+                            element.type === 'Literal' &&
+                            typeof element.value === 'string') {
+                            requestedProperties.push(element.value);
+                        }
+                        else {
+                            logger.warn(`Invalid property type in useCrmProperties: ${element ? element.type : 'undefined'}`);
+                        }
+                    });
+                    if (requestedProperties.length > 0) {
+                        output.dataDependencies.dependencies.push({
+                            /**
+                             * This refID is a hash of the property type and the requested properties.
+                             * This should allow us to create the same hash at execution time, to find the correct data from BE.
+                             */
+                            referenceId: (0, utils_1.generateHash)(propertyType, requestedProperties),
+                            properties: {
+                                type: propertyType,
+                                recordProperties: requestedProperties,
+                            },
+                        });
+                    }
+                }
             }
         }
     }
@@ -131,7 +185,10 @@ function traverseAbstractSyntaxTree(ast, checks, extensionPath, logger) {
     const state = {
         functions: {},
         badImports: [],
-        dataDependencies: [],
+        dataDependencies: {
+            importedHooks: {},
+            dependencies: [],
+        },
     };
     try {
         (0, estraverse_1.traverse)(ast, {

package/dist/lib/constants.d.ts

@@ -21,7 +21,9 @@ export declare const SERVER_CAPABILITIES: string[];
 export declare const PLATFORM_VERSION: {
     readonly V20231: "2023.1";
     readonly V20232: "2023.2";
+    readonly V20251: "2025.1";
     readonly V20252: "2025.2";
+    readonly UNSTABLE: "unstable";
 };
 export declare const PUBLIC_APP = "public-app";
 export declare const PRIVATE_APP = "private-app";

package/dist/lib/constants.js

@@ -31,7 +31,9 @@ exports.SERVER_CAPABILITIES = [
 exports.PLATFORM_VERSION = {
     V20231: '2023.1',
     V20232: '2023.2',
+    V20251: '2025.1',
     V20252: '2025.2',
+    UNSTABLE: 'unstable',
 };
 exports.PUBLIC_APP = 'public-app';
 // You gotta be kidding me linter

package/dist/lib/plugins/codeBlockingPlugin.js

@@ -14,7 +14,10 @@ const codeBlockingPlugin = ({ logger, extensionPath }) => {
             let sourceCodeMetadata = {
                 functions: {},
                 badImports: [],
-                dataDependencies: [],
+                dataDependencies: {
+                    importedHooks: {},
+                    dependencies: [],
+                },
             };
             const requireFunctionName = 'require';
             try {

package/dist/lib/plugins/manifestPlugin.js

@@ -34,7 +34,7 @@ const manifestPlugin = (options) => {
                 const sourceCodeMetadata = (0, ast_1.traverseAbstractSyntaxTree)(ast, [], options.extensionPath || process.cwd(), logger);
                 allDataDependencies = [
                     ...allDataDependencies,
-                    ...sourceCodeMetadata.dataDependencies,
+                    ...sourceCodeMetadata.dataDependencies.dependencies,
                 ];
             }
             catch (e) {

package/dist/lib/types.d.ts

@@ -209,7 +209,12 @@ export interface SourceCodeMetadata {
         [functionName: string]: FunctionMetadata;
     };
     badImports: string[];
-    dataDependencies: DataDependency[];
+    dataDependencies: {
+        importedHooks: {
+            [hookName: string]: string;
+        };
+        dependencies: DataDependency[];
+    };
 }
 export interface FunctionInvocationCheck {
     functionName: string;

package/package.json

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