Skip to content

Tighten setup executor ownership and learned setup plans#73

Merged
shinpr merged 6 commits into
mainfrom
follow-up/setup-executor-ownership
May 25, 2026
Merged

Tighten setup executor ownership and learned setup plans#73
shinpr merged 6 commits into
mainfrom
follow-up/setup-executor-ownership

Conversation

@shinpr
Copy link
Copy Markdown
Owner

@shinpr shinpr commented May 25, 2026

Summary

  • Delegate persisted setup plans to the setup executor instead of running them directly in the daemon, so failed setup can be diagnosed and repaired in the same executor context.
  • Persist learned setup plans back to environment.yaml only after the setup executor returns a valid ready result with executable setup commands.
  • Tighten setup-result validation across daemon, Claude guard, schema, and prompts so readiness checks cannot be saved as setup commands.
  • Update Galley skill/profile guidance and plugin metadata for setup.commands[] and setup failure troubleshooting.

Closes #72

Verification

  • go test ./...
  • python3 -m py_compile internal/runner/claude_guard_plugin/scripts/require-final-json.py
  • claude plugin validate plugins/galley
  • claude plugin validate .claude-plugin/marketplace.json
  • git diff --check

@shinpr shinpr merged commit 0b936fb into main May 25, 2026
4 checks passed
@shinpr shinpr deleted the follow-up/setup-executor-ownership branch May 25, 2026 21:42
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.

Mention environment setup commands in Galley skill profile-authoring flow

1 participant