A few things:

1. Optimistic state should be updated using useTransition() rather than just a normal function.
2. Unlike a form action, your state won't be refetched from the server when your optimistic state finishes. You need to manually call router.refresh() or something when using useTransition() and updating the state on the server.