Feat: enhance Result reporting and implement Vedantu learning module#295
Feat: enhance Result reporting and implement Vedantu learning module#295
Conversation
kaifcodec
commented
Mar 11, 2026
- Introduces a dynamic extra field to the Result class for capturing auxiliary data during scans.
- Updates the reporting core to support the new field across JSON, CSV, and debug string outputs.
- Adds localized terminal display logic for extra information using cyan formatting for better readability.
- Synchronizes the test suite to ensure full coverage of the new data fields and output formats.
- Implements the Vedantu learning module with support for capturing masked phone numbers via the extra field.
VamatoHD
left a comment
There was a problem hiding this comment.
In my opinion, the new extra should appear in another place, since Vedantu [Phone: +9*******78] (___@gmail.com): Registered adds a bit of visual clutter.
Perhaps showing it in a new line, like:
Vedantu (___@gmail.com): Registered
Extra: Phone number: "+9*******78"
Or showing it at the end of the line, in case the message is small:
Vedantu (___@gmail.com): Registered [Phone: "+9*******78"]
|
Yeah, that needs to be changed, it was just a demo. Iβm planning to show the [i] Scan complete.
# Extra info (only if it gets any)
Total hits: 1This approach keeps the output cleaner. Also, I donβt expect too many extra details in most cases, especially in I mainly added this as a placeholder in case we move forward with extracting additional details from usernames in the future. Let me know if you think something else would work better. |
|
@kaifcodec I'll look into it. Also, I don't think showing it at the end is better. |
|
@VamatoHD Ok, when it's ready let me know what you are going to do. |
|
@kaifcodec After trying a couple of approaches, I really liked the following: |
|
@VamatoHD Yeah this approach is good, but we also need to adapt it later when we move forward for the deep scans in future to return the |
