Add missing descriptions to 119 artifacts#940
Merged
Conversation
Fill in empty "description" fields in the __artifacts_v2__ block for 119 artifacts across 86 files. Each description is derived from the artifact's actual paths/query/columns and states what the parser extracts and its source. The description renders in the HTML report header and is carried into LAVA, so these artifacts previously showed no explanation. Two touched files also get the standard top-of-file '# pylint: disable=W0613' pragma (matching 255 other artifact modules) for their unused signature args, and airtagAndroid discards the unused blackboxprotobuf type-defs via '_' so the now-touched files lint at 10.00. No parser logic changed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Fills in empty `description` fields in the `artifacts_v2` metadata block for 119 artifacts across 86 files. The `description` renders in the artifact's HTML report header and is carried into LAVA, so these artifacts previously showed no explanation of what they do.
Each description was written from the artifact's actual paths, SQL query and output columns (not guessed), following the existing house style ("Parses X from Y").
Coverage
This clears every remaining empty-description artifact in ALEAPP (verified: 0 left;
PluginLoaderloads 583 plugins). Files span messaging (Viber, LINE, Skype, Teams, TextNow, IMO, TikTok, MeWe, Tango, Skout, Words With Friends), browsers (Firefox), file transfer (Xender, Zapya, SHAREit, MEGA, PikPak), torrents, GEO/location, Samsung system artifacts (My Files, Weather Clock, Members, ContextLog, recovery/wipe), installed-apps, permissions, Bluetooth, AirTag detection, and more.Notes
teams.py×5,slopes.py×3,airtagAndroid.py×4).accounts_de.py,airtagAndroid.py) also get the standard top-of-file `# pylint: disable=W0613` pragma that 255 other artifact modules already carry (for the unused `report_folder/seeker/wrap_text` signature args), andairtagAndroiddiscards the unusedblackboxprotobuftype-defs via `_`, so the now-touched files lint at 10.00/10. No behavior change.