fix: custom titles of code snippets inside code groups

[Raiondesu]

This PR fixes #1785, by allowing the following syntax when a snippet is placed in a code group:

::: code-group

<<< @/snippets/snippet.js [snippet.js]

<!-- with custom tab titles -->
<<< @/snippets/snippet-with-region.js [snippet with region custom title]

<!-- filename is now grabbed by-default -->
<<< @/snippets/snippet.cs{1,2,4-6 c#}

:::

See updated docs for an example.

[sky124380729]

when would it be merged, i need this feature

[Raiondesu]

@brc-dd, please take a look at this 🙏

It's a small fix, but oh so useful

[MarkusKeck]

I would also appreciate if this gets merged.
Does the proposed solution allow to include icons (in my case fontawesome) as description, judging from the regex it does not?
The current implementation does allow it. I would suggest a uniform behavior.

Example (Changed ` for ' so it gets displayed correctly)

'''java [<i class="fa-sharp fa-solid fa-flag-checkered"></i> Alternate Solution]
 ... content
'''

[Raiondesu]

@MarkusKeck, technically the regex doesn't disallow the markup inside the [title] brackets, though I didn't test for this.

This PR's code simply passes the parsed title to already existing logic for processing.

[MarkusKeck]

@Raiondesu Even better.
I just saw you added the following regex ?(?:\[([\w0-9_\-\.]+)\])? in src/node/markdown/plugins/snippet.ts:101
I checked it with https://regex101.com/ and the icon version did pass. I am not completly sure why, but it seems your addition did not affect the default behavior much. Someone with more experience in regex should take a look at this ;)

I really like the approach and implementation. It will make my project much cleaner. At the moment I am using a "workaround" which allows me to externalize my snippets but also allows me to change the displayed name.

::: code-group

<!--@include: ./../../../snippets/xxx.java-->
<!--@include: ./../../../snippets/yyy.java-->
<!--@include: ./../../../snippets/zzz.java-->

:::

Inside every file I have to add the codetab markup (see example above). It is not really pretty but the only way I could figure out to make this work. Looking forward to getting this merged. Thanks for the implementation.

[Raiondesu]

@MarkusKeck, thanks! I believe the regex you mentioned was actually changed to ?(?:\[(.+)\])?$ in this commit, though I must admit I didn't expect the original regex to work too! 😅

[Raiondesu]

@brc-dd, what's keeping this from being merged?
Maybe I need to change something in the pr?

[MarkusKeck]

Thank you @brc-dd

[RaniAgus]

Thank you! @Raiondesu @brc-dd