fix: use IsProjectMemberOrReadOnly for TaxaListViewSet permissions

ObjectPermission doesn't work for M2M-to-project models because
BaseModel.get_project() returns None when get_project_accessor()
returns "projects". This caused all write operations (create, update,
delete) on taxa lists to be denied for every user.

Switch to IsProjectMemberOrReadOnly which resolves the project via
ProjectMixin.get_active_project() (from query param) instead of
through the model instance. Add 10 permission tests covering member
CRUD, anonymous read-only, non-member rejection, and owner access.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: mihow

ami/main/api/views.py

@@ -1634,7 +1634,7 @@ class TaxaListViewSet(DefaultViewSet, ProjectMixin):
         "created_at",
         "updated_at",
     ]
-    permission_classes = [ObjectPermission]
+    permission_classes = [IsProjectMemberOrReadOnly]
     require_project = True
 
     def get_queryset(self):
@@ -1652,8 +1652,6 @@ def perform_create(self, serializer):
 
         Users cannot manually assign taxa lists to projects for security reasons.
         A taxa list is always created in the context of the active project.
-
-        @TODO Do we need to check permissions here? Is this user allowed to add taxa lists to this project?
         """
         instance = serializer.save()
         project = self.get_active_project()

ami/main/tests.py

@@ -3445,6 +3445,71 @@ def test_taxon_detail_visible_when_excluded_from_list(self):
         self.assertEqual(res.status_code, status.HTTP_200_OK)
 
 
+class TaxaListViewSetPermissionTestCase(TestCase):
+    """Test TaxaListViewSet CRUD permissions for project members vs anonymous users."""
+
+    def setUp(self):
+        self.owner = User.objects.create_user(email="owner@example.com", password="testpass")
+        self.member = User.objects.create_user(email="member@example.com", password="testpass")
+        self.non_member = User.objects.create_user(email="nonmember@example.com", password="testpass")
+        self.project = Project.objects.create(name="Test Project", owner=self.owner)
+        self.project.members.add(self.member)
+        self.taxa_list = TaxaList.objects.create(name="Existing List", description="A list")
+        self.taxa_list.projects.add(self.project)
+        self.client = APIClient()
+        self.list_url = f"/api/v2/taxa/lists/?project_id={self.project.pk}"
+        self.detail_url = f"/api/v2/taxa/lists/{self.taxa_list.pk}/?project_id={self.project.pk}"
+
+    def test_member_can_list_taxa_lists(self):
+        self.client.force_authenticate(self.member)
+        response = self.client.get(self.list_url)
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+
+    def test_member_can_create_taxa_list(self):
+        self.client.force_authenticate(self.member)
+        response = self.client.post(self.list_url, {"name": "New List"})
+        self.assertEqual(response.status_code, status.HTTP_201_CREATED)
+
+    def test_member_can_update_taxa_list(self):
+        self.client.force_authenticate(self.member)
+        response = self.client.patch(self.detail_url, {"name": "Renamed"})
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+        self.taxa_list.refresh_from_db()
+        self.assertEqual(self.taxa_list.name, "Renamed")
+
+    def test_member_can_delete_taxa_list(self):
+        self.client.force_authenticate(self.member)
+        response = self.client.delete(self.detail_url)
+        self.assertEqual(response.status_code, status.HTTP_204_NO_CONTENT)
+
+    def test_anonymous_can_read_taxa_lists(self):
+        response = self.client.get(self.list_url)
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+
+    def test_anonymous_cannot_create_taxa_list(self):
+        response = self.client.post(self.list_url, {"name": "Anon List"})
+        self.assertIn(response.status_code, [status.HTTP_401_UNAUTHORIZED, status.HTTP_403_FORBIDDEN])
+
+    def test_anonymous_cannot_update_taxa_list(self):
+        response = self.client.patch(self.detail_url, {"name": "Hacked"})
+        self.assertIn(response.status_code, [status.HTTP_401_UNAUTHORIZED, status.HTTP_403_FORBIDDEN])
+
+    def test_non_member_cannot_create_taxa_list(self):
+        self.client.force_authenticate(self.non_member)
+        response = self.client.post(self.list_url, {"name": "Intruder List"})
+        self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
+
+    def test_non_member_cannot_update_taxa_list(self):
+        self.client.force_authenticate(self.non_member)
+        response = self.client.patch(self.detail_url, {"name": "Hacked"})
+        self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
+
+    def test_owner_can_manage_taxa_list(self):
+        self.client.force_authenticate(self.owner)
+        response = self.client.post(self.list_url, {"name": "Owner List"})
+        self.assertEqual(response.status_code, status.HTTP_201_CREATED)
+
+
 class TaxaListTaxonAPITestCase(TestCase):
     """Test TaxaList taxa management operations via API."""