Repository metrics

Stars: (48,455 stars)
PR merge metrics: (平均マージ 6d 17h) (30d で 9 merged PRs)

説明

Bug Report

2022-02-16T20:50:15.000Z

# Bug Report ### 🔎 Search Terms - quick fix - code action - jsdoc ### 🕗 Version & Regression Information 4.7.0-dev.20220216 This is new behavior and not a regression ### 💻 Repo 1. For the TypeScript: 2. ```ts /** * @param c * @param b * * bla bla bla */ function add(a, b) { } ``` 2. Trigger the `delete unused @param tag` quick fix on `c` inside the JSDoc. ### 🙁 Actual behavior This results in: ``` /** * @param b bla bla bla */ function add(a, b) { } ``` ### 🙂 Expected behavior I'd sped that only the `@param c` is removed and the rest of the jsdoc remains unchanged: ```js /** * @param b * * bla bla bla */ function add(a, b) { } ``` ### Related issues - #47922

🔎 Search Terms

quick fix
code action
jsdoc

🕗 Version & Regression Information

4.7.0-dev.20220216

This is new behavior and not a regression

💻 Repo

For the TypeScript:

/**
 * @param c
 * @param b
 * 
 * bla bla bla
 */
function add(a, b) { }

Trigger the delete unused @param tag quick fix on c inside the JSDoc.

🙁 Actual behavior

This results in:

/**
 * @param b bla bla bla
 */
function add(a, b) { }

🙂 Expected behavior

I'd sped that only the @param c is removed and the rest of the jsdoc remains unchanged:

/**
 * @param b
 * 
 * bla bla bla
 */
function add(a, b) { }

Related issues

#47922

コントリビューターガイド

調査方針: TypeScript のコードベースにある「未使用の @param タグを削除」クイック修正を調査します。この機能はおそらく src/services/codefixes/deleteUnusedJSDocParam.ts にあります。現在のクイック修正はタグのみを削除しますが、続くテキスト（例：「bla bla bla」）も次のパラメータと結合してしまいます。削除後のコメントテキストがどのように再構築されるかを調べ、指定されたタグ行のみを削除して元の書式を保持する変更を実装してください。ソースマップまたは AST 操作を使用して改行を維持することを検討してください。
技術スタック: typescript
領域: developer experience
Issue 種別: バグ
難度: 2
推定時間: 1-3時間
活動状況: アクティブ
明確さ: 明確
前提条件: なし
初心者向け度: 60