Skip to content

get: print short_text body inline with headers#5

Merged
markmnl merged 2 commits intomainfrom
copilot/fix-get-command-headers-data
May 6, 2026
Merged

get: print short_text body inline with headers#5
markmnl merged 2 commits intomainfrom
copilot/fix-get-command-headers-data

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 6, 2026

get only showed headers, requiring a separate get-data call to read even a short plain-text message body.

Changes

  • internal/api/client.go — added ShortText *string \json:"short_text"`toMessage`; the field was already returned by the API but silently dropped during JSON decode
  • cmd/get.go — after headers, prints a blank line followed by short_text when present; users only need get-data for non-text or oversized messages
  • README.md — updated get command description to reflect the new output

Example output

From: @alice@example.com
To:   ["@bob@example.com"]
Type: text/plain
Size: 12
Time: 1746523200.000000

Hello, world!

Copilot AI linked an issue May 6, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix get command to include headers and short text body get: print short_text body inline with headers May 6, 2026
Copilot finished work on behalf of markmnl May 6, 2026 10:16
Copilot AI requested a review from markmnl May 6, 2026 10:16
@markmnl markmnl marked this pull request as ready for review May 6, 2026 10:33
@markmnl markmnl merged commit dd578e0 into main May 6, 2026
@markmnl markmnl deleted the copilot/fix-get-command-headers-data branch May 6, 2026 10:34
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.

get should include headers and data

2 participants