Skip to content

[DO NOT MERGE] An AI-assisted attempt to convert Weld from classfilewriter to Gizmo 2.#3302

Draft
manovotn wants to merge 1 commit intoweld:masterfrom
manovotn:gizmoTest
Draft

[DO NOT MERGE] An AI-assisted attempt to convert Weld from classfilewriter to Gizmo 2.#3302
manovotn wants to merge 1 commit intoweld:masterfrom
manovotn:gizmoTest

Conversation

@manovotn
Copy link
Member

This is just a result of an experiment I did with Claude; just wanted to put it out here to have a record of it.

The migration is NOT complete (see ignored tests, 4 are still failing) and the result obviously cannot run on EE servers (they lack Gizmo dependency).
That said, 99+% of tests outside of container pass.

Code state is NOT final and would need a revision:

  • Serialization of proxies was omitted (possibly as it cannot be tested without EE server)
  • Some parts of migrated code now look like a dead end that's not even used
    • Classes inside org.jboss.weld.util.bytecode package are a good example of that

The migration is NOT complete (see ignored tests, 4 are still failing) and the result obviously cannot run on EE servers (they lack Gizmo dependency).
That said, 99+% of tests outside of container pass which is pretty nice.
Code state is NOT final and would need a revision too!
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.

1 participant