Tianyi Cui
774d460889
The audit swept every packages/*/* plugin for the new AGENTS.md
convention (no hardcoded tunables in plugins) and exposes each finding
as a defaulted, validated Config field. Defaults are the previously
hardcoded values throughout, so no deployment or golden changes.
- tool-fs (had NO Config): readLimit, readMaxLineLength, readMaxBytes,
readStreamMinSize. The caps thread through ReadToolCaps/ReadWindow —
read-render already documented that the consumer applies the caps, so
they become explicit per-request fields.
- tool-web: searchMaxResults (WEB_SEARCH_MAX_RESULTS stays as the
schemastery default). Also fixes the stale GREP_LIMIT references in
search.ts and the web-capability-seam RFC (no such constant exists).
- bash-local: graceMs (SIGTERM->SIGKILL escalation grace). The
RunInternals.graceMs test seam is gone: graceMs is now a required
SpawnSpec field filled from config, so tests exercise the real
config path and the defaults live in exactly one place.
- subagent-acp: disposeEofGraceMs / disposeGraceMs. The AcpRunSpec
fields become required for the same one-defaulting-layer reason.
- session-persistence-sqlite: journalMode ('wal' default; the
rollback-journal modes serve filesystems where WAL's shared-memory
files do not work, e.g. network mounts).
- hooks-claude + hooks-codex: stderrSummaryMaxChars for the persisted
hook/result stderr summary. The duplicated summarize() helpers merge
into hook-protocol's summarizeStderr(stderr, maxChars), beside the
HookResultRecord field it feeds, with the bound parameterized the
same way runHook's defaultTimeoutMs already is.
- compact-basic: charsPerToken for the token estimator (default 4, the
English-text heuristic; CJK-heavy deployments need ~1-2 or compaction
fires far too late). Also corrects the BasicCompactService class doc,
which claimed defaults the required-field config never had.
- fs-local: deletes the dead STREAM_MIN_SIZE constant and the dead
FsIoInternals.streamMinSize seam — the read-routing bound lives in
the consumer (tool-fs), where it is now config. This is item 1 of
the proposed prune-write-only-fs-surface RFC, annotated accordingly.
Every new field gets range validation (following the existing
assertPositiveFinite pattern), a README row, and tests covering the
configured behavior, the schema default, and load-time rejection.
39 lines
1.8 KiB
TypeScript
39 lines
1.8 KiB
TypeScript
/**
|
|
* `@deepseek-ai/dsh-hook-protocol` — the shared core of the Claude Code / Codex
|
|
* hook wire protocol. NOT a cordis plugin: it registers nothing and injects
|
|
* nothing. It is a LIBRARY of dialect-neutral primitives the two bridge plugins
|
|
* (`dsh-hooks-claude`, `dsh-hooks-codex`) import to avoid re-implementing the
|
|
* identical halves of the protocol:
|
|
*
|
|
* - {@link matchesMatcher} — the matcher primitive (literal-or-regex by dialect).
|
|
* - {@link runHook} + {@link parseHookOutput} — run a command hook via `ctx.bash`
|
|
* (stdin payload + env) and decode its exit-code/stdout/stderr into a neutral
|
|
* {@link HookOutput}.
|
|
* - {@link mergeHookOutputs} — fold multiple matched hooks into one
|
|
* most-restrictive {@link MergedHookOutcome} (deny > ask > allow).
|
|
* - {@link appendHookInvoked} / {@link appendHookResult} — the log-only `hook/*`
|
|
* session-event helpers (declaration-merged into `SessionEventMap`).
|
|
*
|
|
* Each bridge owns what genuinely DIFFERS: building the per-event stdin payload
|
|
* (CC vs Codex field sets), the dialect's env/substitution, and mapping the
|
|
* neutral outcome onto the harness's seam-specific typed Decisions.
|
|
*
|
|
* @module @deepseek-ai/dsh-hook-protocol
|
|
*/
|
|
|
|
export type {
|
|
CommandHook,
|
|
HookDialect,
|
|
HookOutput,
|
|
MatcherGroup,
|
|
MatcherMode,
|
|
} from './types.ts'
|
|
export { matchesMatcher } from './matcher.ts'
|
|
export { BLOCKING_EXIT_CODE, parseHookOutput } from './codec.ts'
|
|
export { runHook } from './runner.ts'
|
|
export type { RunHookOptions, RunHookResult } from './runner.ts'
|
|
export { mergeHookOutputs } from './merge.ts'
|
|
export type { MergedDecision, MergedHookOutcome } from './merge.ts'
|
|
export { appendHookInvoked, appendHookResult, summarizeStderr } from './events.ts'
|
|
export type { HookInvocation, HookResultRecord } from './events.ts'
|