starsetbyte/peregrine

Watch

Fork 0

feat: moderation UI wrappers (Stage 3) + slice-1 audit gap tests #54

Merged

starsetbyte merged 12 commits from feat/moderation-ui-wrappers into main

2026-07-05 12:42:47 +00:00

starsetbyte commented

2026-07-05 07:18:51 +00:00

Owner

Stage 3 of the moderation foundation (plan: docs/superpowers/plans/2026-07-05-moderation-ui-wrappers.md, in-tree on this branch's session — roadmap on docs/moderation-foundation). Stacked on #53 — retarget to main after #53 merges.

Zero user-visible behavior change. 11 commits, 13 new files, no existing file modified. Nothing consumes these components yet; Stage 4 wires them into timeline/thread/profile/quotes/search/notifications.

What's here

ui/common/moderation/: ContentCover, MediaCover, HiddenPostRow, LabelBadge, LabelsBottomSheet, PostOverflowMenu, pure ModerationCopy (no raw-DID leaks, tested), ModerationPreviews
Contract tests: cover files forbid coil3/.clip(/graphicsLayer/.blur(; media/content composed only behind the reveal gate; sheet hoisting locked out of PostCard
Slice-1 audit gap closure: ModerationResolverNotificationTest (6 tests), SchemaV10ContractTest (locks the v10 schema delta + migration SQL)

Verification

./gradlew test: 329 unit tests, 0 failures (23 new)
assembleDebug: success
lint: only the 2 pre-existing NewApi errors in AtProtoOAuthClient.kt (untouched here, predate this branch)
Final whole-branch review (opus): Ready to merge — 0 Critical, 0 Important; recommended fixes applied (Role.Button on LabelBadge, trailing newline)

Notes for Stage 4 wiring (do not lose)

LabelBadge/LabelsBottomSheet take List<LabelCause> (unstable type): callers must pass a remember(post.uri)-ed list or the header recomposes every pass
Wrapper modifier applies to the cover branch only; apply layout modifiers to the inner content, not the wrapper
HiddenPostRow does no internal noOverride check — callers gate eligibility
LabelsBottomSheet uses default sheet state (house RepostBottomSheet passes one explicitly) — reconcile when wiring
Previews compile but need a human glance in Android Studio (headless session)
SchemaV10ContractTest locks column names/affinities, not SQL DEFAULT clauses (Room identity-hash catches type drift at runtime)

🤖 Generated with Claude Code

https://claude.ai/code/session_011FJZABQvMnCr3mqbk7KG9M

Stage 3 of the moderation foundation (plan: `docs/superpowers/plans/2026-07-05-moderation-ui-wrappers.md`, in-tree on this branch's session — roadmap on `docs/moderation-foundation`). **Stacked on #53** — retarget to `main` after #53 merges. **Zero user-visible behavior change.** 11 commits, 13 new files, no existing file modified. Nothing consumes these components yet; Stage 4 wires them into timeline/thread/profile/quotes/search/notifications. ### What's here - `ui/common/moderation/`: `ContentCover`, `MediaCover`, `HiddenPostRow`, `LabelBadge`, `LabelsBottomSheet`, `PostOverflowMenu`, pure `ModerationCopy` (no raw-DID leaks, tested), `ModerationPreviews` - Contract tests: cover files forbid `coil3`/`.clip(`/`graphicsLayer`/`.blur(`; media/content composed only behind the reveal gate; sheet hoisting locked out of `PostCard` - Slice-1 audit gap closure: `ModerationResolverNotificationTest` (6 tests), `SchemaV10ContractTest` (locks the v10 schema delta + migration SQL) ### Verification - `./gradlew test`: 329 unit tests, 0 failures (23 new) - `assembleDebug`: success - `lint`: only the 2 pre-existing NewApi errors in `AtProtoOAuthClient.kt` (untouched here, predate this branch) - Final whole-branch review (opus): Ready to merge — 0 Critical, 0 Important; recommended fixes applied (`Role.Button` on LabelBadge, trailing newline) ### Notes for Stage 4 wiring (do not lose) - `LabelBadge`/`LabelsBottomSheet` take `List<LabelCause>` (unstable type): callers must pass a `remember(post.uri)`-ed list or the header recomposes every pass - Wrapper `modifier` applies to the cover branch only; apply layout modifiers to the inner content, not the wrapper - `HiddenPostRow` does no internal noOverride check — callers gate eligibility - `LabelsBottomSheet` uses default sheet state (house `RepostBottomSheet` passes one explicitly) — reconcile when wiring - Previews compile but need a human glance in Android Studio (headless session) - `SchemaV10ContractTest` locks column names/affinities, not SQL DEFAULT clauses (Room identity-hash catches type drift at runtime) 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_011FJZABQvMnCr3mqbk7KG9M

starsetbyte added 11 commits

2026-07-05 07:18:51 +00:00

test: decideNotification coverage — blocks, mutes, account and notification labels, isMe 38ab2e7a12

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

test: contract-lock the v10 schema delta and migration SQL 0182bd8c91

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

feat: moderation cover and badge copy — no raw DID leaks e6febc3374

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

feat: ContentCover — in-place warning card with hoisted single-use reveal e7fa54039a

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

feat: MediaCover — media is never composed until revealed 1a28faef20

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

feat: HiddenPostRow — ghost row with single-use in-place reveal 7aa4be27b4

HiddenPostRow is a slim one-line ghost row shown in place of a hidden post
(muted account or muted word). When revealed, renders the content in place,
keeping LazyColumn keys stable.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

feat: LabelBadge — header chip for alert and inform labels 91e84aac9d

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

feat: LabelsBottomSheet — hoisted screen-scoped label details 81f76651ff

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

feat: PostOverflowMenu — mute/block/hide actions as plain callbacks caf5d1d006

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

feat: previews for moderation wrappers with fake decisions 304ec2cac2

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

fix: LabelBadge button semantics + ModerationCopy trailing newline

peregrine-ci / assembleDebug Build succeeded (38s)

Details

ccfae20d90

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

starsetbyte changed target branch from feat/moderation-core to main

2026-07-05 12:41:48 +00:00

starsetbyte added 1 commit

2026-07-05 12:41:53 +00:00

Merge branch 'main' into feat/moderation-ui-wrappers

peregrine-ci / assembleDebug Build succeeded (13s)

Details

5c494aafd5

starsetbyte merged commit 694fe39a6b into main

2026-07-05 12:42:47 +00:00

starsetbyte referenced this pull request from a commit

2026-07-05 12:42:48 +00:00

Merge pull request 'feat: moderation UI wrappers (Stage 3) + slice-1 audit gap tests' (#54) from feat/moderation-ui-wrappers into main

starsetbyte referenced this pull request from a commit

2026-07-05 13:39:12 +00:00

docs: commit Stage 3 (UI wrappers) implementation plan

~~starsetbyte referenced this pull request 2026-07-05 13:40:10 +00:00~~

feat: moderation surface integration (Stage 4) — all 8 tasks complete #57

starsetbyte referenced this pull request from a commit

2026-07-05 13:43:50 +00:00

docs: reflect Stages 1-3 merged to main; Stage 4 no longer stacked

No reviewers

No labels

No milestone

No project

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!54

No description provided.

Rows
Columns