Skip to content

Align pi-supervisor status, defaults, and docs#2

Open
shanelindsay wants to merge 1 commit intotintinweb:masterfrom
shanelindsay:docs/pi-supervisor-runtime-alignment
Open

Align pi-supervisor status, defaults, and docs#2
shanelindsay wants to merge 1 commit intotintinweb:masterfrom
shanelindsay:docs/pi-supervisor-runtime-alignment

Conversation

@shanelindsay
Copy link
Copy Markdown

Summary

  • align README and runtime behaviour around goal-focused supervision
  • make inactive model and sensitivity changes persist as defaults for the next /supervise
  • clarify status/settings handling and user-facing goal terminology
  • prevent idle supervisor continue responses from leaving the run stalled

Detailed breakdown

  • README.md
    • updates command wording and settings text to match current behaviour
    • clarifies the supervisor's limited context and no-tools model
    • documents goal drafting guidance and the inactive-defaults behaviour
  • src/index.ts
    • aligns user-facing copy around goal rather than outcome
    • makes /supervise status open settings when active
    • resolves model/sensitivity defaults more consistently for future runs
    • updates notifications and tool responses to match goal-focused wording
  • src/state.ts and src/types.ts
    • add persisted supervisor preferences so inactive model/sensitivity changes can be reused on the next run
  • src/engine.ts
    • converts idle continue decisions into a steer so the run does not stall
  • src/ui/settings-panel.ts
    • renames the active settings field from Outcome to Goal

Testing

  • ./node_modules/.bin/tsc --noEmit

Copilot AI review requested due to automatic review settings April 17, 2026 19:58
Copy link
Copy Markdown

Copilot AI left a comment

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 aligns pi-supervisor terminology and UI/docs around a goal-focused supervision concept, while improving runtime defaults persistence and preventing idle supervisor decisions from stalling the run.

Changes:

  • Updates user-facing wording from “outcome” to “goal” across UI text, notifications, and documentation.
  • Persists model/provider and sensitivity selections as session-scoped defaults for the next /supervise, even when supervision is inactive.
  • Converts idle continue supervisor decisions into a steer so agent_end runs don’t stall.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
README.md Updates commands/terminology and documents defaults + idle-continue behavior.
src/index.ts Implements default resolution via persisted prefs, updates command behavior/copy, and applies prefs during tool-initiated starts.
src/state.ts Adds persisted supervisor prefs alongside persisted supervisor state.
src/types.ts Introduces SupervisorPrefs type for session-scoped defaults.
src/engine.ts Converts idle continue into steer to avoid stalled runs.
src/ui/settings-panel.ts Renames the active field label from “Outcome” to “Goal”.

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

Comment thread src/index.ts
Comment on lines 399 to 406
@@ -397,8 +406,8 @@ export default function (pi: ExtensionAPI) {
} else {
Comment thread src/index.ts
Comment on lines 424 to 425
"info"
);
Comment thread src/engine.ts
Comment on lines +233 to +237
if (agentIsIdle && decision.action === "continue") {
return {
action: "steer",
message: "Please continue working toward the goal.",
reasoning: decision.reasoning || "Idle continue converted to steer",
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