Repository metrics

Stars: (48,455 stars)
PR merge metrics: (Avg merge 6d 17h) (9 merged PRs in 30d)

Description

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

Contributor guide

Research direction: Inspect the source code for the 'delete unused @param tag' code action in TypeScript's services layer. The likely root cause is that the quick fix is reformatting the entire JSDoc comment instead of only removing the specific line.
Tech stack: typescript
Domain: developer experience
Issue type: Bug
Difficulty: 2
Estimated time: 1-3 hours
Activity status: Active
Clarity: Clear
Prerequisites: None
Newbie friendliness: 60