Skip to content

Fix response type of putExtensibleStringValue in special-words spec#9953

Open
Copilot wants to merge 5 commits intomainfrom
copilot/fix-response-type-case
Open

Fix response type of putExtensibleStringValue in special-words spec#9953
Copilot wants to merge 5 commits intomainfrom
copilot/fix-response-type-case

Conversation

Copy link
Contributor

Copilot AI commented Mar 6, 2026

Follow-up to #9785: the putExtensibleStringValue operation was missing an explicit response content-type header, inconsistent with the pattern used by other extensible enum endpoints.

  • Updated return type in packages/http-specs/specs/special-words/main.tsp:
putExtensibleStringValue(@body body: ExtensibleString): {
    @header contentType: "application/json";
    @body body: ExtensibleString;
};

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • telemetry.astro.build
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node /home/REDACTED/work/typespec/typespec/website/node_modules/.bin/../astro/astro.js build sh s/.b�� ../../website/sr--llmstxt sh _modules/pnpm/dist/node-gyp-bin/node --no-emit git _modules/pnpm/digenerate-scenarios-summary node tobu�� build.json sh /.bin/sh ld.json && pnpm node pnpm tobuf/reference node (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…icit content-type header

Co-authored-by: msyyc <70930885+msyyc@users.noreply.github.com>
@microsoft-github-policy-service microsoft-github-policy-service bot added the lib:http-specs For issues/prs related to the @typespec/http-specs package label Mar 6, 2026
Copilot AI changed the title [WIP] Fix response type of case introduced in PR #9785 Fix response type of putExtensibleStringValue in special-words spec Mar 6, 2026
Copilot finished work on behalf of msyyc March 6, 2026 12:24
@msyyc msyyc marked this pull request as ready for review March 6, 2026 13:00
@azure-sdk
Copy link
Collaborator

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 9, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/http-specs@9953

commit: 71079ae

@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2026

All changed packages have been documented.

  • @typespec/http-specs
Show changes

@typespec/http-specs - fix ✏️

Fix response type of putExtensibleStringValue to include explicit content-type header

@msyyc msyyc enabled auto-merge March 9, 2026 02:12
@msyyc
Copy link
Contributor

msyyc commented Mar 9, 2026

@timotheeguerin Pls help review the fix PR, thanks!

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

Labels

lib:http-specs For issues/prs related to the @typespec/http-specs package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants