Skip to content

Skillz based flow#457

Closed
tcapelle wants to merge 6 commits intomainfrom
skillz
Closed

Skillz based flow#457
tcapelle wants to merge 6 commits intomainfrom
skillz

Conversation

@tcapelle
Copy link
Copy Markdown
Contributor

Refactor advisor workflow from a monolithic instruction doc into composable Claude Code skills

The advisor's CLAUDE-ADVISOR.md was a 182-line file with the entire workflow hardcoded — inline gh commands, PR templates,
label swap patterns, sub-agent instructions, decision criteria. This PR extracts those into 3 standalone skills:

  • check-student-progress — survey W&B metrics, list PRs, identify idle students, review status:review PRs
    (merge/request-changes/close logic, label management, conflict handling)
  • generate-research-ideas — launch an Opus sub-agent to review past experiments and produce ranked hypotheses (plateau
    protocol, prioritization guidelines)
  • assign-work — create experiment branches and labeled draft PRs for idle students (PR body template, instruction quality
    guidelines)

CLAUDE-ADVISOR.md is now a 67-line strategy document: identity, boundaries, a 4-step loop that references the skills by
name, and high-level principles. All mechanical details live in the skills.

Additionally:

  • Skills use $ADVISOR_BRANCH, $STUDENT_NAMES etc. directly (real ConfigMap env vars) instead of
    placeholders
  • Restores k8s/orchestrator-rbac.yaml (deleted in aa244b4) with added apps/deployments permissions so kubectl get
    deployments actually works

tcapelle and others added 6 commits March 16, 2026 23:04
Extract the advisor's hardcoded gh-command workflows into three
reusable Claude Code skills (check-student-progress, generate-research-ideas,
assign-work) and simplify CLAUDE-ADVISOR.md to a high-level strategy doc
that references them. Restore orchestrator-rbac.yaml (deleted in aa244b4)
with added apps/deployments permissions for kubectl get deployments.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Keep skill-based loop from skillz branch with updated file references
(train.py) from main.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Update assign-work and generate-research-ideas skills to reference
train.py after the repo restructure.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…<advisor>/

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…nches to <advisor>/

Resolves conflict in CLAUDE-ADVISOR.md by keeping the skill-based loop
from skillz and propagating the label/branch rename into the skills.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tcapelle tcapelle closed this Apr 2, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant