Skip to content

fix:Added zod schema unit tests for all 6 MCP tool input contracts - #74

Open
VALENTINE-it wants to merge 1 commit into
hoainho:mainfrom
VALENTINE-it:val/tests
Open

fix:Added zod schema unit tests for all 6 MCP tool input contracts#74
VALENTINE-it wants to merge 1 commit into
hoainho:mainfrom
VALENTINE-it:val/tests

Conversation

@VALENTINE-it

@VALENTINE-it VALENTINE-it commented Aug 7, 2026

Copy link
Copy Markdown

#9

Linked issue

Closes #___

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • Feature (non-breaking change that adds functionality)
  • Performance improvement
  • Documentation only
  • Refactor (no behavior change)
  • Test additions or fixes
  • Chore (tooling, deps, build config)

What changed

Testing notes

$ npm run test:run
...

$ npm run build
...

Claim confirmation

If this PR closes a good first issue or help wanted issue, please confirm:

  • I starred the repo ⭐ — see CONTRIBUTING.md → How to claim. This is now enforced by CI (.github/workflows/star-check.yml) — the "Star Check" status will block merge until you star.
  • I commented I'll take this (or similar) on the issue before starting work, so two people don't accidentally race on the same issue. (Honor-system; not CI-enforced.)

If this PR is from a maintainer or a follow-up to a tracked plan, both can be skipped — just delete this section. Maintainer/bot/tracked-plan-labeled PRs are auto-exempted by the Star Check workflow.

Checklist

  • npm run test:run passes locally (or pre-existing failures are unrelated to this change)
  • npm run build succeeds locally
  • CHANGELOG.md entry added under ## [Unreleased] (skip for docs-only or test-only PRs)
  • PR title follows Conventional Commits (e.g., fix(panel): ...)
  • No new runtime dependencies added without prior discussion in a Discussion

Screenshots / GIFs

Additional notes

@VALENTINE-it

Copy link
Copy Markdown
Author
Screenshot from 2026-08-07 21-26-20 Screenshot from 2026-08-07 21-28-00

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