Skip to content

feat: add FaceExtra and QFaceExtra messages for enhanced functionality#8

Merged
Redmomn merged 2 commits intomasterfrom
qface
Mar 10, 2026
Merged

feat: add FaceExtra and QFaceExtra messages for enhanced functionality#8
Redmomn merged 2 commits intomasterfrom
qface

Conversation

@Redmomn
Copy link
Collaborator

@Redmomn Redmomn commented Mar 10, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 10, 2026 14:58
@github-actions
Copy link

github-actions bot commented Mar 10, 2026

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed⏩ skippedMar 10, 2026, 3:00 PM

@github-actions
Copy link

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed❌ failed (1)✅ passed⏩ skippedMar 10, 2026, 2:59 PM

@Redmomn Redmomn merged commit 22ea851 into master Mar 10, 2026
2 checks passed
@Redmomn Redmomn deleted the qface branch March 10, 2026 15:00
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds two new protobuf message definitions (FaceExtra and QFaceExtra) to the v2 message proto file. These messages are the v2 equivalents of the existing v1 definitions in proto/message/v1/element.proto, adapted to use proto2 syntax with explicit optional keywords on all fields. They provide structured types for face/sticker metadata used by downstream consumers (e.g., LagrangeGo) to decode CommonElem.pb_elem bytes.

Changes:

  • Added FaceExtra message with a single face_id field for basic face element metadata.
  • Added QFaceExtra message with fields for sticker packs, sticker IDs, source/type info, and random type — mirroring the v1 definition exactly.

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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants