Repository navigation
spawn only session-loaded profiles; find poteto-mode in devin's plugin cache - #28
Conversation
…n cache - devin loads subagent profiles when a session starts. a pstack-M profile written mid-session returns "Subagent failed to start." in that session, even on a later turn. poteto-mode and the routed skills now spawn only listed profiles, fall back to the role's line or the auto mapping, and write the missing profile for the next session. - the delegate fallback in poteto-agent and the setup-pstack pstack-M block searched ~/.config/devin/ and .devin/, where a plugin install never lands. it now searches ~/.local/share/devin/cli/plugins/, the counterpart of upstream's ~/.cursor/plugins/. - drop the fusion pitfall bullet and the poteto-help port note left by #13, and say Adaptive instead of Auto. Closes #16 Closes #17 Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
caelaxie
left a comment
There was a problem hiding this comment.
verdict: changes requested (blocked on harness proof only).
content passes the bar on 341bc58:
- #16: fusion pitfall bullet dropped, guide page 10 matches upstream 88d4751 again.
- #17: port note dropped, "Auto" -> "Adaptive", as the issue asked.
- session-loaded profile fallback in poteto-mode + the six routed skills, and the delegate search path moved to
~/.local/share/devin/cli/plugins/: devin-only harness fixes, minimal, upstream text otherwise untouched. version unchanged is fine. Closes #16/Closes #17linkage is right. ci (Validate plugin) green.
missing: this touches plugin content, so it needs real devin proof on the exact head before merge. post it as a comment: devin plugins install --local wrapper pinned to the full head sha 341bc58, devin plugins list, the installed text of poteto-mode / setup-pstack / poteto-agent showing the new search path and fallback, and the live non-fusion + fusion turns you promised (a spawn of an unlisted pstack-M falling back to auto with a notice). symlink-only proof won't count. i'll merge on the next sweep once it's up.
harness proofreal devin cli on a local box (no cloud agent, no git clone). pr head
1. install / list2. install cache: cmp and symlinksinstalled text excerpts3. live turns (read-only)
the box's always-on both turns use the same prompt ( regular (non-fusion), session
|
caelaxie
left a comment
There was a problem hiding this comment.
pass. harness proof on 341bc58 (comment 6400677882) meets the devin bar: --local wrapper pinned to the full head sha, list shows pstack v0.15.32, 200/200 cache files cmp-identical with 0 symlinks, changed text present in the installed copy, and live regular + fusion turns fall back to subagent_explore with a notice when the profile is unlisted. validate plugin is green, the content passed last review, and it closes #16/#17. squash-merging.
Closes #16
Closes #17
harness-only fixes for devin, in both regular and fusion chats. version unchanged.
profiles load at session start
devin reads custom subagent profiles when a session starts, so a profile written later can't be spawned in that session. the port told agents to create a missing
pstack-Mprofile and then spawn it, so that spawn always failed.probe on devin 3000.11.3 (
swe-2-max,--permission-mode accept-edits):fabulous-ancient:probe-pre, written before the session started, returnsPONG-PRE.probe-mid(project.devin/agents/) andzz-pstack-probe-user(~/.config/devin/agents/), both written by the session, returnSubagent failed to start., and again aftersleep 30.devin -r fabulous-ancient(new process, same session): both complete.hilarious-scorpio: a second turn in the same process still getsSubagent failed to start.small-adverbloweredhow explainertoswe-2-high, spawnedpstack-swe-2-high-ro(written two minutes after the session started), and gotSubagent failed to start.poteto-mode's Subagents section and the six routed skills (
how,why,reflect,swarm,arena,interrogate) now spawn only listed profiles. An unlisted one falls back to the role's line as written, else to theautomapping, with a notice. The missing profile is still written, for the next session.setup-pstackstep 7 already says its profiles apply to new sessions.delegate fallback path
With no path from its parent,
poteto-agentand thepstack-Mblock insetup-pstackstep 6 searched~/.config/devin/and the project's.devin/.devin plugins installputs pstack in~/.local/share/devin/cli/plugins/cache/<source>/<version>/, so the search never found it. Both now search~/.local/share/devin/cli/plugins/, the counterpart of upstream's~/.cursor/plugins/, and thepstack-Mblock gains poteto-agent's highest-version tiebreak. Existingpstack-Mprofiles keep the old text until/pstack:setup-pstackruns again.#16, #17
The review trims from #13: drop the fusion pitfall bullet on guide page 10 and the poteto-help port note. poteto-help says Adaptive instead of Auto.
not changed
run_subagentfor panels and fan-out) and thefusion-*rejectionHarness proof (wrapper pinned to the head sha, installed text, live non-fusion and fusion turns) will follow as a comment.
Generated with Devin