Skip to content

fix(prompt): trust tool results + nudge toward list filters#31

Merged
oxyc merged 1 commit into
mainfrom
fix/v0.4.1-tool-result-trust
May 29, 2026
Merged

fix(prompt): trust tool results + nudge toward list filters#31
oxyc merged 1 commit into
mainfrom
fix/v0.4.1-tool-result-trust

Conversation

@oxyc

@oxyc oxyc commented May 29, 2026

Copy link
Copy Markdown
Member

A v0.4.0 session repeatedly re-called `gds/forms-read` on the same form, writing "didn't return the full details" each time, even though every return contained all 10 fields. The model was treating long JSON returns as "incomplete".

Adds an explicit prompt block under "Bulk operations":

  • The JSON return IS the full result.
  • Truncation is always explicit (a `truncated: true` flag or a stated "first N items" note).
  • If a list result is missing an item the user expects, check whether default filters hide it (`gds/forms-list` defaults to active + non-trashed — pass `include_inactive` / `include_trashed`) before assuming the item is gone.

Companion to gds-mcp #xx which adds those filter inputs and fixes the silent restoreForm bug.

A v0.4.0 session repeatedly re-called gds/forms-read for the same
form, writing "didn't return the full details" each time, even though
every response shipped all 10 fields. The model was treating a long
JSON return as "incomplete" and burning a turn per re-read.

Add an explicit guidance block:
  - the JSON return IS the full result;
  - truncation is always explicit (a `truncated: true` flag or a
    stated "first N items" note) — without that marker, treat the
    result as complete;
  - and if a list result is missing an item the user expects,
    check whether default filters hide it (gds/forms-list defaults
    to active + non-trashed) before assuming the item is gone.

Companion to gds-mcp v0.5.0 which adds include_inactive /
include_trashed inputs to gds/forms-list so the model has a real
escape hatch when the user is hunting for a disabled form.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@oxyc oxyc merged commit a84124c into main May 29, 2026
3 checks passed
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.

1 participant