Skip to content

Preserve scheduler verification context in v0.4.1 - #10

Merged
amittell merged 2 commits into
mainfrom
codex/verification-context-v0.4.1
Jul 13, 2026
Merged

Preserve scheduler verification context in v0.4.1#10
amittell merged 2 commits into
mainfrom
codex/verification-context-v0.4.1

Conversation

@amittell

Copy link
Copy Markdown
Owner

Summary

  • run scheduler-projected post-success verification from the shell task working directory
  • retain runtime-provided execution environment without persisting environment values
  • reject expanded verifier commands that exceed the scheduler storage limit
  • make the integration assertion compatible with handoff v2 and v3
  • bump the package to 0.4.1

Validation

  • npm run lint
  • npm test (786 passed)
  • npm pack --dry-run
  • packed 0.4.1 artifact passed the openclaw-scheduler compile, apply, approval, restart, execute, verify, and inspect black box

Copilot AI review requested due to automatic review settings July 13, 2026 05:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the openclaw-scheduler compile target so post-success verification for shell tasks runs from the task’s declared shell.cwd while relying on the scheduler/runtime-provided execution environment (without persisting env values), adds coverage for the new behavior/limits, and bumps the package version to 0.4.1.

Changes:

  • Wrap scheduler verify_shell with a cd <task cwd> && … prelude for shell tasks (using existing POSIX-safe rendering) and enforce the scheduler’s verify_shell storage limit.
  • Add conformance + integration regressions covering quoted CWD paths, runtime environment retention, verifier-length rejection, and handoff v2/v3-compatible field-version assertions.
  • Bump version to 0.4.1 and document the behavior in docs + changelog.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
test/scheduler-conformance.test.js Adds conformance tests for verify CWD binding, runtime env retention, and verify-shell length validation.
test/integration-scheduler.test.js Updates the handoff assertion to be compatible with negotiated handoff v2 and v3.
src/compiler/openclaw-scheduler.js Implements verify-shell CWD wrapping for shell tasks and adds verify_shell to scheduler string-length validation.
package.json Bumps package version to 0.4.1.
package-lock.json Updates lockfile version metadata to 0.4.1.
docs/execution-identity.md Documents scheduler verify behavior (CWD binding + runtime env retention constraints).
CHANGELOG.md Adds 0.4.1 release notes describing scheduler verify changes and added regressions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/compiler/openclaw-scheduler.js Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 6 out of 7 changed files in this pull request and generated no new comments.

@amittell
amittell merged commit f1fed6d into main Jul 13, 2026
4 checks passed
@amittell
amittell deleted the codex/verification-context-v0.4.1 branch July 13, 2026 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants