25 lines
1.7 KiB
Markdown
25 lines
1.7 KiB
Markdown
|
# Implementation: Wire DashboardConfig + Add Caching + Add Size Limits
|
||
|
|
|
||
|
|
## Summary
|
||
|
|
- Added `GET /api/config` endpoint returning `config.to_dict()`
|
||
|
|
- Added `PUT /api/config` endpoint accepting JSON body, validating via `DashboardConfig.validate()`, and persisting to `dashboard-config.json`
|
||
|
|
- Added server-side task cache with 1-second TTL (`_get_cached_tasks`, `_invalidate_task_cache`)
|
||
|
|
- `_serve_tasks` and `_serve_task` now use cached task list instead of calling `discover_tasks()` on every request
|
||
|
|
- `_serve_review_summary` uses cached task list instead of iterating filesystem
|
||
|
|
- Writing a review (`POST /api/task/{name}/review`) invalidates cache
|
||
|
|
- R3 (content-length bounds) already implemented in harden-dashboard-security task
|
||
|
|
- Dashboard.js now fetches `/api/config` on init and applies `theme`, `default_view`, `column_width`, `show_timelines`, `auto_refresh_interval`
|
||
|
|
- Removed hardcoded `setInterval(refreshData, 2000)` — refresh interval is now config-driven
|
||
|
|
- CORS headers updated to include `PUT` method
|
||
|
|
|
||
|
|
## Changes
|
||
|
|
- `automaton/dashboard/ui/app.py`: Added `_task_cache`, `_get_cached_tasks`, `_invalidate_task_cache`, `_serve_config`, `_handle_config_update`, `do_PUT`; wired `DashboardHandler.config` class attribute; `_serve_tasks`, `_serve_task`, `_serve_review_summary` use cache; cache invalidation in `_handle_review`; CORS updated with PUT
|
||
|
|
- `automaton/dashboard/html/dashboard.js`: Added `fetchConfig`, `applyConfig`; updated `startAutoRefresh` to accept interval; `DOMContentLoaded` now async and fetches config before starting refresh
|
||
|
|
- `tests/test_app.py`: Added `TestTaskCache` (3 tests) and `TestConfigEndpoint` (4 tests)
|
||
|
|
|
||
|
|
## Test Results
|
||
|
|
126 passed in 0.09s
|
||
|
|
Dashboard starts (port conflict on 8080 is environmental, not a bug)
|
||
|
|
|
||
|
|
## Blockers
|
||
|
|
None
|