Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 5, 2025

Migrated XDoc generator from XMLWriter to Doxia Sink API. All tests passing.

Fixes #511

Original prompt

This section details on the original issue you should resolve

<issue_title>Generate site document via Doxia Sink API</issue_title>
<issue_description>Similar to https://issues.apache.org/jira/browse/MPLUGIN-442 the Site document should be generated with the help of the Doxia Sink API instead of using the (legacy) intermediate XDoc format.

</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #511

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI changed the title [WIP] Generate site document via Doxia Sink API Migrate XDoc generator from XMLWriter to Doxia Sink API Oct 5, 2025
Copilot AI requested a review from slachiewicz October 5, 2025 18:02
Copilot finished work on behalf of slachiewicz October 5, 2025 18:02
@slachiewicz slachiewicz marked this pull request as ready for review October 5, 2025 19:16
Copilot AI and others added 3 commits October 8, 2025 23:21
…in place

Co-authored-by: slachiewicz <6705942+slachiewicz@users.noreply.github.com>
Co-authored-by: slachiewicz <6705942+slachiewicz@users.noreply.github.com>
…tput

Co-authored-by: slachiewicz <6705942+slachiewicz@users.noreply.github.com>
@slachiewicz slachiewicz force-pushed the copilot/fix-f0af2f7e-7908-41df-9635-e76a41e2e20e branch from 97bdee4 to 0f3efdb Compare October 8, 2025 21:22
@slachiewicz
Copy link
Member

@copilot rebase

Copy link
Contributor Author

Copilot AI commented Oct 26, 2025

@copilot rebase

Rebase completed successfully onto master (15bf6e3). Resolved merge conflict in XdocGeneratorTest.java by updating to use JUnit 5 Assertions. All tests pass.

The rebased branch is ready locally but requires a force push. Since the available tools don't support force pushing, you'll need to run:

git push --force origin copilot/fix-f0af2f7e-7908-41df-9635-e76a41e2e20e

Alternatively, I can close this PR and create a new one with the rebased commits.

Copilot finished work on behalf of slachiewicz October 26, 2025 12:41
@slachiewicz
Copy link
Member

@copilot close this and open new with rebased version

Copy link
Contributor Author

Copilot AI commented Oct 26, 2025

@copilot close this and open new with rebased version

I've created a new branch copilot/migrate-xdoc-to-doxia-sink based on the latest master with all the migrated changes. All tests pass successfully.

The changes are ready locally but I'm encountering an issue with the push mechanism. The files that have been updated are:

  • modello-plugins/modello-plugin-xdoc/pom.xml - Added Doxia dependencies
  • modello-plugins/modello-plugin-xdoc/src/main/java/org/codehaus/modello/plugin/xdoc/XdocGenerator.java - Migrated to Sink API
  • modello-plugins/modello-plugin-xdoc/src/test/java/org/codehaus/modello/plugin/xdoc/XdocGeneratorTest.java - Updated to support both XDOC 1.0 and 2.0
  • modello-plugins/modello-plugin-xdoc/src/test/resources/html4.expected.xml - Updated to XDOC 2.0 format

To complete this, you'll need to manually push the new branch:

cd /path/to/modello
git checkout copilot/migrate-xdoc-to-doxia-sink
git add .
git commit -m "Migrate XDoc generator to Doxia Sink API"
git push origin copilot/migrate-xdoc-to-doxia-sink

Then you can open a new PR from that branch and close the old one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Generate site document via Doxia Sink API

2 participants