Remove copy function

Author: tolyo

dist/angular-ts.cjs.js

@@ -1,6 +1,5 @@
 import {
   minErr,
-  copy,
   extend,
   forEach,
   getNgAttribute,
@@ -63,7 +62,6 @@ export class Angular {
 
     // Utility methods kept for backwards purposes
     this.bind = bind;
-    this.copy = copy;
     this.equals = equals;
     this.element = jqLite;
     this.extend = extend;

dist/angular-ts.esm.js

@@ -1,5 +1,3 @@
-import { copy } from "./utils";
-
 /**
  * @ngdoc service
  * @name $animateCss
@@ -14,17 +12,16 @@ import { copy } from "./utils";
  */
 export function CoreAnimateCssProvider() {
   this.$get = [
-    "$q",
     "$$AnimateRunner",
-    ($q, $$AnimateRunner) =>
+    ($$AnimateRunner) =>
       function (element, initialOptions) {
         // all of the animation functions should create
         // a copy of the options data, however, if a
         // parent service has already created a copy then
         // we should stick to using that
         let options = initialOptions || {};
         if (!options.$$prepared) {
-          options = copy(options);
+          options = structuredClone(options);
         }
 
         // there is no point in applying the styles since

dist/angular-ts.umd.js

@@ -270,7 +270,7 @@ import { createEventDirective } from "./directive/ngEventDirs";
  *              }
  *              if (!angular.equals(this.items_clone, this.items)) {
  *                this.log.push('doCheck: items mutated');
- *                this.items_clone = angular.copy(this.items);
+ *                this.items_clone = structuredClone(this.items);
  *              }
  *            };
  *          }

src/loader.js

@@ -1,7 +1,6 @@
 import { NODE_TYPE_COMMENT } from "../../constants";
 import { jqLite, jqLiteRemove, startingTag } from "../../jqLite";
 import {
-  copy,
   equals,
   forEach,
   hashKey,
@@ -356,7 +355,9 @@ export const ngOptionsDirective = [
             getViewValueFromOption(option) {
               // If the viewValue could be an object that may be mutated by the application,
               // we need to make a copy and not return the reference to the value on the option.
-              return trackBy ? copy(option.viewValue) : option.viewValue;
+              return trackBy
+                ? structuredClone(option.viewValue)
+                : option.viewValue;
             },
           };
         },

src/ng/animateCss.js

@@ -121,11 +121,11 @@ import { getBlockNodes } from "../../jqLite";
         };
 
         $scope.copy = function() {
-          $scope.friends = angular.copy($scope.friends);
+          $scope.friends = structuredClone($scope.friends);
         };
 
         $scope.reset = function() {
-          $scope.friends = angular.copy(friends);
+          $scope.friends = structuredClone(friends);
         };
 
         $scope.reset();

src/ng/compile.js

@@ -1,7 +1,6 @@
 import { jqLite } from "../jqLite";
 import { urlResolve } from "./urlUtils";
 import {
-  copy,
   encodeUriSegment,
   forEach,
   isBoolean,
@@ -582,7 +581,7 @@ const locationPrototype = {
           search = search.toString();
           this.$$search = parseKeyValue(search);
         } else if (isObject(search)) {
-          search = copy(search, {});
+          search = structuredClone(search, {});
           // remove object undefined or null properties
           forEach(search, (value, key) => {
             if (value == null) delete search[key];

src/ng/directive/ngOptions.js

@@ -1,9 +1,4 @@
-/* eslint-disable no-param-reassign */
-/* eslint-disable max-classes-per-file */
-/* eslint-disable no-use-before-define */
-
 import {
-  copy,
   createMap,
   csp,
   forEach,
@@ -523,7 +518,7 @@ AST.prototype = {
         this.peek().text,
       )
     ) {
-      primary = copy(this.selfReferential[this.consume().text]);
+      primary = structuredClone(this.selfReferential[this.consume().text]);
     } else if (
       Object.prototype.hasOwnProperty.call(
         this.options.literals,
@@ -2082,7 +2077,7 @@ export function $ParseProvider() {
       var noUnsafeEval = csp().noUnsafeEval;
       var $parseOptions = {
         csp: noUnsafeEval,
-        literals: copy(literals),
+        literals: structuredClone(literals),
         isIdentifierStart: isFunction(identStart) && identStart,
         isIdentifierContinue: isFunction(identContinue) && identContinue,
       };

src/ng/directive/ngRepeat.js

@@ -8,7 +8,6 @@ import {
   isObject,
   isArrayLike,
   isNumberNaN,
-  copy,
   arrayRemove,
   equals,
 } from "./utils";
@@ -294,10 +293,10 @@ class Scope {
  *   (see next point)
  * - When `objectEquality == true`, inequality of the `watchExpression` is determined
  *   according to the {@link angular.equals} function. To save the value of the object for
- *   later comparison, the {@link angular.copy} function is used. This therefore means that
+ *   later comparison, the {@link structuredClone} function is used. This therefore means that
  *   watching complex objects will have adverse memory and performance implications.
  * - This should not be used to watch for changes in objects that are (or contain)
- *   [File](https://developer.mozilla.org/docs/Web/API/File) objects due to limitations with {@link angular.copy `angular.copy`}.
+ *   [File](https://developer.mozilla.org/docs/Web/API/File) objects due to limitations with {@link structuredClone `structuredClone`}.
  * - The watch `listener` may change the model, which may trigger other `listener`s to fire.
  *   This is achieved by rerunning the watchers until no changes are detected. The rerun
  *   iteration limit is 10 to prevent an infinite loop deadlock.
@@ -822,7 +821,7 @@ class Scope {
                 ) {
                   dirty = true;
                   lastDirtyWatch = watch;
-                  watch.last = watch.eq ? copy(value, null) : value;
+                  watch.last = watch.eq ? structuredClone(value) : value;
                   fn = watch.fn;
                   fn(value, last === initWatchVal ? value : last, current);
                   if (ttl < 5) {

src/ng/location.js

@@ -625,174 +625,6 @@ export function arrayRemove(array, value) {
   return index;
 }
 
-/**
- * Creates a deep copy of the source, which should be an object or an array.
- * - If no destination is supplied, a copy of the object or array is created.
- * - If a destination is provided, all of its elements (for arrays) or properties (for objects) are deleted,
- *   and then all elements/properties from the source are copied to it.
- * - If the source is not an object or array (including null and undefined), the source is returned.
- * - If the source is identical to the destination, an exception will be thrown.
- *
- * @template T The type of the source and destination.
- * @param {T} source The source that will be used to make a copy. Can be any type, including primitives, null, and undefined.
- * @param {T} [destination] Destination into which the source is copied. If provided, must be of the same type as source.
- * @returns {T} A deep copy of the source.
- */
-export function copy(source, destination, maxDepth) {
-  const stackSource = [];
-  const stackDest = [];
-  maxDepth = isValidObjectMaxDepth(maxDepth) ? maxDepth : NaN;
-
-  if (destination) {
-    if (isTypedArray(destination) || isArrayBuffer(destination)) {
-      throw ngMinErr(
-        "cpta",
-        "Can't copy! TypedArray destination cannot be mutated.",
-      );
-    }
-    if (source === destination) {
-      throw ngMinErr(
-        "cpi",
-        "Can't copy! Source and destination are identical.",
-      );
-    }
-
-    // Empty the destination object
-    if (isArray(destination)) {
-      destination.length = 0;
-    } else {
-      forEach(destination, (value, key) => {
-        if (key !== "$$hashKey") {
-          delete destination[key];
-        }
-      });
-    }
-
-    stackSource.push(source);
-    stackDest.push(destination);
-    return copyRecurse(source, destination, maxDepth);
-  }
-
-  return copyElement(source, maxDepth);
-
-  function copyRecurse(source, destination, maxDepth) {
-    maxDepth--;
-    if (maxDepth < 0) {
-      return "...";
-    }
-    const h = destination.$$hashKey;
-    let key;
-    if (isArray(source)) {
-      for (let i = 0, ii = source.length; i < ii; i++) {
-        destination.push(copyElement(source[i], maxDepth));
-      }
-    } else if (isBlankObject(source)) {
-      // createMap() fast path --- Safe to avoid hasOwnProperty check because prototype chain is empty
-      for (key in source) {
-        destination[key] = copyElement(source[key], maxDepth);
-      }
-    } else if (source && typeof source.hasOwnProperty === "function") {
-      // Slow path, which must rely on hasOwnProperty
-      for (key in source) {
-        if (Object.prototype.hasOwnProperty.call(source, key)) {
-          destination[key] = copyElement(source[key], maxDepth);
-        }
-      }
-    }
-    setHashKey(destination, h);
-    return destination;
-  }
-
-  function copyElement(source, maxDepth) {
-    // Simple values
-    if (!isObject(source)) {
-      return source;
-    }
-
-    // Already copied values
-    const index = stackSource.indexOf(source);
-    if (index !== -1) {
-      return stackDest[index];
-    }
-
-    if (isWindow(source) || isScope(source)) {
-      throw ngMinErr(
-        "cpws",
-        "Can't copy! Making copies of Window or Scope instances is not supported.",
-      );
-    }
-
-    let needsRecurse = false;
-    let destination = copyType(source);
-
-    if (destination === undefined) {
-      destination = isArray(source)
-        ? []
-        : Object.create(Object.getPrototypeOf(source));
-      needsRecurse = true;
-    }
-
-    stackSource.push(source);
-    stackDest.push(destination);
-
-    return needsRecurse
-      ? copyRecurse(source, destination, maxDepth)
-      : destination;
-  }
-
-  function copyType(source) {
-    switch (toString.call(source)) {
-      case "[object Int8Array]":
-      case "[object Int16Array]":
-      case "[object Int32Array]":
-      case "[object Float32Array]":
-      case "[object Float64Array]":
-      case "[object Uint8Array]":
-      case "[object Uint8ClampedArray]":
-      case "[object Uint16Array]":
-      case "[object Uint32Array]":
-        return new source.constructor(
-          copyElement(source.buffer),
-          source.byteOffset,
-          source.length,
-        );
-
-      case "[object ArrayBuffer]":
-        // Support: IE10
-        if (!source.slice) {
-          // If we're in this case we know the environment supports ArrayBuffer
-          /* eslint-disable no-undef */
-          const copied = new ArrayBuffer(source.byteLength);
-          new Uint8Array(copied).set(new Uint8Array(source));
-          /* eslint-enable */
-          return copied;
-        }
-        return source.slice(0);
-
-      case "[object Boolean]":
-      case "[object Number]":
-      case "[object String]":
-      case "[object Date]":
-        return new source.constructor(source.valueOf());
-
-      case "[object RegExp]":
-        const re = new RegExp(
-          source.source,
-          source.toString().match(/[^/]*$/)[0],
-        );
-        re.lastIndex = source.lastIndex;
-        return re;
-
-      case "[object Blob]":
-        return new source.constructor([source], { type: source.type });
-    }
-
-    if (isFunction(source.cloneNode)) {
-      return source.cloneNode(true);
-    }
-  }
-}
-
 export function simpleCompare(a, b) {
   return a === b || (a !== a && b !== b);
 }
@@ -1407,7 +1239,7 @@ export function minErr(module) {
     const template = arguments[1];
     let message = `[${module ? `${module}:` : ""}${code}] `;
     const templateArgs = sliceArgs(arguments, 2).map((arg) =>
-      toDebugString(arg, minErrConfig.objectMaxDepth),
+      toDebugString(arg),
     );
     let paramPrefix;
     let i;
@@ -1443,17 +1275,9 @@ export function minErr(module) {
   };
 }
 
-export function serializeObject(obj, maxDepth) {
+export function serializeObject(obj) {
   const seen = [];
-  let copyObj = obj;
-  // There is no direct way to stringify object until reaching a specific depth
-  // and a very deep object can cause a performance issue, so we copy the object
-  // based on this specific depth and then stringify it.
-  if (isValidObjectMaxDepth(maxDepth)) {
-    // This file is also included in `angular-loader`, so `copy()` might not always be available in
-    // the closure. Therefore, it is lazily retrieved as `angular.copy()` when needed.
-    copyObj = copy(obj, null, maxDepth);
-  }
+  let copyObj = structuredClone(obj);
   return JSON.stringify(copyObj, (key, val) => {
     const replace = toJsonReplacer(key, val);
     if (isObject(replace)) {
@@ -1465,15 +1289,15 @@ export function serializeObject(obj, maxDepth) {
   });
 }
 
-export function toDebugString(obj, maxDepth) {
+export function toDebugString(obj) {
   if (typeof obj === "function") {
     return obj.toString().replace(/ \{[\s\S]*$/, "");
   }
   if (isUndefined(obj)) {
     return "undefined";
   }
   if (typeof obj !== "string") {
-    return serializeObject(obj, maxDepth);
+    return serializeObject(obj);
   }
   return obj;
 }

src/ng/parse.js

@@ -6,7 +6,6 @@ import {
   isArray,
   isString,
   isElement,
-  copy,
   isDefined,
   extend,
 } from "../ng/utils";
@@ -400,7 +399,7 @@ export const $$AnimateQueueProvider = [
           // we always make a copy of the options since
           // there should never be any side effects on
           // the input data when running `$animateCss`.
-          let options = copy(initialOptions);
+          let options = structuredClone(initialOptions);
 
           let element = stripCommentsFromElement(originalElement);
           const node = getDomNode(element);