Skip to content

Add more Radar tools #139

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

Merged
merged 4 commits into from
May 2, 2025
Merged

Add more Radar tools #139

merged 4 commits into from
May 2, 2025

Conversation

andre-j3sus
Copy link
Collaborator

@andre-j3sus andre-j3sus commented May 2, 2025

Add tools to retrieve information about:

  • Layer 3 attacks
  • Internet services ranking
  • Internet speed
  • DNS
  • Email Routing
  • Email Security

Created the resolveAndInvoke function to resolve and invoke a method from the TS library dynamically based on the provided slugs. This is useful for Radar because, for each dataset, we have many different endpoints (one for each dimension and data format). This approach allows us to create only one tool per dataset, as creating a tool for each endpoint seems excessive. I'm not sure if this is the best approach, and I'm open to suggestions on how to improve it. Also, I tried to improve the typing for that method, but I'm having a hard time.

@andre-j3sus andre-j3sus self-assigned this May 2, 2025
@andre-j3sus andre-j3sus marked this pull request as ready for review May 2, 2025 11:28
@andre-j3sus andre-j3sus requested a review from Maximo-Guk May 2, 2025 12:57
@andre-j3sus andre-j3sus merged commit 766d426 into main May 2, 2025
5 of 6 checks passed
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.

2 participants