[ryanmeowy]

Issue resolved by this Pull Request:
Resolves #4749

Checklist:

• Documentation has been updated, if necessary. (Not needed. The reader is internal.)
• Examples have been added, if necessary. (Not needed.)
• Tests have been added, if necessary. tests/test_backend_opendocument.py builds an ODT for a table of contents and for each other index kind, and asserts the text and the item labels.

Review evidence:

Behavior before. The body of the ODT holds two paragraphs and a table of contents with an index title and two entries. The export holds the two paragraphs only:

Intro paragraph.

Outro paragraph.

The document has two text items. _add_odf_child() sends the index element to the branch that reads images only, so the paragraphs inside text:index-body are never visited. Every index kind of ODF 1.2 has its own tag and meets the same branch.

Behavior after.

Intro paragraph.

Table of Contents

Chapter One	1

Section One	2

Outro paragraph.

Document tree: the index title and the entries are DocItemLabel.TEXT items in document order, between the two paragraphs, and no other item is added. The text:index-title-template of text:table-of-content-source stays out of the document: it is a template, and LibreOffice renders the text:index-title paragraph of the index body.

The index containers are transparent now, the same way text:section is. The containers are text:index-body and text:index-title, and they hold the entries of every index kind: text:table-of-content, text:alphabetical-index, text:illustration-index, text:table-index, text:object-index, text:user-index and text:bibliography. The hidden section behavior stays: text:display="none" sends the content to the invisible content layer, and the same holds for a hidden index.

Checks.

Check	Command	Result
Table of contents test, red first	pytest tests/test_backend_opendocument.py -k table_of_contents without the source change	1 failed
Index kind tests, red first	pytest tests/test_backend_opendocument.py -k index without the source change	6 failed
New tests, after the fix	pytest tests/test_backend_opendocument.py	68 passed
Other ODF callers	pytest tests/test_cli.py tests/test_input_doc.py	123 passed
Format and lint	ruff format --check and ruff check on the two changed files	All checks passed
Line limit	scripts/check_max_lines.py	16 files over the limit, the same as main; the two changed files are exempted in .github/max-lines-ignore
Type check	ty check	618 diagnostics, the same as main, none on the new lines

No reference-data changes: no fixture under tests/data/ holds an index element, and the new tests build their own documents.

Scope: this branch is rebased onto main at the writing time. The rebase keeps the hidden section handling of #4860.

[github-actions]

✅ DCO Check Passed

Thanks @ryanmeowy, all your commits are properly signed off. 🎉

[mergify]

Merge Protections

🟢 Merge protection satisfied — ready to merge.

Show 1 satisfied protection

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

• title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[github-actions]

AI triage (advisory)

Checked commit e59721f16c. The analysis read the diff only. It did not run code from this pull request.

The PR fixes the ODF backend so it keeps the text of a table of contents and of other index kinds. It treats the index container elements as transparent, in the same way as text:section. It adds tests for this.

CI safety: no concern found. A maintainer must still approve CI.

_{The ai:* labels are advisory. They do not approve CI and they do not add tests:full. A new push runs this check again.}

[github-actions]

AI first review (advisory): no blocker found

This review covers the full diff at c6beb713ea. It read the code and did not run it.

The fix is small and handles a table of contents correctly. The main risk is that the same data loss stays for the other ODF index types. The reviewer must decide whether to widen the tag set in this PR or to track those types in a follow-up issue.

_{A maintainer makes the merge decision. This review never approves the pull request.}

[github-actions]

AI first review (advisory): no blocker found

This review covers the full diff at e59721f16c. It read the code and did not run it.

I found no finding that decides the merge. The change makes the index containers transparent, the same way text:section already is, and the two new tests cover the table of contents and the other index kinds. The odfdo package in the environment has no class for these tags, so the tag-based match is needed.

_{A maintainer makes the merge decision. This review never approves the pull request.}