fix(Changelogs): storyId as string, fix parse last line

Author: Feverqwe

src/transform/plugins/changelog/collect.ts

@@ -6,7 +6,7 @@ import imsize from '../imsize';
 import {MarkdownItPluginOpts} from '../typings';
 
 const BLOCK_START = '{% changelog %}';
-const BLOCK_END = '{% endchangelog %}\n';
+const BLOCK_END = '{% endchangelog %}';
 
 function parseChangelogs(str: string, path?: string) {
     const {parse, compile, env} = initMarkdownit({
@@ -28,33 +28,38 @@ type Options = Pick<MarkdownItPluginOpts, 'path' | 'log'> & {
 const collect = (input: string, {path: filepath, log, changelogs, extractChangelogs}: Options) => {
     let result = input;
     let lastPos = 0;
-    const rawChanges = [];
+    const rawChangelogs = [];
 
     // eslint-disable-next-line no-constant-condition
     while (true) {
-        const pos = result.indexOf(BLOCK_START, lastPos);
-        lastPos = pos;
-        if (pos === -1) {
+        const startPos = result.indexOf(BLOCK_START, lastPos);
+        lastPos = startPos;
+        if (startPos === -1) {
             break;
         }
-        const endPos = result.indexOf(BLOCK_END, pos + BLOCK_START.length);
-        if (endPos === -1) {
+
+        const endBlockPos = result.indexOf(BLOCK_END, startPos + BLOCK_START.length);
+        if (endBlockPos === -1) {
             log.error(`Changelog block must be closed${filepath ? ` in ${bold(filepath)}` : ''}`);
             break;
         }
+        let endPos = endBlockPos + BLOCK_END.length;
+        if (result[endPos + 1] === '\n') {
+            endPos += 1;
+        }
 
-        const change = result.slice(pos, endPos + BLOCK_END.length);
+        const changelog = result.slice(startPos, endPos);
 
-        rawChanges.push(change);
+        rawChangelogs.push(changelog);
 
-        result = result.slice(0, pos) + result.slice(endPos + BLOCK_END.length);
+        result = result.slice(0, startPos) + result.slice(endPos);
     }
 
-    if (rawChanges.length && changelogs && extractChangelogs) {
-        const parsedChangelogs = parseChangelogs(rawChanges.join('\n\n'), filepath);
-        if (parsedChangelogs.length !== rawChanges.length) {
+    if (rawChangelogs.length && changelogs && extractChangelogs) {
+        const parsedChangelogs = parseChangelogs(rawChangelogs.join('\n\n'), filepath);
+        if (parsedChangelogs.length !== rawChangelogs.length) {
             log.error(
-                `Parsed cahngelogs less than expected${filepath ? ` in ${bold(filepath)}` : ''}`,
+                `Parsed changelogs less than expected${filepath ? ` in ${bold(filepath)}` : ''}`,
             );
         }
         changelogs.push(...parsedChangelogs);

src/transform/plugins/changelog/index.ts

@@ -55,10 +55,10 @@ function parseBody(tokens: Token[], state: StateCore) {
         throw new Error('Metadata tag not found');
     }
 
-    let metadata: object = {};
+    let metadata: Record<string, unknown> = {};
     const rawMetadata = yaml.load(metadataToken.content, {
         schema: yaml.JSON_SCHEMA,
-    });
+    }) as Record<string, unknown>;
     if (rawMetadata && typeof rawMetadata === 'object') {
         metadata = rawMetadata;
     }
@@ -93,6 +93,10 @@ function parseBody(tokens: Token[], state: StateCore) {
 
     const description = md.renderer.render(tokens, md.options, env);
 
+    if (typeof metadata.storyId === 'number') {
+        metadata.storyId = String(metadata.storyId);
+    }
+
     return {
         ...metadata,
         title,
@@ -135,13 +139,13 @@ const changelog: MarkdownItPluginCb<Options> = function (md, {extractChangelogs,
                 content.splice(-3);
 
                 try {
-                    const change = parseBody(content, state);
+                    const changelogLocal = parseBody(content, state);
 
                     if (!env.changelogs) {
                         env.changelogs = [];
                     }
 
-                    env.changelogs.push(change);
+                    env.changelogs.push(changelogLocal);
                 } catch (err) {
                     log.error(`Changelog error: ${(err as Error).message} in ${bold(path)}`);
                     continue;

test/changelog.test.ts

@@ -12,7 +12,7 @@ describe('Changelog', () => {
         return {
             date,
             index,
-            storyId: 123321,
+            storyId: '123321',
             title: 'Change log title',
             image: {
                 alt: 'My image',