remove unnecessary includes check from ctx.error

Author: ssalbdivad

ark/schema/shared/errors.ts

@@ -224,6 +224,7 @@ export class ArkErrors
 	add(error: ArkError): void {
 		const existing = this.byPath[error.propString]
 		if (existing) {
+			// only add if it's not already in the errors collection
 			if (error === existing) return
 			// If the existing error is an error for a value constrained to "never",
 			// then we don't want to intersect the error messages.

ark/schema/shared/traversal.ts

@@ -257,9 +257,7 @@ export class Traversal {
 
 			if (result instanceof ArkError) {
 				// if an ArkError was returned, ensure it has been added to errors
-				// (it may have already been added via ctx.error() within the morph)
-				// Only add if it's not already in the errors collection
-				if (!this.errors.includes(result)) this.errors.add(result)
+				this.errors.add(result)
 
 				// skip any remaining morphs at the current path
 				break