Remove Application Insights telemetry — zero logging policy#46
Merged
Conversation
- Add 93 new tests (CoreCommandsCoverage, ResultType invariants, serialization, ShapeHelpers, parameter validation) - Fix ~30 COM object leaks across 12 command files by moving ComUtilities.Release() calls into finally blocks - Add ArgumentException.ThrowIfNullOrWhiteSpace() guards to 25+ Core Command methods across 10 files Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Update title, description, badges for PowerPoint MCP Server - Replace all Excel references with PowerPoint equivalents - Update tool counts to accurate 20 tools / 84 operations - Update example prompts for PowerPoint workflows - Update architecture section for PowerPoint.Application COM API - Simplify Quick Start and CLI sections Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Update copilot-instructions.md and all 11 instruction files - Replace Excel references with PowerPoint equivalents throughout - Update COM API references (Excel.Application → PowerPoint.Application) - Replace workbook/worksheet terminology with presentation/slide - Update repo URLs to trsdn/mcp-server-ppt - Adapt Excel-specific concepts (Power Query, PivotTable, etc.) to PowerPoint equivalents (slides, shapes, animations, etc.) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
New tools (15 new operations): - Comment (4 ops): list, add, delete, clear slide comments - Placeholder (2 ops): list placeholders, set placeholder text - Background (3 ops): get info, set solid color, reset to master - HeaderFooter (2 ops): get/set footer, slide numbers, date - SmartArt (2 ops): get diagram info, add nodes - ShapeAlign (2 ops): align and distribute shapes Renamed instruction files: - excel-com-interop.instructions.md → ppt-com-interop.instructions.md - excel-connection-types-guide.instructions.md → ppt-com-patterns-guide.instructions.md - Updated all internal references to new file names Updated tests: 12 new coverage tests for new command areas Updated README: 26 tools / 101 operations Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…, Tags) New tools (9 new operations): - CustomShow (3 ops): list, create, delete custom slide shows - PageSetup (2 ops): get slide size/orientation, set slide dimensions - SlideImport (1 op): import slides from another .pptx file - Tags (3 ops): list/set/delete custom metadata on slides and shapes Total: 30 tools with 110 operations All 236 tests passing (116 Core + 120 MCP) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
New shape operations (4 new): - copy-to-slide: Copy shapes between slides - set-shadow: Shadow effects (on/off, offset) - add-connector: Connector lines between shapes (straight/elbow/curve) - merge: Boolean shape operations (union/combine/fragment/intersect/subtract) New export operation: - to-video: Export presentation as MP4 video Total: 30 tools with 115 operations Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Export: add print operation (copies, slide range) - Design: add list-color-schemes operation - All PowerPoint COM API features now implemented Total: 30 tools with 117 operations (started at 20/84) All 236 tests passing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Rewrite FEATURES.md from scratch for 30 PowerPoint tools / 104 operations - Fix Excel→PowerPoint in McpServer: PptToolsBase, PptTools, Program.cs, PptResourceProvider, Telemetry files - Fix CLI: Remove PowerPivot paths, update tray icon strings - Fix Service: Update comment in PptMcpService - Fix ComInterop: Update timeout comment in PptBatch - Fix Generators: Update example in Models.cs Only 3 legitimate Excel references remain (comparing PowerPoint vs Excel behavior) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- docs/: Updated 7 key docs (INSTALLATION, CONTRIBUTING, DEVELOPMENT, BREAKING-CHANGES, NUGET-GUIDE, AUTHOR, RELEASE-STRATEGY) - skills/: Updated ~50 files across shared/, ppt-cli/, ppt-mcp/, templates/ - CHANGELOG.md: 37 replacements across all version entries - Consistent PowerPoint branding throughout Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
… docs - PRIVACY.md, SECURITY.md: Full Excel→PowerPoint replacement - Issue templates + PR template: Updated all Excel references - docs/INSTALLATION.md: Fix remaining npx/config references - docs/CLAUDE-MCPB-SUBMISSION.md, MCP_REGISTRY_PUBLISHING.md: Updated - README.md: Fix last 2 Excel references Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Replace entire Excel version history (v1.0-v1.8.25) with fresh PowerPoint MCP changelog starting from [Unreleased]. Lists all 30 tools with 117 operations as initial feature set. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…content New operations: - shape duplicate: Copy shape on same slide - export save-as: Save as .pptx/.pptm/.potx/.ppsx/.pdf/.xps/.odp - background set-image: Set image as slide background Cleanup: - Removed 9 Excel-only skill reference files (Power Query, PivotTable, Data Model, DMV, M code, Range, Slicer, Worksheet, ConditionalFormat) - Removed Excel-only spec files and docs/archive/ - Rewrote agent mode file for PowerPoint workflows - Renamed excel_agent_mode.md → ppt_agent_mode.md Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
New operations: - slide set-name: Rename slides programmatically - shape flip: Flip shapes horizontally/vertically - notes append: Append to existing speaker notes - transition copy-to-all: Apply one slide's transition to all - window set-zoom: Set zoom level Cleanup: - Deleted 9 Excel-only skill reference files from ppt-cli + ppt-mcp - Deleted docs/COM-API-BEHAVIOR-FINDINGS.md (Excel COM) - Deleted docs/PIA-COVERAGE.md (Excel PIA) - Deleted 5 Excel-only spec files + specs/003-datamodel-llm-guidance/ - Renamed excel_agent_mode.md → ppt_agent_mode.md in skill refs Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- specs/: No longer doing spec-driven development - gh-pages/: Will be rebuilt from scratch later Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Rename excelProcess/excelProc → pptProcess in PptBatch.cs - Fix mcp-excel → mcp-ppt in 5 example config files - Fix EXCEL_DEFAULT_PRIVACY_LEVEL → PPT_DEFAULT_PRIVACY_LEVEL - Fix excel-skills → ppt-skills in release strategy - Fix excel_batch → ppt_batch in readme management Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- ServiceBridge: excelPath → presentationPath (3 methods) - PptMcpTelemetry: excelPath → presentationPath - PptBatch variable: excelPid → pptPid - READMEs: mcp-excel → mcp-ppt - packages/ppt-cli-skill: excel-cli-skill → ppt-cli-skill Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- PptBatchMessagePumpTests: excelProcess/excelCpu* → pptProcess/pptCpu* - PptBatchTimeoutTests: excelPid → pptPid - SessionManagerTimeoutTests: excelPid → pptPid - PptContextTests: excel → powerpoint in comments - SkillMdQualityTests: excel-cli → ppt-cli paths - McpServerIntegrationTests: excelPath → presentationPath comment Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Counts were inconsistent (84/104/117) after adding new operations. Actual count from interfaces: 30 tools, 125 operations. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ies + fix stale refs New operations: - text format-advanced: underline, strikethrough, subscript, superscript - slidetable format-cell: cell fill color, font bold/size, text alignment - docproperty get-custom/set-custom: custom document properties Stale content fixes: - llm-tests/: ExcelMcp→PptMcp, excelcli→pptcli paths - mcpb + vscode-extension: Replace Excel keywords with PowerPoint - scripts: \→\, EXCEL→POWERPNT - PptResourceProvider: worksheet→slide - Generators: Remove Excel-specific loadDestination/targetSheet Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- .gitattributes: xlsx/xlsm → pptx/pptm - Issue templates: Replace Excel tool/error examples with PowerPoint - copilot-instructions: Replace Excel quirks with PowerPoint quirks - critical-rules: Replace all PivotTable/PowerQuery/Worksheet code examples with Slide/Shape/SlideMaster PowerPoint equivalents Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Resolved: - #1: Deleted InspectTemplate.csx (Excel artifact) - #2: Rewrote ppt-com-patterns-guide.instructions.md for PowerPoint - #3: Added 9 ResultType invariant tests (278 Core tests now) - #4: Expanded ParameterValidationTests for all 30 command areas - #11: Made GetPlaceholderTypeName private Core.Tests: 278 passed (was 116, +162 new tests) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…LM tests) Resolved: - #13: Deleted orphaned deploy-gh-pages.yml workflow - #14: Deleted 2 disabled Excel-specific workflows - #15: Renamed excelcli.ico → pptcli.ico, updated .csproj - #16: Fixed all ExcelMcp refs in vscode-extension/src/extension.ts - #17: Fixed .gitignore Excel comments + stale paths - #18: Rewrote SKILL.cli.sbn + SKILL.mcp.sbn templates for PowerPoint - #19: Deleted 6 Excel-only LLM tests, renamed fixtures in 23 files - #20: Deleted batch-test-static.xlsx 398 tests passing (278 Core + 120 MCP) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Build was generating SKILL.md into skills/excel-cli/ instead of skills/ppt-cli/. Updated PptMcp.CLI.csproj output paths. Deleted stale skills/excel-cli/ generated directory. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…lone, master edit) New operations: - #7 text word-count: Count words across slides - #7 text alt-text-audit: Report shapes missing alt text - #7 text empty-placeholder-audit: Find unfilled placeholders - #8 hyperlink validate: Check all hyperlinks for broken/empty URLs - #9 master list-shapes: List shapes on slide master - #9 master edit-shape-text: Edit text on master shapes - #10 slide clone-with-replace: Clone slides with text substitution - #12 Refactored TextCommands.GetText (extracted ReadParagraphs helper) Total: 30 tools, 133 operations 398 tests passing (278 Core + 120 MCP) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…eplacement) New operations: - #5 chart set-data: Edit chart data via embedded Excel worksheet - #6 placeholder set-image: Replace placeholder with image at same position Total: 30 tools, 135 operations 398 tests passing (278 Core + 120 MCP) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…s, counts) Critical fixes: - #21: ToolCommandName mcp-excel → mcp-ppt in McpServer.csproj - #22: Fix Excel refs in infrastructure Bicep files - #23: Fix SKILL.cli.sbn template reference links (removed 7 Excel refs) - #24: Fix scripts (Build-AgentSkills, pre-commit, Test-CliWorkflow) - #25: Fix operation counts to 132 across README/FEATURES/CHANGELOG New features: - #26: Table read-cell operation - #27: Export all-slides-to-images (batch PNG export) Total: 30 tools, 132 operations 398 tests passing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- LICENSE: Added 'Copyright (c) 2026 Torsten Mahr' alongside original - 10 .cs files: Added second copyright line below original Sbroenne copyright - Original copyright preserved per MIT License requirements Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Bug fixes: - #30: Fixed 5 COM leaks + unsafe bool cast in 6 files - #31: Infrastructure azure runner naming excel→ppt - #32: pre-commit.ps1 excel-cli→ppt-cli references New operations (10): - #28: shape set-text-frame (margins, word wrap, autosize) - #33: text set-spacing (line/paragraph/character spacing) - #34: text set-bullets (bullet type, character, indent level) - #35: export extract-text + extract-images (content extraction) - #36: shape set-gradient-fill (two-color gradients) - #37: shape set-glow + set-reflection (visual effects) Total: 30 tools, 142 operations 398 tests passing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
New operations: - slide hide/unhide: Hide slides from slideshow - shape set-opacity: Set shape transparency - shape set-text-frame: Already committed, part of this cycle - chart set-legend: Show/hide/position legend - text insert-link: Inline text hyperlinks - animation set-timing: Set duration, delay, trigger type Operation count fixed to accurate 150 (was incorrectly 130/155) Total: 30 tools, 150 operations 398 tests passing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Bug fixes: - CloneWithReplace now handles grouped shapes (recursive helper) - Fixed conftest.py excelcli docstring New operations (6): - shape read-fill: Read fill type, color, transparency - shape read-line: Read line visibility, color, weight - slidetable write-row: Batch write row values - slidetable read-row: Batch read row values - slide get-thumbnail: Export slide as PNG thumbnail - notes read-all: Read all speaker notes at once Total: 30 tools, 156 operations 398 tests passing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
New tools: - #39 accessibility (3 ops): audit, get-reading-order, set-reading-order - #40 proofing (3 ops): check-spelling, set-language, get-language LLM tests cleanup (#38): - Deleted 7 Excel-only test files - Renamed file_worksheet → file_slide tests - Rewrote prompts in 15 files (Excel→PowerPoint) Added coverage tests for new tools Updated expected tool list (32 tools) Updated counts to 32 tools, 162 operations Total: 32 tools, 162 operations 398 tests passing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
… crop) - #41: Deleted stale uv.lock (excelmcp name) - #42: shape find-by-type (find shapes by MsoShapeType) - #43: shape copy-formatting (PickUp/Apply format painter) - #44: window set-view + get-view (Normal/Outline/SlideSorter/Notes/Master) - #45: image crop (CropLeft/Right/Top/Bottom) Total: 32 tools, 167 operations 398 tests passing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- background set-gradient: Slide gradient backgrounds - slidetable set-cell-border: Table cell border styling - slide summary: Presentation metadata (slide count, dimensions, etc.) - Fixed last 2 stale Excel refs in azure-runner.bicep Total: 32 tools, 170 operations 398 tests passing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Deleted: - docs/AZURE_SELFHOSTED_RUNNER_SETUP.md (runners deprovisioned) - docs/BREAKING-CHANGES.md (Excel-era breaking changes) - docs/CLAUDE-MCPB-SUBMISSION.md (old submission instructions) - infrastructure/ directory (Azure runners removed) Fixed: - mcpb/README.md: Replaced Power Query/PivotTable/Data Model examples with PowerPoint presentation examples Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Remove all telemetry code, dependencies, and documentation: Code: - Delete Telemetry/ folder (PptMcpTelemetry, TelemetryInitializer, SensitiveDataRedactor) - Remove App Insights from Program.cs (ConfigureTelemetry, InitializeTelemetryClient) - Remove TrackToolInvocation from PptToolsBase.cs - Remove App Insights NuGet packages from csproj and Directory.Packages.props - Remove GenerateTelemetryConfig MSBuild target - Delete Directory.Build.props.user.template Tests: - Delete TelemetryTests.cs and TelemetryIntegrationTests.cs - Remove telemetry DI setup and test from McpServerIntegrationTests.cs Docs: - Delete PRIVACY.md - Remove privacy links from all READMEs - Remove App Insights section from DEVELOPMENT.md - Remove APPINSIGHTS_CONNECTION_STRING from release workflow and docs - Add upstream credit to Stefan Broenner in README acknowledgments Build: 0 warnings, 0 errors (full solution) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Dependency ReviewThe following issues were found:
License Issuessrc/PptMcp.Build.Tasks/PptMcp.Build.Tasks.csproj
src/PptMcp.CLI/PptMcp.CLI.csproj
src/PptMcp.ComInterop/PptMcp.ComInterop.csproj
src/PptMcp.Core/PptMcp.Core.csproj
src/PptMcp.Generators.Cli/PptMcp.Generators.Cli.csproj
src/PptMcp.Generators.Mcp/PptMcp.Generators.Mcp.csproj
src/PptMcp.Generators.Shared/PptMcp.Generators.Shared.csproj
src/PptMcp.Generators/PptMcp.Generators.csproj
src/PptMcp.McpServer/PptMcp.McpServer.csproj
src/PptMcp.Service/PptMcp.Service.csproj
tests/PptMcp.CLI.Tests/PptMcp.CLI.Tests.csproj
tests/PptMcp.ComInterop.Tests/PptMcp.ComInterop.Tests.csproj
tests/PptMcp.Core.Tests/PptMcp.Core.Tests.csproj
tests/PptMcp.Diagnostics.Tests/PptMcp.Diagnostics.Tests.csproj
tests/PptMcp.McpServer.Tests/PptMcp.McpServer.Tests.csproj
tests/PptMcp.SkillGeneration.Tests/PptMcp.SkillGeneration.Tests.csproj
OpenSSF ScorecardScorecard details
Scanned Files
|
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
1 participant
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.
Remove all telemetry code, dependencies, and documentation. 23 files, -1181 lines.
Code
Tests
Docs
CI/CD
Build: 0 warnings, 0 errors (full solution)