claude-autopm 1.18.0 → 1.20.1

package/autopm/.claude/hooks/docker-first-enforcement.sh

@@ -89,7 +89,7 @@ check_docker_files() {
     if [[ ${#missing_files[@]} -gt 0 ]]; then
         echo ""
         echo "📝 MISSING FILES: ${missing_files[*]}"
-        echo "💡 TIP: Use docker-expert agent to create Docker files:"
+        echo "💡 TIP: Use docker-containerization-expert agent to create Docker files:"
         echo "   Task: Create Docker development environment for this project"
         echo ""
     fi

package/autopm/.claude/mcp/MCP-REGISTRY.md

@@ -96,7 +96,7 @@ autopm mcp add
 cp .claude/examples/mcp/context7.md .claude/mcp/
 
 # Or from installed framework
-cp /path/to/framework/autopm/.claude/examples/mcp/playwright-mcp.md .claude/mcp/
+cp .claude/examples/mcp/playwright-mcp.md .claude/mcp/
 ```
 
 ### ✅ Enabling/Disabling Servers

package/autopm/.claude/mcp/playwright-mcp.md

@@ -74,7 +74,7 @@ autopm mcp sync
 ### Integration with Agents
 
 Used extensively with:
-- `e2e-test-engineer` - For E2E test creation
+- `frontend-testing-engineer` - For E2E test creation
 - `react-frontend-engineer` - For UI testing
 - `ux-design-expert` - For visual regression
 
@@ -309,4 +309,4 @@ npx playwright show-trace trace.zip
 
 - [Playwright Documentation](https://playwright.dev)
 - [MCP Browser Control](https://modelcontextprotocol.org/browser)
-- [E2E Test Engineer Agent](../agents/frameworks/e2e-test-engineer.md)
+- [E2E Test Engineer Agent](../agents/frameworks/frontend-testing-engineer.md)

package/autopm/.claude/rules/agent-coordination.md

@@ -20,10 +20,10 @@ Comprehensive coordination system for multi-agent collaboration in ClaudeAutoPM.
 | python-backend-expert | Backend APIs | `*.py`, `requirements.txt`, `pyproject.toml` | react-ui-expert, kubernetes, cloud architects | framework: fastapi/flask/django |
 | react-ui-expert | Frontend UI | `*.tsx`, `*.jsx`, `*.css`, `package.json` | python-backend, e2e-test | framework: mui/chakra/antd/bootstrap |
 | e2e-test-engineer | E2E Tests | `tests/`, `*.spec.ts`, `playwright.config.ts` | react-ui, python-backend | tool: playwright/cypress |
-| multi-cloud-architect | Cloud Infrastructure | `terraform/`, `*.tf`, cloud configs | kubernetes, docker | provider: aws/azure/gcp |
+| terraform-infrastructure-expert | Cloud Infrastructure | `terraform/`, `*.tf`, cloud configs | kubernetes, docker, cloud architects | provider: aws/azure/gcp |
 | kubernetes-orchestrator | K8s Manifests | `k8s/`, `charts/`, `*.yaml` | cloud architects, github-ops | tool: helm/kustomize |
 | docker-containerization-expert | Containers | `Dockerfile*`, `docker-compose*.yml` | All deployment agents | strategy: multi-stage/compose |
-| database-architect | Database | `migrations/`, `*.sql`, schema files | python-backend | engine: postgresql/mysql/mongodb |
+| postgresql-expert / mongodb-expert | Database | `migrations/`, `*.sql`, schema files | python-backend-engineer | engine: postgresql/mongodb/redis |
 | github-operations-specialist | CI/CD | `.github/`, `*.yml`, `*.yaml` | All agents | platform: actions/azure-devops |
 | mcp-context-manager | Context | `.claude/mcp-servers.json`, context pools | All agents | pool: shared/isolated |
 
@@ -32,7 +32,7 @@ Comprehensive coordination system for multi-agent collaboration in ClaudeAutoPM.
 | Agent | Use Case | Replaces |
 |-------|----------|----------|
 | azure-devops-specialist | Azure DevOps specific | Part of github-operations |
-| playwright-mcp-frontend-tester | MCP browser control | Enhanced e2e-test |
+| e2e-test-engineer | E2E testing with Playwright/Cypress | Support role |
 | code-analyzer | Read-only analysis | Support role |
 | file-analyzer | Log/file summarization | Support role |
 | test-runner | Test execution only | Support role |
@@ -103,7 +103,7 @@ Notes: Added rate limiting middleware
 
 ```yaml
 backend-context:
-  agents: [python-backend-expert, database-architect]
+  agents: [python-backend-expert, postgresql-expert, mongodb-expert]
   sources: [context7, context7]
   filters: [python, fastapi, flask, django, sqlalchemy, databases]
   persistence: true
@@ -115,7 +115,7 @@ frontend-context:
   persistence: true
 
 infrastructure-context:
-  agents: [multi-cloud-architect, kubernetes-orchestrator, docker-containerization-expert]
+  agents: [terraform-infrastructure-expert, kubernetes-orchestrator, docker-containerization-expert]
   sources: [context7, terraform-registry]
   filters: [terraform, kubernetes, docker, cloud-providers]
   persistence: true
@@ -159,7 +159,7 @@ Stream B: Frontend Development
   Priority: P1
 
 Stream C: Infrastructure
-  Agent: multi-cloud-architect
+  Agent: terraform-infrastructure-expert
   Parameters:
     provider: aws
     iac: terraform
@@ -243,7 +243,7 @@ graph LR
   B -->|UI Ready| C[e2e-test-engineer]
   C -->|Tests Pass| D[github-operations-specialist]
   D -->|CI/CD Ready| E[kubernetes-orchestrator]
-  E -->|Deployment Config| F[multi-cloud-architect]
+  E -->|Deployment Config| F[terraform-infrastructure-expert]
   F -->|Infrastructure| G[Production]
 ```
 
@@ -261,7 +261,7 @@ graph LR
 
 ```mermaid
 graph LR
-  A[multi-cloud-architect] -->|New Infrastructure| B[docker-containerization-expert]
+  A[terraform-infrastructure-expert] -->|New Infrastructure| B[docker-containerization-expert]
   B -->|Updated Images| C[kubernetes-orchestrator]
   C -->|Manifests Ready| D[github-operations-specialist]
   D -->|Blue-Green Deploy| E[Production]
@@ -325,11 +325,11 @@ git commit -m "Multiple changes"
 
 | Scenario | Priority | Lead Agent | Support Agents | SLA |
 |----------|----------|------------|----------------|-----|
-| Production Down | P0 | kubernetes-orchestrator | multi-cloud, python-backend | 15 min |
+| Production Down | P0 | kubernetes-orchestrator | terraform-infrastructure-expert, python-backend | 15 min |
 | Security Vulnerability | P0 | code-analyzer | python-backend, github-ops | 30 min |
-| Data Loss Risk | P0 | database-architect | multi-cloud, kubernetes | 15 min |
+| Data Loss Risk | P0 | postgresql-expert | terraform-infrastructure-expert, kubernetes | 15 min |
 | Failed Deployment | P1 | github-operations-specialist | kubernetes-orchestrator | 1 hour |
-| Performance Degradation | P1 | python-backend-expert | database-architect, multi-cloud | 2 hours |
+| Performance Degradation | P1 | python-backend-expert | postgresql-expert, terraform-infrastructure-expert | 2 hours |
 | Test Failures | P2 | e2e-test-engineer | react-ui, python-backend | 4 hours |
 | Feature Development | P3 | Varies by stream | Full stack team | 1-5 days |
 | Technical Debt | P4 | code-analyzer | All relevant agents | Best effort |
@@ -338,13 +338,13 @@ git commit -m "Multiple changes"
 
 ### Never Modify Without Coordination
 
-- **Production configurations** - Requires multi-cloud-architect + kubernetes-orchestrator
+- **Production configurations** - Requires terraform-infrastructure-expert + kubernetes-orchestrator
 - **Security settings** - Requires security review + python-backend-expert
 - **API contracts** - Requires python-backend + react-ui agreement
-- **Database schemas** - Requires database-architect + python-backend
+- **Database schemas** - Requires postgresql-expert + python-backend
 - **Authentication flows** - Requires security review + full stack team
 - **Payment processing** - Requires security + compliance review
-- **User data handling** - Requires privacy review + database-architect
+- **User data handling** - Requires privacy review + postgresql-expert
 
 ### Safe for Independent Work
 
@@ -405,31 +405,31 @@ git commit -m "chore: Sync checkpoint"
 ### System Down
 
 1. **kubernetes-orchestrator** takes incident command
-2. **multi-cloud-architect** checks infrastructure health
+2. **terraform-infrastructure-expert** checks infrastructure health
 3. **python-backend-expert** validates application state
-4. **database-architect** verifies data integrity
+4. **postgresql-expert** verifies data integrity
 5. **github-operations-specialist** prepares rollback if needed
 
 ### Security Breach
 
 1. **code-analyzer** identifies scope and entry point
-2. **multi-cloud-architect** isolates affected resources
+2. **terraform-infrastructure-expert** isolates affected resources
 3. **python-backend-expert** patches vulnerability
-4. **database-architect** audits data access
+4. **postgresql-expert** audits data access
 5. **github-operations-specialist** deploys emergency fix
 
 ### Performance Crisis
 
 1. **python-backend-expert** profiles application bottlenecks
-2. **database-architect** analyzes and optimizes queries
+2. **postgresql-expert** analyzes and optimizes queries
 3. **docker-containerization-expert** optimizes container resources
 4. **kubernetes-orchestrator** scales resources horizontally
-5. **multi-cloud-architect** provisions additional infrastructure
+5. **terraform-infrastructure-expert** provisions additional infrastructure
 
 ### Data Corruption
 
-1. **database-architect** stops writes immediately
-2. **multi-cloud-architect** initiates backup restoration
+1. **postgresql-expert** stops writes immediately
+2. **terraform-infrastructure-expert** initiates backup restoration
 3. **kubernetes-orchestrator** redirects traffic to healthy replicas
 4. **python-backend-expert** validates data integrity checks
 5. **e2e-test-engineer** runs full regression suite
@@ -444,11 +444,13 @@ git commit -m "chore: Sync checkpoint"
 | flask-backend-engineer | python-backend-expert | framework: flask |
 | mui-react-expert | react-ui-expert | framework: mui |
 | chakra-ui-expert | react-ui-expert | framework: chakra |
+| antd-react-expert | react-ui-expert | framework: antd |
+| bootstrap-ui-expert | react-ui-expert | framework: bootstrap |
 | docker-expert | docker-containerization-expert | focus: dockerfile |
 | docker-compose-expert | docker-containerization-expert | focus: compose |
-| aws-cloud-architect | multi-cloud-architect | provider: aws |
-| gcp-cloud-architect | multi-cloud-architect | provider: gcp |
+| docker-development-orchestrator | docker-containerization-expert | focus: development |
 | playwright-test-engineer | e2e-test-engineer | tool: playwright |
+| playwright-mcp-frontend-tester | e2e-test-engineer | browser_control: mcp-enhanced |
 
 ### Backward Compatibility
 

package/autopm/.claude/rules/docker-first-development.md

@@ -43,7 +43,7 @@ project/
 ```bash
 # 1. Check if Docker files exist
 if [ ! -f "Dockerfile" ]; then
-  Use docker-expert agent to create Dockerfile
+  Use docker-containerization-expert agent to create Dockerfile
 fi
 
 # 2. Build development image

package/autopm/.claude/rules/infrastructure-pipeline.md

@@ -23,7 +23,7 @@
 **Sequence**:
 
 ```
-1. docker-expert → Optimize Dockerfile
+1. docker-containerization-expert → Optimize Dockerfile
 2. Security scan with Trivy/Snyk
 3. Build multi-stage image
 4. test-runner → Test container functionality

package/autopm/.claude/rules/ui-development-standards.md

@@ -6,7 +6,7 @@
 
 ### 1. FRAMEWORK DECISION MATRIX
 
-**Bootstrap** - Use `bootstrap-ui-expert`:
+**Bootstrap** - Use `react-ui-expert` (framework=bootstrap):
 
 - Rapid prototyping and MVP development
 - Component-heavy applications (dashboards, admin panels)

package/autopm/.claude/rules/visual-testing.md

@@ -179,8 +179,8 @@ describe('Accessibility', () => {
 
 ### Required Agents for Visual Testing
 
-- **playwright-test-engineer**: Visual regression tests
-- **playwright-mcp-frontend-tester**: Browser automation
+- **frontend-testing-engineer**: Visual regression tests
+- **frontend-testing-engineer**: Browser automation
 - **react-frontend-engineer**: Component development
 - **code-analyzer**: Review UI code quality
 
@@ -188,7 +188,7 @@ describe('Accessibility', () => {
 
 ```
 1. Implement UI change following TDD
-2. playwright-test-engineer → Create visual tests
+2. frontend-testing-engineer → Create visual tests
 3. Run visual regression tests at all breakpoints
 4. code-analyzer → Verify accessibility
 5. Capture screenshots for PR

package/autopm/.claude/scripts/azure/active-work.js

@@ -8,8 +8,8 @@
 const path = require('path');
 const fs = require('fs');
 const chalk = require('chalk');
-const AzureDevOpsClient = require('../../lib/azure/client');
-const AzureFormatter = require('../../lib/azure/formatter');
+const AzureDevOpsClient = require('../../providers/azure/lib/client');
+const AzureFormatter = require('../../providers/azure/lib/formatter');
 const { table } = require('table');
 
 class AzureActiveWork {

package/autopm/.claude/scripts/azure/blocked.js

@@ -18,22 +18,22 @@ const https = require('https');
 const yargs = require('yargs/yargs');
 const { hideBin } = require('yargs/helpers');
 
-// Import utilities
-const Logger = require('../../lib/utils/logger');
-const FileSystem = require('../../lib/utils/filesystem');
-const Config = require('../../lib/utils/config');
+// TODO: Implement utility modules before using this script
+// const Logger = require('../../lib/utils/logger');
+// const FileSystem = require('../../lib/utils/filesystem');
+// const Config = require('../../lib/utils/config');
 
 class AzureBlocked {
   constructor(options = {}) {
-    // Initialize utilities
-    const loggerOptions = {
-      verbose: options.verbose || false,
-      silent: options.silent || false
-    };
-
-    this.logger = new Logger(loggerOptions);
-    this.fs = new FileSystem(this.logger);
-    this.config = new Config(this.logger);
+    // Initialize utilities (commented out until utils are implemented)
+    // const loggerOptions = {
+    //   verbose: options.verbose || false,
+    //   silent: options.silent || false
+    // };
+
+    // this.logger = new Logger(loggerOptions);
+    // this.fs = new FileSystem(this.logger);
+    // this.config = new Config(this.logger);
 
     // Set options
     this.options = {

package/autopm/.claude/scripts/azure/daily.js

@@ -10,7 +10,7 @@ const path = require('path');
 const fs = require('fs').promises;
 const chalk = require('chalk');
 const dotenv = require('dotenv');
-const AzureDevOpsClient = require('../../lib/azure/client');
+const AzureDevOpsClient = require('../../providers/azure/lib/client');
 
 class AzureDaily {
   constructor(options = {}) {

package/autopm/.claude/scripts/azure/dashboard.js

@@ -7,7 +7,7 @@
 
 const path = require('path');
 const fs = require('fs');
-const AzureDevOpsClient = require('../../lib/azure/client');
+const AzureDevOpsClient = require('../../providers/azure/lib/client');
 
 // Simple chalk replacement for stub
 const chalk = {

package/autopm/.claude/scripts/azure/feature-list.js

@@ -5,8 +5,8 @@
  * Lists all features in the current project
  */
 
-const AzureDevOpsClient = require('../../lib/azure/client');
-const AzureFormatter = require('../../lib/azure/formatter');
+const AzureDevOpsClient = require('../../providers/azure/lib/client');
+const AzureFormatter = require('../../providers/azure/lib/formatter');
 const chalk = require('chalk');
 const path = require('path');
 const fs = require('fs');

package/autopm/.claude/scripts/azure/feature-status.js

@@ -10,7 +10,7 @@ const path = require('path');
 const fs = require('fs').promises;
 const chalk = require('chalk');
 const dotenv = require('dotenv');
-const AzureDevOpsClient = require('../../lib/azure/client');
+const AzureDevOpsClient = require('../../providers/azure/lib/client');
 
 class AzureFeatureStatus {
   constructor(options = {}) {

package/autopm/.claude/scripts/azure/next-task.js

@@ -10,7 +10,7 @@ const path = require('path');
 const fs = require('fs').promises;
 const chalk = require('chalk');
 const dotenv = require('dotenv');
-const AzureDevOpsClient = require('../../lib/azure/client');
+const AzureDevOpsClient = require('../../providers/azure/lib/client');
 
 class AzureNextTask {
   constructor(options = {}) {

package/autopm/.claude/scripts/azure/search.js

@@ -10,7 +10,7 @@ const path = require('path');
 const fs = require('fs').promises;
 const chalk = require('chalk');
 const dotenv = require('dotenv');
-const AzureDevOpsClient = require('../../lib/azure/client');
+const AzureDevOpsClient = require('../../providers/azure/lib/client');
 
 class AzureSearch {
   constructor(options = {}) {

package/autopm/.claude/scripts/azure/setup.js

@@ -22,24 +22,24 @@ const yargs = require('yargs/yargs');
 const { hideBin } = require('yargs/helpers');
 const yaml = require('js-yaml');
 
-// Import utilities
-const Logger = require('../../lib/utils/logger');
-const FileSystem = require('../../lib/utils/filesystem');
-const Prompts = require('../../lib/utils/prompts');
-const Config = require('../../lib/utils/config');
+// TODO: Implement utility modules before using this script
+// const Logger = require('../../lib/utils/logger');
+// const FileSystem = require('../../lib/utils/filesystem');
+// const Prompts = require('../../lib/utils/prompts');
+// const Config = require('../../lib/utils/config');
 
 class AzureSetup {
   constructor(options = {}) {
-    // Initialize utilities
-    const loggerOptions = {
-      verbose: options.verbose || false,
-      silent: options.silent || false
-    };
-
-    this.logger = new Logger(loggerOptions);
-    this.fs = new FileSystem(this.logger);
-    this.prompts = new Prompts(this.logger);
-    this.config = new Config(this.logger);
+    // Initialize utilities (commented out until utils are implemented)
+    // const loggerOptions = {
+    //   verbose: options.verbose || false,
+    //   silent: options.silent || false
+    // };
+
+    // this.logger = new Logger(loggerOptions);
+    // this.fs = new FileSystem(this.logger);
+    // this.prompts = new Prompts(this.logger);
+    // this.config = new Config(this.logger);
 
     // Set options
     this.options = {

package/autopm/.claude/scripts/azure/sprint-report.js

@@ -9,8 +9,8 @@
 const path = require('path');
 const fs = require('fs');
 const chalk = require('chalk');
-const AzureDevOpsClient = require('../../lib/azure/client');
-const AzureFormatter = require('../../lib/azure/formatter');
+const AzureDevOpsClient = require('../../providers/azure/lib/client');
+const AzureFormatter = require('../../providers/azure/lib/formatter');
 const { table } = require('table');
 
 class AzureSprintReport {

package/autopm/.claude/scripts/azure/sync.js

@@ -7,7 +7,7 @@
 
 const path = require('path');
 const fs = require('fs').promises;
-const AzureDevOpsClient = require('../../lib/azure/client');
+const AzureDevOpsClient = require('../../providers/azure/lib/client');
 
 // Simple chalk replacement for stub
 const chalk = {

package/autopm/.claude/scripts/azure/us-list.js

@@ -9,7 +9,7 @@ const path = require('path');
 const fs = require('fs').promises;
 const chalk = require('chalk');
 const dotenv = require('dotenv');
-const AzureDevOpsClient = require('../../lib/azure/client');
+const AzureDevOpsClient = require('../../providers/azure/lib/client');
 
 class AzureUserStoryList {
   constructor(options = {}) {

package/autopm/.claude/scripts/azure/us-status.js

@@ -9,7 +9,7 @@ const path = require('path');
 const fs = require('fs').promises;
 const chalk = require('chalk');
 const dotenv = require('dotenv');
-const AzureDevOpsClient = require('../../lib/azure/client');
+const AzureDevOpsClient = require('../../providers/azure/lib/client');
 
 class AzureUserStoryStatus {
   constructor(options = {}) {

package/autopm/.claude/scripts/azure/validate.js

@@ -18,22 +18,22 @@ const https = require('https');
 const yargs = require('yargs/yargs');
 const { hideBin } = require('yargs/helpers');
 
-// Import utilities
-const Logger = require('../../lib/utils/logger');
-const FileSystem = require('../../lib/utils/filesystem');
-const Config = require('../../lib/utils/config');
+// TODO: Implement utility modules before using this script
+// const Logger = require('../../lib/utils/logger');
+// const FileSystem = require('../../lib/utils/filesystem');
+// const Config = require('../../lib/utils/config');
 
 class AzureValidate {
   constructor(options = {}) {
-    // Initialize utilities
-    const loggerOptions = {
-      verbose: options.verbose || false,
-      silent: options.silent || false
-    };
-
-    this.logger = new Logger(loggerOptions);
-    this.fs = new FileSystem(this.logger);
-    this.config = new Config(this.logger);
+    // Initialize utilities (commented out until utils are implemented)
+    // const loggerOptions = {
+    //   verbose: options.verbose || false,
+    //   silent: options.silent || false
+    // };
+
+    // this.logger = new Logger(loggerOptions);
+    // this.fs = new FileSystem(this.logger);
+    // this.config = new Config(this.logger);
 
     // Set options
     this.options = {

package/autopm/.claude/scripts/lib/frontmatter-utils.sh

@@ -9,6 +9,21 @@ SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 source "${SCRIPT_DIR}/logging-utils.sh"
 source "${SCRIPT_DIR}/datetime-utils.sh"
 
+# Escape special characters for use in sed pattern
+sed_escape_pattern() {
+    local str="$1"
+    # Escape characters that have special meaning in sed patterns: . * [ ] ^ $ \ /
+    printf '%s\n' "$str" | sed 's/[]\/$*.^[]/\\&/g'
+}
+
+# Escape special characters for use in sed replacement
+sed_escape_replacement() {
+    local str="$1"
+    # Escape backslashes first, then other special characters
+    # This prevents double-escaping issues
+    printf '%s\n' "$str" | sed 's/\\/\\\\/g; s/&/\\&/g; s/|/\\|/g'
+}
+
 # Update or add a field in frontmatter
 update_frontmatter_field() {
     local file_path="$1"
@@ -25,18 +40,32 @@ update_frontmatter_field() {
     # Create backup
     cp "$file_path" "${file_path}.bak"
 
+    # Escape field name and value for safe use in sed
+    local escaped_field_name
+    local escaped_field_value
+    escaped_field_name=$(sed_escape_pattern "$field_name")
+    escaped_field_value=$(sed_escape_replacement "$field_value")
+
     # Check if field exists
     if grep -q "^${field_name}:" "$file_path"; then
-        # Update existing field
-        sed -i.tmp "/^${field_name}:/c\\${field_name}: ${field_value}" "$file_path"
+        # Update existing field - delete old line and insert new one
+        # This approach avoids sed replacement string escaping issues
+        grep -v "^${escaped_field_name}:" "$file_path" > "${file_path}.tmp"
+        {
+            head -1 "${file_path}.tmp"  # First --- line
+            printf '%s: %s\n' "$field_name" "$field_value"  # New field value
+            tail -n +2 "${file_path}.tmp"  # Rest of file
+        } > "${file_path}.tmp2" && mv "${file_path}.tmp2" "$file_path"
         rm -f "${file_path}.tmp"
         log_debug "Updated existing field: $field_name"
     else
         # Add new field after the first line of frontmatter (after opening ---)
-        awk -v field="${field_name}: ${field_value}" '
-            /^---$/ && NR==1 { print; print field; next }
-            { print }
-        ' "$file_path" > "${file_path}.tmp" && mv "${file_path}.tmp" "$file_path"
+        # Write the new line directly to avoid variable expansion issues
+        {
+            head -1 "$file_path"  # First --- line
+            printf '%s: %s\n' "$field_name" "$field_value"  # New field (preserves all chars)
+            tail -n +2 "$file_path"  # Rest of file
+        } > "${file_path}.tmp" && mv "${file_path}.tmp" "$file_path"
         log_debug "Added new field: $field_name"
     fi
 
@@ -60,8 +89,14 @@ get_frontmatter_field() {
         return 1
     fi
 
+    # Escape field name for safe use in patterns
+    local escaped_field_name
+    escaped_field_name=$(sed_escape_pattern "$field_name")
+
     local field_value
-    field_value=$(grep "^${field_name}:" "$file_path" | sed "s/^${field_name}: *//" | head -1)
+    # Use | delimiter to avoid conflicts with / in values
+    # Only remove the field name, colon, and exactly one space (YAML format)
+    field_value=$(grep "^${field_name}:" "$file_path" | sed "s|^${escaped_field_name}: ||" | head -1)
 
     log_debug "Retrieved field $field_name: '$field_value'"
     log_function_exit "get_frontmatter_field"

package/autopm/.claude/scripts/lib/logging-utils.sh

@@ -4,22 +4,26 @@
 
 set -euo pipefail
 
-# Colors for output
-readonly RED='\033[0;31m'
-readonly GREEN='\033[0;32m'
-readonly YELLOW='\033[1;33m'
-readonly BLUE='\033[0;34m'
-readonly PURPLE='\033[0;35m'
-readonly CYAN='\033[0;36m'
-readonly WHITE='\033[1;37m'
-readonly NC='\033[0m' # No Color
-
-# Log levels
-readonly LOG_LEVEL_DEBUG=0
-readonly LOG_LEVEL_INFO=1
-readonly LOG_LEVEL_WARNING=2
-readonly LOG_LEVEL_ERROR=3
-readonly LOG_LEVEL_SUCCESS=4
+# Colors for output (only define if not already defined)
+if [[ -z "${RED:-}" ]]; then
+    readonly RED='\033[0;31m'
+    readonly GREEN='\033[0;32m'
+    readonly YELLOW='\033[1;33m'
+    readonly BLUE='\033[0;34m'
+    readonly PURPLE='\033[0;35m'
+    readonly CYAN='\033[0;36m'
+    readonly WHITE='\033[1;37m'
+    readonly NC='\033[0m' # No Color
+fi
+
+# Log levels (only define if not already defined)
+if [[ -z "${LOG_LEVEL_DEBUG:-}" ]]; then
+    readonly LOG_LEVEL_DEBUG=0
+    readonly LOG_LEVEL_INFO=1
+    readonly LOG_LEVEL_WARNING=2
+    readonly LOG_LEVEL_ERROR=3
+    readonly LOG_LEVEL_SUCCESS=4
+fi
 
 # Default log level (can be overridden by AUTOPM_LOG_LEVEL env var)
 LOG_LEVEL=${AUTOPM_LOG_LEVEL:-$LOG_LEVEL_INFO}

package/autopm/.claude/scripts/lib/validation-utils.sh

@@ -12,7 +12,7 @@ source "${SCRIPT_DIR}/logging-utils.sh"
 validate_required_commands() {
     local commands=("$@")
 
-    log_function_entry "validate_required_commands" "${commands[@]}"
+    log_function_entry "validate_required_commands" "${commands[@]:-}"
 
     local missing_commands=()