Skip to content

fix(http): return diagnostics from resolution - #12574

Closed
Mary Gao (MaryGao) wants to merge 1 commit into
microsoft:mainfrom
MaryGao:fix/http-resolution-diagnostics
Closed

Mary Gao (MaryGao) wants to merge 1 commit into
microsoft:mainfrom
MaryGao:fix/http-resolution-diagnostics

Conversation

@MaryGao

@MaryGao Mary Gao (MaryGao) commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Return structured-file and multiple-status-code diagnostics from HTTP resolution APIs rather than reporting them directly.
  • Preserve compiler validation diagnostics without adding warnings when callers inspect HTTP metadata repeatedly.

Checks

  • pnpm --filter @typespec/http build
  • pnpm --filter @typespec/http test (434 passed)
  • pnpm --filter @typespec/openapi3 exec vitest run test/file.test.ts (90 passed)
  • Targeted formatting, type-aware lint, and strict test-file type-check

The broader tsc --noEmit -p packages/http/tsconfig.json reports existing errors in unchanged test/cache.test.ts and vitest.config.ts; the changed tests type-check in isolation.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@pkg-pr-new

pkg-pr-new Bot commented Nov 13, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/http@12574

commit: a0d7443

@github-actions

Copy link
Copy Markdown
Contributor

All changed packages have been documented.

  • ✅ @typespec/http
Show changes

@typespec/http - fix ✏️

Return structured-file and multiple-status diagnostics from HTTP resolution APIs instead of reporting them directly.

@azure-sdk-automation

Copy link
Copy Markdown

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant