Avoid constructor source scanning for classes without a base class in @loopback/context

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#11,878 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

@zuhairkhalid229 y travaille déjà.

Depuis le 5/11/2026.

  • #11884 par @zuhairkhalid229 — ouverte

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
72/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
node.js, typescript
Domaine
backend, performance

Piste de recherche

Commencez par lire describeInjectedArguments() et shouldSkipBaseConstructorInjection() dans @loopback/context, puis examinez les tests existants pour l’injection de constructeur et l’héritage. Confirmez que les classes sans classe de base évitent l’analyse du code source, tandis que les classes avec une classe de base conservent le comportement existant ; exécutez les tests pertinents pour vérifier les deux cas.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

bug
Describe the bug

On the first call to describeInjectedArguments() for a constructor, LoopBack calls shouldSkipBaseConstructorInjection(), which reads the class’s source and runs regular expressions to determine whether constructor injections should be inherited.

This work also runs for classes that do not extend another class. For those classes, there is no base constructor to inspect. The metadata cache avoids repeated work for the same class, but applications inspecting many distinct classes still incur this cost during startup.

The proposed change in describeInjectedArguments() is:

if (method === '') {
  if (
    Object.getPrototypeOf(target) !== Function.prototype &&
    shouldSkipBaseConstructorInjection(target)
  ) {
    options.ownMetadataOnly = true;
  }
}
Logs
Additional information

We measured the following application startup timings with @loopback/context 9.0.0, Node.js 22.23.4, and macOS 26.7 on an Apple M4 Pro:

Measurement Original With proposed guard
Constructor inspection 147.9 ms 18.5 ms
Overall component registration 224 ms 98 ms

These are medians of three fresh processes per variant, alternating between variants with other startup settings held constant. Of 3,948 inspected classes, 3,384 had no base class.

The improvement saves approximately 126 ms during registration. Constructor inspection is included in registration, so these savings are not additive or a claim about total application startup time.

Reproduction

https://github.com/tonynelson19/loopback-constructor-repro

Langage dominant
TypeScript
Étoiles
5.1k
Forks
1.1k
Merge moyen
1 j 19 h
PR mergées (30 j)
47

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de loopbackio/loopback-next

Toutes les issues de loopbackio/loopback-next

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.