@malloydata/malloy 0.0.324 → 0.0.326

package/CONTEXT.md

@@ -0,0 +1,57 @@
+# Malloy Core Package
+
+The `malloy` package is the heart of the Malloy language implementation. It contains the compiler, translator, and runtime system that powers Malloy's semantic modeling and query capabilities.
+
+## Package Structure
+
+```
+packages/malloy/
+├── src/
+│   ├── lang/              # Translator: Parse tree → AST → IR (see src/lang/CONTEXT.md)
+│   ├── model/             # Compiler: IR → SQL (see src/model/CONTEXT.md)
+│   ├── dialect/           # Database-specific SQL generation
+│   ├── api/               # Public API interfaces
+│   ├── connection/        # Database connection abstractions
+│   └── malloy.ts          # Main entry point
+```
+
+## Two-Phase Architecture
+
+The Malloy compilation process is split into two distinct phases:
+
+### Phase 1: Translation (src/lang/)
+The translator takes Malloy source code and transforms it into an Intermediate Representation (IR).
+
+**Process:**
+1. ANTLR parser generates parse tree from source code
+2. Parse tree is transformed into Abstract Syntax Tree (AST)
+3. AST is analyzed and transformed into IR
+
+**Key characteristics:**
+- IR is a **serializable data format** (plain objects, not class instances)
+- IR fully describes the semantic model independent of SQL
+- IR can be cached, transmitted, and reused across compilations
+
+For detailed information about the translator, see [src/lang/CONTEXT.md](src/lang/CONTEXT.md).
+
+### Phase 2: Compilation (src/model/)
+The compiler takes IR and generates SQL queries for specific database dialects.
+
+**Process:**
+1. IR is read and analyzed
+2. Query operations are transformed into SQL expressions
+3. Dialect-specific SQL is generated
+4. Metadata is generated for result processing
+
+**Key characteristics:**
+- Produces SQL that can be executed on target database
+- Includes metadata to interpret and render results
+- Dialect-agnostic until final SQL generation step
+
+For detailed information about the compiler, see [src/model/CONTEXT.md](src/model/CONTEXT.md).
+
+## Subsystem Context
+
+For deeper details on specific subsystems:
+- [src/lang/CONTEXT.md](src/lang/CONTEXT.md) - Translator architecture (grammar, AST, IR generation)
+- [src/model/CONTEXT.md](src/model/CONTEXT.md) - Compiler architecture (SQL generation, expression compilation)

package/dist/connection/base_connection.js

@@ -19,19 +19,21 @@ class BaseConnection {
             try {
                 const cacheResponse = await fillCache();
                 if (typeof cacheResponse === 'string') {
-                    cached = { error: cacheResponse };
+                    // Don't cache errors - just return them
+                    return { error: cacheResponse };
                 }
                 else {
                     cached = {
                         schema: cacheResponse,
                         timestamp: refreshTimestamp !== null && refreshTimestamp !== void 0 ? refreshTimestamp : Date.now(),
                     };
+                    this.schemaCache[schemaKey] = cached;
                 }
             }
             catch (uncaught) {
-                cached = { error: uncaught.message };
+                // Don't cache errors - just return them
+                return { error: uncaught.message };
             }
-            this.schemaCache[schemaKey] = cached;
         }
         if (cached.error) {
             return cached;

package/dist/lang/ast/source-properties/renames.js

@@ -43,6 +43,11 @@ class RenameField extends malloy_element_1.MalloyElement {
             this.logError('invalid-rename-with-same-name', "Can't rename field to itself");
             return;
         }
+        // Check if the new name would shadow an existing field
+        if (fs.entry(this.newName)) {
+            this.logError('invalid-rename-with-same-name', `Can't rename to '${this.newName}', field already exists`);
+            return;
+        }
         const oldValue = this.oldName.getField(fs);
         /*
     

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const MALLOY_VERSION = "0.0.324";
+export declare const MALLOY_VERSION = "0.0.326";

package/dist/version.js

@@ -2,5 +2,5 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.MALLOY_VERSION = void 0;
 // generated with 'generate-version-file' script; do not edit manually
-exports.MALLOY_VERSION = '0.0.324';
+exports.MALLOY_VERSION = '0.0.326';
 //# sourceMappingURL=version.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@malloydata/malloy",
-  "version": "0.0.324",
+  "version": "0.0.326",
   "license": "MIT",
   "exports": {
     ".": "./dist/index.js",
@@ -41,9 +41,9 @@
     "generate-version-file": "VERSION=$(npm pkg get version --workspaces=false | tr -d \\\")\necho \"// generated with 'generate-version-file' script; do not edit manually\\nexport const MALLOY_VERSION = '$VERSION';\" > src/version.ts"
   },
   "dependencies": {
-    "@malloydata/malloy-filter": "0.0.324",
-    "@malloydata/malloy-interfaces": "0.0.324",
-    "@malloydata/malloy-tag": "0.0.324",
+    "@malloydata/malloy-filter": "0.0.326",
+    "@malloydata/malloy-interfaces": "0.0.326",
+    "@malloydata/malloy-tag": "0.0.326",
     "antlr4ts": "^0.5.0-alpha.4",
     "assert": "^2.0.0",
     "jaro-winkler": "^0.2.8",