fix: add trailing periods to MCP tool descriptions#52
Merged
Patrik Simek (patriksimek) merged 1 commit intomainfrom Apr 16, 2026
Merged
fix: add trailing periods to MCP tool descriptions#52Patrik Simek (patriksimek) merged 1 commit intomainfrom
Patrik Simek (patriksimek) merged 1 commit intomainfrom
Conversation
Patrik Simek (patriksimek)
added
the
no-production-impact
Contributor
There was a problem hiding this comment.
Pull request overview
Standardizes MCP tool description strings by adding trailing periods across multiple endpoint tool definitions, and updates package versioning metadata.
Changes:
- Add trailing periods to MCP tool
descriptionfields across many endpoint tool definitions. - Bump SDK package version from
1.2.0to1.2.1. - Update
package-lock.json(including a changed resolved version forhono).
Reviewed changes
Copilot reviewed 23 out of 24 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/endpoints/users.mcp.ts | Add trailing period to tool description. |
| src/endpoints/teams.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/sdk/webhooks.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/sdk/rpcs.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/sdk/modules.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/sdk/functions.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/sdk/connections.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/sdk/apps.mcp.ts | Add trailing periods to tool descriptions (also exposes “a SDK” grammar on edited lines). |
| src/endpoints/scenarios.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/organizations.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/keys.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/incomplete-executions.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/hooks.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/functions.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/folders.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/executions.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/enums.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/devices.mcp.ts | Add trailing period to tool description. |
| src/endpoints/data-structures.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/data-stores.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/data-store-records.mcp.ts | Add trailing periods to tool descriptions. |
| src/endpoints/connections.mcp.ts | Add trailing periods to tool descriptions. |
| package.json | Bump package version to 1.2.1. |
| package-lock.json | Sync lockfile version fields to 1.2.1 and includes a hono resolution change. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| name: 'sdk-apps_delete', | ||
| title: 'Delete SDK app', | ||
| description: 'Delete a SDK app by name and version', | ||
| description: 'Delete a SDK app by name and version.', |
There was a problem hiding this comment.
Grammar: use "an" before "SDK". Consider changing this to "Delete an SDK app by name and version."
Suggested change
| description: 'Delete a SDK app by name and version.', | |
| description: 'Delete an SDK app by name and version.', |
| name: 'sdk-apps_get-section', | ||
| title: 'Get SDK app section', | ||
| description: 'Get a specific section of a SDK app', | ||
| description: 'Get a specific section of a SDK app.', |
There was a problem hiding this comment.
Grammar: use "an" before "SDK". Consider changing this to "Get a specific section of an SDK app."
Suggested change
| description: 'Get a specific section of a SDK app.', | |
| description: 'Get a specific section of an SDK app.', |
| name: 'sdk-apps_set-section', | ||
| title: 'Set SDK app section', | ||
| description: 'Set/update a specific section of a SDK app', | ||
| description: 'Set/update a specific section of a SDK app.', |
There was a problem hiding this comment.
Grammar: use "an" before "SDK". Consider changing this to "Set/update a specific section of an SDK app."
Suggested change
| description: 'Set/update a specific section of a SDK app.', | |
| description: 'Set/update a specific section of an SDK app.', |
| { | ||
| "name": "@makehq/sdk", | ||
| "version": "1.2.0", | ||
| "version": "1.2.1", |
There was a problem hiding this comment.
PR title indicates a punctuation-only change, but this also bumps the package version. Please confirm this release bump is intended for this change; otherwise keep the version unchanged (or move the version bump into a separate release PR).
| name: 'sdk-apps_get', | ||
| title: 'Get SDK app', | ||
| description: 'Get a SDK app by name and version', | ||
| description: 'Get a SDK app by name and version.', |
There was a problem hiding this comment.
Grammar: use "an" before an initialism pronounced with a leading vowel sound ("SDK"). Consider changing this to "Get an SDK app by name and version."
Suggested change
| description: 'Get a SDK app by name and version.', | |
| description: 'Get an SDK app by name and version.', |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.