kopscript 0.12.0 → 0.13.0

package/LLM.md

@@ -327,16 +327,30 @@ inspect a type parameter at runtime.
   "Cannot access member 'Foo' on type 'T'"). This is correct, not a bug — same as an
   unconstrained type parameter in C#/Java/TypeScript.
 
+**Generic inheritance**: a class/interface base list CAN name a generic base, with its own
+type arguments, in either direction:
+```ks
+class IntBox : Box<number> { }        // concrete — threads a fixed type through
+class Container<T> : Box<T> {          // generic — threads its OWN T through
+  public T GetAgain() { return this.Get(); }
+}
+interface INumberContainer : IContainer<number> { }   // interfaces work the same way
+```
+Member resolution (fields, methods, an inherited constructor via `base(...)` or an implicit
+default) composes substitutions correctly however many generic-base links up the chain a
+member was actually declared — `Container<number>.Get()` (declared on `Box<T>`, two links
+away in a longer chain) resolves to `number`, not the abstract `T`. **v1 limit**: at most
+one generic entry across a class/interface's *whole* base list (its superclass, or one
+implemented/extended interface — never more than one) — `class Foo<T> : Box<T>,
+IContainer<T>` is a compile error ("has more than one generic entry in its base list") even
+though either half alone would work.
+
 **Does not exist (v1 scope cuts, each deliberate)**:
 - **Generic functions.** `T Identity<T>(T x)` doesn't parse — only classes and interfaces
   take type parameters, not free functions/methods themselves (a method *inside* a
   generic class can use that class's own type parameters freely, same as any other member).
 - **Constraints.** No `T : IComparable` — a type parameter is always fully unconstrained,
   which is why you can't call members on a bare one (see above).
-- **Generic inheritance.** A class/interface's base list can only name a *non-generic*
-  type — `class Foo : Box<number>` and even `class Foo<T> : SomeGenericBase<T>` are both
-  compile errors ("cannot extend/implement generic type '...' — not supported in v1"). A
-  generic class/interface can still extend/implement ordinary non-generic bases normally.
 - **Variance.** No `out`/`in`/covariance/contravariance — see invariance above.
 
 ## Nullable types — `T?`
@@ -414,11 +428,13 @@ KopScript-declared name exactly. Extern class members use real JS member names v
 (camelCase, no rename mechanism). No inheritance modeling between two `extern class`
 declarations — each stands alone.
 
-`extern class` can carry `<T>` (`extern class Box<T> { constructor(T v); T Value { get; } }
-from "some-package";`) — identical rules to a real generic class (see "Generics" above:
-one unconstrained invariant parameter, erased, no base-list generics), so a generic type
-from another package instantiates and type-checks exactly like a local one
-(`Box<number>`, arity/invariance errors included).
+`extern class` can carry type parameters (`extern class Box<T> { constructor(T v); T Value
+{ get; } } from "some-package";`) — identical rules to a real generic class (see "Generics"
+above: one or more unconstrained invariant parameters, erased, generic inheritance
+included), so a generic type from another package instantiates and type-checks exactly
+like a local one (`Box<number>`, arity/invariance errors included) — and a real KopScript
+class can extend a generic `extern class` with a concrete or threaded-through type
+argument, same as extending a real generic base.
 
 **Never write `async` on an extern function/method signature** — declare its return type
 as `task`/`task<T>` directly (`task<string> text();`, not `async task<string> text();`).
@@ -505,9 +521,10 @@ Locals/params: **camelCase**. This is convention, not enforced by the compiler.
 
 ## Does not exist (don't reach for these)
 
-Generics beyond one or more unconstrained type parameters (no `T : IFoo` constraints, no
-generic functions, no generic inheritance, no variance — see Generics above for what *is*
-supported) · `any`/`unknown` annotations ·
+Generics beyond one or more unconstrained type parameters and single-generic-base
+inheritance (no `T : IFoo` constraints, no generic functions, no variance, no more than one
+generic entry per base list — see Generics above for what *is* supported) ·
+`any`/`unknown` annotations ·
 a `let`/`var` keyword (locals are just `Type name = value;`) · decorators/annotations ·
 reflection · type inference on declarations · ternary expression · union/tuple types ·
 **object-literal syntax** (`{ key: value }` as a value — this doesn't exist *anywhere* in

package/README.md

@@ -263,10 +263,23 @@ inside v1:
 - **No generic functions.** `T Identity<T>(T x)` isn't supported — only classes and
   interfaces take type parameters. A method *inside* a generic class can still use that
   class's own type parameters freely; it's just not introducing one of its own.
-- **No generic inheritance.** A class or interface's base list can only name a
-  *non-generic* type. `class Foo : Box<number>` and `class Foo<T> : SomeBase<T>` are both
-  compile errors — a generic class can still extend/implement ordinary non-generic bases
-  normally, it just can't be the one on either side of a generic base relationship.
+
+A class or interface *can* extend/implement a generic base, type arguments and all:
+
+```ks
+class IntBox : Box<number> { }                   // concrete: threads a fixed type through
+class Container<T> : Box<T> {                     // generic: threads its OWN T through
+  public T GetAgain() { return this.Get(); }
+}
+Container<number> c = new Container<number>(5);
+print(c.GetAgain());                               // 5, and c.Value/c.Get() work too
+```
+
+Member lookup (fields, methods, an inherited constructor) resolves correctly however many
+generic-base links up the chain a member was actually declared, substituting all the way
+down. **v1 limit**: at most one generic entry across a whole base list (the superclass, or
+one implemented interface — not several at once) — `class Foo<T> : Box<T>, IContainer<T>`
+is a compile error even though each half would work alone.
 
 See `LLM.md`'s "Generics" section for the exhaustive rules if you're generating code
 against this — nested generics (`Box<Box<number>>`, `Box<number>?[]`, `Pair<number,
@@ -372,9 +385,9 @@ globals within a single project.
 
 An `extern class` can carry its own type parameter(s), `extern class Box<T> { ... }` or
 `extern class Pair<K, V> { ... }` — exactly the same rules as a real generic class (see
-"Generics" above: invariant, unconstrained, erased, can't appear in a base list), so a
-generic type from another package (e.g. Kopular's `FormField<T>`) can be described and
-instantiated generically, not just per concrete type:
+"Generics" above: invariant, unconstrained, erased, and a real class can extend it as a
+generic base), so a generic type from another package (e.g. Kopular's `FormField<T>`) can
+be described and instantiated generically, not just per concrete type:
 
 ```ks
 extern class Box<T> {

package/dist/checker.js

@@ -204,7 +204,9 @@ export class Checker {
             name: decl.name,
             typeParams: decl.typeParams.map((p) => p.name),
             superclass: null,
+            superclassTypeArgs: null,
             interfaces: [],
+            interfaceTypeArgs: new Map(),
             fields,
             methods,
             staticFields,
@@ -459,6 +461,59 @@ export class Checker {
                 return type;
         }
     }
+    // Builds the bindings map for a generic reference (a `ClassType`/
+    // `InterfaceType`'s own `typeArgs`, zipped against its declared
+    // `typeParams`) — the "own bindings" every member-lookup/conformance
+    // check starts from. Null when the reference isn't actually generic
+    // (no declared params) or carries no type arguments (an arity error
+    // already diagnosed elsewhere — see validateGenericArity).
+    ownBindings(typeParams, typeArgs) {
+        if (typeParams.length === 0 || !typeArgs)
+            return null;
+        return new Map(typeParams.map((name, i) => [name, typeArgs[i]]));
+    }
+    // Maps `typeParams` to themselves, as abstract TypeParamTypes — used when
+    // resolving a base-list/interface type argument that may reference the
+    // *declaring* class/interface's own (still-abstract) type parameter,
+    // rather than a concrete instantiation (see checkInterfaceConformance and
+    // the base(...) call check in checkClassBodyInner).
+    identityBindings(typeParams) {
+        return new Map(typeParams.map((name) => [name, T.typeParamType(name)]));
+    }
+    // One step of substitution composition, walking from a class/interface
+    // ("owner") to one of its own generic base-list entries. `baseArgs` are
+    // the type arguments `owner` supplied to `baseName` in its own base list
+    // — still in `owner`'s own type-param-name space (may reference `owner`'s
+    // own type parameters, not yet concrete). `ownerBindings` is `owner`'s
+    // current bindings at the point of this lookup. Returns `baseName`'s own
+    // bindings map (its declared type-param names -> resolved types), ready
+    // for the next composition step or a final substituteTypeParams call.
+    composeBaseBindings(baseName, baseArgs, ownerBindings) {
+        const baseParams = this.genericTypeParams.get(baseName) ?? [];
+        const resolvedArgs = baseArgs.map((t) => this.substituteTypeParams(t, ownerBindings));
+        return new Map(baseParams.map((name, i) => [name, resolvedArgs[i]]));
+    }
+    // Starting from `className` with its own bindings (from ownBindings,
+    // above — empty if non-generic or no type arguments), walks up the
+    // superclass chain composing each generic link's stored
+    // superclassTypeArgs, until it reaches `targetOwner` (the ancestor that
+    // actually declared the member being resolved — see lookupField/
+    // lookupMethod/etc.'s `owner` result). Returns the bindings in
+    // `targetOwner`'s own type-param-name space. A non-generic link along the
+    // way (superclassTypeArgs null) resets to empty bindings, same as today's
+    // behavior of never substituting through a non-generic ancestor.
+    bindingsAtAncestor(className, ownBindings, targetOwner) {
+        let current = className;
+        let bindings = ownBindings;
+        while (current !== targetOwner) {
+            const info = this.classes.get(current);
+            if (!info || !info.superclass)
+                return bindings;
+            bindings = info.superclassTypeArgs ? this.composeBaseBindings(info.superclass, info.superclassTypeArgs, bindings) : new Map();
+            current = info.superclass;
+        }
+        return bindings;
+    }
     registerEnum(decl) {
         const members = new Map();
         decl.members.forEach((name, index) => {
@@ -472,28 +527,31 @@ export class Checker {
         this.recordHover(decl.line, decl.col, `enum ${decl.name}`);
     }
     registerInterface(decl) {
-        const methods = this.withTypeParamsInScope(decl.typeParams, () => decl.methods.map((m) => {
-            const params = m.params.map((p) => this.resolveType(p.type, m.line, m.col));
-            const returnType = this.resolveType(m.returnType, m.line, m.col);
-            this.recordHover(m.nameLine, m.nameCol, `${m.name}(${params.map(T.typeToString).join(", ")}): ${T.typeToString(returnType)}`);
-            return { name: m.name, params, returnType };
-        }));
-        const bases = [];
-        for (const baseName of decl.baseList) {
-            if (this.namedTypes.get(baseName) !== "interface") {
-                this.diagnostics.error("KS4012", `Interface '${decl.name}' can only extend other interfaces (unknown interface '${baseName}')`, decl.line, decl.col);
-                continue;
-            }
-            // v1 generics can't appear in a base list at all — only as a
-            // standalone type (field/param/return/local, `new Box<T>()`). A
-            // class/interface always implements/extends a *bare* name.
-            if (this.genericTypeParams.has(baseName)) {
-                this.diagnostics.error("KS4013", `Interface '${decl.name}' cannot extend generic interface '${baseName}' — not supported in v1`, decl.line, decl.col);
-                continue;
+        const genericBaseCount = { count: 0 };
+        const { methods, bases, baseTypeArgs } = this.withTypeParamsInScope(decl.typeParams, () => {
+            const methods = decl.methods.map((m) => {
+                const params = m.params.map((p) => this.resolveType(p.type, m.line, m.col));
+                const returnType = this.resolveType(m.returnType, m.line, m.col);
+                this.recordHover(m.nameLine, m.nameCol, `${m.name}(${params.map(T.typeToString).join(", ")}): ${T.typeToString(returnType)}`);
+                return { name: m.name, params, returnType };
+            });
+            const bases = [];
+            const baseTypeArgs = new Map();
+            for (const entry of decl.baseList) {
+                if (this.namedTypes.get(entry.name) !== "interface") {
+                    this.diagnostics.error("KS4012", `Interface '${decl.name}' can only extend other interfaces (unknown interface '${entry.name}')`, decl.line, decl.col);
+                    continue;
+                }
+                bases.push(entry.name);
+                if (entry.typeArgs || this.genericTypeParams.has(entry.name)) {
+                    const resolved = this.resolveBaseTypeArgs(decl.name, entry.name, entry.typeArgs, decl.line, decl.col, genericBaseCount);
+                    if (resolved)
+                        baseTypeArgs.set(entry.name, resolved);
+                }
             }
-            bases.push(baseName);
-        }
-        this.interfaces.set(decl.name, { name: decl.name, typeParams: decl.typeParams.map((p) => p.name), bases, methods });
+            return { methods, bases, baseTypeArgs };
+        });
+        this.interfaces.set(decl.name, { name: decl.name, typeParams: decl.typeParams.map((p) => p.name), bases, baseTypeArgs, methods });
         this.recordHover(decl.line, decl.col, decl.typeParams.length > 0 ? `interface ${decl.name}<${decl.typeParams.map((p) => p.name).join(", ")}>` : `interface ${decl.name}`);
     }
     checkInterfaceHierarchy(decl) {
@@ -517,16 +575,31 @@ export class Checker {
         }
     }
     // All methods required to satisfy an interface: its own plus every
-    // transitively-inherited parent interface's (cycle-safe).
-    collectInterfaceMethods(interfaceName, seen = new Set()) {
+    // transitively-inherited parent interface's (cycle-safe). `bindings` maps
+    // `interfaceName`'s own type-param names to resolved types (from
+    // ownBindings, at the root call — see its two call sites) — every
+    // returned signature is substituted through it, and a generic base link
+    // along the way (`interface IDerived<T> : IBase<T>`) composes its own
+    // stored baseTypeArgs through `bindings` before recursing, so a signature
+    // inherited from several generic-base links up still resolves correctly.
+    collectInterfaceMethods(interfaceName, bindings = new Map(), seen = new Set()) {
         if (seen.has(interfaceName))
             return [];
         seen.add(interfaceName);
         const info = this.interfaces.get(interfaceName);
         if (!info)
             return [];
-        const inherited = info.bases.flatMap((b) => this.collectInterfaceMethods(b, seen));
-        return [...inherited, ...info.methods];
+        const inherited = info.bases.flatMap((b) => {
+            const baseArgs = info.baseTypeArgs.get(b);
+            const baseBindings = baseArgs ? this.composeBaseBindings(b, baseArgs, bindings) : new Map();
+            return this.collectInterfaceMethods(b, baseBindings, seen);
+        });
+        const ownMethods = info.methods.map((m) => ({
+            name: m.name,
+            params: m.params.map((p) => this.substituteTypeParams(p, bindings)),
+            returnType: this.substituteTypeParams(m.returnType, bindings),
+        }));
+        return [...inherited, ...ownMethods];
     }
     interfaceExtends(sub, sup) {
         if (sub === sup)
@@ -538,7 +611,8 @@ export class Checker {
     }
     registerClass(decl) {
         this.recordHover(decl.line, decl.col, decl.typeParams.length > 0 ? `class ${decl.name}<${decl.typeParams.map((p) => p.name).join(", ")}>` : `class ${decl.name}`);
-        const { fields, staticFields, methods, staticMethods, ownCtorParams } = this.withTypeParamsInScope(decl.typeParams, () => {
+        const genericBaseCount = { count: 0 };
+        const { fields, staticFields, methods, staticMethods, ownCtorParams, superclass, superclassTypeArgs, interfaces, interfaceTypeArgs } = this.withTypeParamsInScope(decl.typeParams, () => {
             const fields = new Map();
             const staticFields = new Map();
             for (const f of decl.fields) {
@@ -567,39 +641,46 @@ export class Checker {
             const ownCtorParams = decl.constructor
                 ? decl.constructor.params.map((p) => this.resolveType(p.type, decl.constructor.line, decl.constructor.col))
                 : null;
-            return { fields, staticFields, methods, staticMethods, ownCtorParams };
-        });
-        let superclass = null;
-        const interfaces = [];
-        for (const baseName of decl.baseList) {
-            // v1 generics can't appear in a base list at all — only as a
-            // standalone type (field/param/return/local, `new Box<T>()`). A
-            // class always extends/implements a *bare* name.
-            if (this.genericTypeParams.has(baseName)) {
-                this.diagnostics.error("KS4015", `Class '${decl.name}' cannot extend/implement generic type '${baseName}' — not supported in v1`, decl.line, decl.col);
-                continue;
-            }
-            const kind = this.namedTypes.get(baseName);
-            if (kind === "class") {
-                if (superclass !== null) {
-                    this.diagnostics.error("KS4016", `Class '${decl.name}' cannot extend multiple classes ('${superclass}' and '${baseName}')`, decl.line, decl.col);
+            // Base-list resolution runs in this same scope — so `Box<T>` in
+            // `class Container<T> : Box<T>` resolves its inner `T` as
+            // Container's own type parameter, exactly like a field type would.
+            let superclass = null;
+            let superclassTypeArgs = null;
+            const interfaces = [];
+            const interfaceTypeArgs = new Map();
+            for (const entry of decl.baseList) {
+                const kind = this.namedTypes.get(entry.name);
+                if (kind === "class") {
+                    if (superclass !== null) {
+                        this.diagnostics.error("KS4016", `Class '${decl.name}' cannot extend multiple classes ('${superclass}' and '${entry.name}')`, decl.line, decl.col);
+                        continue;
+                    }
+                    superclass = entry.name;
+                    if (entry.typeArgs || this.genericTypeParams.has(entry.name)) {
+                        superclassTypeArgs = this.resolveBaseTypeArgs(decl.name, entry.name, entry.typeArgs, decl.line, decl.col, genericBaseCount);
+                    }
+                }
+                else if (kind === "interface") {
+                    interfaces.push(entry.name);
+                    if (entry.typeArgs || this.genericTypeParams.has(entry.name)) {
+                        const resolved = this.resolveBaseTypeArgs(decl.name, entry.name, entry.typeArgs, decl.line, decl.col, genericBaseCount);
+                        if (resolved)
+                            interfaceTypeArgs.set(entry.name, resolved);
+                    }
                 }
                 else {
-                    superclass = baseName;
+                    this.diagnostics.error("KS4017", `Unknown base class or interface '${entry.name}'`, decl.line, decl.col);
                 }
             }
-            else if (kind === "interface") {
-                interfaces.push(baseName);
-            }
-            else {
-                this.diagnostics.error("KS4017", `Unknown base class or interface '${baseName}'`, decl.line, decl.col);
-            }
-        }
+            return { fields, staticFields, methods, staticMethods, ownCtorParams, superclass, superclassTypeArgs, interfaces, interfaceTypeArgs };
+        });
         this.classes.set(decl.name, {
             name: decl.name,
             typeParams: decl.typeParams.map((p) => p.name),
             superclass,
+            superclassTypeArgs,
             interfaces,
+            interfaceTypeArgs,
             fields,
             methods,
             staticFields,
@@ -607,6 +688,35 @@ export class Checker {
             ownCtorParams,
         });
     }
+    // Resolves a base-list entry's own type arguments against `baseName`'s
+    // declared arity — `typeArgNodes` is null when the entry was written bare
+    // (`: Box`, not `: Box<number>`). Must be called from inside the
+    // declaring class/interface's own withTypeParamsInScope, so a type
+    // argument that references the declaring type's own parameter (`Box<T>`
+    // in `class Container<T> : Box<T>`) resolves correctly. `genericBaseCount`
+    // is a shared mutable counter across one declaration's whole base list,
+    // enforcing v1's "at most one generic entry per base list" cut.
+    resolveBaseTypeArgs(declName, baseName, typeArgNodes, declLine, declCol, genericBaseCount) {
+        const declaredParams = this.genericTypeParams.get(baseName);
+        if (!declaredParams) {
+            if (typeArgNodes)
+                this.diagnostics.error("KS4092", `'${baseName}' is not generic — it doesn't take a type argument`, declLine, declCol);
+            return null;
+        }
+        if (!typeArgNodes) {
+            this.diagnostics.error("KS4093", `Generic base '${baseName}' requires ${this.describeArity(declaredParams)} (e.g. '${baseName}<${declaredParams.join(", ")}>')`, declLine, declCol);
+            return null;
+        }
+        if (typeArgNodes.length !== declaredParams.length) {
+            this.diagnostics.error("KS4094", `Generic base '${baseName}' expects ${this.describeArity(declaredParams)}, got ${typeArgNodes.length}`, declLine, declCol);
+            return null;
+        }
+        genericBaseCount.count++;
+        if (genericBaseCount.count > 1) {
+            this.diagnostics.error("KS4095", `'${declName}' has more than one generic entry in its base list — v1 supports at most one`, declLine, declCol);
+        }
+        return typeArgNodes.map((a) => this.resolveType(a, declLine, declCol));
+    }
     checkClassHierarchy(decl) {
         const info = this.classes.get(decl.name);
         this.checkBaseCall(decl, info);
@@ -684,10 +794,19 @@ export class Checker {
     }
     checkInterfaceConformance(decl) {
         const classInfo = this.classes.get(decl.name);
+        // This is a conformance check against the class's own *declared*
+        // (still-abstract) signatures, not an instantiated use site, so a
+        // generic interface arg that references the class's own type param
+        // (`IContainer<T>` in `class Container<T> : IContainer<T>`) should
+        // compose down to that same abstract TypeParamType, matching how the
+        // class's own methods are stored — see identityBindings.
+        const selfBindings = this.identityBindings(classInfo.typeParams);
         for (const ifaceName of classInfo.interfaces) {
             if (!this.interfaces.has(ifaceName))
                 continue; // already reported as an unknown base type
-            for (const sig of this.collectInterfaceMethods(ifaceName)) {
+            const ifaceArgs = classInfo.interfaceTypeArgs.get(ifaceName);
+            const bindings = ifaceArgs ? this.composeBaseBindings(ifaceName, ifaceArgs, selfBindings) : new Map();
+            for (const sig of this.collectInterfaceMethods(ifaceName, bindings)) {
                 const found = this.lookupMethod(decl.name, sig.name);
                 if (!found) {
                     this.diagnostics.error("KS4026", `Class '${decl.name}' does not implement method '${sig.name}' required by interface '${ifaceName}'`, decl.line, decl.col);
@@ -896,7 +1015,18 @@ export class Checker {
             // params but no `this` — matching real base()/super() semantics,
             // which must run before `this` becomes available.
             if (decl.constructor.baseArgs && info.superclass) {
-                const baseCtorParams = this.lookupCtorParams(info.superclass);
+                // One step from `decl` to its immediate superclass (Container's own
+                // type params, still abstract, composed through whatever it
+                // supplied Box in its base list — see identityBindings), then
+                // however many further steps lookupCtorParams' own `owner` implies
+                // (Box itself has no constructor of its own, inherits its
+                // superclass's) via bindingsAtAncestor.
+                const superBindings = info.superclassTypeArgs
+                    ? this.composeBaseBindings(info.superclass, info.superclassTypeArgs, this.identityBindings(info.typeParams))
+                    : new Map();
+                const { params: rawBaseCtorParams, owner: baseCtorOwner } = this.lookupCtorParams(info.superclass);
+                const baseCtorBindings = this.bindingsAtAncestor(info.superclass, superBindings, baseCtorOwner);
+                const baseCtorParams = rawBaseCtorParams.map((p) => this.substituteTypeParams(p, baseCtorBindings));
                 const baseCtx = { returnType: T.VOID, currentClass: info, inConstructor: false, loopDepth: 0, isAsync: false };
                 if (decl.constructor.baseArgs.length !== baseCtorParams.length) {
                     this.diagnostics.error("KS4032", `Expected ${baseCtorParams.length} base constructor argument(s), got ${decl.constructor.baseArgs.length}`, decl.constructor.line, decl.constructor.col);
@@ -1498,11 +1628,16 @@ export class Checker {
             typeArgs = expr.typeArgs.map((a) => this.resolveType(a, expr.line, expr.col));
         }
         this.recordHover(expr.line, expr.col, typeArgs ? `class ${expr.className}<${typeArgs.map(T.typeToString).join(", ")}>` : `class ${expr.className}`);
-        let ctorParams = this.lookupCtorParams(expr.className);
-        if (info.typeParams.length > 0 && typeArgs) {
-            const bindings = new Map(info.typeParams.map((name, i) => [name, typeArgs[i]]));
-            ctorParams = ctorParams.map((p) => this.substituteTypeParams(p, bindings));
-        }
+        // The constructor being called may be inherited from a generic
+        // ancestor several base-list links up (e.g. `new NumberBox(5)` where
+        // `NumberBox : Box<number>` has no constructor of its own) — compose
+        // bindings all the way to whichever class's own constructor
+        // lookupCtorParams actually found (bindingsAtAncestor is a no-op single
+        // step when it's expr.className's own, the common case).
+        const { params: rawCtorParams, owner: ctorOwner } = this.lookupCtorParams(expr.className);
+        const ownCtorBindings = this.ownBindings(info.typeParams, typeArgs) ?? new Map();
+        const ctorBindings = this.bindingsAtAncestor(expr.className, ownCtorBindings, ctorOwner);
+        const ctorParams = rawCtorParams.map((p) => this.substituteTypeParams(p, ctorBindings));
         if (expr.args.length !== ctorParams.length) {
             this.diagnostics.error("KS4071", `Expected ${ctorParams.length} constructor argument(s), got ${expr.args.length}`, expr.line, expr.col);
         }
@@ -1515,17 +1650,23 @@ export class Checker {
         });
         return T.classType(expr.className, typeArgs);
     }
+    // `owner` is which class up the chain actually declared the constructor
+    // being inherited (itself, or the nearest ancestor with one) — needed so
+    // a caller can compose the right type-parameter bindings when that
+    // ancestor is a generic base (see bindingsAtAncestor). `params` is stored
+    // abstractly in `owner`'s own type-param-name space, same as a field or
+    // method's type would be.
     lookupCtorParams(className) {
         let current = className;
         while (current) {
             const info = this.classes.get(current);
             if (!info)
-                return [];
+                return { params: [], owner: className };
             if (info.ownCtorParams !== null)
-                return info.ownCtorParams;
+                return { params: info.ownCtorParams, owner: current };
             current = info.superclass;
         }
-        return [];
+        return { params: [], owner: className };
     }
     // Array stdlib with a fixed (non-polymorphic) signature, given the
     // array's own element type — everything except Map, which checkCall
@@ -1663,13 +1804,14 @@ export class Checker {
             return { type: T.UNKNOWN, methodInfo: null };
         }
         if (objectType.kind === "class") {
-            // v1 has no generic inheritance (a generic class's base list can only
-            // name non-generic types), so a member found on a generic instance
-            // was always declared directly on that same class — substituting by
-            // its own type parameters, not some ancestor's, is always correct.
-            const declaredParams = objectType.typeArgs ? this.genericTypeParams.get(objectType.name) : undefined;
-            const bindings = declaredParams && objectType.typeArgs ? new Map(declaredParams.map((name, i) => [name, objectType.typeArgs[i]])) : null;
-            const substitute = (type) => (bindings ? this.substituteTypeParams(type, bindings) : type);
+            // A member found on an *ancestor* (owner !== objectType.name — v1 now
+            // supports a generic superclass, see registerClass/
+            // bindingsAtAncestor) needs bindings composed through however many
+            // generic base links separate objectType from that ancestor, not just
+            // objectType's own type arguments — bindingsAtAncestor does that walk
+            // (a no-op single step when owner IS objectType.name, the common
+            // case, same result as before this existed).
+            const ownBindings = this.ownBindings(this.genericTypeParams.get(objectType.name) ?? [], objectType.typeArgs) ?? new Map();
             const field = this.lookupField(objectType.name, expr.property);
             if (field) {
                 this.checkAccessibility(field.info.visibility, field.owner, ctx, expr.property, expr.line, expr.col);
@@ -1679,29 +1821,32 @@ export class Checker {
                         this.diagnostics.error("KS4082", `'${expr.property}' has no setter and can only be assigned within ${field.owner}'s constructor`, expr.line, expr.col);
                     }
                 }
-                return { type: substitute(field.info.type), methodInfo: null };
+                const bindings = this.bindingsAtAncestor(objectType.name, ownBindings, field.owner);
+                return { type: this.substituteTypeParams(field.info.type, bindings), methodInfo: null };
             }
             const method = this.lookupMethod(objectType.name, expr.property);
             if (method) {
                 this.checkAccessibility(method.info.visibility, method.owner, ctx, expr.property, expr.line, expr.col);
-                const info = bindings ? { ...method.info, params: method.info.params.map(substitute), returnType: substitute(method.info.returnType) } : method.info;
+                const bindings = this.bindingsAtAncestor(objectType.name, ownBindings, method.owner);
+                const info = {
+                    ...method.info,
+                    params: method.info.params.map((p) => this.substituteTypeParams(p, bindings)),
+                    returnType: this.substituteTypeParams(method.info.returnType, bindings),
+                };
                 return { type: info.returnType, methodInfo: info };
             }
             this.diagnostics.error("KS4083", `Class '${objectType.name}' has no member '${expr.property}'`, expr.line, expr.col);
             return { type: T.UNKNOWN, methodInfo: null };
         }
         if (objectType.kind === "interface") {
-            const sig = this.collectInterfaceMethods(objectType.name).find((m) => m.name === expr.property);
+            // collectInterfaceMethods itself now substitutes through `bindings` —
+            // including across a generic base-interface link — so passing
+            // objectType's own bindings once at the root is enough; no separate
+            // re-substitution needed here (see its own header comment).
+            const ownBindings = this.ownBindings(this.genericTypeParams.get(objectType.name) ?? [], objectType.typeArgs) ?? new Map();
+            const sig = this.collectInterfaceMethods(objectType.name, ownBindings).find((m) => m.name === expr.property);
             if (sig) {
-                // v1 has no generic interface inheritance either (same restriction
-                // as classes — see registerInterface), so a signature found here
-                // was always declared directly on this same interface.
-                const declaredParams = objectType.typeArgs ? this.genericTypeParams.get(objectType.name) : undefined;
-                const bindings = declaredParams && objectType.typeArgs ? new Map(declaredParams.map((name, i) => [name, objectType.typeArgs[i]])) : null;
-                const substitute = (type) => (bindings ? this.substituteTypeParams(type, bindings) : type);
-                const params = sig.params.map(substitute);
-                const returnType = substitute(sig.returnType);
-                return { type: returnType, methodInfo: { params, returnType, visibility: "public", isVirtual: false, isOverride: false } };
+                return { type: sig.returnType, methodInfo: { params: sig.params, returnType: sig.returnType, visibility: "public", isVirtual: false, isOverride: false } };
             }
             this.diagnostics.error("KS4084", `Interface '${objectType.name}' has no member '${expr.property}'`, expr.line, expr.col);
             return { type: T.UNKNOWN, methodInfo: null };

package/dist/codegen.js

@@ -286,7 +286,7 @@ export class CodeGenerator {
         const pad = this.indentStr(indent);
         // The base list mixes an optional superclass with interface names (checker-validated);
         // only the non-interface entry, if any, becomes a JS `extends` clause.
-        const superclass = decl.baseList.find((n) => !this.interfaceNames.has(n)) ?? null;
+        const superclass = decl.baseList.find((b) => !this.interfaceNames.has(b.name))?.name ?? null;
         const header = superclass ? `class ${decl.name} extends ${superclass} {` : `class ${decl.name} {`;
         const memberPad = this.indentStr(indent + 1);
         const memberCol = memberPad.length;

package/dist/parser.js

@@ -258,6 +258,21 @@ export class Parser {
         this.consume(TokenKind.Gt, "Expected '>' after type parameter list");
         return params;
     }
+    // `Bar<T>` or `IBaz` — one base-list entry, with or without its own type
+    // arguments. Used by both class and interface base lists.
+    parseBaseListEntry() {
+        const name = this.consume(TokenKind.Identifier, "Expected base class or interface name").lexeme;
+        let typeArgs = null;
+        if (this.check(TokenKind.Lt)) {
+            this.advance();
+            typeArgs = [this.parseType()];
+            while (this.match(TokenKind.Comma)) {
+                typeArgs.push(this.parseType());
+            }
+            this.consume(TokenKind.Gt, "Expected '>' after type argument list");
+        }
+        return { name, typeArgs };
+    }
     parseClassDecl(isExported) {
         const start = this.advance(); // 'class'
         const name = this.consume(TokenKind.Identifier, "Expected class name").lexeme;
@@ -265,7 +280,7 @@ export class Parser {
         const baseList = [];
         if (this.match(TokenKind.Colon)) {
             do {
-                baseList.push(this.consume(TokenKind.Identifier, "Expected base class or interface name").lexeme);
+                baseList.push(this.parseBaseListEntry());
             } while (this.match(TokenKind.Comma));
         }
         this.consume(TokenKind.LBrace, "Expected '{' before class body");
@@ -426,7 +441,7 @@ export class Parser {
         const baseList = [];
         if (this.match(TokenKind.Colon)) {
             do {
-                baseList.push(this.consume(TokenKind.Identifier, "Expected base interface name").lexeme);
+                baseList.push(this.parseBaseListEntry());
             } while (this.match(TokenKind.Comma));
         }
         this.consume(TokenKind.LBrace, "Expected '{' before interface body");

package/dist/printer.js

@@ -118,6 +118,9 @@ export class Printer {
     printParams(params) {
         return params.map((p) => `${this.printType(p.type)} ${p.name}`).join(", ");
     }
+    printBaseListEntry(entry) {
+        return entry.typeArgs ? `${entry.name}<${entry.typeArgs.map((t) => this.printType(t)).join(", ")}>` : entry.name;
+    }
     // ---------- top-level declarations ----------
     printStatement(stmt, indent) {
         const pad = indentStr(indent);
@@ -213,7 +216,7 @@ export class Printer {
         const pad = indentStr(indent);
         const memberPad = indentStr(indent + 1);
         const nameWithTypeParam = decl.typeParams.length > 0 ? `${decl.name}<${decl.typeParams.map((p) => p.name).join(", ")}>` : decl.name;
-        const header = decl.baseList.length > 0 ? `class ${nameWithTypeParam} : ${decl.baseList.join(", ")} {` : `class ${nameWithTypeParam} {`;
+        const header = decl.baseList.length > 0 ? `class ${nameWithTypeParam} : ${decl.baseList.map((b) => this.printBaseListEntry(b)).join(", ")} {` : `class ${nameWithTypeParam} {`;
         const prefix = decl.isExported ? "" : "private ";
         const memberParts = [];
         const pushMember = (node, text) => {
@@ -251,7 +254,7 @@ export class Printer {
         const pad = indentStr(indent);
         const memberPad = indentStr(indent + 1);
         const nameWithTypeParam = decl.typeParams.length > 0 ? `${decl.name}<${decl.typeParams.map((p) => p.name).join(", ")}>` : decl.name;
-        const header = decl.baseList.length > 0 ? `interface ${nameWithTypeParam} : ${decl.baseList.join(", ")} {` : `interface ${nameWithTypeParam} {`;
+        const header = decl.baseList.length > 0 ? `interface ${nameWithTypeParam} : ${decl.baseList.map((b) => this.printBaseListEntry(b)).join(", ")} {` : `interface ${nameWithTypeParam} {`;
         const prefix = decl.isExported ? "" : "private ";
         if (decl.methods.length === 0)
             return `${pad}${prefix}${header}\n${pad}}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kopscript",
-  "version": "0.12.0",
+  "version": "0.13.0",
   "description": "KopScript: a small OOP, strongly-typed language that transpiles to JavaScript, with generics and nullable types",
   "type": "module",
   "license": "MIT",