fix: code style

packages/core/src/types.ts

@@ -255,11 +255,7 @@ export interface AgentErrorEvent {
  * Union type for all history events
  */
 export type HistoricalEvent =
-	| AgentStepEvent
+	AgentStepEvent | ObservationEvent | UserTakeoverEvent | RetryEvent | AgentErrorEvent
-	| ObservationEvent
-	| UserTakeoverEvent
-	| RetryEvent
-	| AgentErrorEvent
 
 /**
  * Agent lifecycle status.

packages/page-controller/src/dom/dom_tree/index.js

@@ -598,14 +598,12 @@ export default (
 					isAnyRectVisible = true
 
 					// Viewport check for this rect
-					if (
+					if (!(
-						!(
 						rect.bottom < -viewportExpansion ||
 						rect.top > window.innerHeight + viewportExpansion ||
 						rect.right < -viewportExpansion ||
 						rect.left > window.innerWidth + viewportExpansion
-						)
+					)) {
-					) {
 						isAnyRectInViewport = true
 						break // Found a visible rect in viewport, no need to check others
 					}
@@ -985,13 +983,11 @@ export default (
 				rect.height > 0 &&
 				!(
 					// Only check non-empty rects
-					(
 					rect.bottom < -viewportExpansion ||
 					rect.top > window.innerHeight + viewportExpansion ||
 					rect.right < -viewportExpansion ||
 					rect.left > window.innerWidth + viewportExpansion
 				)
-				)
 			) {
 				isAnyRectInViewport = true
 				break
@@ -1103,14 +1099,12 @@ export default (
 		for (const rect of rects) {
 			if (rect.width === 0 || rect.height === 0) continue // Skip empty rects
 
-			if (
+			if (!(
-				!(
 				rect.bottom < -viewportExpansion ||
 				rect.top > window.innerHeight + viewportExpansion ||
 				rect.right < -viewportExpansion ||
 				rect.left > window.innerWidth + viewportExpansion
-				)
+			)) {
-			) {
 				return true // Found at least one rect in the viewport
 			}
 		}