Skip to content

Conversation

@GeekInTheNorth
Copy link
Owner

Closes #89

Copy link
Contributor

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

GeekInTheNorth and others added 2 commits September 15, 2025 14:19
Copy link
Contributor

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

@GeekInTheNorth GeekInTheNorth merged commit 04c5f1a into develop Sep 15, 2025
2 checks passed
@GeekInTheNorth GeekInTheNorth deleted the feature/opal-tools branch September 15, 2025 22:32
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