rebase

rebase

rebase and land

rebase before landing

include the validator change

Let's move the inline CSS styles to the CSS file

Looks like CI was network issue. I'll restart

The invariants in thread-reducer were removed later in the stack. Specifically in D10609 and D10624.

Going to land as is, created a task to track making ServerStateSyncFullSocketPayload ReadOnly: https://linear.app/comm/issue/ENG-6515/make-serverstatesyncfullsocketpayload-readonly

Nice, def cleaner

Seems fine to me as long as we're consistent with this sort of thing.

Unsolicited, but would prefer a simple "No results" without the "Please try searching a different username."

Look good. Personally would've preferred deduplicating BlockListModal, but not sure how involved that would've been.

stage more

include more changes

In D10622#307511, @bartek wrote:

Re. failing Codegen JSI

fatal: detected dubious ownership in repository at '/comm'
To add an exception for this directory, call:
 
	git config --global --add safe.directory /comm

Not sure what started to cause this, but I had the exact same problem with the "gRPC-Web Codegen CI" job and fixed it by adding the following command:

steps:
  - label: 'Foo'
    command:
      - 'git config --global --add safe.directory "*"'
      - ...

include some more

land

make cookieChange readonly

make readonly

remove extraneous change

address feedback (make readonly)

Thanks for linking the Linear task!

pre-commit hooks aren't catching flow issues in keyserver, will try a clean install