pi-lens 2.2.9 → 3.0.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +198 -0
- package/README.md +709 -519
- package/clients/__tests__/file-time.test.js +216 -0
- package/clients/__tests__/file-time.test.ts +276 -0
- package/clients/__tests__/format-service.test.js +245 -0
- package/clients/__tests__/format-service.test.ts +339 -0
- package/clients/__tests__/formatters.test.js +271 -0
- package/clients/__tests__/formatters.test.ts +401 -0
- package/clients/amain-types.js +164 -0
- package/clients/amain-types.ts +165 -0
- package/clients/architect-client.js +56 -12
- package/clients/architect-client.ts +81 -16
- package/clients/ast-grep-client.js +2 -2
- package/clients/ast-grep-client.ts +14 -39
- package/clients/ast-grep-parser.ts +1 -1
- package/clients/ast-grep-rule-manager.js +8 -0
- package/clients/ast-grep-rule-manager.ts +10 -1
- package/clients/ast-grep-types.js +9 -0
- package/clients/ast-grep-types.ts +106 -0
- package/clients/auto-loop.js +10 -0
- package/clients/auto-loop.ts +14 -1
- package/clients/biome-client.js +81 -19
- package/clients/biome-client.ts +103 -22
- package/clients/bus/bus.js +191 -0
- package/clients/bus/bus.ts +251 -0
- package/clients/bus/events.js +214 -0
- package/clients/bus/events.ts +279 -0
- package/clients/bus/index.js +8 -0
- package/clients/bus/index.ts +9 -0
- package/clients/bus/integration.js +158 -0
- package/clients/bus/integration.ts +214 -0
- package/clients/complexity-client.js +13 -7
- package/clients/complexity-client.ts +13 -7
- package/clients/config-validator.js +465 -0
- package/clients/config-validator.ts +558 -0
- package/clients/dependency-checker.js +4 -10
- package/clients/dependency-checker.ts +4 -10
- package/clients/dispatch/__tests__/autofix-integration.test.js +245 -0
- package/clients/dispatch/__tests__/autofix-integration.test.ts +300 -0
- package/clients/dispatch/__tests__/runner-registration.test.js +236 -0
- package/clients/dispatch/__tests__/runner-registration.test.ts +282 -0
- package/clients/dispatch/bus-dispatcher.js +177 -0
- package/clients/dispatch/bus-dispatcher.ts +251 -0
- package/clients/dispatch/dispatcher.edge.test.js +82 -0
- package/clients/dispatch/dispatcher.edge.test.ts +100 -0
- package/clients/dispatch/dispatcher.format.test.js +46 -0
- package/clients/dispatch/dispatcher.format.test.ts +58 -0
- package/clients/dispatch/dispatcher.inline.test.js +74 -0
- package/clients/dispatch/dispatcher.inline.test.ts +93 -0
- package/clients/dispatch/dispatcher.js +19 -53
- package/clients/dispatch/dispatcher.ts +20 -67
- package/clients/dispatch/plan.js +9 -4
- package/clients/dispatch/plan.ts +9 -4
- package/clients/dispatch/runners/architect.js +21 -7
- package/clients/dispatch/runners/architect.test.js +138 -0
- package/clients/dispatch/runners/architect.test.ts +162 -0
- package/clients/dispatch/runners/architect.ts +22 -7
- package/clients/dispatch/runners/ast-grep-napi.js +462 -0
- package/clients/dispatch/runners/ast-grep-napi.test.js +111 -0
- package/clients/dispatch/runners/ast-grep-napi.test.ts +133 -0
- package/clients/dispatch/runners/ast-grep-napi.ts +506 -0
- package/clients/dispatch/runners/ast-grep.js +62 -19
- package/clients/dispatch/runners/ast-grep.ts +70 -18
- package/clients/dispatch/runners/biome.js +29 -53
- package/clients/dispatch/runners/biome.ts +29 -63
- package/clients/dispatch/runners/config-validation.js +67 -0
- package/clients/dispatch/runners/config-validation.ts +82 -0
- package/clients/dispatch/runners/go-vet.js +4 -28
- package/clients/dispatch/runners/go-vet.ts +4 -32
- package/clients/dispatch/runners/index.js +30 -10
- package/clients/dispatch/runners/index.ts +30 -10
- package/clients/dispatch/runners/oxlint.js +141 -0
- package/clients/dispatch/runners/oxlint.test.js +230 -0
- package/clients/dispatch/runners/oxlint.test.ts +303 -0
- package/clients/dispatch/runners/oxlint.ts +175 -0
- package/clients/dispatch/runners/pyright.js +40 -70
- package/clients/dispatch/runners/pyright.test.js +16 -2
- package/clients/dispatch/runners/pyright.test.ts +14 -2
- package/clients/dispatch/runners/pyright.ts +48 -91
- package/clients/dispatch/runners/python-slop.js +97 -0
- package/clients/dispatch/runners/python-slop.test.js +203 -0
- package/clients/dispatch/runners/python-slop.test.ts +298 -0
- package/clients/dispatch/runners/python-slop.ts +124 -0
- package/clients/dispatch/runners/ruff.js +18 -71
- package/clients/dispatch/runners/ruff.ts +19 -79
- package/clients/dispatch/runners/rust-clippy.js +28 -32
- package/clients/dispatch/runners/rust-clippy.ts +29 -31
- package/clients/dispatch/runners/scan_codebase.test.js +89 -0
- package/clients/dispatch/runners/scan_codebase.test.ts +105 -0
- package/clients/dispatch/runners/shellcheck.js +147 -0
- package/clients/dispatch/runners/shellcheck.test.js +98 -0
- package/clients/dispatch/runners/shellcheck.test.ts +129 -0
- package/clients/dispatch/runners/shellcheck.ts +188 -0
- package/clients/dispatch/runners/similarity.js +230 -0
- package/clients/dispatch/runners/similarity.ts +339 -0
- package/clients/dispatch/runners/spellcheck.js +106 -0
- package/clients/dispatch/runners/spellcheck.test.js +158 -0
- package/clients/dispatch/runners/spellcheck.test.ts +214 -0
- package/clients/dispatch/runners/spellcheck.ts +136 -0
- package/clients/dispatch/runners/tree-sitter.js +107 -0
- package/clients/dispatch/runners/tree-sitter.ts +135 -0
- package/clients/dispatch/runners/ts-lsp.js +104 -33
- package/clients/dispatch/runners/ts-lsp.ts +120 -38
- package/clients/dispatch/runners/ts-slop.js +113 -0
- package/clients/dispatch/runners/ts-slop.test.js +180 -0
- package/clients/dispatch/runners/ts-slop.test.ts +230 -0
- package/clients/dispatch/runners/ts-slop.ts +142 -0
- package/clients/dispatch/runners/utils/diagnostic-parsers.js +134 -0
- package/clients/dispatch/runners/utils/diagnostic-parsers.ts +186 -0
- package/clients/dispatch/runners/utils/runner-helpers.js +115 -0
- package/clients/dispatch/runners/utils/runner-helpers.ts +167 -0
- package/clients/dispatch/runners/utils.js +2 -4
- package/clients/dispatch/runners/utils.ts +2 -4
- package/clients/dispatch/types.ts +1 -1
- package/clients/dispatch/utils/format-utils.js +49 -0
- package/clients/dispatch/utils/format-utils.ts +60 -0
- package/clients/dogfood.test.js +201 -0
- package/clients/dogfood.test.ts +269 -0
- package/clients/file-time.js +152 -0
- package/clients/file-time.ts +208 -0
- package/clients/file-utils.js +40 -0
- package/clients/file-utils.ts +44 -0
- package/clients/fix-scanners.js +10 -20
- package/clients/fix-scanners.ts +10 -22
- package/clients/format-service.js +172 -0
- package/clients/format-service.ts +254 -0
- package/clients/formatters.js +435 -0
- package/clients/formatters.ts +508 -0
- package/clients/go-client.js +5 -14
- package/clients/go-client.ts +5 -13
- package/clients/installer/index.js +356 -0
- package/clients/installer/index.ts +426 -0
- package/clients/jscpd-client.js +11 -9
- package/clients/jscpd-client.ts +12 -8
- package/clients/knip-client.js +3 -7
- package/clients/knip-client.ts +3 -6
- package/clients/lsp/__tests__/client.test.js +325 -0
- package/clients/lsp/__tests__/client.test.ts +434 -0
- package/clients/lsp/__tests__/config.test.js +166 -0
- package/clients/lsp/__tests__/config.test.ts +209 -0
- package/clients/lsp/__tests__/error-recovery.test.js +213 -0
- package/clients/lsp/__tests__/error-recovery.test.ts +279 -0
- package/clients/lsp/__tests__/integration.test.js +127 -0
- package/clients/lsp/__tests__/integration.test.ts +160 -0
- package/clients/lsp/__tests__/launch.test.js +260 -0
- package/clients/lsp/__tests__/launch.test.ts +329 -0
- package/clients/lsp/__tests__/server.test.js +259 -0
- package/clients/lsp/__tests__/server.test.ts +332 -0
- package/clients/lsp/__tests__/service.test.js +417 -0
- package/clients/lsp/__tests__/service.test.ts +499 -0
- package/clients/lsp/client.js +235 -0
- package/clients/lsp/client.ts +328 -0
- package/clients/lsp/config.js +115 -0
- package/clients/lsp/config.ts +149 -0
- package/clients/lsp/index.js +222 -0
- package/clients/lsp/index.ts +280 -0
- package/clients/lsp/installer/index.js +391 -0
- package/clients/lsp/interactive-install.js +210 -0
- package/clients/lsp/interactive-install.ts +251 -0
- package/clients/lsp/language.js +170 -0
- package/clients/lsp/language.ts +216 -0
- package/clients/lsp/launch.js +174 -0
- package/clients/lsp/launch.ts +240 -0
- package/clients/lsp/lsp/launch.js +116 -0
- package/clients/lsp/lsp/server.js +532 -0
- package/clients/lsp/lsp-index.js +10 -0
- package/clients/lsp/lsp-index.ts +11 -0
- package/clients/lsp/path-utils.js +48 -0
- package/clients/lsp/path-utils.ts +52 -0
- package/clients/lsp/server.js +615 -0
- package/clients/lsp/server.ts +800 -0
- package/clients/lsp/test-py-spawn/requirements.txt +1 -0
- package/clients/lsp/test-py-spawn/test.py +3 -0
- package/clients/lsp/test-py-svc/requirements.txt +1 -0
- package/clients/lsp/test-py-svc/test.py +3 -0
- package/clients/lsp/test-python-project/requirements.txt +1 -0
- package/clients/lsp/test-python-project/test.py +5 -0
- package/clients/metrics-history.js +2 -2
- package/clients/metrics-history.ts +2 -2
- package/clients/production-readiness.js +522 -0
- package/clients/production-readiness.ts +556 -0
- package/clients/project-index.js +255 -0
- package/clients/project-index.ts +383 -0
- package/clients/project-metadata.js +531 -0
- package/clients/project-metadata.ts +624 -0
- package/clients/ruff-client.js +56 -16
- package/clients/ruff-client.ts +72 -15
- package/clients/runner-tracker.js +152 -0
- package/clients/runner-tracker.ts +213 -0
- package/clients/rust-client.js +4 -11
- package/clients/rust-client.ts +5 -11
- package/clients/safe-spawn.js +96 -0
- package/clients/safe-spawn.ts +128 -0
- package/clients/scan-architectural-debt.js +3 -6
- package/clients/scan-architectural-debt.ts +3 -6
- package/clients/scan-utils.js +5 -20
- package/clients/scan-utils.ts +5 -29
- package/clients/secrets-scanner.js +3 -17
- package/clients/secrets-scanner.ts +4 -20
- package/clients/services/__tests__/effect-integration.test.js +86 -0
- package/clients/services/__tests__/effect-integration.test.ts +111 -0
- package/clients/services/effect-integration.js +194 -0
- package/clients/services/effect-integration.ts +268 -0
- package/clients/services/index.js +7 -0
- package/clients/services/index.ts +8 -0
- package/clients/services/runner-service.js +105 -0
- package/clients/services/runner-service.ts +179 -0
- package/clients/sg-runner.js +87 -13
- package/clients/sg-runner.ts +97 -13
- package/clients/state-matrix.js +160 -0
- package/clients/state-matrix.ts +202 -0
- package/clients/subprocess-client.js +10 -9
- package/clients/subprocess-client.ts +10 -8
- package/clients/test-runner-client.js +3 -7
- package/clients/test-runner-client.ts +3 -6
- package/clients/tool-availability.js +4 -10
- package/clients/tool-availability.ts +4 -9
- package/clients/tree-sitter-client.js +564 -0
- package/clients/tree-sitter-client.ts +797 -0
- package/clients/tree-sitter-query-loader.js +355 -0
- package/clients/tree-sitter-query-loader.ts +425 -0
- package/clients/type-coverage-client.js +3 -7
- package/clients/type-coverage-client.ts +3 -6
- package/clients/typescript-client.codefix.test.js +157 -0
- package/clients/typescript-client.codefix.test.ts +186 -0
- package/clients/typescript-client.js +43 -0
- package/clients/typescript-client.ts +98 -0
- package/commands/booboo.js +799 -219
- package/commands/booboo.ts +1004 -225
- package/commands/clients/ast-grep-client.js +250 -0
- package/commands/clients/ast-grep-parser.js +86 -0
- package/commands/clients/ast-grep-rule-manager.js +91 -0
- package/commands/clients/ast-grep-types.js +9 -0
- package/commands/clients/biome-client.js +380 -0
- package/commands/clients/complexity-client.js +667 -0
- package/commands/clients/file-kinds.js +177 -0
- package/commands/clients/file-utils.js +40 -0
- package/commands/clients/jscpd-client.js +169 -0
- package/commands/clients/knip-client.js +211 -0
- package/commands/clients/ruff-client.js +297 -0
- package/commands/clients/safe-spawn.js +88 -0
- package/commands/clients/scan-utils.js +83 -0
- package/commands/clients/sg-runner.js +190 -0
- package/commands/clients/types.js +11 -0
- package/commands/clients/typescript-client.js +505 -0
- package/commands/fix-from-booboo.js +398 -0
- package/commands/fix-from-booboo.ts +485 -0
- package/commands/fix-simplified.js +618 -0
- package/commands/fix-simplified.ts +768 -0
- package/commands/rate.js +10 -14
- package/commands/rate.ts +9 -16
- package/default-architect.yaml +59 -15
- package/index.ts +342 -429
- package/package.json +16 -3
- package/rules/ast-grep-rules/rules/empty-catch.yml +38 -13
- package/rules/ast-grep-rules/rules/no-array-constructor.yml +1 -0
- package/rules/ast-grep-rules/rules/no-debugger.yml +2 -0
- package/rules/python-slop-rules/.sgconfig.yml +4 -0
- package/rules/python-slop-rules/rules/slop-rules.yml +647 -0
- package/rules/tree-sitter-queries/python/bare-except.yml +54 -0
- package/rules/tree-sitter-queries/python/eval-exec.yml +50 -0
- package/rules/tree-sitter-queries/python/is-vs-equals.yml +60 -0
- package/rules/tree-sitter-queries/python/mutable-default-arg.yml +57 -0
- package/rules/tree-sitter-queries/python/unreachable-except.yml +60 -0
- package/rules/tree-sitter-queries/python/wildcard-import.yml +46 -0
- package/rules/tree-sitter-queries/tsx/dangerously-set-inner-html.yml +63 -0
- package/rules/tree-sitter-queries/typescript/await-in-loop.yml +56 -0
- package/rules/tree-sitter-queries/typescript/console-statement.yml +47 -0
- package/rules/tree-sitter-queries/typescript/debugger.yml +47 -0
- package/rules/tree-sitter-queries/typescript/deep-nesting.yml +117 -0
- package/rules/tree-sitter-queries/typescript/deep-promise-chain.yml +73 -0
- package/rules/tree-sitter-queries/typescript/empty-catch.yml +64 -0
- package/rules/tree-sitter-queries/typescript/eval.yml +48 -0
- package/rules/tree-sitter-queries/typescript/hardcoded-secrets.yml +78 -0
- package/rules/tree-sitter-queries/typescript/long-parameter-list.yml +62 -0
- package/rules/tree-sitter-queries/typescript/mixed-async-styles.yml +49 -0
- package/rules/tree-sitter-queries/typescript/nested-ternary.yml +45 -0
- package/rules/ts-slop-rules/.sgconfig.yml +4 -0
- package/rules/ts-slop-rules/rules/in-correct-optional-input-type.yml +10 -0
- package/rules/ts-slop-rules/rules/jwt-no-verify.yml +13 -0
- package/rules/ts-slop-rules/rules/no-architecture-violation.yml +10 -0
- package/rules/ts-slop-rules/rules/no-case-declarations.yml +10 -0
- package/rules/ts-slop-rules/rules/no-dangerously-set-inner-html.yml +10 -0
- package/rules/ts-slop-rules/rules/no-debugger.yml +10 -0
- package/rules/ts-slop-rules/rules/no-dupe-args.yml +10 -0
- package/rules/ts-slop-rules/rules/no-dupe-class-members.yml +10 -0
- package/rules/ts-slop-rules/rules/no-dupe-keys.yml +10 -0
- package/rules/ts-slop-rules/rules/no-eval.yml +13 -0
- package/rules/ts-slop-rules/rules/no-hardcoded-secrets.yml +12 -0
- package/rules/ts-slop-rules/rules/no-implied-eval.yml +12 -0
- package/rules/ts-slop-rules/rules/no-inner-html.yml +13 -0
- package/rules/ts-slop-rules/rules/no-javascript-url.yml +10 -0
- package/rules/ts-slop-rules/rules/no-mutable-default.yml +10 -0
- package/rules/ts-slop-rules/rules/no-nested-links.yml +12 -0
- package/rules/ts-slop-rules/rules/no-new-symbol.yml +10 -0
- package/rules/ts-slop-rules/rules/no-new-wrappers.yml +13 -0
- package/rules/ts-slop-rules/rules/no-open-redirect.yml +16 -0
- package/rules/ts-slop-rules/rules/slop-rules.yml +455 -0
- package/rules/ts-slop-rules/rules/weak-rsa-key.yml +12 -0
- package/skills/ast-grep/SKILL.md +182 -0
- package/clients/dispatch/runners/secrets.js +0 -109
- package/commands/fix.js +0 -244
- package/commands/fix.ts +0 -373
- package/rules/ast-grep-rules/rules/no-lonely-if.yml +0 -13
|
@@ -0,0 +1,380 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Biome Client for pi-lens
|
|
3
|
+
*
|
|
4
|
+
* All-in-one: formatting + linting for JS/TS/JSX/TSX/CSS/JSON
|
|
5
|
+
* Replaces Prettier with 15-50x faster Rust-based tool.
|
|
6
|
+
*
|
|
7
|
+
* Requires: npm install @biomejs/biome (or npx @biomejs/biome)
|
|
8
|
+
* Docs: https://biomejs.dev/
|
|
9
|
+
*/
|
|
10
|
+
import * as fs from "node:fs";
|
|
11
|
+
import * as path from "node:path";
|
|
12
|
+
import { isFileKind } from "./file-kinds.js";
|
|
13
|
+
import { safeSpawn } from "./safe-spawn.js";
|
|
14
|
+
// --- Client ---
|
|
15
|
+
export class BiomeClient {
|
|
16
|
+
biomeAvailable = null;
|
|
17
|
+
log;
|
|
18
|
+
constructor(verbose = false) {
|
|
19
|
+
this.log = verbose
|
|
20
|
+
? (msg) => console.error(`[biome] ${msg}`)
|
|
21
|
+
: () => { };
|
|
22
|
+
}
|
|
23
|
+
/**
|
|
24
|
+
* Check if biome CLI is available
|
|
25
|
+
*/
|
|
26
|
+
isAvailable() {
|
|
27
|
+
if (this.biomeAvailable !== null)
|
|
28
|
+
return this.biomeAvailable;
|
|
29
|
+
// Try npx biome first (works without global install)
|
|
30
|
+
const result = safeSpawn("npx", ["@biomejs/biome", "--version"], {
|
|
31
|
+
timeout: 10000,
|
|
32
|
+
});
|
|
33
|
+
this.biomeAvailable = !result.error && result.status === 0;
|
|
34
|
+
if (this.biomeAvailable) {
|
|
35
|
+
const version = result.stdout?.trim() || "unknown";
|
|
36
|
+
this.log(`Biome found: ${version}`);
|
|
37
|
+
}
|
|
38
|
+
else {
|
|
39
|
+
this.log("Biome not available — install with: npm install -D @biomejs/biome");
|
|
40
|
+
}
|
|
41
|
+
return this.biomeAvailable;
|
|
42
|
+
}
|
|
43
|
+
/**
|
|
44
|
+
* Check if a file is supported by Biome
|
|
45
|
+
*/
|
|
46
|
+
isSupportedFile(filePath) {
|
|
47
|
+
return isFileKind(filePath, ["jsts", "json", "css"]);
|
|
48
|
+
}
|
|
49
|
+
// --- Internal helpers ---
|
|
50
|
+
/**
|
|
51
|
+
* Validate path and availability — returns path or null on failure
|
|
52
|
+
*/
|
|
53
|
+
withValidatedPath(filePath) {
|
|
54
|
+
if (!this.isAvailable())
|
|
55
|
+
return null;
|
|
56
|
+
const absolutePath = path.resolve(filePath);
|
|
57
|
+
if (!fs.existsSync(absolutePath))
|
|
58
|
+
return null;
|
|
59
|
+
return absolutePath;
|
|
60
|
+
}
|
|
61
|
+
/**
|
|
62
|
+
* Run biome check (format + lint) without fixing — returns diagnostics
|
|
63
|
+
*/
|
|
64
|
+
checkFile(filePath) {
|
|
65
|
+
const absolutePath = this.withValidatedPath(filePath);
|
|
66
|
+
if (!absolutePath)
|
|
67
|
+
return [];
|
|
68
|
+
try {
|
|
69
|
+
const result = safeSpawn("npx", [
|
|
70
|
+
"@biomejs/biome",
|
|
71
|
+
"check",
|
|
72
|
+
"--reporter=json",
|
|
73
|
+
"--max-diagnostics=50",
|
|
74
|
+
absolutePath,
|
|
75
|
+
], {
|
|
76
|
+
timeout: 15000,
|
|
77
|
+
});
|
|
78
|
+
// Biome exits 0 on success, 1 on issues found
|
|
79
|
+
const output = result.stdout || "";
|
|
80
|
+
if (!output.trim())
|
|
81
|
+
return [];
|
|
82
|
+
return this.parseDiagnostics(output, absolutePath);
|
|
83
|
+
}
|
|
84
|
+
catch (err) {
|
|
85
|
+
this.log(`Check error: ${err instanceof Error ? err.message : String(err)}`);
|
|
86
|
+
return [];
|
|
87
|
+
}
|
|
88
|
+
}
|
|
89
|
+
/**
|
|
90
|
+
* Format a file (writes to disk)
|
|
91
|
+
*/
|
|
92
|
+
formatFile(filePath) {
|
|
93
|
+
const absolutePath = this.withValidatedPath(filePath);
|
|
94
|
+
if (!absolutePath)
|
|
95
|
+
return {
|
|
96
|
+
success: false,
|
|
97
|
+
changed: false,
|
|
98
|
+
error: this.isAvailable() ? "File not found" : "Biome not available",
|
|
99
|
+
};
|
|
100
|
+
const content = fs.readFileSync(absolutePath, "utf-8");
|
|
101
|
+
try {
|
|
102
|
+
const result = safeSpawn("npx", ["@biomejs/biome", "format", "--write", absolutePath], {
|
|
103
|
+
timeout: 15000,
|
|
104
|
+
});
|
|
105
|
+
if (result.error) {
|
|
106
|
+
return { success: false, changed: false, error: result.error.message };
|
|
107
|
+
}
|
|
108
|
+
// Re-read to see if changed
|
|
109
|
+
const formatted = fs.readFileSync(absolutePath, "utf-8");
|
|
110
|
+
const changed = content !== formatted;
|
|
111
|
+
if (changed) {
|
|
112
|
+
this.log(`Formatted ${path.basename(filePath)}`);
|
|
113
|
+
}
|
|
114
|
+
return { success: true, changed };
|
|
115
|
+
}
|
|
116
|
+
catch (err) {
|
|
117
|
+
return {
|
|
118
|
+
success: false,
|
|
119
|
+
changed: false,
|
|
120
|
+
error: err instanceof Error ? err.message : String(err),
|
|
121
|
+
};
|
|
122
|
+
}
|
|
123
|
+
}
|
|
124
|
+
/**
|
|
125
|
+
* Fix both formatting and linting issues (writes to disk)
|
|
126
|
+
*/
|
|
127
|
+
fixFile(filePath) {
|
|
128
|
+
const absolutePath = this.withValidatedPath(filePath);
|
|
129
|
+
if (!absolutePath)
|
|
130
|
+
return {
|
|
131
|
+
success: false,
|
|
132
|
+
changed: false,
|
|
133
|
+
fixed: 0,
|
|
134
|
+
error: this.isAvailable() ? "File not found" : "Biome not available",
|
|
135
|
+
};
|
|
136
|
+
const content = fs.readFileSync(absolutePath, "utf-8");
|
|
137
|
+
try {
|
|
138
|
+
// First, count issues before fixing
|
|
139
|
+
const beforeDiags = this.checkFile(filePath);
|
|
140
|
+
const fixableCount = beforeDiags.filter((d) => d.fixable).length;
|
|
141
|
+
// Apply fixes
|
|
142
|
+
const result = safeSpawn("npx", [
|
|
143
|
+
"@biomejs/biome",
|
|
144
|
+
"check",
|
|
145
|
+
"--write",
|
|
146
|
+
"--unsafe", // Apply unsafe fixes too
|
|
147
|
+
absolutePath,
|
|
148
|
+
], {
|
|
149
|
+
timeout: 15000,
|
|
150
|
+
});
|
|
151
|
+
if (result.error) {
|
|
152
|
+
return {
|
|
153
|
+
success: false,
|
|
154
|
+
changed: false,
|
|
155
|
+
fixed: 0,
|
|
156
|
+
error: result.error.message,
|
|
157
|
+
};
|
|
158
|
+
}
|
|
159
|
+
const fixed = fs.readFileSync(absolutePath, "utf-8");
|
|
160
|
+
const changed = content !== fixed;
|
|
161
|
+
if (changed) {
|
|
162
|
+
this.log(`Fixed ${fixableCount} issue(s) in ${path.basename(filePath)}`);
|
|
163
|
+
}
|
|
164
|
+
return { success: true, changed, fixed: fixableCount };
|
|
165
|
+
}
|
|
166
|
+
catch (err) {
|
|
167
|
+
return {
|
|
168
|
+
success: false,
|
|
169
|
+
changed: false,
|
|
170
|
+
fixed: 0,
|
|
171
|
+
error: err instanceof Error ? err.message : String(err),
|
|
172
|
+
};
|
|
173
|
+
}
|
|
174
|
+
}
|
|
175
|
+
/**
|
|
176
|
+
* Fix multiple files at once (much faster than file-by-file)
|
|
177
|
+
*/
|
|
178
|
+
fixFiles(filePaths) {
|
|
179
|
+
if (!this.isAvailable()) {
|
|
180
|
+
return {
|
|
181
|
+
success: false,
|
|
182
|
+
fixed: 0,
|
|
183
|
+
changed: 0,
|
|
184
|
+
error: "Biome not available",
|
|
185
|
+
};
|
|
186
|
+
}
|
|
187
|
+
// Filter to existing files
|
|
188
|
+
const validFiles = filePaths
|
|
189
|
+
.map(f => path.resolve(f))
|
|
190
|
+
.filter(f => fs.existsSync(f));
|
|
191
|
+
if (validFiles.length === 0) {
|
|
192
|
+
return { success: true, fixed: 0, changed: 0 };
|
|
193
|
+
}
|
|
194
|
+
try {
|
|
195
|
+
// Count fixable issues before fixing
|
|
196
|
+
let totalFixable = 0;
|
|
197
|
+
for (const file of validFiles) {
|
|
198
|
+
const diags = this.checkFile(file);
|
|
199
|
+
totalFixable += diags.filter(d => d.fixable).length;
|
|
200
|
+
}
|
|
201
|
+
// Run biome once on all files - much faster than npx per file
|
|
202
|
+
const result = safeSpawn("npx", [
|
|
203
|
+
"@biomejs/biome",
|
|
204
|
+
"check",
|
|
205
|
+
"--write",
|
|
206
|
+
"--unsafe",
|
|
207
|
+
...validFiles,
|
|
208
|
+
], {
|
|
209
|
+
timeout: 60000, // Longer timeout for batch
|
|
210
|
+
});
|
|
211
|
+
if (result.error) {
|
|
212
|
+
return {
|
|
213
|
+
success: false,
|
|
214
|
+
fixed: 0,
|
|
215
|
+
changed: 0,
|
|
216
|
+
error: result.error.message,
|
|
217
|
+
};
|
|
218
|
+
}
|
|
219
|
+
// Count how many files actually changed
|
|
220
|
+
let changedCount = 0;
|
|
221
|
+
for (const file of validFiles) {
|
|
222
|
+
// We don't know exactly which files changed without re-reading,
|
|
223
|
+
// so we report total files processed
|
|
224
|
+
changedCount++;
|
|
225
|
+
}
|
|
226
|
+
this.log(`Fixed ${totalFixable} issue(s) in ${validFiles.length} file(s)`);
|
|
227
|
+
return { success: true, fixed: totalFixable, changed: changedCount };
|
|
228
|
+
}
|
|
229
|
+
catch (err) {
|
|
230
|
+
return {
|
|
231
|
+
success: false,
|
|
232
|
+
fixed: 0,
|
|
233
|
+
changed: 0,
|
|
234
|
+
error: err instanceof Error ? err.message : String(err),
|
|
235
|
+
};
|
|
236
|
+
}
|
|
237
|
+
}
|
|
238
|
+
/**
|
|
239
|
+
* Format diagnostics for LLM consumption
|
|
240
|
+
*/
|
|
241
|
+
formatDiagnostics(diags, _filename) {
|
|
242
|
+
if (diags.length === 0)
|
|
243
|
+
return "";
|
|
244
|
+
const lintIssues = diags.filter((d) => d.category === "lint");
|
|
245
|
+
const formatIssues = diags.filter((d) => d.category === "format");
|
|
246
|
+
const errors = diags.filter((d) => d.severity === "error");
|
|
247
|
+
const fixable = diags.filter((d) => d.fixable);
|
|
248
|
+
let result = `[Biome] ${diags.length} issue(s)`;
|
|
249
|
+
if (lintIssues.length)
|
|
250
|
+
result += ` — ${lintIssues.length} lint`;
|
|
251
|
+
if (formatIssues.length)
|
|
252
|
+
result += ` — ${formatIssues.length} format`;
|
|
253
|
+
if (errors.length)
|
|
254
|
+
result += ` — ${errors.length} error(s)`;
|
|
255
|
+
if (fixable.length)
|
|
256
|
+
result += ` — ${fixable.length} fixable`;
|
|
257
|
+
result += ":\n";
|
|
258
|
+
for (const d of diags.slice(0, 15)) {
|
|
259
|
+
const loc = d.line === d.endLine
|
|
260
|
+
? `L${d.line}:${d.column}`
|
|
261
|
+
: `L${d.line}:${d.column}-L${d.endLine}:${d.endColumn}`;
|
|
262
|
+
const rule = d.rule ? ` [${d.rule}]` : "";
|
|
263
|
+
const fix = d.fixable ? " ✓" : "";
|
|
264
|
+
result += ` ${loc}${rule} ${d.message}${fix}\n`;
|
|
265
|
+
}
|
|
266
|
+
if (diags.length > 15) {
|
|
267
|
+
result += ` ... and ${diags.length - 15} more\n`;
|
|
268
|
+
}
|
|
269
|
+
return result;
|
|
270
|
+
}
|
|
271
|
+
/**
|
|
272
|
+
* Generate a diff-like summary of formatting changes
|
|
273
|
+
*/
|
|
274
|
+
getFormatDiff(filePath) {
|
|
275
|
+
const absolutePath = this.withValidatedPath(filePath);
|
|
276
|
+
if (!absolutePath)
|
|
277
|
+
return "";
|
|
278
|
+
const content = fs.readFileSync(absolutePath, "utf-8");
|
|
279
|
+
try {
|
|
280
|
+
// Get formatted output without writing
|
|
281
|
+
const result = safeSpawn("npx", ["@biomejs/biome", "format", absolutePath], {
|
|
282
|
+
timeout: 15000,
|
|
283
|
+
});
|
|
284
|
+
if (result.error || !result.stdout)
|
|
285
|
+
return "";
|
|
286
|
+
const formatted = result.stdout;
|
|
287
|
+
if (content === formatted)
|
|
288
|
+
return "";
|
|
289
|
+
return this.computeDiff(content, formatted);
|
|
290
|
+
}
|
|
291
|
+
catch (err) {
|
|
292
|
+
void err;
|
|
293
|
+
return "";
|
|
294
|
+
}
|
|
295
|
+
}
|
|
296
|
+
// --- Internal ---
|
|
297
|
+
parseDiagnostics(output, filterFile) {
|
|
298
|
+
try {
|
|
299
|
+
// Biome JSON output: {"summary": {...}, "diagnostics": [...], ...}
|
|
300
|
+
const result = JSON.parse(output);
|
|
301
|
+
const diagnostics = [];
|
|
302
|
+
const diags = result.diagnostics || [];
|
|
303
|
+
const filterPath = path.resolve(filterFile);
|
|
304
|
+
for (const item of diags) {
|
|
305
|
+
// Filter to our file
|
|
306
|
+
const itemPath = item.location?.path;
|
|
307
|
+
if (itemPath && path.resolve(itemPath) !== filterPath)
|
|
308
|
+
continue;
|
|
309
|
+
const loc = item.location || {};
|
|
310
|
+
const start = loc.start || {};
|
|
311
|
+
const end = loc.end || start;
|
|
312
|
+
const isLint = item.category?.startsWith("lint/") || false;
|
|
313
|
+
const isFormat = item.category === "format";
|
|
314
|
+
const isAssist = item.category?.startsWith("assist/");
|
|
315
|
+
// Skip non-lint/format diagnostics (like summaries)
|
|
316
|
+
if (!isLint && !isFormat && !isAssist)
|
|
317
|
+
continue;
|
|
318
|
+
// Determine if fixable based on category
|
|
319
|
+
const fixable = isFormat ||
|
|
320
|
+
isAssist ||
|
|
321
|
+
item.category?.includes("organizeImports") ||
|
|
322
|
+
item.message?.includes("fix");
|
|
323
|
+
diagnostics.push({
|
|
324
|
+
line: start.line ?? 1,
|
|
325
|
+
column: start.column ?? 1,
|
|
326
|
+
endLine: end.line ?? start.line ?? 1,
|
|
327
|
+
endColumn: end.column ?? start.column ?? 1,
|
|
328
|
+
severity: item.severity || "warning",
|
|
329
|
+
message: item.message || "Unknown issue",
|
|
330
|
+
rule: isLint ? item.category?.replace("lint/", "") : undefined,
|
|
331
|
+
category: isLint ? "lint" : "format",
|
|
332
|
+
fixable,
|
|
333
|
+
});
|
|
334
|
+
}
|
|
335
|
+
return diagnostics;
|
|
336
|
+
}
|
|
337
|
+
catch (err) {
|
|
338
|
+
void err;
|
|
339
|
+
this.log("Failed to parse biome JSON output");
|
|
340
|
+
return [];
|
|
341
|
+
}
|
|
342
|
+
}
|
|
343
|
+
computeDiff(original, formatted) {
|
|
344
|
+
const origLines = original.split("\n");
|
|
345
|
+
const formLines = formatted.split("\n");
|
|
346
|
+
let changedLines = 0;
|
|
347
|
+
const changes = [];
|
|
348
|
+
const maxLen = Math.max(origLines.length, formLines.length);
|
|
349
|
+
for (let i = 0; i < maxLen; i++) {
|
|
350
|
+
const orig = origLines[i] ?? "";
|
|
351
|
+
const form = formLines[i] ?? "";
|
|
352
|
+
if (orig !== form) {
|
|
353
|
+
changedLines++;
|
|
354
|
+
if (changes.length < 5) {
|
|
355
|
+
if (orig && form) {
|
|
356
|
+
changes.push(` L${i + 1}: \`${orig.trim()}\` → \`${form.trim()}\``);
|
|
357
|
+
}
|
|
358
|
+
else if (!form) {
|
|
359
|
+
changes.push(` L${i + 1}: remove line`);
|
|
360
|
+
}
|
|
361
|
+
else {
|
|
362
|
+
changes.push(` L${i + 1}: add line`);
|
|
363
|
+
}
|
|
364
|
+
}
|
|
365
|
+
}
|
|
366
|
+
}
|
|
367
|
+
let result = ` ${changedLines} line(s) would change`;
|
|
368
|
+
if (origLines.length !== formLines.length) {
|
|
369
|
+
result += ` (${origLines.length} → ${formLines.length} lines)`;
|
|
370
|
+
}
|
|
371
|
+
result += "\n";
|
|
372
|
+
for (const c of changes) {
|
|
373
|
+
result += `${c}\n`;
|
|
374
|
+
}
|
|
375
|
+
if (changedLines > 5) {
|
|
376
|
+
result += ` ... and ${changedLines - 5} more\n`;
|
|
377
|
+
}
|
|
378
|
+
return result;
|
|
379
|
+
}
|
|
380
|
+
}
|