@scriptc/compiler 0.4.2 → 0.4.3

package/dist/frontend/lowering/lower-stmts.js

@@ -2,7 +2,7 @@ import { lowerTupleAliasDecl, stripTypeWrappers } from "./typed-arguments.js";
 import { declaredMemberOf, isPromisePrototypeRead } from "./expressions/property-access.js";
 import { hostsAll } from "./expressions/hosted-narrowing.js";
 import { requestGeneralStreamSurface } from "./general-streams.js";
-import { lowerForAwaitDynStream } from "./dyn-streams.js";
+import { lowerForAwaitDynStream, nodeStreamInterfaceSide } from "./dyn-streams.js";
 import { lowerForAwaitValueStream } from "./builtins/hash-stream.js";
 import { lowerChildPropertyWrite } from "./handle-properties.js";
 import { matchAllRowsReadByElements, storedMatchAllRowsReadByElements, } from "./containers/match-row-uses.js";
@@ -22,6 +22,7 @@ import { InternalCompilerError } from "../../errors.js";
  * flow (if/while/for/for-of/do, switch, try/catch, abrupt completions),
  * explicit resource management, and blocked-binding poisoning. */
 import * as ts from "../ts7/adapter.js";
+import { lowerOptionalIndex } from "./undefined-index.js";
 import { processDataMemberValue, programReadsExitCode } from "./builtins/process.js";
 import { requireInitFlagStmt } from "./require-initialization.js";
 import { isJsonRequireDecl } from "./json-require.js";
@@ -33,12 +34,13 @@ import { isDeadTscHelperDecl, tscInteropBindingOfDecl } from "./cjs-tsc-interop.
 import { arrayValueRead, arrayValueStore, arrayValueType, unionArrayValueRead, lowerSafeIndexRead, } from "./array-values.js";
 import { lowerForAwaitGenerator, lowerForOfGenerator, lowerYieldStarStatement, } from "./lower-generators.js";
 import { classAsyncGeneratorIterator, lowerForAwaitBuiltin } from "./lower-async-iteration.js";
-import { BOOL, BYTES_U8, CAUGHT, DYN, F64, STRING, UNDEFINED_T, VOID, arrayOf, isDynTypedRefType, isUnitType, shapeHasAccessorSlots, typeEquals, } from "../../ir/ir.js";
+import { BOOL, BYTES_U8, CAUGHT, DYN, F64, SEARCH_PARAMS_T, STRING, UNDEFINED_T, VOID, arrayOf, isDynTypedRefType, isUnitType, shapeHasAccessorSlots, typeEquals, } from "../../ir/ir.js";
 import { PoisonError, boundIdentifiersOf, dynFallbackType, neverTaintedJsType, staticImportNamespaceType, } from "./lowerer.js";
 import { enforceLibBoundary, lowerSurplusCalls } from "./lib-boundary.js";
 import { cjsExportAssignmentOf, cjsExportChainOf, cjsVoidInitPreambleReals, cjsExportDiscardReason, cjsExportTargetLiteral, isCjsJsFile, isEsModuleStamp, isJsSourceFile, locOf, requireEarlyReference, requireSpecOf, } from "../program.js";
 import { isStrictCode } from "./strict-mode.js";
-import { COMPOUND_ASSIGN_OPS, STR_METHODS, UNSUPPORTED_STMT, isStdlibMember, sideEffectFreeOptionValue, stdlibGlobalAliasDecl, stdlibGlobalAliasNameOf, bufferNamespaceMember, stdlibGlobalNameOf, } from "./surfaces.js";
+import { markSloppyRecordWrites } from "./frozen-records.js";
+import { COMPOUND_ASSIGN_OPS, STR_METHODS, NARROW_FIRST, UNSUPPORTED_STMT, isStdlibMember, sideEffectFreeOptionValue, stdlibGlobalAliasDecl, stdlibGlobalAliasNameOf, bufferNamespaceMember, stdlibGlobalNameOf, } from "./surfaces.js";
 import { isNativeBuiltinValueInitializer } from "./lower-builtin-values.js";
 import { isProvenanceSourceFile } from "../provenance-registry.js";
 import { ambientUndefVarRootOf, lowerImportEquals, nsUndefRead, nsWritableTarget, trapDeclRootOf, } from "./lower-namespaces.js";
@@ -59,7 +61,7 @@ import { objectFactorySignature } from "./object-factory-new.js";
 import { isClassCallback, lowerClassCallbackAssign } from "./class-callbacks.js";
 import { classForInHasKey, classPropertiesHelper } from "./class-dynamic-dispatch.js";
 import { genericIfaceBindingKeepsClass, superConditionLead } from "./lower-classes.js";
-import { lowerStreamUnderscoreAssign, streamClassAliasDecl } from "./lower-stream.js";
+import { lowerStreamUnderscoreAssign, streamClassAliasDecl, streamSidesOf, } from "./lower-stream.js";
 import { staticAssertInstanceDecl } from "./assert-instances.js";
 import { lowerHttpResPropertyAssignment, lowerHttpServerTimeoutAssignment, lowerNetServerMaxConnectionsAssignment, lowerServerCloseOverrideAssignment, } from "./lower-server.js";
 import { lowerUrlAssignment } from "./builtins/url.js";
@@ -70,7 +72,7 @@ import { lowerNodeModuleCall } from "./builtins/module-resolution.js";
 import { registerBuiltinCallableAlias } from "./builtins/callable-aliases.js";
 import { builtinClassAliasOwnsNoSlot, registerBuiltinClassAlias } from "./builtins/class-values.js";
 import { lowerEnumDeclaration } from "./lower-enums.js";
-import { abstractPropertyDeclOf, aliasTypeofNarrows, isMatchSliceType, lowerAbsenceProbe, lowerCompoundValueToTarget, lowerElementCompoundStatement, lowerIncDec, lowerIncDecTarget, lowerOptionalNumber, lowerGroupsProjection, matchResultNamedGroupsOf, runtimeOptionalGuardIds, runtimeOptionalTrueIds, symbolFieldInfo, withRuntimeOptionalNarrowed, } from "./lower-exprs.js";
+import { abstractPropertyDeclOf, aliasTypeofNarrows, isMatchSliceType, lowerAbsenceProbe, lowerCompoundValueToTarget, lowerElementCompoundStatement, lowerIncDec, lowerIncDecTarget, lowerGroupsProjection, matchResultNamedGroupsOf, runtimeOptionalGuardIds, runtimeOptionalTrueIds, symbolFieldInfo, withRuntimeOptionalNarrowed, } from "./lower-exprs.js";
 import { globalSymbolKey } from "./expressions/global-symbols.js";
 import { identifiersNamed } from "./identifier-uses.js";
 import { lowerShortCircuitAssignment } from "./expressions/nullish-assignment.js";
@@ -89,6 +91,7 @@ import { npmStaticPackageOfPath } from "../npm-static.js";
 import { fenceNodeModuleMutation } from "./lower-node-module.js";
 import { fencePerfEntryKeyWalk } from "./builtins/performance.js";
 import { classGeneratorIterable } from "./class-iterables.js";
+import { iterableUnionOf, lowerForOfIterable } from "./iterable-values.js";
 /** An opted package's directly-returned empty array may take the function's
  * explicit static return element type. This is the common JS accumulator
  * shape (`const result = []; result.push(value); return result`) and is
@@ -808,6 +811,46 @@ export function predeclareForwardCapture(lowerer, symbol) {
     }
     return false;
 }
+/** The CLASS-DECLARATION twin of predeclareForwardCapture: a function
+ * hoisted above a local `class D` (`function make() { return new D(); }`
+ * reached through an earlier function) captures the class binding, which
+ * is in its temporal dead zone until the class statement runs. The class
+ * cannot be collected yet (its heritage may name bindings declared between
+ * here and the statement), so the binding is pre-declared as a TDZ box
+ * holding the class as a checked-dynamic constructor; the class statement
+ * later initializes it, and every reference reads it from there. */
+export function predeclareForwardClass(lowerer, symbol) {
+    const decl = symbol.valueDeclaration;
+    if (!decl || !ts.isClassDeclaration(decl) || decl.name === undefined)
+        return false;
+    if (!isJsSourceFile(ts.getSourceFileOfNode(decl)))
+        return false;
+    for (let i = lowerer.activeStmtLists.length - 1; i >= 0; i--) {
+        const entry = lowerer.activeStmtLists[i];
+        const idx = entry.stmts.indexOf(decl);
+        if (idx < 0)
+            continue;
+        if (idx < entry.index || entry.ctx === lowerer.ctx || entry.ctx.tdzPredeclared.has(symbol))
+            return false;
+        const name = decl.name.text;
+        const count = entry.ctx.localCounters.get(name) ?? 0;
+        entry.ctx.localCounters.set(name, count + 1);
+        const local = {
+            id: `${name}.${count}`,
+            name,
+            type: DYN,
+            mutable: false,
+            boxed: true,
+            tdz: true,
+        };
+        entry.ctx.locals.push(local);
+        entry.frame.set(symbol, local);
+        entry.out.push({ kind: "varDecl", localId: local.id, init: null, loc: locOf(decl) });
+        entry.ctx.tdzPredeclared.set(symbol, local);
+        return true;
+    }
+    return false;
+}
 /** The FUNCTION-DECLARATION twin of predeclareForwardCapture: JS hoists a
  * nested `function f() {}` to scope entry — the binding is LIVE from the
  * first statement, so a reference lexically above the declaration in the
@@ -999,6 +1042,32 @@ function lowerJsVarFromExisting(lowerer, decl, symbol) {
         loc: locOf(decl),
     };
 }
+/** Whether a JavaScript declaration's object value changes shape after its
+ * initializer (jsOpenObjectType, jsObjectBindingExpands) or carries
+ * methods, so only the checked-dynamic representation holds it.
+ * `inferred` is the declaration's mapped checker type. */
+function jsDeclarationExpandsObject(lowerer, decl, inferred) {
+    const arms = inferred?.kind === "union"
+        ? (lowerer.unions.get(inferred.unionId)?.arms ?? [])
+        : inferred
+            ? [inferred]
+            : [];
+    return ((!decl.type &&
+        !hasJsTypeAnnotation(decl) &&
+        inferred !== null &&
+        jsOpenObjectType(decl, inferred, lowerer.shapes, lowerer.unions).kind === "dyn") ||
+        arms.some((arm) => arm.kind === "record" && jsObjectBindingExpands(lowerer, decl, arm)) ||
+        (isJsSourceFile(ts.getSourceFileOfNode(decl)) &&
+            !decl.type &&
+            !hasJsTypeAnnotation(decl) &&
+            decl.initializer !== undefined &&
+            ts.isObjectLiteralExpression(decl.initializer) &&
+            decl.initializer.properties.nodes.some((property) => ts.isMethodDeclaration(property) ||
+                ts.isAccessor(property) ||
+                (ts.isPropertyAssignment(property) &&
+                    (ts.isFunctionExpression(property.initializer) ||
+                        ts.isArrowFunction(property.initializer))))));
+}
 /** The `var` symbol's function-scoped binding TYPE — the declared type at
  * the binding name, with the JS-source fallbacks every mutable binding
  * takes (an empty-object-literal type is checked-dynamic because tsc
@@ -1030,6 +1099,17 @@ function varBindingType(lowerer, nameNode) {
     }
     if (!type || type.kind === "void")
         return null;
+    // A JavaScript `var` holds whatever its writes store, as a `let` does:
+    // the checker types it by its initializer and checks no later write
+    // (`var fraction = 0;`, later `fraction = match[7].slice(0, 3)`), and an
+    // object it holds may gain members (`mark.snippet = snippet(mark)`).
+    if (type.kind !== "dyn" &&
+        ts.isVariableDeclaration(nameNode.parent) &&
+        nameNode.parent.name === nameNode &&
+        isJsSourceFile(ts.getSourceFileOfNode(nameNode)) &&
+        (jsBindingHasOpenWrites(lowerer, nameNode.parent) ||
+            jsDeclarationExpandsObject(lowerer, nameNode.parent, type)))
+        return DYN;
     // A binding readable before its first store holds undefined until then.
     return lowerer.runtimeOptionalBindingType(nameNode, type);
 }
@@ -1299,8 +1379,24 @@ export function lowerStmt(lowerer, stmt) {
         if (!stmt.name)
             lowerer.unsupported("SC1090", stmt, "anonymous local class declarations");
         const info = lowerer.lowerClassExpressionInfo(stmt);
+        // An earlier hoisted function captured the class: its TDZ box exists.
+        const symbol = lowerer.checker.getSymbolAtLocation(stmt.name);
+        const pre = symbol ? lowerer.ctx.tdzPredeclared.get(symbol) : undefined;
+        if (pre) {
+            lowerer.ctx.tdzPredeclared.delete(symbol);
+            const value = lowerer.coerceToExpected(lowerer.lowerClassExpression(stmt), DYN);
+            return { kind: "assign", localId: pre.id, value, initializes: true, loc: locOf(stmt) };
+        }
         const type = { kind: "classval", className: info.def.name };
         const local = lowerer.declareLocal(stmt.name, stmt.name.text, type, true);
+        // Static members of a function-local class run while the class value is
+        // built and may read the class by name: the class value assigns the
+        // binding before them.
+        if (info.localClass && (info.runtimeStatics || info.runtimePrototypeMembers))
+            return [
+                { kind: "varDecl", localId: local.id, init: null, loc: locOf(stmt) },
+                { kind: "exprStmt", expr: lowerer.lowerClassExpression(stmt, local), loc: locOf(stmt) },
+            ];
         const init = lowerer.lowerClassExpression(stmt);
         return { kind: "varDecl", localId: local.id, init, loc: locOf(stmt) };
     }
@@ -2718,6 +2814,18 @@ export function lowerBindingPattern(lowerer, pattern, srcRef, srcType, isLet, ou
         lowerModuleNamespacePattern(lowerer, pattern, nsSource, isLet, out);
         return;
     }
+    if (srcType.kind === "url") {
+        lowerUrlObjectPattern(lowerer, pattern, srcRef, isLet, out);
+        return;
+    }
+    if (srcType.kind === "union" &&
+        !isJsSourceFile(ts.getSourceFileOfNode(pattern)) &&
+        lowerer.unions
+            .get(srcType.unionId)
+            ?.arms.every((arm) => arm.kind === "record" || arm.kind === "object")) {
+        lowerUnionObjectPattern(lowerer, pattern, srcRef, srcType, isLet, out);
+        return;
+    }
     if (srcType.kind !== "record") {
         lowerer.unsupported("SC1031", pattern, srcType.kind === "object"
             ? "object destructuring of class instances (read the fields directly — accessors make the desugar observable)"
@@ -2900,7 +3008,10 @@ function patternSourceValue(lowerer, value, arrayPattern, out, loc, spelling) {
     if (!arms ||
         !present?.length ||
         present.length === arms.length ||
-        (present.length > 1 && !(arrayPattern && present.every((arm) => arm.kind === "array"))))
+        (present.length > 1 &&
+            !(arrayPattern
+                ? present.every((arm) => arm.kind === "array")
+                : present.every((arm) => arm.kind === "record" || arm.kind === "object"))))
         return value;
     for (const [tag, arm] of arms.entries()) {
         if (!isUnitType(arm))
@@ -2994,6 +3105,122 @@ function lowerModuleNamespacePattern(lowerer, pattern, source, isLet, out) {
         lowerer.bindPatternTarget(el.name, value, isLet, out);
     }
 }
+/** The URL getters an object pattern reads (`const { protocol, hostname }
+ * = new URL(text)`), each a read of the URL's current serialization. */
+const URL_PATTERN_GETTERS = {
+    href: "url.href",
+    origin: "url.origin",
+    protocol: "url.protocol",
+    username: "url.username",
+    password: "url.password",
+    host: "url.host",
+    hostname: "url.hostname",
+    port: "url.port",
+    pathname: "url.pathname",
+    search: "url.search",
+    hash: "url.hash",
+};
+/** An object pattern over a URL: each element reads its getter in pattern
+ * order, as `url.name` does; `searchParams` binds the URL's live view.
+ * Methods and rest elements stay fenced. */
+function lowerUrlObjectPattern(lowerer, pattern, srcRef, isLet, out) {
+    for (const el of pattern.elements.nodes) {
+        if (el.name === undefined)
+            continue;
+        if (el.dotDotDotToken) {
+            lowerer.unsupported("SC1031", el, "rest bindings over URL sources (a URL's properties are prototype getters, which rest elements do not copy)");
+        }
+        const prop = el.propertyName ?? el.name;
+        const propName = ts.isIdentifier(prop) ||
+            ts.isComputedPropertyName(prop) ||
+            ts.isStringLiteralLike(prop) ||
+            ts.isNumericLiteral(prop)
+            ? patternKeyNameOf(lowerer, prop)
+            : null;
+        const loc = locOf(el);
+        let value;
+        if (propName !== null && Object.hasOwn(URL_PATTERN_GETTERS, propName)) {
+            value = {
+                kind: "libCall",
+                fn: URL_PATTERN_GETTERS[propName],
+                args: [srcRef()],
+                type: STRING,
+                loc,
+            };
+        }
+        else if (propName === "searchParams") {
+            value = {
+                kind: "libCall",
+                fn: "url.searchParams",
+                args: [srcRef()],
+                type: SEARCH_PARAMS_T,
+                loc,
+            };
+        }
+        else {
+            lowerer.unsupported("SC1031", el, propName === null
+                ? "destructuring with computed keys that do not fold to one property name"
+                : `destructuring '${propName}' from a URL (${Object.keys(URL_PATTERN_GETTERS).join(", ")} and searchParams are the supported URL properties)`);
+        }
+        if (el.initializer)
+            value = applyBindingDefault(lowerer, el, value);
+        lowerer.bindPatternTarget(el.name, value, isLet, out);
+    }
+}
+/** An object pattern over a union of records and class instances
+ * (`function f({ kind, path }: Add | Remove)`): each element reads a data
+ * field every arm declares with one type, through the runtime tag, as
+ * `value.field` does. Rest elements and fields the arms type differently
+ * stay fenced. */
+function lowerUnionObjectPattern(lowerer, pattern, srcRef, srcType, isLet, out) {
+    const arms = lowerer.unions.get(srcType.unionId)?.arms ?? [];
+    for (const el of pattern.elements.nodes) {
+        if (el.name === undefined)
+            continue;
+        if (el.dotDotDotToken) {
+            lowerer.unsupported("SC1031", el, `rest bindings over union sources (${NARROW_FIRST})`);
+        }
+        const prop = el.propertyName ?? el.name;
+        const propName = ts.isIdentifier(prop) ||
+            ts.isComputedPropertyName(prop) ||
+            ts.isStringLiteralLike(prop) ||
+            ts.isNumericLiteral(prop)
+            ? patternKeyNameOf(lowerer, prop)
+            : null;
+        if (propName === null) {
+            lowerer.unsupported("SC1031", el, "destructuring with computed keys that do not fold to one property name");
+        }
+        let common = null;
+        for (const arm of arms) {
+            let ft;
+            if (arm.kind === "record") {
+                ft = lowerer.shapes.get(arm.shapeId)?.fields.find((f) => f.name === propName)?.type;
+            }
+            else if (arm.kind === "object") {
+                ft = lowerer.classes.get(arm.className)?.fields.get(propName);
+            }
+            if (!ft || ft.kind === "void" || isUnitType(ft) || (common && !typeEquals(common, ft))) {
+                common = null;
+                break;
+            }
+            common = ft;
+        }
+        if (common === null) {
+            lowerer.unsupported("SC1031", el, `destructuring '${propName}' from a union whose arms type it differently (every arm must be a record or class with a same-typed field '${propName}' — ${NARROW_FIRST})`);
+        }
+        let value = {
+            kind: "unionDisc",
+            unionId: srcType.unionId,
+            field: propName,
+            value: srcRef(),
+            type: common,
+            loc: locOf(el),
+        };
+        if (el.initializer)
+            value = applyBindingDefault(lowerer, el, value);
+        lowerer.bindPatternTarget(el.name, value, isLet, out);
+    }
+}
 /** Whether a pattern's first element has a computed key that does not
  * fold to one property name, so the key evaluates at run time. */
 function startsWithRuntimeKey(lowerer, pattern) {
@@ -3881,11 +4108,15 @@ export function bindPatternTarget(lowerer, name, value, isLet, out, allowDynObje
         }
         // Preserve checked-value pattern bindings when the inferred type
         // cannot represent them without a refusal or an alias-breaking copy.
+        // A name destructured from a container of an evolving array
+        // (evolvingArrayHold) may hold the array, which the checker types by
+        // the elements written before it.
         const dynStays = value.type.kind === "dyn" &&
-            (() => {
+            ((() => {
                 const mapped = lowerer.mapTypeOf(lowerer.typeOf(name));
                 return mapped === null || jsBindingKeepsCheckedObject(lowerer, name, mapped);
-            })();
+            })() ||
+                readsHeldEvolvingArray(lowerer, name));
         let type = dynStays
             ? DYN
             : (lowerer.runtimeOptionalBindingType(name, lowerer.irTypeOf(name)) ??
@@ -3969,9 +4200,10 @@ export function lowerVarDeclList(lowerer, list) {
  * `never[]` or `any[]` and each later read by the elements written before
  * it, and reads inside closures as the unsettled array. When every read
  * with a settled element type maps to the same array type, the binding has
- * that type. When reads disagree or the binding is reassigned, it is
- * checked-dynamic, as the same binding is at module scope. Null for other
- * bindings. */
+ * that type. When reads disagree, a write (`xs.push(v)`, `xs.unshift(v)`,
+ * `xs.push(...values)`, `xs[i] = v`) stores a value that type's element does
+ * not hold, or the binding is reassigned, it is checked-dynamic, as the same
+ * binding is at module scope. Null for other bindings. */
 function evolvingArrayType(lowerer, decl) {
     if (!ts.isIdentifier(decl.name) || decl.initializer === undefined)
         return null;
@@ -3999,13 +4231,19 @@ function evolvingArrayType(lowerer, decl) {
     };
     const scope = owner(decl);
     let result = null;
+    // The values the binding's writes store, checked against the element
+    // type once the reads settle it.
+    const written = [];
     // The file's identifiers are indexed by name once (identifiersNamed), so
-    // each binding resolves only the uses spelled like it.
+    // each binding resolves only the uses spelled like it. A shorthand
+    // property (`{ xs }`) is a use too.
     for (const node of identifiersNamed(lowerer, ts.getSourceFileOfNode(decl), decl.name.text)) {
         if (node === decl.name ||
             node.pos < scope.pos ||
             node.end > scope.end ||
-            lowerer.checker.getSymbolAtLocation(node) !== symbol)
+            (ts.isShorthandPropertyAssignment(node.parent) && node.parent.name === node
+                ? lowerer.checker.getShorthandAssignmentValueSymbol(node.parent)
+                : lowerer.checker.getSymbolAtLocation(node)) !== symbol)
             continue;
         let target = node;
         while (ts.isParenthesizedExpression(target.parent))
@@ -4017,6 +4255,18 @@ function evolvingArrayType(lowerer, decl) {
             parent.operatorToken.kind <= ts.SyntaxKind.LastAssignment;
         if (reassigned)
             return DYN;
+        if (ts.isPropertyAccessExpression(parent) &&
+            parent.expression === target &&
+            (parent.name.text === "push" || parent.name.text === "unshift") &&
+            ts.isCallExpression(parent.parent) &&
+            parent.parent.expression === parent)
+            written.push(...parent.parent.arguments.nodes);
+        else if (ts.isElementAccessExpression(parent) &&
+            parent.expression === target &&
+            ts.isBinaryExpression(parent.parent) &&
+            parent.parent.left === parent &&
+            parent.parent.operatorToken.kind === ts.SyntaxKind.EqualsToken)
+            written.push(parent.parent.right);
         const t = lowerer.typeOf(node);
         if (unsettled(t))
             continue;
@@ -4025,7 +4275,132 @@ function evolvingArrayType(lowerer, decl) {
             return DYN;
         result = mapped;
     }
-    return result ?? DYN;
+    if (result === null || result.kind !== "array")
+        return DYN;
+    // The element types a write stores: its value's, or for a spread
+    // (`xs.push(...values)`) the source array's element or tuple's elements.
+    // Another spread source stores elements this check cannot see.
+    const storedTypes = (value) => {
+        if (!ts.isSpreadElement(value))
+            return [lowerer.typeOf(value)];
+        const source = lowerer.typeOf(value.expression);
+        if (lowerer.checker.isArrayType(source) || lowerer.checker.isTupleType(source))
+            return [...lowerer.checker.getTypeArguments(source)];
+        return null;
+    };
+    for (const value of written) {
+        const stored = storedTypes(value);
+        if (stored === null)
+            return DYN;
+        for (const type of stored) {
+            const t = lowerer.mapTypeOf(lowerer.checker.getBaseTypeOfLiteralType(type));
+            if (!t || (!typeEquals(t, result.elem) && lowerer.widthLiftPlan(t, result.elem) === null))
+                return DYN;
+        }
+    }
+    return result;
+}
+/** How a TypeScript expression holds an evolving array binding whose reads
+ * disagree (evolvingArrayType answers checked-dynamic): "evolving" for the
+ * binding itself, "alias" for an unannotated binding initialized with it or
+ * with an alias of it (`const alias = xs`), and "container" for an object or
+ * array literal with it as a property, element or spread source at any
+ * depth (`{ items: xs }`, `[xs]`), an unannotated binding initialized with
+ * or destructured from such a literal, and a property or element read of a
+ * container (`box.items`). The checker types an alias or container by the
+ * elements written before it was taken, but it holds the same array. Null
+ * for other expressions. */
+function evolvingArrayHold(lowerer, expr, depth = 0) {
+    while (ts.isParenthesizedExpression(expr))
+        expr = expr.expression;
+    if (depth >= 8)
+        return null;
+    const binding = (symbol) => {
+        const source = symbol?.valueDeclaration;
+        // A name destructured from a container (`const { items } = box`) may
+        // be the array or another container.
+        if (source !== undefined && ts.isBindingElement(source) && ts.isIdentifier(source.name)) {
+            const host = hostVariableDeclarationOf(source.name);
+            return host !== null &&
+                !host.type &&
+                host.initializer !== undefined &&
+                evolvingArrayHold(lowerer, host.initializer, depth + 1) === "container"
+                ? "container"
+                : null;
+        }
+        if (source === undefined || !ts.isVariableDeclaration(source) || source.type)
+            return null;
+        const evolved = evolvingArrayType(lowerer, source);
+        if (evolved !== null)
+            return evolved.kind === "dyn" ? "evolving" : null;
+        if (source.initializer === undefined)
+            return null;
+        const held = evolvingArrayHold(lowerer, source.initializer, depth + 1);
+        return held === "evolving" ? "alias" : held;
+    };
+    if (ts.isIdentifier(expr))
+        return binding(lowerer.checker.getSymbolAtLocation(expr));
+    if (ts.isPropertyAccessExpression(expr) || ts.isElementAccessExpression(expr))
+        return evolvingArrayHold(lowerer, expr.expression, depth + 1) === "container"
+            ? "container"
+            : null;
+    // A spread of the array itself (`{ ...xs }`, `[...xs]`) copies its
+    // elements, as in Node; a spread of a container copies its references.
+    if (ts.isObjectLiteralExpression(expr)) {
+        for (const property of expr.properties.nodes) {
+            const held = ts.isPropertyAssignment(property)
+                ? evolvingArrayHold(lowerer, property.initializer, depth + 1)
+                : ts.isShorthandPropertyAssignment(property)
+                    ? binding(lowerer.checker.getShorthandAssignmentValueSymbol(property))
+                    : ts.isSpreadAssignment(property) &&
+                        evolvingArrayHold(lowerer, property.expression, depth + 1) === "container"
+                        ? "container"
+                        : null;
+            if (held !== null)
+                return "container";
+        }
+        return null;
+    }
+    if (ts.isArrayLiteralExpression(expr)) {
+        for (const element of expr.elements.nodes) {
+            const held = ts.isSpreadElement(element)
+                ? evolvingArrayHold(lowerer, element.expression, depth + 1) === "container"
+                    ? "container"
+                    : null
+                : evolvingArrayHold(lowerer, element, depth + 1);
+            if (held !== null)
+                return "container";
+        }
+    }
+    return null;
+}
+/** Whether a TypeScript expression is an alias or container of an evolving
+ * array binding whose reads disagree (evolvingArrayHold). Its value is
+ * checked-dynamic, and the checker's type for it lists only the elements
+ * written before it was taken, so element reads through it stay
+ * checked-dynamic instead of checking against that type. */
+export function readsHeldEvolvingArray(lowerer, expr) {
+    if (isJsSourceFile(ts.getSourceFileOfNode(expr)))
+        return false;
+    const held = evolvingArrayHold(lowerer, expr);
+    return held === "alias" || held === "container";
+}
+/** Whether a TypeScript object or array literal holds an evolving array
+ * binding whose reads disagree (evolvingArrayHold), so it builds as a
+ * checked-dynamic literal that keeps the array. */
+export function holdsEvolvingArray(lowerer, expr) {
+    if (isJsSourceFile(ts.getSourceFileOfNode(expr)))
+        return false;
+    return evolvingArrayHold(lowerer, expr) === "container";
+}
+/** An unannotated binding initialized with an evolving array binding whose
+ * reads disagree, or with an alias or container of one (evolvingArrayHold):
+ * it takes the array's checked-dynamic type, so it sees the later writes.
+ * Null for other bindings. */
+function evolvingArrayHolderType(lowerer, decl) {
+    if (decl.initializer === undefined)
+        return null;
+    return evolvingArrayHold(lowerer, decl.initializer) === null ? null : DYN;
 }
 /** A single static type for an initializer-less JavaScript `let` whose
  * later plain writes all agree. An evolving JS local reads as `undefined`
@@ -4488,20 +4863,7 @@ export function lowerVarDecl(lowerer, decl, isLet) {
         : inferredObjectType
             ? [inferredObjectType]
             : [];
-    const expandsObject = (!decl.type &&
-        !hasJsTypeAnnotation(decl) &&
-        inferredObjectType !== null &&
-        jsOpenObjectType(decl, inferredObjectType, lowerer.shapes, lowerer.unions).kind === "dyn") ||
-        inferredObjectArms.some((arm) => arm.kind === "record" && jsObjectBindingExpands(lowerer, decl, arm)) ||
-        (isJsSourceFile(ts.getSourceFileOfNode(decl)) &&
-            !decl.type &&
-            !hasJsTypeAnnotation(decl) &&
-            ts.isObjectLiteralExpression(decl.initializer) &&
-            decl.initializer.properties.nodes.some((property) => ts.isMethodDeclaration(property) ||
-                ts.isAccessor(property) ||
-                (ts.isPropertyAssignment(property) &&
-                    (ts.isFunctionExpression(property.initializer) ||
-                        ts.isArrowFunction(property.initializer)))));
+    const expandsObject = jsDeclarationExpandsObject(lowerer, decl, inferredObjectType);
     let init;
     try {
         init =
@@ -4666,7 +5028,7 @@ export function lowerVarDecl(lowerer, decl, isLet) {
     if (returnedEmptyArrayType === null &&
         !decl.type &&
         !isJsSourceFile(ts.getSourceFileOfNode(decl))) {
-        const evolved = evolvingArrayType(lowerer, decl);
+        const evolved = evolvingArrayType(lowerer, decl) ?? evolvingArrayHolderType(lowerer, decl);
         if (evolved !== null) {
             type = evolved;
             init = lowerer.lowerExprExpecting(decl.initializer, evolved);
@@ -5143,17 +5505,12 @@ export function hasJsTypeAnnotation(decl) {
     return (!!statement &&
         /@type\b/.test(ts.getSourceFileOfNode(decl).text.slice(statement.pos, ts.tokenStartOfNode(decl))));
 }
-/** Inferred JS scalar initializers do not constrain later writes through
- * an any-valued callback. Keep those bindings open, including captured writes. */
-export function jsBindingHasOpenWrites(lowerer, decl) {
-    if (!isJsSourceFile(ts.getSourceFileOfNode(decl)) ||
-        decl.type ||
-        hasJsTypeAnnotation(decl) ||
-        !ts.isIdentifier(decl.name))
-        return false;
-    const symbol = lowerer.resolveValueSymbol(decl.name);
+/** The assignments to a declaration's binding in its function (or file),
+ * `=` and compound alike. Null when the name has no symbol. */
+function jsBindingWrites(lowerer, decl, name) {
+    const symbol = lowerer.resolveValueSymbol(name);
     if (!symbol)
-        return false;
+        return null;
     let owner = decl;
     while (owner.parent && !ts.isFunctionLike(owner) && !ts.isSourceFile(owner))
         owner = owner.parent;
@@ -5175,9 +5532,41 @@ export function jsBindingHasOpenWrites(lowerer, decl) {
         writes = indexed;
         lowerer.jsBindingWritesByOwner.set(owner, writes);
     }
-    const candidates = (writes.get(decl.name.text) ?? []).filter((node) => lowerer.resolveValueSymbol(node.left) === symbol);
+    return (writes.get(name.text) ?? []).filter((node) => lowerer.resolveValueSymbol(node.left) === symbol);
+}
+/** Whether a JavaScript binding always holds a string: the checker types it
+ * as one, and every write appends to it (`result += part`), which yields a
+ * string whatever the part is. */
+export function jsBindingAppendsOnly(lowerer, decl) {
+    if (!ts.isIdentifier(decl.name))
+        return false;
+    if (lowerer.mapTypeOf(lowerer.typeOf(decl.name))?.kind !== "string")
+        return false;
+    const writes = jsBindingWrites(lowerer, decl, decl.name);
+    return (writes !== null &&
+        writes.every((node) => node.operatorToken.kind === ts.SyntaxKind.PlusEqualsToken));
+}
+/** Inferred JS scalar initializers do not constrain later writes through
+ * an any-valued callback. Keep those bindings open, including captured writes. */
+export function jsBindingHasOpenWrites(lowerer, decl) {
+    if (!isJsSourceFile(ts.getSourceFileOfNode(decl)) ||
+        decl.type ||
+        hasJsTypeAnnotation(decl) ||
+        !ts.isIdentifier(decl.name))
+        return false;
+    const candidates = jsBindingWrites(lowerer, decl, decl.name);
+    if (candidates === null)
+        return false;
     if (candidates.some((node) => (lowerer.typeOf(node.right).flags & (ts.TypeFlags.Any | ts.TypeFlags.Unknown)) !== 0))
         return true;
+    // A `+=` of a string turns a number or boolean binding into a string
+    // (`n += "px"`); the checker keeps the declared type.
+    if (candidates.some((node) => node.operatorToken.kind === ts.SyntaxKind.PlusEqualsToken &&
+        (lowerer.typeOf(node.right).flags & ts.TypeFlags.StringLike) !== 0)) {
+        const declared = lowerer.mapTypeOf(lowerer.typeOf(decl.name));
+        if (declared?.kind === "f64" || declared?.kind === "bool")
+            return true;
+    }
     // A write that can store null or undefined into a binding the checker
     // types as a primitive (`offset = match[3] || null`): JavaScript files
     // are checked without strict null checks, so the declared type does not
@@ -5700,7 +6089,7 @@ function lowerDeleteStatement(lowerer, expr) {
     if (obj.type.kind === "record") {
         const shape = lowerer.shapes.get(obj.type.shapeId);
         if (shape?.indexValue && shape.fields.length === 0 && !shape.tuple) {
-            return { kind: "recordKeyDelete", obj, shapeId: obj.type.shapeId, key: lowerKey(), loc };
+            return markSloppyRecordWrites(lowerer, { kind: "recordKeyDelete", obj, shapeId: obj.type.shapeId, key: lowerKey(), loc }, expr);
         }
         // `delete r.f` of a declared OPTIONAL field (undefined-armed slot)
         // writes the field's ABSENT state: a monomorphic shape cannot remove
@@ -5720,14 +6109,14 @@ function lowerDeleteStatement(lowerer, expr) {
         if (field) {
             const absent = lowerer.absentFieldValue(field.type, loc);
             if (absent) {
-                return {
+                return markSloppyRecordWrites(lowerer, {
                     kind: "recordSet",
                     obj,
                     shapeId: obj.type.shapeId,
                     field: field.name,
                     value: absent,
                     loc,
-                };
+                }, expr, true);
             }
         }
         if (shape?.indexValue) {
@@ -7671,7 +8060,7 @@ function lowerAssignTargetInto(lowerer, out, targetNode, blame, dflt, valueOf) {
             if (arr.type.kind === "array") {
                 const recv = lowerer.declareHiddenLocal("%dtRecv", arr.type);
                 out.push({ kind: "varDecl", localId: recv.id, init: arr, loc });
-                const index = lowerOptionalNumber(lowerer, lowerer.lowerExpr(t.argumentExpression), locOf(t.argumentExpression), t.argumentExpression);
+                const index = lowerOptionalIndex(lowerer, lowerer.lowerExpr(t.argumentExpression), locOf(t.argumentExpression), t.argumentExpression);
                 if (index.type.kind !== "f64") {
                     lowerer.unsupported("SC1090", t.argumentExpression, "indexing with non-number keys");
                 }
@@ -8060,6 +8449,16 @@ export function lowerForOf(lowerer, stmt) {
             lowerer.stdlibGlobalMember(stmt.expression, "process") === "stdin") {
             return lowerForAwaitStdin(lowerer, stmt);
         }
+        // NodeJS.WritableStream is what @types/node answers for pipeline()
+        // with a function destination, whose value is a PassThrough.
+        if (nodeStreamInterfaceSide(lowerer, lowerer.typeOf(stmt.expression)) === "w") {
+            const mapped = lowerer.mapTypeOf(lowerer.typeOf(stmt.expression));
+            const sides = mapped?.kind === "object"
+                ? streamSidesOf(lowerer, lowerer.classes.get(mapped.className))
+                : null;
+            if (sides !== "r" && sides !== "rw")
+                lowerer.noLowering("for await over a NodeJS.WritableStream", stmt.expression, "type the stream as the readable class it is: pipeline() answers a PassThrough for a function destination (pipeline(...) as PassThrough, or a JSDoc @type)");
+        }
         // A checked-dynamic NodeJS.ReadableStream iterates its native stream,
         // and a Hash, Hmac, Cipheriv or Decipheriv iterates its Transform.
         const dynStream = lowerForAwaitDynStream(lowerer, stmt, () => lowerForAwaitBuiltin(lowerer, stmt)) ??
@@ -8232,6 +8631,11 @@ export function lowerForOf(lowerer, stmt) {
     }
     iterSrc ??= stmt.expression;
     let iterable = lowerer.lowerExpr(iterSrc);
+    {
+        const iterableUnion = iterableUnionOf(lowerer, iterable.type);
+        if (iterableUnion)
+            return lowerForOfIterable(lowerer, stmt, iterable, iterableUnion, labels);
+    }
     let tupleSource = null;
     // A nested loop receiver can be an OOB-safe array read from the outer
     // loop (`for (const row of grid) for (const cell of row)`). The checker
@@ -9890,11 +10294,13 @@ function lowerForIn(lowerer, stmt) {
                     loc,
                 });
                 const keys = objectIterOverIndexShape(lowerer, stmt.expression, "keys", receiver.type, rShape, recvRef(), keysT, loc);
+                // The overflow map's hashed key set answers presence directly; a
+                // fresh key list per visit would make the walk quadratic.
                 const guard = (kRef) => ({
-                    kind: "arrIntrinsic",
-                    method: "includes",
-                    receiver: { kind: "recordOvfKeys", obj: recvRef(), shapeId, type: keysT, loc },
-                    args: [kRef],
+                    kind: "recordOvfHas",
+                    obj: recvRef(),
+                    shapeId,
+                    key: kRef,
                     type: BOOL,
                     loc,
                 });