Skip to content

[codex] fix(ci): initialize DingTalk request before debug logging - #571

Open
eetoc wants to merge 1 commit into
mainfrom
codex/fix-dingtalk-request-debug-order
Open

[codex] fix(ci): initialize DingTalk request before debug logging#571
eetoc wants to merge 1 commit into
mainfrom
codex/fix-dingtalk-request-debug-order

Conversation

@eetoc

@eetoc eetoc commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Move the DingTalk LLM request debug output after request_body is initialized.
  • Prevent the release notification workflow from exiting under set -u before the LLM request is sent.

The failure was introduced when ${#request_body} was logged before the variable assignment, producing request_body: unbound variable.

Testing

  • Extracted the embedded workflow Bash and validated it with bash -n.
  • Ran the complete embedded script locally with mocked GitHub release, LLM, and DingTalk responses.
  • Ran git diff --check.

Checklist

  • Documentation updated when behavior or configuration changed (not applicable; no documented behavior changed).
  • Tests added or updated for user-visible behavior (no repository test added; validated with a local mocked workflow smoke test).
  • No secrets, private endpoints, internal certificates, or local runtime state included.

@monkeyscan

monkeyscan Bot commented Aug 7, 2026

Copy link
Copy Markdown

评审任务执行失败。

@eetoc
eetoc marked this pull request as ready for review August 7, 2026 13:38
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.

1 participant