Skip to content

Conversation

@Saurabh7019
Copy link
Collaborator

🎯 Aim

Adds copy_spo_page Language Model Tool for modern SharePoint page copy operation. Closes #587

πŸ“· Result

image

βœ… What was done

  • Implemented by referring to other page-related LM tools

πŸ”— Related issue

Closes: #587

Copilot AI review requested due to automatic review settings December 15, 2025 14:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for copying SharePoint pages through a new Language Model Tool called copy_spo_page. The implementation enables GitHub Copilot to assist users in duplicating modern SharePoint pages within the same site or across different sites, with support for overwriting existing pages and copying page templates.

Key changes:

  • Implemented SharePointPageCopy LM tool following established patterns for SPO page operations
  • Added tool registration and exports in alphabetical order with other page operations
  • Updated documentation to reflect the new page copy capability

Reviewed changes

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

Show a summary per file
File Description
src/chat/tools/spo/page/PageCopy.ts New Language Model Tool implementation for copying SharePoint pages with auth validation, CLI execution, and user confirmation
src/chat/tools/spo/index.ts Export added for PageCopy class in alphabetical order
src/chat/tools/ChatTools.ts Registered copy_spo_page tool with import and instantiation
package.json Added comprehensive tool configuration with schema, descriptions, and parameters for the copy_spo_page tool
.github/copilot-instructions.md Updated Language Model Tools documentation to include the full set of page operations including copy_spo_page

@Adam-it Adam-it force-pushed the dev branch 2 times, most recently from 4ca0a6a to 8716fdf Compare December 17, 2025 22:10
@Adam-it Adam-it marked this pull request as draft December 17, 2025 22:10
@Adam-it
Copy link
Member

Adam-it commented Dec 17, 2025

@Saurabh7019 I just did a minor and in this process the dev gets rebased/aligned with main.
Please perform the rebase on your side before we proceed

@Saurabh7019 Saurabh7019 marked this pull request as ready for review December 19, 2025 08:44
@Adam-it Adam-it self-assigned this Jan 15, 2026
Adam-it
Adam-it previously approved these changes Jan 15, 2026
Copy link
Member

@Adam-it Adam-it left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checked locally βœ…

Image

awesome work πŸ‘
nothing to add

@Adam-it Adam-it changed the title Add copy_spo_page LM tool. Closes #587 Adds copy_spo_page LM tool. Closes #587 Jan 15, 2026
@Adam-it Adam-it merged commit 0937694 into pnp:dev Jan 15, 2026
1 check passed
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.

πŸ’‘ [Feature]: Add copy_spo_page Language Model Tool for modern SharePoint page copy operation

2 participants