edwh-editorjs 2.5.0a4__tar.gz → 2.6.1__tar.gz

{edwh_editorjs-2.5.0a4 → edwh_editorjs-2.6.1}/CHANGELOG.md

@@ -2,6 +2,24 @@
 
 <!--next-version-placeholder-->
 
+## v2.6.1 (2026-02-05)
+
+### Fix
+
+* **paragraph:** Handle nested HTML in alignment tag closing detection ([`7f910e1`](https://github.com/educationwarehouse/edwh-editorjs/commit/7f910e15103fe25ae0e4d1158deecc15d267d926))
+
+## v2.6.0 (2026-02-05)
+
+### Fix
+
+* **custom:** Replace recursive html parser with lxml to only parse outer layer and keep inner HTML intact ([`8a8908c`](https://github.com/educationwarehouse/edwh-editorjs/commit/8a8908c30708573df11f2011b24fda8c4984de21))
+
+## v2.5.0 (2025-03-20)
+
+### Feature
+
+* Improve markdown conversion and image handling in EditorJS blocks ([`80d0fc0`](https://github.com/educationwarehouse/edwh-editorjs/commit/80d0fc00a4e39b279de35998d1a85a37282c58a7))
+
 ## v2.4.0 (2024-12-02)
 
 ### Feature

{edwh_editorjs-2.5.0a4 → edwh_editorjs-2.6.1}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.3
+Metadata-Version: 2.4
 Name: edwh-editorjs
-Version: 2.5.0a4
+Version: 2.6.1
 Summary: EditorJS.py
 Project-URL: Homepage, https://github.com/educationwarehouse/edwh-EditorJS
 Author-email: SKevo <skevo.cw@gmail.com>, Robin van der Noord <robin.vdn@educationwarehouse.nl>
@@ -15,6 +15,7 @@ Classifier: Programming Language :: Python :: 3.13
 Requires-Python: >=3.10
 Requires-Dist: html2markdown
 Requires-Dist: humanize
+Requires-Dist: lxml
 Requires-Dist: markdown2
 Requires-Dist: mdast
 Provides-Extra: dev

edwh_editorjs-2.6.1/blocks-diff.txt

@@ -0,0 +1,42 @@
+diff --git a/editorjs/blocks.py b/editorjs/blocks.py
+index 9627a82..bc93cee 100644
+--- a/editorjs/blocks.py
++++ b/editorjs/blocks.py
+@@ -210,14 +210,30 @@ class ParagraphBlock(EditorJSBlock):
+ 
+                 if child.get("value", "").endswith("/>"):
+                     # self-closing
+-                    result.append(EditorJSCustom.to_json(node))
++                    result.append(EditorJSCustom.to_json({"children": [child]}))
+                 else:
+-                    # <editorjs>something</editorjs> = 3 children
+-                    result.extend(
+-                        EditorJSCustom.to_json({"children": nodes[idx : idx + 2]})
+-                    )
+-
+-                    skip = 2
++                    # <editorjs>...</editorjs> may include nested HTML; find the closing tag
++                    end_idx = None
++                    for j, next_child in enumerate(nodes[idx + 1 :], start=idx + 1):
++                        if next_child.get("type") == "html" and next_child.get(
++                            "value", ""
++                        ).strip() == "</editorjs>":
++                            end_idx = j
++                            break
++
++                    if end_idx is None:
++                        # fallback to previous behavior if tag is malformed
++                        result.extend(
++                            EditorJSCustom.to_json({"children": nodes[idx : idx + 2]})
++                        )
++                        skip = 2
++                    else:
++                        result.extend(
++                            EditorJSCustom.to_json(
++                                {"children": nodes[idx : end_idx + 1]}
++                            )
++                        )
++                        skip = end_idx - idx
+ 
+                 continue
+ 

edwh_editorjs-2.6.1/blog.md

@@ -0,0 +1,86 @@
+Via de pilot **‘Ambtenaar in de Praktijk’** doen Leidse ambtenaren gerichte praktijkervaring op in de stad. Zo loopt Lies Timmering, projectleider van de **Leidse Gelijke Kansen Aanpak**, elke woensdagmorgen mee bij **BuZz**, waar zij te vinden is bij de buurtsoep en meewerkt met bewoners en vrijwilligers. In deze blog deelt Lies haar persoonlijke reflectie op wat deze ervaringen voor haar betekenen en hoe ze helpen om beleid beter te laten aansluiten op wat er écht speelt in de stad.
+
+<editorjs type="image" caption="Foto: BuZz Leiden&nbsp;" border="" background="" stretched="1" url="https://py4web.onderwijsindeleidseregio.nl/thumb/upload/af1ac9a1-fd7a-416e-a5dc-6ee97266397d.jpg?hash=075d9622bd25e284eb23ea34a22537dd707d8437" />
+
+***
+
+Vandaag pakte ik mijn eed en belofte er nog eens bij. Ik beloofde als ambtenaar bij de start van mijn functie ‘plechtig’ het volgende:  
+
+1. Ik zal de gerechtigheid dienen. 
+2. Ik zal trouw zijn aan de grondwet en de overige wetten van het rijk.
+3. Ik zal mij inzetten voor het welzijn en de rechten van alle burgers van Leiden.
+4. Ik zal onpartijdig handelen en de democratische beginselen en procedures respecteren.
+5. Ik ben loyaal ten opzichte van de bestuursorganen van de stad en het door hen vastgestelde beleid.
+6. Ik zal van de overheidsmacht die mij is toevertrouwd geen misbruik maken.
+7. Ik zal zorgvuldig omgaan met informatie.
+8. Ik zal de geloofwaardigheid van het ambt niet schaden.
+9. Ik zal het vertrouwen, dat de burger in mij mag stellen, niet beschamen.
+10. Ik zal mij een zelfstandig oordeel vormen over de morele juistheid van mijn handelen.
+
+Tevens verklaar ik dat ik bekend ben met de gedragscode voor ambtenaren van de gemeente Leiden en deze als leidraad zal hanteren. 
+
+**Dat verklaar en beloof ik!**
+
+Toen ik mocht starten bij de gemeente had ik weinig beeld bij wat een ambtenaar nou precies deed. Wat het betekende om te werken voor een gemeente. Mensen zeiden me: “Bij de gemeente, jij? Dat gaat jou toch allemaal veel te traag, dat is niks voor jou?!” Maar de inhoud van de functie sprak me ontzettend aan, het team waarin ik werd ontvangen was ongelofelijk warm en de rest zou ik nog ontdekken.  
+
+<editorjs type='alignment' tag='p' alignment='center'> <b>Werk dat ertoe doet </b> </editorjs>
+
+Toen ik mocht starten bij de gemeente had ik weinig beeld bij wat een ambtenaar nou precies deed. Wat het betekende om te werken voor een gemeente. Mensen zeiden me: “Bij de gemeente, jij? Dat gaat jou toch allemaal veel te traag, dat is niks voor jou?!” Maar de inhoud van de functie sprak me ontzettend aan, het team waarin ik werd ontvangen was ongelofelijk warm en de rest zou ik nog ontdekken. 
+
+Later kon ik al deze mensen vertellen dat ik een geweldige baan had gevonden. Een baan waarvan mijn hart sneller ging kloppen. Ik mocht werk doen dat ertoe deed. Niks ging langzaam, eerder te snel. Nog nooit werkte ik in zo’n dynamische omgeving als een gemeente, geen dag is hetzelfde!  
+
+Iedereen die mij spreekt, die weet: ik geniet van mijn werk. Ik mag met ontzettend fijne mensen werken en ik mag werk doen dat ertoe doet. Ik mag mij inzetten voor het welzijn en de rechten van burgers in Leiden. Net zoals ik heb beloofd.  
+
+Wat ik wel merkte toen ik vanuit mijn functie met zoveel mogelijk partners en burgers in de stad ging praten, was dat er een flinke kloof was tussen de beleidswereld (gemeente, overheid) en de leefwereld van mensen, de wijken, de scholen, de plekken waarvoor het beleid werd gemaakt. Daar ervaarden mensen soms afstand, onbegrip en frustratie doordat de realiteit weerbarstig is en het beleid zwart-wit. Dit is natuurlijk niet iets Leids. Dit is landelijk zo, dit is internationaal zo. De afstand tussen overheid en ‘burgers’ lijkt groot, en bovendien ook steeds groter te worden. Regels, procedures, systemen. Maar ook simpelweg taal. Alsof het verschillende talen zijn, maar het zijn zeker verschillende belevingen...  
+
+Wat in mijn eed en belofte niet echt scherp wordt, is het waartoe van mijn werk. Waartoe beloof ik te doen wat ik als ambtenaar ga doen? Zeker, trouw aan de wet, zorgvuldigheid, inzetten voor het welzijn, loyaal aan het stadsbestuur. Ja. Maar waartoe doe ik dit dan eigenlijk?  
+
+Volgens AI bestaan “\[...] ambtenaren om het functioneren van de samenleving te waarborgen door het uitvoeren van overheidstaken, het ontwikkelen en uitvoeren van beleid, en het leveren van publieke diensten. Zij zorgen voor de handhaving van wetten, innen belastingen, verlenen vergunningen en ondersteunen politieke bestuurders.”  
+
+Herkenbaar en zeker, dat doen we. Maar nogmaals, waartoe doen we dit? Anno 2026 is een ambtenaar toch zeker iemand die zich bewust is van het feit dat de functie bestaat bij de gratie van deze vorm van democratie? Dat het doel is om de inwoners te dienen, en dus ook nabij te zijn om zelf te ervaren dat je werk goed uitpakt, en dus dienend is aan het welzijn en de rechten van de burgers? Soms raak ik zelf zo overweldigd door de systemen waarin we allen ons werk proberen uit te voeren dat het eerder voelt alsof je trouw hebt beloofd te zijn aan Excel-sheets en processen van besluitvorming, dan aan je inzetten voor het welzijn van de inwoners. De praktijk van een ambtenaar is ook best weerbarstig…   
+
+Tim ‘s Jongers  <a href="https://decorrespondent.nl/15957/beleid-wordt-beter-als-ambtenaren-achter-hun-laptop-vandaan-komen-daarom-een-pleidooi-voor-een-maatschappelijke-diensttijd/91ead8ad-ffb7-0a2a-15bf-316fe8e4578a" target="_blank">schreef eens dat beleid beter wordt als ambtenaren achter hun laptops vandaan komen </a> en onderdeel worden van de leefwereld waarvoor zij beleid maken. In Leiden kennen wij gelukkig verschillende werkprincipes die ons hierop wijzen, zo ook: wij werken van buiten (wat leeft in de stad) naar binnen (we maken beleid dat aansluit op wat leeft in de stad).  
+
+<editorjs type='alignment' tag='h2' alignment='center'>Leidse Ambtenaren in de Praktijk  </editorjs>
+
+Eén van de projecten die vanuit de Leidse Gelijke Kansen Aanpak is opgestart is de pilot ‘Ambtenaar in de Praktijk’. Via Ambtenaar in de Praktijk, doen ambtenaren van gemeente Leiden gerichte praktijkervaring op. Op die manier ervaren zij direct hoe beleid uitpakt in de praktijk en die kennis en ervaring brengen zij mee terug naar hun beleidsafdelingen.  
+
+“Ik krijg door het meewerken in de praktijk van een brugfunctionaris meer inzicht in de problematieken die scholen binnenkomen. Ik realiseerde me dat problemen zichtbaar worden via het kind maar dat de problemen altijd verder reiken.”    —  Ambtenaar in de praktijk 
+
+<editorjs type='alignment' tag='p' alignment='center'> <b>Gerichte praktijkervaring via partners in de stad  </b> </editorjs>
+
+In aanloop naar deze pilot hebben we de standpunten van onze samenwerkingspartners geïnventariseerd en hen gevraagd: ‘stel dat we dit gaan doen, wat zou je dan waardevolle praktijkervaring vinden voor een ambtenaar en hoe kun je dat faciliteren?’  
+
+Op die manier zijn we gekomen tot een soort menukaart met daarop min of meer kant-en-klare ideeën voor het opdoen van praktijkervaringen. Denk aan het meewerken in een buurthuis of buurtontmoetingsplek tijdens een inloop voor inwoners. Of denk aan het meewerken bij een speelgroep van JES Rijnland waar ouders met zeer jonge kinderen komen. Of misschien het meewerken bij een spreekuur rondom schulden. Of meewerken op een Leidse basisschool om zo meer mee te krijgen van wat er zoals speelt in en rond het onderwijs.   
+
+“Bij de speelgroep merk ik dat er vaak sprake is van een taalbarrière. Ouders gebruiken de ontmoetingen bij een speelgroep dus ook om onderling informatie uit te wisselen. Het is een soort vindplaats. Ouders en de professional(s) van de speelgroep delen kennis en helpen elkaar dus via deze plek.” —  Ambtenaar in de praktijk
+
+<editorjs type='alignment' tag='p' alignment='center'> <b>De gemeente organiseert en faciliteert  </b> </editorjs>
+
+Het is geweldig fijn dat gemeente Leiden haar ambtenaren hiervoor 32u op jaarbasis ruimte biedt. Op die manier geven we echt uitvoering aan ‘werken van buiten naar binnen’. De coördinatie van de pilot gebeurt vanuit de Leidse Gelijke Kansen Aanpak. Het is natuurlijk niet de bedoeling dat dit een verzwaring oplevert voor de partners in de stad. Het moet elkaar gaan versterken.  
+
+“Het is alleen al prachtig dat we nu zo nabij kunnen zijn. De partners voelen zich gezien en gehoord in hun werk. En andersom kun je ook een inkijkje geven in hoe het dan binnen de gemeente loopt, waarom dingen soms wat tijd vragen. Of gewoon door te zeggen, “Ja, wij balen er ook enorm van dat dit nog wat rommelig loopt. We werken er hard aan om dit beter te maken.” Het vertrouwen waarmee de partners deze pilot in de stad hebben ontvangen is geweldig. En nu op naar verdere uitbreiding!”  —  Ambtenaar in de praktijk
+
+<editorjs type='alignment' tag='p' alignment='center'> <b>Ongelofelijk leerzame weken  </b> </editorjs>
+
+De reacties in de stad zijn dan ook positief; de partnerorganisaties benadrukken het belang van praktijkervaring voor ambtenaren. Het is zeer leerzaam om in de praktijk te ervaren dat jouw beeld van hoe iets uitwerkt, misschien moet worden bijgesteld omdat de praktijk nu eenmaal weerbarstig is.  
+
+“Op papier staat het mooi, maar in de praktijk valt het me op hoeveel je vraagt van partijen op het gebied van samenwerking en uitwisseling. Als je je handen dagelijks vol hebt aan het ondersteunen van inwoners, dan is het niet realistisch om te verwachten dat men ook nog zelf op zoek gaat naar allerlei samenwerkingen. Zonde, want je ziet wel hoeveel men onderling van die kennis zou kunnen profiteren. Zowel de professionals als inwoners.” —  Ambtenaar in de praktijk   
+
+<editorjs type='alignment' tag='p' alignment='center'> <b>De volgende ronde gaat in maart van start  </b> </editorjs>
+
+De eerste ronde van deze mooie pilot is bijna afgerond. De betrokken ambtenaren ronden de pilot met elkaar af door de geleerde en ervaren lessen met elkaar om te zetten naar advies terug naar de beleidswereld. Onderling worden deze natuurlijk ook vast gedeeld met de volgende groep ambtenaren en met de betrokken partijen. Het is een prachtig leerproces. Wat leren wij van hen? Wat leren zij van ons? Wat betekent dit en hoe kunnen we dit leren omzetten in het daadwerkelijk verkleinen van die kloof tussen praktijk en beleid?   
+
+Zo geeft een collega aan dat ze ziet hoe brieven in duidelijke taal vanuit de gemeente, toch vaak niet duidelijk genoeg lijken te zijn voor inwoners. Dat kun je niet weten, tenzij je spreekt met de inwoners zelf én met de professionals die hen ondersteunen.  
+
+“Bij het Goede Buur spreekuur werk ik met gepassioneerde hulpverleners die de inwoner zo goed mogelijk willen helpen.” —  Ambtenaar in de praktijk
+
+Eén ding is zeker: dit zou onderdeel moeten kunnen zijn van je werk. Gemeente Leiden werkt graag kennisgericht, en via deze praktijkervaring maken we daar ook praktijkervaringskennisgericht werken van. Minimaal ieder jaar een week meewerken in de praktijk. De mouwen opstropen, doen, ondergaan en ervaren. Dit zou toch voor eenieder moeten gelden in elke positie waarbij jouw beleid invloed heeft op de ander? 
+
+<editorjs type='alignment' tag='p' alignment='center'> <b>College in de Praktijk?  </b> </editorjs>
+
+Pasgeleden opperde een collega dat we misschien ook moeten denken aan een uitbreiding van de pilot; ‘College in de Praktijk’, zodat na de verkiezingen het college van wethouders en burgemeester ook de Leidse praktijk echt kan ervaren. Een werkbezoek is namelijk echt iets heel anders als gewoon even ergens zelf de koffie zetten, stoelen uitklappen voor de vele inwoners die binnenkomen, samen soep koken, luisteren naar de verhalen van zij die even steun nodig hebben of vragen komen stellen. Zij die even warmte opzoeken omdat het thuis koud is. Of even stoom komen afblazen omdat ze niet als volwaardig worden gezien... 
+
+“Heel eerlijk? Ik had geen idee. Ik wist het wel, maar ik realiseerde me het niet. Zelf voelen hoe het moet zijn om geen netwerk te hebben, om geen idee te hebben van de wereld om je heen. Zoveel te bieden, zo slim, getalenteerd. Zo betrokken en sociaal. Maar toch vast blijven zitten omdat je nu eenmaal de weg niet kent. Je denkt, dat hebben we allemaal goed ingeregeld! Maar in de praktijk zie je dat het niet zo makkelijk is...”  —  Ambtenaar in de praktijk
+
+***

edwh_editorjs-2.6.1/debug.py

@@ -0,0 +1,22 @@
+from pathlib import Path
+
+from editorjs import EditorJS
+
+
+def main():
+    # blog = Path("blog.md").read_text()
+
+    ejs = EditorJS.from_json(
+        '{"time":1770292823347,"blocks":[{"id":"QKelxSHz2Y","type":"paragraph","data":{"text":"rechts basis"},"tunes":{"alignmentTune":{"alignment":"right"}}},{"id":"IUX70yigzz","type":"paragraph","data":{"text":"links"},"tunes":{"alignmentTune":{"alignment":"left"}}},{"id":"SaWthD_Vlr","type":"paragraph","data":{"text":"<b>rechts duur</b>"},"tunes":{"alignmentTune":{"alignment":"right"}}}],"version":"2.30.7"}'
+    )
+
+    # if html = <editorjs type='alignment' tag='p' alignment='center'> <b>Werk dat ertoe doet </b> </editorjs>
+    # assert parse_html(html).type == 'alignment'
+
+    print(ejs.to_html())  # good
+    print(ejs.to_json())  # bad, fixme
+    # {"time": 1770292880600, "blocks": [{"type": "paragraph", "data": {"text": ""}, "tunes": {"alignmentTune": {"alignment": "right"}}}, {"type": "paragraph", "data": {"text": "links"}}, {"type": "paragraph", "data": {"text": "<b></b>"}, "tunes": {"alignmentTune": {"alignment": "right"}}}, {"type": "raw", "data": {"html": "</b></editorjs>"}}], "version": "2.30.6"}
+
+
+if __name__ == "__main__":
+    main()

edwh_editorjs-2.6.1/editorjs/__about__.py

@@ -0,0 +1 @@
+__version__ = "2.6.1"

{edwh_editorjs-2.5.0a4 → edwh_editorjs-2.6.1}/editorjs/blocks.py

@@ -5,15 +5,15 @@ mdast to editorjs
 import abc
 import re
 import typing as t
-from html.parser import HTMLParser
 from urllib.parse import urlparse
 
+import html2markdown
 import humanize
+import lxml.html
 import markdown2
 
 from .exceptions import TODO, Unreachable
 from .types import EditorChildData, MDChildNode
-import html2markdown
 
 
 class EditorJSBlock(abc.ABC):
@@ -55,8 +55,6 @@ def process_styled_content(item: MDChildNode, strict: bool = True) -> str:
     """
     _type = item.get("type")
 
-    print('process_styled_content', _type)
-
     html_wrappers = {
         "text": "{value}",
         "html": "{value}",
@@ -110,9 +108,9 @@ class HeadingBlock(EditorJSBlock):
             raise ValueError("Header level must be between 1 and 6.")
 
         if (
-                tunes.get("alignmentTune")
-                and (alignment := tunes["alignmentTune"].get("alignment"))
-                and (alignment != "left")
+            tunes.get("alignmentTune")
+            and (alignment := tunes["alignmentTune"].get("alignment"))
+            and (alignment != "left")
         ):
             # can't just return regular HTML because then it will turn into a raw block
             return AlignmentBlock.to_markdown(
@@ -172,9 +170,9 @@ class ParagraphBlock(EditorJSBlock):
         tunes = data.get("tunes", {})
 
         if (
-                tunes.get("alignmentTune")
-                and (alignment := tunes["alignmentTune"].get("alignment"))
-                and (alignment != "left")
+            tunes.get("alignmentTune")
+            and (alignment := tunes["alignmentTune"].get("alignment"))
+            and (alignment != "left")
         ):
             return AlignmentBlock.to_markdown(
                 {
@@ -189,6 +187,18 @@ class ParagraphBlock(EditorJSBlock):
 
         return f"{text}\n\n"
 
+    @staticmethod
+    def _find_closing_editorjs_tag(nodes: list, start_idx: int) -> int | None:
+        """Find index of closing </editorjs> tag."""
+        for idx, node in enumerate(nodes[start_idx:], start=start_idx):
+            if (
+                node.get("type") == "html"
+                and node.get("value", "").strip() == "</editorjs>"
+            ):
+                return idx
+
+        return None
+
     @classmethod
     def to_json(cls, node: MDChildNode) -> list[dict]:
         result = []
@@ -212,14 +222,17 @@ class ParagraphBlock(EditorJSBlock):
 
                 if child.get("value", "").endswith("/>"):
                     # self-closing
-                    result.append(EditorJSCustom.to_json(node))
+                    result.append(EditorJSCustom.to_json({"children": [child]}))
                 else:
-                    # <editorjs>something</editorjs> = 3 children
-                    result.extend(
-                        EditorJSCustom.to_json({"children": nodes[idx: idx + 2]})
+                    # <editorjs>...</editorjs> may include nested HTML; find the closing tag
+                    end_idx = cls._find_closing_editorjs_tag(nodes, idx + 1)
+                    children_slice = (
+                        nodes[idx : idx + 2]
+                        if end_idx is None
+                        else nodes[idx : end_idx + 1]
                     )
-
-                    skip = 2
+                    skip = 2 if end_idx is None else end_idx - idx
+                    result.extend(EditorJSCustom.to_json({"children": children_slice}))
 
                 continue
 
@@ -398,7 +411,7 @@ class CodeBlock(EditorJSBlock):
     @classmethod
     def to_markdown(cls, data: EditorChildData) -> str:
         code = data.get("code", "")
-        return f"```\n" f"{code}" f"\n```\n"
+        return f"```\n{code}\n```\n"
 
     @classmethod
     def to_json(cls, node: MDChildNode) -> list[dict]:
@@ -457,9 +470,9 @@ class ImageBlock(EditorJSBlock):
         border = node.get("border") or ""
 
         return f"""
-        <div class="ce-block {stretched and 'ce-block--stretched'}">
+        <div class="ce-block {stretched and "ce-block--stretched"}">
             <div class="ce-block__content">
-            <div class="cdx-block image-tool image-tool--filled {background and 'image-tool--withBackground'} {stretched and 'image-tool--stretched'} {border and 'image-tool--withBorder'}">
+            <div class="cdx-block image-tool image-tool--filled {background and "image-tool--withBackground"} {stretched and "image-tool--stretched"} {border and "image-tool--withBorder"}">
                 <div class="image-tool__image">
                     <figure>
                         <img class="image-tool__image-picture" src="{url}" title="{caption}" alt="{caption}">
@@ -514,7 +527,6 @@ class QuoteBlock(EditorJSBlock):
 
 @block("raw", "html")
 class RawBlock(EditorJSBlock):
-
     @classmethod
     def to_markdown(cls, data: EditorChildData) -> str:
         text = data.get("html", "")
@@ -537,7 +549,6 @@ class RawBlock(EditorJSBlock):
 
 @block("table")
 class TableBlock(EditorJSBlock):
-
     @classmethod
     def to_markdown(cls, data: EditorChildData) -> str:
         """
@@ -656,7 +667,6 @@ class LinkBlock(EditorJSBlock):
 
 @block("attaches")
 class AttachmentBlock(EditorJSBlock):
-
     @classmethod
     def to_markdown(cls, data: EditorChildData) -> str:
         title = data.get("title", "")
@@ -721,7 +731,7 @@ class AttachmentBlock(EditorJSBlock):
                 </div>
                 {file_size}
             </div>
-            <a class="cdx-attaches__download-button" href="{node.get('file', '')}" target="_blank" rel="nofollow noindex noreferrer" title="{node.get('name', '')}">
+            <a class="cdx-attaches__download-button" href="{node.get("file", "")}" target="_blank" rel="nofollow noindex noreferrer" title="{node.get("name", "")}">
                 <svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" fill="none" viewBox="0 0 24 24"><path stroke="currentColor" stroke-linecap="round" stroke-width="2" d="M7 10L11.8586 14.8586C11.9367 14.9367 12.0633 14.9367 12.1414 14.8586L17 10"></path></svg>
             </a>
         </div>
@@ -773,7 +783,6 @@ class AlignmentBlock(EditorJSBlock):
 
 @block("embed")
 class EmbedBlock(EditorJSBlock):
-
     @classmethod
     def to_markdown(cls, data: EditorChildData) -> str:
         service = data.get("service", "")
@@ -802,20 +811,6 @@ class EmbedBlock(EditorJSBlock):
 ### end blocks
 
 
-class AttributeParser(HTMLParser):
-    def __init__(self):
-        super().__init__()
-        self.attributes = {}
-        self.data = None
-
-    def handle_starttag(self, tag, attrs):
-        # Collect attributes when the tag is encountered
-        self.attributes = dict(attrs)
-
-    def handle_data(self, data):
-        self.data = data
-
-
 class EditorJSCustom(EditorJSBlock, markdown2.Extra):
     """
     Special type of block to deal with custom attributes.
@@ -828,10 +823,29 @@ class EditorJSCustom(EditorJSBlock, markdown2.Extra):
 
     @classmethod
     def parse_html(cls, html: str):
-        parser = AttributeParser()
-        parser.feed(html)
+        """
+        Extract attributes from the outermost HTML element and return its inner HTML.
+
+        This function parses the provided markup, identifies the root element,
+        returns its attributes as a dictionary, and serializes all direct child
+        nodes back into an HTML string.
+
+        Args:
+            html: A string containing a single root HTML element.
+
+        Returns:
+            A tuple of:
+                - dict[str, str]: Attributes of the root element
+                - str: Inner HTML of the root element
+        """
+        root = lxml.html.fromstring(html)
+
+        attributes = dict(root.attrib)
+        inner_html = "".join(
+            lxml.html.tostring(child, encoding="unicode") for child in root
+        )
 
-        return parser.attributes, parser.data
+        return attributes, inner_html
 
     @classmethod
     def to_markdown(cls, data: EditorChildData) -> str:
@@ -846,6 +860,7 @@ class EditorJSCustom(EditorJSBlock, markdown2.Extra):
         handler = BLOCKS.get(_type)
 
         if not handler:
+            raise ValueError(f"debug: {attrs = } {body = }")  # fixme
             raise ValueError(f"Unknown custom type {_type}")
 
         return handler, attrs

{edwh_editorjs-2.5.0a4 → edwh_editorjs-2.6.1}/editorjs/core.py

@@ -100,13 +100,11 @@ class EditorJS:
             try:
                 blocks.extend(block.to_json(child))
             except Exception as e:
-
                 warnings.warn(
                     "to_json: Oh oh, unexpected block failure!",
                     category=RuntimeWarning,
                     source=e,
                 )
-                raise e
                 # if isinstance(e, TODO):
                 #     raise e
 

{edwh_editorjs-2.5.0a4 → edwh_editorjs-2.6.1}/pyproject.toml

@@ -31,6 +31,7 @@ dependencies = [
     "markdown2",  # markdown -> html
     "html2markdown", # html -> markdown
     "humanize",
+    "lxml",
 ]
 
 [project.optional-dependencies]

{edwh_editorjs-2.5.0a4 → edwh_editorjs-2.6.1}/tests/test_core.py

@@ -2,6 +2,7 @@ import json
 import textwrap
 
 from editorjs import EditorJS
+from editorjs.blocks import EditorJSCustom
 
 EXAMPLE_MD = textwrap.dedent("""
     # Heading
@@ -185,7 +186,8 @@ asdfsdajgdsjaklgkjds
 
 
 def test_code():
-    e = EditorJS.from_markdown(textwrap.dedent("""
+    e = EditorJS.from_markdown(
+        textwrap.dedent("""
     Read code:
     
     ```
@@ -193,7 +195,8 @@ def test_code():
     ```
     
     End of code
-    """))
+    """)
+    )
 
     blocks = json.loads(e.to_json())
 
@@ -266,6 +269,7 @@ def test_figcaption():
 
     assert "figcaption" in html
 
+
 def test_bold():
     js = """{"time":1742475802066,"blocks":[{"id":"v_Kc51dnJH","type":"paragraph","data":{"text":"Deze tekst is <b>half bold</b> en half niet"},"tunes":{"alignmentTune":{"alignment":"left"}}},{"id":"q_fkuEFcY5","type":"paragraph","data":{"text":"<b>Deze tekst is heel bold</b>"},"tunes":{"alignmentTune":{"alignment":"left"}}}],"version":"2.30.7"}"""
 
@@ -291,3 +295,81 @@ def test_quotes():
         e.to_markdown(),
     )
 
+
+# test case based on issue HETNIEUWELEIDEN-019c28cd-56b8-7778-be0c-60fb9e930638
+
+
+def test_nonrecusrive_attr_parser():
+    html = "<div type='outer'><div type='inner'>contents</div></div>"
+
+    attributes, data = EditorJSCustom.parse_html(html)
+
+    assert attributes == {"type": "outer"}
+    assert data == '<div type="inner">contents</div>'
+
+
+def test_editorjs_alignment_tag():
+    """Test that editorjs alignment tags with nested HTML are parsed correctly."""
+    md_input = "<editorjs type='alignment' tag='p' alignment='center'> <b>Werk dat ertoe doet </b> </editorjs>"
+
+    e = EditorJS.from_markdown(md_input)
+
+    print("MDAST:", e.to_mdast())
+    print("Markdown:", e.to_markdown())
+    print("JSON:", e.to_json())
+
+    html = e.to_html()
+    print("HTML:", html)
+
+    # The type should be detected as 'alignment' (or mapped to 'paragraph' with alignment tune)
+    blocks = json.loads(e.to_json())
+
+    assert blocks
+
+    assert "<b>Werk dat ertoe doet" in html
+    assert "<editorjs" not in html
+    assert "<code" not in html
+
+
+def test_alignment_with_nested_bold_roundtrip():
+    """
+    Test that alignment tags preserve nested bold HTML through JSON roundtrip.
+
+    Regression test for issue where nested HTML in alignment tags causes:
+    - Text content to disappear
+    - Malformed closing tags to appear as separate raw blocks
+    """
+    # Input: three paragraphs with different alignments, one with nested bold
+    input_json = r"""{"time":1770292823347,"blocks":[{"id":"QKelxSHz2Y","type":"paragraph","data":{"text":"rechts basis"},"tunes":{"alignmentTune":{"alignment":"right"}}},{"id":"IUX70yigzz","type":"paragraph","data":{"text":"links"},"tunes":{"alignmentTune":{"alignment":"left"}}},{"id":"SaWthD_Vlr","type":"paragraph","data":{"text":"<b>rechts duur</b>"},"tunes":{"alignmentTune":{"alignment":"right"}}}],"version":"2.30.7"}"""
+    # faulty output: {"time": 1770292880600, "blocks": [{"type": "paragraph", "data": {"text": ""}, "tunes": {"alignmentTune": {"alignment": "right"}}}, {"type": "paragraph", "data": {"text": "links"}}, {"type": "paragraph", "data": {"text": "<b></b>"}, "tunes": {"alignmentTune": {"alignment": "right"}}}, {"type": "raw", "data": {"html": "</b></editorjs>"}}], "version": "2.30.6"}
+
+    e = EditorJS.from_json(input_json)
+
+    # Convert through markdown and back to JSON
+    md = e.to_markdown()
+    print("Markdown output:", md)
+
+    e2 = EditorJS.from_markdown(md)
+    output_json = json.loads(e2.to_json())
+
+    output_str = json.dumps(output_json, indent=2)
+    print("Output JSON:", output_str)
+
+    # Should have exactly 3 blocks (not 4 with a malformed raw block)
+    assert len(output_json["blocks"]) == 3
+
+    # All should be paragraphs (not raw blocks)
+    for block in output_json["blocks"]:
+        assert block["type"] == "paragraph"
+
+    # Third block should preserve the bold content
+    assert (
+        "<b>rechts duur</b>" in output_json["blocks"][2]["data"]["text"]
+        or "<b>rechts duur </b>" in output_json["blocks"][2]["data"]["text"]
+    )
+
+    # Third block should preserve alignment
+    assert output_json["blocks"][2]["tunes"]["alignmentTune"]["alignment"] == "right"
+
+    assert "raw" not in output_str
+    assert "html" not in output_str

edwh_editorjs-2.5.0a4/editorjs/__about__.py

@@ -1 +0,0 @@
-__version__ = "2.5.0a4"