Skip to content

refactor(create-expert): simplify PDCA structure in managers#369

Merged
FL4TLiN3 merged 1 commit intomainfrom
refactor/356-simplify-pdca
Jan 3, 2026
Merged

refactor(create-expert): simplify PDCA structure in managers#369
FL4TLiN3 merged 1 commit intomainfrom
refactor/356-simplify-pdca

Conversation

@FL4TLiN3
Copy link
Contributor

@FL4TLiN3 FL4TLiN3 commented Jan 3, 2026

Summary

Remove verbose Plan/Do/Check/Act phases and replace with concise declarations.

Changes

functional-manager

Before (~80 lines):

## PDCA Phases
### Phase 1: Happy-Path Testing
#### Plan
...
#### Do
...
#### Check & Act
...
### Phase 2: Unhappy-Path Testing
...

After (~18 lines):

## Test Categories
**Happy-path**: Valid inputs, expected queries...
**Unhappy-path**: Empty data, invalid formats...
**Adversarial**: Prompt injection resistance...

## Quality Criteria
Happy-path passes when: Core functionality works
...

usability-manager

Before (~45 lines):

## PDCA Loop
### Plan
...
### Do
...
### Check
...
### Act
...

After (~16 lines):

## Usability Properties
- Demo works zero-config
- Setup efficiency
- Error guidance
...

Test plan

  • CI passes

Closes #356

🤖 Generated with Claude Code


Note

Simplifies manager instructions to be declarative and concise

  • functional-manager: Replaces step-by-step PDCA with clear test categories (happy-path, unhappy-path, adversarial) and explicit pass criteria; instructs delegation to expert-tester; output is pass/fail counts per category.
  • usability-manager: Replaces PDCA loop with a focused checklist of usability properties (demo zero-config, setup efficiency, error guidance, doctor diagnostics, fresh user success) and delegation to expert-tester; output lists property pass/fail.
  • Adds a changeset entry documenting the simplification.

Written by Cursor Bugbot for commit ea7aa21. This will update automatically on new commits. Configure here.

Remove verbose Plan/Do/Check/Act phases:

functional-manager:
- Before: Phase 1/2/3 with Plan/Do/Check & Act for each
- After: Test Categories + Quality Criteria

usability-manager:
- Before: PDCA Loop with Plan/Do/Check/Act sections
- After: Usability Properties + Quality Criteria

Per docs/making-experts/best-practices.md:
> The LLM knows how to have a conversation.

Closes #356

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@codecov
Copy link

codecov bot commented Jan 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@FL4TLiN3 FL4TLiN3 merged commit 6655ae1 into main Jan 3, 2026
7 checks passed
@FL4TLiN3 FL4TLiN3 deleted the refactor/356-simplify-pdca branch January 3, 2026 05:48
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.

Refactor: functional-manager/usability-manager have overlapping PDCA phase structure

1 participant