-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
area: frontendVue.js UIVue.js UIdocumentationImprovements or additions to documentationImprovements or additions to documentationpriority: mediumImportant but not urgentImportant but not urgentsprint-finalDocumentation wrap-upDocumentation wrap-up
Description
Context
docs/frontend/frontend-review.md (dated 2025-12-02) is severely outdated. An audit of the actual codebase reveals major discrepancies.
Audit Findings (2026-02-15)
Codebase Scale vs Review Claims
| Metric | Review Claims | Actual | Delta |
|---|---|---|---|
| Pages | 21 | 27 | +6 |
| Stores | 5 | 13 | +8 |
| Components | 14 | 44 | +30 |
Features Built Since Review
- Drift Detection: 3 pages (DriftDetectionPage, DriftSchedulesPage, DriftReportsPage)
- Notifications: 4 pages (Dashboard, Rules, History, Channels)
- Provisioning: 4 pages (Dashboard, Tasks, TaskDetail, Agents)
- Device Config: 2 pages (DeviceConfigPage, DeviceDetailPage)
- Templates: 2 pages (TemplatesPage, TemplateDetailPage)
Component Optimization Achievements
| Component | Before | After | Reduction |
|---|---|---|---|
| BackupManagement | 1625 lines | 1026 lines | -37% |
| GitOps | 1351 lines | 578 lines | -57% |
| Plugin | 1151 lines | 597 lines | -48% |
API Coverage Gap Analysis
Overall: 48/108 API functions actually used (44%)
| Module | Functions | Used | Coverage |
|---|---|---|---|
| configNew.ts | 12 | 12 | 100% |
| templates.ts | 6 | 5 | 83% |
| plugin.ts | 9 | 7 | 78% |
| devices.ts | 8 | 6 | 75% |
| notification.ts | 8 | 5 | 63% |
| schedule.ts | 8 | 5 | 63% |
| metrics.ts | 14 | 8 | 57% |
| typedConfig.ts | 8 | 3 | 38% |
| export.ts | 42 | 12 | 29% |
| bulk.ts | 4 | 1 | 25% |
| drift.ts | 11 | 2 | 18% |
| deviceConfig.ts | 11 | 2 | 18% |
| provisioning.ts | 7 | 0 | 0% |
| admin.ts | 1 | 0 | 0% |
Pattern: Pages and stores were scaffolded, API modules were created, but actual wiring is incomplete in many areas.
Related Issues Created
API integration completion issues have been created:
- Complete provisioning system API integration #128 - Provisioning API (0% → 100%)
- Complete drift detection API integration #129 - Drift Detection API (18% → 100%)
- Complete device config API integration #130 - Device Config API (18% → 100%)
- Complete export system API integration #131 - Export System API (29% → 100%)
- Complete device management CRUD #132 - Device Management CRUD (75% → 100%)
- Complete notification system API integration #133 - Notification API (63% → 100%)
- Complete metrics advanced features #134 - Metrics Advanced Features (57% → 100%)
- Complete typed config integration #135 - Typed Config (38% → 100%)
- Complete bulk operations integration #136 - Bulk Operations (25% → 100%)
- Add capability-specific config UI #137 - Capability Config UI (new API module)
- Wire admin key rotation #138 - Admin Key Rotation (0% → 100%)
- Complete export/import stub pages #139 - Export/Import Stub Pages
Success Criteria
- Rewrite frontend-review.md to reflect actual current state
- Update API coverage metrics with data from this audit
- Remove stale task sections referencing completed tasks (251-411)
- Document the 12 API integration issues as the current Phase 7 work items
- Update Phase 7 progress indicators based on actual coverage (44%)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
area: frontendVue.js UIVue.js UIdocumentationImprovements or additions to documentationImprovements or additions to documentationpriority: mediumImportant but not urgentImportant but not urgentsprint-finalDocumentation wrap-upDocumentation wrap-up