gp-opengraph, gp-sitemap(fix[logging]): attach NullHandler + drop trailing periods

why: PR #22 code review flagged two CLAUDE.md#logging violations in the
two new packages. These are house-style invariants that every existing
workspace package (gp_sphinx, sphinx_fonts, sphinx_ux_badges, …)
already follows:

1. "Add ``NullHandler`` in library ``__init__.py`` files" (CLAUDE.md
   line 553) — both new package __init__.py files declared a
   module-level logger without one. Importing either package outside a
   Sphinx build could surface a "no handlers could be found" warning
   from stdlib logging.
2. "No trailing punctuation" in log messages (CLAUDE.md line 576) —
   two log calls introduced in commit ab3a6a6 and 04f0121 ended with a
   period.

what:
- packages/gp-opengraph/src/gp_opengraph/__init__.py:
  * After ``logger = logging.getLogger(__name__)`` attach
    ``logger.addHandler(logging.NullHandler())`` to match the
    sphinx-fonts pattern exactly.
  * ``_warn_if_social_cards_used``: message reworded to "ogp_social_cards
    ignored — gp-opengraph ships no card generator; use a static PNG
    via ogp_image (site default) or per-page 'og:image' frontmatter"
    (no trailing period; also trimmed the "is ignored — gp-opengraph
    does not bundle a card generator" phrasing into "ignored — …
    ships no card generator" which reads cleaner as a warning).
- packages/gp-sitemap/src/gp_sitemap/__init__.py:
  * Import ``logging`` from stdlib and add
    ``logging.getLogger(__name__).addHandler(logging.NullHandler())``
    alongside the existing ``logger = getLogger(__name__)`` from
    ``sphinx.util.logging``. The Sphinx adapter is kept because it
    supports the ``type=``/``subtype=`` kwargs used elsewhere for
    warning classification; the NullHandler attaches to the underlying
    stdlib logger with the same name so the library is well-behaved
    when imported outside Sphinx. Inline comment memorializes the
    two-getLogger dance so future readers don't try to delete one.
  * ``_write_sitemap`` "skipping sitemap" info message: trailing period
    removed.

CI gate (all green before commit):
- uv run ruff check . --fix --show-fixes: clean
- uv run ruff format .: no changes
- uv run mypy: Success (176 source files)
- uv run py.test --reruns 0: 1199 passed, 3 skipped
- just build-docs: build succeeded

Author: tony

packages/gp-opengraph/src/gp_opengraph/__init__.py

@@ -38,6 +38,7 @@
     from sphinx.config import Config
 
 logger = logging.getLogger(__name__)
+logger.addHandler(logging.NullHandler())
 
 _EXTENSION_VERSION = "0.0.1a9"
 
@@ -260,9 +261,9 @@ def _warn_if_social_cards_used(app: Sphinx, config: Config) -> None:
     del app  # unused; required by Sphinx's config-inited signature
     if config.ogp_social_cards:
         logger.warning(
-            "gp-opengraph: ogp_social_cards is ignored — gp-opengraph does "
-            "not bundle a card generator. Use a static PNG via ogp_image "
-            "(site default) or per-page 'og:image' frontmatter.",
+            "gp-opengraph: ogp_social_cards ignored — gp-opengraph ships "
+            "no card generator; use a static PNG via ogp_image (site "
+            "default) or per-page 'og:image' frontmatter",
         )
 
 

packages/gp-sitemap/src/gp_sitemap/__init__.py

@@ -26,6 +26,7 @@
 import contextlib
 import datetime as dt
 import fnmatch
+import logging
 import pathlib
 import typing as t
 from xml.etree import ElementTree
@@ -44,7 +45,12 @@
 _SITEMAP_NS = "http://www.sitemaps.org/schemas/sitemap/0.9"
 _XHTML_NS = "http://www.w3.org/1999/xhtml"
 
+# gp-sitemap uses Sphinx's logger adapter so ``type=``/``subtype=`` kwargs
+# work for warning classification, but still attaches NullHandler to the
+# underlying stdlib logger so the library doesn't emit "no handlers" warnings
+# when imported outside a Sphinx build (per CLAUDE.md #Logger setup).
 logger = getLogger(__name__)
+logging.getLogger(__name__).addHandler(logging.NullHandler())
 
 SitemapLink = tuple[str, str | None]  # (relative link, last_updated ISO8601 or None)
 
@@ -229,7 +235,7 @@ def _write_sitemap(app: Sphinx, exception: BaseException | None) -> None:
         # that run with ``-W``.
         logger.info(
             "gp-sitemap: skipping sitemap — set site_url or html_baseurl "
-            "in conf.py to enable.",
+            "in conf.py to enable",
             type="sitemap",
             subtype="configuration",
         )