[jackohilts]

Summary

Adds two rules from mtuchi's review of the openstreetmap adaptor (#1833) to the adaptor generation and review guidance. Stacked on #1829: it targets that branch and adds to the Design Rules section #1829 introduces.

Details

The two rules:

1. Name functions with a verb for what they do, not after the endpoint they call. runQuery, not interpreter. This applies to helpers in Utils.js too.
2. Options with a unit say it in the name. For example timeoutMs and queryTimeoutSeconds, and never two options with the same name in different units. In Add openstreetmap adaptor (read-only, via the Overpass API) #1833, RequestOptions.timeout was in milliseconds and QueryOptions.timeout was in seconds.

Where they go:

• claude.md: two new bullets under Design Rules, so generation applies them.
• wiki/best-practice.md: a new "Naming" section under Common Patterns, with the reasoning.
• wiki/code-review.md: two checklist items under API design, linking to the new section. Both are at the default severity for API design (Recommended).

The rules apply to new code. Common's request keeps its existing timeout (ms) option.

Notes:

• claude.md and both wiki files already fail prettier --check on Improve adaptor generation and review guidance #1829's branch. I left that alone to keep this diff to the 24 added lines.
• Docs only, so no changeset.

AI Usage

Please disclose how you've used AI in this work (it's cool, we just want to
know!):

• I have used Claude Code
• I have used another model
• I have not used AI

You can read more details in our
Responsible AI Policy

Review Checklist

Before merging, the reviewer should check the following items:

• Does the PR do what it claims to do?
• If this is a new adaptor, added the adaptor on marketing website ?
• If this PR includes breaking changes, do we need to update any jobs in
  production? Is it safe to release?
• Are there any unit tests?
• Is there a changeset associated with this PR? Should there be? Note that
  dev only changes don't need a changeset.
• If there is a changeset, was pnpm run version used to bump versions (not
  pnpm changeset version directly)? This ensures changelog dates are stamped correctly.
• Have you ticked a box under AI Usage?

---

Generated by Claude Code

[mtuchi]

Hey @jackohilts thanks for the suggestion, I have incorporated you suggestion in this pr #1829