This patch looks allright. Another option is to remove the ClassPool stuff from "doInitialize" and create a LifecycleEventHandler (registered in LifecycleState.PRE_INIT) to extract out the logic from the server itself (I recommend this approach).
This patch leads spi-test to fail with:
java.lang.NoClassDefFoundError: org/jboss/classpool/scoped/ScopedClassPoolRepository
...which is just an issue of adding the dep to the spi-test POM config to be copied into target/lib.
S,
ALR