eslint-plugin-use-agnostic 1.6.9 → 1.6.10

package/library/_commons/constants/bases.js

@@ -46,6 +46,8 @@ export const importBreaksCommentedImportRulesMessageId =
 export const noCommentedDirectiveMessageId = "no-commented-directive-detected";
 export const commentedDirectiveVerificationFailedMessageId =
   "commented-directive-verification-failed";
+export const commentedDirectiveReactDirectiveFailedMessageId =
+  "commented-directive-correct-react-directive-failed";
 export const importNotStrategizedMessageId =
   "import-from-use-agnostic-strategies-not-strategized";
 export const exportNotStrategizedMessageId =

package/library/directive21/_commons/constants/bases.js

@@ -11,6 +11,11 @@ import {
   USE_AGNOSTIC_CONDITIONS as COMMONS_USE_AGNOSTIC_CONDITIONS,
   USE_AGNOSTIC_STRATEGIES as COMMONS_USE_AGNOSTIC_STRATEGIES,
 } from "../../../_commons/constants/bases.js";
+import {
+  USE_SERVER,
+  USE_CLIENT,
+  USE_AGNOSTIC,
+} from "../../../agnostic20/_commons/constants/bases.js";
 
 import { makeIntroForSpecificViolationMessage } from "../../../_commons/utilities/helpers.js";
 
@@ -129,6 +134,8 @@ export const importedFileCommentedDirective = "importedFileCommentedDirective";
 export const commentedDirectiveMessage = "commentedDirectiveMessage";
 export const specificViolationMessage = "specificViolationMessage";
 export const specificFailure = "specificFailure";
+export const verifiedCommentedDirective = "verifiedCommentedDirective";
+export const expectedReactDirectiveAsText = "expectedReactDirectiveAsText";
 export const currentFileEnvironment = "currentFileEnvironment";
 export const importedFileEnvironment = "importedFileEnvironment";
 
@@ -388,3 +395,24 @@ export const environments_allowedChainImportEnvironments = Object.freeze({
   [CLIENT]: client_allowedChainImportEnvironments,
   [AGNOSTIC]: agnostic_allowedChainImportEnvironments,
 });
+
+export const commentedDirectives_reactDirectives = Object.freeze({
+  [USE_SERVER_LOGICS]: null,
+  [USE_CLIENT_LOGICS]: "use client",
+  [USE_AGNOSTIC_LOGICS]: "use agnostic",
+  [USE_SERVER_COMPONENTS]: null,
+  [USE_CLIENT_COMPONENTS]: "use client",
+  [USE_AGNOSTIC_COMPONENTS]: "use agnostic",
+  [USE_SERVER_FUNCTIONS]: "use server",
+  [USE_CLIENT_CONTEXTS]: "use client",
+  [USE_AGNOSTIC_CONDITIONS]: null,
+  [USE_AGNOSTIC_STRATEGIES]: null,
+});
+
+/** @type {Map<typeof USE_SERVER | typeof USE_CLIENT | typeof USE_AGNOSTIC | null, `the React "${typeof USE_SERVER}"` | `the React "${typeof USE_CLIENT}"` | `the React "${typeof USE_AGNOSTIC}"` | "no React" `>} */
+export const reactDirectives_asTexts = new Map([
+  [USE_SERVER, `the React "${USE_SERVER}"`],
+  [USE_CLIENT, `the React "${USE_CLIENT}"`],
+  [USE_AGNOSTIC, `the React "${USE_AGNOSTIC}"`],
+  [null, "no React"],
+]);

package/library/directive21/_commons/rules/import-rules.js

@@ -3,6 +3,7 @@ import {
   importBreaksCommentedImportRulesMessageId,
   noCommentedDirectiveMessageId,
   commentedDirectiveVerificationFailedMessageId,
+  commentedDirectiveReactDirectiveFailedMessageId,
   importNotStrategizedMessageId,
   exportNotStrategizedMessageId,
   cantChainImportAcrossEnvironmentsMessageId,
@@ -13,6 +14,8 @@ import {
   commentedDirectiveMessage,
   specificViolationMessage,
   specificFailure,
+  verifiedCommentedDirective,
+  expectedReactDirectiveAsText,
   currentFileEnvironment,
   importedFileEnvironment,
 } from "../constants/bases.js";
@@ -45,6 +48,7 @@ In this case, {{ ${specificViolationMessage} }} `,
 All targeted modules need to be marked with their respective directives (\`// "use server logics"\`, etc.) for the purpose of this linting rule, evaluated from the first JavaScript comment starting on the first column within the first three lines of a module. `,
       [commentedDirectiveVerificationFailedMessageId]: `The commented directive could not pass verification due to an incompatible combination with its file extension.
 In this context, {{ ${specificFailure} }} `,
+      [commentedDirectiveReactDirectiveFailedMessageId]: `Commented directive "{{ ${verifiedCommentedDirective} }}" requires {{ ${expectedReactDirectiveAsText} }} directive in order to communicate accordingly with the React architecture at hand. `,
       [importNotStrategizedMessageId]: `Imports from Agnostic Strategies Modules must be strategized (\`/* @serverLogics */\`, etc.).  
 Please include a Strategy that corresponds to the kind of module this import would be mapped to. `,
       [exportNotStrategizedMessageId]: `Exports from Agnostic Strategies Modules must be strategized (\`/* @serverLogics */\`, etc.).  

package/library/directive21/_commons/utilities/flows.js

@@ -9,6 +9,7 @@ import {
   importBreaksCommentedImportRulesMessageId,
   noCommentedDirectiveMessageId,
   commentedDirectiveVerificationFailedMessageId,
+  commentedDirectiveReactDirectiveFailedMessageId,
   importNotStrategizedMessageId,
   cantChainImportAcrossEnvironmentsMessageId,
   skip,
@@ -23,10 +24,15 @@ import {
   commentedDirectiveMessage,
   specificViolationMessage,
   specificFailure,
+  // verifiedCommentedDirective,
+  // expectedReactDirectiveAsText,
   environments_allowedChainImportEnvironments,
+  commentedDirectives_reactDirectives,
+  reactDirectives_asTexts,
 } from "../constants/bases.js";
 
 import { highlightFirstLineOfCode } from "../../../_commons/utilities/helpers.js";
+import { getDirectiveFromCurrentModule } from "../../../agnostic20/_commons/utilities/helpers.js";
 import {
   getCommentedDirectiveFromCurrentModule,
   getVerifiedCommentedDirective,
@@ -104,6 +110,39 @@ export const currentFileFlow = (context) => {
     return skipTrue;
   }
 
+  // GETTING THE DIRECTIVE (or lack thereof) OF THE CURRENT FILE
+  const currentFileDirective = getDirectiveFromCurrentModule(context);
+
+  if (
+    commentedDirectives_reactDirectives[verifiedCommentedDirective] !==
+    currentFileDirective
+  ) {
+    const expectedReactDirective =
+      commentedDirectives_reactDirectives[verifiedCommentedDirective];
+    const expectedReactDirectiveAsText = reactDirectives_asTexts.get(
+      expectedReactDirective
+    );
+
+    if (!expectedReactDirectiveAsText) {
+      console.warn(
+        `Somehow, expectedReactDirectiveAsText for ${expectedReactDirective} is undefined.`
+      );
+      return { skip: undefined, verifiedCommentedDirective }; // at this time, behave as if the new implementation didn't exist yet
+    }
+
+    context.report({
+      loc: highlightFirstLineOfCode(context),
+      messageId: commentedDirectiveReactDirectiveFailedMessageId, // ADD NEW
+      data: {
+        // verifiedCommentedDirective
+        verifiedCommentedDirective,
+        // expectedReactDirectiveAsText
+        expectedReactDirectiveAsText,
+      },
+    });
+    return skipTrue;
+  }
+
   return { skip: undefined, verifiedCommentedDirective };
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-use-agnostic",
-  "version": "1.6.9",
+  "version": "1.6.10",
   "description": "Highlights problematic server-client imports in projects made with the Fullstack React Architecture.",
   "keywords": [
     "eslint",