alexei.dolgolyov
f2871319cb
Lint & Test / test (push) Failing after 48s
Security: tighten CORS defaults, add webhook rate limiting, fix XSS in automations, guard WebSocket JSON.parse, validate ADB address input, seal debug exception leak, URL-encode WS tokens, CSS.escape in selectors. Code quality: add Pydantic models for brightness/power endpoints, fix thread safety and name uniqueness in DeviceStore, immutable update pattern, split 6 oversized files into 16 focused modules, enable TypeScript strictNullChecks (741→102 errors), type state variables, add dom-utils helper, migrate 3 modules from inline onclick to event delegation, ProcessorDependencies dataclass. Performance: async store saves, health endpoint log level, command palette debounce, optimized entity-events comparison, fix service worker precache list. Testing: expand from 45 to 293 passing tests — add store tests (141), route tests (25), core logic tests (42), E2E flow tests (33), organize into tests/api/, tests/storage/, tests/core/, tests/e2e/. DevOps: CI test pipeline, pre-commit config, Dockerfile multi-stage build with non-root user and health check, docker-compose improvements, version bump to 0.2.0. Docs: rewrite CLAUDE.md (202→56 lines), server/CLAUDE.md (212→76), create contexts/server-operations.md, fix .js→.ts references, fix env var prefix in README, rewrite INSTALLATION.md, add CONTRIBUTING.md and .env.example.
85 lines
2.9 KiB
JavaScript
85 lines
2.9 KiB
JavaScript
/**
|
|
* Service Worker for LED Grab PWA.
|
|
*
|
|
* Strategy:
|
|
* - Static assets (/static/): stale-while-revalidate
|
|
* - API / config requests: network-only (device control must be live)
|
|
* - Navigation: network-first with offline fallback
|
|
*/
|
|
|
|
const CACHE_NAME = 'ledgrab-v33';
|
|
|
|
// Only pre-cache static assets (no auth required).
|
|
// Do NOT pre-cache '/' — it requires API key auth and would cache an error page.
|
|
const PRECACHE_URLS = [
|
|
'/static/dist/app.bundle.css',
|
|
'/static/dist/app.bundle.js',
|
|
'/static/icons/icon-192.png',
|
|
'/static/icons/icon-512.png',
|
|
];
|
|
|
|
// Install: pre-cache core shell
|
|
self.addEventListener('install', (event) => {
|
|
event.waitUntil(
|
|
caches.open(CACHE_NAME)
|
|
.then((cache) => cache.addAll(PRECACHE_URLS))
|
|
.then(() => self.skipWaiting())
|
|
);
|
|
});
|
|
|
|
// Activate: clean old caches
|
|
self.addEventListener('activate', (event) => {
|
|
event.waitUntil(
|
|
caches.keys()
|
|
.then((keys) => Promise.all(
|
|
keys.filter((k) => k !== CACHE_NAME).map((k) => caches.delete(k))
|
|
))
|
|
.then(() => self.clients.claim())
|
|
);
|
|
});
|
|
|
|
// Fetch handler
|
|
self.addEventListener('fetch', (event) => {
|
|
const url = new URL(event.request.url);
|
|
|
|
// API and config: always network (device control must be live)
|
|
if (url.pathname.startsWith('/api/') || url.pathname.startsWith('/config/')) {
|
|
return; // fall through to default network fetch
|
|
}
|
|
|
|
// Static assets: stale-while-revalidate
|
|
if (url.pathname.startsWith('/static/')) {
|
|
event.respondWith(
|
|
caches.open(CACHE_NAME).then((cache) =>
|
|
cache.match(event.request).then((cached) => {
|
|
const fetchPromise = fetch(event.request).then((response) => {
|
|
if (response.ok) {
|
|
cache.put(event.request, response.clone());
|
|
}
|
|
return response;
|
|
}).catch(() => cached);
|
|
|
|
return cached || fetchPromise;
|
|
})
|
|
)
|
|
);
|
|
return;
|
|
}
|
|
|
|
// Navigation: network-only (page requires auth, no useful offline fallback)
|
|
if (event.request.mode === 'navigate') {
|
|
event.respondWith(
|
|
fetch(event.request).catch(() =>
|
|
new Response(
|
|
'<html><body style="font-family:system-ui;text-align:center;padding:60px 20px;background:#1a1a1a;color:#ccc">' +
|
|
'<h2>LED Grab</h2><p>Cannot reach the server. Check that it is running and you are on the same network.</p>' +
|
|
'<button onclick="location.reload()" style="margin-top:20px;padding:10px 24px;border-radius:8px;border:none;background:#4CAF50;color:#fff;font-size:1rem;cursor:pointer">Retry</button>' +
|
|
'</body></html>',
|
|
{ status: 503, headers: { 'Content-Type': 'text/html' } }
|
|
)
|
|
)
|
|
);
|
|
return;
|
|
}
|
|
});
|