Skip to content

fix: restore message + phone IDs in list output, clarify direction - #31

Merged
abhiramkast merged 1 commit into
mainfrom
fix/restore-list-ids
May 20, 2026
Merged

fix: restore message + phone IDs in list output, clarify direction#31
abhiramkast merged 1 commit into
mainfrom
fix/restore-list-ids

Conversation

@abhiramkast

Copy link
Copy Markdown
Contributor

Problem

The v2 rewrite dropped the ID column from linq messages list and linq phonenumbers — so an agent (or a human) can't grab an ID to act on a specific message or phone without falling back to --json.

messages list was also confusing:

  • The arrow + number (→ +1 (205) 222-2918) was ambiguous — does it mean "sent to" or "sent from"?
  • A sent / delivered / read status was shown for every row, but "sent" is meaningless for an inbound message.

messages list --json was leaking the raw SDK envelope (options, response, a duplicated payload) — noisy and coupled to SDK internals.

Solution

  • messages list: bring back the message ID (cyan, leftmost, copy-paste ready); switch to absolute timestamps; drop the status column (still in linq messages get); replace the ambiguous arrow with explicit you → +1 (...) / +1 (...) → you.
  • phonenumbers: bring back the phone number's UUID before the formatted phone.
  • messages list --json: emit just { messages, next_cursor } instead of the whole SDK wrapper.

Notes

Other commands still leak the SDK envelope in --json. Follow-up PR will clean those up uniformly.

@abhiramkast
abhiramkast requested a review from jflayhart as a code owner May 20, 2026 16:36
@abhiramkast
abhiramkast merged commit 9566e64 into main May 20, 2026
4 checks passed
@abhiramkast
abhiramkast deleted the fix/restore-list-ids branch May 20, 2026 19:00
@jflayhart

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 2.0.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants