Pushed a small follow-up in cdec97d to address patch coverage:

• added a missing regression test for create() without resource_id (expects ValueError)

Local run on this branch now shows 100% coverage for pyinaturalist.controllers.annotation_controller, so this should clear the codecov/patch check on rerun.

Thanks for the review. I pushed follow-up commit 6d7acef with the requested changes:

• switched annotation create tests to use requests_mock.post (instead of patching controller post)
• replaced manual try/except assertions with pytest.raises
• updated term label resolution to use AnnotationController.term_lookup cache instead of calling all()

Validation run on this branch:

• uv run pytest test/controllers/test_annotation_controller.py (11 passed)

If this looks good, please re-review when you have a moment.

All review threads from the previous round are now addressed and marked resolved, and CI is green on the latest commit.\n\nWhen you have a chance, could you take a final re-review pass?

Good catch, thanks. I pushed follow-up commit 542bc3c to make resource_id a required positional parameter in annotations.create().

What changed:

• updated AnnotationController.create() signature so resource_id is required
• removed the now-redundant runtime resource_id is None guard
• updated examples/tests accordingly

Validation:

• uv run pytest test/controllers/test_annotation_controller.py test/docs/test_signatures.py (25 passed)

Looks good. Thank you!