Skip to content

fix(cli): cron create by agent name, correct create/list response par… - #1276

Open
89rat wants to merge 1 commit into
RightNow-AI:mainfrom
89rat:fix/cli-cron-name-resolution
Open

fix(cli): cron create by agent name, correct create/list response par…#1276
89rat wants to merge 1 commit into
RightNow-AI:mainfrom
89rat:fix/cli-cron-name-resolution

Conversation

@89rat

@89rat 89rat commented Aug 2, 2026

Copy link
Copy Markdown

…sing, data_dir in status

  • cron create: resolve agent name -> UUID via /api/agents (help text already promised names); clear 'Unknown agent' error with fix hint
  • cron create: parse the actual API response shape ({"result": json-string}) instead of a nonexistent top-level id, eliminating the false 'Failed: ?' on success; surface real error bodies
  • cron list: read the {jobs, total} wrapper (bare array still accepted), render schedule from schedule.expr/secs and prompt from action.message, and show agent names instead of raw UUIDs
  • status: include data_dir in GET /api/status; CLI falls back to home_dir for older daemons (fixes 'Data dir: ?')

Verified: cargo clippy -D warnings clean on both crates; 196 tests passing; live daemon round trip confirmed all paths.

Claude-Session: https://claude.ai/code/session_013nK3VtM7ZcEYG5mdrjQJcB

Summary

Changes

Testing

  • cargo clippy --workspace --all-targets -- -D warnings passes
  • cargo test --workspace passes
  • Live integration tested (if applicable)

Security

  • No new unsafe code
  • No secrets or API keys in diff
  • User input validated at boundaries

…sing, data_dir in status

- cron create: resolve agent name -> UUID via /api/agents (help text already
  promised names); clear 'Unknown agent' error with fix hint
- cron create: parse the actual API response shape ({"result": json-string})
  instead of a nonexistent top-level id, eliminating the false 'Failed: ?' on
  success; surface real error bodies
- cron list: read the {jobs, total} wrapper (bare array still accepted), render
  schedule from schedule.expr/secs and prompt from action.message, and show
  agent names instead of raw UUIDs
- status: include data_dir in GET /api/status; CLI falls back to home_dir for
  older daemons (fixes 'Data dir: ?')

Verified: cargo clippy -D warnings clean on both crates; 196 tests passing;
live daemon round trip confirmed all paths.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013nK3VtM7ZcEYG5mdrjQJcB
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.

2 participants