API Extractor not able to parse "import * as __ from __"

[marlass]

Current solution for namespaces that we use relies on export * as grouping them.
Different solution is to rely on namespaces provided by TS.

The current solution makes it impossible to use API extractor for finding breaking changes in our API.

We can try if namespaces are usable solution for this case at all or we should completely drop namespacing actions/selectors and expose them directly.

To consider:

• splitting namespaces in multiple files
• refactoring (problems, ease of catching errors)
• namespaces support by API extractor

[Platonn]

It looks that the issue with import * as _ from _ has been fixed recently in PR microsoft/rushstack#1796
(available in 7.17.0 microsoft/rushstack#1029 (comment))

So we should change version referenced in this line:

spartacus/.github/api-extractor-action/src/setup.ts

Line 24 in 30f87d7

await exec.exec('npm', ['i', '-g', '@microsoft/api-extractor@^7.12.0']);

And see how the report will change

[Platonn]

btw. another problem with api-extractor ERROR: The expression contains an import() type, which is not yet supported by API Extractor: also seems to be solved in 7.18.0! See microsoft/rushstack#2140 (comment)

[marlass]

Our current setup automatically picked up these changes as 7.18 matches the semver we used in the install.

There are however a couple of things that we can now improve:

• remove some sections from the generated comments (entry points that can't be analyzed, debugging instructions)
• cleanup code related to this specific error handling we did for these errors