Skip to content

Conversation

@steven10a
Copy link
Collaborator

@steven10asteven10a commented Oct 9, 2025

  • Added safety header oaig --> oai_guardrails
  • Added testing around safety header
  • Other changes are Ruff formatting

Copy link

CopilotAI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a safety header oaig to OpenAI clients used by guardrails and includes various Ruff-based formatting changes. The safety header is automatically added to identify Guardrails usage, and the PR includes comprehensive testing for the new functionality along with code style improvements throughout the codebase.

  • Introduces a safety identifier header system for OpenAI client identification
  • Adds comprehensive test coverage for the new safety header functionality
  • Applies Ruff formatting improvements to enhance code style consistency

Reviewed Changes

Copilot reviewed 53 out of 53 changed files in this pull request and generated no comments.

FileDescription
src/guardrails/_openai_utils.pyNew utility module for safety header management
tests/unit/test_openai_utils.pyTest coverage for OpenAI utilities functionality
src/guardrails/client.pyIntegration of safety header into client initialization
Multiple filesRuff formatting improvements across codebase

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

CopilotAI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 55 out of 55 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Collaborator

@gabor-openaigabor-openai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TY

@gabor-openaigabor-openai merged commit 293b1ae into mainOct 9, 2025
3 checks passed
@gabor-openaigabor-openai deleted the dev/steven/safety_header branch October 9, 2025 21:07
Sign up for freeto 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.

3 participants

@steven10a@gabor-openai