elastic/kibana

[saved objects] Improve type safety in the saved objects repository

Open

#239.468 geöffnet am 17. Okt. 2025

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (1 zugewiesene Person)TypeScript (8.021 Forks)batch import
Feature:Saved ObjectsTeam:Coregood first issue💝community

Repository-Metriken

Stars
 (19.065 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 1T 16h) (999 gemergte PRs in 30 T)

Beschreibung

The is an existing pattern of using // @ts-expect-error MultiGetHit._source is optional comments in the saved object repository code that weakens the type safety. We should use a type guard instead

/**
 * Type guard to check if an item is a GetGetResult.
 */
export function isGetGetResult<TDocument = unknown>(
  item: MgetResponseItem<TDocument>
): item is GetGetResult<TDocument> {
  return (item as MgetMultiGetError).error === undefined;
}

Originally posted by @rudolf in https://github.com/elastic/kibana/pull/224411#discussion_r2348988329

Contributor Guide

Research-Richtung
Überprüfen Sie den Code des Repositorys für gespeicherte Objekte auf TypeScript Typsicherheitsprobleme, insbesondere wo `@ts expect error` verwendet wird, und wenden Sie das bereitgestellte Typwächter Muster an.
Tech Stack
typescript
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitTypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75