From e3fd08672da39c4e60ee6d16957925579aa7e5ac Mon Sep 17 00:00:00 2001
From: Ricardo Devis Agullo <mkredz@gmail.com>
Date: Wed, 27 May 2026 11:20:31 +0200
Subject: [PATCH 1/2] perf(get-component): hoist Set creation, eliminate
 template double-lookup, anchor regex

- Move customHeadersToSkipSet from per-request to registry scope
- Store repository.getTemplate result once and reuse, removing redundant
  try/catch and second lookup
- Use anchored regex for staticPath protocol stripping
---
 .../registry/routes/helpers/get-component.ts  | 38 ++++---------------
 1 file changed, 7 insertions(+), 31 deletions(-)

diff --git a/packages/oc/src/registry/routes/helpers/get-component.ts b/packages/oc/src/registry/routes/helpers/get-component.ts
index f0a873a2..7d60532e 100644
--- a/packages/oc/src/registry/routes/helpers/get-component.ts
+++ b/packages/oc/src/registry/routes/helpers/get-component.ts
@@ -10,13 +10,7 @@ import emptyResponseHandler from 'oc-empty-response-handler';
 import { fromPromise } from 'universalify';
 import strings from '../../../resources';
 import settings from '../../../resources/settings';
-import type {
-  Component,
-  Config,
-  PluginContext,
-  Plugins,
-  Template
-} from '../../../types';
+import type { Component, Config, PluginContext, Plugins } from '../../../types';
 import isTemplateLegacy from '../../../utils/is-template-legacy';
 import eventsHandler from '../../domain/events-handler';
 import NestedRenderer from '../../domain/nested-renderer';
@@ -146,6 +140,9 @@ export default function getComponent(conf: Config, repository: Repository) {
     refreshInterval: conf.refreshInterval
   });
   const convertPlugins = pluginConverter(conf.plugins);
+  const customHeadersToSkipSet = conf.customHeadersToSkipOnWeakVersion?.length
+    ? new Set(conf.customHeadersToSkipOnWeakVersion)
+    : undefined;
 
   const getEnv = async (
     component: Component
@@ -315,7 +312,8 @@ export default function getComponent(conf: Config, repository: Repository) {
           templateType = `oc-template-${templateType}`;
         }
 
-        if (!repository.getTemplate(templateType)) {
+        const ocTemplate = repository.getTemplate(templateType);
+        if (!ocTemplate) {
           return callback({
             status: 400,
             response: {
@@ -326,11 +324,6 @@ export default function getComponent(conf: Config, repository: Repository) {
           });
         }
 
-        const customHeadersToSkip = conf.customHeadersToSkipOnWeakVersion;
-        const customHeadersToSkipSet = customHeadersToSkip?.length
-          ? new Set(customHeadersToSkip)
-          : undefined;
-
         const filterCustomHeaders = (
           headers: Record<string, string>,
           requestedVersion: string,
@@ -551,23 +544,6 @@ export default function getComponent(conf: Config, repository: Repository) {
                 component.name,
                 component.version,
                 (_err, templateText) => {
-                  let ocTemplate: Template;
-
-                  try {
-                    ocTemplate = repository.getTemplate(templateType);
-                  } catch {
-                    return callback({
-                      status: 400,
-                      response: {
-                        code: 'TEMPLATE_NOT_SUPPORTED',
-                        error:
-                          strings.errors.registry.TEMPLATE_NOT_SUPPORTED(
-                            templateType
-                          )
-                      }
-                    });
-                  }
-
                   const template = ocTemplate.getCompiledTemplate(
                     templateText,
                     key
@@ -581,7 +557,7 @@ export default function getComponent(conf: Config, repository: Repository) {
         };
         const staticPath = repository
           .getStaticFilePath(component.name, component.version, '')
-          .replace('https:', '');
+          .replace(/^https:/, '');
 
         if (!component.oc.files.dataProvider) {
           const { __oc_Retry, ...props } = params;

From 4704146969c1d09b104e289707500db701cde081 Mon Sep 17 00:00:00 2001
From: Ricardo Devis Agullo <mkredz@gmail.com>
Date: Wed, 27 May 2026 11:50:20 +0200
Subject: [PATCH 2/2] fix: compute customHeadersToSkipSet from options.conf,
 not outer conf

---
 packages/oc/src/registry/routes/helpers/get-component.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/oc/src/registry/routes/helpers/get-component.ts b/packages/oc/src/registry/routes/helpers/get-component.ts
index 7d60532e..e2f73135 100644
--- a/packages/oc/src/registry/routes/helpers/get-component.ts
+++ b/packages/oc/src/registry/routes/helpers/get-component.ts
@@ -140,9 +140,6 @@ export default function getComponent(conf: Config, repository: Repository) {
     refreshInterval: conf.refreshInterval
   });
   const convertPlugins = pluginConverter(conf.plugins);
-  const customHeadersToSkipSet = conf.customHeadersToSkipOnWeakVersion?.length
-    ? new Set(conf.customHeadersToSkipOnWeakVersion)
-    : undefined;
 
   const getEnv = async (
     component: Component
@@ -200,6 +197,9 @@ export default function getComponent(conf: Config, repository: Repository) {
 
     let componentCallbackDone = false;
     const conf = options.conf;
+    const customHeadersToSkipSet = conf.customHeadersToSkipOnWeakVersion?.length
+      ? new Set(conf.customHeadersToSkipOnWeakVersion)
+      : undefined;
     const acceptLanguage = getLanguage();
     const requestedComponent = {
       name: options.name,