Add regression coverage for update_issue MCP body length handling above 10KB

[Copilot]

update_issue was previously vulnerable to an MCP pre-validation mismatch where large bodies could be rejected by the default 10KB string guard instead of honoring the tool schema’s explicit body limit. This PR adds focused regression tests so update_issue.body behavior stays aligned with the 65KB-class schema contract used by sibling issue/comment tools.

• Regression scope: MCP string-length pre-validation vs tool schema

  • Adds test coverage in actions/setup/js/mcp_scripts_validation.test.cjs for update_issue using the real safe_outputs_tools.json schema.
  • Verifies that a body larger than MAX_STRING_INPUT_BYTES is accepted when update_issue.body.maxLength is declared.
  • Verifies that values over schema boundary (65537) are still rejected with a body max-length violation.

• Fixture/schema loading hardening

  • Introduces a small helper to load update_issue schema from safe_outputs_tools.json.
  • Improves failure diagnostics when schema shape/tool entries are invalid or missing.

const updateIssueSchema = loadUpdateIssueToolSchema();

expect(updateIssueSchema.properties.body.maxLength).toBe(65536);
expect(
  validateStringInputLengths({ body: "a".repeat(MAX_STRING_INPUT_BYTES + 1) }, updateIssueSchema)
).toEqual([]);

const violations = validateStringInputLengths({ body: "a".repeat(65537) }, updateIssueSchema);
expect(violations[0]).toMatchObject({ field: "body", limit: 65536, unit: "characters" });

• Fixes update_issue body is capped at 10 KB while create_issue / add_comment allow 65 KB #45787

[github-actions]

👋 Thanks for the fix! This PR looks great — it's a focused test-addition that adds solid regression coverage for the update_issue body limit schema fix.

What this does well:

• Adds two targeted regression tests that verify update_issue.body accepts up to 65536 characters (matching the schema)
• Verifies the boundary case (65537 bytes) correctly triggers a validation error
• Keeps the change minimal and scoped — just the tests, no unrelated edits
• Clear reference to the upstream issue (update_issue body is capped at 10 KB while create_issue / add_comment allow 65 KB #45787)

Ready for review — the tests look comprehensive and the implementation follows the test structure in place. 🎉

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• patchdiff.githubusercontent.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "patchdiff.githubusercontent.com"

See Network Configuration for more information.

Generated by ✅ Contribution Check · 64.1 AIC · ⌖ 8.09 AIC · ⊞ 6.2K · ◷

[Copilot]

Pull request overview

Adds regression coverage ensuring update_issue.body honors its explicit schema length limit instead of the default 10KB MCP guard.

Changes:

• Loads the real update_issue tool schema with actionable failures.
• Tests acceptance above 10KB and rejection above 65,536 characters.

Show a summary per file

File	Description
actions/setup/js/mcp_scripts_validation.test.cjs	Adds schema-backed body-length regression tests.

Review details

Tip

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

• Files reviewed: 1/1 changed files
• Comments generated: 0
• Review effort level: Medium

[github-actions]

🎉 This pull request is included in a new release.

Release: v0.82.14