oxlint json parse failure fallback

[jbolda]

What does this PR do?

In some failure or no-op cases, oxlint seems to return output which is not json-parseable even with --json passed. I suspect these are mostly edge cases which we don't highly value handling. Went with a simple try/catch and log out the result instead.

Example

https://github.com/bombshell-dev/repo v0.0.0 using v22.14.0
❯ pnpm lint

> @bomb.sh/ui-workspace@0.0.0 lint /dev/github/bombshell-dev/repo
> bsh lint

SyntaxError: Unexpected token 'N', "No files f"... is not valid JSON
    at JSON.parse (<anonymous>)
    at runOxlint (file:///dev/github/bombshell-dev/repo/node_modules/.pnpm/@bomb.sh+tools@0.5.1_oxc-resolver@11.20.0_vite@8.0.16_jiti@2.7.0_yaml@2.9.0_/node_modules/@bomb.sh/tools/dist/commands/lint.mjs:16:15)
    at async Promise.allSettled (index 0)
    at async collectViolations (file:///dev/github/bombshell-dev/repo/node_modules/.pnpm/@bomb.sh+tools@0.5.1_oxc-resolver@11.20.0_vite@8.0.16_jiti@2.7.0_yaml@2.9.0_/node_modules/@bomb.sh/tools/dist/commands/lint.mjs:149:18)
    at async lint (file:///dev/github/bombshell-dev/repo/node_modules/.pnpm/@bomb.sh+tools@0.5.1_oxc-resolver@11.20.0_vite@8.0.16_jiti@2.7.0_yaml@2.9.0_/node_modules/@bomb.sh/tools/dist/commands/lint.mjs:172:21)
    at async main (file:///dev/github/bombshell-dev/repo/node_modules/.pnpm/@bomb.sh+tools@0.5.1_oxc-resolver@11.20.0_vite@8.0.16_jiti@2.7.0_yaml@2.9.0_/node_modules/@bomb.sh/tools/dist/bin.mjs:33:2)

packages/new-pkg/src/index.ts
  -       warning     Unused file  knip/unused-file

1 warning

Result

https://github.com/bombshell-dev/repo on main [!] v0.0.0 using v22.14.0 took 2s
❯ pnpm lint

> @bomb.sh/new-repo-workspace@0.0.0 lint /dev/github/bombshell-dev/repo
> bsh lint

No files found to lint. Please check your paths and ignore patterns.
{ "diagnostics": [],
              "number_of_files": 0,
              "number_of_rules": 145,
              "threads_count": 8,
              "start_time": 0.084649375
            }


No issues found.

Type of change

• Bug fix
• Feature
• Refactor (no behavior change)
• Documentation
• Performance improvement
• Tests
• Chore (dependencies, CI, tooling)

Checklist

• All tests pass (pnpm test)
• Files are formatted (pnpm format)
• I have added/updated tests for my changes (if applicable)
• I have added a changeset

AI-generated code disclosure

• This PR includes AI-generated code

[changeset-bot]

🦋 Changeset detected

Latest commit: acf2725

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@bomb.sh/tools	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@bomb.sh/tools@37

commit: acf2725

[dreyfus92]

lgtm, have you tested this fix already? 👀

[jbolda]

Aye, the "Result" snippet is running the preview package in the same repo!