chore: upgrade glimmer/component for types to work properly

Author: shleewhite

.changeset/khaki-bananas-know.md

@@ -2,4 +2,12 @@
 "@hashicorp/design-system-components": patch
 ---
 
-Dependency ember-a11y-refocus upgraded
+Dependency `ember-a11y-refocus` and `@glimmer/component` upgraded.
+
+<!-- START components/side-nav -->
+`SideNav` - Fixed the type of `@a11yRefocusNavigationText` to match the expected type from `ember-a11y-refocus`. The new type is `(transition: Transition) => boolean` instead of `string`.
+<!-- END -->
+
+<!-- START components/app-header -->
+`AppHeader` - Fixed the type of `@a11yRefocusNavigationText` to match the expected type from `ember-a11y-refocus`. The new type is `(transition: Transition) => boolean` instead of `string`.
+<!-- END -->

packages/components/package.json

@@ -85,7 +85,7 @@
     "@ember/test-helpers": "^4.0.4",
     "@embroider/addon-dev": "^7.1.5",
     "@eslint/js": "^9.23.0",
-    "@glimmer/component": "^1.1.2",
+    "@glimmer/component": "^2.0.0",
     "@glint/core": "^1.5.2",
     "@glint/environment-ember-loose": "^1.5.2",
     "@glint/environment-ember-template-imports": "^1.5.2",

packages/components/src/components/hds/advanced-table/th-resize-handle.ts

@@ -9,6 +9,7 @@ import { action } from '@ember/object';
 import { modifier } from 'ember-modifier';
 import { requestAnimationFrameWaiter } from './utils.ts';
 import { BORDER_WIDTH } from './index.ts';
+import type Owner from '@ember/owner';
 
 import type HdsAdvancedTableColumn from './models/column.ts';
 import type { HdsAdvancedTableSignature } from './index.ts';
@@ -90,7 +91,7 @@ export default class HdsAdvancedTableThResizeHandle extends Component<HdsAdvance
   );
 
   constructor(
-    owner: unknown,
+    owner: Owner,
     args: HdsAdvancedTableThResizeHandleSignature['Args']
   ) {
     super(owner, args);

packages/components/src/components/hds/app-header/index.hbs

@@ -10,7 +10,6 @@
   ...attributes
 >
   {{#if (and this.hasA11yRefocus (not this._isOpen))}}
-    {{! @glint-expect-error - `ember-a11y-refocus` doesn't expose types yet }}
     <NavigationNarrator
       @routeChangeValidator={{@a11yRefocusRouteChangeValidator}}
       @skipTo={{this.a11yRefocusSkipTo}}

packages/components/src/components/hds/app-header/index.ts

@@ -9,6 +9,7 @@ import { guidFor } from '@ember/object/internals';
 import { tracked } from '@glimmer/tracking';
 import { registerDestructor } from '@ember/destroyable';
 import type Owner from '@ember/owner';
+import type {NavigationNarratorSignature} from 'ember-a11y-refocus/components/navigation-narrator';
 
 import { hdsBreakpoints } from '../../../utils/hds-breakpoints.ts';
 
@@ -19,7 +20,7 @@ export interface HdsAppHeaderSignature {
     a11yRefocusSkipTo?: string;
     a11yRefocusSkipText?: string;
     a11yRefocusNavigationText?: string;
-    a11yRefocusRouteChangeValidator?: string;
+    a11yRefocusRouteChangeValidator?: NavigationNarratorSignature['Args']['routeChangeValidator'];
     a11yRefocusExcludeAllQueryParams?: boolean;
   };
   Blocks: {

packages/components/src/components/hds/side-nav/index.hbs

@@ -20,7 +20,6 @@
 >
   <:root>
     {{#if this.hasA11yRefocus}}
-      {{! @glint-expect-error - `ember-a11y-refocus` doesn't expose types yet }}
       <NavigationNarrator
         @routeChangeValidator={{@a11yRefocusRouteChangeValidator}}
         @skipTo={{this.a11yRefocusSkipTo}}

packages/components/src/components/hds/side-nav/index.ts

@@ -11,6 +11,7 @@ import { registerDestructor } from '@ember/destroyable';
 
 import type { HdsSideNavBaseSignature } from './base';
 import type Owner from '@ember/owner';
+import type {NavigationNarratorSignature} from 'ember-a11y-refocus/components/navigation-narrator';
 
 export interface HdsSideNavSignature {
   Args: {
@@ -21,7 +22,7 @@ export interface HdsSideNavSignature {
     a11yRefocusSkipTo?: string;
     a11yRefocusSkipText?: string;
     a11yRefocusNavigationText?: string;
-    a11yRefocusRouteChangeValidator?: string;
+    a11yRefocusRouteChangeValidator?: NavigationNarratorSignature['Args']['routeChangeValidator'];
     a11yRefocusExcludeAllQueryParams?: boolean;
     /**
      * @deprecated The `@ariaLabel` argument for "Hds::SideNav" has been deprecated. It is replaced by aria-labelledby and aria-expanded on the toggle button

packages/components/unpublished-development-types/global.d.ts

@@ -15,6 +15,7 @@ import type { EmbroiderUtilRegistry } from '@embroider/util';
 import type RenderModifiersRegistry from '@ember/render-modifiers/template-registry';
 import type EmberComposableHelpersRegistry from '@nullvoxpopuli/ember-composable-helpers/template-registry';
 import type EmberIntlRegistry from 'ember-intl/template-registry';
+import type EmberA11yRefocusRegistry from 'ember-a11y-refocus/template-registry';
 
 declare module '@glint/environment-ember-loose/registry' {
   export default interface Registry
@@ -28,6 +29,7 @@ declare module '@glint/environment-ember-loose/registry' {
       RenderModifiersRegistry,
       EmberComposableHelpersRegistry,
       EmbroiderUtilRegistry,
+      EmberA11yRefocusRegistry,
       EmberIntlRegistry /*, other addon registries */ {
     // local entries
     Portal: typeof Portal;

pnpm-lock.yaml

@@ -49,7 +49,7 @@
     "@embroider/macros": "^1.18.1",
     "@embroider/test-setup": "^4.0.0",
     "@eslint/js": "^9.27.0",
-    "@glimmer/component": "^1.1.2",
+    "@glimmer/component": "^2.0.0",
     "@glimmer/tracking": "^1.1.2",
     "@glint/core": "^1.5.2",
     "@glint/environment-ember-loose": "^1.5.2",