moost 0.6.33 → 0.6.35

package/dist/index.cjs

@@ -48,7 +48,7 @@ let stamped = false;
 	if (stamped) return;
 	stamped = true;
 	stampModuleIdentity({
-		version: "0.6.33",
+		version: "0.6.35",
 		path: typeof __filename === "string" ? __filename : require("url").pathToFileURL(__filename).href
 	});
 }
@@ -593,21 +593,41 @@ function nextScopeId() {
 
 //#endregion
 //#region packages/moost/src/binding/utils.ts
-/** Returns the own method names of an instance, including inherited methods from parent classes. */ function getInstanceOwnMethods(instance) {
-	const proto = Object.getPrototypeOf(instance);
-	return [...new Set([
-		...getParentProps((0, _prostojs_mate.getConstructor)(instance)),
-		...Object.getOwnPropertyNames(proto),
-		...Object.getOwnPropertyNames(instance)
-	])].filter((m) => typeof instance[m] === "function");
-}
-/** Returns the own non-method property names of an instance, including inherited properties. */ function getInstanceOwnProps(instance) {
-	const proto = Object.getPrototypeOf(instance);
-	return [...new Set([
-		...getParentProps((0, _prostojs_mate.getConstructor)(instance)),
-		...Object.getOwnPropertyNames(proto),
-		...Object.getOwnPropertyNames(instance)
-	])].filter((m) => typeof instance[m] !== "function");
+/**
+* Returns the own method names of an instance, including inherited methods
+* from parent classes.
+*
+* Classification is descriptor-based and NEVER invokes accessors: a property
+* is a method only when its nearest descriptor is a DATA property holding a
+* function. Accessor properties (get/set) are classified as props, not
+* methods — evaluating `instance[name]` to test them would fire the getter,
+* and getters are allowed to throw (e.g. moost-db's `.table` throws for
+* view-bound controllers), which used to turn a mere method scan into a
+* crash.
+*/ function getInstanceOwnMethods(instance) {
+	return collectByDescriptor(instance, true);
+}
+/** Returns the own non-method property names of an instance, including inherited properties. Accessor properties are always included (their getters are never invoked — see {@link getInstanceOwnMethods}). */ function getInstanceOwnProps(instance) {
+	return collectByDescriptor(instance, false);
+}
+/**
+* Walks the prototype chain (instance own names first, then each prototype up
+* to but excluding `Object.prototype`) and classifies every name by its
+* NEAREST descriptor — matching JS property-resolution order, so an instance
+* field shadowing a prototype method is classified by the instance field.
+*/ function collectByDescriptor(instance, wantMethods) {
+	const seen = /* @__PURE__ */ new Set();
+	const out = [];
+	let obj = instance;
+	while (obj && obj !== Object.prototype) {
+		for (const name of Object.getOwnPropertyNames(obj)) {
+			if (seen.has(name)) continue;
+			seen.add(name);
+			if (typeof Object.getOwnPropertyDescriptor(obj, name).value === "function" === wantMethods) out.push(name);
+		}
+		obj = Object.getPrototypeOf(obj);
+	}
+	return out;
 }
 /**
 * Walks the user-defined ancestor classes of a constructor, closest first.
@@ -619,11 +639,6 @@ function nextScopeId() {
 		parent = Object.getPrototypeOf(parent);
 	}
 }
-function getParentProps(constructor) {
-	const props = [];
-	for (const parent of ancestorsOf(constructor)) props.unshift(...Object.getOwnPropertyNames(parent.prototype));
-	return props;
-}
 
 //#endregion
 //#region packages/moost/src/decorators/circular.decorator.ts

package/dist/index.d.ts

@@ -1114,9 +1114,20 @@ declare function registerEventScope(scopeId: string): () => void;
  */
 declare function defineMoostEventHandler<T>(options: TMoostEventHandlerOptions<T>): () => unknown;
 
-/** Returns the own method names of an instance, including inherited methods from parent classes. */
+/**
+ * Returns the own method names of an instance, including inherited methods
+ * from parent classes.
+ *
+ * Classification is descriptor-based and NEVER invokes accessors: a property
+ * is a method only when its nearest descriptor is a DATA property holding a
+ * function. Accessor properties (get/set) are classified as props, not
+ * methods — evaluating `instance[name]` to test them would fire the getter,
+ * and getters are allowed to throw (e.g. moost-db's `.table` throws for
+ * view-bound controllers), which used to turn a mere method scan into a
+ * crash.
+ */
 declare function getInstanceOwnMethods<T = TAny>(instance: T): (keyof T)[];
-/** Returns the own non-method property names of an instance, including inherited properties. */
+/** Returns the own non-method property names of an instance, including inherited properties. Accessor properties are always included (their getters are never invoked — see {@link getInstanceOwnMethods}). */
 declare function getInstanceOwnProps<T = TAny>(instance: T): (keyof T)[];
 
 /**

package/dist/index.mjs

@@ -47,7 +47,7 @@ let stamped = false;
 	if (stamped) return;
 	stamped = true;
 	stampModuleIdentity({
-		version: "0.6.33",
+		version: "0.6.35",
 		path: typeof __filename === "string" ? __filename : import.meta.url
 	});
 }
@@ -592,21 +592,41 @@ function nextScopeId() {
 
 //#endregion
 //#region packages/moost/src/binding/utils.ts
-/** Returns the own method names of an instance, including inherited methods from parent classes. */ function getInstanceOwnMethods(instance) {
-	const proto = Object.getPrototypeOf(instance);
-	return [...new Set([
-		...getParentProps(getConstructor$1(instance)),
-		...Object.getOwnPropertyNames(proto),
-		...Object.getOwnPropertyNames(instance)
-	])].filter((m) => typeof instance[m] === "function");
-}
-/** Returns the own non-method property names of an instance, including inherited properties. */ function getInstanceOwnProps(instance) {
-	const proto = Object.getPrototypeOf(instance);
-	return [...new Set([
-		...getParentProps(getConstructor$1(instance)),
-		...Object.getOwnPropertyNames(proto),
-		...Object.getOwnPropertyNames(instance)
-	])].filter((m) => typeof instance[m] !== "function");
+/**
+* Returns the own method names of an instance, including inherited methods
+* from parent classes.
+*
+* Classification is descriptor-based and NEVER invokes accessors: a property
+* is a method only when its nearest descriptor is a DATA property holding a
+* function. Accessor properties (get/set) are classified as props, not
+* methods — evaluating `instance[name]` to test them would fire the getter,
+* and getters are allowed to throw (e.g. moost-db's `.table` throws for
+* view-bound controllers), which used to turn a mere method scan into a
+* crash.
+*/ function getInstanceOwnMethods(instance) {
+	return collectByDescriptor(instance, true);
+}
+/** Returns the own non-method property names of an instance, including inherited properties. Accessor properties are always included (their getters are never invoked — see {@link getInstanceOwnMethods}). */ function getInstanceOwnProps(instance) {
+	return collectByDescriptor(instance, false);
+}
+/**
+* Walks the prototype chain (instance own names first, then each prototype up
+* to but excluding `Object.prototype`) and classifies every name by its
+* NEAREST descriptor — matching JS property-resolution order, so an instance
+* field shadowing a prototype method is classified by the instance field.
+*/ function collectByDescriptor(instance, wantMethods) {
+	const seen = /* @__PURE__ */ new Set();
+	const out = [];
+	let obj = instance;
+	while (obj && obj !== Object.prototype) {
+		for (const name of Object.getOwnPropertyNames(obj)) {
+			if (seen.has(name)) continue;
+			seen.add(name);
+			if (typeof Object.getOwnPropertyDescriptor(obj, name).value === "function" === wantMethods) out.push(name);
+		}
+		obj = Object.getPrototypeOf(obj);
+	}
+	return out;
 }
 /**
 * Walks the user-defined ancestor classes of a constructor, closest first.
@@ -618,11 +638,6 @@ function nextScopeId() {
 		parent = Object.getPrototypeOf(parent);
 	}
 }
-function getParentProps(constructor) {
-	const props = [];
-	for (const parent of ancestorsOf(constructor)) props.unshift(...Object.getOwnPropertyNames(parent.prototype));
-	return props;
-}
 
 //#endregion
 //#region packages/moost/src/decorators/circular.decorator.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "moost",
-  "version": "0.6.33",
+  "version": "0.6.35",
   "description": "moost",
   "keywords": [
     "composables",
@@ -41,15 +41,15 @@
     "@prostojs/infact": "^0.5.0",
     "@prostojs/logger": "^0.4.3",
     "@prostojs/mate": "^0.5.0",
-    "@wooksjs/event-core": "^0.7.21",
+    "@wooksjs/event-core": "^0.7.22",
     "hookable": "^5.5.3",
-    "wooks": "^0.7.21"
+    "wooks": "^0.7.22"
   },
   "devDependencies": {
-    "@wooksjs/event-http": "^0.7.21",
-    "@wooksjs/http-body": "^0.7.21",
-    "vitest": "3.2.4",
-    "@moostjs/event-http": "^0.6.33"
+    "@wooksjs/event-http": "^0.7.22",
+    "@wooksjs/http-body": "^0.7.22",
+    "vitest": "3.2.7",
+    "@moostjs/event-http": "^0.6.35"
   },
   "scripts": {
     "pub": "pnpm publish --access public",