fix: prefer prompt names for multi-skill cron jobs
This commit is contained in:
parent
a6dc73fa07
commit
2536ff328b
2 changed files with 13 additions and 1 deletions
|
|
@ -323,7 +323,7 @@ def create_job(
|
||||||
now = _hermes_now().isoformat()
|
now = _hermes_now().isoformat()
|
||||||
|
|
||||||
normalized_skills = _normalize_skill_list(skill, skills)
|
normalized_skills = _normalize_skill_list(skill, skills)
|
||||||
label_source = (normalized_skills[0] if normalized_skills else prompt) or "cron job"
|
label_source = (prompt or (normalized_skills[0] if normalized_skills else None)) or "cron job"
|
||||||
job = {
|
job = {
|
||||||
"id": job_id,
|
"id": job_id,
|
||||||
"name": name or label_source[:50].strip(),
|
"name": name or label_source[:50].strip(),
|
||||||
|
|
|
||||||
|
|
@ -262,6 +262,18 @@ class TestUnifiedCronjobTool:
|
||||||
listing = json.loads(cronjob(action="list"))
|
listing = json.loads(cronjob(action="list"))
|
||||||
assert listing["jobs"][0]["skills"] == ["blogwatcher", "find-nearby"]
|
assert listing["jobs"][0]["skills"] == ["blogwatcher", "find-nearby"]
|
||||||
|
|
||||||
|
def test_multi_skill_default_name_prefers_prompt_when_present(self):
|
||||||
|
result = json.loads(
|
||||||
|
cronjob(
|
||||||
|
action="create",
|
||||||
|
skills=["blogwatcher", "find-nearby"],
|
||||||
|
prompt="Use both skills and combine the result.",
|
||||||
|
schedule="every 1h",
|
||||||
|
)
|
||||||
|
)
|
||||||
|
assert result["success"] is True
|
||||||
|
assert result["name"] == "Use both skills and combine the result."
|
||||||
|
|
||||||
def test_update_can_clear_skills(self):
|
def test_update_can_clear_skills(self):
|
||||||
created = json.loads(
|
created = json.loads(
|
||||||
cronjob(
|
cronjob(
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue