Migrate cspell config to per-service files ***NO_CI***

[maorleger]

What

Adopt the latest Azure SDK spellcheck guidance
(spellcheck.md):

• Move service-scoped overrides out of the central .vscode/cspell.json
  into per-service sdk/<service>/cspell.yaml files that import the central
  config (the import is required).
• Convert legacy per-service cspell.json → cspell.yaml, add the missing
  import to existing per-service cspell.yaml files, and consolidate each
  service down to a single sdk/<service>/cspell.yaml.
• Alphabetize word lists (case-insensitive) and dedupe.

Cross-cutting words stay in the central config (they have no reliable
per-service attribution).

Why

Keeping service words in per-service files reduces merge conflicts in the hot
central file and makes the lists easier to read — exactly what the guidance now
recommends.

Safety / validation

• cspell uses the nearest config for each file, so consolidating package
  configs up to the service directory preserves coverage (verified empirically
  with cspell 10, the pinned version).
• Conservation check: the full case-insensitive word union is identical
  before and after — 0 words lost.
• Spot-checked affected files (e.g. voicelive/keyvault review docs, storage &
  cosmos sources) still report 0 issues.

CI

Config-only change across many packages, so the commit is tagged ***NO_CI***
to avoid a repo-wide build storm. Opened as a draft.

[azure-pipelines]

Azure Pipelines:
Successfully started running 3 pipeline(s).
7 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

[maorleger]

Reference: this migration follows the Azure SDK spellcheck guidance — Migrating from .vscode/cspell.json to sdk/<service>/cspell.yaml.

[maorleger]

Updated to fix the spell-check failure seen in the earlier CI run
(build 6573147).

Root cause: the initial version kept service words inside per-file overrides
blocks. When CI spell-checks the changed cspell.yaml file itself, top-level
words self-cover it but words nested in overrides do not, so tokens like
BYOK/coreai were flagged as unknown in sdk/agentserver/cspell.yaml.

Fix: follow the guidance literally — one sdk/<service>/cspell.yaml per
service with all words flattened to top-level words (no overrides). The file
now self-covers, and a single config per service keeps words scoped to that
service. Verified locally with cspell 10: every generated cspell.yaml passes a
self-lint, no words were lost vs. the previous config, and affected source files
still pass.

[maorleger]

Refreshed onto the latest upstream/main (~3 weeks of changes) and removed the
***NO_CI*** token from the branch commit so PR validation actually runs — the
token was causing ADO to treat the PR builds as informational (skipped) runs, so
we never got a green signal. ***NO_CI*** will be re-added to the squash-merge
commit at merge time to still avoid the post-merge build storm.

The rebase was clean (cspell-only). One new package config that landed upstream
in the meantime (azure-ai-agentserver-activity/cspell.json) was folded into the
service-level sdk/agentserver/cspell.yaml. Re-validated with cspell 10: every
generated cspell.yaml self-lints clean and no words were lost.

[maorleger]

CI status: spell-check passes; remaining failures are pre-existing/environmental

The change this PR is actually about — the Check spelling (cspell) step — passes (e.g. python - cosmos - ci → Analyze → Check spelling (cspell): succeeded). That's the exact step that used to fail on the per-file overrides.

Why so many other jobs run (and fail)

This PR adds sdk/<service>/cspell.yaml across ~50 services. Per
eng/scripts/Language-Settings.ps1
(the isOther / sdk/<service>/<file> logic), a change to a service-level file pulls every package in that service into validation. So this config-only PR ends up running full CI (tests, docs, api-stub, min-dependency) for ~130 packages across those ~50 services. No other current PR triggers this breadth, which is why the failure count looks large.

Every failure is pre-existing and environmental — none caused by this change

All failures are in packages this PR does not modify, and reproduce independently of it:

Failing check	Root cause	Proof it's pre-existing
autorest - pr	pnpm 9.5.0 npm-registry signature can't be verified	Also failing on unrelated PRs #48286, #48202 (×5)
cosmos - ci EmulatorTests	pypi network WinError 10013 + Python 3.14	Identical job failures on main build 6620924 (no PR)
pullrequest ci_test_base_*	Python 3.14 removed the cgi stdlib module → old azure-core under .venv_mindependency fails to import; plus test-proxy playback recording drift (e.g. azure-ai-language-conversations-authoring)	cgi removal is deterministic on 3.14; the same playback failures were present weeks ago
Build Docs	No module named 'msrest' in the doc-gen env	dependency/env issue in an unrelated package
Build Extended	azure-maps-geolocation apistub: Unexpected keyword: raises	apistub tooling issue in an unrelated package

None of these relate to spelling or to the cspell configuration content. The diff is 100% cspell files (.vscode/cspell.json + per-service cspell.yaml), and the sibling JS PR (Azure/azure-sdk-for-js#39307) is fully green.

Net: the spelling migration is validated (spell-check green, no words lost). The red checks are pre-existing environmental/recording failures in unrelated packages that this PR merely triggers by touching many service directories.

[azure-pipelines]

Azure Pipelines:
Successfully started running 3 pipeline(s).
7 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

[Copilot]

Pull request overview

Migrates spellcheck configuration to imported, service-level YAML files and removes legacy package configs. Several migrated ignore globs have incorrect or broadened scopes and require correction.

Changes:

• Added or normalized per-service cspell.yaml files.
• Consolidated, sorted, and deduplicated service word lists.
• Removed legacy package configs and service overrides from the central config.

Show a summary per file

File	Description
.vscode/cspell.json	Removes service overrides and normalizes global words.
sdk/webpubsub/cspell.yaml	Adds service configuration.
sdk/webpubsub/cspell.json	Removes legacy configuration.
sdk/voicelive/cspell.yaml	Adds consolidated configuration.
sdk/voicelive/azure-ai-voicelive/cspell.json	Removes package configuration.
sdk/vision/cspell.yaml	Adds service configuration.
sdk/videoanalyzer/cspell.yaml	Adds service configuration.
sdk/translation/cspell.yaml	Expands and normalizes service words.
sdk/transcription/cspell.yaml	Adds consolidated configuration.
sdk/transcription/azure-ai-transcription/cspell.json	Removes package configuration.
sdk/textanalytics/cspell.yaml	Adds service configuration.
sdk/tables/cspell.yaml	Adds service configuration.
sdk/synapse/cspell.yaml	Adds service configuration.
sdk/storage/cspell.yaml	Consolidates storage words.
sdk/schemaregistry/cspell.yaml	Adds service configuration.
sdk/remoterendering/cspell.yaml	Adds service configuration.
sdk/purview/cspell.yaml	Adds service configuration.
sdk/postgresql/cspell.yaml	Normalizes existing configuration.
sdk/planetarycomputer/cspell.yaml	Normalizes and expands words.
sdk/personalizer/cspell.yaml	Adds service configuration.
sdk/openai/cspell.yaml	Adds service configuration.
sdk/monitor/cspell.yaml	Adds consolidated configuration.
sdk/ml/cspell.yaml	Adds consolidated ML configuration.
sdk/ml/azure-ai-ml/cspell.json	Removes package configuration.
sdk/mixedreality/cspell.yaml	Adds service configuration.
sdk/metricsadvisor/cspell.yaml	Adds service configuration.
sdk/maps/cspell.yaml	Adds service configuration.
sdk/loadtestservice/cspell.yaml	Adds service configuration.
sdk/loadtesting/cspell.yaml	Adds consolidated configuration.
sdk/loadtesting/azure-developer-loadtesting/cspell.json	Removes package configuration.
sdk/keyvault/cspell.yaml	Adds consolidated configuration.
sdk/keyvault/cspell.json	Removes legacy configuration.
sdk/iothub/cspell.yaml	Adds service configuration.
sdk/identity/cspell.yaml	Adds service configuration.
sdk/healthinsights/cspell.yaml	Adds consolidated configuration.
sdk/healthdataaiservices/cspell.yaml	Adds service configuration.
sdk/formrecognizer/cspell.yaml	Adds service configuration.
sdk/face/cspell.yaml	Adds service configuration.
sdk/eventhub/cspell.yaml	Adds service configuration.
sdk/eventgrid/cspell.yaml	Adds service configuration.
sdk/evaluation/cspell.yaml	Adds consolidated configuration and ignores.
sdk/evaluation/azure-ai-evaluation/cspell.json	Removes package configuration.
sdk/documentintelligence/cspell.yaml	Adds service configuration.
sdk/discovery/cspell.yaml	Adds consolidated configuration and ignores.
sdk/discovery/azure-ai-discovery/cspell.json	Removes package configuration.
sdk/deviceupdate/cspell.yaml	Adds central import and normalizes metadata.
sdk/devcenter/cspell.yaml	Adds service configuration.
sdk/cosmos/cspell.yaml	Adds consolidated configuration.
sdk/cosmos/azure-cosmos/cspell.json	Removes package configuration.
sdk/core/cspell.yaml	Adds service configuration.
sdk/contentunderstanding/cspell.yaml	Adds consolidated configuration and ignores.
sdk/contentunderstanding/azure-ai-contentunderstanding/cspell.json	Removes package configuration.
sdk/contentsafety/cspell.yaml	Adds service configuration.
sdk/containerregistry/cspell.yaml	Adds service configuration.
sdk/confidentialledger/cspell.yaml	Adds service configuration.
sdk/communication/cspell.yaml	Adds consolidated configuration.
sdk/communication/cspell.json	Removes legacy configuration.
sdk/communication/azure-communication-callautomation/cspell.yaml	Removes package configuration.
sdk/cognitiveservices/cspell.yaml	Adds service configuration.
sdk/cognitivelanguage/cspell.yaml	Adds service configuration.
sdk/batch/cspell.yaml	Adds import and sorts words.
sdk/attestation/cspell.yaml	Adds service configuration.
sdk/appconfiguration/cspell.yaml	Adds service configuration.
sdk/ai/cspell.yaml	Adds consolidated AI configuration and ignores.
sdk/ai/azure-ai-projects/cspell.json	Removes package configuration.
sdk/ai/azure-ai-agents/cspell.json	Removes package configuration.
sdk/agrifood/cspell.yaml	Adds service configuration.
sdk/agentserver/cspell.yaml	Consolidates Agent Server configurations.
sdk/agentserver/azure-ai-agentserver-responses/cspell.json	Removes package configuration.
sdk/agentserver/azure-ai-agentserver-optimization/cspell.json	Removes package configuration.
sdk/agentserver/azure-ai-agentserver-invocations/cspell.json	Removes package configuration.
sdk/agentserver/azure-ai-agentserver-ghcopilot/cspell.json	Removes package configuration.
sdk/agentserver/azure-ai-agentserver-core/cspell.json	Removes package configuration.
sdk/agentserver/azure-ai-agentserver-activity/cspell.json	Removes package configuration.

Review details

• Files reviewed: 74/74 changed files
• Comments generated: 5
• Review effort level: Medium

[maorleger]

I'm looking into all of these 👍

[maorleger]

Good catch — addressed in the latest push. Each ignorePaths glob is now scoped to its originating package with a repo-root-relative path (e.g. sdk/ai/azure-ai-projects/**/*.json and sdk/ai/azure-ai-agents/**/*.json instead of a bare *.json). I verified against the real repo that JSON under sibling packages like azure-ai-inference is spell-checked again (as it was on main), and that no file previously ignored is now checked.

[maorleger]

Fixed in the latest push. The bare extension globs and samples/** are now emitted per-package with repo-root-relative paths (e.g. sdk/agentserver/azure-ai-agentserver-core/**/*.json and sdk/agentserver/azure-ai-agentserver-core/samples/**), so they no longer affect the whole service and samples/** correctly targets each package's nested sample dir. Verified the ignored-file set is a superset of the previous behavior (nothing newly exposed).

[maorleger]

I don't think this one holds. The Azure spellcheck tooling (eng/common/spelling/Invoke-Cspell.ps1) runs cspell with --root <repo root>, and cspell resolves ignorePaths relative to that root — not to the config file's directory. So sdk/discovery/azure-ai-discovery/**/generated_samples/** is not double-prefixed to sdk/discovery/sdk/discovery/...; it matches the real path. I verified against the actual repo: sdk/discovery/azure-ai-discovery/.../generated_samples/... reports skipped (Excluded by ignorePaths.) identically on this branch and on main. No change needed.

[maorleger]

Same as the discovery thread — cspell resolves ignorePaths relative to --root (the repo root passed by Invoke-Cspell.ps1), not the config directory, so the sdk/evaluation/... prefix is correct rather than duplicated. Verified on the real repo: .../simulator/_data_sources/grounding.json and the other listed paths are excluded identically on this branch and on main. (Confirmed they are NOT excluded by the central config alone, so the service-level ignore is doing the work.) No change needed.

[maorleger]

cspell resolves ignorePaths relative to --root (repo root), not the config file location, so sdk/contentunderstanding/... is not duplicated. Verified against the real repo: .../training_samples/*.json files report skipped (Excluded by ignorePaths.) identically on this branch and on main. No change needed.

[Copilot]

Review details

Comments suppressed due to low confidence (2)

sdk/agentserver/cspell.yaml:70

• Moving these package-local exclusions to the service root broadens them to every sdk/agentserver package. For example, azure-ai-agentserver-ghcopilot previously had no ignorePaths, while it now skips CSV/JSON/RST, samples, and Makefiles; optimization likewise newly skips samples. This contradicts the stated coverage-preservation guarantee and can hide future spelling errors. Keep the exclusions package-qualified when consolidating them.

ignorePaths:
  - '*.csv'
  - '*.json'
  - '*.rst'
  - samples/**

sdk/ai/cspell.yaml:131

• These exclusions were package-local before consolidation: JSONL applied only to azure-ai-projects, and CSV/JSON only to Projects and Agents. At the service root they now suppress those file types across every sdk/ai package, reducing spellcheck coverage rather than preserving it. Scope each glob to the package(s) that originally owned it.

ignorePaths:
  - '*.csv'
  - '*.json'
  - '*.jsonl'

• Files reviewed: 74/74 changed files
• Comments generated: 0 new
• Review effort level: Medium

[Copilot]

Review details

• Files reviewed: 74/74 changed files
• Comments generated: 1
• Review effort level: Medium

[github-actions]

[Pilot] PR Pipeline Failure Analysis

A CI pipeline failed on this pull request. Here is an automated analysis of what went wrong and how to get the build green.

What failed

Three failures were reported across two Azure DevOps builds:

1. Build 6623191 — Cosmos Emulator tests, Windows Python 3.10 (test): A large number of azure-cosmos emulator tests failed across many test files (TestFaultInjectionTransportAsync, TestReadItems, TestSession, TestTimeoutRetryPolicy, TestTransactionalBatch, and others — both sync and async variants). Failures for azure-agrifood-farming and azure-ai-language-* were also reported. Because this PR is a cspell config–only change (tagged ***NO_CI***), these failures are very likely pre-existing or intermittent and unrelated to this PR.

2. Build 6623192 — pnpm install step (infrastructure): The build failed because pnpm@9.5.0 could not be verified against its npm registry signature (Refusing to run pnpm@9.5.0: its npm registry signature could not be verified). This is a transient infrastructure/registry issue unrelated to the PR code.

3. Build 6623193 — API Stub check (validation): sdk/maps/azure-maps-geolocation reported apistub FAIL(1) while all other 104 packages passed. This is also likely pre-existing and unrelated to this cspell-config-only PR.

Recommended next steps

• Re-run build 6623192 — the pnpm registry signature error is an infrastructure flake; no code changes needed.
• Confirm Cosmos/AgriFood/Language test failures are pre-existing on main and not introduced by this PR (it only changes cspell.yaml/cspell.json files).
• Investigate azure-maps-geolocation apistub failure in build 6623193 separately; verify whether it also fails on main to confirm it is pre-existing.
• See the CI troubleshooting guide: https://aka.ms/ci-fix
• Push new commits to address failures if any are confirmed to be caused by this PR; this comment updates automatically on the next failing run.

Raw pipeline analysis (azsdk ci analyze)

Analyzing pipeline https://github.com/Azure/azure-sdk-for-python/pull/48094...

--- Failed Tests (build 6623191, cosmos emulator sdist & whl) ---
  tests.test_cosmos_http_logging_policy_async.TestFaultInjectionTransportAsync.*
  tests.test_excluded_locations_emulator_async.TestFaultInjectionTransportAsync.*
  tests.test_fault_injection_transport_async.TestFaultInjectionTransportAsync.*
  tests.test_read_items.TestReadItems.*
  tests.test_read_items_async.TestReadItemsAsync.*
  tests.test_retryable_writes_async.TestFaultInjectionTransportAsync.*
  tests.test_session.TestSession.*
  tests.test_session_async.TestSessionAsync.*
  tests.test_timeout_and_failover_retry_policy.TestTimeoutRetryPolicy.*
  tests.test_transactional_batch.TestTransactionalBatch.*
  tests.test_transactional_batch_async.TestTransactionalBatchAsync.*
  tests.test_ttl.TestTimeToLive.test_collection_and_document_ttl_values
  ... (agrifood & cognitivelanguage tests also failing)

--- Failed Task (build 6623192) ---
  ERROR  Refusing to run pnpm@9.5.0: its npm registry signature could not be verified.
  ##[error]Bash exited with code '1'.

Pipeline: https://dev.azure.com/azure-sdk/public/_build/results?buildId=6623192

--- Failed Task (build 6623193) ---
  /sdk/maps/azure-maps-geolocation  apistub  FAIL(1)  10.99
  Total checks: 105 | Failed: 1 | Worst exit code: 1

Pipeline: https://dev.azure.com/azure-sdk/public/_build/results?buildId=6623193

Copilot detected the failing pipeline and generated the analysis above. To have it attempt a fix automatically, reply with @copilot please fix the failing pipeline on this PR.

Generated by Pipeline Analysis - Next Steps · 48.3 AIC · ⌖ 8.87 AIC · ⊞ 6.6K · ◷