Name format

src/rules/name-format.js

@@ -3,16 +3,32 @@ const LintIssue = require('./../LintIssue');
 
 const lintId = 'name-format';
 const nodeName = 'name';
-const message = 'Format should be all lowercase';
 const ruleType = 'standard';
+const maxLength = 214;
 
 const lint = (packageJsonData, severity) => {
-  if (!isLowercase(packageJsonData, nodeName)) {
-    return new LintIssue(lintId, severity, nodeName, message);
+  if (!packageJsonData.hasOwnProperty(nodeName)) {
+    return true;
+  }
+
+  const name = packageJsonData[nodeName];
+
+  if (!isLowercase(name)) {
+    return new LintIssue(lintId, severity, nodeName, 'Format should be all lowercase');
+  }
+
+  if (name.length > maxLength) {
+    return new LintIssue(lintId, severity, nodeName, `name should be less than or equal to ${maxLength} characters.`);
+  }
+
+  if (name.startsWith('.') || name.startsWith('_')) {
+    return new LintIssue(lintId, severity, nodeName, 'name should not start with . or _');
   }
 
   return true;
 };
 
-module.exports.lint = lint;
-module.exports.ruleType = ruleType;
+module.exports = {
+  lint,
+  ruleType
+};

src/validators/format.js

@@ -6,10 +6,6 @@ const semver = require('semver');
  * @return {boolean}      True if the string is lowercase or is missing. False if it is not.
  */
 const isLowercase = name => {
-  if (typeof name !== 'string') {
-    return true;
-  }
-
   return name === name.toLowerCase();
 };
 

test/unit/rules/name-format.test.js

@@ -10,7 +10,7 @@ describe('name-format Unit Tests', () => {
   });
 
   describe('when package.json has node with incorrect format', () => {
-    test('LintIssue object should be returned', () => {
+    test('not lowercase - LintIssue object should be returned', () => {
       const packageJsonData = {
         name: 'ImNotLowercase'
       };