Skip to content

docs: the PM owns board triage, not the spawned team's onboarding scan#26

Merged
mabry1985 merged 1 commit into
mainfrom
docs/pm-owns-triage
Jul 3, 2026
Merged

docs: the PM owns board triage, not the spawned team's onboarding scan#26
mabry1985 merged 1 commit into
mainfrom
docs/pm-owns-triage

Conversation

@mabry1985

Copy link
Copy Markdown
Contributor

Companion to portfolio-plugin's onboarding redesign (portfolio-plugin#<TBD>): the team's onboarding pass is now a read-only scan (no board features, no code changes) — the PM reviews the readiness summary and dispatches what's needed.

Updates the archetype soul so a PM spun up from this bundle actually knows the new contract, rather than silently assuming the team boards its own onboarding gaps (the old, buggy assumption — see the linked PR for the live dogfooding postmortem: a freshly-spawned team double-called board_create_feature for the same onboarding item, and even "safe" auto-fix items ended up as board features).

🤖 Generated with Claude Code

Companion to portfolio-plugin's onboarding redesign (the team's onboarding
pass is now scan-and-report only -- no board writes). Update the archetype
soul so a PM built from this bundle knows: after spin-up, review the
readiness summary yourself and portfolio_dispatch whatever needs work --
the team never self-boards its own onboarding findings.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@protoquinn

protoquinn Bot commented Jul 3, 2026

Copy link
Copy Markdown

👀 Quinn is reviewing — verdict (PASS / WARN / FAIL) + findings to follow.

@protoquinn protoquinn Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA Audit — PR #26 | docs: the PM owns board triage, not the spawned team's onboarding scan

VERDICT: WARN (non-terminal CI; expected PASS once verify completes)


CI Status

  • bump: skipped
  • verify: queued

Diff Review

  • Single-file docs change to protoagent.bundle.yaml (+3 lines, archetype soul)
  • Adds clear contract language: team spin-up runs a read-only readiness scan; the PM reviews the summary and dispatches work via portfolio_dispatch; the team never self-boards
  • No code paths, no logic changes — purely an archetype prompt update

Observations

  • LOW: CI still queued — re-review once verify is terminal green. The diff is docs-only so risk is nil, but the formal PASS needs terminal CI.

— Quinn, QA Engineer

@protoquinn

protoquinn Bot commented Jul 3, 2026

Copy link
Copy Markdown

Submitted COMMENT review on protoLabsAI/portfolio-manager-stack#26.

@protoquinn protoquinn Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI terminal-green, no blockers on prior review — auto-approving on green (#748).

@mabry1985 mabry1985 merged commit efb6efe into main Jul 3, 2026
2 checks passed
@mabry1985 mabry1985 deleted the docs/pm-owns-triage branch July 3, 2026 20:17
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.

1 participant