Skip to content

ChatGPTの定期確認で起動失敗を成功扱いしない - #336

Open
mashfromband wants to merge 2 commits into
mainfrom
scheduler/todo-3232-script-repair
Open

mashfromband wants to merge 2 commits into
mainfrom
scheduler/todo-3232-script-repair

Conversation

@mashfromband

@mashfromband mashfromband commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

毎日のChatGPT確認で、呼先の実行ファイルが見つからなくてもWindows Script Hostが0を返すため、実処理が動いていない状態を成功と誤認していた。既存VBSで起動例外を1へ変換し、checkの非0とresume-readyの実終了コードを保持する。

  • 仕様: docs/chatgpt-hidden-check-3232.md
  • Red: 実WSHでmissing check/resumeの2件が誤った0を返して失敗。既存の失敗コード・成功引数3件は成功。
  • Green: 実WSHのGo fixture全5件、go vet成功。
  • 既存のWindows workflowへ同じwrapper fixtureを追加し、-count=1でVBSだけの変更でも毎回実行する。

元ToDo3232は既存タスクのActionを存在する正規sourceへ変更したが、配布先の実行ファイルの資格・正式配布と実checkは未完。このPRの試験を実ChromeのPlus確認や1974の完了とは扱わない。保護先・guard実装・1974をこのPRで変更しない。原5ファイルのClaude Sonnet5.5/maxレビューを受領し、唯一のMustだった-count=1不足を修正。追加2ファイルだけのClaude Sonnet5.5/max確認でMust0を受領済み。最終CIを受領してからmergeする。

Closes #335

Refs #335; ToDo3232. Verify missing children and real exit propagation through Windows Script Host.
@cursor

cursor Bot commented Oct 21, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: f61b7ef5-1c79-4cc1-9222-0369013e1a2e)

Address PR336 review: disable Go test result caching for module-external wrapper input; document WSH hosts.
@cursor

cursor Bot commented Oct 21, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: d86d3e22-c800-45ba-aa3e-e4159f7d14e0)

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.

ChatGPTの定期確認で起動失敗を成功扱いしない

1 participant