Skip to content

fix(db): limit event transactions to listener mention kinds - #7932

Merged
jsibbison-square merged 1 commit into
mainfrom
jsib-260928-limit-listener-mention-transactions
Sep 28, 2026
Merged

jsibbison-square merged 1 commit into
mainfrom
jsib-260928-limit-listener-mention-transactions

Conversation

@jsibbison-square

Copy link
Copy Markdown
Contributor

Summary

  • Keep non-listener-mention event writes on the plain insert path without opening a transaction.
  • Preserve the transaction around listener mention event insertion and outbox enqueue.

Validation

  • cargo fmt --all -- --check (Hermit)
  • Tests not run.

Follow-up to Baxen’s review of #7793, point 1.

Signed-off-by: Implementor <691cca7a870db1dad6990d5938d1b4a2a2ca9647ca81290b536c17b06b2e473f@buzz.block.builderlab.xyz>
@github-actions

Copy link
Copy Markdown

🔐 Codex Security Review

Status: review required for the current range.

The current range is c4c86006f2d67b2ea9055780129ee7ce95c54e0a...56ba689f976af8dd34a08f22636eedd29a31ee77.
A new review must complete for this exact range. When manual authorization
is required, a Block organization member must comment exactly
@buzz-security-review 56ba689f976af8dd34a08f22636eedd29a31ee77 to authorize a new review.
Any previous review applies only to its recorded range.

@jsibbison-square
jsibbison-square marked this pull request as ready for review September 28, 2026 05:02
@jsibbison-square
jsibbison-square requested a review from a team as a code owner September 28, 2026 05:02
@jsibbison-square
jsibbison-square marked this pull request as draft September 28, 2026 05:07
@jsibbison-square
jsibbison-square marked this pull request as ready for review September 28, 2026 05:10
@jsibbison-square
jsibbison-square merged commit 1846094 into main Sep 28, 2026
154 of 157 checks passed
@jsibbison-square
jsibbison-square deleted the jsib-260928-limit-listener-mention-transactions branch September 28, 2026 13:17
wpfleger96 pushed a commit that referenced this pull request Sep 28, 2026
Co-authored-by: Will Pfleger <pfleger.will@gmail.com>
Signed-off-by: Will Pfleger <pfleger.will@gmail.com>

* origin/main:
  fix(relay): fail startup on invalid operator listener config (#7933)
  fix(db): limit event transactions to listener mention kinds (#7932)
  feat(relay): deliver pubkey mentions to relay companions (#7793)
  docs(protocol): propose simplified channel artifacts (#7791)
  feat(push): support configurable HTTP(S) delivery URLs (#7877)
  fix(ci): select runtime suites from PR changes only (#7843)
  test(desktop): synchronize upload edit smoke test (#7903)

Signed-off-by: Hayt <211b96e6a2b7f45fd4047988976c7bbbeeda0c15f3ae7b32eec20834b5a55118@buzz.block.builderlab.xyz>
wpfleger96 pushed a commit that referenced this pull request Sep 28, 2026
* origin/main:
  🤖 docs(nip-fi): remove implementation references from the spec (#7912)
  fix(relay): fail startup on invalid operator listener config (#7933)
  fix(db): limit event transactions to listener mention kinds (#7932)
  feat(relay): deliver pubkey mentions to relay companions (#7793)
  docs(protocol): propose simplified channel artifacts (#7791)
  feat(push): support configurable HTTP(S) delivery URLs (#7877)
  fix(ci): select runtime suites from PR changes only (#7843)
  test(desktop): synchronize upload edit smoke test (#7903)

Signed-off-by: Duncan <dcfd242e557282d7a1e2cf2e6877522682f1e5c6156dc92ca7d90eaedd3b0f95@buzz.block.builderlab.xyz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants