microsoft / microsoft/vs-solutionpersistence
Name validation is stricter than VS and MSBuild
@richardstanton is already working on this.
Since Jun 4, 2026.
- Dominant language
- C#
- Stars
- 213
- Forks
- 14
- PR merge metrics
- No merged PRs in 30d
Description
Parsing a valid .sln file with a solution folder named CI/CD throws SolutionException because ValidateName rejects the / character. More broadly, the library rejects / ? : * " < > | and DOS reserved names (CON, AUX, LPT4, etc.) in folder names, project names, and configuration names — even though these are filesystem concerns that don't apply to display names stored in .sln files.
How other parsers handle this
We surveyed two other .sln parsers to understand established behavior:
VS's original parser (src/env/vscore/package/Solutions/SolutionPersistence.cs):
- Display names: no character validation at all during read. On write, project names are truncated at the first
\r,\n, or\t— that's the only sanitization. - No reserved name checks (CON, AUX, etc.).
- Solution folder names are treated as plain display strings with no special validation.
MSBuild's parser (src/Build/Construction/Solution/SolutionFile.cs):
- Display names: zero character validation.
ParseFirstProjectLineregex-extracts the name and assigns it directly. Empty names get a synthesized placeholder; that's the only special handling. - Project relative paths: validated against
Path.GetInvalidPathChars()(control characters only), notGetInvalidFileNameChars(). - Configuration names: only structural validation —
BuildType|Platformmust split into exactly two parts on|. No character validation on the individual parts. - When names need to become MSBuild XML identifiers,
MakeIntoSafeItemNameandCleanseProjectNametransform invalid characters to_rather than rejecting them. - No reserved name checks.
This library (Microsoft.VisualStudio.SolutionPersistence):
ValidateName(used for solution folder names only — viaCreateFolder,AddFolder, and theNamesetter) rejects/ ? : \ * " < > |, control characters, DOS reserved names,.,.., and names over 260 characters.- The same validation is shared for configuration names (
AddBuildType,AddPlatform). - Project display names are not validated by
ValidateName— they only get duplicate-name checking.
Summary
| Concern | VS original | MSBuild | This library |
|---|---|---|---|
/ ? : * " < > | in display names |
Allowed | Allowed | Rejected |
\ (backslash) in display names |
Allowed | Allowed | Rejected |
| DOS reserved names (CON, AUX, etc.) | Allowed | Allowed | Rejected |
. and .. as names |
Allowed | Allowed | Rejected |
| Control characters | Stripped on write | Rejected in paths only | Rejected |
| Configuration name characters | No validation | Only | split must yield 2 parts |
Rejected: / ? : \ * " < > | |
The key principle in both VS and MSBuild is that display names are arbitrary strings, not filesystem paths. Filesystem-invalid characters and reserved names are only relevant when a name maps to something on disk (like a project's relative path), not for logical names like solution folders or build configurations.
Considerations
ValidateName is only called for solution folder names — the one type of name that is purely virtual and never touches the filesystem. Ironically, project display names (which do derive from filesystem paths) skip this validation entirely.
The \ restriction is the one character that has a structural justification in this library's model, since the folder path format (/Folder/Subfolder/) normalizes backslashes. The / restriction is also structurally motivated for configuration names, since it's the folder-path segment delimiter, and | is the BuildType|Platform separator.
The remaining restrictions (? : * " < >, DOS reserved names, ./..) have no structural justification in the model — they are carried over from filesystem naming rules that don't apply to display names.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.