PDX-484: carry construct-vs-amend contract into MCP tool titles

[mrdailey99]

Summary

• Update the title: fields on provar_testcase_generate (→ Generate Test Case (full steps in one call), 43 chars) and provar_testcase_step_edit (→ Amend Existing Test Case Step, 29 chars).
• Both titles clear the cross-client chip-render comfort threshold (≤50 chars) and carry the construct-vs-amend split at the chip-level surface that some MCP clients render exclusively (Claude Desktop tool-picker chips, Cursor audit pane, inline tool-call refs in chat threads).
• Adds title-level assertions in unit tests and in scripts/pdx-482-validate.cjs (run in both standard and compact schema modes — titles are mode-independent, but asserting in both surfaces drift early).

Jira

https://provartesting.atlassian.net/browse/PDX-484

Test plan

• yarn compile passes
• node_modules/.bin/nyc node_modules/.bin/mocha "test/**/*.test.ts" — 1133 passing, 0 failing
• node scripts/mcp-smoke.cjs — 54/54 PASS
• yarn lint clean
• node scripts/pdx-482-validate.cjs — 32/32 PASS (12 new title assertions across the two modes)

Changes

• src/mcp/tools/testCaseGenerate.ts — title updated, contract intent commented at the call site
• src/mcp/tools/testCaseStepTools.ts — title updated, contract intent commented at the call site
• test/unit/mcp/testCaseGenerate.test.ts — MockMcpServer captures title; added phrasing + length assertions
• test/unit/mcp/testCaseStepTools.test.ts — same pattern for the step_edit suite
• scripts/pdx-482-validate.cjs — added titleAssertions helper, invoked from both standard and compact passes
• docs/mcp.md — one-line title-level contract note in each tool section
• docs/mcp-pilot-guide.md — Scenario 12 mentions the title-level contract as evidence of the construct-vs-amend split

Downstream consumer check

Repo-wide search for the literal old title strings ('Generate Test Case', 'Edit Test Case Step', double-quoted variants) found no other callers — the only references were in the two source files updated by this PR. No follow-up breakage to flag.

Cross-client pilot evaluation

Manual pilot evaluation in Claude Desktop and Cursor is part of the AC and has not been captured in this PR. Reviewer should verify both new titles render without truncation in those two clients' default tool-picker / audit surfaces before landing. If a client truncates either title, choose the alternate candidate listed in the Jira ticket.

Public docs

docs/provar-mcp-public-docs.md and docs/university-of-provar-mcp-course.md are maintained separately by the Provar team — they may want to mention the new chip-level titles in customer-facing release notes for the next pilot drop.

[github-actions]

Quality Orchestrator

🟢 LOW · 4 / 100 · All changed files have mapped tests.

---

🧪 Tests to Run · Running 2 of 51 tests

• unit/mcp/testCaseGenerate.test.ts
• unit/mcp/testCaseStepTools.test.ts

▶ Run command

npx vitest run \
  unit/mcp/testCaseGenerate.test.ts \
  unit/mcp/testCaseStepTools.test.ts

---

_{⚡ quality-orchestrator  ·  /qo stub <file>  ·  qo analyze-local}

[Copilot]

Pull request overview

Updates the title: fields on two MCP tools (provar_testcase_generate and provar_testcase_step_edit) to carry the construct-vs-amend contract at the chip-level surface that some MCP clients render exclusively, and adds title-level assertions to tests, the PDX-482 validation script, and docs.

Changes:

• New titles: Generate Test Case (full steps in one call) (43 chars) and Amend Existing Test Case Step (29 chars).
• Test mocks now capture title; new assertions enforce canonical phrasing + ≤50-char length.
• Docs (mcp.md, mcp-pilot-guide.md) and scripts/pdx-482-validate.cjs note the title-level contract.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/mcp/tools/testCaseGenerate.ts	Updates title to include "(full steps in one call)" with explanatory comment.
src/mcp/tools/testCaseStepTools.ts	Updates title to "Amend Existing Test Case Step" with explanatory comment.
test/unit/mcp/testCaseGenerate.test.ts	MockMcpServer now captures title; adds phrasing + length assertions.
test/unit/mcp/testCaseStepTools.test.ts	Same pattern for the step_edit suite.
scripts/pdx-482-validate.cjs	Adds titleAssertions helper invoked from both standard and compact passes.
docs/mcp.md	Adds title-level contract note in each tool section.
docs/mcp-pilot-guide.md	Scenario 12 mentions the title-level contract.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.