-
Notifications
You must be signed in to change notification settings - Fork 19
feat: add schedule creation and update functionality #28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
dmcclure-pdt
wants to merge
18
commits into
PagerDuty:main
Choose a base branch
from
dmcclure-pdt:feature/add-schedule-creation
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
feat: add schedule creation and update functionality #28
dmcclure-pdt
wants to merge
18
commits into
PagerDuty:main
from
dmcclure-pdt:feature/add-schedule-creation
Conversation
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
- Add Golden Template for extending PagerDuty MCP Server - Add detailed requirements for schedule creation feature 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Provides step-by-step instructions for implementing the schedule creation feature, including: - Model definitions - Tool implementation - Testing strategy - Git workflow guidance 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
…le restrictions The PagerDuty API requires start_day_of_week values for all restriction types, even for daily_restriction where it's not logically needed. This change: 1. Updates ScheduleLayerRestriction model to require start_day_of_week with default=1 2. Improves create_schedule to ensure this field is always provided 3. Adds documentation explaining this API requirement This fixes the issue where schedule creation with restrictions would fail with 'Start day of week must be in 1, 2, 3, 4, 5, 6, 7...' error. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Added create_schedule function to the tools table in README.md to document its availability. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Add update_schedule function to update existing schedules - Add ScheduleUpdateRequest model for validation - Add helper function get_layer_differences for comparing layers - Update README.md with new tools - Add proper validation for schedule parameters and restrictions 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
|
👍 for this. This functionality is a requirement for us to make use of the PagerDuty MCP. |
|
@imjaroiswebdev Can this PR be merged please? |
|
Hi @kyub, thank you for sharing how interested you are in leveraging this feature. This PR will get merged hopefully before the end of this same week. |
|
Great, thanks! |
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.
Description
This PR adds schedule creation and update support to the PagerDuty MCP server, enabling users to create and manage complex on-call schedules directly through the MCP interface. The implementation follows the PagerDuty API specification for schedule operations, allowing for configuration of:
Issue number: #XXX (replace with actual issue number if applicable)
Type of change
Checklist
Additional Notes
Feature Overview
create_scheduletool for schedule creation operationsupdate_scheduletool for schedule updatesScheduleCreateRequest,ScheduleUpdateRequest, and related models__init__.pyTesting
Test Prompts
The following test prompts were used to verify the functionality:
Schedule Update Patterns
The implementation includes special handling for schedule updates, which have unique behavior in the PagerDuty API:
schedule_layers=[]parameter