Skip to content

docs(skill): VM gate-sweep exact procedure + IS_SANDBOX gotcha - #637

Merged
100yenadmin merged 1 commit into
mainfrom
docs/skillify-vm-sweep
Jun 3, 2026
Merged

docs(skill): VM gate-sweep exact procedure + IS_SANDBOX gotcha#637
100yenadmin merged 1 commit into
mainfrom
docs/skillify-vm-sweep

Conversation

@100yenadmin

@100yenadmin 100yenadmin commented Jun 3, 2026

Copy link
Copy Markdown
Member

Skillifies the heavy VM gate-sweep so we stop re-deriving it each session (owner directive 2026-06-03).

worldos-dev skill gains a ## VM GATE SWEEP — exact procedure section: the VM coordinates, the mandatory IS_SANDBOX=1 (the real root+bypassPermissions blocker — claude -p --dangerously-skip-permissions is refused as root → silent "player produced no intro" abort; sweep_v2.sh sets it, standalone run_duo.sh needs it), the art path (content/worlds/_private/baldurs-gate/images, not top-level _private), the copy-pasteable scout→ff→smoke→sweep_v2.sh→read sequence, the per-gate mapping (score-*.json → G1/G2/G3; duo lenses → G5; ui_audit → G4), the honest-score/derived-sat guard, and the RRI rollup rule.

Runbook gets the same IS_SANDBOX gotcha + a pointer to the skill + a 2026-06-03 status update superseding the stale 2026-06-01 scout note (the VM is verified ready: git-fetch works now, claude authed, art present).

Docs-only. Memory note project_worldos_vm_sweep_lane.md already updated in parallel.

Summary by CodeRabbit

  • Documentation
    • Updated internal developer documentation for VM testing procedures and configuration workflows.

…re-deriving it)

The worldos-dev skill + GUI runbook had the VM-lane WARNING but not the actionable HOW — so each
session re-derived IS_SANDBOX=1 (the real root+bypassPermissions blocker that silently aborts every
claude-lane run as "player produced no intro"), the sweep_v2.sh one-command invocation, and the art
path (content/worlds/_private/baldurs-gate/images, NOT top-level _private).

Add "VM GATE SWEEP — exact procedure" to worldos-dev (scout → ff → smoke → sweep_v2 → read results →
RRI rollup, with the per-gate mapping + the honest-score/derived-sat guard), and close the same gap in
WorldOS-GUI-RUNBOOK.md + supersede the stale 2026-06-01 scout note (the VM is verified READY 2026-06-03:
git-fetch works, claude authed, art present). Owner directive: testing process must live in skills,
updated as the harness changes, so a cold post-compaction agent never re-learns it.
@coderabbitai

coderabbitai Bot commented Jun 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d5ecf1ad-a788-46f3-bab8-553cbc5b0ce6

📥 Commits

Reviewing files that changed from the base of the PR and between fe78cc2 and 109133e.

📒 Files selected for processing (2)
  • .claude/skills/worldos-dev/SKILL.md
  • WorldOS-GUI-RUNBOOK.md

📝 Walkthrough

Walkthrough

This PR adds authoritative documentation for the evaos-support VM gate sweep procedure across two files: a complete technical reference in the WorldOS dev skill doc with command steps, result parsing, and RRI integration; and a runbook update with VM-lane requirements and current readiness status confirming the workflow is executable.

Changes

VM Gate Sweep Procedures

Layer / File(s) Summary
Authoritative VM gate sweep procedure
.claude/skills/worldos-dev/SKILL.md
Complete heavy-part-B VM gate sweep procedure documented: mandatory IS_SANDBOX=1, specific SSH/repo/harness paths, scout/FF/smoke/full sweep command sequences, score-*.json result parsing and gate-to-log mappings, score thresholds, and same-SHA rollup/RRI steps using macOS app handoff plus qa/release_readiness.py.
Runbook integration and VM readiness status
WorldOS-GUI-RUNBOOK.md
CLAUDE VM sweep lane requirements clarified: IS_SANDBOX=1 mandate, root/sandbox behavior with --permission-mode, art directory path, and external procedure consistency flag. VM readiness updated to 2026-06-03: git-fetch resolved, claude/codex CLI present and authenticated, disk and art assets available, sweep_v2.sh operational.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • electricsheephq/WorldOS#471: Both PRs update WorldOS-GUI-RUNBOOK.md's "support VM / heavy sweep" instructions—this PR expands it with the evaos-support gate-sweep one-command procedure building directly on the VM-lane routing guidance introduced in #471.
  • electricsheephq/WorldOS#578: Both PRs update .claude/skills/worldos-dev/SKILL.md with support/VM "heavy sweep" lane guidance; these PRs overlap at the same doc section level for WorldOS gate/sweep procedures.
  • electricsheephq/WorldOS#509: Both PRs document the QA/app handoff + RRI release-gating workflow and reference shared tooling (qa/release_readiness.py); this PR's sweep/run guidance is part of the same documented gate truth.

Poem

🐰 The VM now gates with certainty,
One sandbox command, sweep so pure,
Score-files tell the truth,
RRI rolls with macOS in hand—
Heavy part-B, ready to run! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main documentation change: adding the exact VM gate-sweep procedure and highlighting the critical IS_SANDBOX gotcha.
Description check ✅ Passed The description is detailed and complete. It covers what changed (two files updated with VM sweep procedure), why (owner directive to skillify the procedure), key technical details (IS_SANDBOX requirement, art path, mappings, RRI rollup), and the validation context (docs-only, parallel memory note already updated).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@100yenadmin
100yenadmin merged commit 3c6c59d into main Jun 3, 2026
14 checks passed
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.

1 participant