Skip to content

feat: add vk_users_search tool#5

Merged
bulatko merged 1 commit into
bulatko:masterfrom
sergiygoncharov:feat/users-search
Jul 25, 2026
Merged

feat: add vk_users_search tool#5
bulatko merged 1 commit into
bulatko:masterfrom
sergiygoncharov:feat/users-search

Conversation

@sergiygoncharov

Copy link
Copy Markdown
Contributor

Summary

API

users.searchhttps://dev.vk.com/ru/method/users.search

Test plan

  • Tested live against VK API: query "Иван Иванов" returns 897 983 results with photo_200, online fields
  • Optional filters (city, sex, age_from/age_to) are correctly omitted when not provided

🤖 Generated with Claude Code

Search VK users by name with filters: city, country, sex, age range.
Includes null/undefined params fix to prevent API serialization errors.

Tested: returns 897 983 results for "Иван Иванов" with photo_200, online fields.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@bulatko
bulatko merged commit a5616bc into bulatko:master Jul 25, 2026
bulatko added a commit that referenced this pull request Jul 25, 2026
- unskipped the omitted-argument test now that #4 is merged; it guards the
  regression where undefined reached VK as the literal string
- handler defaults now use ?? instead of ||, so an explicit 0 (count, offset,
  intervals_count) is no longer replaced by the default; test unskipped
- protocol and request tests extended to the 19-tool surface
- new test walks tools/list and calls each tool, catching any tool that is
  advertised but missing from the handler switch

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
bulatko added a commit that referenced this pull request Jul 25, 2026
The tool table listed 10 of 19 — the wall management and photo upload tools
from #2 were never added, nor the search/members/join/likes tools from #5#8.
Grouped by area and marked the ones that write to VK on the token owner's
behalf.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants