Skip to content

script: add send-prompt.sh to send prompt to clawapp via proxy#12

Merged
1186258278 merged 1 commit intoqingchencloud:mainfrom
youyli03:script/send-prompt-by-script
Mar 8, 2026
Merged

script: add send-prompt.sh to send prompt to clawapp via proxy#12
1186258278 merged 1 commit intoqingchencloud:mainfrom
youyli03:script/send-prompt-by-script

Conversation

@youyli03
Copy link
Contributor

@youyli03 youyli03 commented Mar 6, 2026

so that we can call send-prompt.sh in cron jobs to trigger openclaw to do some tasks at specific time.

so that we can call send-prompt.sh in cron jobs to trigger openclaw to do some tasks at specific time.
@1186258278
Copy link
Contributor

感谢 @youyli03 持续贡献!第 7 个 PR 了,cron 定时发送 prompt 的需求很实用 👍

代码整体质量不错,有几个小建议:

  1. eval 安全问题(第 31 行):eval ENV_${key}='${val}' 存在注入风险,建议改用 declare

    declare ENV_${key}=${val}
    
  2. python3 依赖:项目已经要求 Node.js,可以考虑用 node -e 替代 python3 -c 做 JSON 解析,减少一个系统依赖。例如:

    json_field() {
      node -e "const d=JSON.parse(process.argv[1]);let r=d;process.argv[2].split('.').forEach(k=>r=r[k]);console.log(r)" "$1" "$2"
    }
    

改完这两点就可以合并了!继续期待你的贡献 🎉

@1186258278 1186258278 merged commit f4a0d60 into qingchencloud:main Mar 8, 2026
@1186258278
Copy link
Contributor

已合并 ✅ 上面提到的 eval 安全问题和 python3 依赖我这边直接改了,感谢持续贡献!🎉

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants