Move the agent monitor to psf-monitor - #28
Merged
Merged
Conversation
The monitor now lives in its own plugin, thisguymartin/psf-monitor. Remove the monitor skill and scripts/monitor, drop them from the scripts' test and typecheck runs, and point the docs at psf-monitor. pstack-runner keeps writing the lane journal, which psf-monitor reads. docs/reference.md now documents the journal as the contract between the two.
thisguymartin
marked this pull request as ready for review
October 5, 2026 04:22
thisguymartin
pushed a commit
that referenced
this pull request
Oct 5, 2026
…tor) Main moved the agent monitor to its own repository, psf-monitor (#28). Take that: drop the in-repo pstack-monitor plugin this branch had split out, with its marketplace entries, CI steps, check.sh steps, .gitignore line, and static checks. Keep main's psf-monitor docs and lane journal wording; keep this branch's rename, intake, diff-behavior, doc fixes, check.sh, and LIVE-GATE.md, with the monitor row replaced by a lane journal check. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LSWNeBzAMt5FQZVziZCGLs
Merged
1 of 4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #27
What changed
The agent monitor moved to its own plugin, psf-monitor. pstack users who want the live view install that plugin next to pstack. pstack itself loses the
monitorskill.skills/monitor/andskills/poteto-mode/scripts/monitor/, and dropmonitorfrom the scripts'testandtypecheckruns.pstack-runner's lane journal unchanged.docs/reference.mdnow documents it as the contract psf-monitor reads:lane.jsonschema version 1,stream.jsonl,receipt.json, and SIGTERM to the runner pid as the cancel path.psf-monitor was built from this repo's monitor at
71f51a1. It adds server-side pstack-only filtering, with no--all, and page controls to cancel a lane, copy a resume command, toggle the journal, and stop the monitor.Verification
Checks run on
259c1b1:bun run testpassed 208 of 208,bun run typecheckpassed, the manifest parse andPSTACK_STATIC_ONLY=1 bash tests/skill-collision-repro.shpassed, andclaude plugin validate .passed.Live evidence:
Still needed for this candidate: install the branch in Claude Code and Codex, confirm
pstack:monitoris gone, and confirm apstack-runnerlane still journals and appears in psf-monitor.Already observed with the installed pstack 1.5.0 runner, whose journal code this PR changes only in a comment, and psf-monitor 0.1.1 installed in Claude Code:
/psf-monitor:monitorfrom a fresh Claude Code process ran the launcher and printed the link.pstack-runnerCodex lane labeledpsf live cancel testappeared as running. Clicking Cancel lane and then Confirm turned it cancelled, with receiptstatus: cancelled, signal: SIGTERM.A pull request without live evidence remains a draft. Do not merge, tag, release, or roll it out.