fix(check-property-names): check duplicate nested names

Author: brettz9

README.md

@@ -1862,6 +1862,41 @@ The following patterns are considered problems:
  */
 // Message: Duplicate @property "foo"
 
+/**
+ * @typedef (SomeType) SomeTypedef
+ * @property cfg
+ * @property cfg.foo
+ * @property cfg.foo
+ */
+function quux ({foo, bar}) {
+
+}
+// Message: Duplicate @property "cfg.foo"
+
+/**
+ * @typedef (SomeType) SomeTypedef
+ * @property cfg
+ * @property cfg.foo
+ * @property [cfg.foo]
+ * @property baz
+ */
+function quux ({foo, bar}, baz) {
+
+}
+// Message: Duplicate @property "cfg.foo"
+
+/**
+ * @typedef (SomeType) SomeTypedef
+ * @property cfg
+ * @property cfg.foo
+ * @property [cfg.foo="with a default"]
+ * @property baz
+ */
+function quux ({foo, bar}, baz) {
+
+}
+// Message: Duplicate @property "cfg.foo"
+
 /**
  * @typedef (SomeType) SomeTypedef
  * @prop foo
@@ -1937,6 +1972,14 @@ function quux (code = 1) {
   this.error = new Error('oops');
   this.code = code;
 }
+
+/**
+ * @typedef (SomeType) SomeTypedef
+ * @property foo
+ * @property foo.bar
+ * @property foo.baz
+ * @property bar
+ */
 ````
 
 

src/rules/checkPropertyNames.js

@@ -6,7 +6,7 @@ const validatePropertyNames = (
   jsdoc, jsdocNode, utils,
 ) => {
   const propertyTags = entries(jsdoc.tags).filter(([, tag]) => {
-    return tag.tag === targetTagName && !tag.name.includes('.');
+    return tag.tag === targetTagName;
   });
 
   return propertyTags.some(([, tag], index) => {

test/rules/assertions/checkPropertyNames.js

@@ -56,6 +56,65 @@ export default {
            */
       `,
     },
+    {
+      code: `
+          /**
+           * @typedef (SomeType) SomeTypedef
+           * @property cfg
+           * @property cfg.foo
+           * @property cfg.foo
+           */
+          function quux ({foo, bar}) {
+
+          }
+      `,
+      errors: [
+        {
+          line: 6,
+          message: 'Duplicate @property "cfg.foo"',
+        },
+      ],
+    },
+    {
+      code: `
+          /**
+           * @typedef (SomeType) SomeTypedef
+           * @property cfg
+           * @property cfg.foo
+           * @property [cfg.foo]
+           * @property baz
+           */
+          function quux ({foo, bar}, baz) {
+
+          }
+      `,
+      errors: [
+        {
+          line: 6,
+          message: 'Duplicate @property "cfg.foo"',
+        },
+      ],
+    },
+    {
+      code: `
+          /**
+           * @typedef (SomeType) SomeTypedef
+           * @property cfg
+           * @property cfg.foo
+           * @property [cfg.foo="with a default"]
+           * @property baz
+           */
+          function quux ({foo, bar}, baz) {
+
+          }
+      `,
+      errors: [
+        {
+          line: 6,
+          message: 'Duplicate @property "cfg.foo"',
+        },
+      ],
+    },
     {
       code: `
           /**
@@ -191,5 +250,16 @@ export default {
           }
       `,
     },
+    {
+      code: `
+          /**
+           * @typedef (SomeType) SomeTypedef
+           * @property foo
+           * @property foo.bar
+           * @property foo.baz
+           * @property bar
+           */
+      `,
+    },
   ],
 };