Skip to content

Revert "Add Custom model endpoint provider" - #1431

Merged
Paul Lizer (paullizer) merged 1 commit into
Developmentfrom
revert-1228-paullizer-custom-model-endpoints
Sep 4, 2026
Merged

Revert "Add Custom model endpoint provider"#1431
Paul Lizer (paullizer) merged 1 commit into
Developmentfrom
revert-1228-paullizer-custom-model-endpoints

Conversation

@paullizer

Copy link
Copy Markdown
Contributor

Reverts #1228

@paullizer
Paul Lizer (paullizer) merged commit 4a29f67 into Development Sep 4, 2026
10 checks passed
Paul Lizer (paullizer) added a commit to zarg-dev/simplechat that referenced this pull request Sep 4, 2026
Take Development through the revert of the Custom model endpoint provider
(microsoft#1431 reverting microsoft#1228). Only two files conflicted, both mechanical.

* config.py: kept VERSION 0.261.011. The revert returned Development to
  0.261.009, so this is still ahead and needs no renumber.
* release_notes.md: kept the v0.261.011 DLP section and took Development's side
  of the region below it, so the reverted v0.261.010 Custom model endpoint
  section is not reinstated.
* Regenerated docs/explanation/release-notes/ so the generated pages drop the
  reverted v0.261.010 entry with the source. docs/_data/app_surface.yml was
  regenerated and is unchanged, since the revert did not alter the application
  surface this PR documents.

Re-verified the DLP enforcement paths against the reverted file contents rather
than assuming they carried over, because microsoft#1228 had touched both files:

* save_chunks and save_chunks_batch still evaluate upload DLP before any
  embedding call, and the embedding character clamp still only narrows
  sanitized_chunk_text.
* perform_web_search still has no user_message fallback, so only the
  DLP-approved query can reach the outbound search.
* Video enhanced-citation upload still gates on the caller-supplied
  enable_enhanced_citations flag that upload DLP clears.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant