mirror of
https://github.com/affaan-m/everything-claude-code.git
synced 2026-07-01 20:41:26 +08:00
affaan
6f01d26420
The inline node -e resolver blob was duplicated ~60x across hooks.json, command docs, and translations. Each copy inlined the full ~700-char plugin-root search using a spread over nested array literals (p.join(d,'plugins',...s) over [['ecc'],...]), which breaks Windows hook execution due to shell quoting (#2368). Collapse every copy to a 250-char locator that loads the committed resolve-ecc-root module and delegates to resolveEccRoot() — no spread, no nested array literals, no escaped double quotes. The real search logic now lives in one tested module. Also route session-start-bootstrap.js through resolveEccRoot() instead of its own duplicated reimplementation, and fix the auto-update.md 'marketplace' (singular) typo along the way. Guard tests updated: discovery behavior is asserted against resolveEccRoot(); the inline is asserted to delegate and to contain no Windows-fragile constructs. Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
86 lines
2.9 KiB
JavaScript
86 lines
2.9 KiB
JavaScript
#!/usr/bin/env node
|
|
'use strict';
|
|
|
|
/**
|
|
* session-start-bootstrap.js
|
|
*
|
|
* Bootstrap loader for the ECC SessionStart hook.
|
|
*
|
|
* Problem this solves: the previous approach embedded this logic as an inline
|
|
* `node -e "..."` string inside hooks.json. Characters like `!` (used in
|
|
* `!org.isDirectory()`) can trigger bash history expansion or other shell
|
|
* interpretation issues depending on the environment, causing
|
|
* "SessionStart:startup hook error" to appear in the Claude Code CLI header.
|
|
*
|
|
* By extracting to a standalone file, the shell never sees the JavaScript
|
|
* source and the `!` characters are safe. Behaviour is otherwise identical.
|
|
*
|
|
* How it works:
|
|
* 1. Reads the raw JSON event from stdin (passed by Claude Code).
|
|
* 2. Resolves the ECC plugin root directory (via CLAUDE_PLUGIN_ROOT env var
|
|
* or a set of well-known fallback paths).
|
|
* 3. Delegates to `scripts/hooks/run-with-flags.js` with the `session:start`
|
|
* event, which applies hook-profile gating and then runs session-start.js.
|
|
* 4. Passes stdout/stderr through and forwards the child exit code.
|
|
* 5. If the plugin root cannot be found, emits a warning and passes stdin
|
|
* through unchanged so Claude Code can continue normally.
|
|
*/
|
|
|
|
const fs = require('fs');
|
|
const path = require('path');
|
|
const { spawnSync } = require('child_process');
|
|
const { resolveEccRoot } = require('../lib/resolve-ecc-root');
|
|
|
|
// Read the raw JSON event from stdin
|
|
const raw = fs.readFileSync(0, 'utf8');
|
|
|
|
// Path (relative to plugin root) to the hook runner
|
|
const rel = path.join('scripts', 'hooks', 'run-with-flags.js');
|
|
|
|
// Resolve the ECC plugin root via the shared resolver, probing for the runner
|
|
// so a valid root is one that actually contains run-with-flags.js.
|
|
const root = resolveEccRoot({ probe: rel });
|
|
const script = path.join(root, rel);
|
|
|
|
if (fs.existsSync(script)) {
|
|
const result = spawnSync(
|
|
process.execPath,
|
|
[script, 'session:start', 'scripts/hooks/session-start.js', 'minimal,standard,strict'],
|
|
{
|
|
input: raw,
|
|
encoding: 'utf8',
|
|
env: process.env,
|
|
cwd: process.cwd(),
|
|
timeout: 30000,
|
|
}
|
|
);
|
|
|
|
const stdout = typeof result.stdout === 'string' ? result.stdout : '';
|
|
if (stdout) {
|
|
process.stdout.write(stdout);
|
|
} else {
|
|
process.stdout.write(raw);
|
|
}
|
|
|
|
if (result.stderr) {
|
|
process.stderr.write(result.stderr);
|
|
}
|
|
|
|
if (result.error || result.status === null || result.signal) {
|
|
const reason = result.error
|
|
? result.error.message
|
|
: result.signal
|
|
? 'signal ' + result.signal
|
|
: 'missing exit status';
|
|
process.stderr.write('[SessionStart] ERROR: session-start hook failed: ' + reason + '\n');
|
|
process.exit(1);
|
|
}
|
|
|
|
process.exit(Number.isInteger(result.status) ? result.status : 0);
|
|
}
|
|
|
|
process.stderr.write(
|
|
'[SessionStart] WARNING: could not resolve ECC plugin root; skipping session-start hook\n'
|
|
);
|
|
process.stdout.write(raw);
|