lytedev/nix
1
0
Fork 2
Code Pull requests 9 Activity Actions

feat(iamb): bump the fork for thread mentions in the inbox #1197

Open
lytedev wants to merge 1 commit from iamb-thread-mentions into main
pull from: iamb-thread-mentions
merge into: lytedev:main
Conversation 0 Commits 1 Files changed 1 +10 -2
lytedev commented 2026-09-15 09:52:06 -05:00
Owner
Copy link

Bumps the iamb pin so that :unreadmentions can show an @-mention that arrived inside a thread. Please read the commits; the overlay comment in lib/overlays/default.nix carries the reasoning that belongs with the code.

Verified with nix build .#iamb against the new rev.

🤖 Generated with Claude Code

https://claude.ai/code/session_01R2biq8jbdUGW47SfFqeXKQ

Bumps the iamb pin so that `:unreadmentions` can show an @-mention that arrived inside a thread. Please read the commits; the overlay comment in `lib/overlays/default.nix` carries the reasoning that belongs with the code. Verified with `nix build .#iamb` against the new rev. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01R2biq8jbdUGW47SfFqeXKQ
feat(iamb): bump the fork for thread mentions in the inbox
Some checks failed
/ check-format (push) Successful in 13s
Details
/ build (push) Failing after 3h12m37s
Details
25d470526b 
`:unreadmentions` was built from rooms and DMs alone, so an @-mention that
arrived inside a thread had nowhere to appear: a room's unread state comes
from its main scrollback, a threaded reply never lands there, and the room
therefore read as read. The window that exists for mentions was the one
window that could not show a threaded one.

The fork now lists followed threads there beside rooms, the way
`:unreadthreads` already does, and being named in a thread is itself enough
to follow it -- without which a mention in a thread the user has never
posted in stays invisible to every window that lists followed threads.

Tested by building .#iamb against the new rev, and by the fork's own tests
over the mention detection and the thread-following rule.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01R2biq8jbdUGW47SfFqeXKQ
Some checks failed
/ check-format (push) Successful in 13s
Required
Details
/ build (push) Failing after 3h12m37s
Required
Details
Some required checks were not successful.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin iamb-thread-mentions:iamb-thread-mentions
git switch iamb-thread-mentions
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug
Something is wrong

  
enhancement
New feature

  
question
Awaiting information

  
scary
Potentially problematic change

  
upstream
Awaiting upstream changes

No labels
bug
enhancement
question
scary
upstream
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
lytedev/nix!1197
No description provided.