Compare commits

...
Author SHA1 Message Date
gorkem-bwl 249e109b7b feat: add expandable location rows to ChecksTable
## Changes
- Add LocationResult type and locationResults field to client Check type
- Add expandable rows to ChecksTable showing per-location Globalping results (location name, status, response time, status code)
- Pass globalpingEnabled and locationLabels props from Details page to ChecksTable
- Add translation keys for expanded row columns

## Benefits
Users can now click a check row to see how each Globalping location
performed for that specific monitoring cycle, providing at-a-glance
multi-location visibility.
2026-02-20 08:20:25 -05:00
gorkem-bwl a23e4de1d3 refactor: wire backfill pipeline and remove per-location Check documents
## Changes
- Remove location field from CheckMetadata, MonitorStatusResponse, Check model metadata schema, and location index
- Remove location from checkService.buildCheck() and MongoChecksRepository toDocument/mapMetadata
- Rewrite SuperSimpleQueueHelper Step 2b: instead of creating separate Check documents per Globalping location, backfill locationResults into the same Check document after the local check is buffered
- Rewrite findLocationChecksByMonitorId to use $unwind on locationResults subdocument array instead of querying by metadata.location
- Remove metadata.location filter from findUptimeDateRangeChecks
- Pass locationResults through in toEntity

## Benefits
Single Check document per monitoring cycle now contains both local results
(top-level fields) and per-location Globalping results (locationResults array).
This eliminates document proliferation and simplifies queries.
2026-02-20 08:20:16 -05:00
10 changed files with 152 additions and 49 deletions

No files matched your search

@@ -1,5 +1,7 @@
import { Table, Pagination, StatusLabel } from "@/Components/v2/design-elements";
import Box from "@mui/material/Box";
import Stack from "@mui/material/Stack";
import Typography from "@mui/material/Typography";
import type { Header } from "@/Components/v2/design-elements/";
import type { Check } from "@/Types/Check";
@@ -8,6 +10,7 @@ import { useTranslation } from "react-i18next";
import { formatDateWithTz } from "@/Utils/TimeUtils";
import type { RootState } from "@/Types/state";
import { useSelector } from "react-redux";
import { useTheme } from "@mui/material/styles";
const getHeaders = (t: Function, uiTimezone: string) => {
const headers: Header<Check>[] = [
@@ -50,6 +53,8 @@ export const ChecksTable = ({
setPage,
rowsPerPage,
setRowsPerPage,
globalpingEnabled = false,
locationLabels = {},
}: {
checks: Check[];
count: number;
@@ -57,9 +62,12 @@ export const ChecksTable = ({
setPage: (page: number) => void;
rowsPerPage: number;
setRowsPerPage: (rowsPerPage: number) => void;
globalpingEnabled?: boolean;
locationLabels?: Record<string, string>;
}) => {
const navigate = useNavigate();
const { t } = useTranslation();
const theme = useTheme();
const uiTimezone = useSelector((state: RootState) => state.ui.timezone);
const headers = getHeaders(t, uiTimezone);
@@ -78,6 +86,87 @@ export const ChecksTable = ({
setRowsPerPage(value);
};
const renderExpandedContent = (row: Check) => {
if (!row.locationResults || row.locationResults.length === 0) {
return null;
}
return (
<Stack gap={theme.spacing(2)}>
<Stack
direction="row"
sx={{
px: theme.spacing(4),
py: theme.spacing(2),
borderBottom: `1px solid ${theme.palette.divider}`,
}}
>
<Typography
variant="body2"
fontWeight={600}
sx={{ flex: 2 }}
>
{t("pages.checks.expandedRow.location")}
</Typography>
<Typography
variant="body2"
fontWeight={600}
sx={{ flex: 1 }}
>
{t("common.table.headers.status")}
</Typography>
<Typography
variant="body2"
fontWeight={600}
sx={{ flex: 1 }}
>
{t("pages.checks.expandedRow.responseTime")}
</Typography>
<Typography
variant="body2"
fontWeight={600}
sx={{ flex: 1 }}
>
{t("pages.checks.expandedRow.statusCode")}
</Typography>
</Stack>
{row.locationResults.map((lr) => (
<Stack
key={lr.location}
direction="row"
sx={{
px: theme.spacing(4),
py: theme.spacing(1),
alignItems: "center",
}}
>
<Typography
variant="body2"
sx={{ flex: 2 }}
>
{locationLabels[lr.location] ?? lr.location}
</Typography>
<Box sx={{ flex: 1 }}>
<StatusLabel status={lr.status ? "up" : "down"} />
</Box>
<Typography
variant="body2"
sx={{ flex: 1 }}
>
{Math.round(lr.responseTime)} ms
</Typography>
<Typography
variant="body2"
sx={{ flex: 1 }}
>
{lr.statusCode || "N/A"}
</Typography>
</Stack>
))}
</Stack>
);
};
return (
<Box>
<Table
@@ -86,6 +175,8 @@ export const ChecksTable = ({
onRowClick={(row) => {
navigate(`/checks/${row.id}`);
}}
expandableRows={globalpingEnabled}
renderExpandedContent={globalpingEnabled ? renderExpandedContent : undefined}
/>
<Pagination
component="div"
@@ -245,6 +245,8 @@ const UptimeDetailsPage = () => {
setPage={setPage}
rowsPerPage={rowsPerPage}
setRowsPerPage={setRowsPerPage}
globalpingEnabled={monitor?.globalpingEnabled ?? false}
locationLabels={locationLabels}
/>
</BasePage>
);
+9
View File
@@ -117,6 +117,14 @@ export interface CheckTimings {
};
}
export interface LocationResult {
location: string;
status: boolean;
responseTime: number;
statusCode: number;
message: string;
}
export interface Check {
id: string;
metadata: CheckMetadata;
@@ -140,6 +148,7 @@ export interface Check {
seo?: number;
performance?: number;
audits?: CheckAudits;
locationResults?: LocationResult[];
__v: number;
createdAt: string;
updatedAt: string;
+5
View File
@@ -368,6 +368,11 @@
"dateTime": "Date & time",
"statusCode": "Status code"
}
},
"expandedRow": {
"location": "Location",
"responseTime": "Response time",
"statusCode": "Status code"
}
},
"common": {
-5
View File
@@ -20,7 +20,6 @@ type CheckMetadataDocument = Omit<CheckMetadata, "monitorId" | "teamId"> & {
monitorId: Types.ObjectId;
teamId: Types.ObjectId;
type: MonitorType;
location?: string;
};
type CheckDocumentBase = Omit<Check, "id" | "metadata" | "expiry" | "ackAt" | "createdAt" | "updatedAt"> & {
@@ -210,9 +209,6 @@ const metadataSchema = new Schema<CheckMetadataDocument>(
required: true,
index: true,
},
location: {
type: String,
},
},
{ _id: false }
);
@@ -319,7 +315,6 @@ CheckSchema.index({ createdAt: 1 });
CheckSchema.index({ "metadata.teamId": 1, createdAt: -1 });
CheckSchema.index({ "metadata.monitorId": 1, "metadata.type": 1, createdAt: -1 });
CheckSchema.index({ "metadata.teamId": 1, status: 1, createdAt: -1 });
CheckSchema.index({ "metadata.monitorId": 1, "metadata.location": 1, createdAt: -1 });
const CheckModel = model<CheckDocument>("Check", CheckSchema);
@@ -170,7 +170,6 @@ class MongoChecksRepository implements IChecksRepository {
monitorId: toStringId(metadata.monitorId),
teamId: toStringId(metadata.teamId),
type: metadata.type,
...(metadata.location ? { location: metadata.location } : {}),
});
return {
@@ -196,6 +195,7 @@ class MongoChecksRepository implements IChecksRepository {
seo: doc.seo,
performance: doc.performance,
audits: mapAudits(doc.audits),
locationResults: (doc as any).locationResults,
__v: doc.__v ?? 0,
createdAt: toDateString(doc.createdAt),
updatedAt: toDateString(doc.updatedAt),
@@ -221,7 +221,6 @@ class MongoChecksRepository implements IChecksRepository {
monitorId: new mongoose.Types.ObjectId(metadata.monitorId),
teamId: new mongoose.Types.ObjectId(metadata.teamId),
type: metadata.type,
...(metadata.location ? { location: metadata.location } : {}),
},
...rest,
} as unknown as CheckDocument;
@@ -429,24 +428,25 @@ class MongoChecksRepository implements IChecksRepository {
findLocationChecksByMonitorId = async (monitorId: string, startDate: Date, endDate: Date, dateString: string): Promise<Record<string, any>> => {
const monitorObjectId = new mongoose.Types.ObjectId(monitorId);
const matchStage = {
"metadata.monitorId": monitorObjectId,
"metadata.location": { $exists: true, $ne: null },
createdAt: { $gte: startDate, $lte: endDate },
};
const results = await CheckModel.aggregate([
{ $match: matchStage },
{ $sort: { createdAt: 1 } },
{
$match: {
"metadata.monitorId": monitorObjectId,
createdAt: { $gte: startDate, $lte: endDate },
locationResults: { $type: "array", $ne: [] },
},
},
{ $unwind: "$locationResults" },
{
$group: {
_id: {
location: "$metadata.location",
location: "$locationResults.location",
bucket: { $dateToString: { format: dateString, date: "$createdAt" } },
},
avgResponseTime: { $avg: "$responseTime" },
avgResponseTime: { $avg: "$locationResults.responseTime" },
totalChecks: { $sum: 1 },
upChecks: { $sum: { $cond: [{ $eq: ["$status", true] }, 1, 0] } },
upChecks: { $sum: { $cond: [{ $eq: ["$locationResults.status", true] }, 1, 0] } },
},
},
{ $sort: { "_id.bucket": 1 } },
@@ -490,7 +490,6 @@ class MongoChecksRepository implements IChecksRepository {
const matchStage: Record<string, any> = {
"metadata.monitorId": monitorObjectId,
createdAt: { $gte: startDate, $lte: endDate },
"metadata.location": { $exists: false },
};
const [result] = await CheckModel.aggregate([
{ $match: matchStage },
+1 -2
View File
@@ -37,7 +37,7 @@ class CheckService {
};
buildCheck = (statusResponse: MonitorStatusResponse<PageSpeedStatusPayload | HardwareStatusPayload | undefined>) => {
const { monitorId, teamId, type, status, responseTime, code, message, payload, timings, location } = statusResponse;
const { monitorId, teamId, type, status, responseTime, code, message, payload, timings } = statusResponse;
const check: Partial<Check> = {
id: new Types.ObjectId().toString(),
@@ -46,7 +46,6 @@ class CheckService {
monitorId,
teamId,
type,
...(location ? { location } : {}),
},
status,
statusCode: code,
@@ -127,33 +127,6 @@ class SuperSimpleQueueHelper {
throw new Error("No network response");
}
// Step 2b. Run Globalping checks (fire-and-forget, doesn't affect local check pipeline)
if (monitor.globalpingEnabled && (monitor.type === "http" || monitor.type === "ping")) {
this.globalpingService
.runChecks(monitor)
.then((locationStatuses) => {
for (const locStatus of locationStatuses) {
const locCheck = this.checkService.buildCheck(locStatus);
if (locCheck) {
this.buffer.addToBuffer(locCheck);
} else {
this.logger.warn({
message: `Failed to build Globalping check for monitor ${monitorId}, location: ${locStatus.location}`,
service: SERVICE_NAME,
method: "getMonitorJob",
});
}
}
})
.catch((err: any) => {
this.logger.warn({
message: `Globalping checks failed for monitor ${monitorId}: ${err.message}`,
service: SERVICE_NAME,
method: "getMonitorJob",
});
});
}
// Step 3. Build check
const check = await this.checkService.buildCheck(status);
if (!check) {
@@ -167,6 +140,38 @@ class SuperSimpleQueueHelper {
}
// Step 4. Add check to buffer
this.buffer.addToBuffer(check);
// Step 4b. Run Globalping checks and backfill into the same check document.
// Fire-and-forget: Globalping results are supplementary and must not block the
// local check pipeline. The dual-write strategy below is safe against mid-flush
// races because flushBuffer() replaces the buffer array reference atomically
// (this.buffer = []) — if findInBuffer returns a hit, the check is still in the
// current buffer and our mutation will be included in the upcoming flush; if it
// returns undefined the check has already been flushed, so we fall back to a
// direct DB update.
if (monitor.globalpingEnabled && (monitor.type === "http" || monitor.type === "ping") && check.id) {
const checkId = check.id;
this.globalpingService
.runChecks(monitor)
.then(async (locationResults) => {
if (locationResults.length > 0) {
const buffered = this.buffer.findInBuffer(checkId);
if (buffered) {
buffered.locationResults = locationResults;
} else {
await this.checksRepository.updateLocationResults(checkId, locationResults);
}
}
})
.catch((err: any) => {
this.logger.warn({
message: `Globalping checks failed for monitor ${monitorId}: ${err.message}`,
service: SERVICE_NAME,
method: "getMonitorJob",
});
});
}
// Step 5. Update monitor status
const statusChangeResult = await this.statusService.updateMonitorStatus(status, check);
-1
View File
@@ -6,7 +6,6 @@ export interface CheckMetadata {
monitorId: string;
teamId: string;
type: MonitorType;
location?: string;
}
export interface LocationResult {
-1
View File
@@ -34,7 +34,6 @@ export interface MonitorStatusResponse<T = any> {
matchMethod?: MonitorMatchMethod;
expectedValue?: string;
extracted?: unknown;
location?: string;
}
export interface GlobalpingCheckResult {