@prover-coder-ai/component-tagger 1.0.25 → 1.0.27

package/babel.cjs

@@ -22,6 +22,7 @@
 
 const path = require("node:path")
 
+const babelPluginName = "component-path-babel-tagger"
 const componentPathAttributeName = "data-path"
 const jsxFilePattern = /\.(tsx|jsx)(\?.*)?$/u
 
@@ -37,7 +38,7 @@ const attrExists = (node, attrName, t) =>
 
 module.exports = function componentTaggerBabelPlugin({ types: t }) {
   return {
-    name: "component-path-babel-tagger",
+    name: babelPluginName,
     visitor: {
       JSXOpeningElement(nodePath, state) {
         const { node } = nodePath

package/dist/core/component-path.d.ts

@@ -1,3 +1,19 @@
+export declare const babelPluginName = "component-path-babel-tagger";
+/**
+ * Normalizes a module ID by stripping query parameters.
+ *
+ * Vite and other bundlers may append query parameters to module IDs
+ * (e.g., "src/App.tsx?import" or "src/App.tsx?v=123"). This function
+ * returns the clean file path without query string.
+ *
+ * @param id - Module ID (may include query parameters).
+ * @returns Clean path without query string.
+ *
+ * @pure true
+ * @invariant ∀ id: normalizeModuleId(id) does not contain '?'
+ * @complexity O(n) time / O(1) space where n = |id|
+ */
+export declare const normalizeModuleId: (id: string) => string;
 export declare const componentPathAttributeName = "data-path";
 /**
  * Checks whether the Vite id represents a JSX or TSX module.

package/dist/core/component-path.js

@@ -1,4 +1,43 @@
 const jsxFilePattern = /\.(tsx|jsx)(\?.*)?$/u;
+// CHANGE: define canonical Babel plugin name for component path tagging.
+// WHY: eliminate magic string duplication across plugin implementations.
+// QUOTE(ТЗ): "Вынести строки имён плагинов в константы (чтобы не дублировать \"component-path-babel-tagger\")."
+// REF: issue-19
+// SOURCE: n/a
+// FORMAT THEOREM: forall p in PluginName: p = "component-path-babel-tagger"
+// PURITY: CORE
+// EFFECT: n/a
+// INVARIANT: plugin name remains stable across all implementations
+// COMPLEXITY: O(1)/O(1)
+export const babelPluginName = "component-path-babel-tagger";
+/**
+ * Normalizes a module ID by stripping query parameters.
+ *
+ * Vite and other bundlers may append query parameters to module IDs
+ * (e.g., "src/App.tsx?import" or "src/App.tsx?v=123"). This function
+ * returns the clean file path without query string.
+ *
+ * @param id - Module ID (may include query parameters).
+ * @returns Clean path without query string.
+ *
+ * @pure true
+ * @invariant ∀ id: normalizeModuleId(id) does not contain '?'
+ * @complexity O(n) time / O(1) space where n = |id|
+ */
+// CHANGE: centralize query stripping as a pure function in core.
+// WHY: unify module ID normalization in one place as requested in issue #18.
+// QUOTE(ТЗ): "Вынести stripQuery() (или normalizeModuleId()) в core, использовать в Vite и (при желании) в isJsxFile."
+// REF: REQ-18 (issue #18)
+// SOURCE: n/a
+// FORMAT THEOREM: ∀ id: normalizeModuleId(id) = id.split('?')[0]
+// PURITY: CORE
+// EFFECT: n/a
+// INVARIANT: result contains no query string
+// COMPLEXITY: O(n)/O(1)
+export const normalizeModuleId = (id) => {
+    const queryIndex = id.indexOf("?");
+    return queryIndex === -1 ? id : id.slice(0, queryIndex);
+};
 // CHANGE: rename attribute from "path" to "data-path" for HTML5 compliance.
 // WHY: data-* attributes are standard HTML5 custom data attributes, improving compatibility.
 // QUOTE(issue-14): "Rename attribute path → data-path (breaking change)"
@@ -22,7 +61,7 @@ export const componentPathAttributeName = "data-path";
  */
 // CHANGE: centralize JSX file detection as a pure predicate.
 // WHY: keep file filtering in the functional core for testability.
-// QUOTE(TZ): "\u0421\u0430\u043c \u043a\u043e\u043c\u043f\u043e\u043d\u0435\u043d\u0442 \u0434\u043e\u043b\u0436\u0435\u043d \u0431\u044b\u0442\u044c \u0432 \u0442\u0435\u043a\u0443\u0449\u0435\u043c app \u043d\u043e \u0432\u043e\u0442 \u0447\u0442\u043e \u0431\u044b \u0435\u0433\u043e \u043f\u0440\u043e\u0442\u0435\u0441\u0442\u0438\u0440\u043e\u0432\u0430\u0442\u044c \u043d\u0430\u0434\u043e \u0441\u043e\u0437\u0434\u0430\u0442\u044c \u0435\u0449\u0451 \u043e\u0434\u0438\u043d \u043f\u0440\u043e\u0435\u043a\u0442 \u043a\u043e\u0442\u043e\u0440\u044b\u0439 \u043d\u0430\u0448 \u0442\u0435\u043a\u0443\u0449\u0438\u0439 \u0430\u043f\u043f \u0431\u0443\u0434\u0435\u0442 \u043f\u043e\u0434\u043a\u043b\u044e\u0447\u0430\u0442\u044c"
+// QUOTE(TZ): "Сам компонент должен быть в текущем app но вот что бы его протестировать надо создать ещё один проект который наш текущий апп будет подключать"
 // REF: user-2026-01-14-frontend-consumer
 // SOURCE: n/a
 // FORMAT THEOREM: forall id in ModuleId: isJsxFile(id) -> matches(id, jsxFilePattern)
@@ -45,7 +84,7 @@ export const isJsxFile = (id) => jsxFilePattern.test(id);
  */
 // CHANGE: provide a pure formatter for component location payloads.
 // WHY: reuse a single, deterministic encoding for UI metadata.
-// QUOTE(TZ): "\u0421\u0430\u043c \u043a\u043e\u043c\u043f\u043e\u043d\u0435\u043d\u0442 \u0434\u043e\u043b\u0436\u0435\u043d \u0431\u044b\u0442\u044c \u0432 \u0442\u0435\u043a\u0443\u0449\u0435\u043c app \u043d\u043e \u0432\u043e\u0442 \u0447\u0442\u043e \u0431\u044b \u0435\u0433\u043e \u043f\u0440\u043e\u0442\u0435\u0441\u0442\u0438\u0440\u043e\u0432\u0430\u0442\u044c \u043d\u0430\u0434\u043e \u0441\u043e\u0437\u0434\u0430\u0442\u044c \u0435\u0449\u0451 \u043e\u0434\u0438\u043d \u043f\u0440\u043e\u0435\u043a\u0442 \u043a\u043e\u0442\u043e\u0440\u044b\u0439 \u043d\u0430\u0448 \u0442\u0435\u043a\u0443\u0449\u0438\u0439 \u0430\u043f\u043f \u0431\u0443\u0434\u0435\u0442 \u043f\u043e\u0434\u043a\u043b\u044e\u0447\u0430\u0442\u044c"
+// QUOTE(TZ): "Сам компонент должен быть в текущем app но вот что бы его протестировать надо создать ещё один проект который наш текущий апп будет подключать"
 // REF: user-2026-01-14-frontend-consumer
 // SOURCE: n/a
 // FORMAT THEOREM: forall p,l,c: formatComponentPathValue(p,l,c) = concat(p, ":", l, ":", c)

package/dist/index.d.ts

@@ -1,4 +1,4 @@
-export { componentPathAttributeName, formatComponentPathValue, isJsxFile } from "./core/component-path.js";
+export { componentPathAttributeName, formatComponentPathValue, isJsxFile, normalizeModuleId } from "./core/component-path.js";
 export { attrExists, createJsxTaggerVisitor, createPathAttribute, type JsxTaggerContext, processJsxElement } from "./core/jsx-tagger.js";
 export { componentTaggerBabelPlugin, type ComponentTaggerBabelPluginOptions } from "./shell/babel-plugin.js";
 export { componentTagger, type ComponentTaggerOptions } from "./shell/component-tagger.js";

package/dist/index.js

@@ -8,7 +8,7 @@
 // EFFECT: n/a
 // INVARIANT: exports remain stable for consumers
 // COMPLEXITY: O(1)/O(1)
-export { componentPathAttributeName, formatComponentPathValue, isJsxFile } from "./core/component-path.js";
+export { componentPathAttributeName, formatComponentPathValue, isJsxFile, normalizeModuleId } from "./core/component-path.js";
 export { attrExists, createJsxTaggerVisitor, createPathAttribute, processJsxElement } from "./core/jsx-tagger.js";
 export { componentTaggerBabelPlugin } from "./shell/babel-plugin.js";
 export { componentTagger } from "./shell/component-tagger.js";

package/dist/shell/babel-plugin.js

@@ -1,5 +1,5 @@
 import { types as t } from "@babel/core";
-import { componentPathAttributeName, isJsxFile } from "../core/component-path.js";
+import { babelPluginName, componentPathAttributeName, isJsxFile } from "../core/component-path.js";
 import { createJsxTaggerVisitor } from "../core/jsx-tagger.js";
 import { computeRelativePath } from "../core/path-service.js";
 /**
@@ -83,7 +83,7 @@ const getContextFromState = (state) => {
 // INVARIANT: each JSX opening element has at most one path attribute
 // COMPLEXITY: O(n)/O(1)
 export const componentTaggerBabelPlugin = () => ({
-    name: "component-path-babel-tagger",
+    name: babelPluginName,
     visitor: createJsxTaggerVisitor(getContextFromState, t)
 });
 /**

package/dist/shell/component-tagger.js

@@ -1,6 +1,6 @@
 import { transformAsync, types as t } from "@babel/core";
 import { Effect, pipe } from "effect";
-import { componentPathAttributeName, isJsxFile } from "../core/component-path.js";
+import { babelPluginName, componentPathAttributeName, isJsxFile, normalizeModuleId } from "../core/component-path.js";
 import { createJsxTaggerVisitor } from "../core/jsx-tagger.js";
 import { NodePathLayer, relativeFromRoot } from "../core/path-service.js";
 class ComponentTaggerError extends Error {
@@ -11,10 +11,6 @@ class ComponentTaggerError extends Error {
         this.cause = cause;
     }
 }
-const stripQuery = (id) => {
-    const queryIndex = id.indexOf("?");
-    return queryIndex === -1 ? id : id.slice(0, queryIndex);
-};
 const toViteResult = (result) => {
     if (result === null || result.code === null || result.code === undefined) {
         return null;
@@ -25,10 +21,20 @@ const toViteResult = (result) => {
         map: result.map ?? null
     };
 };
+// CHANGE: use unified JSX tagger visitor from core module.
+// WHY: share business logic between Vite and Babel plugins as requested.
+// QUOTE(TZ): "А ты можешь сделать что бы бизнес логика оставалось одной? Ну типо переиспользуй код с vite версии на babel"
+// REF: issue-12-comment (unified interface request)
+// SOURCE: n/a
+// FORMAT THEOREM: forall f in JSXOpeningElement: rendered(f) -> annotated(f)
+// PURITY: SHELL
+// EFFECT: Babel AST transformation
+// INVARIANT: each JSX opening element has at most one path attribute
+// COMPLEXITY: O(n)/O(1), n = number of JSX elements
 const makeBabelTagger = (relativeFilename, attributeName) => {
     const context = { relativeFilename, attributeName };
     return {
-        name: "component-path-babel-tagger",
+        name: babelPluginName,
         visitor: createJsxTaggerVisitor(() => context, t)
     };
 };
@@ -55,7 +61,7 @@ const makeBabelTagger = (relativeFilename, attributeName) => {
 // INVARIANT: errors are surfaced as ComponentTaggerError only
 // COMPLEXITY: O(n)/O(1)
 const runTransform = (code, id, rootDir, attributeName) => {
-    const cleanId = stripQuery(id);
+    const cleanId = normalizeModuleId(id);
     return pipe(relativeFromRoot(rootDir, cleanId), Effect.flatMap((relative) => Effect.tryPromise({
         try: () => transformAsync(code, {
             filename: cleanId,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prover-coder-ai/component-tagger",
-  "version": "1.0.25",
+  "version": "1.0.27",
   "description": "Component tagger Vite plugin and Babel plugin for JSX metadata",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",