starsetbyte/peregrine
Watch
1
0
Fork
You've already forked peregrine
 0
Code Issues 14 Pull requests 1 Projects Releases Packages Wiki Activity Actions

WIP: feat/moderation #11

Closed
starsetbyte wants to merge 26 commits from feat/moderation into main
pull from: feat/moderation
merge into: starsetbyte:main
Conversation 3 Commits 26 Files changed 59 +6858 -27
starsetbyte commented 2026-06-16 23:43:26 +00:00
Owner
Copy link
No description provided. 
Covers ModerationEngine (FilterResult sealed class), data layer
(ModerationRepository, LabelerRepository, new Room entities), UI layer
(PostCard ghost rows + warning cards + overflow menu, per-labeler
content filtering screens, mute words chip cloud), and like/repost
consistency fix for SearchScreen.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Replaces MVP_IMPLEMENTATION_PLAN.md. Reflects current status (phases 0-6,
search, and feed sync complete), adds moderation as the next active phase
with pointers to the spec and plan, removes Play Store as a hard gate,
and adds muteActor/block/labeler lexicon facts to the AT Protocol reference.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Move Label from NotificationModels to FeedModels (same package, no import change needed)
- Add Label.neg field for negation support
- Add labels: List<Label> to Author and PostView
- Add blocking: String? to ViewerState (AT-URI of own block record)
- Add LabelersPref + LabelerPrefItem to PreferenceModels with serializer case
- Create LabelerModels.kt: LabelBehavior enum, LabelDefinition, LabelLocale, LabelerView, LabelerPolicies, GetLabelerServicesResponse
- Create MuteWord.kt: text + expiresAt (epoch millis, null = permanent)
- Add labelsJson: String? to PostEntity (DB version bump deferred to Task 2)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- LabelsBottomSheet: ModalBottomSheet listing all labels with advisory/normal
  icon distinction and labeler attribution
- PostOverflowMenu: Dropdown with mute/unmute and block/unblock toggles
- PostCard: new label bubble (tag emoji + count) in PostHeader row that opens
  LabelsBottomSheet; PostOverflowMenu added at end of PostActions row
- New PostCard params (all defaulted): onMute, onBlock, labelerNameFor
- FeedViewModel, CustomFeedViewModel, ThreadViewModel, SearchViewModel, ProfileViewModel: inject ModerationEngine + LabelerRepository, add checkModeration/labelerNameFor/muteAccount/blockAccount
- FeedRoute, ThreadScreen, SearchScreen, FeedHostScreen: pass filterResult and moderation lambdas to PostCard, skip Blocked items
LabelerRepository.init fires at app startup before OAuth session exists.
syncFromServer() calls getPreferences() → 401 → swallowed by runCatching.
Default labeler subscribed locally but labeler views never loaded.

Fix:
- Inject SessionManager, collect authState flow
- Wait for Authenticated state before making any network calls
- After subscribing default labeler, pre-load getLabelerViews() so
  label definitions (descriptions, severity, blurs) are available
  for content filtering immediately
- Add diagnostic logging (LabelerRepo, DpopInterceptor, PdsRouting)
- Add testOptions.returnDefaultValues for android.util.Log in tests
fix(postcard): align overflow icon with the other four action buttons
All checks were successful
peregrine-ci / assembleDebug Build succeeded (1m5s)
Details
03486c3e2a 
PostOverflowMenu used Material3's IconButton (40.dp default touch target),
while reply/repost/like/share in PostActions are a custom Row+clickable+
padding(6.dp) wrapping an 18.dp icon — a visibly smaller box. That size
mismatch made the overflow icon the only one of the five not lined up.
Rebuilt it with the same Box+clickable+padding(6.dp)+size(18.dp) pattern.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
starsetbyte commented 2026-06-16 23:44:03 +00:00
Author
Owner
Copy link

this is still a work in progress and is not ready to commit. labels do not work as intended currently and do not apply consistently

this is still a work in progress and is not ready to commit. labels do not work as intended currently and do not apply consistently
starsetbyte changed title from feat/moderation to WIP: feat/moderation 2026-06-17 00:00:14 +00:00
starsetbyte commented 2026-07-13 01:30:21 +00:00
Author
Owner
Copy link

Triage review — reconstruction required before content review

This draft is not reviewable against current main: it is 294 commits behind and Forgejo reports mergeable=false. A real synthetic merge against current origin/main confirms conflicts across the build configuration, schema, API models/services, data layer, navigation, feed/profile/search surfaces, and moderation UI/tests.

Please rebase/reconstruct from current main and split the current intent into mergeable slices before requesting code review. Reviewing the present 6,858-line diff would not give a trustworthy merge verdict.

## Triage review — reconstruction required before content review This draft is not reviewable against current `main`: it is **294 commits behind** and Forgejo reports `mergeable=false`. A real synthetic merge against current `origin/main` confirms conflicts across the build configuration, schema, API models/services, data layer, navigation, feed/profile/search surfaces, and moderation UI/tests. Please rebase/reconstruct from current `main` and split the current intent into mergeable slices before requesting code review. Reviewing the present 6,858-line diff would not give a trustworthy merge verdict.
starsetbyte commented 2026-07-26 03:22:34 +00:00
Author
Owner
Copy link

Closing as abandoned. This was reference material only and was never to be merged — recorded as such in CLAUDE.md. Moderation shipped separately and completely via PRs #53–54, #57–66. Both feat/moderation and worktree-feat+moderation are retained as reference; neither has been deleted.

Reconciled in docs/reviews/2026-07-25-branch-pr-reconciliation.md, which landed on main via #103.

Closing as abandoned. This was reference material only and was never to be merged — recorded as such in `CLAUDE.md`. Moderation shipped separately and completely via PRs #53–54, #57–66. Both `feat/moderation` and `worktree-feat+moderation` are **retained** as reference; neither has been deleted. Reconciled in `docs/reviews/2026-07-25-branch-pr-reconciliation.md`, which landed on `main` via #103.
starsetbyte closed this pull request 2026-07-26 03:22:34 +00:00
All checks were successful
peregrine-ci / assembleDebug Build succeeded (1m5s)
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
auth
Authentication and OAuth

  
bug
Something is broken

  
composer
Post composition and drafts

  
correctness
Logic and state correctness

  
database
Room, migrations, persistence

  
durability
Data safety, persistence, recovery

  
enhancement
New feature or request

  
feeds
Feed management and preferences

  
interactions
Like, repost, reply interactions

  
notifications
Notification system

  
performance
Performance issues

  
reliability
Resilience against interruptions

  
safety
User safety and trust

  
security
Security concerns

No labels
auth
bug
composer
correctness
database
durability
enhancement
feeds
interactions
notifications
performance
reliability
safety
security
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
starsetbyte/peregrine!11
No description provided.