BUG: Incorrect type definition for StyleManager.addProperty

[duclet]

GrapesJS version

• I confirm to use the latest version of GrapesJS

What browser are you using?

All

Reproducible demo link

None

Describe the bug

How to reproduce the bug?

1. Per the definition here: https://github.com/GrapesJS/grapesjs/blob/dev/src/style_manager/index.ts#L304, you should be providing the options as options for PropertyProps.
2. However, PropertyProps as defined here: https://github.com/GrapesJS/grapesjs/blob/dev/src/style_manager/model/Property.ts#L9, does not have such a property. This is causing the TS compiler to complain.

What is the expected behavior?
Definition should exists.

What is the current behavior?
Definition is missing.

Code of Conduct

• I agree to follow this project's Code of Conduct

[SachinPuthiyoth]

Seems like the type definition in StyleManagerConfig.sectors is unchanged :

grapesjs/src/style_manager/config/config.ts

Line 8 in 5c89044

sectors?: (Omit<SectorProperties, 'properties'> & { properties?: (string | PropertyProps)[] })[];

Shouldn't we be updating this to use PropertyTypes as in :

grapesjs/src/style_manager/index.ts

Line 86 in 5c89044

export type PropertyTypes = PropertyStackProps | PropertySelectProps | PropertyNumberProps;

Opened an issue here - #5613
Shall I raise a PR for the same?