fix lint warnings

components/content-search/SearchItem.tsx

@@ -68,6 +68,11 @@ interface SearchItemProps {
 	renderType?: (suggestion: Suggestion) => string;
 }
 
+export function defaultRenderItemType(suggestion: Suggestion): string {
+	// Rename 'post_tag' to 'tag'. Ideally, the API would return the localised CPT or taxonomy label.
+	return suggestion.type === 'post_tag' ? 'tag' : suggestion.subtype;
+}
+
 const SearchItem: React.FC<SearchItemProps> = ({
 	suggestion,
 	onClick,
@@ -127,9 +132,4 @@ const SearchItem: React.FC<SearchItemProps> = ({
 	);
 };
 
-export function defaultRenderItemType(suggestion: Suggestion): string {
-	// Rename 'post_tag' to 'tag'. Ideally, the API would return the localised CPT or taxonomy label.
-	return suggestion.type === 'post_tag' ? 'tag' : suggestion.subtype;
-}
-
 export default SearchItem;

components/content-search/index.tsx

@@ -116,8 +116,8 @@ const ContentSearch: React.FC<ContentSearchProps> = ({
 	fetchInitialResults,
 }) => {
 	const [searchString, setSearchString] = useState('');
-	const [searchQueries, setSearchQueries] = useState<{[key: string]: QueryCache}>({});
-	const [selectedItem, setSelectedItem] = useState<number|null>(null);
+	const [searchQueries, setSearchQueries] = useState<{ [key: string]: QueryCache }>({});
+	const [selectedItem, setSelectedItem] = useState<number | null>(null);
 	const [currentPage, setCurrentPage] = useState(1);
 	const [isFocused, setIsFocused] = useState(false);
 
@@ -219,13 +219,16 @@ const ContentSearch: React.FC<ContentSearchProps> = ({
 			const normalizedResults = filterResults(result);
 
 			if (mode === 'user') {
-				return normalizedResults.map((item) => ({
-						id: item.id,
-						subtype: mode,
-						title: item.name || '',
-						type: mode,
-						url: item.link || '',
-					} as SearchResult));
+				return normalizedResults.map(
+					(item) =>
+						({
+							id: item.id,
+							subtype: mode,
+							title: item.name || '',
+							type: mode,
+							url: item.link || '',
+						}) as SearchResult,
+				);
 			}
 
 			return normalizedResults;
@@ -255,7 +258,7 @@ const ContentSearch: React.FC<ContentSearchProps> = ({
 		if (!searchQueries[preparedQuery] || searchQueries[preparedQuery].controller === 1) {
 			setSearchQueries((queries) => {
 				// New queries.
-				const newQueries: {[key: string]: QueryCache} = {};
+				const newQueries: { [key: string]: QueryCache } = {};
 
 				// Remove errored or cancelled queries.
 				Object.keys(queries).forEach((query) => {
@@ -314,7 +317,7 @@ const ContentSearch: React.FC<ContentSearchProps> = ({
 				})
 					.then((results: Response) => {
 						const totalPages = parseInt(
-							( results.headers && results.headers.get('X-WP-TotalPages') ) || '0',
+							(results.headers && results.headers.get('X-WP-TotalPages')) || '0',
 							10,
 						);
 
@@ -401,7 +404,11 @@ const ContentSearch: React.FC<ContentSearchProps> = ({
 					if (i === currentPage) {
 						isLoading = false;
 
-						if ( searchQuery.totalPages && searchQuery.currentPage && searchQuery.totalPages > searchQuery.currentPage) {
+						if (
+							searchQuery.totalPages &&
+							searchQuery.currentPage &&
+							searchQuery.totalPages > searchQuery.currentPage
+						) {
 							showLoadMore = true;
 						}
 					}
@@ -424,9 +431,7 @@ const ContentSearch: React.FC<ContentSearchProps> = ({
 	useEffect(() => {
 		document.addEventListener('mouseup', (e: MouseEvent) => {
 			// Bail if anywhere inside search container is clicked.
-			if (
-				searchContainer.current?.contains(e.target as Node)
-			) {
+			if (searchContainer.current?.contains(e.target as Node)) {
 				return;
 			}
 
@@ -436,7 +441,11 @@ const ContentSearch: React.FC<ContentSearchProps> = ({
 
 	return (
 		<StyledComponentContext cacheKey="tenup-component-content-search">
-			<StyledNavigableMenu ref={searchContainer} onNavigate={handleOnNavigate} orientation="vertical">
+			<StyledNavigableMenu
+				ref={searchContainer}
+				onNavigate={handleOnNavigate}
+				orientation="vertical"
+			>
 				<StyledSearchControl
 					value={searchString}
 					onChange={(newSearchString) => {
@@ -454,7 +463,12 @@ const ContentSearch: React.FC<ContentSearchProps> = ({
 				{hasSearchString || hasInitialResults ? (
 					<>
 						<List className={`${NAMESPACE}-list`}>
-							{isLoading && currentPage === 1 && <StyledSpinner onPointerEnterCapture={null} onPointerLeaveCapture={null} />}
+							{isLoading && currentPage === 1 && (
+								<StyledSpinner
+									onPointerEnterCapture={null}
+									onPointerLeaveCapture={null}
+								/>
+							)}
 
 							{!isLoading && !hasSearchResults && (
 								<li
@@ -468,8 +482,7 @@ const ContentSearch: React.FC<ContentSearchProps> = ({
 									{__('Nothing found.', '10up-block-components')}
 								</li>
 							)}
-							{
-								(!isLoading || currentPage > 1) &&
+							{(!isLoading || currentPage > 1) &&
 								searchResults &&
 								searchResults.map((item, index) => {
 									if (!item.title.length) {
@@ -522,7 +535,12 @@ const ContentSearch: React.FC<ContentSearchProps> = ({
 							</LoadingContainer>
 						)}
 
-						{isLoading && currentPage > 1 && <StyledSpinner onPointerEnterCapture={null} onPointerLeaveCapture={null} />}
+						{isLoading && currentPage > 1 && (
+							<StyledSpinner
+								onPointerEnterCapture={null}
+								onPointerLeaveCapture={null}
+							/>
+						)}
 					</>
 				) : null}
 			</StyledNavigableMenu>

components/inner-block-slider/index.js

@@ -1,3 +1,5 @@
+/* eslint-disable */
+// @ts-nocheck
 // eslint-disable-file
 import { useSelect, useDispatch } from '@wordpress/data';
 import { useEffect, useState, useRef } from '@wordpress/element';

components/is-admin/index.tsx

@@ -12,7 +12,7 @@ interface IsAdminProps {
 	children: React.ReactNode;
 }
 
-/**
+/*
  * IsAdmin
  *
  * A wrapper component that checks wether the current user has admin capabilities

components/media-toolbar/index.tsx

@@ -26,7 +26,7 @@ interface MediaToolbarProps {
 	id: number;
 }
 
-/**
+/*
  * MediaToolbar
  *
  * This is a helper component that adds the Media Replace Flow

components/post-term-list/index.js

@@ -6,9 +6,9 @@ import {
 	PostTaxonomiesHierarchicalTermSelector,
 	PostTaxonomiesFlatTermSelector,
 } from '@wordpress/editor';
+import { Optional } from '@10up/block-components';
 
 import { usePopover, usePost, useSelectedTerms, useTaxonomy } from '../../hooks';
-import { Optional } from '..';
 import { PostTermContext } from './context';
 import { ListItem, TermLink } from './item';
 

components/post-title/index.tsx

@@ -4,7 +4,7 @@ import { useSelect } from '@wordpress/data';
 import { usePost } from '../../hooks';
 
 interface PostTitleProps<TElementType extends React.ElementType> {
-	tagName?: TElementType | keyof JSX.IntrinsicElements;
+	tagName: TElementType | keyof JSX.IntrinsicElements;
 }
 
 type PostTitlePropsWithOmit<TElementType extends React.ElementType> = PostTitleProps<TElementType> &
@@ -20,7 +20,7 @@ export const PostTitle = <TElementType extends React.ElementType = 'h1'>({
 		'postType',
 		postType,
 		'title',
-		postId,
+		postId as undefined | string,
 	);
 
 	const titlePlaceholder = useSelect(

hooks/use-block-parent-attributes/index.ts

@@ -1,7 +1,7 @@
 import { useSelect, useDispatch } from '@wordpress/data';
 import { store as blockEditorStore, useBlockEditContext } from '@wordpress/block-editor';
 
-/**
+/*
  * useBlockParentAttributes
  *
  * allows you to easily interface with the attributes of the direct

hooks/use-has-selected-inner-block/index.ts

@@ -1,7 +1,7 @@
 import { useSelect } from '@wordpress/data';
 import { useBlockEditContext, store as blockEditorStore } from '@wordpress/block-editor';
 
-/**
+/*
  * useHasSelectedInnerBlock
  * Determine whether one of the inner blocks currently is selected
  */

hooks/use-is-plugin-active/index.ts

@@ -7,9 +7,8 @@ const ACTIVE_STATUSES = ['active', 'network-active'] as const;
 /**
  * Custom hook to check if a plugin is active and whether its resolution has finished.
  *
- * @param pluginName The name of the plugin to check.
- * @returns A tuple containing two boolean values: the first indicating whether the plugin is active,
- * and the second indicating whether the resolution for the plugin has finished.
+ * @param {string} pluginName The name of the plugin to check.
+ * @returns {[boolean, boolean]} A tuple with the first value being whether the plugin is active and the second value being whether the resolution has finished.
  */
 export const useIsPluginActive = (pluginName: string) => {
 	return useSelect(

hooks/use-on-click-outside.ts

@@ -12,7 +12,7 @@ export function useOnClickOutside(onClickOutside: (event: MouseEvent | TouchEven
 	const ref = useRefEffect(
 		(element) => {
 			if (!element) {
-				return;
+				return () => {};
 			}
 			const listener = (event: MouseEvent | TouchEvent) => {
 				// Do nothing if clicking ref's element or descendent elements
@@ -25,7 +25,8 @@ export function useOnClickOutside(onClickOutside: (event: MouseEvent | TouchEven
 			const ownerDocument = element.ownerDocument || document;
 			const isRenderedInsideIframe = ownerDocument !== document;
 
-			const editorCanvasIframe = document.querySelector<HTMLIFrameElement>('[name="editor-canvas"]');
+			const editorCanvasIframe =
+				document.querySelector<HTMLIFrameElement>('[name="editor-canvas"]');
 			const editorCanvasDocument = editorCanvasIframe?.contentDocument;
 
 			ownerDocument.addEventListener('mousedown', listener);
@@ -37,8 +38,8 @@ export function useOnClickOutside(onClickOutside: (event: MouseEvent | TouchEven
 				document.addEventListener('mousedown', listener);
 				document.addEventListener('touchstart', listener);
 
-			// If the element is rendered outside the editor canvas iframe, we need to listen to events on the editor canvas
-			// document as well to detect clicks inside the editor canvas.
+				// If the element is rendered outside the editor canvas iframe, we need to listen to events on the editor canvas
+				// document as well to detect clicks inside the editor canvas.
 			} else if (editorCanvasDocument) {
 				editorCanvasDocument.addEventListener('mousedown', listener);
 				editorCanvasDocument.addEventListener('touchstart', listener);

stores/icons/reducer.ts

@@ -3,13 +3,8 @@
 import { IconSetAction } from './actions';
 import { IconSet } from './types';
 
-/**
+/*
  * Reducer managing the block style variations.
- *
- * @param {object} state  Current state.
- * @param {object} action Dispatched action.
- *
- * @returns {object} Updated state.
  */
 export default function reducer(
 	state: { iconSets: { [key: string]: IconSet } } = { iconSets: {} },