Mirrors/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-03-28 20:16:28 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix(hooks): avoid repo-wide format churn

Browse Source
This commit is contained in:
Peter Steinberger
2026-03-27 20:19:36 +00:00
parent 4430805719
commit 888be707cf
2 changed files with 4 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
git-hooks/pre-commit
View File

@@ -50,15 +50,14 @@ git add -- "${files[@]}"
# This hook is also exercised from lightweight temp repos in tests, where the
# staged-file safety behavior matters but the full OpenClaw workspace does not
# exist. Only run the repo-wide gate inside a real checkout.
# exist. Only run the repo-wide validation gate inside a real checkout.
if [[ -f "$ROOT_DIR/package.json" ]] && [[ -f "$ROOT_DIR/pnpm-lock.yaml" ]]; then
cd "$ROOT_DIR"
case "${FAST_COMMIT:-}" in
1|true|TRUE|yes|YES|on|ON)
echo "FAST_COMMIT enabled: skipping pnpm format and pnpm check in pre-commit hook."
echo "FAST_COMMIT enabled: skipping pnpm check in pre-commit hook."
;;
*)
pnpm format
pnpm check
;;
esac

6
test/git-hooks-pre-commit.test.ts
View File

@@ -73,7 +73,7 @@ describe("git-hooks/pre-commit (integration)", () => {
expect(staged).toEqual(["--all"]);
});
it("skips pnpm format and pnpm check when FAST_COMMIT is enabled", () => {
it("skips pnpm check when FAST_COMMIT is enabled", () => {
const dir = mkdtempSync(path.join(os.tmpdir(), "openclaw-pre-commit-yolo-"));
run(dir, "git", ["init", "-q", "--initial-branch=main"]);
@@ -116,8 +116,6 @@ describe("git-hooks/pre-commit (integration)", () => {
FAST_COMMIT: "1",
});
expect(output).toContain(
"FAST_COMMIT enabled: skipping pnpm format and pnpm check in pre-commit hook.",
);
expect(output).toContain("FAST_COMMIT enabled: skipping pnpm check in pre-commit hook.");
});
});