Skip to content

refactor(core): centralize lint result adaptation - #138

Merged
luojiyin1987 merged 1 commit into
masterfrom
refactor/core-result-adapter
Aug 10, 2026
Merged

refactor(core): centralize lint result adaptation#138
luojiyin1987 merged 1 commit into
masterfrom
refactor/core-result-adapter

Conversation

@luojiyin1987

Copy link
Copy Markdown
Contributor

Summary

  • Add one adapter for core lint results.
  • Use it for stdin lint and fix paths.
  • Use it for worker lint and fix paths.
  • Add direct adapter tests.

Why

The CLI mapped the same core fields in three places.

A shared adapter keeps this boundary consistent.

The adapter excludes core diagnostics from BatchLintItem.

Impact

CLI output and exit behavior stay unchanged.

Lint results now include fixedResult with a null value.

Validation

  • npm run typecheck
  • Prettier check for changed files
  • npm test -- --runInBand
  • 24 test suites passed
  • 186 tests passed

Closes #137

@luojiyin1987
luojiyin1987 marked this pull request as ready for review August 10, 2026 01:02
@luojiyin1987
luojiyin1987 merged commit 8837c68 into master Aug 10, 2026
6 checks passed
@luojiyin1987
luojiyin1987 deleted the refactor/core-result-adapter branch August 10, 2026 01:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Centralize core lint result adaptation

1 participant