- 
                Notifications
    You must be signed in to change notification settings 
- Fork 8
Create static GitHub Copilot instructions file and restore environment setup workflow #771
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
          
     Merged
      
      
            BenjaminMichaelis
  merged 11 commits into
  main
from
copilot/fix-fde5fa8c-b82a-4f64-b892-775e442dc677
  
      
      
   
  Aug 29, 2025 
      
    
      
        
          +329
        
        
          −0
        
        
          
        
      
    
  
  
     Merged
                    Changes from 9 commits
      Commits
    
    
            Show all changes
          
          
            11 commits
          
        
        Select commit
          Hold shift + click to select a range
      
      8396b17
              
                Initial plan
              
              
                Copilot 5376714
              
                Add initial copilot-setup-steps.yml workflow
              
              
                Copilot f973a70
              
                Enhance copilot-setup-steps.yml with comprehensive tooling and instru…
              
              
                Copilot 79bdbee
              
                Finalize copilot-setup-steps.yml - remove unnecessary Node.js caching
              
              
                Copilot 7ceea8c
              
                Simplify copilot-setup-steps.yml to generate instructions file only
              
              
                Copilot 0427302
              
                Create static copilot-instructions.md and restore environment setup w…
              
              
                Copilot c18fe77
              
                Update .github/workflows/copilot-setup-steps.yml
              
              
                BenjaminMichaelis 5cf396c
              
                Update .github/workflows/copilot-setup-steps.yml
              
              
                BenjaminMichaelis 11ef8df
              
                Update .github/workflows/copilot-setup-steps.yml
              
              
                BenjaminMichaelis 44a1433
              
                Update .github/workflows/copilot-setup-steps.yml
              
              
                BenjaminMichaelis 9cc8d23
              
                update
              
              
                BenjaminMichaelis File filter
Filter by extension
Conversations
          Failed to load comments.   
        
        
          
      Loading
        
  Jump to
        
          Jump to file
        
      
      
          Failed to load files.   
        
        
          
      Loading
        
  Diff view
Diff view
There are no files selected for viewing
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              | Original file line number | Diff line number | Diff line change | 
|---|---|---|
| @@ -0,0 +1,265 @@ | ||
| # GitHub Copilot Instructions for EssentialCSharp.Web | ||
|  | ||
| ## Project Overview & Core Purpose | ||
| This is a comprehensive ASP.NET Core 9.0 web application ecosystem for the **Essential C#** programming education platform. The project serves as the technical foundation for [essentialcsharp.com](https://essentialcsharp.com/), providing educational content, AI-powered chat assistance, and user management for C# learning resources. | ||
|  | ||
| **Key Value**: Provides an interactive learning platform where developers can access Essential C# content, engage with AI-powered assistance for C# questions, and track their learning progress through a modern web interface. | ||
|  | ||
| **Target Audience**: C# developers at all levels, students learning C#, and educators teaching C# programming concepts. | ||
|  | ||
| ## Project Structure & Architecture | ||
| This solution follows a modular architecture with clear separation of concerns: | ||
|  | ||
| ``` | ||
| EssentialCSharp.Web.sln | ||
| ├── EssentialCSharp.Web/ # Main ASP.NET Core web application | ||
| │ ├── Areas/ # Identity and feature-specific areas | ||
| │ ├── Controllers/ # MVC controllers | ||
| │ ├── Views/ # Razor views and layouts | ||
| │ ├── Models/ # View models and data models | ||
| │ ├── Services/ # Business logic services | ||
| │ ├── Migrations/ # Entity Framework migrations | ||
| │ └── wwwroot/ # Static assets (CSS, JS, images) | ||
| ├── EssentialCSharp.Chat/ # Console application for AI chat | ||
| ├── EssentialCSharp.Chat.Shared/ # Shared models and services | ||
| ├── EssentialCSharp.Web.Tests/ # Integration tests for web app | ||
| └── EssentialCSharp.Chat.Tests/ # Unit tests for chat functionality | ||
| ``` | ||
|  | ||
| ## Tech Stack & Core Technologies | ||
|  | ||
| ### Framework & Runtime | ||
| - **.NET 9.0** with **C# 13** language features | ||
| - **ASP.NET Core 9.0** for web application framework | ||
| - **Entity Framework Core 8.0.10** for data access with SQL Server | ||
| - **ASP.NET Core Identity** for user authentication and authorization | ||
|  | ||
| ### AI & Chat Integration | ||
| - **Microsoft Semantic Kernel 1.60.0** for AI orchestration | ||
| - **Azure OpenAI** integration for chat functionality | ||
| - **pgvector with PostgreSQL** for vector database operations | ||
| - **Model Context Protocol (MCP)** for AI agent integration | ||
|  | ||
| ### Authentication & Security | ||
| - **GitHub OAuth** integration for developer authentication | ||
| - **Microsoft Account** authentication support | ||
| - **HCaptcha** for bot protection | ||
| - **JWT tokens** for API authentication | ||
|  | ||
| ### Development & Deployment | ||
| - **Docker** containerization with multi-stage builds | ||
| - **Azure Application Insights** for telemetry and monitoring | ||
| - **Azure Monitor OpenTelemetry** for observability | ||
| - **Mailjet API** for email services | ||
|  | ||
| ### Package Management & Build | ||
| - **Central Package Management** via `Directory.Packages.props` | ||
| - **Private NuGet feed** from Azure DevOps for internal packages | ||
| - **Source Link** for debugging support | ||
| - **Build versioning** with continuous integration support | ||
|  | ||
| ## Coding Conventions & Development Patterns | ||
|  | ||
| ### Project Structure Conventions | ||
| - **Areas**: Use for Identity and distinct feature modules | ||
| - **Controllers**: Follow MVC pattern with async actions | ||
| - **Services**: Implement business logic with dependency injection | ||
| - **Models**: Separate view models from domain models | ||
| - **Extensions**: Place extension methods in dedicated Extension classes | ||
|  | ||
| ### Code Style & Patterns | ||
| - **Async/Await**: Use consistently for all I/O operations | ||
| - **Dependency Injection**: Register services in `Program.cs` | ||
| - **Repository Pattern**: Implement for data access abstraction | ||
| - **Error Handling**: Use structured logging and custom exceptions | ||
| - **Configuration**: Use strongly-typed configuration classes | ||
|  | ||
| ### File Naming & Organization | ||
| - **Controllers**: `{Feature}Controller.cs` (e.g., `HomeController.cs`) | ||
| - **Services**: `{Feature}Service.cs` and `I{Feature}Service.cs` | ||
| - **Models**: `{Entity}Model.cs` for view models, `{Entity}.cs` for domain | ||
| - **Extensions**: `{Type}Extensions.cs` | ||
| - **Tests**: `{ClassUnderTest}Tests.cs` | ||
|  | ||
| ## Testing Strategy & Frameworks | ||
|  | ||
| ### Test Organization | ||
| - **Integration Tests**: `EssentialCSharp.Web.Tests` using `Microsoft.AspNetCore.Mvc.Testing` | ||
| - **Unit Tests**: `EssentialCSharp.Chat.Tests` using `xUnit` and `Moq` | ||
| - **Test Structure**: Follow AAA pattern (Arrange, Act, Assert) | ||
|  | ||
| ### Testing Tools | ||
| - **xUnit 2.9.3** as the primary testing framework | ||
| - **Moq 4.20.72** for mocking dependencies | ||
| - **Coverlet** for code coverage collection | ||
| - **Microsoft.AspNetCore.Mvc.Testing** for integration testing | ||
|  | ||
| ### Test Conventions | ||
| ```csharp | ||
| [Fact] | ||
| public async Task MethodName_Scenario_ExpectedBehavior() | ||
| { | ||
| // Arrange | ||
| var service = new TestService(); | ||
|  | ||
| // Act | ||
| var result = await service.MethodAsync(); | ||
|  | ||
| // Assert | ||
| Assert.NotNull(result); | ||
| } | ||
| ``` | ||
|  | ||
| ## Build & Development Commands | ||
|  | ||
| ### Essential Commands | ||
| ```bash | ||
| # Restore all packages | ||
| dotnet restore | ||
|  | ||
| # Build entire solution | ||
| dotnet build --configuration Release --no-restore | ||
|  | ||
| # Run all tests | ||
| dotnet test --no-build --configuration Release | ||
|  | ||
| # Run web application | ||
| dotnet run --project EssentialCSharp.Web | ||
|  | ||
| # Run chat application | ||
| dotnet run --project EssentialCSharp.Chat | ||
|  | ||
| # Entity Framework operations | ||
| dotnet ef migrations add MigrationName --project EssentialCSharp.Web | ||
| dotnet ef database update --project EssentialCSharp.Web | ||
| ``` | ||
|  | ||
| ### Docker Operations | ||
| ```bash | ||
| # Build Docker image | ||
| docker build -t essentialcsharp-web -f EssentialCSharp.Web/Dockerfile . | ||
|  | ||
| # Run with Docker Compose (if available) | ||
| docker-compose up --build | ||
| ``` | ||
|  | ||
| ## Configuration & Environment Setup | ||
|  | ||
| ### Required Secrets (Use dotnet user-secrets) | ||
| ```bash | ||
| # Email configuration | ||
| dotnet user-secrets set "AuthMessageSender:SendFromName" "Essential C# Team" | ||
| dotnet user-secrets set "AuthMessageSender:SendFromEmail" "[email protected]" | ||
| dotnet user-secrets set "AuthMessageSender:SecretKey" "your-mailjet-secret" | ||
| dotnet user-secrets set "AuthMessageSender:APIKey" "your-mailjet-api-key" | ||
|  | ||
| # OAuth providers | ||
| dotnet user-secrets set "Authentication:Microsoft:ClientSecret" "microsoft-oauth-secret" | ||
| dotnet user-secrets set "Authentication:Microsoft:ClientId" "microsoft-oauth-client-id" | ||
| dotnet user-secrets set "Authentication:github:clientSecret" "github-oauth-secret" | ||
| dotnet user-secrets set "Authentication:github:clientId" "github-oauth-client-id" | ||
|  | ||
| # Security | ||
| dotnet user-secrets set "HCaptcha:SiteKey" "hcaptcha-site-key" | ||
| dotnet user-secrets set "HCaptcha:SecretKey" "hcaptcha-secret-key" | ||
|  | ||
| # Application Insights | ||
| dotnet user-secrets set "APPLICATIONINSIGHTS_CONNECTION_STRING" "your-app-insights-connection" | ||
| ``` | ||
|  | ||
| ### Package Feed Configuration | ||
| - Set `<AccessToNugetFeed>false</AccessToNugetFeed>` in `Directory.Packages.props` if you don't have access to private Azure DevOps feed | ||
| - Private feed contains `EssentialCSharp.Shared.Models` and content packages | ||
|  | ||
| ## AI Integration Patterns | ||
|  | ||
| ### Semantic Kernel Usage | ||
| - **Chat Services**: Implement AI chat functionality using Semantic Kernel | ||
| - **Vector Operations**: Use pgvector for semantic search and retrieval | ||
| - **Model Context Protocol**: Integrate with MCP for agent communication | ||
| - **Prompt Engineering**: Store prompts as structured templates | ||
|  | ||
| ### AI Service Patterns | ||
| ```csharp | ||
| public interface IChatService | ||
| { | ||
| Task<ChatResponse> ProcessMessageAsync(string message, CancellationToken cancellationToken); | ||
| Task<IEnumerable<SearchResult>> SearchContentAsync(string query, CancellationToken cancellationToken); | ||
| } | ||
| ``` | ||
|  | ||
| ## Development Workflow & Best Practices | ||
|  | ||
| ### Code Review Guidelines | ||
| - Ensure all new code includes appropriate tests | ||
| - Follow established naming conventions and patterns | ||
| - Use async/await for all I/O operations | ||
| - Implement proper error handling and logging | ||
| - Add XML documentation for public APIs | ||
|  | ||
| ### Performance Considerations | ||
| - Use Entity Framework efficiently (avoid N+1 queries) | ||
| - Implement caching where appropriate | ||
| - Use async patterns for database and API calls | ||
| - Optimize Docker image size with multi-stage builds | ||
|  | ||
| ### Security Best Practices | ||
| - Never commit secrets to source control | ||
| - Use HTTPS for all external communications | ||
| - Implement proper input validation | ||
| - Follow OWASP security guidelines | ||
| - Use parameterized queries for database operations | ||
|  | ||
| ## Common Patterns & Utilities | ||
|  | ||
| ### Service Registration Pattern | ||
| ```csharp | ||
| // In Program.cs | ||
| builder.Services.AddScoped<IFeatureService, FeatureService>(); | ||
| builder.Services.AddSingleton<ICacheService, CacheService>(); | ||
| ``` | ||
|  | ||
| ### Configuration Pattern | ||
| ```csharp | ||
| public class FeatureOptions | ||
| { | ||
| public const string SectionName = "Feature"; | ||
| public string ApiKey { get; set; } = string.Empty; | ||
| public int TimeoutSeconds { get; set; } = 30; | ||
| } | ||
|  | ||
| // Registration | ||
| builder.Services.Configure<FeatureOptions>( | ||
| builder.Configuration.GetSection(FeatureOptions.SectionName)); | ||
| ``` | ||
|  | ||
| ### Error Handling Pattern | ||
| ```csharp | ||
| public async Task<Result<T>> MethodAsync<T>() | ||
| { | ||
| try | ||
| { | ||
| var result = await SomeOperationAsync(); | ||
| return Result<T>.Success(result); | ||
| } | ||
| catch (SpecificException ex) | ||
| { | ||
| logger.LogError(ex, "Operation failed"); | ||
| return Result<T>.Failure(ex.Message); | ||
| } | ||
| } | ||
| ``` | ||
|  | ||
| ## Future Roadmap Considerations | ||
| - **Microservices Evolution**: Consider splitting into microservices as features grow | ||
| - **Performance Optimization**: Implement advanced caching and CDN strategies | ||
| - **AI Enhancement**: Expand AI capabilities with more sophisticated models | ||
| - **Mobile Support**: Potential mobile app integration | ||
| - **API Expansion**: RESTful API for third-party integrations | ||
|  | ||
| ## Anti-Patterns & Gotchas | ||
| - **Avoid**: Synchronous calls in async methods (use ConfigureAwait(false)) | ||
| - **Avoid**: Large Entity Framework queries without pagination | ||
| - **Avoid**: Hardcoded configuration values (use appsettings.json) | ||
| - **Avoid**: Missing error handling in async operations | ||
| - **Security**: Never expose sensitive configuration in client-side code | 
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              | Original file line number | Diff line number | Diff line change | 
|---|---|---|
| @@ -0,0 +1,104 @@ | ||
| name: Setup GitHub Copilot Agent Environment | ||
|  | ||
| # This workflow configures the environment for GitHub Copilot agents | ||
| # It reuses configuration from the main Build-Test-And-Deploy workflow | ||
| # to ensure consistency and reduce duplication | ||
|  | ||
| on: | ||
| workflow_dispatch: | ||
|  | ||
| permissions: | ||
| contents: read | ||
|  | ||
| jobs: | ||
| setup-environment: | ||
| runs-on: ubuntu-latest | ||
| environment: "BuildAndUploadImage" | ||
|  | ||
| steps: | ||
| - uses: actions/checkout@v5 | ||
|  | ||
| - name: Set up .NET Core | ||
| uses: actions/setup-dotnet@v4 | ||
| with: | ||
| global-json-file: global.json | ||
| source-url: https://pkgs.dev.azure.com/intelliTect/_packaging/EssentialCSharp/nuget/v3/index.json | ||
| env: | ||
| NUGET_AUTH_TOKEN: ${{ secrets.AZURE_DEVOPS_PAT }} | ||
|  | ||
| - name: Set up dependency caching for faster builds | ||
| uses: actions/cache@v4 | ||
| id: nuget-cache | ||
| with: | ||
| path: | | ||
| ~/.nuget/packages | ||
| ${{ github.workspace }}/**/obj/project.assets.json | ||
| key: ${{ runner.os }}-nuget-${{ hashFiles('**/packages.lock.json') }} | ||
| restore-keys: | | ||
| ${{ runner.os }}-nuget-${{ hashFiles('**/packages.lock.json') }} | ||
| ${{ runner.os }}-nuget- | ||
|  | ||
| - name: Restore with dotnet | ||
| run: dotnet restore | ||
|  | ||
| - name: Build with dotnet | ||
| run: dotnet build -p:ContinuousIntegrationBuild=True -p:ReleaseDateAttribute=True --configuration Release --no-restore | ||
|  | ||
| - name: Run .NET Tests | ||
| run: dotnet test --no-build --configuration Release | ||
|  | ||
| - name: Set up Docker Buildx | ||
| uses: docker/setup-buildx-action@v3 | ||
|  | ||
| - name: Validate Docker build | ||
| uses: docker/build-push-action@v6 | ||
| with: | ||
| push: false | ||
| tags: copilot-env-validation | ||
| file: ./EssentialCSharp.Web/Dockerfile | ||
| context: . | ||
| secrets: | | ||
| "nuget_auth_token=${{ secrets.AZURE_DEVOPS_PAT }}" | ||
| cache-from: type=gha | ||
| cache-to: type=gha,mode=max | ||
|  | ||
| - name: Set up Node.js for frontend development | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: '20' | ||
|  | ||
| - name: Install additional development tools | ||
| run: | | ||
| # Install common development tools that Copilot agents might need | ||
| echo "Installing additional tools for Copilot agent environment..." | ||
|  | ||
| # Install EF Core tools globally | ||
| dotnet tool install --global dotnet-ef | ||
|  | ||
| - name: Validate development tools | ||
| run: | | ||
| echo "🔍 Validating installed tools..." | ||
| echo "EF Core CLI: $(dotnet ef --version || echo 'Not available')" | ||
| echo "Node.js: $(node --version || echo 'Not available')" | ||
| echo "npm: $(npm --version || echo 'Not available')" | ||
|  | ||
|  | ||
|  | ||
|  | ||
| - name: Display environment information | ||
| run: | | ||
| echo "✅ Environment setup complete for GitHub Copilot agents" | ||
| echo "📦 .NET Version: $(dotnet --version)" | ||
| echo "🏗️ Build Status: Success" | ||
| echo "🐳 Docker Status: Ready" | ||
| echo "📂 Repository: ${{ github.repository }}" | ||
| echo "🔧 This environment is configured for:" | ||
| echo " - .NET 9.0 development with C# 13" | ||
| echo " - ASP.NET Core web applications" | ||
| echo " - Entity Framework with SQL Server" | ||
| echo " - Semantic Kernel AI integration" | ||
| echo " - Azure OpenAI and vector databases" | ||
| echo " - Docker containerization" | ||
| echo " - Azure CLI and GitHub CLI tools" | ||
| echo " - NuGet package management (public and private feeds)" | ||
| echo " - Node.js for frontend tooling" | ||
  Add this suggestion to a batch that can be applied as a single commit.
  This suggestion is invalid because no changes were made to the code.
  Suggestions cannot be applied while the pull request is closed.
  Suggestions cannot be applied while viewing a subset of changes.
  Only one suggestion per line can be applied in a batch.
  Add this suggestion to a batch that can be applied as a single commit.
  Applying suggestions on deleted lines is not supported.
  You must change the existing code in this line in order to create a valid suggestion.
  Outdated suggestions cannot be applied.
  This suggestion has been applied or marked resolved.
  Suggestions cannot be applied from pending reviews.
  Suggestions cannot be applied on multi-line comments.
  Suggestions cannot be applied while the pull request is queued to merge.
  Suggestion cannot be applied right now. Please check back later.
  
    
  
    
Uh oh!
There was an error while loading. Please reload this page.