build: ensure Markdown nodes are not inserted when links array is empty

Planeshifter · Planeshifter · commit bf71cdee618b · 2024-01-07T17:38:57.000-05:00
diff --git a/lib/node_modules/@stdlib/_tools/remark/plugins/remark-namespace-toc/lib/transformer.js b/lib/node_modules/@stdlib/_tools/remark/plugins/remark-namespace-toc/lib/transformer.js
@@ -158,21 +158,21 @@ function transformer( tree, file ) {
 			if ( res.packages && res.packages.length > 0 ) {
 				usedPackages = usedPackages.concat( res.packages );
 			}
-			newNode = {
-				'type': 'html',
-				'value': res.toc
-			};
-			debug( 'Index: %d', index );
-			for ( i = index; i < parent.children.length; i++ ) {
-				if ( parent.children[ i ].value === TOC_END ) {
-					parent.children.splice( index+1, i-index-1, newNode );
-					break;
-				}
-			}
-			if ( i === parent.children.length - 1 ) {
-				throw new Error( 'invalid node. Invalid table of contents (toc) comment. Ensure that the Markdown file includes both starting and ending `toc` comments. Node: `' + node.value + '`.' );
-			}
 			if ( res.links.length > 0 ) {
+				newNode = {
+					'type': 'html',
+					'value': res.toc
+				};
+				debug( 'Index: %d', index );
+				for ( i = index; i < parent.children.length; i++ ) {
+					if ( parent.children[ i ].value === TOC_END ) {
+						parent.children.splice( index+1, i-index-1, newNode );
+						break;
+					}
+				}
+				if ( i === parent.children.length - 1 ) {
+					throw new Error( 'invalid node. Invalid table of contents (toc) comment. Ensure that the Markdown file includes both starting and ending `toc` comments. Node: `' + node.value + '`.' );
+				}
 				if ( found ) {
 					debug( 'Append links to links section...' );
 					newNode = {
diff --git a/lib/node_modules/@stdlib/_tools/remark/plugins/remark-stdlib-related/lib/transformer.js b/lib/node_modules/@stdlib/_tools/remark/plugins/remark-stdlib-related/lib/transformer.js
@@ -188,48 +188,50 @@ function transformer( tree, file ) {
 			if ( i === parent.children.length - 1 ) {
 				throw new Error( format( 'invalid node. Ensure that the Markdown file includes both a starting `<section class="related">` and closing `</section>\n\n<!-- /.related -->`. Node: `%s`.', node.value ) );
 			}
-			if ( found ) {
-				debug( 'Append links to links section...' );
-				newNode = {
-					'type': 'html',
-					'value': links.join( '\n\n' )
-				};
-				for ( i = 0; i < parent.children.length; i++ ) {
-					child = parent.children[ i ];
-					if (
-						child.type === 'html' &&
-						child.value === '<!-- <related-links> -->'
-					) {
-						parent.children.splice( i+1, 0, newNode );
-						break;
-					}
-				}
-			} else {
-				debug( 'Create <related-links> section...' );
-				newNodes = [
-					{
-						'type': 'html',
-						'value': '<!-- <related-links> -->'
-					},
-					{
+			if ( links.length > 0 ) {
+				if ( found ) {
+					debug( 'Append links to links section...' );
+					newNode = {
 						'type': 'html',
 						'value': links.join( '\n\n' )
-					},
-					{
-						'type': 'html',
-						'value': '<!-- </related-links> -->'
+					};
+					for ( i = 0; i < parent.children.length; i++ ) {
+						child = parent.children[ i ];
+						if (
+							child.type === 'html' &&
+							child.value === '<!-- <related-links> -->'
+						) {
+							parent.children.splice( i+1, 0, newNode );
+							break;
+						}
 					}
-				];
-				for ( i = 0; i < parent.children.length; i++ ) {
-					child = parent.children[ i ];
-					if (
-						child.type === 'html' &&
-						child.value === '<!-- /.links -->'
-					) {
-						c = parent.children;
-						c.splice.apply( c, [ i-1, 0 ].concat( newNodes ) );
-						found = true;
-						break;
+				} else {
+					debug( 'Create <related-links> section...' );
+					newNodes = [
+						{
+							'type': 'html',
+							'value': '<!-- <related-links> -->'
+						},
+						{
+							'type': 'html',
+							'value': links.join( '\n\n' )
+						},
+						{
+							'type': 'html',
+							'value': '<!-- </related-links> -->'
+						}
+					];
+					for ( i = 0; i < parent.children.length; i++ ) {
+						child = parent.children[ i ];
+						if (
+							child.type === 'html' &&
+							child.value === '<!-- /.links -->'
+						) {
+							c = parent.children;
+							c.splice.apply( c, [ i-1, 0 ].concat( newNodes ) );
+							found = true;
+							break;
+						}
 					}
 				}
 			}
