Skip to content

Add restrictions on generative AI usage in contributions#1

Merged
7w1 merged 3 commits intoSableClient:devfrom
dozro:patch-1
Mar 9, 2026
Merged

Add restrictions on generative AI usage in contributions#1
7w1 merged 3 commits intoSableClient:devfrom
dozro:patch-1

Conversation

@dozro
Copy link
Copy Markdown
Contributor

@dozro dozro commented Mar 9, 2026

@dozro dozro requested a review from a team March 9, 2026 19:26
@dozro dozro requested a review from 7w1 as a code owner March 9, 2026 19:26
@7w1 7w1 added the internal label Mar 9, 2026
@7w1 7w1 merged commit 454dce2 into SableClient:dev Mar 9, 2026
5 checks passed
@dozro dozro deleted the patch-1 branch March 9, 2026 22:03
Just-Insane added a commit that referenced this pull request Mar 17, 2026
With classic sync, RoomViewHeader creates Thread objects via
room.createThread(id, rootEvent, [], false) — passing no initialEvents.
This means thread.events starts as just [rootEvent] (or empty).

Two bugs resulted:

1. The gate `if (fromThread.length > 0)` blocked the live-timeline fallback.
   Since thread.events = [rootEvent], length was 1 (truthy), but after
   filtering the root out the array was empty — yielding zero replies even
   though the events were present in the main room timeline.
   Fix: compute the filtered array first, then gate on its length so the
   fallback is reached when the thread object has no actual replies yet.

2. Even after #1, subsequent renders and read-receipt logic used thread.events
   (empty) rather than the live timeline.
   Fix: add a mount-time useEffect that backfills matching events from the
   unfiltered live timeline into the Thread object via thread.addEvents() so
   the authoritative source is populated for future interactions.
@SableClient SableClient locked as resolved and limited conversation to collaborators Mar 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants