asciidoctor-rhrev 1.5.3 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6a4a09c2c018c9d8b79c1d819ff29867b1a398b683f22a4074392e0ac75c6ff4
-  data.tar.gz: 7e7118f6e79d519b47380d1aaf52b9e9f8c22db573d66c45c13c9a2cc91aca74
+  metadata.gz: d50d58f6581222d194cf031114bf6aeb82525b1662e6765c4e24c5c4f0e530cd
+  data.tar.gz: b45e1e49cc09002e1267c1e31221658ec9aa26a5ca49d4ef9dc8ea3ba857e0d4
 SHA512:
-  metadata.gz: 4115b2013f9c56541ab3919cb8ca2f71e29167d00af6a105ede4da90d4e5e5bd96ab1ac86678ae1de7550929bd8939770ae52f7a4874f1822136117e0d359568
-  data.tar.gz: 1f984b83343c2735c9fa7e7bd6bac3474eeabe6731a2237386d481a5f4d6f625f83368b4abf02e612bfa1898101b909237d0cb7a13286704894b8470153a2275
+  metadata.gz: 0cf35ff6b4b52a1b9851b8eb103efd5eafc0922f59045d4830b5a390a2a62607060ff79442d5c26e129512efaf06cbd0c4dfc06808f992b14fe3203f94a2e3cf
+  data.tar.gz: 39ac612cba2f168c9cf4b26eddaea213246d18a90a3b25283b38f82c035756ae78597b01090466ae97ffe644f53f813e8f6b907a7957f5bd4f470d8ecc607e73

data/CHANGELOG.adoc

@@ -1,5 +1,10 @@
 = Changelog
 
+== 1.6.0 (2026-09-02)
+
+* Add per-item change bars and revision-history roll-up for individual items in an unordered or ordered list (PDF backend), previously only possible for a whole marked list. `Converter#convert_list_item` brackets asciidoctor-pdf's own per-item render call (`convert_list` calls it once per item directly, a real granular hook, unlike table rows) the same way the existing paragraph/ulist/olist/dlist overrides already bracket their own whole-block calls. No AsciiDoc syntax attaches a block attribute list directly to a list item's own bullet line, confirmed directly against a real parse: the attribute list is silently dropped, not attached to the item, the list, or anywhere else. The working construct, the same shape as an `a|` table cell's own workaround, is a block attribute list on content the item already legitimately attaches via a `+` continuation. `Converter#propagate_list_item_content_marks` copies a marked carrier block's rhrev attribute onto the enclosing item and strips it from the carrier, at prescan time, before real rendering: the strip matters because `convert_list_item`'s own `traverse_list_item` call renders an item's attached blocks through this same converter, so a left-marked carrier would independently trigger its own bar-inking hook too, a second bad bar. `list_item` is now one of the roll-up contexts, resolved via the same nearest-enclosing-section walk every other roll-up context already uses, reusing that mechanism unchanged. A marked sub-list attached to an item keeps its own independent mark; the new marker-carrier context set deliberately excludes ulist/olist/dlist/list_item so a compound item's own nested sub-list is never swallowed into the parent item's mark. A `dlist` using the `unordered`, `ordered`, or `qanda` style gets the same per-item support, since asciidoctor-pdf itself renders those through synthesized list items routed through the same hook; the default and `horizontal` `dlist` styles render each term and description pair inline with no per-item method call to override, the same category of structural gap table-row-level marking has, not attempted here. Verified directly: a real fixture rendered to PNG shows exactly one bar, spanning only the marked item, not the whole list and not bleeding into neighboring items; the revision-history table shows exactly one bullet for the marked item, nothing from its unmarked siblings. New regression file, `tests/regression/list-item-change-bars.rb`, confirmed to fail (a raised `NoMethodError`) without the fix and pass with it
+* Note: gem-only; the Antora extension (HTML output, no page-margin bar geometry the same way) is unaffected, npm still bumps concurrently per the standing 1.5.1 policy
+
 == 1.5.3 (2026-09-01)
 
 * Fix `page_numbering_start_at: cover` still reporting the revision-history table and `pagerhref:` one page low when the theme has no real `front-cover-image`/`cover_front_image` of its own, for example a theme that draws its own cover through some other means. `Converter#compute_rhrev_page_numbering_start_page`'s `cover` branch fell back to plain `body_start` whenever `resolve_background_image` found nothing, matching 1.5.2's own `body`-shaped formula rather than the real one: asciidoctor-pdf itself falls back from `cover` to `title` in that case, not to `body`, and `title`'s own offset is 0 whenever the document has a title page at all (`doctype: book`, or the `title-page` attribute), regardless of whether a real front cover ever rendered. `body_start` only remains correct for the narrow remaining case of no front cover and no title page either. Reported directly against a real document: the table was one page low throughout. New regression case, `cover-no-image`, in `tests/regression/rhrev-page-numbering-start-at.rb`, confirmed to fail without the fix and pass with it

data/lib/asciidoctor/rhrev/converter.rb

@@ -411,6 +411,30 @@ module Asciidoctor
           bracket_change_bar(node) { super }
         end
 
+        # A single item within a ulist/olist (and a dlist using the
+        # unordered/ordered/qanda styles, which asciidoctor-pdf itself
+        # synthesizes into real ListItem objects routed through here the
+        # same way), rather than the whole list. asciidoctor-pdf's own
+        # convert_list calls this once per item directly (not through the
+        # normal node.convert dispatch), a real granular hook, so the same
+        # bracket_change_bar shape as the nine block types above applies
+        # unchanged: bare super forwards list/opts along with node
+        # automatically. The bracket spans traverse_list_item's own call at
+        # the end of the real convert_list_item, which covers the item's
+        # marker, its principal text, and any attached continuation content
+        # together, correct even for a compound, multi-block item.
+        #
+        # A dlist using its default or horizontal style is NOT covered:
+        # asciidoctor-pdf renders those two styles' term/description pairs
+        # entirely inline inside convert_dlist itself, with no per-entry
+        # method call to override, the same category of structural gap
+        # table-row-level marking has (see LIST_ITEM_MARKER_CARRIER_
+        # CONTEXTS's own docblock and the project's declined-features
+        # notes); a bigger, riskier lift, not attempted here.
+        def convert_list_item node, list, opts = {}
+          bracket_change_bar(node) { super }
+        end
+
         def convert_rhrev node
           # The HtmlTreeprocessor (misleadingly named; it has no backend guard,
           # unlike ChangeBarsHtmlTreeprocessor) unshifts a synthetic :rhrev
@@ -528,7 +552,24 @@ module Asciidoctor
         # none yet for the matching revision), rather than needing its own
         # id, its own destination, or a generated label. See
         # rollup_child_revision_entries.
-        ROLLUP_CONTEXTS = [:paragraph, :ulist, :olist, :dlist, :admonition, :open, :quote, :verse, :sidebar].freeze
+        #
+        # list_item is included: enclosing_section's own node.parent walk
+        # already skips past a marked item's containing ulist/olist/dlist to
+        # the nearest section correctly, no special-casing needed there. Its
+        # own mark, however, never arrives via the item's own attributes
+        # directly, see propagate_list_item_content_marks below.
+        ROLLUP_CONTEXTS = [:paragraph, :ulist, :olist, :dlist, :list_item, :admonition, :open, :quote, :verse, :sidebar].freeze
+
+        # Contexts propagate_list_item_content_marks accepts as a list
+        # item's own marker carrier: everything ROLLUP_CONTEXTS covers
+        # except list_item itself (the item is the propagation target, not
+        # a valid source) and ulist/olist/dlist (a nested sub-list attached
+        # to an item already has its own independent, correct mark-and-
+        # roll-up path via convert_ulist/convert_olist/convert_dlist and
+        # this same ROLLUP_CONTEXTS walk; swallowing its mark into the
+        # parent item here would both steal it from the sub-list and give
+        # the item a mark it never earned).
+        LIST_ITEM_MARKER_CARRIER_CONTEXTS = (ROLLUP_CONTEXTS - [:list_item, :ulist, :olist, :dlist]).freeze
 
         # Walks up from node to its nearest enclosing :section, or nil if
         # none exists, i.e. node sits in the document preamble. Shared by
@@ -732,6 +773,60 @@ module Asciidoctor
           end
         end
 
+        # No canonical AsciiDoc syntax attaches a block attribute list
+        # directly to one list item's own bullet line; confirmed directly
+        # (Asciidoctor.load, a real fixture): the attribute list is silently
+        # dropped, not attached to the item, the list, or anywhere else. The
+        # working construct, the same shape as an a| table cell's own
+        # workaround, is a block attribute list on content the item already
+        # legitimately attaches via a `+` continuation:
+        #
+        #   * Item two
+        #   +
+        #   [rhrev1-2="Added item two"]
+        #   --
+        #   --
+        #
+        # or with real visible continuation text instead of an empty open
+        # block. Copies any rhrev-prefixed attribute from the item's own
+        # directly-attached marker block onto the item itself, then strips
+        # it from that block, the same two reasons propagate_cell_content_
+        # marks strips its own source: convert_list_item's traverse_list_
+        # item call renders an item's attached blocks through this same
+        # converter, so a left-marked carrier would independently trigger
+        # its own bar-inking hook too, and the strip keeps
+        # node_carries_own_rhrev?/rollup_child_revision_entries from later
+        # finding and rolling up the same mark a second time from the
+        # carrier's own now-redundant position in the tree.
+        #
+        # Deliberately only the item's own direct children (item.blocks),
+        # not a recursive descent: a compound item nesting a sub-list of
+        # its own must not have that sub-list's independent marks stolen
+        # and reattributed to the outer item, LIST_ITEM_MARKER_CARRIER_
+        # CONTEXTS excludes ulist/olist/dlist/list_item for exactly this
+        # reason, restricting a valid carrier to paragraph/admonition/open/
+        # quote/verse/sidebar.
+        #
+        # Must run before rollup_child_revision_entries and before real
+        # rendering, so the item's own attributes are already in place by
+        # the time either reads them.
+        def propagate_list_item_content_marks doc
+          doc.find_by(context: :list_item).each do |item|
+            item.blocks.each do |node|
+              next unless LIST_ITEM_MARKER_CARRIER_CONTEXTS.include? node.context
+              attr_entries = (node.instance_variable_get(:@attributes) rescue nil)
+              next if attr_entries.nil? || attr_entries.empty?
+
+              attr_entries.keys.each do |key|
+                key_str = key.to_s
+                next unless key_str.start_with?(@revision_prefix)
+                item.set_attr key_str, attr_entries[key]
+                node.remove_attr key_str
+              end
+            end
+          end
+        end
+
         def rollup_table_cell_revision_entries doc
           doc.find_by(context: :table).each do |table|
             cells = (table.rows[:head] + table.rows[:body]).flatten
@@ -964,6 +1059,7 @@ module Asciidoctor
             end
 
             propagate_cell_content_marks doc
+            propagate_list_item_content_marks doc
             rollup_child_revision_entries doc
             rollup_table_cell_revision_entries doc
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: asciidoctor-rhrev
 version: !ruby/object:Gem::Version
-  version: 1.5.3
+  version: 1.6.0
 platform: ruby
 authors:
 - 白一百 baiyibai