redlint 6.5.1 → 6.6.0

package/CONTRIBUTING.md

@@ -0,0 +1,181 @@
+# Contributing to RedLint
+
+## Quick Reference
+
+- **No unused variables or imports** — clean them up before committing.
+- **No `t.pass('everything ok')` in tests** — every assertion must test something meaningful.
+- **100% coverage required** — checked via [`.nycrc.json`](.nycrc.json). Run `redrun coverage` to verify.
+- **Tests use [supertape](https://github.com/coderaiser/supertape)** with `stub()` for mocking.
+- **TDD is encouraged** — write a failing test first, then implement, then `redrun fix:lint test`.
+- **Run `redrun fix:lint test` before every commit** — linter must be clean, all tests green.
+
+## Table of Contents
+
+- [Do / Don't](#do--dont)
+- [Workflow](#workflow)
+- [Architecture](#architecture)
+- [Import Map](#import-map)
+- [File Tree](#file-tree)
+- [Overrides Pattern](#overrides-pattern)
+- [Tools we use](#tools-we-use)
+
+## Do / Don't
+
+| Do                                               | Don't                                            |
+|--------------------------------------------------|------------------------------------------------|
+| Write one assertion per test                     | Use `t.pass('ok')` or `t.comment()`            |
+| Use descriptive test names                       | Use vague names like `'test1'`                 |
+| Inject deps via `overrides`                      | Import real modules inline (hard to stub)      |
+| Use early returns to flatten logic               | Nest deeply with `if/else` chains              |
+| Keep modules small, one concern per file         | Put multiple operations in one file            |
+| Use `async` for tests that `await`               | Forget `await` in async test functions         |
+| Run `redrun fix:lint coverage` before commit         | Commit red linter output                   |
+
+## Workflow
+
+Install [`redrun`](https://github.com/coderaiser/redrun) (faster than `npm run`):
+
+```sh
+bun i redrun -g
+```
+
+Every commit should pass:
+
+```sh
+redrun fix:lint     # runs putout . --fix
+redrun test         # runs all .spec.js tests
+redrun coverage     # run coverage (aim for 100%):
+```
+
+## Architecture
+
+### Worker / Direct Pattern
+
+Each filesystem operation (`scan`, `fix`, `pack`, `extract`, `convert`) has two paths:
+
+| Mode       | Function                                | Thread         |
+|------------|-----------------------------------------|----------------|
+| **Normal** | `master*()` + `slave.js`              | Web Worker       |
+| **Debug**  | Direct import (e.g. `lint()`, `convert()`) | Main thread |
+
+The debug mode is an escape hatch when workers misbehave.
+
+### Module layout
+
+- `lib/<operation>/master.js` — spawns a worker, returns a promise.
+- `lib/<operation>/slave.js` — runs inside the worker, calls the real logic.
+- `lib/<operation>/<operation>.js` — the actual implementation (imported by slave and tests).
+- `lib/<operation>/<operation>.spec.js` — tests.
+- `lib/<operation>/fixture/` — test fixtures.
+
+### Testing
+
+- Framework: [supertape](https://github.com/coderaiser/supertape)
+- Mocking: `stub()` from supertape (no external mocking library)
+- Fixtures: plain `.js` files in `fixture/` directories
+
+## Import Map
+
+The project uses Node.js [`imports`](package.json) for internal aliases:
+
+```json
+{
+    "imports": {
+        "#edit": "./lib/edit/edit.js",
+        "#test": "./test/create-test.js"
+    }
+}
+```
+
+Used in `bin/redlint.js`:
+
+```js
+import {edit, editHelp} from '#edit';
+```
+
+## File Tree
+
+```
+redlint/
+├── bin/redlint.js          # CLI entry point
+├── lib/
+│   ├── convert/            # convert json↔js, rc→flat
+│   ├── edit/               # interactive filesystem editing
+│   ├── extract/            # unpack filesystem.red
+│   ├── help/               # --help output
+│   ├── lint/               # scan & fix filesystem
+│   ├── pack/               # pack to filesystem.red
+│   ├── rename/             # rename js↔jsx
+│   ├── test/               # test runner & plugins
+│   ├── view/               # view file contents
+│   ├── cli/                # version
+│   ├── choose.js           # interactive menu
+│   ├── debug.js            # debug menu
+│   ├── dialog.js           # ask for filename
+│   ├── menu.js             # menu constants & predicates
+│   ├── redlint.js          # buildTree
+│   ├── run.js              # worker runner
+│   ├── simple.js           # simple filesystem format
+│   ├── slave.js            # worker helper
+│   └── spinner.js          # CLI spinner
+├── test/                   # integration tests
+├── .nycrc.json             # coverage config (100% required)
+├── .madrun.js              # task runner scripts
+├── .putout.json            # putout rules
+└── package.json
+```
+
+## Overrides Pattern
+
+When a module calls external dependencies, accept them as optional overrides so tests can inject stubs:
+
+```js
+// run-convert.js
+export const runConvert = async (arg, filesystem, overrides = {}) => {
+    const {
+        askFilename: getFilename = askFilename,
+        convert = masterConvert,
+        isRCToFlat = isConvertRCToFlat,
+    } = overrides;
+    
+    let filename = '.eslintrc.json';
+    
+    if (!isRCToFlat(arg))
+        filename = await getFilename();
+    
+    if (filename)
+        return await convert(filename, arg, filesystem);
+};
+```
+
+Test injects stubs to verify behavior without side effects:
+
+```js
+// run-convert.spec.js
+test('redlint: run-convert: json to js: called with filename', async (t) => {
+    const convert = stub();
+    const askFilename = stub().returns('package.json');
+    const filesystem = '[]';
+    
+    await runConvert(CONVERT_JSON_TO_JS, filesystem, {
+        askFilename,
+        convert,
+    });
+    
+    t.calledWith(convert, ['package.json', CONVERT_JSON_TO_JS, filesystem]);
+    t.end();
+});
+```
+
+This pattern keeps functions pure, easy to unit test, and doesn't require any mocking framework.
+
+## Tools we use
+
+| Tool                                                  | Purpose                                              | Link                 |
+|-------------------------------------------------------|------------------------------------------------------|----------------------|
+| [redrun](https://github.com/coderaiser/redrun)        | Fast task runner (replaces `npm run`)                | `bun i redrun -g`    |
+| [madrun](https://github.com/coderaiser/madrun)        | Define tasks in `.madrun.js`                         | `redrun`             |
+| [putout](https://github.com/coderaiser/putout)        | JavaScript code transformer & linter                 | `redrun fix:lint`    |
+| [supertape](https://github.com/coderaiser/supertape)  | Test framework with built-in `stub()`                | `redrun test`        |
+| [superc8](https://github.com/coderaiser/superc8)      | Enhanced c8 wrapper                                  | `redrun coverage`    |
+| [nodemon](https://github.com/remy/nodemon)            | Watch mode for tests                                 | `redrun watch:test`  |

package/ChangeLog

@@ -1,3 +1,14 @@
+2026.06.06, v6.6.0
+
+feature:
+- 842206b convert: move out
+- eb2885e convert yaml to json: add
+
+2026.06.05, v6.5.2
+
+feature:
+- fa9d692 redlint: convert: move out
+
 2026.06.05, v6.5.1
 
 fix:
@@ -490,7 +501,7 @@ feature:
 2024.02.02, v3.11.1
 
 fix:
-- 779bf51 isConvertToJson
+- 779bf51 isConvertJSToJson
 
 feature:
 - 6edb2f7 redlint: @putout/plugin-filesystem v4.0.0

package/README.md

@@ -45,6 +45,7 @@ To scan your files use `redlint scan`:
 - ✅ [`package-json/find-file`](https://github.com/coderaiser/putout/tree/master/packages/plugin-package-json#find-file);
 - ✅ [`package-json/remove-exports-with-missing-files`](https://github.com/coderaiser/putout/tree/master/packages/plugin-package-json#remove-exports-with-missing-files);
 - ✅ [`esm/resolve-imported-file`](https://github.com/coderaiser/putout/tree/master/packages/plugin-esm#resolve-imported-file);
+- ✅ [`esm/resolve-imported-file-with-extension`](https://github.com/coderaiser/putout/tree/master/packages/plugin-esm#resolve-imported-file-with-extension);
 - ✅ [`esm/shorten-imported-file`](https://github.com/coderaiser/putout/tree/master/packages/plugin-esm#shorten-imported-file);
 - ✅ [`esm/apply-name-to-imported-file`](https://github.com/coderaiser/putout/tree/master/packages/plugin-esm#apply-name-to-imported-file);
 - ✅ [`esm/apply-namespace-to-imported-file`](https://github.com/coderaiser/putout/tree/master/packages/plugin-esm#apply-name-to-imported-file);

package/bin/redlint.js

@@ -27,7 +27,7 @@ import {version} from '../lib/cli/version.js';
 import {chooseConvert} from '../lib/convert/index.js';
 import {chooseRename} from '../lib/rename/index.js';
 import {convert} from '../lib/convert/convert.js';
-import {masterConvert} from '../lib/convert/master.js';
+import {runConvert} from '../lib/convert/run-convert.js';
 import {askFilename} from '../lib/dialog.js';
 import {masterRename} from '../lib/rename/master.js';
 import {view} from '../lib/view/view.js';
@@ -55,7 +55,6 @@ import {
     isBack,
     isExit,
     isBundleDebug,
-    isConvertRCToFlat,
     isEdit,
     isView,
     VIEW,
@@ -184,14 +183,14 @@ async function uiLoop(arg) {
     }
     
     if (isConvertChosen(arg)) {
-        let filename = '.eslintrc.json';
-        
-        if (!isConvertRCToFlat(arg))
-            filename = await askFilename();
-        
-        if (filename)
-            await masterConvert(filename, arg, filesystem);
-        
+        await runConvert(arg, filesystem);
+        return;
+    }
+    
+    if (isConvertChosenDebug(arg)) {
+        await runConvert(arg, filesystem, {
+            convert,
+        });
         return;
     }
     
@@ -205,15 +204,6 @@ async function uiLoop(arg) {
         return;
     }
     
-    if (isConvertChosenDebug(arg)) {
-        const filename = await askFilename();
-        
-        if (filename)
-            await convert(filename, arg, filesystem);
-        
-        return;
-    }
-    
     if (isScan(arg)) {
         const places = await masterLint(filesystem, {
             fix: false,

package/lib/convert/converters/convert-js-to-json.js

@@ -0,0 +1,14 @@
+import * as pluginFilesystem from '@putout/plugin-filesystem';
+
+const [, pluginConvertJsToJson] = pluginFilesystem.rules['convert-js-to-json'];
+
+export const convertJSToJson = (filename) => ({
+    rules: {
+        'filesystem/convert-js-to-json': ['on', {
+            filename,
+        }],
+    },
+    plugins: [
+        ['filesystem/convert-js-to-json', pluginConvertJsToJson],
+    ],
+});

package/lib/convert/converters/convert-json-to-js.js

@@ -0,0 +1,14 @@
+import * as pluginFilesystem from '@putout/plugin-filesystem';
+
+const [, pluginConvertJsonToJs] = pluginFilesystem.rules['convert-json-to-js'];
+
+export const convertJsonToJs = (filename) => ({
+    rules: {
+        'filesystem/convert-json-to-js': ['on', {
+            filename,
+        }],
+    },
+    plugins: [
+        ['filesystem/convert-json-to-js', pluginConvertJsonToJs],
+    ],
+});

package/lib/convert/converters/convert-rc-to-flat.js

@@ -0,0 +1,12 @@
+import * as pluginESLint from '@putout/plugin-eslint';
+
+const [, pluginConvertRCToFlat] = pluginESLint.rules['convert-rc-to-flat'];
+
+export const convertRCToFlat = () => ({
+    rules: {
+        'eslint/convert-rc-to-flat': 'on',
+    },
+    plugins: [
+        ['eslint/convert-rc-to-flat', pluginConvertRCToFlat],
+    ],
+});

package/lib/convert/converters/convert-yaml-to-json.js

@@ -0,0 +1,14 @@
+import * as pluginFilesystem from '@putout/plugin-filesystem';
+
+const [, pluginConvertYamlToJson] = pluginFilesystem.rules['convert-yaml-to-json'];
+
+export const convertYamlToJson = (filename) => ({
+    rules: {
+        'filesystem/convert-yaml-to-json': ['on', {
+            filename,
+        }],
+    },
+    plugins: [
+        ['filesystem/convert-yaml-to-json', pluginConvertYamlToJson],
+    ],
+});

package/lib/convert/create-options.js

@@ -1,49 +1,32 @@
-import * as pluginFilesystem from '@putout/plugin-filesystem';
-import * as pluginESLint from '@putout/plugin-eslint';
 import {
-    isConvertToJs,
-    isConvertToJson,
-    isConvertRCToFlat,
+    CONVERT_JS_TO_JSON,
+    CONVERT_JSON_TO_JS,
+    CONVERT_JSON_TO_JS_DEBUG,
+    CONVERT_JS_TO_JSON_DEBUG,
+    CONVERT_YAML_TO_JSON,
+    CONVERT_YAML_TO_JSON_DEBUG,
+    CONVERT_RC_TO_FLAT,
 } from '../menu.js';
+import {convertYamlToJson} from './converters/convert-yaml-to-json.js';
+import {convertJsonToJs} from './converters/convert-json-to-js.js';
+import {convertJSToJson} from './converters/convert-js-to-json.js';
+import {convertRCToFlat} from './converters/convert-rc-to-flat.js';
 
-const [, pluginConvertJsonToJs] = pluginFilesystem.rules['convert-json-to-js'];
-const [, pluginConvertJsToJson] = pluginFilesystem.rules['convert-js-to-json'];
-const [, convertRCToFlat] = pluginESLint.rules['convert-rc-to-flat'];
+const CONVERTERS = {
+    [CONVERT_JSON_TO_JS]: convertJsonToJs,
+    [CONVERT_JSON_TO_JS_DEBUG]: convertJsonToJs,
+    [CONVERT_JS_TO_JSON]: convertJSToJson,
+    [CONVERT_JS_TO_JSON_DEBUG]: convertJSToJson,
+    [CONVERT_YAML_TO_JSON]: convertYamlToJson,
+    [CONVERT_YAML_TO_JSON_DEBUG]: convertYamlToJson,
+    [CONVERT_RC_TO_FLAT]: convertRCToFlat,
+};
 
 export const createOptions = (filename, type) => {
-    if (isConvertToJs(type))
-        return {
-            rules: {
-                'filesystem/convert-json-to-js': ['on', {
-                    filename,
-                }],
-            },
-            plugins: [
-                ['filesystem/convert-json-to-js', pluginConvertJsonToJs],
-            ],
-        };
-    
-    if (isConvertToJson(type))
-        return {
-            rules: {
-                'filesystem/convert-js-to-json': ['on', {
-                    filename,
-                }],
-            },
-            plugins: [
-                ['filesystem/convert-js-to-json', pluginConvertJsToJson],
-            ],
-        };
+    const converter = CONVERTERS[type];
     
-    if (isConvertRCToFlat(type))
-        return {
-            rules: {
-                'eslint/convert-rc-to-flat': 'on',
-            },
-            plugins: [
-                ['eslint/convert-rc-to-flat', convertRCToFlat],
-            ],
-        };
+    if (!converter)
+        return {};
     
-    return {};
+    return converter(filename);
 };

package/lib/convert/index.js

@@ -5,6 +5,7 @@ import {
     BACK,
     EXIT,
     CONVERT_RC_TO_FLAT,
+    CONVERT_YAML_TO_JSON,
 } from '../menu.js';
 
 export * from './convert.js';
@@ -12,6 +13,7 @@ export const chooseConvert = async () => {
     const command = await chooseDialog('Convert:', [
         CONVERT_JS_TO_JSON,
         CONVERT_JSON_TO_JS,
+        CONVERT_YAML_TO_JSON,
         CONVERT_RC_TO_FLAT,
         BACK,
         EXIT,

package/lib/convert/run-convert.js

@@ -0,0 +1,19 @@
+import {askFilename} from '../dialog.js';
+import {isConvertRCToFlat} from '../menu.js';
+import {masterConvert} from './master.js';
+
+export const runConvert = async (arg, filesystem, overrides = {}) => {
+    const {
+        askFilename: getFilename = askFilename,
+        convert = masterConvert,
+        isRCToFlat = isConvertRCToFlat,
+    } = overrides;
+    
+    let filename = '.eslintrc.json';
+    
+    if (!isRCToFlat(arg))
+        filename = await getFilename();
+    
+    if (filename)
+        return await convert(filename, arg, filesystem);
+};

package/lib/debug.js

@@ -6,6 +6,7 @@ import {
     FIX_DEBUG,
     CONVERT_JS_TO_JSON_DEBUG,
     CONVERT_JSON_TO_JS_DEBUG,
+    CONVERT_YAML_TO_JSON_DEBUG,
     RENAME_JS_TO_JSX_DEBUG,
     BACK,
     EXIT,
@@ -21,6 +22,7 @@ export const debug = async () => {
         PACK_DEBUG,
         CONVERT_JS_TO_JSON_DEBUG,
         CONVERT_JSON_TO_JS_DEBUG,
+        CONVERT_YAML_TO_JSON_DEBUG,
         RENAME_JS_TO_JSX_DEBUG,
         RENAME_JS_TO_JSX_DEBUG,
         BUNDLE_DEBUG,

package/lib/menu.js

@@ -26,6 +26,9 @@ export const CONVERT_JS_TO_JSON_DEBUG = '🦏 convert js to json: debug';
 export const CONVERT_JSON_TO_JS = '🦏 convert json to js';
 export const CONVERT_JSON_TO_JS_DEBUG = '🦏 convert json to js: debug';
 
+export const CONVERT_YAML_TO_JSON = '🦏 convert yaml to json';
+export const CONVERT_YAML_TO_JSON_DEBUG = '🦏 convert yaml to json: debug';
+
 export const RENAME_JS_TO_JSX = '🦏 rename js to jsx';
 export const RENAME_JS_TO_JSX_DEBUG = '🦏 rename js to jsx: debug';
 
@@ -56,15 +59,18 @@ export const isConvertChosen = (a) => {
     return [
         CONVERT_JSON_TO_JS,
         CONVERT_JS_TO_JSON,
+        CONVERT_YAML_TO_JSON,
         CONVERT_RC_TO_FLAT,
     ].includes(a);
 };
 export const isRenameToJsxChosen = (a) => a === RENAME_JS_TO_JSX;
 export const isRenameToJsChosen = (a) => a === RENAME_JSX_TO_JS;
-export const isConvertChosenDebug = (a) => a === CONVERT_JS_TO_JSON_DEBUG || a === CONVERT_JSON_TO_JS_DEBUG;
+export const isConvertChosenDebug = (a) => [
+    CONVERT_JS_TO_JSON_DEBUG,
+    CONVERT_JSON_TO_JS_DEBUG,
+    CONVERT_YAML_TO_JSON_DEBUG,
+].includes(a);
 
-export const isConvertToJson = (a) => a === CONVERT_JS_TO_JSON || a === CONVERT_JS_TO_JSON_DEBUG;
-export const isConvertToJs = (a) => a === CONVERT_JSON_TO_JS || a === CONVERT_JSON_TO_JS_DEBUG;
 export const isConvertRCToFlat = (a) => a === CONVERT_RC_TO_FLAT;
 
 export const isRenameToJs = (a) => a === RENAME_JS_TO_JSX || a === RENAME_JS_TO_JSX_DEBUG;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "redlint",
-  "version": "6.5.1",
+  "version": "6.6.0",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "Lint Filesystem with 🐊Putout",