Repository navigation
feat(web): choose reasoning, speed, and permissions for scheduled tasks - #16592
robertnisipeanu wants to merge 2 commits into
Conversation
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
ApprovabilityVerdict: Not approved Macroscope's review found this PR not approvable — This production web change adds persisted model traits and permission selection to scheduled tasks, changing how future agent runs execute. It spans new controls, provider compatibility handling, and existing-task state persistence, so human review is warranted. You can add or adjust custom eligibility rules. Learn more. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughScheduled-task drafts now store model selections directly. The editor adds model and traits controls, filters runtime-mode choices by provider support, and saves the active selection with the resolved mode. Unthreaded tasks have a permissions selector; thread-bound tasks omit it. ChangesScheduled task editor
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~15 minutes Change: Feature Suggested reviewers: Merge Risk: ⚪ Minimal · up to The scheduled-task editor changes are mergeable after normal checks; no material issue remains established. Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to The change exposes existing execution settings without demonstrating a new authorization bypass or increased maximum authority. Existing task identity and thread permissions remain protected. Provider changes during editing and interrupted saves were assessed statically, not verified at runtime. Retained concerns Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
Resilience and Maintainability Implications
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
A task bound to an existing thread runs with that thread's permissions, so the editor does not offer a choice that would not apply. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Problem
The scheduled-task editor on web only lets you pick a model. Reasoning level, context window, fast mode and permission mode can't be chosen there, even though the chat composer and the mobile task editor already offer them and the server already stores and runs with them.
Change
The Model field in the web editor now has the same traits menu as the composer next to the model picker (reasoning, context window, fast mode, and Ultrathink for Claude models). A new Permissions field picks Supervised, Auto-accept edits, Auto or Full access.
modelSelection. When it'snull, the task follows the project's default model and that model's options. Editing a task keeps its saved options. Picking a different model resets the options to that model's defaults. Picking the model that's already selected changes nothing.Scope and approval
Focused configuration of an established capability. Scheduled tasks already store a full model selection (including provider options) plus a permission mode and launch runs with them; tasks an agent creates copy both from the calling thread, and the mobile editor already exposes them. This PR only adds the matching controls to the web editor, reusing the composer's traits picker and permission options, with no contract or server change.
Verification
tsc --noEmit(apps/web),vp lintandvp fmt --checkon the changed files: pass.scheduledTasksSettings.logic.test.ts: 22/22 pass.options: [effort: high, fastMode: true, contextWindow: 1m]andruntime_mode: approval-required.Screenshots
Implemented with Claude Opus 5.5 in Claude Code (T3 Code).
🤖 Generated with Claude Code