Repository navigation
fix(glean): align meeting_lookup guidance with inclusive date bounds - #30
shashank-tomar-glean wants to merge 2 commits into
Conversation
The meeting_lookup tool served by Glean MCP treats after/before as inclusive and requires both, and no longer accepts extract_transcript or exhaustive. The bundled reference still told the model to pad each bound by a day and to pass those parameters, so "today" or single-date queries pulled in meetings from the neighboring days. Update the using-glean meeting_lookup reference, the meeting_lookup row in the search.md cross-tool table, and the meeting-analyzer agent to the current schema: required inclusive bounds, period keywords, peer by email, cursor paging, and reading transcripts with read_document.
…d names Refer to the transcript URL and attendee/organizer emails generically, and cover responses that report truncation without a cursor, so the guidance holds for both the current MCP response shape and the upcoming cursor-based one.
|
Thanks for this. You're right that the bundled guidance is wrong. It says the date bounds are exclusive and tells the model to pad them, so a single-date question pulls in meetings from the neighboring days. We're closing this in favor of a different fix. We're cutting The parameter details here belong in the |
Summary
The
meeting_lookuptool that Glean MCP serves today treatsafterandbeforeas inclusive and requires both on every call. Its schema no longer listsextract_transcriptorexhaustive.The plugin's bundled guidance still described the old behavior. It told the model that filtering is exclusive and to pad each bound by a day, and its examples passed parameters the schema no longer lists. As a result, "what's on my calendar today?" or a single-date question pulls in meetings from the neighboring days.
Changes
using-glean/reference/meeting-lookup.md: rewrite the parameter table, date handling, examples, and pitfalls to match the current schema:after/beforeare required and inclusive.peermust be an email.calendar_idsmust come from the user.cursorhandles paging when a response returns one; if a response reports truncation without a cursor, narrow the date range.read_document, using the meeting's transcript URL or itsurl.using-glean/reference/search.md: fix themeeting_lookuprow of the cross-tool filter table, whose example padded the dates.agents/meeting-analyzer.md: replace the inline query-string style andextract_transcriptwith structured parameters.Testing
npm testpasses: build plus validate for claude, cursor, and codex.meeting_lookuptool schema.after="today", before="today"returns all of today's meetings and nothing from adjacent days.meeting_lookupto the plugin. Codex readusing-glean/reference/meeting-lookup.mdbefore its calendar calls.meeting_lookupcall withafter="today"andbefore="today", returning exactly that day's events.after="now"andbefore="this_week", covering Monday to Friday with no padded days.meeting_lookup, thenread_documenton the transcript URL from the result. Today's MCP response names that fieldmeetingTranscriptUrl, and the field-agnostic wording handled it.Not in this PR: a few skills (
meeting-prep,catch-up,onboarding,project-awareness,project-handoff,project-synthesizer) still use themeeting_lookup "[topic] [time period]"shorthand. That can be a follow-up. In the same end-to-end runs, "Prep me for my next meeting" went throughmeeting-prepand failed in one of four runs: the model reported that calendar tools were unavailable.