Release: develop -> main - #96
Merged
Merged
Conversation
* 01a057e2 - Omit hasVideo when the forum file is gone List, public GET, and replies advertised a player for rows whose bytes were missing or empty. Stat the media file first and clear the flags. * 01a057e2 - Delete forum rows whose video file is gone A hasVideo note without bytes on disk is not a note. Public GET, list, and replies remove the row (and direct replies) instead of leaving an empty player. * 01a057e2 - Type deleteById on MessageStore mocks Store wrappers in invoice and zap tests must implement the new delete. Narrow videoContentType before unlinking. * 01a057e2 - Use strict equality for videoContentType guards eqeqeq forbids != null. Spell out undefined and null before unlinking. * 01a057e2 - Wrap the missing-video guard for Prettier The hasVideo / videoContentType null check exceeded the print width. * 01a057e2 - Cover deleteById video unlinks and missing-file replies Postgres delete with a video MIME unlinks the file. Replies whose clip is gone are dropped from GET /messages/:id/replies. * Forget zap receipts on delete and recount live replies List drops missing-file video replies before replyCount. Postgres deleteById is one CTE. InMemory releases receipt ids so a replay can be recorded again. * Format InMemory deleteById cascade test * Treat only missing video files as gone Non-ENOENT stat errors no longer delete the note. List replyCount is the stored count minus drops in the replies window. --------- Co-authored-by: TaprootFreakAI <315477232+TaprootFreakAI@users.noreply.github.com>
TaprootFreak
approved these changes
Sep 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automatic Release PR
Commits: 1 new commit(s)