Skip to content

fix #84857: skip CLI runtime harness preflight during compaction#85862

Merged
clawsweeper[bot] merged 3 commits into
mainfrom
clawsweeper/automerge-openclaw-openclaw-84878
May 24, 2026
Merged

fix #84857: skip CLI runtime harness preflight during compaction#85862
clawsweeper[bot] merged 3 commits into
mainfrom
clawsweeper/automerge-openclaw-openclaw-84878

Conversation

@clawsweeper

@clawsweeper clawsweeper Bot commented May 23, 2026

Copy link
Copy Markdown
Contributor

Makes #84878 merge-ready for the ClawSweeper automerge loop.
The edit pass should inspect the live PR diff, review comments, and failing checks; rebase if needed; keep the contributor branch credited; and stop only when validation is green or an external blocker is proven.

ClawSweeper 🐠 replacement reef notes:

  • Repair fallback: GitHub rejected the repair branch push because it updates workflow files and the ClawSweeper app token does not have workflows permission

Inherited issue-closing references from the source PR:
Fixes #84857

Co-author credit kept:

fish notes: model gpt-5.5, reasoning high; reviewed against de553c3.

@clawsweeper clawsweeper Bot added agents Agent runtime and tooling size: XS clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge proof: supplied External PR includes structured after-fix real behavior proof. proof: sufficient ClawSweeper judged the real behavior proof convincing. P1 High-priority user-facing bug, regression, or broken workflow. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. clawsweeper Tracked by ClawSweeper automation labels May 23, 2026
@clawsweeper clawsweeper Bot mentioned this pull request May 23, 2026
Closed
@clawsweeper

clawsweeper Bot commented May 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Codex review: passed.

Latest ClawSweeper review: 2026-05-23 23:56 UTC / May 23, 2026, 7:56 PM ET.

Workflow note: Future ClawSweeper reviews update this same comment in place.

How this review workflow works
  • ClawSweeper keeps one durable marker-backed review comment per issue or PR.
  • Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
  • A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
  • PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
  • Maintainers can also comment @clawsweeper review to request a fresh review only.
  • Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
  • Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
  • Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

PR Surface
Source +12, Tests +33, Docs +1. Total +46 across 6 files.

View PR surface stats
Area Files Added Removed Net
Source 2 14 2 +12
Tests 3 36 3 +33
Docs 1 1 0 +1
Config 0 0 0 0
Generated 0 0 0 0
Other 0 0 0 0
Total 6 51 5 +46

Summary
The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.

Reproducibility: yes. at source level. Current main still routes provider-owned claude-cli runtime compaction preflight through harness selection, where claude-cli is not a registered embedded harness.

PR rating
Overall: 🐚 platinum hermit
Proof: 🐚 platinum hermit
Patch quality: 🐚 platinum hermit
Summary: Normal good bug-fix PR with focused code, regression coverage, inherited terminal proof, and no blocking review findings.

Rank-up moves:

  • none
What the crustacean ranks mean
  • 🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
  • 🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
  • 🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
  • 🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
  • 🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
  • 🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
  • 🌊 off-meta tidepool: rating does not apply to this item.

Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

Real behavior proof
Sufficient (terminal): Sufficient for this bot replacement PR: the source PR includes terminal after-fix proof, and this branch records exact-head validation in the ClawSweeper repair comment.

Next step before merge
No repair lane is needed; this automerge-armed PR should proceed through the exact-head checks and merge gate.

Security
Cleared: The diff changes runtime selection logic, tests, and changelog only; it does not alter dependencies, workflows, secrets, permissions, or package resolution.

Review details

Best possible solution:

Merge the targeted preflight bypass after exact-head checks, and keep broader CLI compaction behavior as a separate follow-up if a runtime API gap appears.

Do we have a high-confidence way to reproduce the issue?

Yes at source level. Current main still routes provider-owned claude-cli runtime compaction preflight through harness selection, where claude-cli is not a registered embedded harness.

Is this the best way to solve the issue?

Yes. The PR keeps the fix at the preflight boundary, returns to the existing context-engine compaction path, and avoids changing normal CLI dispatch.

Label changes:

  • add rating: 🐚 platinum hermit: Current PR rating is 🐚 platinum hermit because proof is 🐚 platinum hermit, patch quality is 🐚 platinum hermit, and Normal good bug-fix PR with focused code, regression coverage, inherited terminal proof, and no blocking review findings.
  • remove rating: 🦪 silver shellfish: Current PR rating is rating: 🐚 platinum hermit, so this older rating label is no longer current.

Label justifications:

  • P1: The PR fixes an urgent agent-runtime regression that can block over-threshold CLI-backed sessions before dispatch and surface the harness error to users.
  • rating: 🐚 platinum hermit: Current PR rating is 🐚 platinum hermit because proof is 🐚 platinum hermit, patch quality is 🐚 platinum hermit, and Normal good bug-fix PR with focused code, regression coverage, inherited terminal proof, and no blocking review findings.
  • status: 🚀 automerge armed: This PR is in ClawSweeper's automerge lane. Sufficient (terminal): Sufficient for this bot replacement PR: the source PR includes terminal after-fix proof, and this branch records exact-head validation in the ClawSweeper repair comment.
  • proof: sufficient: Contributor real behavior proof is sufficient. Sufficient for this bot replacement PR: the source PR includes terminal after-fix proof, and this branch records exact-head validation in the ClawSweeper repair comment.

What I checked:

  • Current main failure path: Current main still enters selectAgentHarness from maybeCompactAgentHarnessSession; for a provider-owned claude-cli runtime config, the current selector has no CLI-provider-id bypass before harness lookup. (src/agents/harness/selection.ts:459, 1e2e6147480c)
  • CLI runtime contract: The docs distinguish CLI backends from embedded harnesses and state that claude-cli is not an embedded harness id that should be passed to AgentHarness selection. Public docs: docs/concepts/agent-runtimes.md. (docs/concepts/agent-runtimes.md:32, 1e2e6147480c)
  • PR implementation: The PR diff adds an early isCliRuntimeProvider return and a resolved-runtime alias guard before harness selection in maybeCompactAgentHarnessSession. (src/agents/harness/selection.ts:459, 1dd8a88d218d)
  • Regression coverage: The PR adds regression tests for both model-scoped claude-cli runtime sessions and provider-owned claude-cli sessions resolving without a thrown harness error. (src/agents/harness/selection.test.ts:639, 1dd8a88d218d)
  • Fallback behavior: The queued compaction caller only returns early when a harness result is present, so undefined from the bypass continues into the regular context-engine compaction path. (src/agents/pi-embedded-runner/compact.queued.ts:129, 1e2e6147480c)
  • Proof and repair validation: The source PR provided terminal after-fix proof for the helper path, and the ClawSweeper repair comment records exact-head validation with pnpm check:changed, the focused selection test, lint, and test types. (1dd8a88d218d)

Likely related people:

  • steipete: Peter Steinberger introduced and evolved the pluggable harness selection behavior that this PR adjusts around CLI runtime aliases. (role: feature-history owner; confidence: high; commits: 44ec4d05de4a, 2d80bbc43de0, 89d7a24a3523; files: src/agents/harness/selection.ts, src/agents/model-runtime-aliases.ts, src/agents/command/attempt-execution.ts)
  • vincentkoc: Vincent Koc split the queued compaction wrapper that calls the harness preflight path affected by this bug. (role: adjacent compaction contributor; confidence: high; commits: f1c4e2f11daf; files: src/agents/pi-embedded-runner/compact.queued.ts)
  • Gio Della-Libera: Recent history shows Gio Della-Libera working near pre-prompt compaction decisions that make this preflight path observable. (role: recent compaction area contributor; confidence: medium; commits: 79be9401306f; files: src/agents/pi-embedded-runner/run/preemptive-compaction.ts)

Codex review notes: model gpt-5.5, reasoning high; reviewed against 1e2e6147480c.

@openclaw-barnacle openclaw-barnacle Bot removed the proof: supplied External PR includes structured after-fix real behavior proof. label May 23, 2026
@clawsweeper

clawsweeper Bot commented May 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

🦞✅
ClawSweeper merged this PR after the passing review.

Source: clawsweeper[bot]
Feedback: structured ClawSweeper verdict: pass (sha=1dd8a88d218d464a4dbc191afbb0acdbd8f1cf02)
Merge status: merged by ClawSweeper automerge
Merged at: 2026-05-24T00:00:56Z
Merge commit: fa39bef3893d

What merged:

  • The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
  • Reproducibility: yes. at source level. Current main still routes provider-owned claude-cli runtime compaction preflight through harness selection, where claude-cli is not a registered embedded harness.

Automerge notes:

The automerge loop is complete.

Automerge progress:

  • 2026-05-23 23:30:14 UTC review queued de553c3ccc59 (queued)
  • 2026-05-23 23:50:05 UTC review queued 1dd8a88d218d (after repair)
  • 2026-05-24 00:00:39 UTC review passed 1dd8a88d218d (structured ClawSweeper verdict: pass (sha=1dd8a88d218d464a4dbc191afbb0acdbd8f1c...)
  • 2026-05-24 00:00:58 UTC merged 1dd8a88d218d (merged by ClawSweeper automerge)

@clawsweeper clawsweeper Bot added rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. status: 🚀 automerge armed This PR is in ClawSweeper's automerge lane. and removed rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels May 23, 2026
@clawsweeper clawsweeper Bot force-pushed the clawsweeper/automerge-openclaw-openclaw-84878 branch from de553c3 to 1dd8a88 Compare May 23, 2026 23:50
@clawsweeper clawsweeper Bot added rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. and removed rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. labels May 24, 2026
@clawsweeper

clawsweeper Bot commented May 24, 2026

Copy link
Copy Markdown
Contributor Author

ClawSweeper PR egg

✨ Hatched: 🥚 common Cosmic Proofling

Hatch command

Comment @clawsweeper hatch when this PR is hatchable.

Hatchability rules:

  • Merged PRs are hatchable.
  • Open PRs are hatchable when they are status: 👀 ready for maintainer look, status: 🚀 automerge armed, or labeled clawsweeper:automerge.
  • Closed unmerged PRs are hatchable only when one of those hatchable labels is still present in the durable record.

Rarity: 🥚 common.
Trait: guards the happy path.
Image traits: location review cove; accessory miniature diff map; palette rose quartz and slate; mood mischievous; pose guarding a tiny green check; shell translucent glimmer shell; lighting warm desk-lamp glow; background soft code-shaped tiles.
Share on X: post this hatch
Copy: My PR egg hatched a 🥚 common Cosmic Proofling in ClawSweeper.

What is this egg doing here?
  • Eggs appear after the PR passes real-behavior proof. It is here for vibes, not verdicts: it does not change labels, ratings, merge decisions, or automation.
  • The shell reacts to review momentum: open follow-up work warms it up, re-review makes it wobble, and a clean final review lets it hatch.
  • Hatchability usually comes from sufficient real-behavior proof, no blocking P0/P1/P2 findings, no security attention needed, and clean correctness. A merged PR is already final, so merge makes the egg hatchable independently.
  • The hatch is seeded from this repository and PR number, so the same PR keeps the same creature; the reviewed head SHA can only change safe visual details.
  • Rarity is just collectible sparkle: 🥚 common, 🌱 uncommon, 💎 rare, ✨ glimmer, and 🌈 legendary.

@clawsweeper clawsweeper Bot merged commit fa39bef into main May 24, 2026
126 of 131 checks passed
@clawsweeper clawsweeper Bot deleted the clawsweeper/automerge-openclaw-openclaw-84878 branch May 24, 2026 00:00
@github-actions github-actions Bot mentioned this pull request May 24, 2026
Open
frankhli843 added a commit to gemmaclaw/gemmaclaw that referenced this pull request May 24, 2026
@frankhli843 @RomneyDa
@clawsweeper @zhangguiping-xydt @Takhoffman @jason-allen-oneal @steipete @itsuzef
fix(upstream-sync): selective cherry-picks from openclaw 58e9628..fa3…
2e56407 
…9bef (#232)

* perf(whatsapp): narrow runtime setter entry (openclaw#85589)

* fix openclaw#84857: skip CLI runtime harness preflight during compaction (openclaw#85862)

Summary:
- The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
- Reproducibility: yes. at source level. Current main still routes provider-owned `claude-cli` runtime compaction preflight through harness selection, where `claude-cli` is not a registered embedded harness.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix openclaw#84857: skip CLI runtime harness preflight during compaction
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8487…

Validation:
- ClawSweeper review passed for head 1dd8a88.
- Required merge gates passed before the squash merge.

Prepared head SHA: 1dd8a88
Review: openclaw#85862 (comment)

Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
# Conflicts:
#	CHANGELOG.md
#	extensions/meeting-notes/index.test.ts
#	extensions/meeting-notes/src/tool.ts
#	src/agents/harness/selection.test.ts
#	src/agents/harness/selection.ts
#	src/gateway/server-startup-post-attach.test.ts

* fix(sandbox): mount workspace skills read-only

* fix(sandbox): block bridge writes to workspace skills
# Conflicts:
#	src/agents/sandbox/fs-paths.ts

* fix(sandbox): block remote bridge writes to skills

* fix(sandbox): filter remote skill mounts by existing roots

* fix: harden sandbox skill mounts (openclaw#85591)

# Conflicts:
#	src/agents/sandbox/workspace-skills-bridge-readonly.test.ts

* fix: guard remote sandbox skill roots (openclaw#85591)

* fix: close remote sandbox skill symlink aliases (openclaw#85591)

* fix: preserve sandbox skill overlay precedence (openclaw#85591)

# Conflicts:
#	src/agents/sandbox/browser.create.test.ts
#	src/agents/sandbox/docker.config-hash-recreate.test.ts

* fix(backup): dereference archive hardlinks

# Conflicts:
#	src/commands/backup-verify.test.ts
#	src/infra/backup-create.test.ts
#	src/infra/backup-create.ts

* fix(backup): reject missing hardlink targets

* fix(whatsapp): serialize Error in auto-reply delivery log (openclaw#85777)

The auto-reply "delivery failed" log path passes a raw Error
under the `err` field. tslog's default JSON serialization
renders bare Error instances as `{}` because Error own data
properties are non-enumerable. Every delivery failure in
production therefore logs `err: {}`, forcing operators to
guess the underlying Baileys error from timestamp alone.

Convert Error to `{ type, message, stack }` plus own-enumerable
properties at the log site, so Boom-style subclass diagnostics
(output.statusCode, data) and custom OutboundDeliveryError
fields (stage, results) survive. Non-Error rejection values
pass through unchanged.

Tests cover Error, Error subclass (Boom-style), string
rejection, and object rejection paths.

AI-assisted: Claude Code (Opus 4.7) authored, codex review
locally addressed.
# Conflicts:
#	extensions/whatsapp/src/auto-reply/monitor/inbound-dispatch.test.ts
#	extensions/whatsapp/src/auto-reply/monitor/inbound-dispatch.ts

* chore: add model-runtime-aliases.ts from upstream (needed by selection.ts)

* chore: add model-runtime-policy.ts from upstream (needed by model-runtime-aliases.ts)

* fix: correct backup-create.ts writeTarArchiveWithRetry structure (cherry-pick conflict resolution artifact)

* fix: add missing upstream types and helpers for build compatibility

- Add AgentRuntimePolicyConfig to types.agents-shared.ts
- Add agentRuntime field to AgentModelEntryConfig, ModelDefinitionConfig, ModelProviderConfig
- Add models field to AgentConfig
- Add relativePathEscapesContainerRoot to sandbox/path-utils.ts
- Add backup-volatile-filter.ts from upstream (isVolatileBackupPath)
- Add writeTarArchiveWithRetry, buildExtensionsNodeModulesFilter to backup-create.ts
- Fix resolveConfiguredAgentHarnessPolicy -> resolveAgentHarnessPolicy in selection.ts

* fix: revert upstream-only harness restructuring and apply minimal c8b75a7 patch

The previous cherry-pick of c8b75a7 (and fa39bef duplicate) introduced
upstream-only files (v2.ts, policy.ts, result-classification.ts, etc.) that
depend on types not present in gemmaclaw, and corrupted several test files.

This commit reverts all wrongly-introduced upstream-only files to their
gemmaclaw branch-point state and applies only the actual c8b75a7 change:
- Add isCliRuntimeProvider/isCliRuntimeAliasForProvider guards to
  maybeCompactAgentHarnessSession to skip compaction preflight for CLI runtimes
- Add minimal gemmaclaw-compatible tests for the new skip behavior
- Remove dead code introduced by the rollback (staging-based archive helpers,
  unused imports, unreachable functions)
- Delete upstream-only meeting-notes extension files not in gemmaclaw
- Restore all test files to their gemmaclaw-compatible versions

* fix(backup): skip unreadable files in tar filter (EACCES from Docker root-owned files)

Docker git clones create root-owned .promisor files that are unreadable on the
host. Add accessSync R_OK check in tarFilter to skip such files gracefully.

Also fix sandbox tests to use real temp directories instead of hardcoded
/tmp/workspace paths for more robust test isolation.

* fix(whatsapp): wire onError into dispatcherOptions for delivery failure logging

The cherry-pick of f0ec730 added logWhatsAppReplyDeliveryError and
normalizeErrForLog but missed wiring onError in dispatcherOptions because
the upstream base already had the onError call site (added in a prior
upstream commit not in our range). Add the missing onError handler so
delivery failures are logged with serialized Error objects.

* fix(whatsapp): restore helper types and functions stripped by revert commit

Commit 0696195 was overly aggressive: it reverted upstream-only harness
files but also stripped ReplyDeliveryInfo, normalizeErrForLog, and
logWhatsAppReplyDeliveryError that were legitimately added by our
WhatsApp cherry-pick (2688cf4). Restore them so onError can call
logWhatsAppReplyDeliveryError correctly.

* fix: add whatsapp runtime-setter-api.js to bundled sidecar paths baseline

extensions/whatsapp/runtime-setter-api.ts was added by cherry-pick but
the generated baseline JSON was not updated. Run runtime-sidecars:gen to
add dist/extensions/whatsapp/runtime-setter-api.js.

---------

Co-authored-by: Dallin Romney <dallinromney@gmail.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
Co-authored-by: Jason O'Neal <jason.allen.oneal@gmail.com>
Co-authored-by: Peter Steinberger <steipete@gmail.com>
Co-authored-by: Youssef Hemimy <53057646+itsuzef@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @zhangguiping-xydt
@Takhoffman @SebTardif
fix openclaw#84857: skip CLI runtime harness preflight during compact…
601749b 
…ion (openclaw#85862)

Summary:
- The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
- Reproducibility: yes. at source level. Current main still routes provider-owned `claude-cli` runtime compaction preflight through harness selection, where `claude-cli` is not a registered embedded harness.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix openclaw#84857: skip CLI runtime harness preflight during compaction
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8487…

Validation:
- ClawSweeper review passed for head 1dd8a88.
- Required merge gates passed before the squash merge.

Prepared head SHA: 1dd8a88
Review: openclaw#85862 (comment)

Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @zhangguiping-xydt
@Takhoffman @SebTardif
fix openclaw#84857: skip CLI runtime harness preflight during compact…
f225b79 
…ion (openclaw#85862)

Summary:
- The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
- Reproducibility: yes. at source level. Current main still routes provider-owned `claude-cli` runtime compaction preflight through harness selection, where `claude-cli` is not a registered embedded harness.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix openclaw#84857: skip CLI runtime harness preflight during compaction
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8487…

Validation:
- ClawSweeper review passed for head 1dd8a88.
- Required merge gates passed before the squash merge.

Prepared head SHA: 1dd8a88
Review: openclaw#85862 (comment)

Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @zhangguiping-xydt
@Takhoffman @SebTardif
fix openclaw#84857: skip CLI runtime harness preflight during compact…
41ba3dd 
…ion (openclaw#85862)

Summary:
- The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
- Reproducibility: yes. at source level. Current main still routes provider-owned `claude-cli` runtime compaction preflight through harness selection, where `claude-cli` is not a registered embedded harness.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix openclaw#84857: skip CLI runtime harness preflight during compaction
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8487…

Validation:
- ClawSweeper review passed for head 1dd8a88.
- Required merge gates passed before the squash merge.

Prepared head SHA: 1dd8a88
Review: openclaw#85862 (comment)

Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 24, 2026
@clawsweeper @zhangguiping-xydt @Takhoffman
fix openclaw#84857: skip CLI runtime harness preflight during compact…
1d2ef65 
…ion (openclaw#85862)

Summary:
- The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
- Reproducibility: yes. at source level. Current main still routes provider-owned `claude-cli` runtime compaction preflight through harness selection, where `claude-cli` is not a registered embedded harness.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix openclaw#84857: skip CLI runtime harness preflight during compaction
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8487…

Validation:
- ClawSweeper review passed for head 1dd8a88.
- Required merge gates passed before the squash merge.

Prepared head SHA: 1dd8a88
Review: openclaw#85862 (comment)

Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @zhangguiping-xydt
@Takhoffman @SebTardif
fix openclaw#84857: skip CLI runtime harness preflight during compact…
845028b 
…ion (openclaw#85862)

Summary:
- The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
- Reproducibility: yes. at source level. Current main still routes provider-owned `claude-cli` runtime compaction preflight through harness selection, where `claude-cli` is not a registered embedded harness.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix openclaw#84857: skip CLI runtime harness preflight during compaction
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8487…

Validation:
- ClawSweeper review passed for head 1dd8a88.
- Required merge gates passed before the squash merge.

Prepared head SHA: 1dd8a88
Review: openclaw#85862 (comment)

Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @zhangguiping-xydt
@Takhoffman @SebTardif
fix openclaw#84857: skip CLI runtime harness preflight during compact…
1e4eef2 
…ion (openclaw#85862)

Summary:
- The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
- Reproducibility: yes. at source level. Current main still routes provider-owned `claude-cli` runtime compaction preflight through harness selection, where `claude-cli` is not a registered embedded harness.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix openclaw#84857: skip CLI runtime harness preflight during compaction
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8487…

Validation:
- ClawSweeper review passed for head 1dd8a88.
- Required merge gates passed before the squash merge.

Prepared head SHA: 1dd8a88
Review: openclaw#85862 (comment)

Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @zhangguiping-xydt
@Takhoffman @SebTardif
fix openclaw#84857: skip CLI runtime harness preflight during compact…
a9be320 
…ion (openclaw#85862)

Summary:
- The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
- Reproducibility: yes. at source level. Current main still routes provider-owned `claude-cli` runtime compaction preflight through harness selection, where `claude-cli` is not a registered embedded harness.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix openclaw#84857: skip CLI runtime harness preflight during compaction
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8487…

Validation:
- ClawSweeper review passed for head 1dd8a88.
- Required merge gates passed before the squash merge.

Prepared head SHA: 1dd8a88
Review: openclaw#85862 (comment)

Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
jameslcowan pushed a commit to jameslcowan/openclaw that referenced this pull request Jun 2, 2026
@clawsweeper @zhangguiping-xydt @Takhoffman
fix openclaw#84857: skip CLI runtime harness preflight during compact…
0261550 
…ion (openclaw#85862)

Summary:
- The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
- Reproducibility: yes. at source level. Current main still routes provider-owned `claude-cli` runtime compaction preflight through harness selection, where `claude-cli` is not a registered embedded harness.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix openclaw#84857: skip CLI runtime harness preflight during compaction
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8487…

Validation:
- ClawSweeper review passed for head 1dd8a88.
- Required merge gates passed before the squash merge.

Prepared head SHA: 1dd8a88
Review: openclaw#85862 (comment)

Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SYU8384 pushed a commit to SYU8384/openclaw that referenced this pull request Jun 3, 2026
@clawsweeper @zhangguiping-xydt @Takhoffman
fix openclaw#84857: skip CLI runtime harness preflight during compact…
e70717e 
…ion (openclaw#85862)

Summary:
- The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
- Reproducibility: yes. at source level. Current main still routes provider-owned `claude-cli` runtime compaction preflight through harness selection, where `claude-cli` is not a registered embedded harness.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix openclaw#84857: skip CLI runtime harness preflight during compaction
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8487…

Validation:
- ClawSweeper review passed for head 1dd8a88.
- Required merge gates passed before the squash merge.

Prepared head SHA: 1dd8a88
Review: openclaw#85862 (comment)

Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
@clawsweeper @zhangguiping-xydt @Takhoffman
fix openclaw#84857: skip CLI runtime harness preflight during compact…
7d2572f 
…ion (openclaw#85862)

Summary:
- The PR skips agent-harness compaction preflight for provider-owned or configured CLI runtime sessions, adds claude-cli regression coverage, includes a changelog entry, and applies small test/type cleanups.
- Reproducibility: yes. at source level. Current main still routes provider-owned `claude-cli` runtime compaction preflight through harness selection, where `claude-cli` is not a registered embedded harness.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix openclaw#84857: skip CLI runtime harness preflight during compaction
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8487…

Validation:
- ClawSweeper review passed for head 1dd8a88.
- Required merge gates passed before the squash merge.

Prepared head SHA: 1dd8a88
Review: openclaw#85862 (comment)

Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

agents Agent runtime and tooling clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge clawsweeper Tracked by ClawSweeper automation gateway Gateway runtime P1 High-priority user-facing bug, regression, or broken workflow. plugin: meeting-notes proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. size: S status: 🚀 automerge armed This PR is in ClawSweeper's automerge lane.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug] Compaction preflight throws MissingAgentHarnessError("claude-cli") for claude-cli runtime sessions over contextThreshold

1 participant

@zhangguiping-xydt