fix: always send live link explicitly, never rely on runtime delivery#8
Open
wiziswiz wants to merge 1 commit intorohunvora:mainfrom
Open
fix: always send live link explicitly, never rely on runtime delivery#8wiziswiz wants to merge 1 commit intorohunvora:mainfrom
wiziswiz wants to merge 1 commit intorohunvora:mainfrom
Conversation
Two issues fixed: 1. Remove 'unless your runtime already delivers it' caveat from Chat UX and execution sequence. Agents should always send the Watch live link explicitly — runtime delivery is not guaranteed (e.g. OpenClaw worker sessions may fail to route output to the user's chat). 2. Always include the source board URL in the final trade summary, even if it was shared earlier. This prevents link loss when mid-pipeline status messages fail to deliver.
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
1 participant
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.
Problem
When running
/tradethrough OpenClaw's worker session dispatch, two things break:Live link never reaches the user — the SKILL.md says to send the Watch live link "unless your runtime already delivers it." OpenClaw's worker sessions don't reliably deliver mid-pipeline status messages, so the agent skips the explicit send and the link is lost.
Final trade summary has no source link — the source board URL is only sent as a mid-pipeline status update, not included in the final trade card. If that status message fails to deliver, the user gets trades with no link to the board.
Fix
paste.trade/s/board URL, even if shared earlier. This makes the trade summary self-contained.Impact
3 lines changed in SKILL.md. No script changes. Makes the skill resilient to runtime delivery failures across any agent platform, not just OpenClaw.