Skip to content

tp leave list --json does not surface the top-level cancelledCount AGENTS.md says it does #62

Description

@jernejk

Found while implementing #52. ListCommand and the MCP GetLeaveEntries tool serialise only the items array, so the list envelope's cancelledCount (and totalItems) never reach the caller, although AGENTS.md states they surface in tp leave list --json.

Expected: either wrap the JSON output in an envelope ({"items": [...], "totalItems": N, "cancelledCount": N}) as a deliberate, announced shape change (0.4.0 candidate, since scripts parse the bare array today), or correct AGENTS.md. Recommend the envelope in 0.4.0 alongside the other contract changes, with MCP mirroring it through LeaveListService.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Priority: P3Nice to haveType: BugA problem with existing functionality

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions