redscript-mc 1.2.15 → 1.2.16

package/dist/cli.js

@@ -48,6 +48,7 @@ const cmdblock_1 = require("./codegen/cmdblock");
 const structure_1 = require("./codegen/structure");
 const diagnostics_1 = require("./diagnostics");
 const repl_1 = require("./repl");
+const metadata_1 = require("./builtins/metadata");
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 // Parse command line arguments
@@ -61,16 +62,18 @@ Usage:
   redscript watch <dir> [-o <outdir>] [--namespace <ns>] [--hot-reload <url>]
   redscript check <file>
   redscript fmt <file.mcrs> [file2.mcrs ...]
+  redscript generate-dts [-o <file>]
   redscript repl
   redscript version
 
 Commands:
-  compile   Compile a RedScript file to a Minecraft datapack
-  watch     Watch a directory for .mcrs file changes, recompile, and hot reload
-  check     Check a RedScript file for errors without generating output
-  fmt       Auto-format RedScript source files
-  repl      Start an interactive RedScript REPL
-  version   Print the RedScript version
+  compile       Compile a RedScript file to a Minecraft datapack
+  watch         Watch a directory for .mcrs file changes, recompile, and hot reload
+  check         Check a RedScript file for errors without generating output
+  fmt           Auto-format RedScript source files
+  generate-dts  Generate builtin function declaration file (builtins.d.mcrs)
+  repl          Start an interactive RedScript REPL
+  version       Print the RedScript version
 
 Options:
   -o, --output <path>    Output directory or file path, depending on target
@@ -414,6 +417,13 @@ async function main() {
             }
             break;
         }
+        case 'generate-dts': {
+            const output = parsed.output ?? 'builtins.d.mcrs';
+            const dtsContent = (0, metadata_1.generateDts)();
+            fs.writeFileSync(output, dtsContent, 'utf-8');
+            console.log(`Generated ${output}`);
+            break;
+        }
         case 'repl':
             await (0, repl_1.startRepl)(parsed.namespace ?? 'repl');
             break;

package/dist/lowering/index.d.ts

@@ -151,6 +151,17 @@ export declare class Lowering {
     private getArrayStorageName;
     private inferLambdaReturnType;
     private inferExprType;
+    /**
+     * Checks a raw() command string for `${...}` interpolation containing runtime variables.
+     * - If the interpolated expression is a numeric literal → OK (MC macro syntax).
+     * - If the interpolated name is a compile-time constant (in constValues) → OK.
+     * - If the interpolated name is a known runtime variable (in varMap) → DiagnosticError.
+     * - Unknown names → OK (could be MC macro params or external constants).
+     *
+     * This catches the common mistake of writing raw("say ${score}") expecting interpolation,
+     * which would silently emit a literal `${score}` in the MC command.
+     */
+    private checkRawCommandInterpolation;
     private resolveInstanceMethod;
     private normalizeType;
     private readArrayElement;

package/dist/lowering/index.js

@@ -704,6 +704,7 @@ class Lowering {
                 this.lowerExecuteStmt(stmt);
                 break;
             case 'raw':
+                this.checkRawCommandInterpolation(stmt.cmd, stmt.span);
                 this.builder.emitRaw(stmt.cmd);
                 break;
         }
@@ -2830,6 +2831,39 @@ class Lowering {
         }
         return undefined;
     }
+    /**
+     * Checks a raw() command string for `${...}` interpolation containing runtime variables.
+     * - If the interpolated expression is a numeric literal → OK (MC macro syntax).
+     * - If the interpolated name is a compile-time constant (in constValues) → OK.
+     * - If the interpolated name is a known runtime variable (in varMap) → DiagnosticError.
+     * - Unknown names → OK (could be MC macro params or external constants).
+     *
+     * This catches the common mistake of writing raw("say ${score}") expecting interpolation,
+     * which would silently emit a literal `${score}` in the MC command.
+     */
+    checkRawCommandInterpolation(cmd, span) {
+        const interpRe = /\$\{([^}]+)\}/g;
+        let match;
+        while ((match = interpRe.exec(cmd)) !== null) {
+            const name = match[1].trim();
+            // Numeric/boolean literals are fine (intentional MC macro syntax)
+            if (/^\d+(\.\d+)?$/.test(name) || name === 'true' || name === 'false') {
+                continue;
+            }
+            // Compile-time constants are fine
+            if (this.constValues.has(name)) {
+                continue;
+            }
+            // Only error if it's a known runtime variable (in varMap or function params)
+            // Unknown identifiers are left alone (could be MC macro params the user intends)
+            if (this.varMap.has(name) || this.currentFnParamNames.has(name)) {
+                const loc = span ?? { line: 1, col: 1 };
+                throw new diagnostics_1.DiagnosticError('LoweringError', `raw() command contains runtime variable interpolation '\${${name}}'. ` +
+                    `Variables cannot be interpolated into raw commands at compile time. ` +
+                    `Use f-string messages (say/tell/announce) or MC macro syntax '$(${name})' for MC 1.20.2+ commands.`, loc);
+            }
+        }
+    }
     resolveInstanceMethod(expr) {
         const receiver = expr.args[0];
         if (!receiver) {