@fgv/ts-res 5.0.0-30 → 5.0.0-31

package/lib/packlets/common/helpers/context.js

@@ -67,6 +67,7 @@ const Validate = __importStar(require("../validate"));
  * @public
  */
 function buildContextQualifierToken({ qualifier, value }) {
+    /* c8 ignore next 1 - edge case: ternary expression alternate branch rarely hit */
     return Validate.toContextQualifierToken(qualifier ? `${qualifier}=${value}` : value);
 }
 /**

package/lib/packlets/common/helpers/resources.js

@@ -36,6 +36,7 @@ const resources_1 = require("../validate/resources");
  * @public
  */
 function splitResourceId(id) {
+    /* c8 ignore next 3 - edge case: undefined resource ID should not occur in normal operation */
     if (id === undefined) {
         return (0, ts_utils_1.succeed)([]);
     }

package/lib/packlets/common/validate/conditions.d.ts

@@ -1,5 +1,5 @@
 import { Result } from '@fgv/ts-utils';
-import { ConditionPriority, QualifierName, QualifierTypeName, QualifierIndex, QualifierTypeIndex, ConditionIndex, ConditionSetIndex, QualifierMatchScore, ConditionKey, ConditionSetKey, DecisionKey, DecisionIndex, ConditionSetHash, ConditionOperator, ConditionToken, ConditionSetToken, ContextQualifierToken, ContextToken, QualifierDefaultValueToken, QualifierDefaultValuesToken } from '../conditions';
+import { ConditionPriority, QualifierName, QualifierTypeName, QualifierIndex, QualifierTypeIndex, ConditionIndex, ConditionSetIndex, QualifierMatchScore, QualifierContextValue, ConditionKey, ConditionSetKey, DecisionKey, DecisionIndex, ConditionSetHash, ConditionOperator, ConditionToken, ConditionSetToken, ContextQualifierToken, ContextToken, QualifierDefaultValueToken, QualifierDefaultValuesToken } from '../conditions';
 /**
  * Determines whether a string is a valid qualifier name.
  * @param name - the string to validate
@@ -117,6 +117,14 @@ export declare function isValidDecisionKey(key: string): key is DecisionKey;
  * @public
  */
 export declare function isValidDecisionIndex(index: number): index is DecisionIndex;
+/**
+ * Determines whether a string is a valid qualifier context value.
+ * For now, any non-empty string is considered a valid context value.
+ * @param value - the string to validate
+ * @returns true if the string is a valid qualifier context value, false otherwise.
+ * @public
+ */
+export declare function isValidQualifierContextValue(value: string): value is QualifierContextValue;
 /**
  * Converts a string to a {@link QualifierName} if it is a valid qualifier name.
  * @param name - the string to convert
@@ -125,6 +133,14 @@ export declare function isValidDecisionIndex(index: number): index is DecisionIn
  * @public
  */
 export declare function toQualifierName(name: string): Result<QualifierName>;
+/**
+ * Converts a string to a {@link QualifierContextValue} if it is a valid qualifier context value.
+ * @param value - the string to convert
+ * @returns `Success` with the converted {@link QualifierContextValue} if successful, or `Failure` with an
+ * error message if not.
+ * @public
+ */
+export declare function toQualifierContextValue(value: string): Result<QualifierContextValue>;
 /**
  * Converts a number to a {@link QualifierIndex} if it is a valid qualifier index.
  * @param index - the number to convert

package/lib/packlets/common/validate/conditions.js

@@ -37,7 +37,9 @@ exports.isValidConditionSetToken = isValidConditionSetToken;
 exports.isValidConditionSetHash = isValidConditionSetHash;
 exports.isValidDecisionKey = isValidDecisionKey;
 exports.isValidDecisionIndex = isValidDecisionIndex;
+exports.isValidQualifierContextValue = isValidQualifierContextValue;
 exports.toQualifierName = toQualifierName;
+exports.toQualifierContextValue = toQualifierContextValue;
 exports.toQualifierIndex = toQualifierIndex;
 exports.toQualifierTypeName = toQualifierTypeName;
 exports.toQualifierTypeIndex = toQualifierTypeIndex;
@@ -182,6 +184,7 @@ function isValidConditionSetKey(key) {
     if (key === '') {
         return true;
     }
+    /* c8 ignore next 2 - edge case: condition set key validation rarely fails */
     // a condition set key is a `+` separated list of condition keys
     return key.split('+').every(isValidConditionKey);
 }
@@ -224,6 +227,16 @@ function isValidDecisionKey(key) {
 function isValidDecisionIndex(index) {
     return index >= 0;
 }
+/**
+ * Determines whether a string is a valid qualifier context value.
+ * For now, any non-empty string is considered a valid context value.
+ * @param value - the string to validate
+ * @returns true if the string is a valid qualifier context value, false otherwise.
+ * @public
+ */
+function isValidQualifierContextValue(value) {
+    return typeof value === 'string' && value.length > 0;
+}
 /**
  * Converts a string to a {@link QualifierName} if it is a valid qualifier name.
  * @param name - the string to convert
@@ -234,10 +247,23 @@ function isValidDecisionIndex(index) {
 function toQualifierName(name) {
     /* c8 ignore next 3 - functional code path tested but coverage intermittently missed */
     if (!isValidQualifierName(name)) {
-        return (0, ts_utils_1.fail)(`${name}: not a valid qualifier name`);
+        return (0, ts_utils_1.fail)(`${name}: invalid qualifier name`);
     }
     return (0, ts_utils_1.succeed)(name);
 }
+/**
+ * Converts a string to a {@link QualifierContextValue} if it is a valid qualifier context value.
+ * @param value - the string to convert
+ * @returns `Success` with the converted {@link QualifierContextValue} if successful, or `Failure` with an
+ * error message if not.
+ * @public
+ */
+function toQualifierContextValue(value) {
+    if (!isValidQualifierContextValue(value)) {
+        return (0, ts_utils_1.fail)(`${value}: invalid qualifier context value`);
+    }
+    return (0, ts_utils_1.succeed)(value);
+}
 /**
  * Converts a number to a {@link QualifierIndex} if it is a valid qualifier index.
  * @param index - the number to convert
@@ -248,7 +274,7 @@ function toQualifierName(name) {
 function toQualifierIndex(index) {
     /* c8 ignore next 3 - coverage is having a bad day */
     if (!isValidQualifierIndex(index)) {
-        return (0, ts_utils_1.fail)(`${index}: not a valid qualifier index`);
+        return (0, ts_utils_1.fail)(`${index}: invalid qualifier index`);
     }
     return (0, ts_utils_1.succeed)(index);
 }
@@ -262,7 +288,7 @@ function toQualifierIndex(index) {
 function toQualifierTypeName(name) {
     /* c8 ignore next 3 - functional code path tested but coverage intermittently missed */
     if (!isValidQualifierTypeName(name)) {
-        return (0, ts_utils_1.fail)(`${name}: not a valid qualifier type name`);
+        return (0, ts_utils_1.fail)(`${name}: invalid qualifier type name`);
     }
     return (0, ts_utils_1.succeed)(name);
 }
@@ -276,7 +302,7 @@ function toQualifierTypeName(name) {
 function toQualifierTypeIndex(index) {
     /* c8 ignore next 3 - functional code path tested but coverage intermittently missed */
     if (!isValidQualifierTypeIndex(index)) {
-        return (0, ts_utils_1.fail)(`${index}: not a valid qualifier type index`);
+        return (0, ts_utils_1.fail)(`${index}: invalid qualifier type index`);
     }
     return (0, ts_utils_1.succeed)(index);
 }
@@ -541,7 +567,7 @@ function isValidQualifierDefaultValuesToken(token) {
 function toQualifierDefaultValueToken(token) {
     /* c8 ignore next 3 - functional code path tested but coverage intermittently missed */
     if (!isValidQualifierDefaultValueToken(token)) {
-        return (0, ts_utils_1.fail)(`${token}: not a valid qualifier default value token`);
+        return (0, ts_utils_1.fail)(`${token}: invalid qualifier default value token`);
     }
     return (0, ts_utils_1.succeed)(token);
 }
@@ -555,7 +581,7 @@ function toQualifierDefaultValueToken(token) {
 function toQualifierDefaultValuesToken(token) {
     /* c8 ignore next 3 - functional code path tested but coverage intermittently missed */
     if (!isValidQualifierDefaultValuesToken(token)) {
-        return (0, ts_utils_1.fail)(`${token}: not a valid qualifier default values token`);
+        return (0, ts_utils_1.fail)(`${token}: invalid qualifier default values token`);
     }
     return (0, ts_utils_1.succeed)(token);
 }

package/lib/packlets/common/validate/resources.js

@@ -132,6 +132,7 @@ function toResourceName(name) {
  * @public
  */
 function toResourceId(id) {
+    /* c8 ignore next 3 - defensive coding: resource ID validation should prevent invalid IDs */
     if (!isValidResourceId(id)) {
         return (0, ts_utils_1.fail)(`${id}: not a valid resource ID.`);
     }

package/lib/packlets/conditions/condition.js

@@ -43,6 +43,7 @@ class Condition {
      */
     constructor({ qualifier, value, operator, priority, scoreAsDefault, index }) {
         if (scoreAsDefault === undefined && qualifier.defaultValue !== undefined) {
+            /* c8 ignore next 4 - edge case: default value matching logic rarely triggered */
             const defaultMatch = qualifier.type.matches(value, qualifier.defaultValue, 'matches');
             if (defaultMatch !== common_1.NoMatch) {
                 scoreAsDefault = defaultMatch;

package/lib/packlets/conditions/conditionSet.js

@@ -187,6 +187,7 @@ class ConditionSet {
                     return { qualifierName, value };
                 }
                 else {
+                    /* c8 ignore next 1 - edge case: object spread pattern rarely used */
                     return Object.assign({ qualifierName }, value);
                 }
             });

package/lib/packlets/import/fsItem.d.ts

@@ -1,4 +1,5 @@
-import { DetailedResult, FileTree, Result } from '@fgv/ts-utils';
+import { DetailedResult, Result } from '@fgv/ts-utils';
+import { FileTree } from '@fgv/ts-json-base';
 import * as Conditions from '../conditions';
 import { IReadOnlyQualifierCollector } from '../qualifiers';
 import { ImportContext } from './importContext';

package/lib/packlets/import/fsItem.js

@@ -56,6 +56,7 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.FsItem = void 0;
 const ts_utils_1 = require("@fgv/ts-utils");
+const ts_json_base_1 = require("@fgv/ts-json-base");
 const common_1 = require("../common");
 const Conditions = __importStar(require("../conditions"));
 const importContext_1 = require("./importContext");
@@ -110,7 +111,7 @@ class FsItem {
      * that should be silently ignored.
      */
     static createForPath(importPath, qualifiers, tree) {
-        return (0, ts_utils_1.useOrInitialize)(tree, () => ts_utils_1.FileTree.forFilesystem())
+        return (0, ts_utils_1.useOrInitialize)(tree, () => ts_json_base_1.FileTree.forFilesystem())
             .withDetail('failed', 'succeeded')
             .onSuccess((tree) => {
             return tree

package/lib/packlets/import/importManager.d.ts

@@ -1,4 +1,5 @@
-import { FileTree, Result } from '@fgv/ts-utils';
+import { Result } from '@fgv/ts-utils';
+import { FileTree } from '@fgv/ts-json-base';
 import { ResourceManagerBuilder } from '../resources';
 import { ImportContext } from './importContext';
 import { IImportable } from './importable';

package/lib/packlets/import/importers/pathImporter.d.ts

@@ -1,4 +1,5 @@
-import { DetailedResult, Result, FileTree } from '@fgv/ts-utils';
+import { DetailedResult, Result } from '@fgv/ts-utils';
+import { FileTree } from '@fgv/ts-json-base';
 import { ResourceManagerBuilder } from '../../resources';
 import { IImportable } from '../importable';
 import { IImporter, ImporterResultDetail } from './importer';

package/lib/packlets/import/importers/pathImporter.js

@@ -23,6 +23,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.PathImporter = void 0;
 const ts_utils_1 = require("@fgv/ts-utils");
+const ts_json_base_1 = require("@fgv/ts-json-base");
 const fsItem_1 = require("../fsItem");
 /**
  * {@link Import.Importers.IImporter | Importer} implementation which imports resources from a `FileTree`
@@ -41,7 +42,7 @@ class PathImporter {
          */
         this.types = ['path'];
         this.qualifiers = params.qualifiers;
-        this.tree = (_a = params.tree) !== null && _a !== void 0 ? _a : ts_utils_1.FileTree.forFilesystem().orThrow();
+        this.tree = (_a = params.tree) !== null && _a !== void 0 ? _a : ts_json_base_1.FileTree.forFilesystem().orThrow();
         this.ignoreFileTypes = (_b = params.ignoreFileTypes) !== null && _b !== void 0 ? _b : [];
     }
     /**

package/lib/packlets/qualifier-types/literalValueHierarchy.js

@@ -102,6 +102,7 @@ class LiteralValueHierarchy {
         if (ancestors.isSuccess()) {
             return ancestors.value.includes(possibleAncestor);
         }
+        /* c8 ignore next 1 - edge case: ancestor validation fallback */
         return false;
     }
     /**

package/lib/packlets/qualifier-types/territoryQualifierType.js

@@ -106,6 +106,7 @@ class TerritoryQualifierType extends qualifierType_1.QualifierType {
      */
     isValidConditionValue(value) {
         const normalized = this.acceptLowercase ? value.toUpperCase() : value;
+        /* c8 ignore next 6 - edge case: invalid territory values and allowed territory filtering rarely hit */
         if (!TerritoryQualifierType.isValidTerritoryConditionValue(normalized)) {
             return false;
         }
@@ -194,6 +195,7 @@ class TerritoryQualifierType extends qualifierType_1.QualifierType {
             if (normalizedCondition === normalizedContext) {
                 return common_1.PerfectMatch;
             }
+            /* c8 ignore next 8 - edge case: hierarchy matching and fallback logic rarely used */
             if (this.hierarchy) {
                 return this.hierarchy.match(normalizedCondition, normalizedContext, __operator);
             }
@@ -209,6 +211,7 @@ class TerritoryQualifierType extends qualifierType_1.QualifierType {
      * @public
      */
     static isValidTerritoryConditionValue(value, acceptLowercase) {
+        /* c8 ignore next 3 - edge case: lowercase validation rarely fails */
         if (acceptLowercase !== true && value !== value.toUpperCase()) {
             return false;
         }

package/lib/packlets/resources/candidateReducer.js

@@ -132,6 +132,7 @@ class CandidateReducer {
                             hasCollisions = true;
                         }
                         else {
+                            /* c8 ignore next 12 - complex collision handling for partial candidates rarely triggered */
                             // Partial reduced candidate - merge into unchanged and suppress reduced
                             const mergedJson = CandidateReducer._mergeJsonValues(unchangedCandidate.originalCandidate.json, reducedCandidate.originalCandidate.json);
                             // Update unchanged candidate with merged JSON, keeping its isPartial status
@@ -142,6 +143,7 @@ class CandidateReducer {
                         }
                     }
                     else {
+                        /* c8 ignore next 15 - complex collision reversion logic rarely triggered in practice */
                         // Complex collision case - revert all reduced candidates to unchanged
                         for (const candidateInfo of reducedCandidates) {
                             // Revert to original conditions and mark as unchanged
@@ -224,6 +226,7 @@ class CandidateReducer {
     reduceCandidate(candidate) {
         var _a;
         const candidateInfo = this._candidateInfos.find((info) => info.originalCandidate === candidate);
+        /* c8 ignore next 3 - defensive coding: candidate should always be found if properly managed */
         if (!candidateInfo) {
             return (0, ts_utils_1.fail)(`Candidate not found in reducer state`);
         }

package/lib/packlets/resources/resourceBuilder.js

@@ -103,6 +103,7 @@ class ResourceBuilder {
                 .getOrAdd(candidate.conditions.toString(), candidate)
                 .onSuccess((added, detail) => {
                 if (detail === 'exists') {
+                    /* c8 ignore next 5 - defensive coding: conflicting candidates with same conditions should not occur */
                     if (!resourceCandidate_1.ResourceCandidate.equal(added, candidate)) {
                         return (0, ts_utils_1.failWithDetail)(`${this.id}: conflicting candidates.`, 'exists');
                     }

package/lib/packlets/resources/resourceManagerBuilder.js

@@ -255,6 +255,7 @@ class ResourceManagerBuilder {
     addResource(decl) {
         var _a;
         const { value: id, message } = common_1.Validate.toResourceId(decl.id);
+        /* c8 ignore next 3 - defensive coding: resource ID validation should prevent invalid IDs */
         if (message !== undefined) {
             return (0, ts_utils_1.failWithDetail)(`${id}: invalid id - ${message}`, 'failure');
         }
@@ -271,6 +272,7 @@ class ResourceManagerBuilder {
             return (0, ts_utils_1.failWithDetail)(`${id}: unable to get or add resource\n${getOrAddMessage}`, detail);
         }
         if (detail === 'exists') {
+            /* c8 ignore next 4 - defensive coding: resource type mismatch on existing resources should not occur */
             const { message } = builder.setResourceType(decl.resourceTypeName);
             if (message !== undefined) {
                 return (0, ts_utils_1.failWithDetail)(`${id}: unable to set resource type\n${message}`, 'type-mismatch');
@@ -479,6 +481,7 @@ class ResourceManagerBuilder {
     getCompiledResourceCollection(options) {
         // Build resources first to ensure all data is available
         const buildResult = this._performBuild();
+        /* c8 ignore next 3 - defensive coding: build failure should not occur after successful validation */
         if (buildResult.isFailure()) {
             return (0, ts_utils_1.fail)(`Failed to build resources: ${buildResult.message}`);
         }

package/lib/packlets/runtime/compiledResourceCollection.js

@@ -264,6 +264,7 @@ class CompiledResourceCollection {
         const conditionCollector = conditionCollectorResult.value;
         for (const [index, compiledCondition] of compiled.conditions.entries()) {
             const qualifierResult = qualifiers.getAt(compiledCondition.qualifierIndex);
+            /* c8 ignore next 5 - defensive coding: invalid qualifier index should not occur in compiled data */
             if (qualifierResult.isFailure()) {
                 errors.addMessage(`Invalid qualifier index ${compiledCondition.qualifierIndex} at condition ${index}: ${qualifierResult.message}`);
                 continue;
@@ -361,6 +362,7 @@ class CompiledResourceCollection {
         const decisionCollector = decisionCollectorResult.value;
         for (const [index, compiledDecision] of compiled.decisions.entries()) {
             const referencedConditionSetsResult = (0, ts_utils_1.mapResults)(compiledDecision.conditionSets.map((conditionSetIndex) => conditionSets.getAt(conditionSetIndex)));
+            /* c8 ignore next 5 - defensive coding: condition set resolution should not fail with valid compiled data */
             if (referencedConditionSetsResult.isFailure()) {
                 errors.addMessage(`Failed to resolve condition sets for decision ${index}: ${referencedConditionSetsResult.message}`);
                 continue;
@@ -394,6 +396,7 @@ class CompiledResourceCollection {
      * @returns
      */
     _getCandidateValue(valueIndex) {
+        /* c8 ignore next 3 - defensive coding: candidate value index should be valid in compiled data */
         if (valueIndex < 0 || valueIndex >= this._candidateValues.length) {
             return (0, ts_utils_1.fail)(`Invalid candidate value index ${valueIndex}`);
         }
@@ -422,31 +425,38 @@ class CompiledResourceCollection {
             })
         });
         for (const compiledResource of compiled.resources) {
-            const resourceTypeResult = resourceTypes.getAt(compiledResource.type);
-            if (resourceTypeResult.isFailure()) {
-                errors.addMessage(`Invalid resource type index ${compiledResource.type} for resource ${compiledResource.id}: ${resourceTypeResult.message}`);
+            const resourceType = resourceTypes
+                .getAt(compiledResource.type)
+                .aggregateError(errors, (message) => `Invalid resource type index ${compiledResource.type} for resource ${compiledResource.id}: ${message}`)
+                .orDefault();
+            /* c8 ignore next 3 - defensive coding: resource type index should be valid in compiled data */
+            if (resourceType === undefined) {
                 continue;
             }
-            const resourceType = resourceTypeResult.value;
-            const decisionResult = decisions.getAt(compiledResource.decision);
-            if (decisionResult.isFailure()) {
-                errors.addMessage(`Invalid decision index ${compiledResource.decision} for resource ${compiledResource.id}: ${decisionResult.message}`);
+            const decision = decisions
+                .getAt(compiledResource.decision)
+                .aggregateError(errors, (message) => `Invalid decision index ${compiledResource.decision} for resource ${compiledResource.id}: ${message}`)
+                .orDefault();
+            /* c8 ignore next 3 - defensive coding: decision index should be valid in compiled data */
+            if (decision === undefined) {
                 continue;
             }
-            const decision = decisionResult.value;
             // Build candidates from compiled data
-            const candidateDeclsResult = (0, ts_utils_1.mapResults)(compiledResource.candidates.map((compiledCandidate) => this._getCandidateValue(compiledCandidate.valueIndex).onSuccess((json) => {
+            const candidateDecls = (0, ts_utils_1.mapResults)(compiledResource.candidates.map((compiledCandidate) => this._getCandidateValue(compiledCandidate.valueIndex)
+                .aggregateError(errors, (message) => `Failed to convert candidate JSON for resource ${compiledResource.id}: ${message}`)
+                .onSuccess((json) => {
                 return (0, ts_utils_1.succeed)({
                     json,
                     isPartial: compiledCandidate.isPartial,
                     mergeMethod: compiledCandidate.mergeMethod
                 });
-            })));
-            if (candidateDeclsResult.isFailure()) {
-                errors.addMessage(`Failed to convert candidate JSON for resource ${compiledResource.id}: ${candidateDeclsResult.message}`);
+            })))
+                .aggregateError(errors, (message) => `Failed to convert candidate JSON for resource ${compiledResource.id}: ${message}`)
+                .orDefault();
+            /* c8 ignore next 3 - defensive coding: candidate JSON conversion should not fail with valid compiled data */
+            if (candidateDecls === undefined) {
                 continue;
             }
-            const candidateDecls = candidateDeclsResult.value;
             // Create minimal candidates that implement IResourceCandidate
             const candidates = candidateDecls.map((candidateDecl) => ({
                 json: candidateDecl.json,
@@ -457,20 +467,21 @@ class CompiledResourceCollection {
                 conditionSet: baseCandidate.conditionSet,
                 value: candidates[idx].json
             }));
-            const concreteDecisionResult = decisions_2.ConcreteDecision.create({
+            const concreteDecision = decisions_2.ConcreteDecision.create({
                 decisions,
                 candidates: candidatesWithConditionSets
-            });
-            /* c8 ignore next 6 - defensive coding for ConcreteDecision creation failure */
-            if (concreteDecisionResult.isFailure()) {
-                errors.addMessage(`Failed to create concrete decision for resource ${compiledResource.id}: ${concreteDecisionResult.message}`);
+            })
+                .aggregateError(errors, (message) => `Failed to create concrete decision for resource ${compiledResource.id}: ${message}`)
+                .orDefault();
+            /* c8 ignore next 3 - defensive coding for ConcreteDecision creation failure */
+            if (concreteDecision === undefined) {
                 continue;
             }
-            const concreteDecision = concreteDecisionResult.value;
-            const { value: name, message: nameError } = common_1.Helpers.getNameForResourceId(compiledResource.id);
-            /* c8 ignore next 4 - defense in depth nearly impossible to reproduce */
-            if (nameError !== undefined) {
-                errors.addMessage(`Failed to get name for resource ${compiledResource.id}: ${nameError}`);
+            const name = common_1.Helpers.getNameForResourceId(compiledResource.id)
+                .aggregateError(errors, (message) => `Failed to get name for resource ${compiledResource.id}: ${message}`)
+                .orDefault();
+            /* c8 ignore next 3 - defensive coding: name should be valid in compiled data */
+            if (name === undefined) {
                 continue;
             }
             // Create minimal resource that implements IResource
@@ -481,13 +492,11 @@ class CompiledResourceCollection {
                 decision: concreteDecision,
                 candidates
             };
-            const setResult = resourceMap.set(compiledResource.id, resource);
-            /* c8 ignore next 3 - defensive coding for resource map set failure */
-            if (setResult.isFailure()) {
-                errors.addMessage(`Failed to add resource ${compiledResource.id}: ${setResult.message}`);
-            }
+            resourceMap
+                .set(compiledResource.id, resource)
+                .aggregateError(errors, (message) => `Failed to add resource ${compiledResource.id}: ${message}`);
         }
-        return errors.hasMessages ? (0, ts_utils_1.fail)(errors.toString()) : (0, ts_utils_1.succeed)(resourceMap);
+        return errors.returnOrReport((0, ts_utils_1.succeed)(resourceMap));
     }
 }
 exports.CompiledResourceCollection = CompiledResourceCollection;

package/lib/packlets/runtime/conditionSetResolutionResult.js

@@ -72,6 +72,7 @@ class ConditionSetResolutionResult {
                 return -1;
         }
         else if (a.matchType === 'matchAsDefault') {
+            /* c8 ignore next 5 - edge case comparison for matchAsDefault rarely hit in practice */
             if (b.matchType === 'match') {
                 return 1;
             }
@@ -80,6 +81,7 @@ class ConditionSetResolutionResult {
             }
         }
         else {
+            /* c8 ignore next 7 - edge case comparison for noMatch rarely hit in practice */
             // a.matchType === 'noMatch'
             if (b.matchType === 'noMatch') {
                 return 0;
@@ -100,6 +102,7 @@ class ConditionSetResolutionResult {
             }
             // If priority matches, compare score
             const scoreDiff = matchB.score - matchA.score;
+            /* c8 ignore next 3 - edge case: score comparison when priorities are equal */
             if (scoreDiff !== 0) {
                 return scoreDiff;
             }

package/lib/packlets/runtime/context/contextQualifierProvider.d.ts

@@ -2,11 +2,12 @@ import { Result } from '@fgv/ts-utils';
 import { QualifierName, QualifierContextValue, QualifierIndex } from '../../common';
 import { IReadOnlyQualifierCollector, Qualifier } from '../../qualifiers';
 /**
- * Abstract interface for providing qualifier values in an optimized runtime context.
+ * Base interface for providing qualifier values in an optimized runtime context.
+ * Contains common read-only operations shared by both mutable and immutable providers.
  * Acts as a property bag using the Result pattern for qualifier value lookups.
  * @public
  */
-export interface IContextQualifierProvider {
+export interface IContextQualifierProviderBase {
     /**
      * Gets a qualifier value by its name, index, or qualifier object.
      * @param nameOrIndexOrQualifier - The {@link QualifierName | qualifier name}, {@link QualifierIndex | index}, or {@link Qualifiers.Qualifier | qualifier object} to look up.
@@ -39,12 +40,61 @@ export interface IContextQualifierProvider {
      */
     readonly qualifiers: IReadOnlyQualifierCollector;
 }
+/**
+ * Read-only interface for providing qualifier values in an optimized runtime context.
+ * Explicitly marked as immutable with compile-time type discrimination.
+ * @public
+ */
+export interface IReadOnlyContextQualifierProvider extends IContextQualifierProviderBase {
+    /**
+     * Explicit mutability marker for compile-time type discrimination.
+     * Always `false` for read-only providers.
+     */
+    readonly mutable: false;
+}
+/**
+ * Mutable interface for providing qualifier values in an optimized runtime context.
+ * Extends the base interface with mutation operations and explicit mutability marker.
+ * @public
+ */
+export interface IMutableContextQualifierProvider extends IContextQualifierProviderBase {
+    /**
+     * Explicit mutability marker for compile-time type discrimination.
+     * Always `true` for mutable providers.
+     */
+    readonly mutable: true;
+    /**
+     * Sets a qualifier value in this provider.
+     * @param name - The {@link QualifierName | qualifier name} to set.
+     * @param value - The {@link QualifierContextValue | qualifier context value} to set.
+     * @returns `Success` with the set {@link QualifierContextValue | qualifier context value} if successful,
+     * or `Failure` with an error message if not.
+     */
+    set(name: QualifierName, value: QualifierContextValue): Result<QualifierContextValue>;
+    /**
+     * Removes a qualifier value from this provider.
+     * @param name - The {@link QualifierName | qualifier name} to remove.
+     * @returns `Success` with the removed {@link QualifierContextValue | qualifier context value} if successful,
+     * or `Failure` with an error message if not found or an error occurs.
+     */
+    remove(name: QualifierName): Result<QualifierContextValue>;
+    /**
+     * Clears all qualifier values from this provider.
+     */
+    clear(): void;
+}
+/**
+ * Union type for context qualifier providers that can be either read-only or mutable.
+ * Provides compile-time type discrimination via the `mutable` property.
+ * @public
+ */
+export type IContextQualifierProvider = IReadOnlyContextQualifierProvider | IMutableContextQualifierProvider;
 /**
  * Abstract base class for implementing context qualifier providers.
  * Provides common functionality and enforces the contract for derived classes.
  * @public
  */
-export declare abstract class ContextQualifierProvider implements IContextQualifierProvider {
+export declare abstract class ContextQualifierProvider implements IContextQualifierProviderBase {
     /**
      * The readonly qualifier collector that defines and validates the qualifiers for this context.
      */