Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Annotation is not identified by Evinse #780

Closed
Lingom-R opened this issue Dec 18, 2023 · 4 comments
Closed

Annotation is not identified by Evinse #780

Lingom-R opened this issue Dec 18, 2023 · 4 comments
Labels
enhancement New feature or request

Comments

@Lingom-R
Copy link

Hi,
I'm likely to use Evinse to identify the dependency reachability. Noticed that Evinse is not looking for annotation from .java files.

Here is an examble:

`package io.abc

import lombok.Data;
import lombok.ToString;

@DaTa
@tostring
public class sample {
......`

Here, I have the dependency (i.e. package) 'Lombok' and mention its function as annotation "@DaTa". However, could not get this in Evinse output.

Pls help me understand, Is that anything I'm missing, or how to get the annotation usage in Evinse result?

@prabhu
Copy link
Collaborator

prabhu commented Dec 18, 2023

@Lingom-R do you have a sample repo for testing?

@prabhu
Copy link
Collaborator

prabhu commented Dec 19, 2023

@Lingom-R, I have a good idea what is going on with lombok. It is currently delomboked by default so we do create/consider the injected methods.

Now trying to find a way to disable delombok to retain the original annotations. Then the next challenge is that usages slicing is currently operating at the method level so need to find a way to inject class annotations.

Test repo used: https://github.com/RameshMF/project-lombok-tutorial

prabhu added a commit that referenced this issue Dec 19, 2023
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu prabhu mentioned this issue Dec 19, 2023
3 tasks
prabhu added a commit that referenced this issue Dec 19, 2023
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
prabhu added a commit that referenced this issue Dec 19, 2023
* Java 21 with gradle 8.5

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>

* Update atom to partly support #780

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>

* Update atom to partly support #780

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>

* Track unused type declarations via usages slices

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>

* container package updates

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>

---------

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu prabhu added the enhancement New feature or request label Dec 19, 2023
@Lingom-R
Copy link
Author

@prabhu Thank you for considering the request and apologize for the delay. Hope, the PR #104 would resolve this.

@prabhu
Copy link
Collaborator

prabhu commented Dec 20, 2023

@Lingom-R, could you test with the latest cdxgen and let me know how it looks?

@prabhu prabhu closed this as completed Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants