@crutchcrew/oxlint-plugin-import-access 1.0.1 → 1.1.0

package/README.md

@@ -38,7 +38,7 @@ This plugin adds a directory-level layer between a file and the whole project: a
 | **Checks unsaved edits in the editor** | ✅           | ✅                                                                                 | ❌ skipped until saved                                                                   |
 | **Same results as the original rule**  | ✅ verified¹ | –                                                                                  | ✅                                                                                       |
 
-¹ With `defaultImportability: "public"`, which makes exports without an access tag `@public` like in the original. This plugin's default is `"package"`.
+¹ With `defaultImportability: "public"`, which makes exports without an access tag `@public` like in the original. This plugin's default is `"package"`. This plugin also checks the members used through namespace imports (`import * as foo`), which the original doesn't check.
 
 ## Install
 

package/dist/core/checkImportability.d.ts

@@ -53,6 +53,11 @@ export declare class ImportabilityPolicy {
     #private;
     readonly options: PackageOptions;
     constructor(options: PackageOptions);
+    /**
+     * Whether some export of `specifier` could be reported. Cheap way to skip namespace imports
+     * of external modules and of modules whose exports are all public.
+     */
+    mayReport(checker: ImportabilityChecker, importerFileName: string, specifier: string): boolean;
     /**
      * Check importing `name` (the export name, `default` for default imports) via `specifier`.
      */

package/dist/index.js

@@ -941,10 +941,17 @@ class ImportabilityPolicy {
     const packageName = lookupPackageName(importerFileName);
     return packageName !== null && (specifier === packageName || specifier.startsWith(`${packageName}/`));
   }
-  check(checker, importerFileName, specifier, name) {
+  #targetModule(checker, importerFileName, specifier) {
     const { defaultImportability, projectDirectory } = this.options;
     const assumeAllPublic = defaultImportability === "public";
-    const module = checker.targetModule(importerFileName, specifier, projectDirectory, assumeAllPublic);
+    return checker.targetModule(importerFileName, specifier, projectDirectory, assumeAllPublic);
+  }
+  mayReport(checker, importerFileName, specifier) {
+    return this.#targetModule(checker, importerFileName, specifier)?.names !== undefined;
+  }
+  check(checker, importerFileName, specifier, name) {
+    const { defaultImportability } = this.options;
+    const module = this.#targetModule(checker, importerFileName, specifier);
     const found = module && checker.findExport(module, name);
     if (found === undefined || !found.entries[0].checked) {
       return;
@@ -1003,6 +1010,21 @@ function realFileName(fileName) {
 function nameOf(node) {
   return node.type === "Literal" ? node.value : node.name;
 }
+function accessedMember(identifier) {
+  const parent = identifier.parent;
+  if (parent.type === "MemberExpression" && parent.object === identifier) {
+    const { property } = parent;
+    if (property.type === "Identifier" && !parent.computed) {
+      return { node: property, name: property.name };
+    }
+    if (property.type === "Literal" && typeof property.value === "string") {
+      return { node: property, name: property.value };
+    }
+  } else if (parent.type === "TSQualifiedName") {
+    return { node: parent.right, name: parent.right.name };
+  }
+  return;
+}
 var rule = {
   meta: {
     type: "problem",
@@ -1060,6 +1082,20 @@ var rule = {
       ImportDefaultSpecifier(node) {
         check(node, node.parent.source.value, "default", false);
       },
+      ImportNamespaceSpecifier(node) {
+        const specifier = node.parent.source.value;
+        if (!policy.mayReport(checker, importer, specifier)) {
+          return;
+        }
+        for (const variable of context.sourceCode.getDeclaredVariables(node)) {
+          for (const { identifier } of variable.references) {
+            const member = accessedMember(identifier);
+            if (member !== undefined) {
+              check(member.node, specifier, member.name, false);
+            }
+          }
+        }
+      },
       ExportSpecifier(node) {
         const { source } = node.parent;
         if (source !== null) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@crutchcrew/oxlint-plugin-import-access",
-	"version": "1.0.1",
+	"version": "1.1.0",
 	"description": "Oxlint JS plugin for import access via JSDoc @public, @package and @private, without a type checker",
 	"keywords": [
 		"import-access",