@backstage/config - npm Package Compare versions

Comparing version 0.1.7 to 0.1.8

CHANGELOG.md

 # @backstage/config
 
+## 0.1.8
+
+### Patch Changes
+
+- 47113f1f1: Only warn once per key when trying to read visibility-filtered values
+
 ## 0.1.7
@@ -4,0 +10,0 @@

dist/index.cjs.js

@@ -49,2 +49,3 @@ 'use strict';
     this.prefix = prefix;
+    this.notifiedFilteredKeys = new Set();
   }
@@ -90,3 +91,4 @@ static fromConfigs(configs) {
           const fullKey = this.fullKey(key);
-          if ((_b = this.filteredKeys) == null ? void 0 : _b.includes(fullKey)) {
+          if (((_b = this.filteredKeys) == null ? void 0 : _b.includes(fullKey)) && !this.notifiedFilteredKeys.has(fullKey)) {
+            this.notifiedFilteredKeys.add(fullKey);
             console.warn(`Failed to read configuration value at '${fullKey}' as it is not visible. See https://backstage.io/docs/conf/defining#visibility for instructions on how to make it visible.`);
@@ -144,3 +146,4 @@ }
         const fullKey = this.fullKey(key);
-        if ((_a = this.filteredKeys) == null ? void 0 : _a.some((k) => k.startsWith(fullKey))) {
+        if (((_a = this.filteredKeys) == null ? void 0 : _a.some((k) => k.startsWith(fullKey))) && !this.notifiedFilteredKeys.has(key)) {
+          this.notifiedFilteredKeys.add(key);
           console.warn(`Failed to read configuration array at '${key}' as it does not have any visible elements. See https://backstage.io/docs/conf/defining#visibility for instructions on how to make it visible.`);
@@ -225,3 +228,4 @@ }
         const fullKey = this.fullKey(key);
-        if ((_a = this.filteredKeys) == null ? void 0 : _a.includes(fullKey)) {
+        if (((_a = this.filteredKeys) == null ? void 0 : _a.includes(fullKey)) && !this.notifiedFilteredKeys.has(fullKey)) {
+          this.notifiedFilteredKeys.add(fullKey);
           console.warn(`Failed to read configuration value at '${fullKey}' as it is not visible. See https://backstage.io/docs/conf/defining#visibility for instructions on how to make it visible.`);
@@ -228,0 +232,0 @@ }

dist/index.d.ts

@@ -55,2 +55,3 @@ declare type JsonPrimitive = number | string | boolean | null;
     private filteredKeys?;
+    private notifiedFilteredKeys;
     static fromConfigs(configs: AppConfig[]): ConfigReader;
@@ -57,0 +58,0 @@ constructor(data: JsonObject | undefined, context?: string, fallback?: ConfigReader | undefined, prefix?: string);

dist/index.esm.js

@@ -40,2 +40,3 @@ import cloneDeep from 'lodash/cloneDeep';
     this.prefix = prefix;
+    this.notifiedFilteredKeys = new Set();
   }
@@ -81,3 +82,4 @@ static fromConfigs(configs) {
           const fullKey = this.fullKey(key);
-          if ((_b = this.filteredKeys) == null ? void 0 : _b.includes(fullKey)) {
+          if (((_b = this.filteredKeys) == null ? void 0 : _b.includes(fullKey)) && !this.notifiedFilteredKeys.has(fullKey)) {
+            this.notifiedFilteredKeys.add(fullKey);
             console.warn(`Failed to read configuration value at '${fullKey}' as it is not visible. See https://backstage.io/docs/conf/defining#visibility for instructions on how to make it visible.`);
@@ -135,3 +137,4 @@ }
         const fullKey = this.fullKey(key);
-        if ((_a = this.filteredKeys) == null ? void 0 : _a.some((k) => k.startsWith(fullKey))) {
+        if (((_a = this.filteredKeys) == null ? void 0 : _a.some((k) => k.startsWith(fullKey))) && !this.notifiedFilteredKeys.has(key)) {
+          this.notifiedFilteredKeys.add(key);
           console.warn(`Failed to read configuration array at '${key}' as it does not have any visible elements. See https://backstage.io/docs/conf/defining#visibility for instructions on how to make it visible.`);
@@ -216,3 +219,4 @@ }
         const fullKey = this.fullKey(key);
-        if ((_a = this.filteredKeys) == null ? void 0 : _a.includes(fullKey)) {
+        if (((_a = this.filteredKeys) == null ? void 0 : _a.includes(fullKey)) && !this.notifiedFilteredKeys.has(fullKey)) {
+          this.notifiedFilteredKeys.add(fullKey);
           console.warn(`Failed to read configuration value at '${fullKey}' as it is not visible. See https://backstage.io/docs/conf/defining#visibility for instructions on how to make it visible.`);
@@ -219,0 +223,0 @@ }

package.json

 {
   "name": "@backstage/config",
   "description": "Config API used by Backstage core, backend, and CLI",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "private": false,
@@ -42,4 +42,4 @@ "publishConfig": {
   ],
-  "gitHead": "2f291dfd04f87a6ff4d6000204d0af7067bcda10",
+  "gitHead": "e436edb5bface99d33b7a0e8dd863d8e204e81fc",
   "module": "dist/index.esm.js"
 }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

69681

increased by2.96%

Lines of code

587

increased by1.56%

No dependency changes