Add pi-supervisor skill

[shanelindsay]

Summary

• add a new pi-supervisor skill under skills/pi-supervisor/SKILL.md
• register the package skill entry in package.json
• provide a self-contained hybrid skill/guide for supervision: when to use it, how goals work, what the supervisor sees, and how SUPERVISOR.md fits in

Detailed breakdown

• skills/pi-supervisor/SKILL.md
  • adds the new skill
  • explains the supervision mental model
  • clarifies goal vs SUPERVISOR.md
  • covers drafting, starting, active-supervision boundaries, and goal-writing patterns
  • clarifies that each supervisor check is a fresh one-shot evaluation with no tools or skills
• package.json
  • registers ./skills so the package exposes the new skill

Testing

• not run (skill/package metadata only)

[Copilot]

Pull request overview

This PR refines the pi-supervisor extension’s user-facing “goal” terminology and improves operational guidance/docs, while adding session-scoped preference persistence so model/sensitivity selections can act as defaults for future supervision runs.

Changes:

• Rename UI/docs language from “outcome” → “goal” in key user-facing surfaces.
• Persist session-scoped supervisor defaults (provider/modelId/sensitivity) separately from active supervision state.
• Add a pi-supervisor skill guide and expand README guidance around goal-writing, supervision behavior, and defaults.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
src/ui/settings-panel.ts	Renames the active “Outcome” settings item label to “Goal”.
src/types.ts	Introduces SupervisorPrefs for session-scoped default settings.
src/state.ts	Adds persistence + restore logic for supervisor-prefs alongside supervisor-state.
src/index.ts	Uses session prefs when resolving defaults; allows setting model/sensitivity even when inactive.
src/engine.ts	Updates prompt wording to “goal” in user prompt; converts idle continue decisions into a steer.
skills/pi-supervisor/SKILL.md	Adds a new skill doc describing how/when to use pi-supervisor and how to write goals.
package.json	Updates repository metadata and registers the skills directory for Pi.
README.md	Updates wording/guidance for goals, settings/status behavior, and idle continue handling.

---

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

[Copilot]

The built-in system prompt still frames the task in terms of a “specific outcome”, but the rest of the extension (UI/README/commands/user prompt) now consistently uses “goal”. Since this text is part of what the supervisor model reads, keeping terminology consistent will reduce ambiguity—consider replacing “outcome” with “goal” throughout the built-in prompt (including the done/steering bullet points later in the prompt).

[Copilot]

loadFromSession now restores both the most recent supervisor-state and supervisor-prefs entries, but the doc comment still says it only finds the most recent supervisor-state entry. Updating the comment to reflect the new behavior will help future maintainers understand the persistence model.

Suggested change

	/** Restore state from session entries (finds the most recent supervisor-state entry). */
	/** Restore the most recent supervisor-state and supervisor-prefs session entries, falling back to prefs derived from state if needed. */

[Copilot]

This description implies that when the agent is idle the supervisor might return continue, but the runtime now converts idle continue decisions into a steer to avoid stalling (and the built-in prompt says idle must be done/steer). Please reword this bullet to reflect that continue is only applicable while the agent is actively working (mid-turn).

Suggested change

	- it can either let the agent continue, inject a steer, or declare the goal
	done when the agent is idle
	- while the agent is actively working (mid-turn), it can let the agent
	continue; when the agent is idle, it can inject a steer or declare the
	goal done

[Copilot]

After switching the repository/homepage URLs to shanelindsay/pi-supervisor, the pi.video and pi.image links still point at tintinweb/pi-supervisor. If the repo move is intentional, these asset URLs should be updated as well (or switched to relative paths) to avoid broken media in package metadata.

Suggested change

	"video": "https://github.com/tintinweb/pi-supervisor/raw/master/media/demo.mp4",
	"image": "https://github.com/tintinweb/pi-supervisor/raw/master/media/screenshot.png"
	"video": "https://github.com/shanelindsay/pi-supervisor/raw/master/media/demo.mp4",
	"image": "https://github.com/shanelindsay/pi-supervisor/raw/master/media/screenshot.png"

[Copilot]

The repository/homepage/bugs URLs were updated to shanelindsay/pi-supervisor, but related package metadata (e.g. the top-level description currently mentioning “outcome”) is now inconsistent with the repo/docs “goal” terminology. Consider updating the remaining metadata fields for consistency so npm/pi users see the same phrasing everywhere.