Closed (fixed)
Project:
/llms.txt
Version:
1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
7 Oct 2025 at 14:58 UTC
Updated:
22 Oct 2025 at 12:49 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
mxr576@a.dmitriiev shared the following code examples, thanks for this!
My spidey senses indicates that cacheability information is lost somewhere here, even if nothing in the call chain of
$view->access($display_id)bubbles up cacheability information - why is that? probably there is a Core issue for this, nvm for now... - . So I think it would be better to run the complete rendering in a dedicated render context and bubbling up the collected cacheability information, just in case.markdownify_viewsis surely needed for this feature. Second guessing a bit but should this feature be included in that module? What is your thought on this @iambatman?I was also thinking about why
views_embed_view()has not been leveraged in the original code, but then I realized that it is not important. The most important part probably doing the same what is done in\Drupal\markdownify_views\Controller\MarkdownifyViewPageController::handle(), or calling that code for consistently generating the markdown version of a Views view.Comment #3
mxr576Maybe recent results in #3548981-4: Output markdown menu links when Markdownify is enabled are just proves that we also do not and cannot serialize any kind of Views view to markdown. Links after the forced transformation may not work.
The special case we are building the ideal examole for transfermarion, maybe we should support those kinds of Views somehow...
Also, the work is being done in #3521569: Allow linking to the md version of an entity could also help with directly generating markdown links for nodes.
Comment #4
mxr576Not alone, after that entity views data also has to be altered so the Markdown link would be available in Views field settings.
\Drupal\views\EntityViewsData::addEntityLinks()Comment #6
mxr576Current state of MR#6 without changes on phpstan baseline so it could be applied on the llm_support recipe.
Comment #7
mxr576Test coverage would be great, especially on cache invalidation.
Comment #8
mxr576Due to preparations for DrupalCon, MR#6 was merged.
Created a follow up for test coverage: #3550976: Test coverage for "Add token for rendering a markdown version of a Views view"
Comment #9
mxr576