feat: implement comprehensive development infrastructure and documentation

- Add GitHub Actions quality gates with backend/frontend/security analysis
- Implement log cleanup background service with configurable retention
- Add comprehensive Serilog configuration for dev/production environments
- Restructure documentation with improved organization and clarity
- Add configuration management UI components and services
- Enhance code quality with StyleCop analyzers and EditorConfig rules
- Update CORS settings and dependency management

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: hemanandr

.editorconfig

@@ -66,6 +66,35 @@ csharp_style_var_for_built_in_types = false:suggestion
 csharp_style_var_when_type_is_apparent = true:suggestion
 csharp_style_var_elsewhere = false:suggestion
 
+# Organize usings
+dotnet_sort_system_directives_first = true
+dotnet_separate_import_directive_groups = false
+
+# Code quality rules (relaxed for internal project)
+dotnet_diagnostic.CA1031.severity = none  # Do not catch general exception types
+dotnet_diagnostic.CA2007.severity = none  # Consider calling ConfigureAwait on the awaited task
+dotnet_diagnostic.CA1062.severity = suggestion  # Validate arguments of public methods
+
+# StyleCop Rules (selective enforcement for internal project)
+dotnet_diagnostic.SA0001.severity = none  # XML comment analysis disabled
+dotnet_diagnostic.SA1633.severity = none  # File should have header
+dotnet_diagnostic.SA1200.severity = none  # Using directives should be placed correctly
+dotnet_diagnostic.SA1101.severity = none  # Prefix local calls with this
+dotnet_diagnostic.SA1309.severity = none  # Field names should not begin with underscore
+dotnet_diagnostic.SA1413.severity = none  # Use trailing comma in multi-line initializers
+dotnet_diagnostic.SA1118.severity = suggestion  # Parameter should not span multiple lines
+dotnet_diagnostic.SA1600.severity = none  # Elements should be documented
+dotnet_diagnostic.SA1601.severity = none  # Partial elements should be documented
+dotnet_diagnostic.SA1602.severity = none  # Enumeration items should be documented
+dotnet_diagnostic.SA1116.severity = suggestion  # Parameters should begin on line after declaration
+dotnet_diagnostic.SA1117.severity = suggestion  # Parameters should be on same line or separate lines
+dotnet_diagnostic.SA1127.severity = suggestion  # Generic constraints should be on own line
+dotnet_diagnostic.SA1108.severity = suggestion  # Block statements should not contain embedded comments
+dotnet_diagnostic.SA1502.severity = suggestion  # Element should not be on single line
+dotnet_diagnostic.SA1516.severity = suggestion  # Elements should be separated by blank line
+dotnet_diagnostic.SA1300.severity = suggestion  # Element should begin with uppercase letter
+dotnet_diagnostic.SA1136.severity = suggestion  # Enum values should be on separate lines
+
 # TypeScript, JavaScript, JSX, TSX files
 [*.{ts,tsx,js,jsx}]
 indent_size = 2

.github/workflows/quality-gates.yml

@@ -0,0 +1,98 @@
+name: Quality Gates
+
+on:
+  push:
+    branches: [ master, develop ]
+  pull_request:
+    branches: [ master, develop ]
+
+jobs:
+  backend-quality:
+    runs-on: ubuntu-latest
+    name: Backend Code Quality
+    
+    steps:
+    - uses: actions/checkout@v4
+    
+    - name: Setup .NET
+      uses: actions/setup-dotnet@v4
+      with:
+        dotnet-version: '8.0.x'
+        
+    - name: Restore dependencies
+      run: dotnet restore ThingConnect.Pulse.Server/ThingConnect.Pulse.Server.csproj
+      
+    - name: Build with analyzers
+      run: dotnet build ThingConnect.Pulse.Server/ThingConnect.Pulse.Server.csproj --no-restore --configuration Release --verbosity normal
+      
+    - name: Run tests (if any)
+      run: |
+        if [ -d "ThingConnect.Pulse.Tests" ]; then
+          dotnet test ThingConnect.Pulse.Tests --no-build --configuration Release
+        else
+          echo "No test project found - skipping tests"
+        fi
+
+  frontend-quality:
+    runs-on: ubuntu-latest
+    name: Frontend Code Quality
+    
+    steps:
+    - uses: actions/checkout@v4
+    
+    - name: Setup Node.js
+      uses: actions/setup-node@v4
+      with:
+        node-version: '18'
+        cache: 'npm'
+        cache-dependency-path: thingconnect.pulse.client/package-lock.json
+        
+    - name: Install dependencies
+      run: |
+        cd thingconnect.pulse.client
+        npm ci
+        
+    - name: Run ESLint
+      run: |
+        cd thingconnect.pulse.client
+        npm run lint
+        
+    - name: Check Prettier formatting
+      run: |
+        cd thingconnect.pulse.client
+        npm run format -- --check
+        
+    - name: Build frontend
+      run: |
+        cd thingconnect.pulse.client
+        npm run build
+
+  security-scan:
+    runs-on: ubuntu-latest
+    name: Security Analysis
+    
+    steps:
+    - uses: actions/checkout@v4
+    
+    - name: Run npm audit
+      run: |
+        cd thingconnect.pulse.client
+        npm audit --audit-level=high
+        
+    - name: Setup .NET for security scan
+      uses: actions/setup-dotnet@v4
+      with:
+        dotnet-version: '8.0.x'
+        
+    - name: Install security scanning tool
+      run: dotnet tool install --global Microsoft.CST.DevSkim.CLI
+      
+    - name: Run DevSkim security analysis
+      run: devskim analyze . --output-file security-report.json --output-format json || true
+      
+    - name: Upload security report
+      uses: actions/upload-artifact@v4
+      if: always()
+      with:
+        name: security-report
+        path: security-report.json

.obsidian/app.json

@@ -0,0 +1 @@
+{}

.obsidian/core-plugins.json

@@ -0,0 +1,33 @@
+{
+  "file-explorer": true,
+  "global-search": true,
+  "switcher": true,
+  "graph": true,
+  "backlink": true,
+  "canvas": true,
+  "outgoing-link": true,
+  "tag-pane": true,
+  "footnotes": false,
+  "properties": false,
+  "page-preview": true,
+  "daily-notes": true,
+  "templates": true,
+  "note-composer": true,
+  "command-palette": true,
+  "slash-command": false,
+  "editor-status": true,
+  "bookmarks": true,
+  "markdown-importer": false,
+  "zk-prefixer": false,
+  "random-note": false,
+  "outline": true,
+  "word-count": true,
+  "slides": false,
+  "audio-recorder": false,
+  "workspaces": false,
+  "file-recovery": true,
+  "publish": false,
+  "sync": true,
+  "bases": true,
+  "webviewer": false
+}

ONE_PAGER.MD

@@ -2,7 +2,9 @@
 
 ## 0) One‑liner & Intent
 
-A free, on‑prem, Windows‑service + web UI tool for manufacturing sites to monitor device **availability** (Ethernet/Wi‑Fi, IPv4/IPv6) using a **YAML** configuration, with **live status**, **historical rollups**, and **CSV export**. Designed for wide adoption to promote the ThingConnect brand.
+**Free, on-premises availability monitoring for manufacturing IT/OT over IPv4/IPv6.**
+
+ThingConnect Pulse provides YAML-configured monitoring with live dashboard, historical rollups, and CSV export. Designed for plant IT/OT admins, production supervisors, and maintenance engineers who need reliable network device monitoring with zero external dependencies and 5-minute setup.
 
 ---
 
@@ -30,9 +32,11 @@ A free, on‑prem, Windows‑service + web UI tool for manufacturing sites to mo
 ## 2) Primary Users & Jobs‑to‑Be‑Done
 
 * **Plant IT/OT Admin:** define targets via YAML; confirm site health; export reports.
-* **Production Supervisor:** glance “all green?” view on phone/tablet before/after shift.
+* **Production Supervisor:** glance "all green?" view on phone/tablet before/after shift.
 * **Maintenance/Controls Engineer:** correlate device outages with line incidents.
 
+*See [detailed personas](./docs/personas.md) for complete user profiles and JTBD analysis.*
+
 **Confidence:** 0.9
 
 ---

README.md

@@ -1,6 +1,17 @@
 # ThingConnect Pulse
 
-Network availability monitoring system for manufacturing sites.
+**Free, on-premises availability monitoring for manufacturing IT/OT over IPv4/IPv6.**
+
+ThingConnect Pulse provides YAML-configured monitoring with live dashboard, historical rollups, and CSV export. Designed for plant IT/OT admins, production supervisors, and maintenance engineers who need reliable network device monitoring with zero external dependencies.
+
+**Value:**
+- **Lightweight**: 5-minute setup with single Windows Service installer
+- **Local-first**: No cloud dependencies, all data stays on-premises  
+- **Readable config**: Simple YAML configuration with explicit Apply workflow
+
+**Non-Goals (v1):**
+- No TLS/auth UI, no WAN exposure, no localization
+- Links: [Complete Specification](./ONE_PAGER.MD) • [Problem & Value](./docs/value.md) • [Scope & Non-Goals](./docs/scope-v1.md) • [Product Roadmap](./docs/roadmap.md)
 
 ## Quick Start
 

ThingConnect.Pulse.Server/Program.cs

@@ -18,18 +18,18 @@ public static async Task Main(string[] args)
         // Initialize path service for directory management
         var pathService = new PathService();
 
-        // Configure Serilog for rolling file logging
+        // Create initial configuration to read Serilog settings
+        var configuration = new ConfigurationBuilder()
+            .AddJsonFile("appsettings.json", optional: false, reloadOnChange: true)
+            .AddJsonFile($"appsettings.{Environment.GetEnvironmentVariable("ASPNETCORE_ENVIRONMENT") ?? "Production"}.json", optional: true, reloadOnChange: true)
+            .Build();
+        
+        // Configure Serilog from configuration files
         Log.Logger = new LoggerConfiguration()
-            .MinimumLevel.Information()
-            .MinimumLevel.Override("Microsoft", LogEventLevel.Warning)
-            .MinimumLevel.Override("Microsoft.Hosting.Lifetime", LogEventLevel.Information)
+            .ReadFrom.Configuration(configuration)
             .Enrich.FromLogContext()
-            .WriteTo.Console()
-            .WriteTo.File(
-                Path.Combine(pathService.GetLogsDirectory(), "pulse-.log"),
-                rollingInterval: RollingInterval.Day,
-                retainedFileCountLimit: 30,
-                shared: true)
+            .Enrich.WithMachineName()
+            .Enrich.WithProcessId()
             .CreateLogger();
 
         try
@@ -76,13 +76,16 @@ public static async Task Main(string[] args)
 
             // Add prune services
             builder.Services.AddScoped<IPruneService, PruneService>();
+            
+            // Add log cleanup service
+            builder.Services.AddHostedService<LogCleanupBackgroundService>();
 
             // Add CORS
             builder.Services.AddCors(options =>
             {
                 options.AddPolicy("AllowFrontend", policy =>
                 {
-                    policy.WithOrigins("https://localhost:55610", "http://localhost:55610", "https://localhost:5173", "http://localhost:5173")
+                    policy.WithOrigins("https://localhost:55610", "http://localhost:55610", "https://localhost:5173", "http://localhost:5173", "https://localhost:55605")
                           .AllowAnyHeader()
                           .AllowAnyMethod()
                           .AllowCredentials();