Skip to content

Conversation

@holmboe
Copy link
Contributor

@holmboe holmboe commented Jan 9, 2026

Summary

Add assignee information to phabfive maniphest show output.

  • Display current Assignee in regular task view (shows (none) if unassigned)
  • Display assignee change history when using --show-history flag
  • Resolve user PHIDs to usernames for readable output

Before/After

Regular task view

Before:

- Link: http://phorge.domain.tld/T47
  Task:
    Name: Set up production code repository
    Status: Open
    Priority: Normal
    Created: '2026-01-09T17:50:29'

After:

- Link: http://phorge.domain.tld/T47
  Task:
    Name: Set up production code repository
    Status: Open
    Priority: Normal
    Assignee: daniel.lindgren
    Created: '2026-01-09T17:50:29'

With --show-history

Before:

  History:
    Priority:
    - 2026-01-09T17:50:29 [↓] Triage → High
    Boards:
      Development:
      - 2026-01-09T18:24:09 [→] Backlog → In Progress

After:

  History:
    Assignee:
    - 2026-01-09T18:23:05 [•] (none) → admin
    - 2026-01-09T18:23:17 [•] admin → daniel.lindgren
    Priority:
    - 2026-01-09T17:50:29 [↓] Triage → High
    Boards:
      Development:
      - 2026-01-09T18:24:09 [→] Backlog → In Progress

Test plan

  • phabfive maniphest show T47 displays Assignee field
  • phabfive maniphest show T1 shows Assignee: (none) for unassigned tasks
  • phabfive maniphest show T47 --show-history displays Assignee transitions
  • uv run tox -e py310 passes

🤖 Generated with Claude Code

@holmboe holmboe requested a review from Timpan4 January 9, 2026 17:59
Copy link
Contributor

@Timpan4 Timpan4 left a comment

Choose a reason for hiding this comment

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

lgtm!

@holmboe holmboe force-pushed the feat-assignee-history branch from 3f50b4d to c6c63de Compare January 10, 2026 19:36
- Display current Assignee in regular task show output
- Display assignee change history when using --show-history flag
- Resolve user PHIDs to usernames for readable output

Changes:
- Add need_assignee parameter to _fetch_all_transactions()
- Handle reassign/core:owner transaction types
- Add _build_assignee_transitions() method
- Include Assignee field in Task section
- Include Assignee section in History output

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@holmboe holmboe force-pushed the feat-assignee-history branch from c6c63de to 71e060b Compare January 10, 2026 19:44
@holmboe holmboe merged commit 69b9292 into master Jan 10, 2026
15 checks passed
@holmboe holmboe deleted the feat-assignee-history branch January 10, 2026 19:45
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.

3 participants