Skip to content

fix(person): Support name lookup and id: prefix syntax#13

Merged
gnapse merged 5 commits into
mainfrom
ernesto/fix-bob-person-cmd
Feb 11, 2026
Merged

fix(person): Support name lookup and id: prefix syntax#13
gnapse merged 5 commits into
mainfrom
ernesto/fix-bob-person-cmd

Conversation

@gnapse

@gnapse gnapse commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix missing Content-Type header for POST requests without body
  • Add name-based person lookup via search API (exact then partial matching)
  • Require explicit id:xxx prefix for ID lookups to eliminate ambiguity
  • Support email addresses for direct lookup
  • Comprehensive help text and error hints for AI agent usability

Test plan

  • bob person Ernesto → name search
  • bob person id:12345 → direct ID lookup
  • bob person user@co.com → direct email lookup
  • All 27 existing tests pass

🤖 Generated with Claude Code

- Fix missing Content-Type header when calling apiPost without body
- Add name-based person lookup via search API (exact match preferred, partial if unambiguous)
- Require explicit id:xxx prefix for direct ID lookups to avoid ambiguity
- Support email addresses for direct lookup (user@domain.com)
- Add comprehensive help text documenting ref formats for AI agents
- Include helpful hints in error messages (e.g., when name not found)

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@gnapse gnapse requested a review from henningmu February 9, 2026 11:49
@gnapse gnapse self-assigned this Feb 9, 2026

@henningmu henningmu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs an update for the skill as well. After that good to go 🙌

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gnapse gnapse requested a review from henningmu February 10, 2026 11:36
gnapse and others added 3 commits February 10, 2026 08:41
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gnapse gnapse merged commit 3c81bda into main Feb 11, 2026
2 checks passed
@gnapse gnapse deleted the ernesto/fix-bob-person-cmd branch February 11, 2026 12:17
rfgamaral pushed a commit that referenced this pull request Mar 25, 2026
🤖 I have created a release *beep* *boop*
---


## [1.3.0](v1.2.0...v1.3.0)
(2026-03-25)


### Features

* add new agent skills and skill auto-update on upgrade
([#17](#17))
([b73b761](b73b761))


### Bug Fixes

* **person:** Support name lookup and id: prefix syntax
([#13](#13))
([3c81bda](3c81bda))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.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