Skip to content

feat: project checks for Info.plist and AndroidManifest - #52

Merged
saimskywalker merged 2 commits into
mainfrom
issue-51-project-checks
Nov 6, 2026
Merged

saimskywalker merged 2 commits into
mainfrom
issue-51-project-checks

Conversation

@saimskywalker

@saimskywalker saimskywalker commented Nov 6, 2026 •

Copy link
Copy Markdown
Owner

Adds a checks: section to storehound.yaml for app-specific pins on Info.plist keys and AndroidManifest attributes, reported as check:<name> with the value's file and line. Privacy manifest and entitlements come in a follow-up.

Verified: dart format, dart analyze --fatal-infos and dart test pass; the production fixture now carries five checks and still runs clean.

Part of #51

A `checks:` section in storehound.yaml pins app-specific values in
Info.plist (nested keys too) and in AndroidManifest attributes, with the
operators exists, absent, equals, absent_or_equals, not_equals, contains,
none_of and matches. Each check needs a reason and runs as the rule
`check:<name>`, so text, JSON, SARIF, ignore, severity and --explain work
as for any rule. Mistakes in a check are configuration errors with a line.

Part of #51.
@coderabbitai

coderabbitai Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: fe25038f-28d2-4766-965c-454541224208
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@saimskywalker
saimskywalker merged commit e30afe6 into main Nov 6, 2026
5 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.

1 participant