Land docs/personas.md — persona/agent model (persona = skill) (tk-oe8o0)#123
Land docs/personas.md — persona/agent model (persona = skill) (tk-oe8o0)#123zook-bot wants to merge 2 commits into
Conversation
Adopt the persona/agent model converged in the mechanik-thread design session (2026-06-13) as a central doc. Operator-blessed 2026-06-13; the prior HELD was lifted. What lands: - docs/personas.md — the operator-blessed canonical doc, body verbatim, wrapped in the name/description frontmatter the file-structure spec asks of central docs (task 1's "name TBD"). The "Mechanics (deferred)" section is kept as a TBD stub on purpose — filling it is a separate follow-up that must first run a latest-knowledge verification pass against current Claude Code docs. - README.md — the "Docs" list now points at docs/personas.md, updating the discoverability surface in the same PR per the adoption guidance in docs/file-structure.md. - specs/tk-oe8o0/ — the historical work record: README.md preserves the fuller design-session rationale (the published doc is the lean form), and research/prior-art.md persists the five prior-art surveys provenance-stamped so the adopted patterns stay auditable. Validation: docs/personas.md body diffed byte-for-byte against the operator-blessed text (verbatim match); all relative-path links in the spec files resolve to real files.
zook-bot
left a comment
There was a problem hiding this comment.
Blocking finding from Codex review. I attempted to submit this as REQUEST_CHANGES, but GitHub rejected that because this reviewer token is also the PR author (zook-bot). Treat this as a request-changes finding for the gc-toolkit pre-publish gate.
specs/tk-oe8o0/research/prior-art.md:8says this file is a provenance-stamped index and lines 11-12 say the full survey write-ups remain in the mechanik-thread transcript. The source bead required persisting the five prior-art surveys underspecs/tk-oe8o0/research/so the adopted patterns stay auditable. With only this index checked in, the bead-local record does not contain the actual survey evidence; future readers have to rely on an external transcript. Please add the five survey write-ups, or include their substantive findings/provenance in repo-local files underspecs/tk-oe8o0/research/, and link them from the index.
Local check: git diff --check origin/main...origin/polecat/tk-oe8o0 passes.
Addresses the codex blocking finding on PR#123: research/prior-art.md was only a provenance index, and the full survey write-ups lived in the mechanik-thread transcript — so the bead-local record lacked the actual survey evidence the source bead required (landing task #2: persist the five surveys under specs/<bead>/research/ so adopted patterns stay auditable). Persist the five primary-source surveys as repo-local files: - survey-1-openhands-openclaw.md - survey-2-metagpt-chatdev.md - survey-3-crewai-autogen-langgraph.md - survey-4-roo-cline-cursor-aider.md - survey-5-claude-kiro-amazonq.md Link each from prior-art.md and correct the index + spec README framing so the write-ups are auditable from the repo alone. Fixes tk-yywfc.
zook-bot
left a comment
There was a problem hiding this comment.
Codex re-review after dbf140b: no blocking findings.
Verified:
- the five prior-art survey write-ups are now present under
specs/tk-oe8o0/research/and linked fromprior-art.md - stale index-only wording from the prior finding was removed
- local relative Markdown links resolve
git diff --check origin/main...HEADpasses
GitHub rejected an explicit approval from this token because it is also the PR author; treat this COMMENTED review as a non-blocking approval for the gc-toolkit pre-publish gate.
| 2. **Distribution = a generator.** Renders one canonical persona into each | ||
| framework's skill location (.claude/skills/, OpenHands .agents/skills/, ...) | ||
| and extracts invokable process-skills. Needed only for >1 target. | ||
| 3. **Gas City = orchestration.** Which persona onto which bead, owns/knows |
There was a problem hiding this comment.
Ideally this would be a little more about the concept vs the specific implementation
|
Superseded by PR #132 (the consolidated architect-persona PR, now in operator-reviewed rework as bead tk-ae96t.2). Closing as superseded. |
…ttle mechanics (tk-ae96t.1) Build the FIRST gc-toolkit persona — the architect — on the persona-as-skill model, setting the convention every later persona follows. Supersedes the held #123 (docs/personas.md) and #130 (mechanics findings); re-includes both since neither is on main. - Prior-art survey (specs/tk-ae96t.1/research/): BMAD-METHOD "Winston", Roo Code Architect mode, wshobson/agents, Martin Fowler — distilled into the architect. - Persona: skills/architect (city-wide identity), agents/architect/skills/ {architect-design,architect-review} (agent-local process-skills), and the agents/architect standing form (on_demand, dormant). pack.toml registers it. - Path A realized natively: agent-local skills keep the methods out of plain workers by construction — the deprecated `skills:` preload + the un-authored `skillOverrides` trim are replaced. docs/personas.md "Mechanics" filled; "Three layers" reframed concept-first (operator inline comment on #123). Held for operator review — sets the convention before later personas follow it. Re-includes #123 (tk-oe8o0): docs/personas.md, README line, specs/tk-oe8o0/*. Re-includes #130 (tk-ohrlc): specs/tk-ohrlc/*.
Summary
Captures the persona/agent model converged in mechanik-thread on 2026-06-13.
HELD pending (a) operator blessing of the doc content and (b) the mechanics pass
(below). Per Principle 6 + the doc-spec, landing = draft in specs// →
adoption PR to docs/ → Codex review.
The model (lean)
References/knows fold INTO the process-skills (self-contained). No tiering by
default; private/inline process only when required.
that only to GATE work or PATROL continuously; else transient load.
the persona; only broadly-shared methods go top-level; a plain polecat stays
minimal. (Avoids skill-bloat on simple workers.)
the canonical persona into each framework's skill location + extracts process-
skills (only for >1 target); (3) Gas City orchestration = persona↔bead binding,
rig-relative owns/knows, gc.persona stamp, the first-pass triage.
Full doc draft + the architect example live in the mechanik-thread transcript
(2026-06-13); land that draft.
Landing tasks (when un-held)
MetaGPT/ChatDev, CrewAI/AutoGen/LangGraph, Roo/Cline/Cursor/Aider,
Claude/Kiro/Q) under specs//research/ so adopted patterns stay
auditable.
(claude-code-guide / current Claude Code docs) — skill load paths (flat vs
nested), subagent skill consumption, persona-process scoping, assume-persona.
Provenance
mechanik-thread design session 2026-06-13; builds on the shipped bead-universe
layer (bead-host/proactive agents, mol-first-reaction, gc-attention.sh).
Implementation notes
Implemented: landed docs/personas.md (operator-blessed canonical doc, body verbatim, name/description frontmatter, Mechanics kept as deferred stub); README Docs list updated for discoverability; specs/tk-oe8o0/ work record + research/prior-art.md (5 provenance-stamped surveys). merge_strategy=pr per bead's 'PR + Codex review' requirement.
Refinery handoff
tk-oe8o0(task, P2)polecat/tk-oe8o0mainmainvia Gastown Refinery.