GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||||||||||||
GeezerVison is showing github.com — feat: add Run now for schedule chat automations by ThomasK33 · Pull Request #30166 · coder/coder Conversation
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@codex review |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Codex Review: Didn't find any major issues. 🚀 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…le and delete Replaces the CODAGT-1209 TODO in ARCHITECTURE.md with prose: disable bumps the queue generation in an automation-only transaction and then deletes older queued rows through DeleteQueuedMessage as chatd; delete removes the automation row first and then its queued rows; the queue promotion guard discards any row the cleanup leaves behind. --- _Generated with [`xum`](https://github.com/coder/xum) • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_
|
@codex review |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Codex Review: Didn't find any major issues. Bravo. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
CI evidence for the exact head 25b8907: Why workflow_dispatch: GitHub creates no pull_request runs for this pushed stacked PR, and its mergeable state stays UNKNOWN. Generated with |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…agt-1211b-automations-run-now No conflicts. make gen produced no diff. --- _Generated with [`xum`](https://github.com/coder/xum) • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_
|
sync with main d9c13dc: no content conflicts; make gen produced no diff
Generated with |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Xum (an AI coding agent) prepared and posted this review on behalf of Mike. Each finding came from automated review passes and was re-verified against the code; please push back on anything that looks wrong.
Run now is small and matches CODAGT-1211: RunAutomation checks the owner before anything else, refuses webhook and disabled automations, and publishes through the shared admission path without an occurrence, so the schedule cursor and revision are never written. After re-verifying the reviewer findings against the code, none was worth asking you to fix.
Verdict: no blocking issues
Severity: P0 0, P1 0, P2 0, P3 0
typesGenerated.ts only): FE2 PASS, FE1 and FE3 to FE10 N/A, generated type naming and consistency PASS
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…agt-1211b-automations-run-now No conflicts. make gen produced no diff. --- _Generated with [`xum`](https://github.com/coder/xum) • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_
|
sync with main 35abc22: no content conflicts; make gen produced no diff
Generated with |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…agt-1211b-automations-run-now No conflicts. make gen produced no diff. --- _Generated with [`xum`](https://github.com/coder/xum) • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_
|
sync with main d11809f: merged #30160
Generated with |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Xum re-reviewed this PR on behalf of @ibetitsmike using GPT-6.1 Sol (high) Exec subagents.
The inherited audit lifetime/request-ID and scanner-plumbing fixes are intact; #30166 has no outstanding prior issue or unanswered thread. New findings: Standards 0, Spec 0, Frontend 0. Final fresh-head logs show Run now, experiment-gate, webhook and selected lifecycle/admission tests passing in coderd (1.483s), chatd (1.386s) and chatstate (0.692s), with the saved layer diff verified. Disconnect survival is source-verified rather than regression-tested; full/required CI is unavailable in the snapshot and Pixel is pending, so this is not an all-CI-green verdict.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…agt-1211b-automations-run-now No conflicts. make gen produced no diff. --- _Generated with [`xum`](https://github.com/coder/xum) • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_
|
sync with main a328bee: merged #30160
Generated with |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Xum validated this PR and approved it on behalf of @ibetitsmike.
Validated head c537251a3654. Revalidated the replacement head after withholding approval of the superseded pin. The introducing-layer payload and Run now contracts remain unchanged; scoped project/store integration is sound. Fresh Run now and caller-gate tests passed on this exact head, and current required CI passed. Generated-only frontend changes introduce no new visual behavior; no Pixel approval or full-stack merge-readiness claim is made.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…agt-1211b-automations-run-now No conflicts. make gen produced no diff. --- _Generated with [`xum`](https://github.com/coder/xum) • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_
|
merged #30160
Generated with |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…agt-1211b-automations-run-now No conflicts. make gen produced no diff. --- _Generated with [`xum`](https://github.com/coder/xum) • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_
|
merged #30160
Generated with |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…agt-1211b-automations-run-now No conflicts. make gen produced no diff. --- _Generated with [`xum`](https://github.com/coder/xum) • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_
Generated with |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…ations-run-now Brings main ff158c0 into this layer. No conflicts. --- _Generated with [`xum`](https://github.com/coder/xum) • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_
…ations-run-now Brings the new_chat automation title source fix from #30134. --- _Generated with [`xum`](https://github.com/coder/xum) • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.