build(style): add ty pre-commit hook
Metadata
Current evaluation
⚠️ This summary may be outdated — the issue content or evaluation prompts have changed since this evaluation was made.
Adds astral-sh/ty-pre-commit hook (v0.0.96) to .pre-commit-config.yaml. Approved by 1 reviewer; one CI check failing on PR title format.
Suggested action: needs review
Reason: PR is approved with 0 unresolved review comments and adds a straightforward 4-line pre-commit hook configuration. One CI check ('policy / Check PR title for Conventional Commits') is failing, likely because the title 'build(style): add ty pre-commit hook' uses an unconventional dual-type prefix. The author should fix the PR title to a single conventional commit type (e.g., 'style: add ty pre-commit hook') before merge.
Evaluated by: mr-cal-bot
Impact:
20
Quick Win:
11
Complexity:
5
Confidence:
80
Actionability:
60
Issue body
Starbase’s pre-commit configuration lacked support for Astral’s `ty` type checker.
- **Hook** — Add `astral-sh/ty-pre-commit`, pinned to `v0.0.96`:
```yaml
- repo: https://github.com/astral-sh/ty-pre-commit
rev: "v0.0.96"
hooks:
- id: ty
```
<!-- START COPILOT CODING AGENT SUFFIX -->
- Fixes #642
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b |
Impact:
20
Quick Win:
11
Complexity:
5
Confidence:
80
Actionability:
60
|
needs review | Adds astral-sh/ty-pre-commit hook (v0.0.96) to .pre-commit-config.yaml. Approved by 1 reviewer; one CI check failing on PR title format. |
Update history
| Date | Change |
|---|---|
| updated | |
| created |
Related work
-
Related To:
starbase#642
(confidence 95%)
This PR directly implements the feature request described in issue #642 to add the ty-pre-commit hook.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #642 Add `ty-pre-commit` hook | starbase | open | Feature request to add astral-sh/ty-pre-commit hook to .pre-commit-config.yaml alongside the existing ruff hook. |