From 3ecf73e471f92e67c7a46a35b6a16ca61cda02e5 Mon Sep 17 00:00:00 2001
From: Marie Lucca <francinelucca@github.com>
Date: Tue, 1 Oct 2024 12:36:23 -0400
Subject: [PATCH 1/2] fix(Banner): rewrite PrimaryAction & SecondaryAction
 types

---
 packages/react/src/Banner/Banner.tsx | 27 +++++++++++++++++----------
 1 file changed, 17 insertions(+), 10 deletions(-)

diff --git a/packages/react/src/Banner/Banner.tsx b/packages/react/src/Banner/Banner.tsx
index 2a9bbfd459b..a34002cb238 100644
--- a/packages/react/src/Banner/Banner.tsx
+++ b/packages/react/src/Banner/Banner.tsx
@@ -1,14 +1,15 @@
 import {clsx} from 'clsx'
-import React, {useEffect} from 'react'
+import React, {forwardRef, useEffect} from 'react'
 import styled from 'styled-components'
 import {AlertIcon, InfoIcon, StopIcon, CheckCircleIcon, XIcon} from '@primer/octicons-react'
-import {Button, IconButton} from '../Button'
+import {Button, IconButton, type ButtonProps} from '../Button'
 import {get} from '../constants'
 import {VisuallyHidden} from '../VisuallyHidden'
 import {useMergedRefs} from '../internal/hooks/useMergedRefs'
 import {useFeatureFlag} from '../FeatureFlags'
 import classes from './Banner.module.css'
 import {toggleStyledComponent} from '../internal/utils/toggleStyledComponent'
+import type {ForwardRefComponent as PolymorphicForwardRefComponent} from '../utils/polymorphic'
 
 type BannerVariant = 'critical' | 'info' | 'success' | 'upsell' | 'warning'
 
@@ -474,22 +475,28 @@ export function BannerActions({primaryAction, secondaryAction}: BannerActionsPro
   )
 }
 
-export type BannerPrimaryActionProps = Omit<React.ComponentPropsWithoutRef<typeof Button>, 'variant'>
+export type BannerPrimaryActionProps = Omit<ButtonProps, 'variant'>
 
-export function BannerPrimaryAction({children, className, ...rest}: BannerPrimaryActionProps) {
+const BannerPrimaryAction = forwardRef(({children, className, ...rest}, forwardedRef) => {
   return (
-    <Button className={clsx('BannerPrimaryAction', className)} variant="default" {...rest}>
+    <Button ref={forwardedRef} className={clsx('BannerPrimaryAction', className)} variant="default" {...rest}>
       {children}
     </Button>
   )
-}
+}) as PolymorphicForwardRefComponent<'button', BannerPrimaryActionProps>
+
+BannerPrimaryAction.displayName = 'BannerPrimaryAction'
 
-export type BannerSecondaryActionProps = Omit<React.ComponentPropsWithoutRef<typeof Button>, 'variant'>
+export type BannerSecondaryActionProps = Omit<ButtonProps, 'variant'>
 
-export function BannerSecondaryAction({children, className, ...rest}: BannerSecondaryActionProps) {
+const BannerSecondaryAction = forwardRef(({children, className, ...rest}, forwardedRef) => {
   return (
-    <Button className={clsx('BannerPrimaryAction', className)} variant="link" {...rest}>
+    <Button ref={forwardedRef} className={clsx('BannerPrimaryAction', className)} variant="link" {...rest}>
       {children}
     </Button>
   )
-}
+}) as PolymorphicForwardRefComponent<'button', BannerSecondaryActionProps>
+
+BannerSecondaryAction.displayName = 'BannerSecondaryAction'
+
+export {BannerPrimaryAction, BannerSecondaryAction}

From 1f9bc822408bf1ca57f395663e8aaee4ce1440d4 Mon Sep 17 00:00:00 2001
From: Marie Lucca <40550942+francinelucca@users.noreply.github.com>
Date: Tue, 1 Oct 2024 13:03:14 -0400
Subject: [PATCH 2/2] Create modern-cooks-invite.md

---
 .changeset/modern-cooks-invite.md | 5 +++++
 1 file changed, 5 insertions(+)
 create mode 100644 .changeset/modern-cooks-invite.md

diff --git a/.changeset/modern-cooks-invite.md b/.changeset/modern-cooks-invite.md
new file mode 100644
index 00000000000..df4fde36b37
--- /dev/null
+++ b/.changeset/modern-cooks-invite.md
@@ -0,0 +1,5 @@
+---
+"@primer/react": patch
+---
+
+fix(Banner): rewrite PrimaryAction & SecondaryAction types