Add the possibility to config the Markdown parser to add heading id attributes and load CommonMark extensions

[giuscris]

No description provided.

[Copilot]

Pull Request Overview

This PR enhances the Markdown parser by centralizing option handling, enabling automatic heading ID generation, and allowing extra CommonMark extensions to be loaded.

• Introduce parseOptions() to extract parser configuration and extensions from input options
• Add support for addHeadingIds via DefaultAttributesExtension and slugified IDs
• Allow custom CommonMark extensions through a new commonmarkExtensions option

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
formwork/src/Parsers/Markdown.php	Refactored parse() to use parseOptions(), load default attributes and user-specified extensions
formwork/src/Parsers/Extensions/CommonMark/FormworkExtension.php	Removed obsolete allowHtml schema entry
formwork/fields/markdown.php	Updated invocation sites to pass new options
formwork/config/views/methods.php	Updated invocation sites to pass new options
formwork/config/system.yaml	Added addHeadingIds and commonmarkExtensions defaults

Comments suppressed due to low confidence (3)

formwork/src/Parsers/Markdown.php:40

• [nitpick] The variable name addedExtensions could be more descriptive (e.g., loadedExtensions) to clarify that it tracks which extensions have been registered.

        $addedExtensions = [];

formwork/src/Parsers/Markdown.php:70

• The docblock for parseOptions() should list the recognized option keys (allowHtml, addHeadingIds, commonmarkExtensions) and their expected types to help future maintainers.

    private static function parseOptions(array $options): array

formwork/src/Parsers/Markdown.php:87

• New behavior for addHeadingIds and slug generation is untested; please add unit tests to cover when this flag is enabled and verify generated IDs.

        if ($options['addHeadingIds']) {

[Copilot]

Flattening all extension config directly into the root config may inadvertently override unrelated settings; consider merging nested configurations under their own namespace or using deep merge.

Suggested change

	$config = [...$config, ...$extensionConfig];
	$config['extensions'] = [...($config['extensions'] ?? []), ...$extensionConfig];