@sap/cds-compiler 2.15.4 → 3.0.0

package/lib/json/from-csn.js

@@ -135,7 +135,7 @@ const schemaClasses = {
   condition: {
     arrayOf: exprOrString,
     type: condition,
-    msgId: 'syntax-csn-expected-term',
+    msgId: 'syntax-expected-term',
     // TODO: also specify requires here, and adapt onlyWith()
     optional: exprProperties,
   },
@@ -145,11 +145,11 @@ const schemaClasses = {
   },
   natnumOrStar: {
     type: natnumOrStar,
-    msgId: 'syntax-csn-expected-cardinality',
+    msgId: 'syntax-expected-cardinality',
   },
   columns: {
     arrayOf: selectItem,
-    msgId: 'syntax-csn-expected-column',
+    msgId: 'syntax-expected-column',
     defaultKind: '$column',
     validKinds: [], // pseudo kind '$column'
     // A column with only as+cast.type is a new association
@@ -290,7 +290,7 @@ const schema = compileSchema( {
   // type properties (except: elements, enum, keys, on): ---------------------
   type: {
     type: artifactRef,
-    msgId: 'syntax-csn-expected-reference',
+    msgId: 'syntax-expected-reference',
     optional: [ 'ref', 'global' ],
     inKind: [ 'element', 'type', 'param', 'mixin', 'event', 'annotation' ],
   },
@@ -366,7 +366,7 @@ const schema = compileSchema( {
   ref: {
     arrayOf: refItem,
     type: renameTo( 'path', arrayOf( refItem ) ),
-    msgId: 'syntax-csn-expected-reference',
+    msgId: 'syntax-expected-reference',
     minLength: 1,
     requires: 'id',
     optional: [ 'id', 'args', 'cardinality', 'where' ],
@@ -582,7 +582,7 @@ const schema = compileSchema( {
     inKind: [ 'element', '$column' ],
   },
   masked: {
-    type: boolOrNull,
+    type: masked,
     inKind: [ 'element' ],
   },
   notNull: {
@@ -779,7 +779,7 @@ function arrayOf( fn, filter = undefined ) {
     } );
     const minLength = spec.minLength || 0;
     if (minLength > val.length) {
-      message( 'syntax-csn-expected-length', location(true),
+      message( 'syntax-expected-length', location(true),
                { prop: spec.prop, n: minLength, '#': minLength === 1 ? 'one' : 'std' });
     }
     if (val.length)
@@ -849,7 +849,7 @@ function object( obj, spec ) {
   if (requires === undefined || requires === true) {
     // console.log(csnProps,JSON.stringify(spec))
     if (!relevantProps) {
-      error( 'syntax-csn-required-subproperty', location(true),
+      error( 'syntax-required-subproperty', location(true),
              {
                prop: spec.msgProp,
                '#': (
@@ -875,7 +875,7 @@ function object( obj, spec ) {
 
 function vZeroDelete( o, spec ) { // for old-CSN property 'origin'
   if (!csnVersionZero) {
-    warning( 'syntax-csn-zero-delete', location(true), { prop: spec.msgProp },
+    warning( 'syntax-zero-delete', location(true), { prop: spec.msgProp },
              'Delete/inline CSN v0.1.0 property $(PROP)' );
   }
   string( o, spec );
@@ -960,7 +960,7 @@ function definition( def, spec, xsn, csn, name ) {
 function dictionaryOf( elementFct ) {
   return function dictionary( dict, spec ) {
     if (!dict || typeof dict !== 'object' || Array.isArray( dict )) {
-      error( 'syntax-csn-expected-object', location(true),
+      error( 'syntax-expected-object', location(true),
              { prop: spec.prop }); // spec.prop, not spec.msgProp!
       return ignore( dict );
     }
@@ -972,7 +972,7 @@ function dictionaryOf( elementFct ) {
     ++virtualLine;
     for (const name of allNames) {
       if (!name) {
-        warning( 'syntax-csn-empty-name', location(true),
+        warning( 'syntax-empty-name', location(true),
                  { prop: spec.prop }, // TODO: Error
                  'Property names in dictionary $(PROP) must not be empty' );
       }
@@ -1049,11 +1049,11 @@ function validKind( val, spec, xsn ) {
   if (val === xsn.kind)         // has been set in definition - the same = ok!
     return undefined;           // already set in definition
   if (val === 'view' && xsn.kind === 'entity') {
-    warning( 'syntax-csn-zero-value', location(true), { prop: spec.msgProp },
+    warning( 'syntax-zero-value', location(true), { prop: spec.msgProp },
              'Replace CSN v0.1.0 value in $(PROP) by something specified' );
   }
   else if ((val === 'entity' || val === 'type') && xsn.kind === 'aspect') {
-    info( 'syntax-csn-aspect', location(true), { kind: 'aspect', '#': val },
+    info( 'syntax-aspect', location(true), { kind: 'aspect', '#': val },
           {
             std: 'Use the dedicated kind $(KIND) for aspect definitions',
             // eslint-disable-next-line max-len
@@ -1061,7 +1061,7 @@ function validKind( val, spec, xsn ) {
           } );
   }
   else {
-    error( 'syntax-csn-expected-valid', location(true), { prop: spec.msgProp },
+    error( 'syntax-expected-valid', location(true), { prop: spec.msgProp },
            'Expected valid string for property $(PROP)' );
   }
   return ignore( val );
@@ -1103,7 +1103,7 @@ function vZeroRef( name, spec, xsn ) {
     return;
   const path = name.split('.');
   if (!path.every( id => id)) {
-    warning( 'syntax-csn-expected-name', location(true), { prop: spec.msgProp },
+    warning( 'syntax-expected-name', location(true), { prop: spec.msgProp },
              'Expected correct name for property $(PROP)' );
   }
   xsn.path = path.map( id => ({ id, location: location() }) );
@@ -1114,7 +1114,7 @@ function vZeroRef( name, spec, xsn ) {
 function boolOrNull( val, spec ) {
   if ([ true, false, null ].includes( val ))
     return { val, location: location() };
-  warning( 'syntax-csn-expected-boolean', location(true), { prop: spec.msgProp },
+  warning( 'syntax-expected-boolean', location(true), { prop: spec.msgProp },
            'Expected boolean or null for property $(PROP)' );
   ignore( val );
   return { val: !!val, location: location() };
@@ -1124,7 +1124,7 @@ function string( val, spec ) {
   if (typeof val === 'string' && val)
     //  XSN TODO: do not require literal
     return val;
-  error( 'syntax-csn-expected-string', location(true), { prop: spec.msgProp },
+  error( 'syntax-expected-string', location(true), { prop: spec.msgProp },
          'Expected non-empty string for property $(PROP)' );
   return ignore( val );
 }
@@ -1133,7 +1133,7 @@ function stringVal( val, spec ) {
   if (typeof val === 'string' && val)
     //  XSN TODO: do not require literal
     return { val, literal: 'string', location: location() };
-  error( 'syntax-csn-expected-string', location(true), { prop: spec.msgProp },
+  error( 'syntax-expected-string', location(true), { prop: spec.msgProp },
          'Expected non-empty string for property $(PROP)' );
   return ignore( val );
 }
@@ -1155,7 +1155,7 @@ function natnum( val, spec ) {
   if (typeof val === 'number' && val >= 0)
     //  XSN TODO: do not require literal
     return { val, literal: 'number', location: location() };
-  error( spec.msgId || 'syntax-csn-expected-natnum', location(true),
+  error( spec.msgId || 'syntax-expected-natnum', location(true),
          { prop: spec.msgProp } );
   return ignore( val );
 }
@@ -1190,10 +1190,8 @@ function annoValue( val, spec ) {
     /** @type {string|boolean} */
     let seenEllipsis = false;
     if (arrayLevelCount > 0) {  // TODO: also inside structure (possible in CSN!)
-      if (val.some( isEllipsis )) {
-        error( 'syntax-csn-unexpected-ellipsis', location(true), { code: '...' },
-               'Unexpected $(CODE) in nested array' );
-      }
+      if (val.some( isEllipsis ))
+        error( 'syntax-unexpected-ellipsis', location(true), { '#': 'nested-array', code: '...' } );
     }
     else {
       for (const item of val) {
@@ -1202,7 +1200,7 @@ function annoValue( val, spec ) {
         }
         else if (isEllipsis( item )) { // with or without UP TO
           // error position at the beginning of the array, but that is fine
-          error( 'syntax-csn-duplicate-ellipsis', location(true), { code: '...' },
+          error( 'syntax-duplicate-ellipsis', location(true), { code: '...' },
                  'Expected no more than one $(CODE)' );
           break;
         }
@@ -1216,7 +1214,7 @@ function annoValue( val, spec ) {
     };
     arrayLevelCount--;
     if (seenEllipsis === 'upTo') {
-      error( 'syntax-csn-expecting-ellipsis', location(true), // at closing bracket
+      error( 'syntax-expecting-ellipsis', location(true), // at closing bracket
              { code: '... up to', newcode: '...' },
              // TODO: should we be more CSN specific in the message?
              'Expecting an array item $(NEWCODE) after an item with $(CODE)' );
@@ -1263,11 +1261,14 @@ function annoValue( val, spec ) {
 }
 
 function annotation( val, spec, xsn, csn, name ) {
-  const variantIndex = name.indexOf('#') + 1 || name.length;
-  const n = refSplit( name.substring( (xsn ? 1 : 0), variantIndex ), spec.msgProp );
+  const absolute = (xsn ? name.substring(1) : name);
+  // TODO: really care about variant (qualifier parts)?
+  const variantIndex = absolute.indexOf('#') + 1 || absolute.length;
+  const n = refSplit( absolute.substring( 0, variantIndex ), spec.msgProp );
   if (!n)
     return undefined;
-  if (variantIndex < name.length)
+  n.absolute = absolute;
+  if (variantIndex < absolute.length)
     n.variant = { id: name.substring( variantIndex ), location: location() };
   const r = annoValue( val, spec );
   r.name = n;
@@ -1282,7 +1283,7 @@ function value( val, spec, xsn ) { // for CSN property 'val'
       xsn.literal = (val === null) ? 'null' : typeof val;
     return val;
   }
-  error( 'syntax-csn-expected-scalar', location(true), { prop: spec.msgProp },
+  error( 'syntax-expected-scalar', location(true), { prop: spec.msgProp },
          'Only scalar values are supported for property $(PROP)' );
   return ignore( val );
 }
@@ -1294,7 +1295,7 @@ function literal( val, spec, xsn, csn ) {
     return val;
   if (typeof val === 'string' && validLiteralsExtra[val] === type)
     return val;
-  error( 'syntax-csn-expected-valid', location(true), { prop: spec.msgProp },
+  error( 'syntax-expected-valid', location(true), { prop: spec.msgProp },
          'Expected valid string for property $(PROP)' );
   return ignore( val );
 }
@@ -1309,7 +1310,7 @@ function func( val, spec, xsn ) {
 function xpr( exprs, spec, xsn, csn ) {
   if (csn.func) {
     if (!exprs.length) {
-      message( 'syntax-csn-expected-length', location(true),
+      message( 'syntax-expected-length', location(true),
                { prop: 'xpr', otherprop: 'func', '#': 'suffix' });
     }
     xsn.suffix = exprArgs( exprs, spec );
@@ -1342,7 +1343,7 @@ function args( exprs, spec ) {
     return arrayOf( exprOrString )( exprs, spec );
   }
   else if (!exprs || typeof exprs !== 'object') {
-    error( 'syntax-csn-expected-args', location(true),
+    error( 'syntax-expected-args', location(true),
            { prop: spec.prop }, // spec.prop, not spec.msgProp!
            'Expected array or object for property $(PROP)' );
     return ignore( exprs );
@@ -1405,12 +1406,12 @@ function condition( cond, spec ) {
 function vZeroValue( obj, spec, xsn ) {
   if (xsn.value) {
     // TODO: also "sign" xsn.value created by inValue to complain about both 'value' and 'ref' etc
-    warning( 'syntax-csn-unexpected-property', location(true), { prop: spec.msgProp },
+    warning( 'syntax-unexpected-property', location(true), { prop: spec.msgProp },
              'Unexpected CSN property $(PROP)' );
     return undefined;
   }
   if (!csnVersionZero) {
-    warning( 'syntax-csn-zero-delete', location(true), { prop: spec.msgProp },
+    warning( 'syntax-zero-delete', location(true), { prop: spec.msgProp },
              'Delete/inline CSN v0.1.0 property $(PROP)' );
   }
   return expr( obj, spec );
@@ -1457,6 +1458,12 @@ function excluding( array, spec, xsn ) {
   xsn.excludingDict = r;
 }
 
+function masked( val, spec ) {
+  message('syntax-invalid-masked', location(), { keyword: 'masked' },
+          'Keyword $(KEYWORD) not supported');
+  return boolOrNull( val, spec );
+}
+
 function duplicateExcluding( name, loc ) {
   error( 'duplicate-excluding', loc, { name, keyword: 'excluding' },
          'Duplicate $(NAME) in the $(KEYWORD) clause' );
@@ -1477,7 +1484,7 @@ function join( val, spec, xsn ) {
 
 function queryArgs( val, spec, xsn, csn ) {
   if (Array.isArray( val ) && val.length > 1 && !csn.op) {
-    warning( 'syntax-csn-expected-property', location(true),
+    warning( 'syntax-expected-property', location(true),
              { prop: 'args', otherprop: 'op' },
              'CSN property $(PROP) expects property $(OTHERPROP) to be specified' );
     xsn.op = { val: 'union', location: location() };
@@ -1496,7 +1503,7 @@ function i18nLang( val, spec, xsn, csn, langKey ) {
 function translations( keyVal, spec, xsn, csn, textKey ) {
   if (typeof keyVal === 'string') // allow empty string
     return { val: keyVal, literal: 'string', location: location() };
-  error( 'syntax-csn-expected-translation', location(true),
+  error( 'syntax-expected-translation', location(true),
          { prop: textKey, otherprop: spec.prop },
          'Expected string for text key $(PROP) of language $(OTHERPROP)' );
   return ignore( keyVal );
@@ -1510,7 +1517,7 @@ function getSpec( parentSpec, csn, prop, xor, expected, kind ) {
   if (!s || s.noPrefix && prop !== p0 ) {
     if (ourpropsRegex.test( prop )) {
       // TODO v2: Warning only with --sloppy
-      warning( 'syntax-csn-unknown-property', location(true), { prop },
+      warning( 'syntax-unknown-property', location(true), { prop },
                'Unknown CSN property $(PROP)' );
     }
     else {                      // TODO v2: always (i.e. also with message) add to $extra
@@ -1521,7 +1528,7 @@ function getSpec( parentSpec, csn, prop, xor, expected, kind ) {
     if (s.ignore)
       return { type: ignore };
     if (s.vZeroIgnore && s.vZeroIgnore === csn[prop]) { // for "op": "call"
-      warning( 'syntax-csn-zero-delete', location(true), { prop },
+      warning( 'syntax-zero-delete', location(true), { prop },
                'Delete/inline CSN v0.1.0 property $(PROP)' );
       return { type: ignore };
     }
@@ -1540,7 +1547,7 @@ function getSpec( parentSpec, csn, prop, xor, expected, kind ) {
     const variant = kind && s.inKind
       ? ([ 'extend', 'annotate' ].includes(kind) ? kind : 'def')
       : (parentSpec.msgProp ? 'std' : 'top');
-    message( 'syntax-csn-unexpected-property', location(true),
+    message( 'syntax-unexpected-property', location(true),
              {
                prop, otherprop: parentSpec.msgProp, kind, '#': variant,
              },
@@ -1598,13 +1605,13 @@ function onlyWith( spec, need, csn, prop, xor, expected ) {
     need = allowed.find( p => !xor[schema[p].xorGroup] ) || allowed[0];
   }
   if (prop) {
-    error( 'syntax-csn-dependent-property', location(true),
+    error( 'syntax-dependent-property', location(true),
            { prop, otherprop: need },
            'CSN property $(PROP) can only be used in combination with $(OTHERPROP)');
     xor['no:req'] = prop;
   }
   else if (!xor['no:req']) {
-    error( 'syntax-csn-required-property', location(true),
+    error( 'syntax-required-property', location(true),
            { prop: need, otherprop: spec.msgProp, '#': spec.prop },
            {  // TODO $(PARENT), TODO: do not use prop===0 hack
              std: 'Object in $(OTHERPROP) must have the property $(PROP)',
@@ -1626,7 +1633,7 @@ function checkAndSetXorGroup( group, prop, xor ) {
   if (prop === 'func' && xor[group] === 'xpr' ||
       prop === 'xpr' && xor[group] === 'func')
     return true;                // hack for window function: both func and xpr is allowed
-  error( 'syntax-csn-excluded-property', location(true),
+  error( 'syntax-excluded-property', location(true),
          { prop, otherprop: xor[group] },
          'CSN property $(PROP) can only be used alternatively to $(OTHERPROP)');
   return false;
@@ -1641,7 +1648,7 @@ function implicitName( ref ) {
 function replaceZeroProp( otherprop, prop ) {
   if (csnVersionZero)
     return;
-  warning( 'syntax-csn-zero-prop', location(true), { prop, otherprop },
+  warning( 'syntax-zero-prop', location(true), { prop, otherprop },
            'Replace CSN v0.1.0 property $(OTHERPROP) by $(PROP)' );
 }
 
@@ -1650,7 +1657,7 @@ function replaceZeroProp( otherprop, prop ) {
 function isArray( array, spec ) {
   if (Array.isArray( array ))
     return array;
-  error( 'syntax-csn-expected-array', location(true), { prop: spec.prop },
+  error( 'syntax-expected-array', location(true), { prop: spec.prop },
          'Expected array for property $(PROP)' );
   return ignore( array );
 }
@@ -1658,7 +1665,7 @@ function isArray( array, spec ) {
 function isObject( obj, spec ) {
   if (obj && typeof obj === 'object' && !Array.isArray( obj ))
     return obj;
-  error( spec.msgId || 'syntax-csn-expected-object', location(true),
+  error( spec.msgId || 'syntax-expected-object', location(true),
          { prop: spec.msgProp });
   return ignore( obj );
 }
@@ -1666,7 +1673,7 @@ function isObject( obj, spec ) {
 function refSplit( name, prop ) {
   const path = name.split('.');
   if (!path.every( id => id)) {
-    warning( 'syntax-csn-expected-name', location(true), { prop },
+    warning( 'syntax-expected-name', location(true), { prop },
              'Expected correct name for property $(PROP)' );
   }
   return { path: path.map( id => ({ id, location: location() }) ), location: location() };
@@ -1674,7 +1681,7 @@ function refSplit( name, prop ) {
 
 function replaceZeroValue( spec ) {
   if (!csnVersionZero && spec.vZeroFor == null) { // but 0 does not match!
-    warning( 'syntax-csn-zero-value', location(true), { prop: spec.msgProp },
+    warning( 'syntax-zero-value', location(true), { prop: spec.msgProp },
              'Replace CSN v0.1.0 value in $(PROP) by something specified' );
   }
 }
@@ -1704,7 +1711,7 @@ function pushLocation( obj ) {
   else if (!loc || typeof loc !== 'string') {
     if (loc)
       dollarLocations.push( null ); // must match with popLocation()
-    error( 'syntax-csn-expected-object', location(true), { prop: '$location' } );
+    error( 'syntax-expected-object', location(true), { prop: '$location' } );
   }
   // hidden feature: string $location
   const m = /:(\d+)(?::(\d+))?$/.exec( loc ); // extra '^'s at end deliberately left out
@@ -1817,7 +1824,7 @@ function parse( source, filename = 'csn.json', options = {}, messageFunctions )
       line,
       col: column,
     };
-    messageFunctions.error( 'syntax-csn-illegal-json', loc, { msg }, 'Illegal JSON: $(MSG)' );
+    messageFunctions.error( 'syntax-illegal-json', loc, { msg }, 'Illegal JSON: $(MSG)' );
     return xsn;
   }
 }

package/lib/json/to-csn.js

@@ -222,7 +222,7 @@ const operators = {
   unboundedFollowing: [ 'unbounded', 'following' ],
   following: postfix( [ 'following' ] ),
   frameBetween: exprs => [ 'between', ...exprs[0], 'and', ...exprs[1] ],
-  // xpr: (exprs) => [].concat( ...exprs ), see below - handled extra
+  ixpr: exprs => [].concat( ...exprs ), // xpr extra, due to extra parentheses
 };
 
 const csnDictionaries = [
@@ -725,7 +725,8 @@ function annotationsAndDocComment( node, annotated ) {
     const val = node[prop];
     // val.$priority isn't set for computed annotations like @Core.Computed
     // and @odata.containment.ignore
-    if (val.$priority && (val.$priority !== 'define') === annotated) {
+    // TODO: use $inferred instead special $priority value
+    if (val.$priority !== undefined && (!!val.$priority) === annotated) {
       // transformer (= value) takes care to exclude $inferred annotation assignments
       const sub = transformer( val );
       // As value() just has one value, so we do not provide ( val, csn, node, prop )
@@ -1588,10 +1589,11 @@ function compactExpr( e ) {     // TODO: options
  */
 function initModuleVars( options = { csnFlavor: 'gensrc' } ) {
   gensrcFlavor = options.parseCdl || options.csnFlavor === 'gensrc' ||
-                 options.toCsn && options.toCsn.flavor === 'gensrc';
-  universalCsn = (options.csnFlavor === 'universal' ||
-                  options.toCsn && options.toCsn.flavor === 'universal' ) &&
-    isBetaEnabled( options, 'enableUniversalCsn' ) && !options.parseCdl;
+               ( options.toCsn && options.toCsn.flavor === 'gensrc');
+  universalCsn = ( options.csnFlavor === 'universal' ||
+                   ( options.toCsn && options.toCsn.flavor === 'universal') ) &&
+                  isBetaEnabled( options, 'enableUniversalCsn' ) &&
+                  !options.parseCdl;
   strictMode = options.testMode;
   const proto = options.dictionaryPrototype;
   // eslint-disable-next-line no-nested-ternary
@@ -1599,8 +1601,8 @@ function initModuleVars( options = { csnFlavor: 'gensrc' } ) {
     ? proto
     : (proto) ? Object.prototype : null;
   withLocations = options.withLocations;
-  parensAsStrings = isDeprecatedEnabled( options, 'parensAsStrings' );
-  projectionAsQuery = isDeprecatedEnabled( options, 'projectionAsQuery' );
+  parensAsStrings = isDeprecatedEnabled( options, '_parensAsStrings' );
+  projectionAsQuery = isDeprecatedEnabled( options, '_projectionAsQuery' );
 }
 
 module.exports = {

package/lib/language/antlrParser.js

@@ -13,8 +13,8 @@ const antlr4 = require('antlr4');
 const { CompileMessage } = require('../base/messages');
 const errorStrategy = require('./errorStrategy');
 
-const Parser = require('../gen/languageParser').languageParser;
-const Lexer = require('../gen/languageLexer').languageLexer;
+const Parser = require('../gen/languageParser').default;
+const Lexer = require('../gen/languageLexer').default;
 
 // Error listener used for ANTLR4-generated parser
 class ErrorListener extends antlr4.error.ErrorListener {

package/lib/language/docCommentParser.js

@@ -1,6 +1,10 @@
 'use strict';
 
-const { splitLines } = require('../utils/file');
+const {
+  isWhitespaceOrNewLineOnly,
+  isWhitespaceCharacterNoNewline,
+  cdlNewLineRegEx,
+} = require('./textUtils');
 
 /**
  * Get the content of a JSDoc-like comment and remove all surrounding asterisks, etc.
@@ -18,39 +22,40 @@ function parseDocComment(comment) {
   if (comment.length <= 5) // at least "/***/"
     return null;
 
-  let lines = splitLines(comment);
+  let lines = comment.split(cdlNewLineRegEx);
 
   if (lines.length === 1) {
-    // special case for one-liners
-    // remove "/***/" and trim white space
-    const content = lines[0].replace(/^\/[*]{2,}/, '').replace(/\*\/$/, '').trim();
-    return isWhiteSpaceOnly(content) ? null : content;
+    // Special case for one-liners.
+    // Remove "/***/" and trim white space and asterisks.
+    const content = lines[0]
+      .replace(/^\/[*]{2,}/, '')
+      .replace(/\*+\/$/, '')
+      .trim();
+    return isWhitespaceOrNewLineOnly(content) ? null : content;
   }
 
-  lines[0] = removeHeaderFence(lines[0]);
+  // If the comment already has content on the first line, i.e. after `/**`,
+  // its leading whitespace is ignored for whitespace trimming.
+  const hasContentOnFirstLine = /\/\*+\s*\S/.test(lines[0]);
+
+  // First line, i.e. header, is always trimmed from left.
+  lines[0] = removeHeaderFence(lines[0]).trimStart();
   lines[lines.length - 1] = removeFooterFence(lines[lines.length - 1]);
 
-  if (isFencedComment(lines)) {
-    lines = lines.map((line, index) => ((index === 0) ? line : removeFence(line)));
-  }
-  else if (lines.length === 2) {
+  if (lines.length === 2) {
     // Comment that is essentially just a header + footer.
-    // First line, i.e. header, is always trimmed from left.
-    lines[0] = lines[0].trimStart();
-
     // If the second line starts with an asterisk then remove it.
-    // Otherwise trim all whitespace.
+    // Otherwise, trim all left whitespace.
     if ((/^\s*[*]/.test(lines[1])))
       lines[1] = removeFence(lines[1]);
     else
       lines[1] = lines[1].trimStart();
   }
+  else if (isFencedComment(lines)) {
+    lines = lines.map((line, index) => ((index === 0) ? line : removeFence(line)));
+  }
   else {
-    const firstNonEmptyLine = lines.find((line, index) => index !== 0 && /[^\s]/.test(line)) || '';
-    // Tabs are regarded as one space.
-    const spacesAtBeginning = firstNonEmptyLine.match(/^\s*/)[0].length;
-    if (spacesAtBeginning > 0)
-      lines = lines.map(line => removeWhitespace(line, spacesAtBeginning));
+    stripCommentIndentation(lines, hasContentOnFirstLine);
   }
 
   // Remove empty header and footer.
@@ -58,18 +63,47 @@ function parseDocComment(comment) {
   const endIndex = (lines[lines.length - 1] === '') ? lines.length - 1 : lines.length;
 
   const content = lines.slice(startIndex, endIndex).join('\n');
-
-  return isWhiteSpaceOnly(content) ? null : content;
+  return isWhitespaceOrNewLineOnly(content) ? null : content;
 }
 
 /**
- * Checks whether the given string is whitespace only, i.e. newline
- * spaces, tabs.
+ * Strips and counts the indentation from the given comment string.
+ * This function is similar to the one in multiLineStringParser.js, but does not
+ * have special handling for the first and last line of the string.
+ *
+ * @example
+ *     |        hello
+ *     |          world
+ *     |        foo bar
+ *   becomes
+ *     | hello
+ *     |   world
+ *     | foo bar
  *
- * @param {string} content
+ * @param {string[]} lines String split into lines.
+ * @param {boolean} ignoreFirstLine Whether to ignore the first line for indentation counting.
  */
-function isWhiteSpaceOnly(content) {
-  return /^\s*$/.test(content);
+function stripCommentIndentation(lines, ignoreFirstLine) {
+  const n = lines.length;
+
+  const minIndent = lines.reduce((min, line, index) => {
+    // Blank lines are ignored.
+    if (isWhitespaceOrNewLineOnly(line) || (index === 0 && ignoreFirstLine))
+      return min;
+
+    let count = 0;
+    const length = Math.min(min, line.length);
+    while (count < length && isWhitespaceCharacterNoNewline(line[count])) {
+      count++;
+    }
+    return Math.min(min, count);
+  }, Number.MAX_SAFE_INTEGER);
+
+  for (let i = (ignoreFirstLine ? 1 : 0); i < n; ++i) {
+    // Note: Line may be empty and have fewer characters than `min`.
+    //       In that case, slice() returns an empty string.
+    lines[i] = lines[i].slice(minIndent);
+  }
 }
 
 /**
@@ -85,17 +119,6 @@ function removeFence(line) {
   return line.replace(/^\s*[*]\s?/, '');
 }
 
-/**
- * Remove the TODO
- *
- * @param {string} line
- * @param {number} spaces Number of whitespace to remove at the beginning of the line
- * @returns {string} line without fence
- */
-function removeWhitespace(line, spaces) {
-  return line.replace(new RegExp(`^\\s{0,${ spaces }}`), ''); // Trailing spaces with '*'? => .replace(/\s+[*]$/, '');
-}
-
 /**
  * Removes a header fence, i.e. '/**'.
  * May remove more than two asterisks e.g. '/*******'