mirror of
https://github.com/tiennm99/ccs.git
synced 2026-10-12 22:14:22 +00:00
fix(websearch): restore hook recovery and force register
This commit is contained in:
1 parent
ed37a2a513
commit
0821c68559
4 files changed
+61
-3
No files matched your search
@@ -216,7 +216,9 @@ export function registerApiProfileOrphans(options?: {
|
|||||||
}
|
}
|
||||||
|
|
||||||
try {
|
try {
|
||||||
ensureProfileHooksOrThrow(orphan.name);
|
if (orphan.validation.valid) {
|
||||||
|
ensureProfileHooksOrThrow(orphan.name);
|
||||||
|
}
|
||||||
registerApiProfileInConfig(orphan.name, options?.target || 'claude', options?.force || false);
|
registerApiProfileInConfig(orphan.name, options?.target || 'claude', options?.force || false);
|
||||||
result.registered.push(orphan.name);
|
result.registered.push(orphan.name);
|
||||||
} catch (error) {
|
} catch (error) {
|
||||||
|
|||||||
@@ -25,8 +25,17 @@ function hasMatchingHookContents(sourcePath: string, destinationPath: string): b
|
|||||||
}
|
}
|
||||||
|
|
||||||
const source = fs.readFileSync(sourcePath);
|
const source = fs.readFileSync(sourcePath);
|
||||||
const destination = fs.readFileSync(destinationPath);
|
try {
|
||||||
return source.equals(destination);
|
const destination = fs.readFileSync(destinationPath);
|
||||||
|
return source.equals(destination);
|
||||||
|
} catch (error) {
|
||||||
|
if (process.env.CCS_DEBUG) {
|
||||||
|
console.error(
|
||||||
|
warn(`Existing WebSearch hook is unreadable; reinstalling: ${(error as Error).message}`)
|
||||||
|
);
|
||||||
|
}
|
||||||
|
return false;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
function getTempHookPath(hookPath: string): string {
|
function getTempHookPath(hookPath: string): string {
|
||||||
|
|||||||
@@ -182,6 +182,26 @@ describe('profile lifecycle service', () => {
|
|||||||
expect(result.skipped).toEqual([]);
|
expect(result.skipped).toEqual([]);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('registers malformed orphan settings when force bypasses validation', async () => {
|
||||||
|
const ccsDir = path.join(tempHome, '.ccs');
|
||||||
|
fs.mkdirSync(ccsDir, { recursive: true });
|
||||||
|
|
||||||
|
const malformedPath = path.join(ccsDir, 'bad.settings.json');
|
||||||
|
fs.writeFileSync(malformedPath, '{ invalid json', 'utf8');
|
||||||
|
fs.writeFileSync(path.join(ccsDir, 'config.json'), JSON.stringify({ profiles: {} }, null, 2) + '\n');
|
||||||
|
|
||||||
|
const result = await runInScopedCcsDir(() =>
|
||||||
|
registerApiProfileOrphans({ names: ['bad'], force: true })
|
||||||
|
);
|
||||||
|
const config = await runInScopedCcsDir(() => loadConfigSafe());
|
||||||
|
|
||||||
|
expect(result.registered).toEqual(['bad']);
|
||||||
|
expect(result.skipped).toEqual([]);
|
||||||
|
expect(config.profiles.bad).toBe('~/.ccs/bad.settings.json');
|
||||||
|
expect(fs.existsSync(path.join(ccsDir, 'hooks', 'websearch-transformer.cjs'))).toBe(false);
|
||||||
|
expect(fs.readFileSync(malformedPath, 'utf8')).toBe('{ invalid json');
|
||||||
|
});
|
||||||
|
|
||||||
it('redacts all sensitive env values during export when includeSecrets=false', async () => {
|
it('redacts all sensitive env values during export when includeSecrets=false', async () => {
|
||||||
const ccsDir = path.join(tempHome, '.ccs');
|
const ccsDir = path.join(tempHome, '.ccs');
|
||||||
fs.mkdirSync(ccsDir, { recursive: true });
|
fs.mkdirSync(ccsDir, { recursive: true });
|
||||||
|
|||||||
@@ -119,6 +119,33 @@ describe('ensureProfileHooks', () => {
|
|||||||
expect(installedHook).toContain('CCS WebSearch Hook');
|
expect(installedHook).toContain('CCS WebSearch Hook');
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('repairs an unreadable existing hook binary instead of failing the profile setup', () => {
|
||||||
|
if (process.platform === 'win32') return;
|
||||||
|
|
||||||
|
setupTempHome();
|
||||||
|
|
||||||
|
const hookPath = getHookPath();
|
||||||
|
fs.mkdirSync(path.dirname(hookPath), { recursive: true });
|
||||||
|
fs.writeFileSync(hookPath, '// unreadable stale hook', 'utf8');
|
||||||
|
fs.chmodSync(hookPath, 0o200);
|
||||||
|
|
||||||
|
try {
|
||||||
|
const ensured = ensureProfileHooks('glm');
|
||||||
|
const settingsPath = path.join(getCcsDir(), 'glm.settings.json');
|
||||||
|
const settings = JSON.parse(fs.readFileSync(settingsPath, 'utf8'));
|
||||||
|
const installedHook = fs.readFileSync(hookPath, 'utf8');
|
||||||
|
|
||||||
|
expect(ensured).toBe(true);
|
||||||
|
expect(installedHook).not.toBe('// unreadable stale hook');
|
||||||
|
expect(installedHook).toContain('CCS WebSearch Hook');
|
||||||
|
expect(settings.hooks.PreToolUse[0].hooks[0].command).toBe(`node "${hookPath}"`);
|
||||||
|
} finally {
|
||||||
|
if (fs.existsSync(hookPath)) {
|
||||||
|
fs.chmodSync(hookPath, 0o644);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
});
|
||||||
|
|
||||||
it('succeeds when another process installs the hook during a failed local install', () => {
|
it('succeeds when another process installs the hook during a failed local install', () => {
|
||||||
setupTempHome();
|
setupTempHome();
|
||||||
|
|
||||||
|
|||||||
Reference in new issue
Block a user