refactor: Check builder support by checking translation_handlers

Author: SilverRainZ

src/sphinxnotes/strike/__init__.py

@@ -17,14 +17,13 @@
 
 from sphinx.application import Sphinx
 from sphinx.builders import Builder
-from sphinx.builders.html import StandaloneHTMLBuilder
-from sphinx.builders.latex import LaTeXBuilder
+from sphinx.util import logging
+from sphinx.environment import BuildEnvironment
 
 from . import meta
 
+logger = logging.getLogger(__name__)
 
-# List of all builders that support render :class:`strike_node`.
-SUPPORTED_BUILDERS: list[type[Builder]] = [StandaloneHTMLBuilder, LaTeXBuilder]
 
 class strike_node(nodes.Inline, nodes.TextElement):
     pass
@@ -39,10 +38,16 @@ def strike_role(
     options: Dict = {},
     content: List[str] = [],
 ) -> Tuple[List[Node], List[system_message]]:
-    env = inliner.document.settings.env  # type: ignore
-
-    if not isinstance(env.app.builder, tuple(SUPPORTED_BUILDERS)):
-        # Builder is not supported, fallback to text.
+    env: BuildEnvironment = inliner.document.settings.env  # type: ignore
+    builder = env.app.builder
+
+    if not _is_supported_builder(builder):
+        logger.warning(
+            f'role {typ} is not supported for builder {builder.name}, fallback to text',
+            location=(env.docname, lineno),
+            type='strike',
+            subtype='unspported_builder',
+        )
         return [Text(unescape(text))], []
 
     node = strike_node(rawtext, unescape(text))
@@ -74,6 +79,15 @@ def latext_depart_strike_node(self, node: strike_node) -> None:
     self.body.append('}')
 
 
+def _is_supported_builder(builder: Builder) -> bool:
+    app = builder.app
+    # a dict of builder name -> dict of node name -> visitor and departure functions
+    return (
+        app.registry.translation_handlers[builder.name].get(strike_node.__name__)
+        is not None
+    )
+
+
 def setup(app: Sphinx):
     """Sphinx extension entrypoint."""
     meta.pre_setup(app)