diff --git a/.kiro/specs/admin-page-overhaul/tasks.md b/.kiro/specs/admin-page-overhaul/tasks.md
index c73c3b0..fe78c43 100644
--- a/.kiro/specs/admin-page-overhaul/tasks.md
+++ b/.kiro/specs/admin-page-overhaul/tasks.md
@@ -6,8 +6,8 @@ Replace the current inline `UserManagement` modal rendering on the admin page wi
## Tasks
-- [ ] 1. Create AdminPage component with page header and tab navigation
- - [ ] 1.1 Create `frontend/src/components/pages/AdminPage.js` with the page shell
+- [x] 1. Create AdminPage component with page header and tab navigation
+ - [x] 1.1 Create `frontend/src/components/pages/AdminPage.js` with the page shell
- Import React, useState, useAuth from AuthContext, and lucide-react icons (Shield, Clock, Activity)
- Define `API_BASE` constant matching project convention
- Define `TABS` array: `[{ id: 'users', label: 'User Management', icon: Shield }, { id: 'audit', label: 'Audit Log', icon: Clock }, { id: 'system', label: 'System Info', icon: Activity }]`
@@ -17,15 +17,15 @@ Replace the current inline `UserManagement` modal rendering on the admin page wi
- Conditionally render placeholder `
` for each panel based on `activeTab`
- _Requirements: 1.1, 1.2, 1.3, 2.1, 2.2, 2.3, 2.4_
- - [ ] 1.2 Integrate AdminPage into App.js
+ - [x] 1.2 Integrate AdminPage into App.js
- Import `AdminPage` from `./components/pages/AdminPage`
- Replace the existing `{currentPage === 'admin' && isAdmin() && (
setCurrentPage('home')} />
)}` block with `{currentPage === 'admin' && isAdmin() && }`
- Add non-admin redirect: `{currentPage === 'admin' && !isAdmin() && setCurrentPage('home')}` (or useEffect equivalent)
- Keep existing `{showUserManagement && }` and `{showAuditLog && }` modal triggers unchanged
- _Requirements: 1.2, 6.1, 6.2, 6.3, 6.4_
-- [ ] 2. Implement UserManagementPanel
- - [ ] 2.1 Build the themed user table and group badges
+- [-] 2. Implement UserManagementPanel
+ - [x] 2.1 Build the themed user table and group badges
- Define `GROUP_BADGE_THEMED` map with themed colors: Admin → danger, Standard_User → accent, Leadership → warning, Read_Only → muted
- Fetch users from `GET /api/users` with `credentials: 'include'` on panel mount
- Render user table with columns: username, email, group, active status, last login
@@ -35,7 +35,7 @@ Replace the current inline `UserManagement` modal rendering on the admin page wi
- Display error banner with `--intel-danger` styling on fetch failure
- _Requirements: 3.1, 3.2, 3.3, 7.1, 7.2_
- - [ ] 2.2 Implement inline add/edit form and CRUD operations
+ - [x] 2.2 Implement inline add/edit form and CRUD operations
- Add "Add User" button styled with `intel-button` primary variant
- Show inline form with `intel-input` styled fields for username, email, password, and group dropdown
- On edit action: populate form with selected user's data (username, email, group; password blank)
@@ -47,7 +47,7 @@ Replace the current inline `UserManagement` modal rendering on the admin page wi
- Display success toast with `--intel-success` color, auto-dismiss after 2 seconds
- _Requirements: 3.4, 3.5, 3.6, 3.7, 3.8, 3.9, 7.3_
- - [ ]* 2.3 Write property test: Group badge color mapping is total and correct
+ - [ ] 2.3 Write property test: Group badge color mapping is total and correct
- **Property 1: Group badge color mapping is total and correct**
- Install `fast-check` as a dev dependency in `frontend/`
- Create test file `frontend/src/components/pages/__tests__/AdminPage.property.test.js`
@@ -56,14 +56,14 @@ Replace the current inline `UserManagement` modal rendering on the admin page wi
- Use `fc.assert(property, { numRuns: 100 })`
- **Validates: Requirements 3.3**
- - [ ]* 2.4 Write property test: Edit form population preserves user data
+ - [ ] 2.4 Write property test: Edit form population preserves user data
- **Property 2: Edit form population preserves user data**
- Generate random user objects with arbitrary username, email, and group values
- Verify that populating the edit form results in `formData.username === user.username`, `formData.email === user.email`, `formData.group === user.group`, and `formData.password === ''`
- Use `fc.assert(property, { numRuns: 100 })`
- **Validates: Requirements 3.5**
- - [ ]* 2.5 Write property test: Self-modification prevention
+ - [ ] 2.5 Write property test: Self-modification prevention
- **Property 3: Self-modification prevention**
- Generate random user lists that include a user matching the current admin's ID
- Verify the admin's own row has group dropdown disabled and active toggle disabled
@@ -71,11 +71,11 @@ Replace the current inline `UserManagement` modal rendering on the admin page wi
- Use `fc.assert(property, { numRuns: 100 })`
- **Validates: Requirements 3.8**
-- [ ] 3. Checkpoint — Verify user management panel
+- [x] 3. Checkpoint — Verify user management panel
- Ensure all tests pass, ask the user if questions arise.
-- [ ] 4. Implement AuditLogPanel
- - [ ] 4.1 Build the themed audit log table with action badges and filters
+- [-] 4. Implement AuditLogPanel
+ - [x] 4.1 Build the themed audit log table with action badges and filters
- Define `ACTION_BADGE_THEMED` map with themed colors: login/success → green, delete → danger, create → accent, update → warning, default → muted
- Fetch audit logs from `GET /api/audit-logs?page=1&limit=25` with `credentials: 'include'` on panel mount
- Fetch action types from `GET /api/audit-logs/actions` for the action filter dropdown
@@ -87,7 +87,7 @@ Replace the current inline `UserManagement` modal rendering on the admin page wi
- Display "No audit log entries found" message with `--text-muted` color when results are empty
- _Requirements: 4.1, 4.2, 4.3, 4.4, 4.9, 4.10, 7.1, 7.2_
- - [ ] 4.2 Implement filter controls and pagination
+ - [x] 4.2 Implement filter controls and pagination
- Render filter bar with: username text input, action type dropdown, entity type dropdown, start date picker, end date picker
- Style all filter controls with `intel-input` and `intel-button` components
- On filter apply: re-fetch audit logs from page 1 with selected filter parameters
@@ -95,22 +95,22 @@ Replace the current inline `UserManagement` modal rendering on the admin page wi
- On page change: fetch the requested page
- _Requirements: 4.5, 4.6, 4.7, 4.8_
- - [ ]* 4.3 Write property test: Action badge color mapping is total and correct
+ - [ ] 4.3 Write property test: Action badge color mapping is total and correct
- **Property 4: Action badge color mapping is total and correct**
- Generate random action strings including all known actions and arbitrary unknown strings
- Verify the badge function returns correct themed colors for known actions and default muted styling for unknown actions
- Use `fc.assert(property, { numRuns: 100 })`
- **Validates: Requirements 4.4**
- - [ ]* 4.4 Write property test: Applying filters resets pagination to page 1
+ - [ ] 4.4 Write property test: Applying filters resets pagination to page 1
- **Property 5: Applying filters resets pagination to page 1**
- Generate random filter combinations (username text, action type, entity type, start date, end date) and random current page numbers
- Verify that applying filters results in a fetch call with `page=1`
- Use `fc.assert(property, { numRuns: 100 })`
- **Validates: Requirements 4.7**
-- [ ] 5. Implement SystemInfoPanel
- - [ ] 5.1 Build stat cards and recent activity list
+- [-] 5. Implement SystemInfoPanel
+ - [x] 5.1 Build stat cards and recent activity list
- Fetch users from `GET /api/users` and recent audit logs from `GET /api/audit-logs?limit=10&page=1` on panel mount
- Compute derived stats: total users (`users.length`), active users (`users.filter(u => u.is_active)`), recent logins (users with `last_login` within last 7 days), total audit entries (from pagination.total)
- Render four stat cards using the `stat-card` pattern with accent-colored top bar and hover lift effect
@@ -119,25 +119,25 @@ Replace the current inline `UserManagement` modal rendering on the admin page wi
- Display loading spinner while fetching
- _Requirements: 5.1, 5.2, 5.3, 5.4, 5.5, 7.1_
- - [ ]* 5.2 Write property test: Recent login count computation
+ - [ ] 5.2 Write property test: Recent login count computation
- **Property 6: Recent login count computation**
- Generate random user lists with random `last_login` timestamps (including null values)
- Verify the computed "recent logins" count equals the number of users whose `last_login` is non-null and falls within the last 7 days
- Use `fc.assert(property, { numRuns: 100 })`
- **Validates: Requirements 5.1**
-- [ ] 6. Checkpoint — Verify all panels and integration
+- [x] 6. Checkpoint — Verify all panels and integration
- Ensure all tests pass, ask the user if questions arise.
-- [ ] 7. Access control and final wiring
- - [ ] 7.1 Verify access control integration
+- [-] 7. Access control and final wiring
+ - [x] 7.1 Verify access control integration
- Confirm `AdminPage` reads auth context via `useAuth()` and only renders content for Admin-group users
- Confirm `App.js` redirects non-admin users to home when `currentPage === 'admin'`
- Confirm `NavDrawer` continues to show "Admin Panel" only for Admin-group users (no changes needed — verify existing behavior)
- Confirm `UserMenu` quick-access links ("Manage Users", "Audit Log") continue to open existing modal components (no changes needed — verify existing behavior)
- _Requirements: 6.1, 6.2, 6.3, 6.4_
- - [ ]* 7.2 Write property test: Admin-only access control
+ - [ ] 7.2 Write property test: Admin-only access control
- **Property 7: Admin-only access control**
- Generate random user objects with random group values
- Verify admin page content renders if and only if `user.group === 'Admin'`
@@ -145,7 +145,7 @@ Replace the current inline `UserManagement` modal rendering on the admin page wi
- Use `fc.assert(property, { numRuns: 100 })`
- **Validates: Requirements 6.1, 6.2**
-- [ ] 8. Final checkpoint — Ensure all tests pass
+- [x] 8. Final checkpoint — Ensure all tests pass
- Ensure all tests pass, ask the user if questions arise.
## Notes
diff --git a/backend/routes/compliance.js b/backend/routes/compliance.js
index d61de7a..02c3614 100644
--- a/backend/routes/compliance.js
+++ b/backend/routes/compliance.js
@@ -891,6 +891,68 @@ function createComplianceRouter(db, upload, requireAuth, requireGroup) {
}
});
+ // -----------------------------------------------------------------------
+ // DELETE /notes/:id
+ // Delete a note (or all notes in the same group_id) by note ID.
+ // Only the note author or an Admin can delete.
+ //
+ // Params: id — note row ID
+ // Query: ?group=true — delete all notes sharing the same group_id
+ // Response: { deleted: number }
+ // -----------------------------------------------------------------------
+ router.delete('/notes/:id', requireGroup('Admin', 'Standard_User'), async (req, res) => {
+ const noteId = parseInt(req.params.id, 10);
+ if (isNaN(noteId)) return res.status(400).json({ error: 'Invalid note ID' });
+
+ const deleteGroup = req.query.group === 'true';
+
+ try {
+ // Fetch the note to verify ownership
+ const note = await dbGet(db,
+ `SELECT id, hostname, metric_id, note, group_id, created_by FROM compliance_notes WHERE id = ?`,
+ [noteId]
+ );
+ if (!note) return res.status(404).json({ error: 'Note not found' });
+
+ // Only the author or an Admin can delete
+ const isAuthor = req.user && String(req.user.id) === String(note.created_by);
+ const isAdminUser = req.user && req.user.group === 'Admin';
+ if (!isAuthor && !isAdminUser) {
+ return res.status(403).json({ error: 'You can only delete your own notes' });
+ }
+
+ let deleted = 0;
+ if (deleteGroup && note.group_id) {
+ const result = await dbRun(db,
+ `DELETE FROM compliance_notes WHERE group_id = ?`,
+ [note.group_id]
+ );
+ deleted = result.changes || 0;
+ } else {
+ const result = await dbRun(db,
+ `DELETE FROM compliance_notes WHERE id = ?`,
+ [noteId]
+ );
+ deleted = result.changes || 0;
+ }
+
+ logAudit(db, {
+ userId: req.user.id,
+ username: req.user.username,
+ action: 'compliance_note_delete',
+ entityType: 'compliance_note',
+ entityId: String(noteId),
+ details: JSON.stringify({ hostname: note.hostname, group_id: note.group_id, deleted_count: deleted }),
+ ipAddress: req.ip,
+ });
+
+ res.json({ deleted });
+ } catch (err) {
+ console.error('[Compliance] DELETE /notes error:', err.message);
+ res.status(500).json({ error: 'Failed to delete note' });
+ }
+ });
+
// -----------------------------------------------------------------------
// GET /trends
// Per-upload active totals + per-team counts for time-series charts.
diff --git a/frontend/src/App.js b/frontend/src/App.js
index b75f9ab..8f9d068 100644
--- a/frontend/src/App.js
+++ b/frontend/src/App.js
@@ -12,6 +12,7 @@ import VulnerabilityTriagePage from './components/pages/ReportingPage';
import KnowledgeBasePage from './components/pages/KnowledgeBasePage';
import ExportsPage from './components/pages/ExportsPage';
import CompliancePage from './components/pages/CompliancePage';
+import AdminPage from './components/pages/AdminPage';
import ArchiveSummaryBar from './components/pages/ArchiveSummaryBar';
import './App.css';
@@ -1012,11 +1013,8 @@ export default function App() {
{currentPage === 'compliance' && }
{currentPage === 'knowledge-base' && }
{currentPage === 'exports' && }
- {currentPage === 'admin' && isAdmin() && (
-