socket-function 0.129.0 → 0.131.0

package/SocketFunctionTypes.ts

@@ -84,6 +84,8 @@ export type ClientHookContext = {
     overrideResult?: unknown;
     // Is called on a result, even if it is from overrideResult
     onResult: ((result: unknown) => MaybePromise<void>)[];
+    // IMPORTANT! This is a unique object per connection, reused within the connection. This
+    //  is allows it to be used as a cache key for connection related info (in a WeakMap).
     connectionId: { nodeId: string };
 };
 export interface SocketFunctionClientHook {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "socket-function",
-	"version": "0.129.0",
+	"version": "0.131.0",
 	"main": "index.js",
 	"license": "MIT",
 	"dependencies": {
@@ -9,7 +9,7 @@
 		"cbor-x": "^1.6.0",
 		"mobx": "^6.6.2",
 		"pako": "^2.1.0",
-		"preact": "10.24.3",
+		"preact": "10.11.3",
 		"typedev": "^0.4.0",
 		"typenode": "^5.12.0",
 		"ws": "^8.17.1"

package/src/batching.ts

@@ -97,10 +97,13 @@ export function batchFunction<Arg, Result = void>(
          */
         throttleWindow?: number;
         name?: string;
+        noMeasure?: boolean;
     },
     fnc: (arg: Arg[]) => (Promise<Result> | Result)
 ): (arg: Arg) => Promise<Result> {
-    fnc = measureWrap(fnc, config.name);
+    if (!config.noMeasure) {
+        fnc = measureWrap(fnc, config.name);
+    }
 
     let prevPromise: Promise<Result> | undefined;
     let batching: {
@@ -171,7 +174,7 @@ export function batchFunction<Arg, Result = void>(
 export function runInSerial<T extends (...args: any[]) => Promise<any>>(fnc: T): T {
     let updateQueue: { promise: Promise<void>; resolve: () => void; }[] = [];
 
-    return (async (...args: any[]) => {
+    return (async function runInSerial(...args: any[]) {
         let promise = {
             promise: null as any as Promise<void>,
             resolve: () => { },

package/src/caching.ts

@@ -263,7 +263,7 @@ export function cacheArgsEqual<Fnc extends AnyFunction>(
     fnc: Fnc,
     limit = 10
 ): Fnc & { clear(...args: Args<Fnc>): void } {
-    let cache = cacheArrayEqual((args: unknown[]) => {
+    let cache = cacheArrayEqual(function cacheArgsEqual(args: unknown[]) {
         return fnc(...args);
     }, limit);
     return Object.assign(

package/src/callManager.ts

@@ -3,7 +3,7 @@ import { CallerContext, CallType, ClientHookContext, FullCallType, FunctionFlags
 import { _setSocketContext } from "../SocketFunction";
 import { entries, isNode } from "./misc";
 import debugbreak from "debugbreak";
-import { measureWrap } from "./profiling/measure";
+import { measureBlock, measureWrap } from "./profiling/measure";
 import { formatTime } from "./formatting/format";
 
 let classes: {
@@ -146,7 +146,9 @@ export const runClientHooks = measureWrap(async function runClientHooks(
     }
     for (let hook of clientHooks) {
         let time = Date.now();
-        await hook(context);
+        await measureBlock(async () => {
+            await hook(context);
+        }, `clientHook|${hook.name || hook.toString().slice(0, 100)}`);
         time = Date.now() - time;
         if (time > 500) {
             console.warn(`Slow (${formatTime(time)}) client hook: ${JSON.stringify(hook.name || hook.toString().slice(0, 100))} for ${callType.classGuid}.${callType.functionName}(...)`);

package/src/misc.ts

@@ -346,7 +346,8 @@ export function binarySearchBasic<T, V>(array: T[], getVal: (val: T) => V, searc
  *      NOTE: If there are duplicates, returns the first match.
  * 
  *      NOTE: If the value can't be found, returns the bitwise negation of the index where it should be inserted.
- *          - If you just want the index which is >=, use `if(index < 0) index = ~index;`
+ * 
+ *      NOTE: With `if (index < 0) index = ~index;` you will get an index of the value >= the target value.
  */
 export function binarySearchIndex(listCount: number, compare: (lhsIndex: number) => number): number {
     if (listCount === 0) {

package/src/profiling/measure.ts

@@ -163,8 +163,8 @@ export function logMeasureTable(
     config?: LogMeasureTableConfig
 ): FormattedMeasureTable | undefined {
     let { useTotalTime, name } = config || {};
-    const thresholdInTable = config?.thresholdInTable ?? 0.05;
-    let minTimeToLog = config?.minTimeToLog ?? 50;
+    const thresholdInTable = config?.thresholdInTable ?? 0.02;
+    let minTimeToLog = config?.minTimeToLog ?? 20;
     const maxTableEntries = config?.maxTableEntries ?? 10;
 
     function getTime(entry: ProfileEntry) {

package/time/trueTimeShim.ts

@@ -54,7 +54,8 @@ export function getTrueTime() {
 export function getTrueTimeOffset() {
     return trueTimeOffset;
 }
-export function waitForFirstTimeSync() {
+export function waitForFirstTimeSync(): Promise<void> | undefined {
+    if (didFirstTimeSync) return undefined;
     return firstTimeSyncPromise;
 }
 let shimmed = false;