shaun smith

I'm not currently in agreement with this PR. This type of longform API/Prompt Engineering information may be better delivered within a normal MCP `Prompt` or using the Servers `instructions` field....

This all looks like useful metadata, however I think that these should be covered either with the upcoming `labels` / `namespacing` features, or better placed in external repositories.

OK. Think I can explain this one, and offer an improvement. Code blocks in markdown can either be fenced (` ```html`) or indented 4 spaces (` `). The issue arises...

Leaving the previous comment, however I am not 100% confident on whether indents are the only cause after more testing. The output below uses `` from list entries 10 onward......

Here is a snippet that shows the issue: - https://gist.github.com/evalstate/6b5ca3f67634602f7ce8dd8c3dbab7a3 - [Marked Demo](https://marked.js.org/demo/?text=%23%23%20Inside%20a%20List%0A%0A-%20This%20is%20a%20test%20(normal%20fences)%0A%0A%60%60%60html%0A%3Cfoo%20%2F%3E%0A%60%60%60%0A%0A-%20This%20is%20another%20test%20(indented%20block)%0A%0A%20%20%20%20%3Cfoo%20%2F%3E%0A%20%20%20%20%20%20%20%20%3Cbar%20%2F%3E%0A%0A-%20This%20is%20a%20further%20test%20(indents%20and%20fences)%0A%0A%20%20%20%20%60%60%60%0A%20%20%20%20%3Cfoo%20%2F%3E%0A%20%20%20%20%20%20%20%3Cbar%20%2F%3E%0A%20%20%20%20%60%60%60%0A%0A-%20Test%20complete%0A%0A%0A%23%23%20Outside%20a%20List%0A%0AThis%20is%20a%20test%20(normal%20fences)%0A%0A%60%60%60html%0A%3Cfoo%20%2F%3E%0A%60%60%60%0A%0AThis%20is%20another%20test%20(indented%20block)%0A%0A%20%20%20%20%3Cfoo%20%2F%3E%0A%20%20%20%20%20%20%20%20%3Cbar%20%2F%3E%0A%0AThis%20is%20another%20test%20(indents%20and%20fences)%0A%0A%20%20%20%20%60%60%60%0A%20%20%20%20%3Cfoo%20%2F%3E%0A%20%20%20%20%20%20%20%3Cbar%20%2F%3E%0A%20%20%20%20%60%60%60%0A%0ATest%20complete&options=%7B%0A%20%22async%22%3A%20false%2C%0A%20%22breaks%22%3A%20false%2C%0A%20%22extensions%22%3A%20null%2C%0A%20%22gfm%22%3A%20true%2C%0A%20%22hooks%22%3A%20null%2C%0A%20%22pedantic%22%3A%20false%2C%0A%20%22silent%22%3A%20false%2C%0A%20%22tokenizer%22%3A%20null%2C%0A%20%22walkTokens%22%3A%20null%0A%7D&version=14.0.0 ) The handling of code blocks in lists changes; asking the LLM via Chat-UI to repeat all...

Final update on this for the moment - the issue also occurs when code blocks are children of lists, causing the parse(token.raw) to show the child codeblock rather than being...

It's because it's a child of a bulleted/numbered list. In this case it doesn't use the CodeBlock component but the marked output. On Tue, 12 Nov 2024, 09:19 Rotem Dan,...

Last reply not helpful - there are 2 separate issues: 1) Code blocks that are children of lists don't get rendered via the CodeBlock component. 2) Those code blocks render...

LGTM after some testing :)