Skip to content

[codex] Fix explicit-any lint warnings - #80

Merged
jack-arturo merged 1 commit into
feat/client-crud-refreshfrom
codex/mcp-toggl-explicit-any-cleanup
Jun 26, 2026
Merged

[codex] Fix explicit-any lint warnings#80
jack-arturo merged 1 commit into
feat/client-crud-refreshfrom
codex/mcp-toggl-explicit-any-cleanup

Conversation

@jack-arturo

Copy link
Copy Markdown
Member

Summary

  • Replaces remaining @typescript-eslint/no-explicit-any surfaces with typed helpers and unknown-based assertions.
  • Preserves the public setAPI testing hook shape while validating it internally.
  • Adds a regression test for empty period validation.

Verification

  • npm run lint -- --max-warnings=0
  • npm run build
  • npm test (55 passed, 4 skipped)
  • git diff --check

Review

  • Independent code-review agent initially found two Important findings.
  • Both were fixed and the reviewer rechecked the final diff with no remaining Critical, Important, or Minor findings.

Stacked on top of feat/client-crud-refresh because that feature branch already has PR #61 open.

@jack-arturo
jack-arturo merged commit e4c508b into feat/client-crud-refresh Jun 26, 2026
2 checks passed
@jack-arturo
jack-arturo deleted the codex/mcp-toggl-explicit-any-cleanup branch June 26, 2026 13:24
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.

1 participant