fix(rules-injector): memoize project-root lookup per process lifecycle
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
This commit is contained in:
@@ -3,6 +3,7 @@ import { createDynamicTruncator } from "../../shared/dynamic-truncator";
|
|||||||
import { getRuleInjectionFilePath } from "./output-path";
|
import { getRuleInjectionFilePath } from "./output-path";
|
||||||
import { createSessionCacheStore } from "./cache";
|
import { createSessionCacheStore } from "./cache";
|
||||||
import { createRuleInjectionProcessor } from "./injector";
|
import { createRuleInjectionProcessor } from "./injector";
|
||||||
|
import { clearProjectRootCache } from "./project-root-finder";
|
||||||
|
|
||||||
interface ToolExecuteInput {
|
interface ToolExecuteInput {
|
||||||
tool: string;
|
tool: string;
|
||||||
@@ -75,6 +76,7 @@ export function createRulesInjectorHook(
|
|||||||
if (sessionInfo?.id) {
|
if (sessionInfo?.id) {
|
||||||
clearSessionCache(sessionInfo.id);
|
clearSessionCache(sessionInfo.id);
|
||||||
}
|
}
|
||||||
|
clearProjectRootCache();
|
||||||
}
|
}
|
||||||
|
|
||||||
if (event.type === "session.compacted") {
|
if (event.type === "session.compacted") {
|
||||||
@@ -83,6 +85,7 @@ export function createRulesInjectorHook(
|
|||||||
if (sessionID) {
|
if (sessionID) {
|
||||||
clearSessionCache(sessionID);
|
clearSessionCache(sessionID);
|
||||||
}
|
}
|
||||||
|
clearProjectRootCache();
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|||||||
@@ -2,6 +2,12 @@ import { existsSync, statSync } from "node:fs";
|
|||||||
import { dirname, join } from "node:path";
|
import { dirname, join } from "node:path";
|
||||||
import { PROJECT_MARKERS } from "./constants";
|
import { PROJECT_MARKERS } from "./constants";
|
||||||
|
|
||||||
|
const projectRootCache = new Map<string, string | null>();
|
||||||
|
|
||||||
|
export function clearProjectRootCache(): void {
|
||||||
|
projectRootCache.clear();
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Find project root by walking up from startPath.
|
* Find project root by walking up from startPath.
|
||||||
* Checks for PROJECT_MARKERS (.git, pyproject.toml, package.json, etc.)
|
* Checks for PROJECT_MARKERS (.git, pyproject.toml, package.json, etc.)
|
||||||
@@ -10,6 +16,16 @@ import { PROJECT_MARKERS } from "./constants";
|
|||||||
* @returns Project root path or null if not found
|
* @returns Project root path or null if not found
|
||||||
*/
|
*/
|
||||||
export function findProjectRoot(startPath: string): string | null {
|
export function findProjectRoot(startPath: string): string | null {
|
||||||
|
if (projectRootCache.has(startPath)) {
|
||||||
|
return projectRootCache.get(startPath) ?? null;
|
||||||
|
}
|
||||||
|
|
||||||
|
const projectRoot = findProjectRootWithoutCache(startPath);
|
||||||
|
projectRootCache.set(startPath, projectRoot);
|
||||||
|
return projectRoot;
|
||||||
|
}
|
||||||
|
|
||||||
|
function findProjectRootWithoutCache(startPath: string): string | null {
|
||||||
let current: string;
|
let current: string;
|
||||||
|
|
||||||
try {
|
try {
|
||||||
|
|||||||
Reference in New Issue
Block a user