From d7adc076641dd7703d44dbfa267a735e1646dde4 Mon Sep 17 00:00:00 2001 From: Anders Kane Date: Tue, 31 Mar 2026 14:01:48 -0400 Subject: [PATCH 1/3] revert(featureserver): revert add indexes default for layer metadata --- .../helpers/feature-layer-metadata.spec.js | 7 ------- .../src/helpers/table-layer-metadata.spec.js | 21 ------------------- .../featureserver/src/metadata-defaults.js | 1 - ...geoservice-client-requests-no-geom.spec.js | 2 -- test/helpers/client-response-fixtures.js | 1 - 5 files changed, 32 deletions(-) diff --git a/packages/featureserver/src/helpers/feature-layer-metadata.spec.js b/packages/featureserver/src/helpers/feature-layer-metadata.spec.js index a38c54ae..016974e3 100644 --- a/packages/featureserver/src/helpers/feature-layer-metadata.spec.js +++ b/packages/featureserver/src/helpers/feature-layer-metadata.spec.js @@ -139,7 +139,6 @@ describe('FeatureLayerMetadata', () => { maxRecordCountFactor: 1, fields: [], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -392,7 +391,6 @@ describe('FeatureLayerMetadata', () => { }, ], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -554,7 +552,6 @@ describe('FeatureLayerMetadata', () => { }, ], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -712,7 +709,6 @@ describe('FeatureLayerMetadata', () => { }, ], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -908,7 +904,6 @@ describe('FeatureLayerMetadata', () => { }, ], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1064,7 +1059,6 @@ describe('FeatureLayerMetadata', () => { }, ], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1233,7 +1227,6 @@ describe('FeatureLayerMetadata', () => { }, ], relationships: [], - indexes: [], capabilities: 'list,of,stuff', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], diff --git a/packages/featureserver/src/helpers/table-layer-metadata.spec.js b/packages/featureserver/src/helpers/table-layer-metadata.spec.js index b5493005..043c4ff6 100644 --- a/packages/featureserver/src/helpers/table-layer-metadata.spec.js +++ b/packages/featureserver/src/helpers/table-layer-metadata.spec.js @@ -130,7 +130,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: [], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -252,7 +251,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -367,7 +365,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -480,7 +477,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -593,7 +589,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -706,7 +701,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -819,7 +813,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -932,7 +925,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1045,7 +1037,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1158,7 +1149,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1271,7 +1261,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1384,7 +1373,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'hello,world', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1497,7 +1485,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query,Extract', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1610,7 +1597,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1723,7 +1709,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1836,7 +1821,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1949,7 +1933,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -1964,7 +1947,6 @@ describe('TableLayerMetadata', () => { { name: 'Hank Williams', relationships: ['something'], - indexes: [], description: 'There is a tear in my beer.', copyrightText: 'copyright-text', templates: ['templates'], @@ -2074,7 +2056,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: ['something'], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -2330,7 +2311,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'list,of,stuff', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], @@ -2452,7 +2432,6 @@ describe('TableLayerMetadata', () => { maxRecordCountFactor: 1, fields: ['fields'], relationships: [], - indexes: [], capabilities: 'list,of,stuff', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, types: [], diff --git a/packages/featureserver/src/metadata-defaults.js b/packages/featureserver/src/metadata-defaults.js index 6a267016..3329f2ae 100644 --- a/packages/featureserver/src/metadata-defaults.js +++ b/packages/featureserver/src/metadata-defaults.js @@ -225,7 +225,6 @@ class MetadataDefaults { maxRecordCountFactor: 1, fields: [], relationships: [], - indexes: [], capabilities: 'Query', ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true, diff --git a/test/geoservice-client-requests-no-geom.spec.js b/test/geoservice-client-requests-no-geom.spec.js index b6246cfe..7441ecb0 100644 --- a/test/geoservice-client-requests-no-geom.spec.js +++ b/test/geoservice-client-requests-no-geom.spec.js @@ -202,7 +202,6 @@ describe('Typical Geoservice Client request sequence: Dataset with no geometry', }, ], relationships: [], - indexes: [], capabilities: 'Query', maxRecordCount: 2000, ownershipBasedAccessControlForFeatures: { @@ -370,7 +369,6 @@ describe('Typical Geoservice Client request sequence: Dataset with no geometry', }, ], relationships: [], - indexes: [], capabilities: 'Query', maxRecordCount: 2000, ownershipBasedAccessControlForFeatures: { allowOthersToQuery: true }, diff --git a/test/helpers/client-response-fixtures.js b/test/helpers/client-response-fixtures.js index 8fa0fc42..795809fa 100644 --- a/test/helpers/client-response-fixtures.js +++ b/test/helpers/client-response-fixtures.js @@ -130,7 +130,6 @@ function getLayersInfo(filename, idField) { }, ], relationships: [], - indexes: [], capabilities: 'Query', maxRecordCount: 2000, supportsStatistics: true, From 885c06086406561f8d3515e26928354b508c21d6 Mon Sep 17 00:00:00 2001 From: Anders Kane Date: Tue, 31 Mar 2026 14:10:09 -0400 Subject: [PATCH 2/3] revert(cache-memory,core,featureserver,logger,output-geoservices,winnow): revert change logs --- .changeset/shiny-lies-arrive.md | 5 +++++ packages/cache-memory/coverage.svg | 8 ++++---- packages/core/CHANGELOG.md | 6 ------ packages/core/coverage.svg | 8 ++++---- packages/core/package.json | 4 ++-- packages/featureserver/CHANGELOG.md | 6 ------ packages/featureserver/coverage.svg | 8 ++++---- packages/featureserver/package.json | 2 +- packages/logger/coverage.svg | 8 ++++---- packages/output-geoservices/CHANGELOG.md | 7 ------- packages/output-geoservices/coverage.svg | 8 ++++---- packages/output-geoservices/package.json | 4 ++-- packages/winnow/coverage-unit.svg | 8 ++++---- 13 files changed, 34 insertions(+), 48 deletions(-) create mode 100644 .changeset/shiny-lies-arrive.md diff --git a/.changeset/shiny-lies-arrive.md b/.changeset/shiny-lies-arrive.md new file mode 100644 index 00000000..2248af65 --- /dev/null +++ b/.changeset/shiny-lies-arrive.md @@ -0,0 +1,5 @@ +--- +'@koopjs/featureserver': minor +--- + +Add indexes default for feature layer metadata diff --git a/packages/cache-memory/coverage.svg b/packages/cache-memory/coverage.svg index 9dd748a7..9c3049ba 100644 --- a/packages/cache-memory/coverage.svg +++ b/packages/cache-memory/coverage.svg @@ -1,14 +1,14 @@ coverage: 100% - + - - + + - +