Skip to content

[WIP] Refactor internal architecture of Permission Analyzer#77

Merged
muhammad7865 merged 1 commit intoPermission_Analyzerfrom
copilot/sub-pr-76
Mar 14, 2026
Merged

[WIP] Refactor internal architecture of Permission Analyzer#77
muhammad7865 merged 1 commit intoPermission_Analyzerfrom
copilot/sub-pr-76

Conversation

Copy link
Contributor

Copilot AI commented Mar 14, 2026

  • Create shared Hashing utility in core/security/ to deduplicate sha256 logic
  • Update MalwareRepositoryImpl to use shared hashing utility
  • Update PermissionRepositoryImpl to use shared hashing utility, replace riskScore with _, fix misleading log label
  • Fix AppCategoryClassifier: remove leading space from ANTIVIRUS label " Security""Security"
  • Fix CyberShieldDatabase: add KDoc to MIGRATION_10_11 stating cache rows are cleared
  • Fix permission_analyzer_commit_summary.md: correct line 11-12 claim "without wiping existing data"
  • Fix PermissionRiskScorerTest: assertTrue(score == 95)assertEquals(95, score)
  • Fix PermissionRiskScorerTest: rename test/fix comment for ML score 99, not 100
  • Fix PermissionRiskScorerTest: replace unused riskLevel with _ in destructuring

📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

Copilot AI mentioned this pull request Mar 14, 2026
@muhammad7865 muhammad7865 marked this pull request as ready for review March 14, 2026 12:55
@muhammad7865 muhammad7865 merged commit 5d602a6 into Permission_Analyzer Mar 14, 2026
1 check failed
Copilot AI requested a review from muhammad7865 March 14, 2026 12:56
Copilot stopped work on behalf of muhammad7865 due to an error March 14, 2026 12:56
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