Commit 9dee1d4
privapps
Squashed commit of the following:
Date: Fri Aug 8 22:38:37 2025 -0700
Update Go version to 1.23, enhance Makefile for multi-OS builds, and improve authentication handling
- Updated Go version in CI and release workflows
- Added Makefile targets for building binaries for different OS/architectures
- Enhanced AuthService to support configurable token refresh and path for tests
- Updated ProxyService to validate tokens and handle errors more effectively
- Improved error handling in API requests and tests for better coverage
Date: Fri Aug 8 15:44:53 2025 -0700
Remove Prometheus metrics integration from proxy and circuit breaker
Eliminate Prometheus client dependencies and related metric registration
from proxy and circuit breaker code. Update documentation to reflect
removal of worker pool metrics and production monitoring. This simplifies
the codebase and reduces external dependencies.
Date: Fri Aug 8 15:37:14 2025 -0700
Add unit tests for ProxyService and Server components
- Implement comprehensive tests for ProxyService including handler, caching, circuit breaker, retry logic, streaming responses, error handling, and concurrent requests.
- Introduce tests for Server creation, configuration, worker pool functionality, HTTP client timeout handling, and memory management.
- Ensure proper handling of various request scenarios and validate server routes and concurrency.
- Utilize mock servers and helper functions to simulate and validate expected behaviors.
Date: Fri Aug 8 12:34:29 2025 -0700
Implement HTTP server with graceful shutdown and worker pool
- Added internal server implementation with HTTP server and worker pool for handling requests.
- Introduced new request/response structures for OpenAI compatibility in transform package.
- Updated integration tests to validate API endpoints, ensuring server is running before tests.
- Refactored test utilities to support new server structure and configuration.
- Created unit tests for authentication, configuration loading, and logger initialization.
- Enhanced error handling and logging throughout the application.
Date: Thu Aug 7 12:15:25 2025 -0700
Refactor: Remove existing tests and server implementation
- Deleted proxy_test.go, server.go, and server_test.go files to clean up the codebase.
- Added valid_config.json and models_response.json fixtures for testing.
- Introduced integration tests for API endpoints in api_test.go.
- Created helper functions in testutils for configuration and server mocking.
- Implemented unit tests for authentication, configuration, and logging functionalities.
Date: Thu Aug 7 01:32:52 2025 -0700
Refactor CI/CD workflows and improve code quality
- Updated GitHub Actions workflows to use the latest versions of actions.
- Enhanced security checks by replacing Gosec with Go vet and go mod verify.
- Removed the Create Release step and replaced it with softprops/action-gh-release for better asset management.
- Improved error handling in authentication and token management functions.
- Refactored timeout constants and validation logic in the configuration.
- Enhanced test coverage and error handling in various test cases.
- Improved logging and graceful shutdown handling in the server.
- Updated HTTP response handling to use http.NoBody for clarity.
Date: Thu Aug 7 00:34:29 2025 -0700
Merge branch 'main' of https://github.com/privapps/github-copilot-svcs into dev
Date: Thu Aug 7 00:34:16 2025 -0700
Add comprehensive tests for logger, main functionality, models, and proxy handling
- Introduced logger tests to validate logging levels and initialization.
- Enhanced main function tests to cover command execution and environment variable handling.
- Added tests for model fetching and validation, including error scenarios.
- Implemented proxy request handling tests to ensure proper request processing and error management.
- Established server tests for health checks and graceful shutdowns.
- Refactored existing code to improve testability and maintainability.
Date: Wed Aug 6 12:13:00 2025 -0700
Add support for customizable HTTP headers and cross-platform builds
- Introduced new build targets for Linux, macOS, and Windows in Makefile.
- Updated README with build instructions for different platforms.
- Enhanced configuration to include customizable HTTP headers.
- Refactored authentication and API request functions to utilize new header configuration.
- Set default header values in config loading.1 parent 85e5217 commit 9dee1d4
File tree
42 files changed
+6668
-1708
lines changed- .github/workflows
- cmd/github-copilot-svcs
- internal
- pkg
- transform
- test
- fixtures
- config
- responses
- integration
- testutils
- unit
- auth
- config
- logger
- models
- proxy
- server
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
42 files changed
+6668
-1708
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| 10 | + | |
10 | 11 | | |
11 | 12 | | |
12 | 13 | | |
13 | 14 | | |
14 | 15 | | |
15 | 16 | | |
16 | | - | |
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
| |||
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
26 | | - | |
| 26 | + | |
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
| |||
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
62 | | - | |
63 | | - | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | | - | |
69 | | - | |
70 | | - | |
71 | | - | |
72 | | - | |
73 | | - | |
74 | | - | |
75 | | - | |
76 | | - | |
77 | | - | |
78 | | - | |
79 | | - | |
80 | | - | |
81 | | - | |
82 | | - | |
83 | | - | |
84 | | - | |
85 | 62 | | |
86 | 63 | | |
87 | 64 | | |
| |||
114 | 91 | | |
115 | 92 | | |
116 | 93 | | |
117 | | - | |
| 94 | + | |
118 | 95 | | |
119 | 96 | | |
120 | 97 | | |
| |||
123 | 100 | | |
124 | 101 | | |
125 | 102 | | |
126 | | - | |
| 103 | + | |
127 | 104 | | |
128 | 105 | | |
129 | 106 | | |
| |||
136 | 113 | | |
137 | 114 | | |
138 | 115 | | |
139 | | - | |
140 | | - | |
141 | | - | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
142 | 163 | | |
143 | | - | |
144 | | - | |
145 | | - | |
146 | | - | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
0 commit comments