Description

This PR enables the noUncheckedIndexedAccess TypeScript compiler option as suggested by @Emyrk.

This option makes array/object indexed access return T | undefined instead of T, improving type safety by ensuring we properly handle cases where indexed access might return undefined.

Progress

Error Breakdown (Original 257)

Remaining Work

The 132 remaining errors are primarily in:

• Test files (.jest.tsx, .test.ts)
• Storybook stories (.stories.tsx)
• Some production code in pages/

Most fixes involve adding:

• Non-null assertion (!) where we're certain the value exists
• Optional chaining (?.) where appropriate
• Guard clauses for genuine undefined checks

Why This Change?

Before this option, TypeScript would assume indexed access always succeeds:

const item = array[0]; // Type: T (assumed to exist)
item.property; // No error, but could crash at runtime!

After this option:

const item = array[0]; // Type: T | undefined (safer)
item.property; // Error! Must check first
if (item) {
  item.property; // OK
}

⚠️ Draft PR - This is a work in progress. The remaining 132 errors need to be fixed before this can be merged.