惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

阮一峰的网络日志
阮一峰的网络日志
博客园 - 司徒正美
D
DataBreaches.Net
宝玉的分享
宝玉的分享
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
博客园 - 【当耐特】
人人都是产品经理
人人都是产品经理
博客园 - Franky
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
IT之家
IT之家
博客园 - 三生石上(FineUI控件)
J
Java Code Geeks
腾讯CDC
博客园_首页
The Cloudflare Blog
S
SegmentFault 最新的问题
C
Check Point Blog
美团技术团队
爱范儿
爱范儿
大猫的无限游戏
大猫的无限游戏
Hugging Face - Blog
Hugging Face - Blog
T
The Blog of Author Tim Ferriss
A
About on SuperTechFans
Blog — PlanetScale
Blog — PlanetScale

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(git-hooks): avoid precommit dependency hydration · op...
vincentkoc · 2026-06-19 · via Recent Commits to openclaw:main
11

// Git hook tests validate pre-commit hook behavior and scripts.

22

import { execFileSync } from "node:child_process";

3-

import { mkdirSync, symlinkSync, writeFileSync } from "node:fs";

3+

import { existsSync, mkdirSync, symlinkSync, writeFileSync } from "node:fs";

44

import path from "node:path";

55

import { afterEach, describe, expect, it } from "vitest";

66

import { cleanupTempDirs, makeTempRepoRoot } from "./helpers/temp-repo.js";

@@ -20,6 +20,35 @@ const run = (cwd: string, cmd: string, args: string[] = [], env?: NodeJS.Process

2020

}).trim();

2121

};

222223+

type FailedCommand = {

24+

status: number;

25+

stderr: string;

26+

stdout: string;

27+

};

28+29+

const runFailure = (

30+

cwd: string,

31+

cmd: string,

32+

args: string[] = [],

33+

env?: NodeJS.ProcessEnv,

34+

): FailedCommand => {

35+

try {

36+

run(cwd, cmd, args, env);

37+

} catch (error) {

38+

if (error instanceof Error && "status" in error) {

39+

const failure = error as Error & { status?: number; stderr?: string; stdout?: string };

40+

return {

41+

status: failure.status ?? 1,

42+

stderr: String(failure.stderr ?? ""),

43+

stdout: String(failure.stdout ?? ""),

44+

};

45+

}

46+

throw error;

47+

}

48+49+

throw new Error("expected command to fail");

50+

};

51+2352

function writeExecutable(dir: string, name: string, contents: string): void {

2453

writeFileSync(path.join(dir, name), contents, {

2554

encoding: "utf8",

@@ -54,6 +83,14 @@ function installPreCommitFixture(dir: string): string {

5483

return fakeBinDir;

5584

}

568586+

function installRunNodeToolFixture(dir: string): void {

87+

mkdirSync(path.join(dir, "scripts", "pre-commit"), { recursive: true });

88+

symlinkSync(

89+

path.join(process.cwd(), "scripts", "pre-commit", "run-node-tool.sh"),

90+

path.join(dir, "scripts", "pre-commit", "run-node-tool.sh"),

91+

);

92+

}

93+5794

function splitNonEmptyLines(output: string): string[] {

5895

const lines: string[] = [];

5996

for (const line of output.split("\n")) {

@@ -163,3 +200,56 @@ describe("git-hooks/pre-commit (integration)", () => {

163200

expect(run(dir, "git", ["diff", "--cached", "--name-only"])).toBe("tracked.txt");

164201

});

165202

});

203+204+

describe("scripts/pre-commit/run-node-tool.sh", () => {

205+

it("runs the installed local tool without invoking pnpm", () => {

206+

const dir = makeTempRepoRoot(tempDirs, "openclaw-run-node-tool-local-");

207+

installRunNodeToolFixture(dir);

208+

writeFileSync(path.join(dir, "pnpm-lock.yaml"), "lockfileVersion: '9.0'\n", "utf8");

209+210+

const fakeBinDir = path.join(dir, "bin");

211+

const toolBinDir = path.join(dir, "node_modules", ".bin");

212+

mkdirSync(fakeBinDir, { recursive: true });

213+

mkdirSync(toolBinDir, { recursive: true });

214+

writeExecutable(

215+

fakeBinDir,

216+

"pnpm",

217+

"#!/usr/bin/env bash\necho 'pnpm should not run from run-node-tool' >&2\nexit 99\n",

218+

);

219+

writeExecutable(toolBinDir, "oxfmt", "#!/usr/bin/env bash\nprintf 'local:%s\\n' \"$*\"\n");

220+221+

expect(

222+

run(dir, "bash", ["scripts/pre-commit/run-node-tool.sh", "oxfmt", "--write", "a.ts"], {

223+

PATH: `${fakeBinDir}:${process.env.PATH ?? ""}`,

224+

}),

225+

).toBe("local:--write a.ts");

226+

});

227+228+

it("fails before pnpm can hydrate dependencies when node_modules is missing", () => {

229+

const dir = makeTempRepoRoot(tempDirs, "openclaw-run-node-tool-missing-deps-");

230+

installRunNodeToolFixture(dir);

231+

writeFileSync(path.join(dir, "pnpm-lock.yaml"), "lockfileVersion: '9.0'\n", "utf8");

232+233+

const fakeBinDir = path.join(dir, "bin");

234+

const markerPath = path.join(dir, "pnpm-called");

235+

mkdirSync(fakeBinDir, { recursive: true });

236+

writeExecutable(

237+

fakeBinDir,

238+

"pnpm",

239+

`#!/usr/bin/env bash\ntouch ${JSON.stringify(markerPath)}\nexit 99\n`,

240+

);

241+242+

const result = runFailure(

243+

dir,

244+

"bash",

245+

["scripts/pre-commit/run-node-tool.sh", "oxfmt", "--write", "a.ts"],

246+

{ PATH: `${fakeBinDir}:${process.env.PATH ?? ""}` },

247+

);

248+249+

expect(result.status).toBe(1);

250+

expect(result.stderr).toContain(

251+

"Missing repo dependencies: cannot run oxfmt without node_modules.",

252+

);

253+

expect(existsSync(markerPath)).toBe(false);

254+

});

255+

});