Repository navigation
fix(ci): discord missing attachments field + Dockerfile crates/ copy - #1100
Merged
Merged
Conversation
The discord channel's poll_channel_mentions emit_message call was missing the required `attachments: vec![]` field, causing WASM compilation failure. Both Dockerfiles were also missing `COPY crates/ crates/` needed for the extracted ironclaw_safety crate. [skip-regression-check] Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
github-actions
Bot
added
scope: sandbox
Contributor
There was a problem hiding this comment.
Pull request overview
Fixes CI/build breakages by aligning the Discord channel’s emitted message struct initialization with the current EmittedMessage shape and ensuring Docker builds include the workspace crates/ directory needed for path dependencies.
Changes:
- Add missing
attachments: vec![]when emitting a Discord “mention” message. - Update
channels-src/discord/Cargo.lockto reflect thediscord-channelcrate version bump to0.2.0. - Include
crates/inDockerfileandDockerfile.testbuild contexts to satisfy workspace path deps during Docker builds.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| channels-src/discord/src/lib.rs | Adds missing attachments field to EmittedMessage initialization in the mention polling path. |
| channels-src/discord/Cargo.lock | Updates lockfile entry to match discord-channel version 0.2.0. |
| Dockerfile.test | Copies crates/ into the image to support workspace path dependency resolution during test builds. |
| Dockerfile | Copies crates/ into the image to support workspace path dependency resolution during production builds. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
zmanian
approved these changes
Mar 13, 2026
This was referenced Mar 13, 2026
Code reviewNo issues found. |
Merged
ilblackdragon
pushed a commit
that referenced
this pull request
Mar 14, 2026
…#1100) The discord channel's poll_channel_mentions emit_message call was missing the required `attachments: vec![]` field, causing WASM compilation failure. Both Dockerfiles were also missing `COPY crates/ crates/` needed for the extracted ironclaw_safety crate. [skip-regression-check] Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
bkutasi
pushed a commit
to bkutasi/ironclaw
that referenced
this pull request
Mar 28, 2026
…nearai#1100) The discord channel's poll_channel_mentions emit_message call was missing the required `attachments: vec![]` field, causing WASM compilation failure. Both Dockerfiles were also missing `COPY crates/ crates/` needed for the extracted ironclaw_safety crate. [skip-regression-check] Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
drchirag1991
pushed a commit
to drchirag1991/ironclaw
that referenced
this pull request
Apr 8, 2026
…nearai#1100) The discord channel's poll_channel_mentions emit_message call was missing the required `attachments: vec![]` field, causing WASM compilation failure. Both Dockerfiles were also missing `COPY crates/ crates/` needed for the extracted ironclaw_safety crate. [skip-regression-check] Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
attachments: vec![]field toEmittedMessagein discord channel (channels-src/discord/src/lib.rs) — the other two call sites already had itCOPY crates/ crates/toDockerfileandDockerfile.testso the extractedironclaw_safetycrate is available during Docker buildsFixes staging CI run #23035039465 (5 failing jobs).
Test plan
cargo clippy --all --benches --tests --examples --all-features— zero warningscargo test— all tests pass🤖 Generated with Claude Code