added build notify by lifeizhou-ap · Pull Request #6891 · block/goose

lifeizhou-ap · 2026-02-02T07:02:45Z

Summary

Why
We have multiple important workflows (CI on main branch, release etc.), but some of them failed sometimes and we did not notice (I guess we don't check our email that much). This has led to delayed feedback and may make it harder to fix the issue with larger changeset. In some cases, issues are only discovered after customers report them. For example, a recent release workflow failure caused a broken download link.

What
Add notifications to discord goose-status channel for failures in key workflows, including merge queue (we added this recently), main CI runs and release.

TODO
will also add the canary build cli and desktop later. Currently they failed on main. will fix this issue first and will then add these workflows for failure notification to avoid the late feedback when generating release artifacts while releasing.

Type of Change

katzdave · 2026-02-06T16:11:25Z

Not sure what's easier on slack vs discord, but either seems fine. I think the audience should be just us, and maybe start with only the release pipeline as that seems like the easiest thing to miss.

* main: (101 commits) fix: lazy provider creation for goose acp (#7026) (#7066) Smoke tests: split compaction test and use debug build (#6984) fix(deps): trim bat to resolve RUSTSEC-2024-0320 (#7061) feat: expose AGENT_SESSION_ID env var to extension child processes (#7072) fix: add XML tool call parsing fallback for Qwen3-coder via Ollama (#6882) Remove clippy too_many_lines lint and decompose long functions (#7064) refactor: move disable_session_naming into AgentConfig (#7062) Add global config switch to disable automatic session naming (#7052) docs: add blog post - 8 Things You Didn't Know About Code Mode (#7059) fix: ensure animated elements are visible when prefers-reduced-motion is enabled (#7047) Show recommended model on failture (#7040) feat(ui): add session content search via API (#7050) docs: fix img url (#7053) Desktop UI for deleting custom providers (#7042) Add blog post: How I Used RPI to Build an OpenClaw Alternative (#7051) Remove build-dependencies section from Cargo.toml (#6946) add /rp-why skill blog post (#6997) fix: fix snake_case function names in code_execution instructions (#7035) Document max_turns settings for recipes and subagents (#7044) feat: update Groq declarative data with Preview Models (#7023) ...

Copilot

Pull request overview

Adds a GitHub Actions workflow that posts a Discord notification to the goose-status channel when key workflows fail, improving visibility into CI/release regressions.

Changes:

Introduces a new workflow_run-triggered workflow to notify on failures for CI and Release.
Filters CI notifications to main-branch runs and merge queue (merge_group) runs.

Copilot · 2026-02-09T03:05:53Z

.github/workflows/build-notify.yml

+          curl -H "Content-Type: application/json" \
+            -d '{
+              "embeds": [{
+                "title": "❌ Build Failed",
+                "description": "**Workflow:** ${{ github.event.workflow_run.name }}\n**Branch:** ${{ github.event.workflow_run.head_branch }}\n**Commit:** ${{ github.event.workflow_run.head_sha }}",
+                "url": "${{ github.event.workflow_run.html_url }}",
+                "color": 15158332
+              }]
+            }' \
+            "$DISCORD_WEBHOOK_URL"


curl will exit 0 on HTTP 4xx/5xx responses, so this step can report success even when Discord rejects the payload; add --fail/--fail-with-body (and optionally retries) so notification delivery failures are visible.

michaelneale

this is a great idea!

* main: added build notify (#6891) test(mcp): add image tool test and consolidate MCP test fixtures (#7019) fix: remove Option from model listing return types, propagate errors (#7074)

lifeizhou-ap · 2026-02-09T07:18:45Z

Not sure what's easier on slack vs discord, but either seems fine.

Based on the team preference, I used discord.

I think the audience should be just us, and maybe start with only the release pipeline as that seems like the easiest thing to miss.

Yes, Tania helped us create a channel goose-status and we keep the number of members very small.

The ci on main and canary build artifacts is also important so that we can find the problem before release. Hopefully (Ideally) we won't have too many notifications

* origin/main: (54 commits) chore: strip posthog for sessions/models/daily only (#7079) tidy: clean up old benchmark and add gym (#7081) fix: use command.process_group(0) for CLI providers, not just MCP (#7083) added build notify (#6891) test(mcp): add image tool test and consolidate MCP test fixtures (#7019) fix: remove Option from model listing return types, propagate errors (#7074) fix: lazy provider creation for goose acp (#7026) (#7066) Smoke tests: split compaction test and use debug build (#6984) fix(deps): trim bat to resolve RUSTSEC-2024-0320 (#7061) feat: expose AGENT_SESSION_ID env var to extension child processes (#7072) fix: add XML tool call parsing fallback for Qwen3-coder via Ollama (#6882) Remove clippy too_many_lines lint and decompose long functions (#7064) refactor: move disable_session_naming into AgentConfig (#7062) Add global config switch to disable automatic session naming (#7052) docs: add blog post - 8 Things You Didn't Know About Code Mode (#7059) fix: ensure animated elements are visible when prefers-reduced-motion is enabled (#7047) Show recommended model on failture (#7040) feat(ui): add session content search via API (#7050) docs: fix img url (#7053) Desktop UI for deleting custom providers (#7042) ...

* origin/main: Docs: require auth optional for custom providers (#7098) fix: improve text-muted contrast for better readability (#7095) Always sync bundled extensions (#7057) feat: Add tom (Top Of Mind) platform extension (#7073) chore(docs): update GOOSE_SESSION_ID -> AGENT_SESSION_ID (#6669) fix(ci): switch from cargo-audit to cargo-deny for advisory scanning (#7032) chore(deps): bump @isaacs/brace-expansion from 5.0.0 to 5.0.1 in /evals/open-model-gym/suite (#7085) chore(deps): bump @modelcontextprotocol/sdk from 1.25.3 to 1.26.0 in /evals/open-model-gym/mcp-harness (#7086) fix: switch to windows msvc (#7080) fix: allow unlisted models for CLI providers (#7090) Use goose port (#7089) chore: strip posthog for sessions/models/daily only (#7079) tidy: clean up old benchmark and add gym (#7081) fix: use command.process_group(0) for CLI providers, not just MCP (#7083) added build notify (#6891)

* main: (125 commits) chore: add a new scenario (#7107) fix: Goose Desktop missing Calendar and Reminders entitlements (#7100) Fix 'Edit In Place' and 'Fork Session' features (#6970) Fix: Only send command content to command injection classifier (excluding part of tool call dict) (#7082) Docs: require auth optional for custom providers (#7098) fix: improve text-muted contrast for better readability (#7095) Always sync bundled extensions (#7057) feat: Add tom (Top Of Mind) platform extension (#7073) chore(docs): update GOOSE_SESSION_ID -> AGENT_SESSION_ID (#6669) fix(ci): switch from cargo-audit to cargo-deny for advisory scanning (#7032) chore(deps): bump @isaacs/brace-expansion from 5.0.0 to 5.0.1 in /evals/open-model-gym/suite (#7085) chore(deps): bump @modelcontextprotocol/sdk from 1.25.3 to 1.26.0 in /evals/open-model-gym/mcp-harness (#7086) fix: switch to windows msvc (#7080) fix: allow unlisted models for CLI providers (#7090) Use goose port (#7089) chore: strip posthog for sessions/models/daily only (#7079) tidy: clean up old benchmark and add gym (#7081) fix: use command.process_group(0) for CLI providers, not just MCP (#7083) added build notify (#6891) test(mcp): add image tool test and consolidate MCP test fixtures (#7019) ...

* main: (85 commits) Fix 'Edit In Place' and 'Fork Session' features (#6970) Fix: Only send command content to command injection classifier (excluding part of tool call dict) (#7082) Docs: require auth optional for custom providers (#7098) fix: improve text-muted contrast for better readability (#7095) Always sync bundled extensions (#7057) feat: Add tom (Top Of Mind) platform extension (#7073) chore(docs): update GOOSE_SESSION_ID -> AGENT_SESSION_ID (#6669) fix(ci): switch from cargo-audit to cargo-deny for advisory scanning (#7032) chore(deps): bump @isaacs/brace-expansion from 5.0.0 to 5.0.1 in /evals/open-model-gym/suite (#7085) chore(deps): bump @modelcontextprotocol/sdk from 1.25.3 to 1.26.0 in /evals/open-model-gym/mcp-harness (#7086) fix: switch to windows msvc (#7080) fix: allow unlisted models for CLI providers (#7090) Use goose port (#7089) chore: strip posthog for sessions/models/daily only (#7079) tidy: clean up old benchmark and add gym (#7081) fix: use command.process_group(0) for CLI providers, not just MCP (#7083) added build notify (#6891) test(mcp): add image tool test and consolidate MCP test fixtures (#7019) fix: remove Option from model listing return types, propagate errors (#7074) fix: lazy provider creation for goose acp (#7026) (#7066) ...

added build notify

c4e5991

katzdave assigned katzdave and unassigned katzdave Feb 6, 2026

lifeizhou-ap added 2 commits February 9, 2026 10:22

notify discord goose-status channel

4e2274a

lifeizhou-ap marked this pull request as ready for review February 9, 2026 03:04

Copilot AI review requested due to automatic review settings February 9, 2026 03:04

Copilot started reviewing on behalf of lifeizhou-ap February 9, 2026 03:04 View session

Copilot AI reviewed Feb 9, 2026

View reviewed changes

fail the workflow if curl returns failure

23eac51

michaelneale approved these changes Feb 9, 2026

View reviewed changes

lifeizhou-ap added this pull request to the merge queue Feb 9, 2026

Merged via the queue into main with commit 5f183a6 Feb 9, 2026
19 checks passed

lifeizhou-ap deleted the lifei/build-monitor branch February 9, 2026 03:31

github-actions bot mentioned this pull request Feb 10, 2026

chore(release): release version 1.24.0 (minor) #7102

Open

Tyler-Hardin pushed a commit to Tyler-Hardin/goose that referenced this pull request Feb 11, 2026

added build notify (block#6891)

b191a8d

Tyler-Hardin pushed a commit to Tyler-Hardin/goose that referenced this pull request Feb 11, 2026

added build notify (block#6891)

9396a3a

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

added build notify#6891

added build notify#6891
lifeizhou-ap merged 4 commits intomainfrom
lifei/build-monitor

lifeizhou-ap commented Feb 2, 2026 •

edited

Loading

Uh oh!

katzdave commented Feb 6, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 9, 2026

Uh oh!

michaelneale left a comment

Uh oh!

Uh oh!

lifeizhou-ap commented Feb 9, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

lifeizhou-ap commented Feb 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Type of Change

Uh oh!

katzdave commented Feb 6, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 9, 2026

Choose a reason for hiding this comment

Uh oh!

michaelneale left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

lifeizhou-ap commented Feb 9, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

lifeizhou-ap commented Feb 2, 2026 •

edited

Loading