Fix Biolink type lookup filters

[shuchenliu]

• Accept both the repeatable biolink_type parameter and the biolink_types alias.
• Remove the optional biolink: prefix correctly using removeprefix.
• Add coverage for singular and plural parameters.

[Copilot]

🟢 Approval recommended

The changes are focused, with only a non-blocking OpenAPI documentation nit remaining.

Pull request overview

Fixes Biolink type lookup filtering by supporting singular/plural query parameters and normalizing prefixes.

Changes:

• Supports biolink_type and biolink_types.
• Removes the optional biolink: prefix.
• Adds coverage for both parameter forms.

File summaries

File	Summary
test/test_lookup_query.py	Adds singular and plural filter coverage.
src/nameres/handlers/lookup.py	Updates alias handling and prefix normalization; OpenAPI documentation for the plural query alias remains a nit.

Review details

Suppressed comments (1)

src/nameres/handlers/lookup.py:268

• This adds biolink_types as a public query alias, but the OpenAPI definitions for both /lookup query operations still advertise only biolink_type; the existing plural schema is a request-body property. Without updating the API specification, generated clients and Swagger users cannot discover or reliably use the alias. Please add the plural query parameter to the relevant OpenAPI operations as well.

        biolink_types = [*self.get_arguments("biolink_types"), *self.get_arguments("biolink_type")]

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.