chore(TS): Group

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## [next]
 
+- chore(TS): migrate Group/ActiveSelection [#8455](https://github.com/fabricjs/fabric.js/pull/8455)
 - chore(TS): migrate text classes/mixins [#8408](https://github.com/fabricjs/fabric.js/pull/8408)
 - chore(TS): migrate Collection [#8433](https://github.com/fabricjs/fabric.js/pull/8433)
 - ci(): Simplify filestats even more [#8449](https://github.com/fabricjs/fabric.js/pull/8449)

src/__types__.ts

@@ -20,5 +20,3 @@ export type StaticCanvas = Record<string, any> & {
   };
   getRetinaScaling(): number;
 } & Observable;
-export type Rect = any;
-export type TObject = any;

src/brushes/spray_brush.class.ts

@@ -1,13 +1,15 @@
 import { fabric } from '../../HEADER';
 import { Point } from '../point.class';
+import { Group } from '../shapes/group.class';
+import { Rect } from '../shapes/rect.class';
 import { getRandomInt } from '../util/internals';
-import { Canvas, Rect } from '../__types__';
+import { Canvas } from '../__types__';
 import { BaseBrush } from './base_brush.class';
 
 /**
  * @todo remove transient
  */
-const { Group, Rect, Shadow } = fabric;
+const { Shadow } = fabric;
 
 export type SprayBrushPoint = {
   x: number;

src/mixins/collection.mixin.ts

@@ -1,7 +1,9 @@
 import { fabric } from '../../HEADER';
 import type { FabricObject } from '../shapes/fabricObject.class';
 
-export function createCollectionMixin(Klass: { new (...args: any[]): any }) {
+export function createCollectionMixin<T extends { new (...args: any[]): any }>(
+  Klass: T
+) {
   return class Collection extends Klass {
     /**
      * @type {FabricObject[]}
@@ -123,7 +125,7 @@ export function createCollectionMixin(Klass: { new (...args: any[]): any }) {
     /**
      * Returns true if collection contains an object.\
      * **Prefer using {@link `FabricObject#isDescendantOf`} for performance reasons**
-     * instead of a.contains(b) use b.isDescendantOf(a)
+     * instead of `a.contains(b)` use `b.isDescendantOf(a)`
      * @param {Object} object Object to check against
      * @param {Boolean} [deep=false] `true` to check all descendants, `false` to check only `_objects`
      * @return {Boolean} `true` if collection contains an object
@@ -133,7 +135,9 @@ export function createCollectionMixin(Klass: { new (...args: any[]): any }) {
         return true;
       } else if (deep) {
         return this._objects.some(
-          (obj) => obj instanceof Collection && obj.contains(object, true)
+          (obj) =>
+            obj instanceof Collection &&
+            (obj as unknown as Collection).contains(object, true)
         );
       }
       return false;

src/mixins/object_interactivity.mixin.ts

@@ -79,7 +79,7 @@ export class InteractiveFabricObject extends FabricObject {
    * Constructor
    * @param {Object} [options] Options object
    */
-  constructor(options: Record<string, unknown>) {
+  constructor(options?: Record<string, unknown>) {
     super(options);
   }