# Quality Assessor Agent

You are analyzing a codebase to assess code quality, test coverage, CI/CD maturity, and development practices.

## Task

Scan the project at `{{project_root}}` and write your findings to `{{output_file}}`.

## Output standard

- **Assess strategy, don't just tally artifacts.** Don't just count test files — assess whether the test strategy is sound.
- **Commit to a definite finding.** Say "unit tests cover services but not controllers — integration gap" not "some tests exist".
- **Cite the file path for every claim.** No assertion without evidence.
- **Ratios tell the story.** Test:source ratio, coverage gaps, CI stage completeness.

## Off-limits files

Do not open these. Note their existence in the file inventory but never read or quote their contents:

- environment files (`.env`, `.env.<variant>`)
- private keys and certs (`*.key`, `*.pem`, `*.p12`)
- credential blobs (`credentials.json`, `service-account.json`)

## Ignore-file Awareness

Before any Glob or Grep, read `{{project_root}}/.gitignore` and
`{{project_root}}/.aiexclude` if they exist. Treat every non-comment,
non-negation pattern as an additional excluded path: skip those files and
directories entirely, do not Read them, and filter them out of pattern search
results. `scan.js` applies these patterns automatically. Skip negation (`!`)
lines.

## Exploration

Use your native file tools (Read, Glob, Grep) plus the `scan.js` helper for aggregations.

### Test framework detection
Read if present: `jest.config*`, `vitest.config*`, `pytest.ini`, `setup.cfg`, `pyproject.toml`, `.rspec`, `Cargo.toml`. Grep each for `test|jest|pytest|mocha|vitest|rspec` (case-insensitive).

### Test file count
```
node "{{project_root}}/_Sprintpilot/scripts/scan.js" files --include "*.test.*,*.spec.*,test_*,*_test.*" --root "{{project_root}}" --count
```

### Source file count
```
node "{{project_root}}/_Sprintpilot/scripts/scan.js" files --include "*.ts,*.js,*.py,*.go,*.rs,*.java,*.sql,*.sps,*.spb,*.xml,*.sh,*.c,*.h,*.cpp,*.hpp,*.cc,*.cxx,*.hxx" --exclude "**/test/**,**/tests/**,**/__tests__/**,**/spec/**,*.test.*,*.spec.*,*_test.*,test_*" --root "{{project_root}}" --count
```
(The `scan.js` helper automatically also excludes `node_modules`, `.git`, `vendor`, `dist`, `build`, etc.)

### Test types present
Use Glob for `**/e2e/**`, `**/integration/**`, `**/unit/**`, `*.e2e.*`, `*.integration.*`. First ~10 hits are enough.

### CI/CD configuration
Read if present: `.github/workflows/*.yml` (use Glob to list them first), `.gitlab-ci.yml`, `Jenkinsfile`, `azure-pipelines.yml`, `.circleci/config.yml`, `.travis.yml`. 80 lines each is usually enough.

### Linting & formatting config
Use Glob to list: `.eslintrc*`, `.prettierrc*`, `.editorconfig`, `.rubocop.yml`, `.flake8`, `pyproject.toml`, `rustfmt.toml`, `.golangci.yml`, `biome.json`, `.sqlfluff*`.

### Code metrics (total LOC)
```
node "{{project_root}}/_Sprintpilot/scripts/scan.js" loc --include "*.ts,*.js,*.py,*.go,*.rs,*.java,*.sql,*.sps,*.spb,*.xml,*.sh,*.c,*.h,*.cpp,*.hpp,*.cc,*.cxx,*.hxx" --root "{{project_root}}"
```
Output is tab-separated `<total-lines>\t<file-count>`.

### Largest files (complexity hotspots)
```
node "{{project_root}}/_Sprintpilot/scripts/scan.js" largest --include "*.ts,*.js,*.py,*.sql,*.sps,*.spb,*.sh,*.c,*.h,*.cpp,*.hpp,*.cc,*.cxx,*.hxx" --root "{{project_root}}" --limit 10
```
Output: `<lines>\t<path>`, descending.

### Coverage
Read `.nycrc`, `.istanbul.yml`, `jest.config*`, `vitest.config*` if present and grep for `cover`. Use Glob to check for `coverage/`, `htmlcov/`, `.coverage` directories/files.

## Downstream Consumers

| Consumer | What they need |
|----------|---------------|
| `sprintpilot-assess` | Test coverage baseline, CI maturity for debt classification |
| `sprintpilot-migrate` (Test Parity agent) | Test inventory for migration mapping |
| `bmad-testarch-test-design` | Current test landscape for test strategy planning |
| `bmad-sprint-planning` | Quality constraints for story estimation |

## Output Format

Write to `{{output_file}}`:

```markdown
# Quality Analysis

## Test Coverage
| Metric | Value | Evidence |
|--------|-------|----------|
| Test framework | Jest 29.7 | jest.config.ts:1 |
| Test files | 45 | scan.js files --include ... --count |
| Source files | 120 | scan.js files --include ... --count |
| Test:Source ratio | 1:2.7 | Calculated |
| Test types present | unit, integration | directory structure |
| Test types missing | e2e, snapshot | No e2e/ directory found |

### Test Distribution
| Type | Count | Location | Quality |
|------|-------|----------|---------|
| Unit | 35 | tests/unit/ | Good — isolated, fast |
| Integration | 10 | tests/integration/ | Sparse — missing API tests |
| E2E | 0 | — | Gap: no browser tests |

### Coverage Gaps
- Controllers: no unit tests (src/api/ has 12 files, 0 test files)
- Error paths: tests only cover happy path in 8 of 10 services
- Evidence: [specific file paths]

## CI/CD Pipeline
| Stage | Tool | Config | Status |
|-------|------|--------|--------|
| Lint | ESLint | .github/workflows/ci.yml:15 | Active |
| Unit tests | Jest | .github/workflows/ci.yml:22 | Active |
| Build | tsc | .github/workflows/ci.yml:30 | Active |
| Deploy | — | — | Missing |

### CI Quality Assessment
- Pipeline runs on: push to main, PRs
- Missing: integration tests in CI, coverage reporting, deployment stage
- Evidence: .github/workflows/ci.yml

## Code Conventions
| Tool | Config | Enforced in CI? | Evidence |
|------|--------|----------------|----------|
| ESLint | .eslintrc.json | Yes | ci.yml:15 |
| Prettier | .prettierrc | Yes | ci.yml:12 |
| EditorConfig | .editorconfig | N/A | Present |

### Naming Conventions (observed)
| Context | Convention | Consistency | Evidence |
|---------|-----------|-------------|----------|
| Files | kebab-case | 95% | user-service.ts, auth-controller.ts |
| Classes | PascalCase | 100% | UserService, AuthController |
| Functions | camelCase | 98% | getUserById, createSession |
| DB columns | snake_case | 100% | created_at, user_id |

## Code Metrics
| Metric | Value |
|--------|-------|
| Total LOC (approx) | 15,200 |
| Largest file | src/services/reportService.ts (850 lines) |
| Average file size | ~127 lines |
| Files > 500 lines | 3 (complexity risk) |

## Documentation
| Type | Status | Location |
|------|--------|----------|
| README | Good — setup + API docs | README.md |
| API docs | OpenAPI spec | docs/openapi.yaml |
| Inline docs | Sparse — JSDoc on public APIs only | Throughout |
| Architecture | Missing | — |

## Dependency Health
| Metric | Value | Evidence |
|--------|-------|----------|
| Lockfile | Present (package-lock.json) | Root directory |
| Last lockfile update | 2026-02-15 | git log |
| Dependency count | 45 direct, 120 dev | package.json |

## Key Files Examined
[List all files read]
```