canvas-native - npm Package Compare versions

Comparing version 2.3.3 to 2.3.4

dist/lib/formatters/summary.js

@@ -23,2 +23,13 @@ 'use strict';
 
+/**
+ * @module Formatters.Summary
+
+/**
+ * Format a CanvasNative document as a summary.
+ *
+ * @static
+ * @param {string} native A CanvasNative string
+ * @return {string} The summarized CanvasNative string
+ */
+
 function format(native) {
@@ -52,3 +63,8 @@ var result = '';
 
-      var next = (0, _removeMarkdown2['default'])(nextContent).replace(/\.?$/, '. ');
+      var next = (0, _removeMarkdown2['default'])(nextContent);
+      if (!/[.?!]$/.test(next)) {
+        next = next + '. ';
+      } else {
+        next = next + ' ';
+      }
 
@@ -55,0 +71,0 @@ if (next.length + result.length > 256) {

lib/formatters/summary.js

@@ -6,2 +6,12 @@ import Scanner from '../scanner';
 
+/**
+ * @module Formatters.Summary
+
+/**
+ * Format a CanvasNative document as a summary.
+ *
+ * @static
+ * @param {string} native A CanvasNative string
+ * @return {string} The summarized CanvasNative string
+ */
 export default function format(native) {
@@ -26,3 +36,8 @@ let result    = '';
 
-    const next = stripMd(nextContent).replace(/\.?$/, '. ');
+    let next = stripMd(nextContent);
+    if (!/[.?!]$/.test(next)) {
+      next = `${next}. `;
+    } else {
+      next = `${next} `;
+    }
 
@@ -29,0 +44,0 @@ if (next.length + result.length > 256) {

package.json

 {
   "name": "canvas-native",
   "description": "Utilities for working with the native Canvas format",
-  "version": "2.3.3",
+  "version": "2.3.4",
   "author": "Jonathan Clem <[e-mail]>",
@@ -6,0 +6,0 @@ "bugs": "https://github.com/usecanvas/canvas-native/issues",

test/lib/formatters/summary-test.js

@@ -22,2 +22,9 @@ import format     from '../../../lib/formatters/summary';
   });
+
+  ['!', '?'].forEach(punct => {
+    it(`does not add a period after existing "${punct}" punctuation`, () => {
+      const doc = parse(`Foo${punct}\nBar`);
+      expect(format(doc)).to.eq(`Foo${punct} Bar.`);
+    });
+  });
 });

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

124214

increased by0.65%

Lines of code

3517

increased by0.98%

Number of medium supply chain risk alerts

0

decreased by-100%

No dependency changes