[Akhil Gupta (akhil9tiet)]

Fixes #12473

param.wire_name is spliced raw into a single-quoted Python literal in serialize_query_header, so a quote in the wire name closes the literal early and the remainder becomes a live Python expression in the generated module (observed: _params["x" + str(1024) + ""], URL carries computed x1024) or a SyntaxError at import; the fix routes all wire-name interpolations through the existing python_string_literal helper, with regression tests for quotes, backslashes and control characters.

[azure-pipelines]

Azure Pipelines:
Successfully started running 1 pipeline(s).
1 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]

🟡 Changes recommended

The delimiter change breaks an existing exact-output unit test, and the user-visible fix is missing the required package changelog entry.

2 open issues

• Preserve single-quoted output for optional content-type keys · New
• Add a changelog entry for the Python generator fix · New

What changed in this PR

This PR addresses #12473 by escaping wire names in Python request builders, preventing specification strings from becoming executable expressions or invalid Python syntax.

Changes:

• Uses python_string_literal for path keys, query/header keys, and keyword-argument lookups.
• Adds regression tests verifying that unusual wire names round-trip as literal data.

File	Description
packages/​http-client-python/​tests/​unit/​test_string_literals.py	Adds literal round-trip tests for query, header, path, and keyword-argument wire names.
packages/​http-client-python/​generator/​pygen/​codegen/​serializers/​parameter_serializer.py	Escapes wire names when generating request-builder literals.

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

The user-visible Python generator fix lacks the required Chronus changelog entry for @typespec/http-client-python, so it would be omitted from the package’s release notes.

1 open issue

• Add a changelog entry for the Python generator fix

1 resolved since last review

• Preserve single-quoted output for optional content-type keys

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

No active review issues are recorded for the Python request-builder wire-name escaping change.

No open issues 🎉

1 resolved since last review

• Add a changelog entry for the Python generator fix

_{Review effort: Balanced}

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/http-client-python@12504

commit: bf71080

[github-actions]

All changed packages have been documented.

• ✅ @typespec/http-client-python

Show changes

@typespec/http-client-python - fix ✏️

Fix wire names with quotes in generated request-builder code by escaping them through the shared string-literal helper, preserving the existing single-quoted output for popped header/query kwargs.

[github-actions]

Python emitter diff

Baseline gh:bb8babf5ca59c9881475e59cdc04c6b288529c4a vs this PR.

No changes to generated output.

Rendered diff: inline on the run summary, or the emitter-diff-html artifact.

Informational check (eng/emitter-diff); does not block the PR.

[Copilot]

🟢 Approval recommended

There are no active review issues for the Python request-builder wire-name escaping change.

No open issues 🎉

_{Review effort: Balanced}