fixes for TypeScript 5.0

Signed-off-by: Patrik Oldsberg <poldsberg@gmail.com>

plugins/catalog-react/src/components/EntityAutocompletePicker/EntityAutocompletePicker.tsx

@@ -114,8 +114,9 @@ export function EntityAutocompletePicker<
     } as Partial<T>);
   }, [name, shouldAddFilter, selectedOptions, Filter, updateFilters]);
 
+  const filter = filters[name];
   if (
-    (filters[name] && !('values' in filters[name])) ||
+    (filter && typeof filter === 'object' && !('values' in filter)) ||
     !availableOptions.length
   ) {
     return null;

plugins/catalog-react/src/components/EntityRefLink/FetchedEntityRefLinks.tsx

@@ -57,11 +57,14 @@ export function FetchedEntityRefLinks<
     error,
   } = useAsync(async () => {
     const refs = entityRefs.reduce((acc, current) => {
-      return 'metadata' in current ? acc : [...acc, parseEntityRef(current)];
+      if (typeof current === 'object' && 'metadata' in current) {
+        return acc;
+      }
+      return [...acc, parseEntityRef(current)];
     }, new Array<CompoundEntityRef>());
 
     const pureEntities = entityRefs.filter(
-      ref => 'metadata' in ref,
+      ref => typeof ref === 'object' && 'metadata' in ref,
     ) as Array<Entity>;
 
     return refs.length > 0

plugins/catalog-react/src/hooks/useEntityListProvider.tsx

@@ -161,8 +161,9 @@ export const EntityListProvider = <EntityFilters extends DefaultEntityFilters>(
 
       const queryParams = Object.keys(requestedFilters).reduce(
         (params, key) => {
-          const filter: EntityFilter | undefined =
-            requestedFilters[key as keyof EntityFilters];
+          const filter = requestedFilters[key as keyof EntityFilters] as
+            | EntityFilter
+            | undefined;
           if (filter?.toQueryValue) {
             params[key] = filter.toQueryValue();
           }

plugins/cicd-statistics/src/charts/logic/daily-summary.ts

@@ -70,10 +70,14 @@ function countTriggersPerDay(builds: ReadonlyArray<Build>) {
   const values = Object.entries(days).map(([epoch, buildsThisDay]) => {
     const datapoint = Object.fromEntries(
       triggerReasons
-        .map(reason => [
-          reason,
-          buildsThisDay.filter(build => build.triggeredBy === reason).length,
-        ])
+        .map(
+          reason =>
+            [
+              reason,
+              buildsThisDay.filter(build => build.triggeredBy === reason)
+                .length,
+            ] as const,
+        )
         .filter(([_type, count]) => count > 0),
     ) as Omit<TriggerReasonsDatapoint, '__epoch'>;
 

plugins/entity-feedback/src/components/StarredRatingButtons/StarredRatingButtons.tsx

@@ -134,7 +134,7 @@ export const StarredRatingButtons = (props: StarredRatingButtonsProps) => {
   return (
     <>
       {Object.values(FeedbackRatings)
-        .filter(o => typeof o === 'number')
+        .filter((o): o is number => typeof o === 'number')
         .map(starRating => (
           <IconButton
             key={starRating}

plugins/github-issues/src/components/GithubIssues/IssuesList/IssuesList.tsx

@@ -73,7 +73,7 @@ export const IssuesList = ({
   const filteredRepos = React.useMemo(
     () =>
       issuesByRepository && activeFilter.length
-        ? activeFilter.reduce(
+        ? activeFilter.reduce<IssuesByRepo>(
             (acc, val) => ({
               [val]: issuesByRepository[val],
               ...acc,

plugins/playlist/src/hooks/usePlaylistList.tsx

@@ -172,8 +172,9 @@ export const PlaylistListProvider = <
 
       const queryParams = Object.keys(requestedFilters).reduce(
         (params, key) => {
-          const filter: PlaylistFilter | undefined =
-            requestedFilters[key as keyof PlaylistFilters];
+          const filter = requestedFilters[key as keyof PlaylistFilters] as
+            | PlaylistFilter
+            | undefined;
           if (filter?.toQueryValue) {
             params[key] = filter.toQueryValue();
           }

plugins/rollbar-backend/src/api/RollbarApi.ts

@@ -104,7 +104,7 @@ export class RollbarApi {
     );
   }
 
-  private async get<T>(url: string, accessToken?: string) {
+  private async get<T extends {}>(url: string, accessToken?: string) {
     const fullUrl = buildUrl(url);
 
     if (this.logger) {
@@ -119,7 +119,7 @@ export class RollbarApi {
       .then(json => camelcaseKeys<T>(json?.result, { deep: true }));
   }
 
-  private async getForProject<T>(
+  private async getForProject<T extends {}>(
     url: string,
     projectName: string,
     useProjectToken = true,

plugins/scaffolder-backend/src/scaffolder/tasks/NunjucksWorkflowRunner.test.ts

@@ -120,7 +120,7 @@ describe('DefaultWorkflowRunner', () => {
             foo: z.number(),
           }),
         },
-      }) as TemplateAction<unknown>,
+      }) as TemplateAction,
     );
 
     actionRegistry.register({

plugins/scaffolder-node/src/actions/createTemplateAction.ts

@@ -22,8 +22,8 @@ import { JsonObject } from '@backstage/types';
 
 /** @public */
 export type TemplateActionOptions<
-  TActionInput = {},
-  TActionOutput = {},
+  TActionInput extends JsonObject = {},
+  TActionOutput extends JsonObject = {},
   TInputSchema extends Schema | z.ZodType = {},
   TOutputSchema extends Schema | z.ZodType = {},
 > = {
@@ -48,10 +48,18 @@ export const createTemplateAction = <
   TOutputParams extends JsonObject = JsonObject,
   TInputSchema extends Schema | z.ZodType = {},
   TOutputSchema extends Schema | z.ZodType = {},
-  TActionInput = TInputSchema extends z.ZodType<any, any, infer IReturn>
+  TActionInput extends JsonObject = TInputSchema extends z.ZodType<
+    any,
+    any,
+    infer IReturn
+  >
     ? IReturn
     : TInputParams,
-  TActionOutput = TOutputSchema extends z.ZodType<any, any, infer IReturn>
+  TActionOutput extends JsonObject = TOutputSchema extends z.ZodType<
+    any,
+    any,
+    infer IReturn
+  >
     ? IReturn
     : TOutputParams,
 >(

plugins/scaffolder-node/src/actions/types.ts

@@ -75,8 +75,8 @@ export type ActionContext<
 
 /** @public */
 export type TemplateAction<
-  TActionInput = unknown,
-  TActionOutput = JsonObject,
+  TActionInput extends JsonObject = JsonObject,
+  TActionOutput extends JsonObject = JsonObject,
 > = {
   id: string;
   description?: string;

plugins/scaffolder-react/src/extensions/types.ts

@@ -64,7 +64,7 @@ export type FieldExtensionOptions<
  */
 export interface FieldExtensionComponentProps<
   TFieldReturnValue,
-  TUiOptions extends {} = {},
+  TUiOptions = unknown,
 > extends FieldProps<TFieldReturnValue> {
   uiSchema: FieldProps['uiSchema'] & {
     'ui:options'?: TUiOptions;

plugins/sonarqube/dev/index.tsx

@@ -66,9 +66,6 @@ createDevApp()
             <EntityGridItem xs={12} md={6} entity={entity('passed')}>
               <EntitySonarQubeCard />
             </EntityGridItem>
-            <EntityGridItem xs={12} entity={entity(undefined)}>
-              <EntitySonarQubeCard />
-            </EntityGridItem>
           </Grid>
         </Content>
       </Page>
@@ -79,7 +76,7 @@ createDevApp()
     deps: {},
     factory: () =>
       ({
-        getFindingSummary: async componentKey => {
+        getFindingSummary: async ({ componentKey }) => {
           switch (componentKey) {
             case 'error':
               throw new Error('Error!');

plugins/stack-overflow/src/home/StackOverflowQuestions/Content.tsx

@@ -59,7 +59,7 @@ export const Content = (props: StackOverflowQuestionsContentProps) => {
     return <Typography paragraph>could not load questions</Typography>;
   }
 
-  const getSecondaryText = (answer_count: Number) =>
+  const getSecondaryText = (answer_count: number) =>
     answer_count > 1 ? `${answer_count} answers` : `${answer_count} answer`;
 
   return (