# 8. Cross-cutting Concepts

## 8.1 State Management

### 8.1.1 Centralized State File

**Concept**: All command state stored in single JSON file

```mermaid
graph TB
    subgraph "State Structure"
        Root[Root Object]
        Regular[Regular Keys]
        Workflow[workflow Key]
        Namespaced[Namespaced Keys]
    end
    
    Root --> Regular
    Root --> Workflow
    Regular --> Namespaced
    
    Namespaced --> Jira[jira.*]
    Namespaced --> FileReview[file_review.*]
    Namespaced --> Background[background.*]
    Namespaced --> Git[commit_message, etc.]
```

**Key Namespaces**:

- `jira.*` - Jira issue operations
- `file_review.*` - PR file review context
- `background.*` - Background task tracking (task_id, timeout, etc.)
- `pull_request_id` - Azure DevOps PR ID
- `workflow` - Workflow state (reserved)

### 8.1.2 File Locking Strategy

**Problem**: Concurrent access to state file

**Solution**: Cross-platform file locking

```python
# Pattern used throughout
from agentic_devtools.state import load_state, save_state

# Load and save with automatic locking
state = load_state(use_locking=True)
state[key] = value
save_state(state, use_locking=True)

# Or use locked file directly
from agentic_devtools.file_locking import locked_state_file

with locked_state_file(state_file_path) as f:
    state = json.load(f)
    state[key] = value
    f.seek(0)
    f.truncate()
    json.dump(state, f)
```

**Implementation**:

- **Windows**: `msvcrt.locking()`
- **Unix**: `fcntl.flock()`
- Automatic retry with fixed 10ms intervals until 5s timeout

## 8.2 Error Handling

### 8.2.1 Error Handling Strategy

```mermaid
flowchart TD
    Start[Command Start] --> Try[Try Block]
    Try --> Success[Execute Logic]
    Success --> Return0[Return 0]
    
    Try --> Error[Exception]
    Error --> Type{Exception Type}
    
    Type -->|ValidationError| Val[Format validation message]
    Type -->|APIError| API[Format API error]
    Type -->|StateError| State[Format state error]
    Type -->|Unknown| Gen[Generic error]
    
    Val --> Log[Log to stderr]
    API --> Log
    State --> Log
    Gen --> Log
    
    Log --> Update{Background task?}
    Update -->|Yes| UpdateTask[Update task state]
    Update -->|No| Return1[Return non-zero]
    UpdateTask --> Return1
```

### 8.2.2 Error Message Format

**Pattern**: Clear, actionable error messages

```text
❌ Error: Missing required state key: jira.issue_key
   Run: agdt-set jira.issue_key PROJECT-1234

❌ Error: API request failed (401 Unauthorized)
   Check your AZURE_DEV_OPS_COPILOT_PAT (or AZURE_DEVOPS_EXT_PAT) environment variable

❌ Error: Background task failed (exit code 1)
   View logs: agdt-task-log
```

## 8.3 Logging and Monitoring

### 8.3.1 Logging Levels

```mermaid
graph LR
    Console[Console Output] --> User[User Messages]
    Files[Log Files] --> Debug[Debug Info]
    Files --> Errors[Error Details]
    
    User --> Success[✓ Success messages]
    User --> Actions[→ Action descriptions]
    User --> Info[ℹ Info messages]
    
    Debug --> API[API requests]
    Debug --> State[State changes]
    Errors --> Stack[Stack traces]
    Errors --> Context[Error context]
```

### 8.3.2 Background Task Logging

**Location**: `.agdt/workflows/…/background-tasks/logs/`

**Format**: One log file per task

```text
[2026-02-13 10:30:00] Starting task: add-jira-comment
[2026-02-13 10:30:01] Reading state: jira.issue_key
[2026-02-13 10:30:01] Found issue key: PROJECT-1234
[2026-02-13 10:30:02] Calling Jira API: POST /issue/PROJECT-1234/comment
[2026-02-13 10:30:03] API response: 200 OK
[2026-02-13 10:30:03] ✓ Task completed successfully
```

## 8.4 Security

### 8.4.1 Credential Management

```mermaid
graph TB
    subgraph "Sources"
        EnvVar[Environment Variables]
    end
    
    subgraph "Never Stored"
        Code[Source Code ✗]
        State[State File ✗]
        Logs[Log Files ✗]
        Output[Output Files ✗]
    end
    
    subgraph "Commands"
        CLI[agdt-* Commands]
    end
    
    EnvVar -->|Read at runtime| CLI
    CLI -.->|NEVER write| Code
    CLI -.->|NEVER write| State
    CLI -.->|NEVER write| Logs
    CLI -.->|NEVER write| Output
    
    style EnvVar fill:#9f9
    style Code fill:#f99
    style State fill:#f99
    style Logs fill:#f99
    style Output fill:#f99
```

**Required Environment Variables**:

- `AZURE_DEV_OPS_COPILOT_PAT` - Azure DevOps Personal Access Token (preferred)
- `AZURE_DEVOPS_EXT_PAT` - Azure DevOps Personal Access Token (fallback)
- `JIRA_COPILOT_PAT` - Jira API token (PAT)
- `JIRA_AUTH_SCHEME` - Jira auth scheme; set to `basic` when using email/username + API token for basic auth

**Optional Environment Variables**:

- `JIRA_EMAIL` / `JIRA_USERNAME` - Jira account identifier used with `JIRA_AUTH_SCHEME=basic`

### 8.4.2 Security Checklist

| Aspect | Implementation |
|--------|---------------|
| **Credentials** | Environment variables only |
| **State File** | No secrets, excluded from Git |
| **API Tokens** | Loaded at runtime, never logged |
| **Output Files** | May contain data, excluded from Git |
| **Error Messages** | Sanitized, no credential leakage |
| **File Permissions** | User-only read/write on temp files |

## 8.5 Testing Strategy

### 8.5.1 Test Pyramid

```mermaid
graph TB
    subgraph "Test Pyramid"
        E2E[End-to-End Tests<br/>~5%]
        Integration[Integration Tests<br/>~25%]
        Unit[Unit Tests<br/>~70%]
    end
    
    E2E --> Integration
    Integration --> Unit
    
    style Unit fill:#9f9
    style Integration fill:#ff9
    style E2E fill:#f99
```

### 8.5.2 Test Coverage Requirements

| Component | Coverage Target | Test Type |
|-----------|----------------|-----------|
| **Core (state, locking)** | 100% | Unit |
| **CLI Commands** | 95% | Unit + Integration |
| **Background Tasks** | 90% | Integration |
| **API Helpers** | 95% | Unit (mocked APIs) |
| **Workflows** | 85% | Integration |

### 8.5.3 Test Execution Pattern

```bash
# Full test suite with coverage
agdt-test

# Quick tests (no coverage)
agdt-test-quick

# Specific file with 100% coverage
agdt-test-file --source-file agentic_devtools/state.py

# Specific test pattern
agdt-test-pattern tests/test_state.py::TestState::test_get_value -v
```

## 8.6 Performance Considerations

### 8.6.1 Background Task Strategy

**Problem**: Long operations block AI agents

**Solution**: Spawn background processes

```mermaid
graph LR
    Sync[Synchronous<br/>❌ Blocks agent] -->|Problem| Long[Long API calls<br/>30-60 seconds]
    Async[Asynchronous<br/>✓ Non-blocking] -->|Solution| BG[Background process<br/>Returns immediately]
    
    style Sync fill:#f99
    style Async fill:#9f9
```

**Threshold**: Operations >5 seconds should be background tasks

### 8.6.2 State File Performance

- **File Size**: Typically <100 KB
- **Read Time**: <10ms
- **Write Time**: <20ms (with lock)
- **Lock Wait**: <100ms (typical)

## 8.7 Usability Patterns

### 8.7.1 Command Approval UX

```mermaid
graph TB
    Generic[Generic Commands<br/>agdt-set, agdt-get] -->|Approve once| AllKeys[Use for all keys]
    Parameterless[Parameterless Actions<br/>agdt-add-jira-comment] -->|Approve once| Reuse[Reuse forever]
    
    BadPattern[❌ Bad: agdt-comment --text "..."] -->|Must approve| EachTime[Each time with<br/>different text]
    
    style Generic fill:#9f9
    style Parameterless fill:#9f9
    style BadPattern fill:#f99
```

### 8.7.2 Output Format Standards

**Console Output Pattern**:

```text
# Success
✓ Task started: 123e4567-e89b-12d3-a456-426614174000
  Status: agdt-task-status
  Logs: agdt-task-log
  Wait: agdt-task-wait

# Progress
→ Fetching PR details for PR #123...
→ Creating Jira comment...

# Info
ℹ Found 3 files to review
ℹ Current workflow: pull-request-review (step: file-review)

# Error
❌ Error: Missing required state key: jira.issue_key
   Run: agdt-set jira.issue_key PROJECT-1234
```

## 8.8 Documentation Standards

### 8.8.1 Code Documentation

- **Docstrings**: All public functions and classes
- **Type Hints**: All function signatures
- **Comments**: Complex logic only
- **Examples**: In docstrings for CLI commands

### 8.8.2 Markdown Standards

**Linting Rules** (`.markdownlint.json`):

- MD041: First line must be h1 heading
- MD022: Blank lines around headings
- MD031/MD032: Blank lines around fences/lists
- MD040: Code fences must specify language
- MD034: No bare URLs
- MD009: No trailing spaces
- MD058: Blank lines around tables

## 8.9 Extensibility

### 8.9.1 Adding New Commands

```mermaid
graph TB
    Create[Create command.py] --> Implement[Implement CLI function]
    Implement --> Entry[Add entry point<br/>in pyproject.toml]
    Entry --> Test[Write tests]
    Test --> Doc[Update docs]
    Doc --> Done[✓ Command available]
```

### 8.9.2 Adding New Workflows

```mermaid
graph TB
    Template[Create prompt templates] --> Agent[Create agent.md file]
    Agent --> Steps[Define workflow steps]
    Steps --> State[Define state keys]
    State --> Commands[Create workflow commands]
    Commands --> Test[Test workflow]
    Test --> Done[✓ Workflow available]
```

## 8.10 Version Control Patterns

### 8.10.1 Excluded from Git

```text
.agdt/                         # All workflow state files
  workflows/…/state.json       # State file
  workflows/…/*.json           # Output files
  workflows/…/background-tasks/# Task logs
__pycache__/               # Python cache
*.pyc                      # Compiled Python
dist/                      # Build artifacts
*.egg-info/                # Package metadata
```

### 8.10.2 Git Worktree Pattern

```mermaid
graph TB
    Main[Main Worktree<br/>main branch] --> WT1[Worktree 1<br/>feature/PROJECT-1234]
    Main --> WT2[Worktree 2<br/>feature/PROJECT-5678]
    
    Main -.->|Shared| Git[.git/]
    WT1 -.->|Shared| Git
    WT2 -.->|Shared| Git
    
    Main -->|Isolated| State1[agdt-state.json]
    WT1 -->|Isolated| State2[agdt-state.json]
    WT2 -->|Isolated| State3[agdt-state.json]
```
