Skip to content

Remove Reflection's broad safety snapshot - #35

Closed
hamzamerzic wants to merge 1 commit into
mobius-os:mainfrom
hamzamerzic:fix/reflection-scoped-commits
Closed

Remove Reflection's broad safety snapshot#35
hamzamerzic wants to merge 1 commit into
mobius-os:mainfrom
hamzamerzic:fix/reflection-scoped-commits

Conversation

@hamzamerzic

Copy link
Copy Markdown
Contributor

Summary

  • remove the pre-run broad safety snapshot from the app-owned runner
  • replace sweeping guidance with revision-anchored, exact-path commits
  • cover both the wrapper and runner against a return to broad unattended commits

Why

mobius-os/mobius#335 retired the same broad snapshot in the former platform-owned Reflection runner because it could commit unrelated concurrent work. app-reflection#20 then made this repository's runner and operating contract canonical, but that app-owned copy still carried the old snapshot and sweeping guidance.

This follow-up applies the established ownership contract at the current source of truth: each change records its starting revision and commits only its exact paths. The runner no longer creates a broad best-effort commit before the agent starts.

Verification

  • npm test — 37 JavaScript checks and 61 Python checks passed
  • package validation — 0 errors, 0 warnings
  • git diff --check

Co-authored-by: Möbius Agent <mobius-agent@users.noreply.github.com>
@hamzamerzic hamzamerzic added the bug Something isn't working label Aug 3, 2026
@hamzamerzic

Copy link
Copy Markdown
Contributor Author

Superseded by #33, which carries this exact-path runner fix together with the dependent terminal-handoff contract and passed the full app suite.

@hamzamerzic hamzamerzic closed this Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant