Если руками открывать и закрывать транзакцию в целевом методе, то тогда не понятно, зачем свою аннотацию делать, если всё равно метод приходится менять, чтобы её использовать. Открывать и закрывать в аспекте - это надо знать, есть ли вообще над целевым методом @Transactional. По-моему, в существующем виде это очень неуниверсальное решение получается.
По-моему, смысл аннотаций в том, что ты ими меняешь / расширяешь поведение метода, не меняя сам метод. Если тебе приходится весь класс раскорячить, чтобы аннотацию прилепить, то от неё один вред и никакой пользы