Skip to content

fix: resolve 'me' to current user in all commands#78

Merged
steveclarke merged 4 commits into
masterfrom
fix/resolve-me-in-all-user-commands
Mar 27, 2026
Merged

fix: resolve 'me' to current user in all commands#78
steveclarke merged 4 commits into
masterfrom
fix/resolve-me-in-all-user-commands

Conversation

@steveclarke
Copy link
Copy Markdown
Owner

@steveclarke steveclarke commented Mar 27, 2026

Summary

  • Adds "me" → current user resolution in UserResolvable#resolve_user, fixing all 9 commands that accept a user argument: cards assign, cards unassign, cards assigned, cards create --owner, spaces add-member, spaces remove-member, activity --user, projects create --owner, projects update --owner
  • Case-insensitive matching (me, Me, ME all work)
  • Cached via @me_cache (consistent with existing @users_cache pattern)
  • Single change in the shared concern — no per-command fixes needed
  • Updated --help desc strings, README tip, and CLI skill docs to reflect me as valid everywhere

Fixes #77

Readiness

  • Simplify: clean (guard clause style, added caching)
  • Code review: clean (0 critical, 2 auto-fixed)
  • Finalize: tests pass (745/0), lint clean

Test plan

  • New unit tests for UserResolvable#resolve_user (7 examples covering nil, ID passthrough, name resolution, not-found error, and me/Me/ME)
  • Updated activity integration tests to stub /users/me endpoint
  • Full suite passes (745 examples, 0 failures)
  • Manual: suth cards assign CARD me assigns current user
  • Manual: suth cards unassign CARD me unassigns current user

Previously only `cards assigned me` worked — other commands like
`cards assign`, `cards unassign`, `spaces add-member`, `--owner`,
and `activity --user` passed 'me' through literally, causing 403s.

Fixes #77
@steveclarke
Copy link
Copy Markdown
Owner Author

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

🤖 Generated with Claude Code

@steveclarke steveclarke merged commit 0aef072 into master Mar 27, 2026
4 checks passed
@steveclarke steveclarke deleted the fix/resolve-me-in-all-user-commands branch March 27, 2026 12:35
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.

cards assign: resolve 'me' to current user

1 participant