[Repo Assist] improve: surface enum allowed-values in operation parameter XmlDocs (+3 tests, 324→327)

[github-actions]

🤖 This PR was created by Repo Assist, an automated AI assistant.

Summary

OpenAPI parameters with enum values (query, path, header, cookie) now include an "Allowed values: ..." hint in their IntelliSense <param> tag, consistent with what already happens for object properties.

Before

/// <summary>List items</summary>
/// <param name="status">Filter by status</param>
```

### After

```
/// <summary>List items</summary>
/// <param name="status">Filter by status
/// Allowed values: active, inactive, pending</param>
```

## Changes

| File | Change |
|---|---|
| `Utils.fs` | Extract `XmlDoc.buildEnumDoc` helper — formats a `JsonNode` enum list into an "Allowed values: ..." string |
| `DefinitionCompiler.fs` | Replace inline `formatEnumValue` / `enumValuesDoc` with a call to `XmlDoc.buildEnumDoc` (no behaviour change for property docs) |
| `OperationCompiler.fs` | Add `buildParamDesc` in the `xmlDoc` block — appends enum allowed-values to each parameter description |
| `Schema.XmlDocTests.fs` | Refactor helpers (`parseSchema`, `getXmlDocAttr`, `getMethodXmlDoc`); add 3 new tests for the operation-parameter enum doc feature |

## Test Status

✅ **327 / 327 tests pass** (up from 324 — 3 new tests added)

```
SwaggerProvider.Tests  Total: 327, Errors: 0, Failed: 0, Skipped: 1, Not Run: 0, Time: 0.923s

Format check: ✅ fantomas --check passes on all changed files.

Generated by 🌈 Repo Assist, see workflow run. Learn more.

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@96b9d4c39aa22359c0b38265927eadb31dcf4e2a

[Copilot]

Pull request overview

Surfaces OpenAPI enum allowed-values as an “Allowed values: …” hint in generated IntelliSense XML docs for operation parameters, aligning with existing behavior for schema properties.

Changes:

• Extracted a shared XmlDoc.buildEnumDoc helper to format enum value lists.
• Reused XmlDoc.buildEnumDoc for schema property XmlDocs (no intended behavior change).
• Appended enum allowed-values to operation parameter descriptions in OperationCompiler, with 3 new tests covering the new behavior.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
tests/SwaggerProvider.Tests/Schema.XmlDocTests.fs	Refactors schema parsing / XmlDoc extraction helpers and adds tests validating enum hints in method <param> docs.
src/SwaggerProvider.DesignTime/Utils.fs	Introduces XmlDoc.buildEnumDoc to format enum values consistently across compilers.
src/SwaggerProvider.DesignTime/OperationCompiler.fs	Adds enum allowed-values into operation parameter XmlDoc content via buildParamDesc.
src/SwaggerProvider.DesignTime/DefinitionCompiler.fs	Replaces inline enum doc formatting with the shared XmlDoc.buildEnumDoc helper.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.