-
Notifications
You must be signed in to change notification settings - Fork 3
Add support for Opal Tools and Headless solutions #95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…rovided by the services.
There was a problem hiding this 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 support for Optimizely Opal Tools and headless solutions by introducing comprehensive API endpoints and token-based authentication system. The implementation allows external tools to manage robots.txt and llms.txt configurations through secure bearer token authentication.
Key changes include:
- Implementation of Opal-compatible API endpoints for robots.txt and llms.txt management
- Token-based authentication system with configurable read/write permissions
- Enhanced UI with token management interface and API documentation
- Discovery endpoint for API function documentation
Reviewed Changes
Copilot reviewed 42 out of 51 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| Opal/* | Complete Opal API implementation including controllers, models, and authentication |
| TokenManagement.jsx | New React component for managing API tokens with scope-based permissions |
| NavigationContainer.jsx | Updated to include token management navigation |
| OpalTokenRepository.cs | Repository pattern implementation for token persistence |
| CSS files | Updated styling for API documentation display |
| Sample project | Updated dependencies and authorization configuration |
| HTTP test files | Comprehensive API testing suite for all endpoints |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
src/Stott.Optimizely.RobotsHandler/Opal/OpalLlmsApiController.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this 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 44 out of 53 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
src/Stott.Optimizely.RobotsHandler/Opal/OpalLlmsApiController.cs:1
- The log message verification in LLMS API controller test references 'robot-txt-configurations' instead of 'llms-txt-configurations'. This should match the actual log message from line 63 in OpalLlmsApiController.cs which logs 'llms-txt-configurations tool'.
using System;
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Closes #89