Skip to content

feat(reports): add partial name filter to list_reports - #323

Merged
dmitrijs-pavlovs-dev merged 5 commits into
mainfrom
feat/list-reports-name-filter
Oct 6, 2026
Merged

dmitrijs-pavlovs-dev merged 5 commits into
mainfrom
feat/list-reports-name-filter

Conversation

@dmitrijs-pavlovs-dev

@dmitrijs-pavlovs-dev dmitrijs-pavlovs-dev commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

list_reports now accepts an optional name and sends it as the API's server-side nameContains filter, so case-insensitive partial matching applies before pagination. The parameter is defined once in Zod and appears in both the derived stdio schema and the hosted argument schema.

Demo-mode searches now apply the same name matching to the sample reports and return the matching row count. Previously a search such as name: "cloud cost" returned all three samples; it now returns only the matching report. Searches with no match follow the handler's existing No reports found response, and searches without a name still return all samples.

Regression tests reproduced the demo bug before the fix. They cover case-insensitive partial matches, multiple matches, no matches, empty and omitted names, fixture preservation, and the real demo-token handler without HTTP calls. The existing request-mapping test verifies the production API parameter.

Validation after synchronizing with main: 1,471 unit tests, 320 integration tests, yarn check:dev, build, and whitespace checks passed. This change used deterministic validation and makes no new live API claim. No standalone validation documentation is included.

@dmitrijs-pavlovs-dev
dmitrijs-pavlovs-dev changed the base branch from main to fix/report-contracts October 6, 2026 09:51
Base automatically changed from fix/report-contracts to main October 6, 2026 10:28
Comment thread src/tools/reports.ts
@dmitrijs-pavlovs-dev
dmitrijs-pavlovs-dev merged commit 67d6b2d into main Oct 6, 2026
5 checks passed
@dmitrijs-pavlovs-dev
dmitrijs-pavlovs-dev deleted the feat/list-reports-name-filter branch October 6, 2026 13:41
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