Improvements and bug fixes

express/backend/src/api/adapter.ts

@@ -1,13 +1,56 @@
 import { Router } from "express";
 import { dbConnect, unescapeObjectKeys } from "../db/utils";
-import { AdapterStats } from "../global/adapter-stats";
+import { AdapterStats, AdapterVersions } from "../global/adapter-stats";
 import { Statistics } from "../global/iobroker";
 
 const router = Router();
 
-router.get("/api/adapter/:name/stats", async function (req, res) {
+router.get("/api/adapter/:name/stats/now", async function (req, res) {
 	try {
 		const { name } = req.params;
+		if (!isValidAdapterName(name)) {
+			res.status(404).send("Adapter not found");
+			return;
+		}
+
+		const db = await dbConnect();
+		const rawStatistics = db.rawStatistics();
+
+		const stats = await rawStatistics
+			.find()
+			.project<Statistics>({
+				adapters: { [name]: 1 },
+				versions: { [name]: 1 },
+				date: 1,
+				_id: 0,
+			})
+			.sort({ date: -1 })
+			.limit(1)
+			.toArray();
+		if (stats.length === 0) {
+			res.status(404).send("Adapter not found");
+			return;
+		}
+
+		const stat = unescapeObjectKeys(stats[0]);
+		const versions: AdapterVersions = {
+			total: stat.adapters[name] ?? 0,
+			versions: stat.versions[name] ?? {},
+		};
+		res.send(versions);
+	} catch (error: any) {
+		console.error(error);
+		res.status(500).send(error.message || error);
+	}
+});
+
+router.get("/api/adapter/:name/stats/history", async function (req, res) {
+	try {
+		const { name } = req.params;
+		if (!isValidAdapterName(name)) {
+			res.status(404).send("Adapter not found");
+			return;
+		}
 		const db = await dbConnect();
 		const rawStatistics = db.rawStatistics();
 		const repoAdapters = db.repoAdapters();
@@ -62,7 +105,7 @@
 
 		console.log(result);
 		if (Object.keys(result.counts).length === 0) {
-			res.status(404).send(`Adapter ${name} not found`);
+			res.status(404).send("Adapter not found");
 			return;
 		}
 
@@ -73,4 +116,14 @@
 	}
 });
 
+function isValidAdapterName(name: string) {
+	const forbiddenChars = /[^a-z0-9\-_]/g;
+	if (forbiddenChars.test(name)) {
+		return false;
+	}
+
+	// the name must start with a letter
+	return /^[a-z]/.test(name);
+}
+
 export default router;

express/backend/src/cron.ts

@@ -64,12 +64,6 @@ async function collectRepos(): Promise<void> {
 		]);
 		const collection = db.repoAdapters();
 
-		// remove "stale" entries
-		const { deletedCount } = await collection.deleteMany({
-			source: { $exists: false },
-		});
-		console.log(`Deleted ${deletedCount || 0} stale entries`);
-
 		await Promise.all([
 			addRepoAdapters(collection, latest, "latest"),
 			addRepoAdapters(collection, stable, "stable"),

express/frontend/src/components/dashboard/LoginButton.tsx

@@ -1,7 +1,7 @@
 import { useUserContext } from "../../contexts/UserContext";
 import { CardButton } from "../CardButton";
 
-export function LoginButton() {
+export function LoginButton({ variant }: { variant?: string }) {
 	const { login } = useUserContext();
-	return <CardButton text="Login" onClick={login} />;
+	return <CardButton text="Login" onClick={login} variant={variant} />;
 }

express/frontend/src/contexts/UserContext.tsx

@@ -27,11 +27,17 @@ export function useUserContext() {
 	return context;
 }
 
+export class UserTokenMissingError extends Error {
+	constructor() {
+		super("User token missing");
+	}
+}
+
 export function useUserToken() {
 	const { user } = useUserContext();
 	const token = user?.token;
 	if (!token) {
-		throw new Error("User token missing");
+		throw new UserTokenMissingError();
 	}
 	return token;
 }

express/frontend/src/lib/ioBroker.ts

@@ -1,5 +1,8 @@
 import axios from "axios";
-import { AdapterStats } from "../../../backend/src/global/adapter-stats";
+import {
+	AdapterStats,
+	AdapterVersions,
+} from "../../../backend/src/global/adapter-stats";
 import {
 	AdapterRatings,
 	AllRatings,
@@ -180,9 +183,16 @@ export const getWeblateAdapterComponents = AsyncCache.of(async () => {
 	return components;
 });
 
-export async function getStatistics(adapterName: string) {
+export async function getCurrentVersions(adapterName: string) {
+	const result = await axios.get<AdapterVersions>(
+		getApiUrl(`adapter/${uc(adapterName)}/stats/now`),
+	);
+	return result.data;
+}
+
+export async function getStatisticsHistory(adapterName: string) {
 	const result = await axios.get<AdapterStats>(
-		getApiUrl(`adapter/${uc(adapterName)}/stats`),
+		getApiUrl(`adapter/${uc(adapterName)}/stats/history`),
 	);
 	return result.data;
 }

express/frontend/src/router.tsx

@@ -1,14 +1,16 @@
-import { createBrowserRouter } from "react-router-dom";
+import { Paper, Typography } from "@mui/material";
+import { createBrowserRouter, useRouteError } from "react-router-dom";
 import { App } from "./App";
 import { Dashboard } from "./components/dashboard/Dashboard";
-import { UserProvider } from "./contexts/UserContext";
+import { LoginButton } from "./components/dashboard/LoginButton";
+import { UserProvider, UserTokenMissingError } from "./contexts/UserContext";
 import { AdapterDashboard } from "./tools/adapter/AdapterDashboard";
 import { AdapterDetails } from "./tools/adapter/AdapterDetails";
 import { AdapterRatings } from "./tools/adapter/AdapterRatings";
-import { AdapterStatistics } from "./tools/adapter/AdapterStatistics";
 import { CreateReleaseDialog } from "./tools/adapter/releases/CreateReleaseDialog";
 import { Releases } from "./tools/adapter/releases/Releases";
 import { UpdateRepositoriesDialog } from "./tools/adapter/releases/UpdateRepositoriesDialog";
+import { Statistics } from "./tools/adapter/statistics/Statistics";
 import { AdapterCheck } from "./tools/AdapterCheck";
 import { StartCreateAdapter } from "./tools/create-adapter/StartCreateAdapter";
 import { Wizard } from "./tools/create-adapter/Wizard";
@@ -24,66 +26,89 @@ export const router = createBrowserRouter([
 
 		children: [
 			{
-				path: "/create-adapter",
+				path: "/",
+				errorElement: <ErrorBoundary />,
 				children: [
 					{
-						index: true,
-						element: <StartCreateAdapter />,
+						path: "/create-adapter",
+						children: [
+							{
+								index: true,
+								element: <StartCreateAdapter />,
+							},
+							{
+								path: "wizard",
+								element: <Wizard />,
+							},
+						],
 					},
 					{
-						path: "wizard",
-						element: <Wizard />,
+						path: "/adapter-check",
+						element: <AdapterCheck />,
 					},
-				],
-			},
-			{
-				path: "/adapter-check",
-				element: <AdapterCheck />,
-			},
-			{
-				path: "/adapter/:name",
-				element: <AdapterDetails />,
-				children: [
 					{
-						index: true,
-						element: <AdapterDashboard />,
-					},
-					{
-						path: "releases",
-						element: <Releases />,
+						path: "/adapter/:name",
+						element: <AdapterDetails />,
 						children: [
 							{
-								path: "~release",
-								element: <CreateReleaseDialog />,
+								index: true,
+								element: <AdapterDashboard />,
 							},
 							{
-								path: "~to-latest",
-								element: (
-									<UpdateRepositoriesDialog action="to-latest" />
-								),
+								path: "releases",
+								element: <Releases />,
+								children: [
+									{
+										path: "~release",
+										element: <CreateReleaseDialog />,
+									},
+									{
+										path: "~to-latest",
+										element: (
+											<UpdateRepositoriesDialog action="to-latest" />
+										),
+									},
+									{
+										path: "~to-stable/:version",
+										element: (
+											<UpdateRepositoriesDialog action="to-stable" />
+										),
+									},
+								],
 							},
 							{
-								path: "~to-stable/:version",
-								element: (
-									<UpdateRepositoriesDialog action="to-stable" />
-								),
+								path: "statistics",
+								element: <Statistics />,
+							},
+							{
+								path: "ratings",
+								element: <AdapterRatings />,
 							},
 						],
 					},
 					{
-						path: "statistics",
-						element: <AdapterStatistics />,
-					},
-					{
-						path: "ratings",
-						element: <AdapterRatings />,
+						index: true,
+						element: <Dashboard />,
 					},
 				],
 			},
-			{
-				index: true,
-				element: <Dashboard />,
-			},
 		],
 	},
 ]);
+
+function ErrorBoundary() {
+	let error = useRouteError();
+	if (error instanceof UserTokenMissingError) {
+		return (
+			<Paper sx={{ padding: 2 }}>
+				<Typography variant="h4">Not logged in</Typography>
+				<p>You need to be logged in to access this page.</p>
+				<p>
+					<LoginButton variant="contained" />
+				</p>
+			</Paper>
+		);
+	}
+
+	throw error;
+}

express/frontend/src/tools/AdapterCheck.tsx

@@ -26,7 +26,7 @@ import { useLocation } from "react-router-dom";
 import { useUserToken } from "../contexts/UserContext";
 import { checkAdapter, CheckResult, getMyAdapterRepos } from "../lib/ioBroker";
 
-const iconStyles = {
+export const iconStyles = {
 	check: {
 		color: "#00b200",
 	},

express/frontend/src/tools/adapter/AdapterDashboard.tsx

@@ -10,7 +10,11 @@ import { CardButton } from "../../components/CardButton";
 import { CardGrid, CardGridProps } from "../../components/dashboard/CardGrid";
 import { DashboardCardProps } from "../../components/dashboard/DashboardCard";
 import { useAdapter } from "../../contexts/AdapterContext";
-import { getAllRatings, getLatest, getStatistics } from "../../lib/ioBroker";
+import {
+	getAllRatings,
+	getCurrentVersions,
+	getLatest,
+} from "../../lib/ioBroker";
 
 const CATEGORY_GENERAL = "General";
 const CATEGORY_FEATURES = "Features";
@@ -28,18 +32,11 @@ export function AdapterDashboard() {
 	useEffect(() => {
 		setCategories(EMPTY_CARDS);
 		const loadCards = async () => {
-			const [latest, ratings] = await Promise.all([
+			const [latest, ratings, versions] = await Promise.all([
 				getLatest(),
 				getAllRatings(),
+				getCurrentVersions(name).catch(() => null),
 			]);
-			let hasStatistics = !!latest[name];
-			if (!hasStatistics) {
-				// check if statistics are available
-				try {
-					await getStatistics(name);
-					hasStatistics = true;
-				} catch {}
-			}
 			const generalCards: DashboardCardProps[] = [];
 			generalCards.push({
 				title: "Releases",
@@ -51,7 +48,7 @@ export function AdapterDashboard() {
 				to: "releases",
 				buttons: [<CardButton text="Manage" to="releases" />],
 			});
-			if (hasStatistics) {
+			if (latest[name] || versions?.total) {
 				generalCards.push({
 					title: "Statistics",
 					text: "Learn more about the usage and distribution of your adapter.",