Summary of Changes

Hello, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request introduces harper-cli as a new grammar linter, expanding the project's built-in linting capabilities. The changes provide configurations for checking both general text content and commit messages, aiming to improve the overall quality and consistency of written materials across the repository.

Highlights

• New Grammar Linter Integration: Integrated harper-cli as a new built-in linter for grammar checking, enhancing the project's ability to maintain high-quality prose and documentation.
• Dedicated Configuration for Text and Commit Messages: Added separate Pkl configurations for harper-cli to specifically lint general text files and commit messages, ensuring targeted grammar checks.
• Tool Stub for harper-cli: Included a tool stub for harper-cli to manage its version and execution within the project's testing environment.

Greptile Summary

This PR adds two new builtins — harper (for prose files) and harper_commit_message (for commit messages) — using harper-cli as the grammar checker, along with the corresponding mise tool stub (harper-cli resolves correctly to aqua:Automattic/harper/harper-cli in the registry). The harper.pkl step includes check tests; harper_commit_message.pkl omits a tests block, which is inconsistent with project convention.

Confidence Score: 4/5

• The PR is close to mergeable but has open feedback from previous review rounds that should be resolved first.
• Previous thread discussions flagged a P1 concern (file-type coverage) that remains unresolved in the current diff. My review adds only a P2 finding (missing tests in harper_commit_message.pkl). Score of 4 reflects the outstanding thread work rather than any new blocking issue.
• pkl/builtins/harper.pkl — open P1 feedback from prior review rounds still applies to this file.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Git Hook Triggered] --> B{Hook Type}
    B -- pre-commit --> C[harper step]
    B -- commit-msg --> D[harper_commit_message step]

    C --> E["types = List('text')\nFilter staged files"]
    E --> F["harper-cli lint {{ files }}"]
    F --> G{Exit code?}
    G -- 0 --> H[Commit proceeds]
    G -- non-zero --> I[Commit blocked\nGrammar issues reported]

    D --> J["harper-cli lint {{commit_msg_file}}"]
    J --> K{Exit code?}
    K -- 0 --> H
    K -- non-zero --> L[Commit blocked\nGrammar issues in message]

_{Reviews (3): Last reviewed commit: "feat(grammar): add harper config to hk b..." | Re-trigger Greptile}