Skip to content

Conversation

@steven10a
Copy link
Collaborator

@steven10a steven10a commented Oct 9, 2025

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

Copy link

Copilot AI 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.

File Description
src/guardrails/_openai_utils.py New utility module for safety header management
tests/unit/test_openai_utils.py Test coverage for OpenAI utilities functionality
src/guardrails/client.py Integration of safety header into client initialization
Multiple files Ruff formatting improvements across codebase

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

@gabor-openai gabor-openai requested a review from Copilot October 9, 2025 20:51
Copy link

Copilot AI 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-openai gabor-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-openai gabor-openai merged commit 293b1ae into main Oct 9, 2025
3 checks passed
@gabor-openai gabor-openai deleted the dev/steven/safety_header branch October 9, 2025 21:07
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.

3 participants