Skip to content

Cover the documented thread calls in DocsSamples - #158

Open
martinmitrevski wants to merge 1 commit into
mainfrom
docs-samples-thread-coverage
Open

Cover the documented thread calls in DocsSamples#158
martinmitrevski wants to merge 1 commit into
mainfrom
docs-samples-thread-coverage

Conversation

@martinmitrevski

Copy link
Copy Markdown
Contributor

Follow-up to #157. This commit was pushed to the branch a moment after #157 was squash-merged, so it never made it onto main.

The Unreal threads docs (docs-content#1495) state that every snippet on the page is mirrored here, which is what makes them compiler-checked: a signature that drifts breaks this build instead of shipping a snippet that does not compile. That claim does not hold on main without this.

Adds the calls the page shows but the module did not yet exercise: GetThread with a participant limit, reading ReplyCount, ThreadParticipants, ParentMessage and LatestReplies off FChatThread, GetReplies on the channel, and IsThreadReply as the way to spot a reply.

DocsSamples is a DeveloperTool module in the sample project, not part of the plugin, so nothing about the shipped SDK changes here.

Verified on UE 5.7 and 5.8.1: strict builds with zero errors and zero warnings, and 59/59 automation tests on each.

🤖 Generated with Claude Code

The threads page now carries Unreal snippets, and DocsSamples is what keeps those honest:
it compiles against the real headers, so a signature that drifts breaks the build instead
of shipping a snippet that does not compile.

Adds the calls the page shows but the module did not yet exercise: GetThread with a
participant limit, reading ReplyCount, ThreadParticipants, ParentMessage and LatestReplies
off FChatThread, GetReplies on the channel, and IsThreadReply as the way to spot a reply.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.

1 participant