[Bug]: several peerDeps are flagged as optional in @o3r/eslint-plugin but usage of it are exposed at the root

[matthieu-crouzet]

Package name

eslint-plugin

Package version

11.4.1

Reproduction steps

Use @o3r/eslint-plugin without having yaml-eslint-parser installed.

Current result

Issue while running linter because yaml-eslint-parser is not installed.

Expected result

To not introduce breaking changes, in v11 we can remove the optional flags.
For v12 we can expose sub-entries like @o3r/eslint-plugin/yaml

Additional comments

Same issue for jsonc-eslint-parser and @angular-eslint/template-parser

[matthieu-crouzet]

Issue referenced in packages/@o3r/eslint-config/schematics/ng-add/index.ts

[kpanot]

Hi,

Can you precise the issue on jsonc-eslint-parser and @angular-eslint/template-parser?
Looking at the code:

• jsonc-eslint-parser seems to be properly flagged as optional, it is used only for typing in

  otter/packages/@o3r/eslint-plugin/src/rules/json/json-dependency-versions-harmonize/json-dependency-versions-harmonize.ts

  Line 5 in 4497355

  import type { AST } from 'jsonc-eslint-parser';

• @angular-eslint/template-parser is not used in the code and it is made to request a compatible option of @angular-eslint/template-parser to ensure the plugin to be tested on this version.

Regarding yaml-eslint-parser, it is properly turned optional in #2354 (see code diff: https://github.com/AmadeusITGroup/otter/pull/2354/files#diff-5faa91646fd6988a6a6817d4291e4a7fba68663d540cf9afa89fc12b9947c4e7)