fix: remove redundant overrides causing npm install to fail

[Iron-56]

Fixes #1031

Changes

• Removed react and react-dom override

PR Checklist

• npm run lint passes
• Inline documentation is included / updated
• Unit tests are included / updated

Problem

Running npm install fails with the following error:

The npm:@preact/compat@latest syntax in the overrides section of package.json causes compatibility issues, preventing contributors from installing dependencies.

Solution

The Preact alias overrides are actually redundant. The Astro Preact integration automatically handles react compatibility using compat: true option of astro.config.mjs:

preact({
  compat: true,  // aliases react
}),

[Iron-56]

Can you look onto this PR @ksen0 ? I am not sure if its the correct branch to merge to. Thank you!

[limzykenneth]

I'm not 100% sure this is redundant/no needed. The override is specifically for using react component libraries (such as @uiw/react-codemirror) with preact, as described here: https://docs.astro.build/en/guides/integrations-guide/preact/#compat

When importing React component libraries, in order to swap out the react and react-dom dependencies as preact/compat, you can use overrides to do so.

Is that not the case?

[Iron-56]

Yeah it was my npm's issue, I recently updated and it's fixed.