Fix: exclude platform_schedule_tool in CLI #6442

lifeizhou-ap · 2026-01-12T03:46:58Z

Summary

Fixed "Scheduler not available" error in issue #6405

Changes:

Exclude schedule tool in CLI mode - Only add platform__manage_schedule tool when scheduler is available. In CLI mode, there's no persistent scheduler service running, so the tool would fail if the LLM tried to use it.
Unified schedule file - CLI and Desktop now both use schedule.json (was using different files). Using schedule.json because Desktop users already have data in this file for scheduled job. For schedules.json created by CLI, the data in this file are not used to execute the recipe with schedule, it only used in run now which triggers the job manually. Losing this data has very minimal user impact.
Refactoring: create_scheduler() helper to reduce duplication
Improved tool description - Added "Goose" to description and all actions to prevent confusion with external schedulers

Type of Change

AI Assistance

This PR was created or reviewed with AI assistance

Testing

Unit test and Manually Testing

Copilot

Pull request overview

This PR fixes the "Scheduler not available" error in CLI mode by conditionally including the schedule management tool only when a scheduler service is present. The PR also standardizes the scheduler storage filename from schedules.json to schedule.json throughout the codebase.

Changes:

Added scheduler availability check before including the schedule management tool in the available tools list
Updated error message in schedule tool handler to reflect that it should only be called when scheduler is available
Standardized scheduler storage filename to schedule.json (singular) across all files
Refactored CLI schedule commands to use a shared helper function for creating scheduler instances

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
crates/goose/src/agents/agent.rs	Added scheduler service availability check in `list_tools` to conditionally include schedule management tool
crates/goose/src/agents/schedule_tool.rs	Updated error message to reflect internal assertion rather than user-facing error
crates/goose/src/agents/platform_tools.rs	Updated tool description to clarify it manages "Goose's internal scheduled recipe execution"
crates/goose/src/scheduler.rs	Standardized storage filename to `schedule.json` and improved error logging with path display
crates/goose/src/execution/manager.rs	Refactored to use `get_default_scheduler_storage_path()` helper function
crates/goose/src/session/diagnostics.rs	Removed duplicate `schedules.json` reference, keeping only `schedule.json`
crates/goose-cli/src/commands/schedule.rs	Added `create_scheduler()` helper function to reduce code duplication
crates/goose/tests/agent.rs	Updated tests to verify tool is not available without scheduler and added scheduler setup to tests requiring it

Copilot

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated no new comments.

DOsinga · 2026-01-12T16:19:37Z

crates/goose/src/agents/platform_tools.rs

+            - "delete": Remove a Goose scheduled job
+            - "kill": Terminate a currently running Goose scheduled job
+            - "inspect": Get details about a running Goose scheduled job
+            - "sessions": List execution history for a Goose scheduled job


goose should not be capitalized. why is this change happening?

I am sorry that I forgot to mention this change in the PR summary. In the original issue, user has some prompts related to k8s that could be related to schedule and agent triggers the platform_schedule_tool. I guess the agent got confused with k8s scheduler with goose scheduler. That's why I added goose in the tool description, so that the agent won't use this tool when having the context with non goose schedule.

I will change to the goose instead of Goose

DOsinga · 2026-01-12T16:21:02Z

crates/goose/src/agents/reply_parts.rs

        let (tools, _toolshim_tools, _system_prompt) =
            agent.prepare_tools_and_prompt(&working_dir).await?;

-        // Ensure both platform and frontend tools are present


or just delete this test

I checked the code, it seems it still worthwhile to use this test to test the ordering of the tools. I changed the test name to reflect the purpose of this test.

DOsinga · 2026-01-12T16:22:34Z

scripts/test_compaction.sh

+LONG_RESPONSE_PROMPT="Count from 1 to 200, one number per line."
+
+echo "Step 1: Creating session with first message (generating tokens for threshold)..."
+(cd "$TESTDIR" && "$GOOSE_BIN" run --text "$LONG_RESPONSE_PROMPT" 2>&1) | tee "$OUTPUT"


what happened here? from a different branch maybe?

Because we removed the platform_schedule_tool from CLI, the context usage is getting smaller and the original test prompt cannot reach to the context size to trigger the compact. So I changed the prompt to generate a bit longer response so that it reached to the threshold of 0.5% to trigger the compact

Copilot

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Copilot · 2026-01-12T23:48:29Z

crates/goose/src/agents/schedule_tool.rs

+                ErrorData::new(
                    ErrorCode::INTERNAL_ERROR,
-                    "Scheduler not available. This tool only works in server mode.".to_string(),
+                    "Scheduler service should be available when schedule tool is called"


This error message will never be seen by users since the tool is now only added when the scheduler is available. Consider removing the defensive check or updating the error to indicate this is an internal error (bug) if it ever triggers.

…ased * 'main' of github.com:block/goose: (23 commits) Use Intl.NumberFormat for token formatting in SessionsInsights (#6466) feat(ui): format large and small token counts for readability (#6449) fix: apply subrecipes when using slash commands (#6460) Fix: exclude platform_schedule_tool in CLI (#6442) Fix: Small update in how ML-based prompt injection determines final result (#6439) docs: remove SSE transport and rename to Streamable HTTP (#6319) fix: correct Cloudinary extension command and env variable (#6453) fix: add gap between buttons in MacDesktopInstallButtons.js (#6452) refactor: include hidden dotfiles folders in file picker search (#6315) upgraded safe npm packages (#6450) chore(deps): bump react-router and react-router-dom in /ui/desktop (#6408) chore(deps): bump lru from 0.12.5 to 0.16.3 (#6379) chore(deps-dev): bump @modelcontextprotocol/sdk from 1.24.0 to 1.25.2 in /ui/desktop (#6375) fix: inconsistent API url requirement between desktop and CLI versions (#6419) feat(vertexai): Add streaming support (#6409) fix deeplink recipe launch cold start (#6210) Spell check setting (#6446) File bug directly (#6413) fix(cli): incorrect bin name in shell completions (#6444) Use crunchy from crates instead of git fork (#6415) ...

* main: (41 commits) Allow customizing the new line keybinding in the CLI (#5956) Ask for permission in the CLI (#6475) docs: add Ralph Loop tutorial for multi-model iterative development (#6455) Remove gitignore fallback from gooseignore docs (#6480) fix: clean up result recording for code mode (#6343) fix(code_execution): handle model quirks with tool calls (#6352) feat(ui): support prefersBorder option for MCP Apps (#6465) fixed line breaks (#6459) Use Intl.NumberFormat for token formatting in SessionsInsights (#6466) feat(ui): format large and small token counts for readability (#6449) fix: apply subrecipes when using slash commands (#6460) Fix: exclude platform_schedule_tool in CLI (#6442) Fix: Small update in how ML-based prompt injection determines final result (#6439) docs: remove SSE transport and rename to Streamable HTTP (#6319) fix: correct Cloudinary extension command and env variable (#6453) fix: add gap between buttons in MacDesktopInstallButtons.js (#6452) refactor: include hidden dotfiles folders in file picker search (#6315) upgraded safe npm packages (#6450) chore(deps): bump react-router and react-router-dom in /ui/desktop (#6408) chore(deps): bump lru from 0.12.5 to 0.16.3 (#6379) ...

lifeizhou-ap added 6 commits January 12, 2026 13:27

not adding chedule_management_tool without scheduler

b1888b2

updated the error handling

ecf7c54

use schedule.json as source of truth

1f716ee

refactored to remove duplicate code for create scheduler

7ffb793

removed all schedules.json reference

a2b0ed5

make the schedule tool description more specific

c2740ea

Copilot AI review requested due to automatic review settings January 12, 2026 03:46

Copilot started reviewing on behalf of lifeizhou-ap January 12, 2026 03:47 View session

Copilot AI reviewed Jan 12, 2026

View reviewed changes

lifeizhou-ap added 2 commits January 12, 2026 15:02

fixed failed test

d8d680b

produce more response to trigger auto compact in smoke test

7de5184

Copilot AI review requested due to automatic review settings January 12, 2026 04:41

Copilot started reviewing on behalf of lifeizhou-ap January 12, 2026 04:41 View session

Copilot AI reviewed Jan 12, 2026

View reviewed changes

renamed variable and generate not too long response

0d288ce

This was referenced Jan 12, 2026

fix: CLI headless recipe runs fail with 'Scheduler not available' error on Linux AARCH64 (v1.18.0+) #6416

Open

CLI headless recipe runs fail with 'Scheduler not available' error on Linux AARCH64 (v1.18.0+) #6405

Closed

DOsinga approved these changes Jan 12, 2026

View reviewed changes

address pr comments

4105408

Copilot AI review requested due to automatic review settings January 12, 2026 23:43

Copilot started reviewing on behalf of lifeizhou-ap January 12, 2026 23:44 View session

Copilot AI reviewed Jan 12, 2026

View reviewed changes

Fixed the test

ab183b4

lifeizhou-ap merged commit 637049c into main Jan 13, 2026
28 of 30 checks passed

lifeizhou-ap deleted the lifei/opt-out-schedule-tool-in-cli branch January 13, 2026 00:36

fbalicchia pushed a commit to fbalicchia/goose that referenced this pull request Jan 13, 2026

Fix: exclude platform_schedule_tool in CLI (block#6442)

0c6ca25

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Fix: exclude platform_schedule_tool in CLI #6442

Fix: exclude platform_schedule_tool in CLI #6442

Uh oh!

lifeizhou-ap commented Jan 12, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

DOsinga Jan 12, 2026

Uh oh!

lifeizhou-ap Jan 12, 2026

Uh oh!

DOsinga Jan 12, 2026

Uh oh!

lifeizhou-ap Jan 12, 2026

Uh oh!

DOsinga Jan 12, 2026

Uh oh!

lifeizhou-ap Jan 12, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Jan 12, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Fix: exclude platform_schedule_tool in CLI #6442

Fix: exclude platform_schedule_tool in CLI #6442

Uh oh!

Conversation

lifeizhou-ap commented Jan 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Type of Change

AI Assistance

Testing

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

DOsinga Jan 12, 2026

Choose a reason for hiding this comment

Uh oh!

lifeizhou-ap Jan 12, 2026

Choose a reason for hiding this comment

Uh oh!

DOsinga Jan 12, 2026

Choose a reason for hiding this comment

Uh oh!

lifeizhou-ap Jan 12, 2026

Choose a reason for hiding this comment

Uh oh!

DOsinga Jan 12, 2026

Choose a reason for hiding this comment

Uh oh!

lifeizhou-ap Jan 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Jan 12, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

lifeizhou-ap commented Jan 12, 2026 •

edited

Loading

lifeizhou-ap Jan 12, 2026 •

edited

Loading