Skip to content

Read Receipts#136

Merged
Nick (ingalls) merged 3 commits into
mainfrom
read-receipts
Jul 4, 2026
Merged

Read Receipts#136
Nick (ingalls) merged 3 commits into
mainfrom
read-receipts

Conversation

@ingalls

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented Jul 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.95918% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/builders/chat.ts 97.36% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@ingalls
Nick (ingalls) requested a review from Copilot July 4, 2026 04:37
@ingalls Nick (ingalls) self-assigned this Jul 4, 2026
@ingalls Nick (ingalls) added the enhancement New feature or request label Jul 4, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds first-class support for DirectChat delivery/read receipts (“Read Receipts”) across parsing, GeoJSON conversion, type schemas, and builders.

Changes:

  • Extend the CoT Detail schema to recognize __chatreceipt.
  • Populate properties.chat from __chatreceipt when converting CoT → GeoJSON.
  • Add CoT.is_chat_receipt() helper, DirectChatReceipt builder, and receipt-focused tests (ATAK + WinTAK styles).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
test/chat.test.ts Adds parsing and builder tests for chat receipt CoTs.
lib/types/types.ts Extends Detail schema to include optional __chatreceipt.
lib/parser/to_geojson.ts Maps __chatreceipt detail into properties.chat for GeoJSON output.
lib/cot.ts Introduces is_chat_receipt() helper for receipt detection.
lib/builders/chat.ts Adds DirectChatReceipt builder and receipt status/type mapping.
CHANGELOG.md Documents new chat receipt parsing, helper, builder, and tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lib/cot.ts
Comment thread test/chat.test.ts
Comment thread test/chat.test.ts Outdated
@ingalls
Nick (ingalls) merged commit f63ebe5 into main Jul 4, 2026
6 checks passed
@ingalls
Nick (ingalls) deleted the read-receipts branch July 4, 2026 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants