feat: agents/skills workflow, agent colors, and CLI/tooling hardening#44
Merged
Conversation
… CLI/tooling WIP This session's work — the agents/skills workflow optimization and agent colors — is shipped together with the larger in-progress working-tree changeset at the maintainer's request (one PR). This session (agents/skills + colors): - wf-* leads documented to run as the main session agent (claude --agent or the "agent" setting) — the only mode where gate stops, /clear, and sub-agent spawning work; wf-development also gains the Agent tool - parallel (P) task dispatch across boundaries via isolation: worktree - prefer csdd_* MCP tools over the npx form in the leads and implementer - wire the orphan test-designer into the flow; fix the phantom "debugger" reference in CLAUDE.md - preload tdd-cycle/verify-change into the implementer via skills:; effort: high on both leads - color: on the six shipped agents + a validated --color flag on `csdd agent create` (internal/cli/color.go, color_test.go, agent_color_test.go) In-progress WIP bundled in (pre-existing in the working tree, not from this session): - CLI: --help handling, workspace.SafeName guards, internal/cli/jsonout.go and a new internal/textutil package; touch-ups across cli, web, validator, workspace, manifest, session, templater, mcp-server, and npm - tooling: .golangci.yml, .gitattributes (normalize line endings to LF), CI - relocate csdd.html -> docs/csdd.html
…d-agent-workflows # Conflicts: # internal/web/handlers.go # internal/web/server.go
…n/mode tests on Windows
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Bundles two things into one PR at the maintainer's request:
textutilpackage, line-ending normalization). It is included as-is; it was not authored in this session.62 files changed, +2963 / −1823.
This session — agents/skills workflow + colors
wf-*leads as the main session agent (claude --agent wf-development/wf-product-discovery, or the"agent"setting). This is the only mode where the phase gates can pause for human approval,/clearworks, and the leads can spawn fresh-context sub-agents.wf-developmentalso gains theAgenttool so it can actually invokecode-reviewer/security-reviewer/test-designer.(P)task dispatch across boundaries viaisolation: worktree— the(P)/_Boundary:_/_Depends:_machinery finally has a consumer, while same-boundary tasks stay sequential.csdd_*MCP tools over thenpxform in the leads and theimplementer.test-designerinto the flow and fix the phantomdebuggerreference inCLAUDE.md.tdd-cycle/verify-changeinto theimplementerviaskills:(resolves "delegate to a skill" without theSkilltool);effort: highon both leads.color:on the six shipped agents (discovery=purple, development=blue, implementer=green, code-reviewer=red, security-reviewer=orange, test-designer=cyan) plus a validated--colorflag oncsdd agent create(internal/cli/color.go+ unit/integration tests), mirroring the existing--effortpattern.--modelhelp updated to mentionfable/ full model IDs.Bundled in-progress WIP (pre-existing, not from this session)
--helphandling,workspace.SafeNameguards on agent/skill lookups, newinternal/cli/jsonout.go(structured output) andinternal/textutilpackage (shared newline/text helpers), plus touch-ups acrosscli,web,validator,workspace,manifest,session,templater,mcp-server, andnpm..golangci.yml,.gitattributes(normalize line endings to LF — this is why many files show large diffs), CI workflow updates.csdd.html→docs/csdd.html(100% rename).Verification
go build ./...✅go test ./...✅ green in an earlier full run; re-confirmed for the changed/critical packages:cli,textutil,templater,frontmatter,validator,workspaceall ✅.--colorflag covered bycolor_test.go(validation set) +agent_color_test.go(create-with-color / no-color / invalid-color-rejected).Notes
.idea/(IDE config, untracked) was deliberately excluded from the commit.update/clean.old-backup tests are flaky on the/mnt/cWSL filesystem (pass/fail non-deterministically, independent of code) — not introduced here.