Skip to content

Commit 1b871eb

Browse files
rysweetclaude
andauthored
fix: Revert inappropriate deprecations from PR #1686 (#1878)
Problem: PR #1686 inappropriately deprecated core ultrathink workflow components and added orphaned hook code not wired into settings.json. Changes: 1. Removed DEPRECATED flags from 3 files: - ultrathink.md command - ultrathink-orchestrator SKILL.md (restored auto_activate: true) - default-workflow SKILL.md (restored auto_activates array) 2. Restored CLAUDE.md ultrathink references (8 mentions): - "ALWAYS use UltraThink" principle - Skill(ultrathink-orchestrator) pattern - /ultrathink command documentation - Workflow and UltraThink Integration section 3. Added Q&A workflow awareness to ultrathink.md: - Q&A keyword detection - Q&A workflow routing - Q&A skill invocation - Q&A fallback path - Q&A workflow example 4. Deleted 8 orphaned files (NOT wired into settings.json): - workflow_gate.py + test - workflow_state.py + test - todo_interceptor.py + test - test_workflow_enforcement.py - docs/workflow-enforcement.md Net Result: -2,533 lines of orphaned code removed while preserving Q&A_WORKFLOW.md and MANDATORY workflow selection section. Fixes #1876 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Sonnet 4.5 (1M context) <noreply@anthropic.com>
1 parent 7dd7f23 commit 1b871eb

File tree

12 files changed

+76
-2609
lines changed

12 files changed

+76
-2609
lines changed

.claude/commands/amplihack/ultrathink.md

Lines changed: 22 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,6 @@
22
name: amplihack:ultrathink
33
version: 1.0.0
44
description: Deep analysis mode with multi-agent orchestration
5-
DEPRECATED: true
6-
deprecation_notice: |
7-
This command is DEPRECATED as of 2025-11-26.
8-
Workflow selection is now handled directly in CLAUDE.md via the
9-
"MANDATORY: Workflow Selection" section. Use workflow classification
10-
directly instead of invoking /ultrathink.
11-
For backward compatibility, this command still works but routes to
12-
workflow classification in CLAUDE.md.
135
triggers:
146
- "Complex multi-step task"
157
- "Need deep analysis"
@@ -42,16 +34,21 @@ You MUST use one of the workflow skills - either default-workflow or investigati
4234
When this command is invoked, you MUST:
4335
Execute this exact sequence for the task: `{TASK_DESCRIPTION}`
4436

45-
1. **First, detect task type** - Check if task is investigation or development
37+
1. **First, detect task type** - Check if task is Q&A, investigation, or development
38+
- **Q&A keywords**: what is, explain briefly, quick question, how do I run, simple question
4639
- **Investigation keywords**: investigate, explain, understand, how does, why does, analyze, research, explore, examine, study
4740
- **Development keywords**: implement, build, create, add feature, fix, refactor, deploy
48-
- **If both types detected**: Use hybrid workflow (investigation first, then development)
49-
- If only investigation keywords found: Use @.claude/workflow/INVESTIGATION_WORKFLOW.md
50-
- If only development keywords found: Use @.claude/workflow/DEFAULT_WORKFLOW.md
41+
- **Priority order**: Q&A detection first (simple questions), then Investigation, then Development
42+
- **If Q&A detected**: Use @.claude/workflow/Q&A_WORKFLOW.md (simple, single-turn answers)
43+
- **If Investigation keywords found**: Use @.claude/workflow/INVESTIGATION_WORKFLOW.md
44+
- **If Development keywords found**: Use @.claude/workflow/DEFAULT_WORKFLOW.md
45+
- **If both Investigation and Development detected**: Use hybrid workflow (investigation first, then development)
5146
2. Mandatory - not doing this wil require rework **Invoke the appropriate workflow skill** using the Skill tool:
47+
- Q&A: Read @.claude/workflow/Q&A_WORKFLOW.md directly (no skill wrapper needed for simple Q&A)
5248
- Investigation: Skill(skill="investigation-workflow")`
5349
- Development: `Skill(skill="default-workflow")`
5450
- **FALLBACK**: If skill invocation fails (skill not found), fall back to reading markdown workflows:
51+
- Q&A: @.claude/workflow/Q&A_WORKFLOW.md
5552
- Investigation: @.claude/workflow/INVESTIGATION_WORKFLOW.md
5653
- Development: @.claude/workflow/DEFAULT_WORKFLOW.md
5754
3. ALWAYS **Create a comprehensive todo list** using TodoWrite tool that includes all workflow steps/phases
@@ -118,6 +115,19 @@ Always use TodoWrite to:
118115

119116
## Example Flow
120117

118+
### Q&A Task Example
119+
120+
```
121+
User: "/ultrathink what is the purpose of the workflow system?"
122+
123+
1. Detect: Q&A task (contains "what is")
124+
2. Select: Q&A workflow (simple, single-turn)
125+
3. Read: `.claude/workflow/Q&A_WORKFLOW.md`
126+
4. Follow Q&A workflow steps (typically 3-4 steps)
127+
5. Provide concise, direct answer
128+
6. No complex agent orchestration needed
129+
```
130+
121131
### Development Task Example
122132

123133
```

.claude/skills/default-workflow/SKILL.md

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,10 @@
22
name: default-workflow
33
version: 1.0.0
44
description: Development workflow for features, bugs, refactoring. Auto-activates for multi-file implementations.
5-
DEPRECATED: true
6-
deprecation_notice: |
7-
This skill is DEPRECATED as of 2025-11-26.
8-
Workflow selection is now handled directly in CLAUDE.md via the
9-
"MANDATORY: Workflow Selection" section. Read the workflow file
10-
directly instead of using this skill wrapper.
11-
auto_activates: []
5+
auto_activates:
6+
- "implement feature spanning multiple files"
7+
- "complex integration across components"
8+
- "refactor affecting 5+ files"
129
explicit_triggers:
1310
- /ultrathink
1411
- /amplihack:default-workflow

.claude/skills/ultrathink-orchestrator/SKILL.md

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,7 @@
22
name: ultrathink-orchestrator
33
version: 1.0.0
44
description: Auto-invokes ultrathink workflow for any work request (default orchestrator)
5-
DEPRECATED: true
6-
deprecation_notice: |
7-
This skill is DEPRECATED as of 2025-11-26.
8-
Workflow selection is now handled directly in CLAUDE.md via the
9-
"MANDATORY: Workflow Selection" section. The indirection through
10-
skills has been eliminated to ensure consistent workflow enforcement.
11-
auto_activate: false
5+
auto_activate: true
126
priority: 5
137
triggers:
148
- "implement"

0 commit comments

Comments
 (0)