@sap/cds-dk 10.0.4 → 10.0.6

package/node_modules/@sap/cds-compiler/CHANGELOG.md

@@ -15,6 +15,32 @@ might use a deprecated flag only for a limited period of time.
 
 
 
+## Version 7.0.3 - 2026-07-21
+
+### Bug Fixes
+
+- **compiler:** When a backlink target is at least distantly related to the entity
+  embedding an association with a backlink comparison, the compiler issues a
+  warning ref-deprecated-backlink instead of reporting an error ref-invalid-backlink;
+  this is now also the case if the target entity is auto-exposed.
+- **sql:** Avoid dump in to.sql.migration with complex models.
+
+
+
+## Version 7.0.2 - 2026-07-14
+
+### Bug Fixes
+
+- **compiler:**
+  + Avoid dump for a new association in a projection with backlink comparison while reporting a missing backlink target element.
+  + Soften incompatibility concerning backlink comparisons: missing target elements which are referred to in the foreign keys or ON-condition of the backlink
+    association now only lead to a warning.
+  + Soften incompatibility concerning structure values for annotations: a structure property like `list` won't be rejected; we avoid an ambiguity with an
+    _expression_ `(1, 2, 3)` by putting `'=': true` next to `list: …` in the CSN, like in compiler v6. A `xpr` expression is rendered without a sibling `=`,
+    though.
+
+
+
 ## Version 7.0.1 - 2026-06-18
 
 ### ⚠ BREAKING CHANGES

package/node_modules/@sap/cds-compiler/bin/cdsse.js

@@ -201,7 +201,7 @@ function offset( buf, alsoSuffix ) { // for line and column
       return false;
   }
   let cursor = pos + column - 1;
-  // eslint-disable-next-line sonarjs/slow-regex
+  // eslint-disable-next-line sonarjs/super-linear-regex
   const prefix = /[a-z_0-9$]*$/i.exec( buf.substring( pos, cursor ) ).index + pos;
   const col = column + prefix - cursor;
   if (alsoSuffix)

package/node_modules/@sap/cds-compiler/bin/cdsv2m.js

@@ -39,7 +39,7 @@ function ria() {
   // 'Choose via $(ANNO) one of $(SORTED_ARTS) as redirection target for $(TARGET) in … $(ART) otherwise'
   // NOTE: regex match on message text not for productive code!
   for (const msgObj of msgs) {
-    // eslint-disable-next-line sonarjs/slow-regex
+    // eslint-disable-next-line sonarjs/super-linear-regex
     const matches = msgObj.message.match( /["“][^"”]+["”]/ug );
     matches.slice( 1, -2 ).forEach( (name) => {
       annotates[name.slice( 1, -1 )] = true;

package/node_modules/@sap/cds-compiler/doc/IncompatibleChanges_v7.md

@@ -173,7 +173,7 @@ In cds-compiler v7, we make `name-deprecated-$self` a non-configurable error.
 Before cds-compiler v7:  
 The CSN representation of an expression-like annotation value is an object with
 properties `=` and one of `ref`, `xpr`, `val`, `func`, `SELECT`, `list`.
-In `=` we capture the (space-normalized, comments removed) string representation of the expression,
+In `=` we capture the (space-normalized, comments removed) string CDL representation of the expression,
 the other one holds the parsed expression as a token stream.
 
 When the annotation is propagated ...
@@ -222,25 +222,32 @@ Resulting CSN:
 ```
 
 In cds-compiler v7:  
-* For anno-xpr that are a simple ref (simple: can be path, but no filter, no parameters), keep current behavior:
-  - Keep both `=` and `ref`.
+* For anno-xpr that are a simple ref (simple: can be path, but no filter, no parameters)
+  in the compiler and runtime CSN:
+  - In `=`, render the `.`-concatenated `ref` items without CDL escaping
+    (like for unchecked references, but unlike compiler v6);
+    e.g. for a reference, written as `(![in])` in CDL, use `{ '=': 'in', ref: ['in'] }` in CSN.
   - When rewriting is necessary, also adapt the `=`.
+* For expression which are a simple enum symbols, with or without surrounding parentheses:
+  - Just render the `#`, omit the `=` property.
+* For expressions which are represented in CSN by a simple `xpr`:
+  - Do not render a `=` property anymore, just render the `xpr`.
 * For all other expressions:
-  - Remove the `=`
-    + both in original and in propagated annotations
-    + no matter whether rewriting is necessary or not
-
-With the removal of `=`, any object with a `ref` or `xpr` or `val` inside an array
-is regarded as an expression value. This could introduce ambiguities. Outside arrays
-there is no danger of ambiguity due to flattening.
-To avoid such ambiguities in CDL, the parser in v7 throws an error for "ambiguous" sources like
+  - Use `=` property with value `true`, do not use the CDL source as value,
+    both in original and in propagated annotations.
+
+With the removal of `=`, any object in a CSN with just a `xpr`
+is regarded as an expression value. With that, a structure inside an array
+with just a `xpr` property could not be represented without ambiguity.
+
+Therefore, the parser in v7 reports an error for such invalid structure values:
 ```cds
 @A_xpr: [(2*foo)]
 @A_str: [{xpr: [{val: 2}, '*', {ref: ['foo']}]}]
 entity E { foo : Integer; }
 ```
-For: ref, xpr, val, func, SELECT, list.
-
+The compiler also reports an error for structures inside arrays containing a `=` property and
+a primary expression property (`ref`, `xpr`, `list`, `val`, `#`, `func`, `SELECT` and `SET`).
 
 ### Default for structures
 

package/node_modules/@sap/cds-compiler/lib/base/builtins.js

@@ -107,7 +107,10 @@ const exprProperties = [
  * Return whether JSON object `val` is a representation for an annotation expression
  */
 function isAnnotationExpression( val ) {
-  return val && primaryExprProperties.some( prop => val[prop] !== undefined );
+  return val && (val['=']
+    ? primaryExprProperties.some( prop => val[prop] !== undefined )
+    : val.xpr !== undefined && Object.keys( val ).length === 1 || val['#'] !== undefined
+  );
 }
 
 module.exports = {

package/node_modules/@sap/cds-compiler/lib/base/message-registry.js

@@ -147,6 +147,8 @@ const centralMessages = {
   'ref-invalid-source': { severity: 'Error' },
   'ref-invalid-target': { severity: 'Error' },
   'ref-invalid-backlink': { severity: 'Error', configurableFor: 'before_v8' },
+  'ref-missing-backlink-element': { severity: 'Warning', errorFor: [ 'v8' ] },
+  'ref-deprecated-backlink': { severity: 'Warning', errorFor: [ 'v8' ] },
   'ref-missing-self-counterpart': { severity: 'Error', configurableFor: true },
   'ref-sloppy-target': { severity: 'Error', configurableFor: 'deprecated_before_v8' },
 
@@ -546,8 +548,10 @@ const centralMessageTexts = {
     colon: 'Use a $(NEWCODE), not a $(CODE) between the element names in a reference',
   },
   'syntax-invalid-anno-struct': {
-    std: 'Most CSN processors interpret a structure with property $(PROP) as expression',
-    ref: 'The value of property $(PROP) must be a non-empty string, which most CSN processors interpret as reference',
+    std: 'Objects with properties $(PROP) and $(SIBLINGPROP) are reserved for expressions and can\'t be used as structures',
+    xpr: 'Objects with $(PROP) as only property are reserved for expressions and can\'t be used as structures',
+    '#': 'Objects with a property $(PROP) are reserved for enum values and can\'t be used as structures',
+    ref: 'The value of property $(PROP) in structures must be a non-empty string',
   },
   // 'syntax-ignoring-doc-comment' (Info)
   'syntax-unexpected-reserved-word': '$(CODE) is a reserved word - write $(DELIMITED) instead if you want to use it as name',
@@ -917,6 +921,7 @@ const centralMessageTexts = {
   'rewrite-undefined-key': {
     std: 'Can\'t redirect association $(NAME), as redirection target $(TARGET) does not project element $(ID), which is required to match foreign key of $(NAME)',
     backlink: 'Can\'t project association, as the current entity does not project element $(ID), which is required to match foreign key of the backlink association',
+    // TODO: delete text variant 'backlink' again
   },
   'type-unexpected-typeof': {
     std: 'Unexpected $(KEYWORD) for the type reference here',
@@ -1180,6 +1185,14 @@ const centralMessageTexts = {
     include: 'The open backlink target $(TARGET) of the aspect is unrelated to the current entity',
     backlink: 'Can\'t use $(ART) as backlink association, because its ON-condition contains a backlink comparison',
   },
+  'ref-deprecated-backlink': {
+    std: 'The target $(TARGET) of the backlink association is just distantly related to the current entity; do not use a comparison with $(ID)',
+    inferred: 'The target $(TARGET) of the backlink association $(NAME) is just distantly related to the current entity; provide an ON-condition not using a comparison with $(ID)',
+  },
+  'ref-missing-backlink-element': {
+    std: 'Element $(ID) must also be projected when projecting this association containing a backlink comparison',
+    def: 'Element $(ID) must be projected when defining this association containing a backlink comparison',
+  },
   'extend-columns': {
     std: 'Artifact $(ART) can\'t be extended with columns, only simple views/projections without JOINs and UNIONs can',
     join: 'Artifact $(ART) can\'t be extended with columns, because it contains a JOIN',

package/node_modules/@sap/cds-compiler/lib/checks/defaultValues.js

@@ -23,7 +23,6 @@ function validateDefaultValues( member, memberName, prop, path ) {
       // TODO: This check only counts the number of leading signs, not inbetween (e.g. 1 - - 1).
       //       The message also needs to be improved.
       if (i > 1)
-        // eslint-disable-next-line cds-compiler/message-no-quotes
         this.error(null, path, {}, 'Illegal number of unary \'+\'/\'-\' operators');
     }
   }

package/node_modules/@sap/cds-compiler/lib/compiler/populate.js

@@ -1094,6 +1094,7 @@ function populate( model ) {
         elem._main?._ancestors?.includes( target )) {
       // source and target of the model association are the same entity, and
       // the current main artifact is a suitable auto-redirection target → return it
+      // TODO: check with auto-exposed entities (which do not have _ancestors)
       return elem._main;
     }
     const elemScope = preferredElemScope( target, service, elem, assoc._main || assoc );

package/node_modules/@sap/cds-compiler/lib/compiler/resolve.js

@@ -1229,7 +1229,8 @@ function resolve( model ) {
   function checkBacklinkPartner( assoc ) {
     checkBareDollarSelf( assoc );
     // console.log('CBP:',assoc.kind,assoc._main?.name?.id,assoc.name.id,!!assoc.$backlink)
-    const target = assoc.$backlink?._origin?.target?._artifact;
+    const origin = assoc.$backlink?._origin;
+    const target = origin?.target?._artifact;
     if (!target)
       return;
     const main = assoc._main;
@@ -1239,7 +1240,7 @@ function resolve( model ) {
     if (chain)                  // standard correct case
       return;
 
-    if (target._ancestors) { // && assocHasForeignKeys( assoc.$backlink?._origin )) {
+    if (target._ancestors || target.$inferred === 'autoexposed') {
       // with non-cyclic projection ancestors, not only for managed assoc, but also
       // unmanaged, see test3/Associations/Backlinks/CrossEyedBacklink.sql-err.cds
       // Current entity is no valid redirection target for the backlink association
@@ -1254,10 +1255,10 @@ function resolve( model ) {
       // TODO: we should do the "simple projection ancestor" probably only if no
       // include is involved in the redirection chain
       if (redirectionChain( null, main, ancestor, true )) {
-        warning( 'ref-dubious-backlink', [ assoc.$backlink.location, assoc ],
-                 { target },
-                 // eslint-disable-next-line @stylistic/max-len
-                 'The target $(TARGET) of the backlink association has just a distant relationship to the current entity' );
+        const msg = (assoc.on.$inferred ? 'inferred' : 'std');
+        message( 'ref-deprecated-backlink', [ assoc.$backlink.location, assoc ],
+                 // eslint-disable-next-line @stylistic/object-curly-newline
+                 { '#': msg, target, id: '$self', name: origin.name.id } );
         return;
       }
     }

package/node_modules/@sap/cds-compiler/lib/compiler/tweak-assocs.js

@@ -450,13 +450,24 @@ function tweakAssocs( model ) {
           if (projectedKey.index > 0)
             targetElement.path.splice(0, projectedKey.index);
         }
+        else if (elem._outer?.$backlink === elem) {
+          const undef = targetElement.path[0];
+          setArtifactLink( undef, null );
+          setArtifactLink( targetElement, null );
+          const outer = elem._outer;
+          message( 'ref-missing-backlink-element',
+                   [ (outer.value || outer.name).location, outer ],
+                   { '#': (outer.value ? 'std' : 'def'), id: undef.id } );
+        // TODO: more info if element was already missing in "intermediate" projection
+        // TODO: collect all missing elements?
+        }
         else {
           setArtifactLink( targetElement.path[0], null );
           setArtifactLink( targetElement, null );
 
           const culprit = !elem.target.$inferred && elem.target ||
-          elem.value?.path?.[elem.value.path.length - 1] ||
-          elem;
+            elem.value?.path?.[elem.value.path.length - 1] ||
+            elem;
           // TODO: probably better to collect the non-projected foreign keys
           // and have one message for all
           error( 'rewrite-undefined-key', [ weakLocation( culprit.location ), elem ], {
@@ -825,8 +836,16 @@ function tweakAssocs( model ) {
       if (!item)
         return;     // just $self
       // console.log('REB:',assoc.name?.id,expr.path.map(i=>i.id))
-      if (item.id === assoc.name.id)
-        rewritePath( expr, item, assoc, assoc, null );
+      if (item.id === assoc.name.id &&
+          rewritePath( expr, item, assoc, assoc, null )) {
+        const undef = expr.path.find( i => !i._artifact );
+        const outer = assoc._outer;
+        message( 'ref-missing-backlink-element',
+                 [ (outer.value || outer.name).location, outer ],
+                 { '#': (outer.value ? 'std' : 'def'), id: undef.id } );
+        // TODO: more info if element was already missing in "intermediate" projection
+        // TODO: collect all missing elements?
+      }
     }
     else if (assoc._main.query) { // from ON cond of mixin element in query
       // here also $calc
@@ -877,7 +896,7 @@ function tweakAssocs( model ) {
           element: 'This element is not originally referred to in the ON-condition of association $(MEMBER) of $(ART)',
         } );
       }
-      rewritePath( expr, item, assoc, elem, null );
+      rewritePath( expr, item, assoc, elem, null ); // TODO: really no error?
     }
   }
 
@@ -1044,6 +1063,8 @@ function tweakAssocs( model ) {
    * Rewrite the reference `ref` with first elem/mixin ref item `item` for user
    * `assoc` (the query element).  `elem` is usually the first (or preferred)
    * query element for item.
+   *
+   * If `location` is falsy, do not issue an error.
    */
   function rewritePath( ref, item, assoc, elem, location ) {
     const { path } = ref;
@@ -1092,7 +1113,7 @@ function tweakAssocs( model ) {
         if (state && state !== true)
           continue;
         if (!state && !location)
-          return true;
+          return true;          // no rewrite possible
         break;
       }
     }

package/node_modules/@sap/cds-compiler/lib/edm/annotations/edmJson.js

@@ -978,6 +978,11 @@ function xpr2edmJson( carrier, anno, location, options, messageFunctions, generi
       return xpr;
     }
 
+    // "'=': true" has no meaning as an OData operator here and must be removed
+    // before transform runs, otherwise transform['='] misinterprets it as $Eq.
+    if (xpr['='] === true)
+      delete xpr['='];
+
     if (xpr.list)
       xpr.list.forEach( preTransformExpression );
 

package/node_modules/@sap/cds-compiler/lib/json/from-csn.js

@@ -117,7 +117,7 @@
 
 const { dictAdd } = require('../compiler/dictionaries');
 const { quotedLiteralPatterns } = require('../compiler/builtins');
-const { exprProperties, primaryExprProperties } = require('../base/builtins');
+const { exprProperties, isAnnotationExpression } = require('../base/builtins');
 const { CompilerAssertion } = require('../base/error');
 const { Location } = require('../base/location');
 const { XsnSource } = require('../compiler/xsn-model');
@@ -1490,10 +1490,10 @@ function annoValue( val, spec ) {
     }
     return retval;
   }
-  else if (primaryExprProperties.some( prop => val[prop] !== undefined )) {
+  else if (isAnnotationExpression( val )) {
     const s = schema['@'].schema['-expr'];
     const r = object( val, s );
-    r.$tokenTexts = true;       // property `=` might not be present
+    r.$tokenTexts = true;
     return r;
   }
   else if (val['...'] !== undefined && Object.keys( val ).length === 1) {

package/node_modules/@sap/cds-compiler/lib/json/to-csn.js

@@ -1149,10 +1149,11 @@ function annoValue( node ) {
   if (!node.$tokenTexts)
     return value( node );
   const csn = expression( node ); // expressions in annotation values
-  return (!csn?.ref || gensrcFlavor || universalCsn ||
-          csn.cast || csn.param || csn.ref.some( id => typeof id !== 'string' ))
-    ? csn
-    : Object.assign({ '=': csn.ref.join( '.' ) }, csn );
+  if (!csn || csn['#'] || csn.xpr && Object.keys( csn ).length === 1)
+    return csn;                 // '#' and 'xpr' without sibling '='
+  const asTrue = !csn?.ref || gensrcFlavor || universalCsn ||
+    csn.cast || csn.param || csn.ref.some( id => typeof id !== 'string' );
+  return Object.assign({ '=': !!asTrue || csn.ref.join( '.' ) }, csn );
 }
 
 function value( node ) {

package/node_modules/@sap/cds-compiler/lib/model/csnUtils.js

@@ -920,12 +920,10 @@ const _dependents = Symbol('_dependents');
  * _dependencies: All artifacts this artifact depends on (because it has a ref to it)
  *
  * @param {CSN.Model} csn A CSN to enrich in-place
- * @param {object} refs csnRefs, only used for artifactRef
  * @returns {object} CSN with _dependents/_dependencies set, "cleanup" function, _dependents/_dependencies Symbol used
  */
-function setDependencies( csn, refs = csnRefs(csn) ) {
+function setDependencies( csn ) {
   const cleanup = [];
-  const { artifactRef } = refs;
 
   forEachDefinition(csn, (artifact, artifactName) => {
     const queries = getNormalizedQuery(artifact).query;
@@ -933,11 +931,18 @@ function setDependencies( csn, refs = csnRefs(csn) ) {
       initDependencies(artifact);
       forAllQueries(queries, (query) => {
         if (query.SELECT?.from) {
-          if (query.SELECT.from.args)
+          if (query.SELECT.from.args) {
             handleArgs(artifact, artifactName, query.SELECT.from.args);
-
-          else if (typeof query.SELECT.from === 'string' || query.SELECT.from.ref)
-            handleDependency(artifactRef.from(query.SELECT.from), artifact, artifactName);
+          }
+          else if (typeof query.SELECT.from === 'string') {
+            handleDependency(csn.definitions[query.SELECT.from], artifact, artifactName);
+          }
+          else if (query.SELECT.from.ref) {
+            if (query.SELECT.from.ref[0].id)
+              handleDependency(csn.definitions[query.SELECT.from.ref[0].id], artifact, artifactName);
+            else
+              handleDependency(csn.definitions[query.SELECT.from.ref[0]], artifact, artifactName);
+          }
         }
       }, [ 'definitions', artifactName, (artifact.projection ? 'projection' : 'query') ]);
     }
@@ -952,7 +957,7 @@ function setDependencies( csn, refs = csnRefs(csn) ) {
       if (arg.args)
         handleArgs(artifact, artifactName, arg.args);
       else if (arg.ref)
-        handleDependency(artifactRef.from(arg), artifact, artifactName);
+        handleDependency(csn.definitions[arg.ref[0].id ? arg.ref[0].id : arg.ref[0]], artifact, artifactName);
     }
   }
 

package/node_modules/@sap/cds-compiler/lib/parsers/AstBuildingParser.js

@@ -1190,15 +1190,26 @@ class AstBuildingParser extends BaseParser {
   }
 
   checkStructProps( dict ) {
+    // see also isAnnotationExpression()
+    if (!dict['=']) {
+      const prop = dict['#'] && '#' ||
+        dict.xpr && Object.keys( dict ).length === 1 && 'xpr';
+      if (!prop)
+        return;
+      this.error( 'syntax-invalid-anno-struct', dict[prop].name.location,
+                  { '#': prop, prop } );
+      return;
+    }
     const prop = primaryExprProperties.find( p => dict[p] !== undefined );
     if (prop) {
       this.error( 'syntax-invalid-anno-struct', dict[prop].name.location,
-                  { '#': 'std', prop } );
+                  { '#': 'std', prop, siblingprop: '=' } );
     }
-    else if (dict['='] && (dict['='].literal !== 'string' || !dict['='].val)) {
+    else if (dict['='].literal !== 'string' || !dict['='].val) {
       this.error( 'syntax-invalid-anno-struct', dict['='].location,
                   { '#': 'ref', prop: '=' } );
     }
+    // TODO: also check '...' ?
   }
 
   // TODO: remove the check from the parser; move it to shared.js
@@ -1635,7 +1646,7 @@ function addOneForDefinition( count, ext ) {
 
 // Significant digits (before exponent) without leading and trailing zeros
 function relevantDigits( val ) {
-  // eslint-disable-next-line sonarjs/slow-regex
+  // eslint-disable-next-line sonarjs/super-linear-regex
   val = val.replace( /e.+$/i, '' ); // this regex has no newlines -> is not slow
 
   const init = /^[-+0.]+/g;     // global flag to have lastIndex

package/node_modules/@sap/cds-compiler/lib/tool-lib/enrichCsn.js

@@ -143,7 +143,7 @@ function enrichCsn( csn, options = {} ) {
     }
     else {
       const record = !isAnnotationExpression( obj ) && assignment;
-      // is record without primary expression property
+      // is record without primary expression property and usually '='
       for (const name of Object.getOwnPropertyNames( obj ) ) {
         const trans = record || transformers[name] || transformers[name.charAt(0)] || standard;
         trans( obj, name, obj[name] );

package/node_modules/@sap/cds-compiler/lib/transform/db/cdsPersistence.js

@@ -51,11 +51,12 @@ function getAssocToSkippedIgnorer( csn, options, messageFunctions, csnUtils ) {
    *
    * @param {CSN.Artifact} artifact
    * @param {string} artifactName
+   * @param {string} _prop
+   * @param {CSN.Path} _path
    */
   function ignoreAssociationToSkippedTarget( artifact, artifactName, _prop, _path ) {
-    if (isPersistedOnDatabase(artifact)) {
+    if (isPersistedOnDatabase(artifact))
       forEachMemberRecursively(artifact, ignore, [ 'definitions', artifactName ]);
-    }
   }
 
   /**

package/node_modules/@sap/cds-compiler/lib/transform/db/views.js

@@ -130,6 +130,7 @@ function getViewTransformer( csn, options, _messageFunctions ) {
    * @param {CSN.Element} elem
    * @param {string} elemName
    * @param {CSN.Path} elementsPath Path pointing to elements
+   * @param {CSN.Path} _queryPath
    */
   function handleAssociationElement( query, elements, columnMap, publishedMixins, elem, elemName, elementsPath, _queryPath ) {
     const isNotMixinByItself = checkIsNotMixinByItself(query, columnMap, elemName);
@@ -391,9 +392,8 @@ function getColumnMap( query, csnUtils ) {
 function ensureColumnNames( query, csnUtils, _options ) {
   const select = query.SELECT || query.projection;
   for (const col of select?.columns || []) {
-    if (col !== '*' && !columnAlias(col)) {
+    if (col !== '*' && !columnAlias(col))
       setProp(col, 'as', csnUtils.getColumnName(col));
-    }
   }
 }
 

package/node_modules/@sap/cds-compiler/lib/transform/transformUtils.js

@@ -218,9 +218,8 @@ function getTransformers(model, options, msgFunctions, pathDelimiter = '_') {
       if (flattenedNames.length === 1) {
         // Single element: propagate default if leaf doesn't have one
         const singleElem = result[flattenedNames[0]];
-        if (singleElem.default?.val === undefined) {
+        if (singleElem.default?.val === undefined)
           singleElem.default = elem.default; // Copy entire default object {val, location}
-        }
       }
       else if (flattenedNames.length > 1) {
         // Multi-element structure: warn that default is ignored

package/node_modules/@sap/cds-compiler/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sap/cds-compiler",
-  "version": "7.0.1",
+  "version": "7.0.3",
   "description": "CDS (Core Data Services) compiler and backends",
   "homepage": "https://cap.cloud.sap/",
   "author": "SAP SE (https://www.sap.com)",

package/node_modules/media-typer/README.md

@@ -1,18 +1,18 @@
 # media-typer
 
-[![NPM Version][npm-version-image]][npm-url]
-[![NPM Downloads][npm-downloads-image]][npm-url]
+[![NPM Version][npm-image]][npm-url]
+[![NPM Downloads][downloads-image]][downloads-url]
 [![Node.js Version][node-version-image]][node-version-url]
-[![Build Status][travis-image]][travis-url]
+[![Build Status][ci-image]][ci-url]
 [![Test Coverage][coveralls-image]][coveralls-url]
 
 Simple RFC 6838 media type parser.
 
-This module will parse a given media type into it's component parts, like type,
+This module will parse a given media type into its component parts, like type,
 subtype, and suffix. A formatter is also provided to put them back together and
 the two can be combined to normalize media types into a canonical form.
 
-If you are looking to parse the string that represents a media type and it's
+If you are looking to parse the string that represents a media type and its
 parameters in HTTP (for example, the `Content-Type` header), use the
 [content-type module](https://www.npmjs.com/package/content-type).
 
@@ -75,19 +75,20 @@ If any of the given object values are invalid, then a `TypeError` is thrown.
 var valid = typer.test('image/svg+xml')
 ```
 
-Validate a media type string. This will return `true` is the string is a well-
+Validate a media type string. This will return `true` if the string is a well-
 formatted media type, or `false` otherwise.
 
 ## License
 
 [MIT](LICENSE)
 
-[coveralls-image]: https://badgen.net/coveralls/c/github/jshttp/media-typer/master
-[coveralls-url]: https://coveralls.io/r/jshttp/media-typer?branch=master
-[node-version-image]: https://badgen.net/npm/node/media-typer
-[node-version-url]: https://nodejs.org/en/download
-[npm-downloads-image]: https://badgen.net/npm/dm/media-typer
+[ci-image]: https://img.shields.io/github/workflow/status/jshttp/media-typer/ci/master?label=ci
+[ci-url]: https://github.com/jshttp/media-typer/actions/workflows/ci.yml
+[npm-image]: https://img.shields.io/npm/v/media-typer.svg
 [npm-url]: https://npmjs.org/package/media-typer
-[npm-version-image]: https://badgen.net/npm/v/media-typer
-[travis-image]: https://badgen.net/travis/jshttp/media-typer/master
-[travis-url]: https://travis-ci.org/jshttp/media-typer
+[node-version-image]: https://img.shields.io/node/v/media-typer.svg
+[node-version-url]: https://nodejs.org/en/
+[coveralls-image]: https://img.shields.io/coveralls/jshttp/media-typer/master.svg
+[coveralls-url]: https://coveralls.io/r/jshttp/media-typer?branch=master
+[downloads-image]: https://img.shields.io/npm/dm/media-typer.svg
+[downloads-url]: https://npmjs.org/package/media-typer

package/node_modules/media-typer/index.js

@@ -83,10 +83,6 @@ function format (obj) {
  */
 
 function test (string) {
-  if (!string) {
-    throw new TypeError('argument string is required')
-  }
-
   if (typeof string !== 'string') {
     throw new TypeError('argument string is required to be a string')
   }
@@ -103,10 +99,6 @@ function test (string) {
  */
 
 function parse (string) {
-  if (!string) {
-    throw new TypeError('argument string is required')
-  }
-
   if (typeof string !== 'string') {
     throw new TypeError('argument string is required to be a string')
   }

package/node_modules/media-typer/package.json

@@ -1,20 +1,24 @@
 {
   "name": "media-typer",
   "description": "Simple RFC 6838 media type parser and formatter",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "author": "Douglas Christopher Wilson <doug@somethingdoug.com>",
   "license": "MIT",
   "repository": "jshttp/media-typer",
+  "funding": {
+    "type": "opencollective",
+    "url": "https://opencollective.com/express"
+  },
   "devDependencies": {
-    "eslint": "5.16.0",
-    "eslint-config-standard": "12.0.0",
-    "eslint-plugin-import": "2.17.2",
-    "eslint-plugin-markdown": "1.0.0",
-    "eslint-plugin-node": "8.0.1",
-    "eslint-plugin-promise": "4.1.1",
-    "eslint-plugin-standard": "4.0.0",
-    "mocha": "6.1.4",
-    "nyc": "14.0.0"
+    "eslint": "7.23.0",
+    "eslint-config-standard": "14.1.1",
+    "eslint-plugin-import": "2.22.1",
+    "eslint-plugin-markdown": "1.0.2",
+    "eslint-plugin-node": "11.1.0",
+    "eslint-plugin-promise": "4.3.1",
+    "eslint-plugin-standard": "4.1.0",
+    "mocha": "9.2.0",
+    "nyc": "15.1.0"
   },
   "files": [
     "LICENSE",
@@ -27,7 +31,7 @@
   "scripts": {
     "lint": "eslint --plugin markdown --ext js,md .",
     "test": "mocha --reporter spec --check-leaks --bail test/",
-    "test-cov": "nyc --reporter=html --reporter=text npm test",
-    "test-travis": "nyc --reporter=text npm test"
+    "test-ci": "nyc --reporter=lcov --reporter=text npm test",
+    "test-cov": "nyc --reporter=html --reporter=text npm test"
   }
 }