eslint-plugin-smarthr 6.9.0 → 6.9.1

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+## [6.9.1](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v6.9.0...eslint-plugin-smarthr-v6.9.1) (2026-04-07)
+
+
+### Bug Fixes
+
+* **best-practice-for-text-component:** 変換不可能なshr-クラスのみの場合に不要なスペースが挿入される問題を修正 ([#1199](https://github.com/kufu/tamatebako/issues/1199)) ([ad942fc](https://github.com/kufu/tamatebako/commit/ad942fcfa2484013fe82fd1c9d9b94c097a0e27a))
+
 ## [6.9.0](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v6.8.0...eslint-plugin-smarthr-v6.9.0) (2026-04-05)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-smarthr",
-  "version": "6.9.0",
+  "version": "6.9.1",
   "author": "SmartHR",
   "license": "MIT",
   "description": "A sharable ESLint plugin for SmartHR",
@@ -37,5 +37,5 @@
     "eslintplugin",
     "smarthr"
   ],
-  "gitHead": "41bc1d6941b48121d18250aaecb83812f87955e1"
+  "gitHead": "5f958652e81f7a69a347489108ff88cb1b3e56dd"
 }

package/rules/best-practice-for-text-component/index.js

@@ -177,16 +177,37 @@ module.exports = {
       // パターン2-1: classNameのみ（asなし）、Text属性なし、shr-クラスあり
       [SELECTOR_CONVERTIBLE_SHR_TO_PROPS]: (classNameAttrNode) => {
         const { nonConvertible, propSuggestions, convertible } = categorizeClassNames(classNameAttrNode)
+        const openingElement = classNameAttrNode.parent
+        const jsxElement = openingElement.parent
+
+        // 変換可能なクラスが0個の場合、spanに変換（パターン1-3と同じ動作）
+        if (!propSuggestions) {
+          const classNameText = `className="${classNameAttrNode.value.value}"`
+          context.report({
+            node: openingElement,
+            message: `Textコンポーネントの機能を使用していないため、ネイティブHTML要素（<span>）に置き換えてください。
+ - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-text-component
+ - 推奨: <span ${classNameText}>
+ - Textコンポーネントの機能（weight、size、color等）を使用しない場合は、直接HTML要素を使用することでシンプルになります`,
+            fix(fixer) {
+              return [
+                fixer.replaceText(openingElement.name, 'span'),
+                fixer.replaceText(jsxElement.closingElement.name, 'span')
+              ]
+            },
+          })
+          return
+        }
 
+        // 変換可能なクラスがある場合、属性に変換
         context.report({
-          node: classNameAttrNode.parent,
+          node: openingElement,
           message: `classNameで指定されたshr-プレフィックスのクラスは、Textコンポーネントの属性に置き換えてください。
  - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-text-component
  - 推奨: <Text ${propSuggestions}${nonConvertible ? ` className="${nonConvertible}"` : ''}>
  - 変換可能なクラス: ${convertible}
  - shr-プレフィックスのクラスをTextの属性に置き換えることで、型安全性が向上し、意図がより明確になります`,
           fix(fixer) {
-            const openingElement = classNameAttrNode.parent
             const sourceCode = context.sourceCode || context.getSourceCode()
             const fixes = []
 
@@ -235,8 +256,39 @@ module.exports = {
       [SELECTOR_CONVERTIBLE_SHR_TO_PROPS_WITH_AS]: (classNameAttrNode) => {
         const { nonConvertible, propSuggestions, convertible } = categorizeClassNames(classNameAttrNode)
         const openingElement = classNameAttrNode.parent
+        const jsxElement = openingElement.parent
         const asValue = getAttributeLiteralValue(openingElement, 'as')
 
+        // 変換可能なクラスが0個の場合、as属性で指定されたタグに変換（パターン1-4と同じ動作）
+        if (!propSuggestions) {
+          const classNameValue = getAttributeLiteralValue(openingElement, 'className')
+          const classNameText = `className="${classNameValue}"`
+          context.report({
+            node: openingElement,
+            message: `Textコンポーネントの機能を使用していないため、ネイティブHTML要素に置き換えてください。
+ - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-text-component
+ - <${asValue}>要素にclassNameを移動してください
+ - Textコンポーネントの機能（weight、size、color等）を使用しない場合は、直接HTML要素を使用することでシンプルになります`,
+            fix(fixer) {
+              const sourceCode = context.sourceCode || context.getSourceCode()
+              const asAttrNode = getAttributeNode(openingElement, 'as')
+
+              // 属性とその前のスペースを含めて削除
+              const tokenBefore = sourceCode.getTokenBefore(asAttrNode)
+              const rangeStart = tokenBefore.range[1]
+              const rangeEnd = asAttrNode.range[1]
+
+              return [
+                fixer.removeRange([rangeStart, rangeEnd]),
+                fixer.replaceText(openingElement.name, asValue),
+                fixer.replaceText(jsxElement.closingElement.name, asValue)
+              ]
+            },
+          })
+          return
+        }
+
+        // 変換可能なクラスがある場合、属性に変換
         context.report({
           node: openingElement,
           message: `classNameで指定されたshr-プレフィックスのクラスは、Textコンポーネントの属性に置き換えてください。

package/test/best-practice-for-text-component.js

@@ -105,6 +105,11 @@ ruleTester.run('best-practice-for-text-component', rule, {
     { code: `<Text as="p" className="shr-text-sm custom-class">text</Text>`, output: `<Text as="p" size="S" className="custom-class">text</Text>`, errors: [{ message: errorConvertibleShr('size="S" className="custom-class"', 'shr-text-sm', 'p') }] },
     { code: `<Text className="shr-text-lg shr-font-bold custom-one custom-two">text</Text>`, output: `<Text size="L" weight="bold" className="custom-one custom-two">text</Text>`, errors: [{ message: errorConvertibleShr('size="L" weight="bold" className="custom-one custom-two"', 'shr-text-lg, shr-font-bold') }] },
 
+    // パターン2-4: shr-プレフィックスがあるが変換不可能なクラスのみ（spanに変換）
+    { code: `<Text className="shr-w-[10rem]">text</Text>`, output: `<span className="shr-w-[10rem]">text</span>`, errors: [{ message: errorUnnecessaryClassName('shr-w-[10rem]') }] },
+    { code: `<Text className="shr-inline-block shr-mr-0.5">text</Text>`, output: `<span className="shr-inline-block shr-mr-0.5">text</span>`, errors: [{ message: errorUnnecessaryClassName('shr-inline-block shr-mr-0.5') }] },
+    { code: `<Text as="p" className="shr-bg-background shr-block">text</Text>`, output: `<p className="shr-bg-background shr-block">text</p>`, errors: [{ message: errorUnnecessaryAsClassName('p') }] },
+
     // パターン3: 属性とclassNameの矛盾
     { code: `<Text size="M" className="shr-text-sm">text</Text>`, errors: [{ message: errorConflictingProps('shr-text-sm') }] },
     { code: `<Text weight="bold" className="shr-font-normal">text</Text>`, errors: [{ message: errorConflictingProps('shr-font-normal') }] },