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
@slachiewicz slachiewicz force-pushed the copilot/fix-f0af2f7e-7908-41df-9635-e76a41e2e20e branch from 97bdee4 to 0f3efdb Compare October 8, 2025 21:22
@slachiewicz

This comment has been minimized.

This comment has been minimized.

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

This comment has been minimized.

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.

Co-authored-by: slachiewicz <6705942+slachiewicz@users.noreply.github.com>
@slachiewicz slachiewicz force-pushed the copilot/fix-f0af2f7e-7908-41df-9635-e76a41e2e20e branch from 0f3efdb to e165abe Compare November 3, 2025 17:52
@slachiewicz slachiewicz marked this pull request as draft November 3, 2025 17:52
@slachiewicz slachiewicz closed this Nov 3, 2025
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