Skip to content

Align the personas with the skills and docs - #82

Merged
TiantianFlow merged 2 commits into
mainfrom
claude/project-thread-ntld6s
Oct 23, 2026
Merged

TiantianFlow merged 2 commits into
mainfrom
claude/project-thread-ntld6s

Conversation

@TiantianFlow

Copy link
Copy Markdown
Owner

Follow-up to #81 (issue #80), from an audit of the personas against the skills, requirements, and READMEs.

What changed

  • Lead skill, README (en and zh-CN), requirements lead row: they now say what the lead persona says: when seats' results disagree, the lead chooses by their evidence or writes the task that settles the point, and never puts its own answer in their place.
  • Lead persona: integrating branches is the lead's job; a design isn't accepted, and the goal isn't done, until a reviewer has tried to break it (as the skill requires). Two independent proposals are two tasks, so they fit the skill's rule for running seats side by side. Two leftover Foil terms are now plain words.
  • Reviewer persona: it reports pass or fail, matching the result contract, instead of "changes needed".
  • Researcher persona: the lead decides, not "the lead and the designer".
  • Memory curator persona: it writes replacement lessons in its result for the lead to file, since a worker's command list has no --replaces.
  • Documentation writer persona: the placeholder tool init example is now a real command.
  • Operator skill: it no longer names an outside persona catalog.

Verified

  • uv run --frozen --extra dev pytest -m "not e2e_live" --deselect tests/test_view_browser.py: 566 passed on this branch's head.
  • ruff check .: clean.
  • git grep -i agency: no hits.
  • Independent audit (consistency with skills, docs, and requirements; Foil mechanics leaking into personas; privacy; publish safety): its findings are fixed in this PR.

Not verified

  • tests/test_view_browser.py locally (the sandbox's Chromium does not match Playwright's version); CI runs it.

Generated by Claude Code

The lead skill and README now say the lead chooses among seats'
proposals by their evidence. The lead persona keeps integration and the
design and final reviews the skill requires. The reviewer reports pass or
fail like the result contract. The documentation writer's example uses a
real command instead of a placeholder.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CD864r5cxkYPNAx1TCWTnb
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CD864r5cxkYPNAx1TCWTnb
@TiantianFlow TiantianFlow self-assigned this Oct 23, 2026
@TiantianFlow
TiantianFlow merged commit e50b849 into main Oct 23, 2026
6 checks passed
@TiantianFlow
TiantianFlow deleted the claude/project-thread-ntld6s branch October 23, 2026 02:32
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