Escape dynamic values in the customizations index - #327475
Conversation
|
Hi Sriraj (@sricursion), thank you for opening a PR! Could we please scope this down to just escaping the input text fields (ie., wrapping |
|
Thanks for the guidance Paul (@pwang347) . I’ve scoped the PR down accordingly. It now only escapes I removed the format-marker, decoder, parser, cache, authorization, and file-path changes. Focused regression tests cover structural-tag injection through the retained text fields, and the typechecks, compilation, lint, focused tests, and full Copilot unit suite pass. The PR description has also been updated to reflect the narrower change. |
Paul (pwang347)
left a comment
There was a problem hiding this comment.
Looks good to me. Thank you for contributing!
Fixes #327467
What changed
name,description,applyTo, andargumentHint.AGENTS.mddescriptions and the truncated skill-name fallback.Validation
npm run typecheck-clientnpm run transpile-clientcomputeAutomaticInstructions.test.ts: 64 passingnpm run eslintnpm run valid-layers-checknpm run define-class-fields-checknpm run typechecknpm run lintnpm run compileautomaticInstructionsCollector.spec.ts: 32 passingnpm run test:unit: 9,553 passing, 129 skippedgit diff --check