Skip to content

Conversation

@pawbana
Copy link
Contributor

@pawbana pawbana commented Dec 23, 2025

Refactor: Reorganize code into packages

Adds following packages

  • config: contains config structs and some constants
  • context: actor related context functions
  • intercept:
    • chatcompletions: OpenAI chat completions interceptors
    • eventstream: common even stream logic beween chatcompletions/messages packages
    • messages: Anthrpic messages interceptors
  • provider: OpenAI/Antropic providers
  • recorder

The main API surface is preserved through type aliases and function wrappers in api.go, ensuring backward compatibility while improving the internal code organization.

Copy link
Contributor Author

pawbana commented Dec 23, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@pawbana pawbana force-pushed the pb/package-refactor branch from 517e67a to 588fd7d Compare December 23, 2025 23:05
Adds following packages
- config: contains config structs and some constants
- context: actor related context functions
- intercept:
  - chatcompletions: OpenAI chat completions interceptors
  - eventstream
  - messages: Anthrpic messages interceptors
- provider: OpenAI/Antropic providers
- recorder
@pawbana pawbana force-pushed the pb/package-refactor branch from 588fd7d to 2fc2413 Compare December 29, 2025 12:17
Copy link
Collaborator

@dannykopping dannykopping left a comment

Choose a reason for hiding this comment

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

Let's merge this as-is for now. There are a few things I'm not 100% in agreement with (from a taste perspective) but I think this does move us forward, and doesn't tie our hands.

Thank you for doing this @pawbana

@pawbana pawbana marked this pull request as ready for review December 29, 2025 14:07
@dannykopping dannykopping merged commit fab3b9d into main Dec 29, 2025
2 checks passed
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.

2 participants