fix(calendar): honor list_events date range aliases (#3283)

* fix(calendar): honor list_events date range aliases

* fix(calendar): reject partially resolved loose range queries
This commit is contained in:
badgerbees
2026-07-04 20:44:46 +08:00
committed by GitHub
parent 6f6cb6ea88
commit 5c16d39e91
5 changed files with 73 additions and 5 deletions
+26
View File
@@ -78,3 +78,29 @@ async def test_list_events_honors_range_aliases(start_key, end_key):
summaries = [event["summary"] for event in res["events"]]
assert summaries == ["Late June planning"]
assert "between 2126-06-01 and 2126-07-01" in res["response"]
async def test_list_events_rejects_partial_loose_range():
from src.tool_implementations import do_manage_calendar
owner = "calendar-partial-" + uuid.uuid4().hex[:8]
# Partial: has query and start, but no end
res = await do_manage_calendar(json.dumps({
"action": "list_events",
"query": "July",
"start_time": "2126-07-01T00:00:00Z",
}), owner=owner)
assert res.get("exit_code", 1) == 1, res
assert "list_events needs explicit start/end" in res.get("error", "")
# Partial: has query and end, but no start
res2 = await do_manage_calendar(json.dumps({
"action": "list_events",
"query": "July",
"end_time": "2126-07-31T23:59:59Z",
}), owner=owner)
assert res2.get("exit_code", 1) == 1, res2
assert "list_events needs explicit start/end" in res2.get("error", "")
+33
View File
@@ -78,3 +78,36 @@ async def test_update_event_dtstart_anchored_to_user_tz(tokyo_offset):
assert bool(ev.is_utc) is True
finally:
db.close()
async def test_list_events_accepts_start_time_end_time_aliases():
from src.tool_implementations import do_manage_calendar
owner = "list-" + uuid.uuid4().hex[:6]
created = await do_manage_calendar(json.dumps({
"action": "create_event",
"summary": "July planning",
"dtstart": "2026-07-15T12:00:00Z",
"dtend": "2026-07-15T13:00:00Z",
}), owner=owner)
assert created.get("exit_code", 0) == 0, created
listed = await do_manage_calendar(json.dumps({
"action": "list_events",
"start_time": "2026-07-01T00:00:00Z",
"end_time": "2026-08-01T00:00:00Z",
}), owner=owner)
assert listed.get("exit_code", 0) == 0, listed
assert "between 2026-07-01 and 2026-08-01" in listed["response"]
assert [event["summary"] for event in listed["events"]] == ["July planning"]
async def test_list_events_query_without_range_does_not_default_to_two_weeks():
from src.tool_implementations import do_manage_calendar
listed = await do_manage_calendar(json.dumps({
"action": "list_events",
"query": "July",
}), owner="list-" + uuid.uuid4().hex[:6])
assert listed.get("exit_code") == 1, listed
assert "explicit start/end" in listed["error"]