@lowdefy/build 0.0.0-experimental-20260113090459 → 0.0.0-experimental-20260114142524

package/dist/build/buildPages/validateStateReferences.js

@@ -16,9 +16,10 @@
 import formatConfigWarning from '../../utils/formatConfigWarning.js';
 import traverseConfig from '../../utils/traverseConfig.js';
 function validateStateReferences({ page, context }) {
-    // Single traversal collects both blockIds and _state references
+    // Single traversal collects blockIds, _state references, and SetState keys
     // More memory-efficient than stringify+regex for massive pages
     const blockIds = new Set();
+    const setStateKeys = new Set();
     const stateRefs = new Map(); // topLevelKey -> configKey (first occurrence)
     traverseConfig({
         config: page,
@@ -27,6 +28,13 @@ function validateStateReferences({ page, context }) {
             if (obj.blockId) {
                 blockIds.add(obj.blockId);
             }
+            // Collect SetState action params to track state keys being initialized
+            if (obj.type === 'SetState' && obj.params) {
+                Object.keys(obj.params).forEach((key)=>{
+                    const topLevelKey = key.split(/[.\[]/)[0];
+                    setStateKeys.add(topLevelKey);
+                });
+            }
             // Collect _state reference if present
             if (obj._state !== undefined) {
                 const topLevelKey = extractOperatorKey({
@@ -40,7 +48,8 @@ function validateStateReferences({ page, context }) {
     });
     // Filter to only undefined references and warn
     stateRefs.forEach((configKey, topLevelKey)=>{
-        if (blockIds.has(topLevelKey)) return;
+        // Skip if state key is from an input block or SetState action
+        if (blockIds.has(topLevelKey) || setStateKeys.has(topLevelKey)) return;
         const message = `_state references "${topLevelKey}" on page "${page.pageId}", ` + `but no input block with id "${topLevelKey}" exists on this page. ` + `State keys are created from input block ids. ` + `Check for typos, add an input block with this id, or initialize the state with SetState.`;
         context.logger.warn(formatConfigWarning({
             message,

package/dist/build/buildRefs/getConfigFile.js

@@ -12,7 +12,8 @@
   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
   See the License for the specific language governing permissions and
   limitations under the License.
-*/ import { type } from '@lowdefy/helpers';
+*/ import path from 'path';
+import { type } from '@lowdefy/helpers';
 async function getConfigFile({ context, refDef, referencedFrom }) {
     if (!type.isString(refDef.path)) {
         throw new Error(`Invalid _ref definition ${JSON.stringify({
@@ -21,8 +22,23 @@ async function getConfigFile({ context, refDef, referencedFrom }) {
     }
     const content = await context.readConfigFile(refDef.path);
     if (content === null) {
+        // Build helpful error message with resolved path information
         const lineInfo = refDef.lineNumber ? `:${refDef.lineNumber}` : '';
-        throw new Error(`Tried to reference file "${refDef.path}" from "${referencedFrom}${lineInfo}", but file does not exist.`);
+        const absolutePath = path.resolve(context.directories.config, refDef.path);
+        let message = `[Config Error] Referenced file does not exist: "${refDef.path}"\n`;
+        message += `  Referenced from: ${referencedFrom}${lineInfo}\n`;
+        message += `  Resolved to: ${absolutePath}\n`;
+        // Help with common mistakes
+        if (refDef.path.startsWith('../')) {
+            const suggestedPath = refDef.path.replace(/^(\.\.\/)+/, '');
+            message += `\n  Tip: Paths in _ref are resolved from your config directory root.`;
+            message += `\n  Did you mean: "${suggestedPath}"?`;
+        } else if (refDef.path.startsWith('./')) {
+            const suggestedPath = refDef.path.substring(2);
+            message += `\n  Tip: Remove the "./" prefix - paths are resolved from config root.`;
+            message += `\n  Did you mean: "${suggestedPath}"?`;
+        }
+        throw new Error(message);
     }
     return content;
 }

package/dist/build/buildRefs/recursiveBuild.js

@@ -14,6 +14,7 @@
   limitations under the License.
 */ import { serializer, type } from '@lowdefy/helpers';
 import evaluateBuildOperators from './evaluateBuildOperators.js';
+import formatConfigError from '../../utils/formatConfigError.js';
 import getKey from './getKey.js';
 import getRefContent from './getRefContent.js';
 import getRefsFromFile from './getRefsFromFile.js';
@@ -21,19 +22,28 @@ import populateRefs from './populateRefs.js';
 import runTransformer from './runTransformer.js';
 async function recursiveBuild({ context, refDef, count, referencedFrom, refChainSet = new Set(), refChainList = [] }) {
     // Detect circular references by tracking the chain of files being resolved
+    // Skip circular reference checking for refs without paths (e.g., resolver refs)
     const currentPath = refDef.path;
-    if (refChainSet.has(currentPath)) {
-        const chainDisplay = [
-            ...refChainList,
-            currentPath
-        ].join(' -> ');
-        throw new Error(`Circular reference detected: ${chainDisplay}`);
+    if (currentPath) {
+        if (refChainSet.has(currentPath)) {
+            const chainDisplay = [
+                ...refChainList,
+                currentPath
+            ].join('\n  -> ');
+            throw new Error(formatConfigError({
+                message: `Circular reference detected.\nFile "${currentPath}" references itself through:\n  -> ${chainDisplay}`,
+                context
+            }));
+        }
+        refChainSet.add(currentPath);
+        refChainList.push(currentPath);
     }
-    refChainSet.add(currentPath);
-    refChainList.push(currentPath);
     // Keep count as a fallback safety limit
     if (count > 10000) {
-        throw new Error(`Maximum recursion depth of references exceeded.`);
+        throw new Error(formatConfigError({
+            message: `Maximum recursion depth of references exceeded (10000 levels). This likely indicates a circular reference.`,
+            context
+        }));
     }
     let fileContent = await getRefContent({
         context,
@@ -97,8 +107,11 @@ async function recursiveBuild({ context, refDef, count, referencedFrom, refChain
         });
     }
     // Backtrack: remove current file from chain so sibling refs can use it
-    refChainSet.delete(currentPath);
-    refChainList.pop();
+    // Only remove if it was added (i.e., if currentPath exists)
+    if (currentPath) {
+        refChainSet.delete(currentPath);
+        refChainList.pop();
+    }
     return populateRefs({
         toPopulate: fileContentBuiltRefs,
         parsedFiles,