antfu/eslint-plugin-command

Comment-as-command for one-off codemod with ESLint.

eslint-plugin-command's Issues

`/// keep-sorted` for `import/export` statements

Clear and concise description of the problem

Hiya! Basically, just a way to keep import/export statements sorted (including the elements imported within them). Some of this might fit into an entirely separate ESLint rule, but I feel like especially the first example below would fit into this plugin.

Suggested solution

basic export

/// keep-sorted
export * from './bbbbbbbbbb.ts';
export * from './aaaaaaaaaaa.ts';

// ->

/// keep-sorted
export * from './aaaaaaaaaaa.ts';
export * from './bbbbbbbbbb.ts';

basic import

/// keep-sorted
import { d, c, b, a } from './bbbbbbbbbb.ts';
import * as bbb from './bbbbbbbbbbc.ts';
import { dd, cc, aa, bb } from './aaaaaaaaaaa.ts';
import * as aaa from './aaaaaaaaaaac.ts';

// everything below isnt sorted i guess? not sure of details
import ....

// ->

/// keep-sorted
import * as aaa from './aaaaaaaaaaac.ts'; // wildcard `as` imports would have more priority based on their aliased name
import * as bbb from './bbbbbbbbbbc.ts';
import { aa, bb, cc, dd } from './aaaaaaaaaaa.ts';
import { a, b, c, d } from './bbbbbbbbbb.ts';

// everything below isnt sorted i guess? not sure of details
import ....

Alternative

No response

Additional context

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guide.
• Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

`/// keep-sorted` for `switch` statements

Clear and concise description of the problem

Hey there, I'm back with another suggestion: /// keep-sorted for switch statements. The idea comes from this file

Suggested solution

Basic

/// keep-sorted
switch (x) {
  case "b":
    doB();
    break
  case "a":
    doA();
    break;
}

// ->

/// keep-sorted
switch (x) {
  case "a":
    doA();
    break
  case "b":
    doB();
    break;
}

Complex/weird

/// keep-sorted
switch (x) {
  case "fallback":
    falling();
  case "c":
  case "b":
    common();
    break;
  case "d":
  case "a":
    aOrD();
    break;
}

// ->

/// keep-sorted
switch (x) {
  case "a": // `a` comes before `fallback`
  case "d":
    aOrD();
    break;
  case "fallback":
    falling();
  case "b":
  case "c":
    common();
    break;
}

Alternative

No response

Additional context

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guide.
• Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Hoist-regex

Clear and concise description of the problem

function foo(file) {
  /// hoist-regex customName
  return file.test(/some-regex/)
}

to

const customName = /some-regex/
function foo(file) {
  return file.test(customName)
}

Suggested solution

Alternative

Later we could support a bunch of other hoist-x commands

Additional context

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guide.
• Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

`/// to-promise-all`

Clear and concise description of the problem

Hey there, I started using eslint-plugin-command and it's a pure joy so far, thanks @antfu!

I'd like a new command to refactor sequential promises to a single Promise.all call to run things in parallel.

Source:

/// to-promise-all
const a = await getA();
const b = await getB();
const c = await getC();

Suggested solution

Result:

const [a, b, c] = await Promise.all([getA(), getB(), getC()]);

Alternative

There might already be an eslint rule that finds and fixes potential performance issues, but I haven't found it yet

Additional context

Maybe the reverse operation /// to-seq-await would make sense if one wants to introduce dependencies between the promises and avoid refactoring by hand:

/// to-seq-await
const [a, b, c] = await Promise.all([getA(b), getB(c), getC()]);

const c = await getC();
const b = await getB(c);
const a = await getA(a);

Validations

• Follow our Code of Conduct
• Read the Contributing Guide.
• Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Interconversion between `template string` and `string`

Clear and concise description of the problem

I would like to use a command to interconvert between template string and string.

Suggested solution

Interconversion would be easy, if there are multiple strings in one line, we can specify the number or all to covert.

Alternative

No response

Additional context

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guide.
• Check that there isn't already an issue that request the same feature to avoid creating a duplicate.