codeblock copy button copies the syntax name instead of just the code

[TFWol]

Describe the Issue
If you have a named codeblock, like batch, it gets placed inside the code and ends up getting copied along with the code which can cause syntax errors if you aren't watching for it.

It should be a label or something outside the box instead.

[Luro223]

@TFWol Does this happen with KoboldAI only? If yes then check Repetition penalty and DRY.

[LostRuins]

This has got nothing to do with rep pen. It's a UI behavior.

@TFWol currently the copy button does not check for syntax and just copies the entirety of the contents of a code block. Some models generate labels and some do not, so it's hard to correctly match them all. For now i'd recommend manually ensuring that unwanted text is excluded.

[TFWol]

When there isn't a label, the element starts with a <br>, otherwise it's the syntax label.
A simple solution could be to have the copy button skip the first line of the codeblock.

I tried to make a userscript, but it didn't work out so great.

✅ Successfully got userscript snippet working without any visible changes to the UI:

(function() {
    'use strict';

    function modifyCopyCode() {
        // Check if the function already exists to avoid conflicts
        if (typeof copyMarkdownCode === 'function') {

          const originalCopyMarkdownCode = copyMarkdownCode;

          copyMarkdownCode = function(btn) {
              const codeContainer = btn.parentElement.querySelector('pre code');
              if (codeContainer) {
                  let codeText = codeContainer.innerText;
                  let codeLines = codeText.split('\n');
                  codeLines.shift(); // This gets the job done by removing the first line
                  let modifiedCodeText = codeLines.join('\n');
                  navigator.clipboard.writeText(modifiedCodeText);
              }
            }
        } else {
          console.error("copyMarkdownCode function not found. Make sure the script loads after the target page.");
        }
    }

    // Call the modifyCopyCode function when the page loads
    window.addEventListener('load', modifyCopyCode);
})();

[TFWol]

@LostRuins Could you replace
https://github.com/LostRuins/koboldcpp/blob/9b324820897fe08d47b4bfec36d3648a14164ed7/klite.embd#L4958-L4963

function copyMarkdownCode(btn)
{
	const codeContainer = btn.parentElement.querySelector('pre code');
	//selectElementContents(codeContainer);
	navigator.clipboard.writeText(codeContainer.innerText);
}

with:

function copyMarkdownCode(btn)
{
	const codeContainer = btn.parentElement.querySelector('pre code');
	//selectElementContents(codeContainer);
	if (codeContainer) 
	{
		let codeText = codeContainer.innerText;
		let codeLines = codeText.split('\n');
		codeLines.shift(); // Remove the first line
		let modifiedCodeText = codeLines.join('\n');
		navigator.clipboard.writeText(modifiedCodeText);
	}
}

I'd do a pull request, but I'm not familiar with the warning on the file: