feat!: Remove deprecations (#2060)

Author: tassoevan

.changeset/little-schools-grow.md

@@ -0,0 +1,15 @@
+---
+'@rocket.chat/fuselage-forms': minor
+'@rocket.chat/fuselage-hooks': minor
+'@rocket.chat/fuselage': minor
+'@rocket.chat/layout': minor
+'@rocket.chat/storybook-dark-mode': patch
+'@rocket.chat/fuselage-toastbar': patch
+'@rocket.chat/onboarding-ui': patch
+'@rocket.chat/css-in-js': patch
+'@rocket.chat/emitter': patch
+'@rocket.chat/styled': patch
+'@rocket.chat/logo': patch
+---
+
+feat: Remove deprecations and keep documentation comments

packages/css-in-js/css-in-js.api.md

@@ -7,7 +7,7 @@
 import { cssSupports } from '@rocket.chat/css-supports';
 import type { Middleware } from 'stylis';
 
-// @public (undocumented)
+// @public
 export const attachRules: RuleAttacher;
 
 // @public (undocumented)
@@ -16,7 +16,7 @@ export const createAnimationName: (content: string) => string;
 // @public (undocumented)
 export const createClassName: (content: string) => string;
 
-// @public (undocumented)
+// @public
 export const css: (slices: TemplateStringsArray, ...values: readonly unknown[]) => cssFn;
 
 // @public (undocumented)
@@ -27,10 +27,10 @@ export { cssSupports }
 // @public (undocumented)
 export const escapeName: (animationOrClassName: string) => string;
 
-// @public (undocumented)
+// @public
 export type Evaluable = <T extends readonly unknown[]>(...args: T) => string;
 
-// @public (undocumented)
+// @public
 export const keyframes: (slices: TemplateStringsArray, ...values: unknown[]) => keyframesFn;
 
 // @public (undocumented)
@@ -41,7 +41,7 @@ export type RuleAttacher = (rules: string, options?: {
     document?: Document;
 }) => () => void;
 
-// @public (undocumented)
+// @public
 export const transpile: (selector: string, content: string, middleware?: Middleware) => string;
 
 // (No @packageDocumentation comment for this package)

packages/emitter/emitter.api.md

@@ -18,19 +18,13 @@ export type AnyEventTypeOf<EventMap extends DefaultEventMap> = keyof EventMap;
 // @public (undocumented)
 export type DefaultEventMap = Record<string | symbol, any>;
 
-// @public (undocumented)
+// @public
 export class Emitter<EventMap extends DefaultEventMap = DefaultEventMap> implements IEmitter<EventMap> {
-    // (undocumented)
     emit<T extends AnyEventOf<EventMap>, EventType extends AnyEventTypeOf<EventMap> = EventTypeOf<EventMap, T>>(type: EventType, ...input: EventOf<EventMap, EventType> extends void ? [undefined?] : [EventOf<EventMap, EventType>]): void;
-    // (undocumented)
     events(): AnyEventTypeOf<EventMap>[];
-    // (undocumented)
     has(key: AnyEventTypeOf<EventMap>): boolean;
-    // (undocumented)
     off<T extends AnyEventOf<EventMap>, EventType extends AnyEventTypeOf<EventMap> = EventTypeOf<EventMap, T>>(type: EventType, handler: EventHandlerOf<EventMap, EventType>): void;
-    // (undocumented)
     on<T extends AnyEventOf<EventMap>, TType extends AnyEventTypeOf<EventMap> = EventTypeOf<EventMap, T>>(type: TType, handler: EventHandlerOf<EventMap, TType>): OffCallbackHandler;
-    // (undocumented)
     once<T extends AnyEventOf<EventMap>, EventType extends AnyEventTypeOf<EventMap> = EventTypeOf<EventMap, T>>(type: EventType, handler: EventHandlerOf<EventMap, EventType>): OffCallbackHandler;
 }
 

packages/fuselage-forms/fuselage-forms.api.md

@@ -24,7 +24,6 @@ import { FieldLabelInfo } from '@rocket.chat/fuselage';
 import { FieldLink } from '@rocket.chat/fuselage';
 import { FieldRow } from '@rocket.chat/fuselage';
 import { IconProps } from '@rocket.chat/fuselage';
-import { InputBoxProps } from '@rocket.chat/fuselage';
 import { JSX } from 'react';
 import { Key } from 'react';
 import { MouseEventHandler } from 'react';
@@ -40,6 +39,7 @@ import { SetStateAction } from 'react';
 import { Slider as Slider_2 } from '@rocket.chat/fuselage';
 import { TelephoneInputProps } from '@rocket.chat/fuselage';
 import { TextAreaInputProps } from '@rocket.chat/fuselage';
+import { TextInputProps } from '@rocket.chat/fuselage';
 import { UrlInputProps } from '@rocket.chat/fuselage';
 
 // @public (undocumented)
@@ -264,10 +264,7 @@ export const TextAreaInput: {
 
 // @public (undocumented)
 export const TextInput: {
-    (props: Omit<InputBoxProps, "ref" | "type"> & RefAttributes<HTMLInputElement> & {
-        addon?: ReactNode;
-        error?: string;
-    }): JSX.Element;
+    (props: TextInputProps & RefAttributes<any>): JSX.Element;
     displayName: string;
 };
 

packages/fuselage-hooks/fuselage-hooks.api.md

@@ -163,9 +163,6 @@ Dispatch<A> & {
 // @public
 export const useDebouncedValue: <V>(value: V, delay: number) => V;
 
-// @public @deprecated
-export const useEffectEvent: <TFunction extends (...args: any[]) => any>(fn: TFunction) => (...args: TFunction extends (...args: infer P) => any ? P : never) => TFunction extends (...args: any) => infer T ? T : never;
-
 // @public (undocumented)
 export const useElementIsVisible: <T extends Element>() => [ref: RefObject<T | null>, isVisible: boolean];
 
@@ -187,9 +184,6 @@ export const useMediaQuery: (query?: string) => boolean;
 // @public
 export const useMergedRefs: <T>(...refs: (Ref<T> | null | undefined)[]) => RefCallback<T>;
 
-// @public @deprecated
-export const useMutableCallback: <TFunction extends (...args: any[]) => any>(fn: TFunction) => (...args: TFunction extends (...args: infer P) => any ? P : never) => TFunction extends (...args: any) => infer T ? T : never;
-
 // @public
 export function useOutsideClick<T extends Element>(elements: RefObject<T | null>[], cb: (e: MouseEvent) => void): void;
 

packages/fuselage-hooks/src/useStableCallback.ts

@@ -30,23 +30,3 @@ export const useStableCallback = <TFunction extends (...args: any[]) => any>(
 
   return stableFnRef.current;
 };
-
-/**
- * Hook to create a stable callback from a mutable one.
- *
- * @deprecated use {@link useStableCallback} instead
- * @param fn - the mutable callback
- * @returns a stable callback
- * @public
- */
-export const useMutableCallback = useStableCallback;
-
-/**
- * Hook to create a stable callback from a mutable one.
- *
- * @deprecated use {@link useStableCallback} instead
- * @param fn - the mutable callback
- * @returns a stable callback
- * @public
- */
-export const useEffectEvent = useStableCallback;

packages/fuselage-hooks/tsconfig.build.json

@@ -1,8 +1,7 @@
 {
   "extends": "./tsconfig.json",
   "compilerOptions": {
-    "rootDir": "./src",
-    "removeComments": false
+    "rootDir": "./src"
   },
   "exclude": ["**/*.spec.*", "jest.config.*"]
 }