Quality: Global mutable call id can grow unbounded across long-lived processes (#2691)

* refactor: global mutable call id can grow unbounded across l

`callId` is module-scoped and incremented on every invocation. In long-running test runners or embedded usage, this can grow indefinitely and eventually lose integer precision semantics for strict ordering comparisons.

Affected files: proxy-invoke.js

Signed-off-by: tuanaiseo <221258316+tuanaiseo@users.noreply.github.com>

* Wrap around for all values that are too high

---------

Signed-off-by: tuanaiseo <221258316+tuanaiseo@users.noreply.github.com>
Co-authored-by: Carl-Erik Kopseng <carlerik@gmail.com>

Author: tuanaiseo

src/sinon/proxy-invoke.js

@@ -8,6 +8,7 @@ const ErrorConstructor = Error.prototype.constructor;
 const { bind } = Function.prototype;
 
 let callId = 0;
+const maxSafeInteger = Number.MAX_SAFE_INTEGER;
 
 /**
  * @callback SinonFunction
@@ -25,7 +26,8 @@ let callId = 0;
  */
 export default function invoke(func, thisValue, args) {
     const matchings = this.matchingFakes(args);
-    const currentCallId = callId++;
+    const currentCallId = callId;
+    callId = callId >= maxSafeInteger ? 0 : callId + 1;
     let exception, returnValue;
 
     proxyCallUtil.incrementCallCount(this);