@nxds/eslint-plugin 0.5.0 → 0.6.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nxds/eslint-plugin",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "type": "module",
   "description": "ESLint rules that enforce nxds conventions in copied components.",
   "keywords": [

package/src/rules/no-render-prop-types.js

@@ -2,7 +2,9 @@
 // should not be typed as render callbacks (`(...) => ReactNode`), component
 // references (`ComponentType<...>`, `FC<...>`), or unions thereof. Event
 // handler props (`onClick`, `onSubmit`, ...) return data, not JSX, and are
-// exempt.
+// exempt. So is a prop whose JSDoc carries `@thirdPartyShape <reason>`: a
+// third-party API mandates its shape. It is a plain tag, not an
+// `eslint-disable`, so copied source lints clean in apps without this plugin.
 //
 // Known scope gaps — these patterns sneak through the AST visitors below and
 // must be caught by code review:
@@ -14,6 +16,8 @@
 //   only).
 
 const EVENT_HANDLER = /^on[A-Z]/;
+// The tag counts only with a reason after it.
+const THIRD_PARTY_TAG = /@thirdPartyShape[ \t]+[^\s*]/;
 
 const REACT_NODE_NAMES = new Set([
   'ReactNode',
@@ -113,6 +117,10 @@ export default {
       if (EVENT_HANDLER.test(name)) {
         return;
       }
+      const comments = context.sourceCode.getCommentsBefore(node);
+      if (comments.some((comment) => THIRD_PARTY_TAG.test(comment.value))) {
+        return;
+      }
       const ann = node.typeAnnotation?.typeAnnotation;
       if (!ann) {
         return;