evgeny/openclaw
1
0
Fork 0
Code Issues Pull Requests 3 Actions 1 Packages Projects Releases Wiki Activity

fix/restore-subagent-announce-v2 #1

Open
evgeny wants to merge 3 commits from fix/restore-subagent-announce-v2 into main
pull from: fix/restore-subagent-announce-v2
merge into: evgeny:main
Conversation 0 Commits 3 Files Changed 7 +417 -1013
evgeny commented 2026-02-22 04:40:45 +00:00
Owner
Copy Link

Summary

Describe the problem and fix in 2–5 bullets:

  • Problem:
  • Why it matters:
  • What changed:
  • What did NOT change (scope boundary):

Change Type (select all)

  • Bug fix
  • Feature
  • Refactor
  • Docs
  • Security hardening
  • Chore/infra

Scope (select all touched areas)

  • Gateway / orchestration
  • Skills / tool execution
  • Auth / tokens
  • Memory / storage
  • Integrations
  • API / contracts
  • UI / DX
  • CI/CD / infra

Linked Issue/PR

  • Closes #
  • Related #

User-visible / Behavior Changes

List user-visible changes (including defaults/config).
If none, write None.

Security Impact (required)

  • New permissions/capabilities? (Yes/No)
  • Secrets/tokens handling changed? (Yes/No)
  • New/changed network calls? (Yes/No)
  • Command/tool execution surface changed? (Yes/No)
  • Data access scope changed? (Yes/No)
  • If any Yes, explain risk + mitigation:

Repro + Verification

Environment

  • OS:
  • Runtime/container:
  • Model/provider:
  • Integration/channel (if any):
  • Relevant config (redacted):

Steps

Expected

Actual

Evidence

Attach at least one:

  • Failing test/log before + passing after
  • Trace/log snippets
  • Screenshot/recording
  • Perf numbers (if relevant)

Human Verification (required)

What you personally verified (not just CI), and how:

  • Verified scenarios:
  • Edge cases checked:
  • What you did not verify:

Compatibility / Migration

  • Backward compatible? (Yes/No)
  • Config/env changes? (Yes/No)
  • Migration needed? (Yes/No)
  • If yes, exact upgrade steps:

Failure Recovery (if this breaks)

  • How to disable/revert this change quickly:
  • Files/config to restore:
  • Known bad symptoms reviewers should watch for:

Risks and Mitigations

List only real risks for this PR. Add/remove entries as needed. If none, write None.

  • Risk:
    • Mitigation:
## Summary Describe the problem and fix in 2–5 bullets: - Problem: - Why it matters: - What changed: - What did NOT change (scope boundary): ## Change Type (select all) - [ ] Bug fix - [ ] Feature - [ ] Refactor - [ ] Docs - [ ] Security hardening - [ ] Chore/infra ## Scope (select all touched areas) - [ ] Gateway / orchestration - [ ] Skills / tool execution - [ ] Auth / tokens - [ ] Memory / storage - [ ] Integrations - [ ] API / contracts - [ ] UI / DX - [ ] CI/CD / infra ## Linked Issue/PR - Closes # - Related # ## User-visible / Behavior Changes List user-visible changes (including defaults/config). If none, write `None`. ## Security Impact (required) - New permissions/capabilities? (`Yes/No`) - Secrets/tokens handling changed? (`Yes/No`) - New/changed network calls? (`Yes/No`) - Command/tool execution surface changed? (`Yes/No`) - Data access scope changed? (`Yes/No`) - If any `Yes`, explain risk + mitigation: ## Repro + Verification ### Environment - OS: - Runtime/container: - Model/provider: - Integration/channel (if any): - Relevant config (redacted): ### Steps 1. 2. 3. ### Expected - ### Actual - ## Evidence Attach at least one: - [ ] Failing test/log before + passing after - [ ] Trace/log snippets - [ ] Screenshot/recording - [ ] Perf numbers (if relevant) ## Human Verification (required) What you personally verified (not just CI), and how: - Verified scenarios: - Edge cases checked: - What you did **not** verify: ## Compatibility / Migration - Backward compatible? (`Yes/No`) - Config/env changes? (`Yes/No`) - Migration needed? (`Yes/No`) - If yes, exact upgrade steps: ## Failure Recovery (if this breaks) - How to disable/revert this change quickly: - Files/config to restore: - Known bad symptoms reviewers should watch for: ## Risks and Mitigations List only real risks for this PR. Add/remove entries as needed. If none, write `None`. - Risk: - Mitigation:
evgeny added 3 commits 2026-02-22 04:40:45 +00:00
fix(agents): remove send path from subagent announce (again) -- routes all completions through agent path so parent LLM processes results. Third time fixing this regression. c20fe0346d
Defer completion announce until subagent descendants finish dd377b0768
fix(agents): restore exact announce logic from fe57bea08 (PR #22223)
Some checks failed
CI / docs-scope (pull_request) Has been cancelled
Details
CI / changed-scope (pull_request) Has been cancelled
Details
CI / build-artifacts (pull_request) Has been cancelled
Details
CI / release-check (pull_request) Has been cancelled
Details
CI / checks (pnpm canvas:a2ui:bundle && bunx vitest run --config vitest.unit.config.ts, bun, test) (pull_request) Has been cancelled
Details
CI / checks (pnpm canvas:a2ui:bundle && pnpm test, node, test) (pull_request) Has been cancelled
Details
CI / checks (pnpm protocol:check, node, protocol) (pull_request) Has been cancelled
Details
CI / check (pull_request) Has been cancelled
Details
CI / dead-code report (pnpm deadcode:report:ci:knip, knip) (pull_request) Has been cancelled
Details
CI / dead-code report (pnpm deadcode:report:ci:ts-prune, ts-prune) (pull_request) Has been cancelled
Details
CI / dead-code report (pnpm deadcode:report:ci:ts-unused, ts-unused-exports) (pull_request) Has been cancelled
Details
CI / check-docs (pull_request) Has been cancelled
Details
Install Smoke / docs-scope (pull_request) Has been cancelled
Details
CI / secrets (pull_request) Has been cancelled
Details
CI / checks-windows (pnpm canvas:a2ui:bundle && pnpm test, node, test) (pull_request) Has been cancelled
Details
CI / checks-windows (pnpm lint, node, lint) (pull_request) Has been cancelled
Details
CI / checks-windows (pnpm protocol:check, node, protocol) (pull_request) Has been cancelled
Details
CI / macos (pull_request) Has been cancelled
Details
CI / ios (pull_request) Has been cancelled
Details
CI / android (./gradlew --no-daemon :app:assembleDebug, build) (pull_request) Has been cancelled
Details
CI / android (./gradlew --no-daemon :app:testDebugUnitTest, test) (pull_request) Has been cancelled
Details
Install Smoke / install-smoke (pull_request) Has been cancelled
Details
Workflow Sanity / no-tabs (pull_request) Has been cancelled
Details
Workflow Sanity / actionlint (pull_request) Has been cancelled
Details
Labeler / label (pull_request_target) Has been cancelled
Details
Labeler / backfill-pr-labels (pull_request_target) Has been cancelled
Details
Labeler / label-issues (pull_request_target) Has been cancelled
Details
cb9b0a5b46
Some checks failed
CI / docs-scope (pull_request) Has been cancelled
Details
CI / changed-scope (pull_request) Has been cancelled
Details
CI / build-artifacts (pull_request) Has been cancelled
Details
CI / release-check (pull_request) Has been cancelled
Details
CI / checks (pnpm canvas:a2ui:bundle && bunx vitest run --config vitest.unit.config.ts, bun, test) (pull_request) Has been cancelled
Details
CI / checks (pnpm canvas:a2ui:bundle && pnpm test, node, test) (pull_request) Has been cancelled
Details
CI / checks (pnpm protocol:check, node, protocol) (pull_request) Has been cancelled
Details
CI / check (pull_request) Has been cancelled
Details
CI / dead-code report (pnpm deadcode:report:ci:knip, knip) (pull_request) Has been cancelled
Details
CI / dead-code report (pnpm deadcode:report:ci:ts-prune, ts-prune) (pull_request) Has been cancelled
Details
CI / dead-code report (pnpm deadcode:report:ci:ts-unused, ts-unused-exports) (pull_request) Has been cancelled
Details
CI / check-docs (pull_request) Has been cancelled
Details
Install Smoke / docs-scope (pull_request) Has been cancelled
Details
CI / secrets (pull_request) Has been cancelled
Details
CI / checks-windows (pnpm canvas:a2ui:bundle && pnpm test, node, test) (pull_request) Has been cancelled
Details
CI / checks-windows (pnpm lint, node, lint) (pull_request) Has been cancelled
Details
CI / checks-windows (pnpm protocol:check, node, protocol) (pull_request) Has been cancelled
Details
CI / macos (pull_request) Has been cancelled
Details
CI / ios (pull_request) Has been cancelled
Details
CI / android (./gradlew --no-daemon :app:assembleDebug, build) (pull_request) Has been cancelled
Details
CI / android (./gradlew --no-daemon :app:testDebugUnitTest, test) (pull_request) Has been cancelled
Details
Install Smoke / install-smoke (pull_request) Has been cancelled
Details
Workflow Sanity / no-tabs (pull_request) Has been cancelled
Details
Workflow Sanity / actionlint (pull_request) Has been cancelled
Details
Labeler / label (pull_request_target) Has been cancelled
Details
Labeler / backfill-pr-labels (pull_request_target) Has been cancelled
Details
Labeler / label-issues (pull_request_target) Has been cancelled
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/restore-subagent-announce-v2:fix/restore-subagent-announce-v2
git checkout fix/restore-subagent-announce-v2
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
evgeny
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: evgeny/openclaw#1
Delete Branch "fix/restore-subagent-announce-v2"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?