Skip to content

refactor(lint): finish #1569 lint hardening (Fixes #1569)#1907

Merged
acoliver merged 458 commits into
mainfrom
issue1569d
May 11, 2026
Merged

refactor(lint): finish #1569 lint hardening (Fixes #1569)#1907
acoliver merged 458 commits into
mainfrom
issue1569d

Conversation

@acoliver
Copy link
Copy Markdown
Collaborator

@acoliver acoliver commented Apr 22, 2026

Summary

This PR completes the issue #1569 lint-hardening follow-through on branch issue1569d.

It eliminates the remaining Phase 5 diagnostics for the configured complexity and size rules by combining targeted decompositions with documented legacy-boundary exemptions where further decomposition would be unsafe for this PR. It also includes the earlier warn-to-error hardening already landed on this branch.

Fixes #1569

Baseline and warning trajectory

Baseline reference: project-plans/issue1569d/BASELINE.md

  • Initial issue1569d baseline: 10456 warnings, 0 errors, 87 distinct rules.
  • Current Phase 5 target scan evidence: chunked package-source scans report 0 diagnostics for complexity, max-lines, and sonarjs/cognitive-complexity across cli, core, a2a-server, and vscode companion source directories.
  • Strict boolean checks were promoted to error after repo-wide cleanup.
  • Several Phase 4 and Phase 5 legacy boundaries are documented with scoped eslint directives rather than hidden by broad config changes.

Notable changes

  • Decomposed large CLI/config/runtime flows including settings loading, extension loading, profile application, non-interactive response handling, session browser tests, bucket failover tests, oauth-manager tests, and VS Code extension update checks.
  • Documented remaining legacy core/provider/CLI/test complexity and size boundaries with explicit Phase 5 justifications.
  • Cleaned stale lint disables and formatter drift surfaced by final verification.

Verification

Final verification attempted on branch issue1569d after final commits:

  • npm run format: exit 0.
  • npm run typecheck: exit 0 after refreshing core build outputs.
  • npm run build: exit 0.
  • Synthetic smoke command: exit 0; provider API call reported an environment/account error, but the command exited successfully.
  • Chunked Phase 5 target scans: target_count 0 for scanned package source groups.
  • npm run test: repeatedly terminated by Signal 15 after core tests were running/passing and before a concrete test failure was reported; log retained at /tmp/issue1569d-final-test.log.
  • npm run lint: repeatedly terminated by Signal 15 with only the command banner and no diagnostics; scoped/chunked eslint checks passed for touched and target-rule scopes.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer:e2e:ok Trusted contributor; maintainer-approved E2E run

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable strict lint and complexity rules on core/cli

2 participants