Skip to content

Conversation

@stdrc
Copy link
Collaborator

@stdrc stdrc commented Dec 26, 2025

Summary

  • split prek hooks by workspace project (root/kosong/kaos)
  • install prek during make prepare and update docs

Testing

  • prek hooks (format/check for kimi-cli/kosong/kaos)

Copilot AI review requested due to automatic review settings December 26, 2025 18:02
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates the repository from pre-commit to prek for git hooks, enabling workspace-aware hook management. The migration splits hooks by workspace project so that only modified projects run their respective formatting and checking commands.

Key changes:

  • Added install-prek target to Makefile that installs prek and sets up git hooks
  • Created workspace-specific pre-commit configs for kosong and kaos packages with orphan mode
  • Updated root pre-commit config to use project-specific make targets (format-kimi-cli, check-kimi-cli)
  • Updated CONTRIBUTING.md documentation to reflect prek usage and workspace mode behavior

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Makefile Added install-prek target and integrated it into prepare target to automatically install prek and hooks during setup
.pre-commit-config.yaml Updated hook IDs and names to use project-specific targets (format-kimi-cli, check-kimi-cli) for the root workspace
packages/kosong/.pre-commit-config.yaml Added orphan config for kosong package with hooks that run format-kosong and check-kosong from repo root
packages/kaos/.pre-commit-config.yaml Added orphan config for kaos package with hooks that run format-pykaos and check-pykaos from repo root
CONTRIBUTING.md Updated documentation from pre-commit to prek, explaining workspace mode and new installation process

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@stdrc stdrc merged commit d096a4e into main Dec 26, 2025
30 checks passed
@stdrc stdrc deleted the rc/prek branch December 26, 2025 18:07
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.

2 participants