refactor(macro): use message descriptor for Trans

[timofei-iatsenko]

Description

Use the same message descriptor with /* i18n */ extraction mark in <Trans> component so extractor will support TransNoContext or runtimeConfigModule without explicit knowledge about them.

Take the input:

<Trans>Hello World</Trans>

Before

<Trans id={"mY42CM"} message={"Hello World"} />;

After

<Trans
  {
    /*i18n*/
    ...{
      id: "mY42CM",
      message: "Hello World",
    }
  }
/>

Where

 /*i18n*/
    ...{
      id: "mY42CM",
      message: "Hello World",
    }

Is standard message descriptor used in t transformation.

A side effect of this change is reduced code duplication between js and jsx macro.

Types of changes

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Examples update

Fixes # (issue)

Checklist

• I have read the CONTRIBUTING and CODE_OF_CONDUCT docs
• I have added tests that prove my fix is effective or that my feature works
• I have added the necessary documentation (if appropriate)

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
js-lingui	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 12, 2024 10:41am

[github-actions]

size-limit report 📦

Path	Size
./packages/core/dist/index.mjs	2.86 KB (0%)
./packages/detect-locale/dist/index.mjs	723 B (0%)
./packages/react/dist/index.mjs	1.65 KB (0%)
./packages/remote-loader/dist/index.mjs	7.26 KB (0%)

[timofei-iatsenko]

this added intentionally to help IDE to understand that this file is executable jest test, otherwise it doesnt have an option to run tests in this file: