mirror of
https://github.com/sbroenne/mcp-server-excel.git
synced 2026-09-19 07:53:08 +08:00
Stefan Broenner
36845831ec
* Add privacy-safe usage analytics reporting Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 915c2266-658e-45b7-ae7d-c9213c810f01 * Fix analytics changeset package Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 915c2266-658e-45b7-ae7d-c9213c810f01 * Make analytics report public-friendly Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 915c2266-658e-45b7-ae7d-c9213c810f01 * Finalize public usage analytics Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 915c2266-658e-45b7-ae7d-c9213c810f01 * Harden analytics validation Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 915c2266-658e-45b7-ae7d-c9213c810f01 --------- Copilot-Session: 915c2266-658e-45b7-ae7d-c9213c810f01
72 lines
2.7 KiB
PowerShell
72 lines
2.7 KiB
PowerShell
<#
|
|
.SYNOPSIS
|
|
Validates Copilot interpretation and adds it to a sanitized analytics report.
|
|
#>
|
|
param(
|
|
[Parameter(Mandatory = $true)]
|
|
[string]$AnalyticsPath,
|
|
|
|
[Parameter(Mandatory = $true)]
|
|
[string]$InterpretationPath,
|
|
|
|
[Parameter(Mandatory = $true)]
|
|
[string]$OutputPath
|
|
)
|
|
|
|
$ErrorActionPreference = "Stop"
|
|
$analyticsText = [IO.File]::ReadAllText([IO.Path]::GetFullPath($AnalyticsPath))
|
|
$analytics = $analyticsText | ConvertFrom-Json
|
|
$interpretation = [IO.File]::ReadAllText([IO.Path]::GetFullPath($InterpretationPath)).Trim()
|
|
|
|
if ($interpretation.Length -lt 100 -or $interpretation.Length -gt 4000) {
|
|
throw "Interpretation must contain between 100 and 4000 characters."
|
|
}
|
|
|
|
$requiredHeadings = @(
|
|
"## What changed",
|
|
"## How well it worked",
|
|
"## How people use it",
|
|
"## What we will improve"
|
|
)
|
|
foreach ($heading in $requiredHeadings) {
|
|
if ($interpretation -notmatch "(?m)^$([regex]::Escape($heading))\s*$") {
|
|
throw "Interpretation is missing required heading '$heading'."
|
|
}
|
|
}
|
|
|
|
$forbiddenPatterns = [ordered]@{
|
|
"link" = 'https?://|\[[^\]]+\]\([^)]+\)'
|
|
"HTML" = '<[A-Za-z!/][^>]*>'
|
|
"email" = '[A-Za-z0-9._%+-]+@[A-Za-z0-9.-]+\.[A-Za-z]{2,}'
|
|
"Windows path" = '[A-Za-z]:\\'
|
|
"UNC path" = '\\\\[^\\\s]+\\'
|
|
"identifier field" = '(?i)\b(UserId|SessionId|FileSessionId|ClientIP|ClientCity|ClientCountryOrRegion|StackTrace)\b'
|
|
"technical jargon" = '(?i)\b(p50|p95|p99|percentile|invocations?|cohort|telemetry|saniti[sz]ed|AggregateException|COMException|tail latency)\b'
|
|
"internal action name" = '(?i)\b[a-z]+(?:_[a-z]+)+\b|\b[a-z]+/[a-z-]+\b'
|
|
}
|
|
foreach ($entry in $forbiddenPatterns.GetEnumerator()) {
|
|
if ($interpretation -match $entry.Value) {
|
|
throw "Interpretation contains forbidden $($entry.Key) content."
|
|
}
|
|
}
|
|
|
|
$allowedNumbers = [Collections.Generic.HashSet[string]]::new([StringComparer]::Ordinal)
|
|
$numberPattern = '(?<![\w.])-?(?:\d{1,3}(?:,\d{3})+|\d+)(?:\.\d+)*'
|
|
foreach ($match in [regex]::Matches($analyticsText, $numberPattern)) {
|
|
[void]$allowedNumbers.Add($match.Value.Replace(",", ""))
|
|
}
|
|
foreach ($match in [regex]::Matches($interpretation, $numberPattern)) {
|
|
$normalizedValue = $match.Value.Replace(",", "")
|
|
if (-not $allowedNumbers.Contains($normalizedValue)) {
|
|
throw "Interpretation contains unsupported numeric claim '$($match.Value)'."
|
|
}
|
|
}
|
|
|
|
$analytics | Add-Member -NotePropertyName interpretation -NotePropertyValue $interpretation
|
|
$analytics | Add-Member -NotePropertyName interpretationModel -NotePropertyValue "GitHub Copilot CLI"
|
|
$json = $analytics | ConvertTo-Json -Depth 10
|
|
[IO.File]::WriteAllText(
|
|
[IO.Path]::GetFullPath($OutputPath),
|
|
$json + "`n",
|
|
[Text.UTF8Encoding]::new($false))
|