mirror of
https://github.com/openclaw/openclaw.git
synced 2026-08-17 16:12:21 -06:00
SunnyShu
16cde04136
* fix(agents): allow required-preflight native Codex compaction Required reply-preflight compaction on a Codex app-server-backed session returns the intentional `ok: true, compacted: false` "codex app-server owns automatic compaction" no-op because the preflight caller never passes `allowNonManualNativeRequest`. The reply/preflight path then misclassifies that successful skip as a failure and throws, dropping the user's turn with "Context is too large and auto-compaction could not recover this turn." The equivalent CLI path was fixed by #88207; this is the second, unpatched caller. Route required-preflight through the existing private `compactAfterContextEngine` harness capability (which already passes `allowNonManualNativeRequest: true`) by adding a typed `nativeCompactionRequest: "required_preflight" | "after_context_engine"` origin on `maybeCompactAgentHarnessSession` and the Codex compact bridge. The non-manual skip guard is bypassed for preflight, so Codex actually compacts the thread. A binding change between the initial read and the native request is a stale-binding race, not a benign skip. For `required_preflight` (and the non-manual CLI path) it now surfaces as the canonical recoverable `stale_thread_binding` failure so the queued harness falls back to the context engine instead of treating an uncompacted `ok: true` result as a completed turn. A genuine post-context-engine request may still skip, because the context engine has already compacted. Required-preflight is also the one scoped exception to the model-locked terminal rule: missing or stale Codex thread bindings recover via the shared context-engine fallback while the persisted harness lock stays intact; other locked failures remain terminal. Rebased onto main after #120740 restructured the guarded native compaction block; the recoverable-binding semantics are reintroduced on the new structure and scoped by `nativeCompactionRequest` so #120740's post-context-engine skip behavior is preserved. Closes #119971. * test(evidence): commit inspectable required-preflight live proof scripts for #119971 Adds the two live codex app-server proof scripts (binding-race + locked-preflight) so the redacted terminal traces in the PR body are inspectable on the exact head. Both drive the real codex binary and real maybeCompactCodexAppServerSession with nativeCompactionRequest: "required_preflight"; neither runs in CI (no codex binary). * fix(agents): scope locked-preflight compaction fallback to Codex Restrict the required-preflight model-lock exception to the Codex harness so missing/stale thread bindings in other locked native harnesses (e.g. Copilot) stay terminal instead of escaping the persisted model-lock boundary via context-engine fallback. Add a model-locked Copilot required-preflight regression covering both missing and stale thread bindings. * fix(codex): require native preflight compaction * chore(plugin-sdk): account for native compaction exports * test(codex): use complete cron authority fixtures * chore(lint): shrink compaction assertion baseline * fix(lint): honor root boundary timeout * fix(lint): extend package boundary timeout * fix(plugins): verify native compaction owner --------- Co-authored-by: fuller-stack-dev <263060202+fuller-stack-dev@users.noreply.github.com>
summary, read_when, title
| summary | read_when | title | |||
|---|---|---|---|---|---|
| Repository script entry points and compatibility notes |
|
Scripts Directory |
Scripts Directory
The scripts/ directory contains repository tooling used by local development,
CI, docs publishing, releases, Docker proof, and maintainer operations. Prefer
the package-script entry points in package.json when one exists, then read the
underlying script before running it directly.
Compatibility
Many scripts are stable paths referenced by package.json, GitHub Actions,
docs, and maintainer runbooks. Do not move, rename, or regroup scripts only to
improve taxonomy. A directory migration needs an explicit maintainer-approved
compatibility plan for package scripts, workflows, docs snippets, and any raw
script paths users may have copied.
This index is a discovery aid for the current flat layout. It does not define a new directory taxonomy.
Common Entry Points
| Area | Prefer | Notes |
|---|---|---|
| Build | pnpm build |
Runs scripts/build-all.mts; use specific build scripts only when debugging a build stage. |
| Changed checks | pnpm changed:lanes --json, pnpm check:changed |
Lane classification lives in scripts/changed-lanes.mjs; changed-file checks live in scripts/check-changed.mjs. |
| Docs | pnpm docs:list, pnpm docs:check-mdx, pnpm docs:check-links |
Backed by scripts/docs-list.js, scripts/check-docs-mdx.mjs, and scripts/docs-link-audit.mjs. |
| Formatting docs | pnpm format:docs:check |
Uses scripts/format-docs.mts; use write mode only when intentionally formatting docs. |
| Lint | pnpm lint, pnpm lint:core, pnpm lint:all |
Wrapper scripts keep oxlint behavior aligned with repo config. |
| Targeted tests | pnpm test <path-or-filter> or node scripts/run-vitest.mjs <path-or-filter> |
Avoid bare vitest; it can start watch mode. |
| Changed tests | pnpm test:changed |
Uses the repo's changed-test resolver instead of a broad Vitest run. |
| Docker proof | pnpm test:docker:all, pnpm test:docker:rerun, pnpm test:docker:timings |
Use the planner/rerun helpers before launching broad Docker work. |
| Live proof | pnpm test:live |
Live checks require the matching environment and credentials. |
| Release checks | pnpm release:check, pnpm release:beta, pnpm release:candidate |
Release scripts are maintainer workflows; read release docs before use. |
| GitHub reads | scripts/gh-read |
Uses a GitHub App read token when configured, leaving normal gh login for writes. |
| Commits | git add <files...> then git commit -m "<message>" |
Stage only the intended files for each commit. |
| Remote proof | node scripts/crabbox-wrapper.mjs ... |
Agent default for tests and heavy work; pre-warm by source trust, sync each run, reuse the lease. |
Script Families
check-*.mts/check-*.ts/ retainedcheck-*.mjs: guardrails for architecture, docs, package contents, boundaries, workflows, and generated artifacts.run-*.mjs/run-*.mts: stable wrappers and typed implementations for Node, Vitest, oxlint, tsgo, and environment setup.test-*.mts/ retainedtest-*.mjs/test-*.sh/test-*.ts: test planners, Docker lanes, live checks, and focused validation helpers.docs-*andcheck-docs-*: docs listing, link auditing, MDX checks, spellcheck, sync, and i18n glossary checks.release-*,openclaw-npm-*, andplugin-*-release-*: release preparation, package verification, and publishing helpers.docker-*,test-docker-*, andtest-live-*-docker.sh: Docker E2E planning, rerun, timing, and live/package lane helpers.gh-read*,label-*,sync-labels.ts, and PR helpers: GitHub read, labeling, and maintainer workflow support.generate-*,write-*,copy-*, andsync-*: generated docs, metadata, package surfaces, and build artifact support.lib/: shared helpers imported by script entry points.
Maintenance Rules
- Read
scripts/AGENTS.mdbefore changing scripts. - Keep package scripts, generators, generated-artifact checks, docs references, and workflow references aligned when touching a script path.
- Prefer existing wrappers instead of introducing a raw tool invocation.
- Add or update focused tests under
test/scripts/when changing script behavior.
See also Scripts for public-facing script guidance.