PR 1: Convert research skills to deterministic YAML wrappers - #38
Merged
Conversation
PR 1 of deterministic workflow conversion. Skills and commands now delegate to workflows/research.yml and workflows/deep-research.yml instead of containing full step-by-step logic. Condensed fallbacks preserved for environments without the Go engine. - skills/research/SKILL.md: 145 → 37 lines - skills/deep-research/SKILL.md: 236 → 45 lines - commands/deep-research.md: 150 → 24 lines
- Restore harness detection fallback in commands/deep-research.md - Restore all 4 escalation triggers in research skill fallback - Align pipeline summaries with actual YAML step IDs - Merge sensitivity check into evidence-matrix step (matches YAML) - Add budget mention (~200k) to research fallback - Restore decompose-with-goals and confidence-calibration rules
6 tasks
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
1 participant
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
First PR of the deterministic workflow conversion project.
research,deep-research) converted from full step-by-step markdown (145/236 lines) to thin wrappers (37/45 lines) that delegate to YAML workflows viadevkit workflow rundeep-research.md) trimmed from 150 → 24 lines, pointing to the YAML workflow + skill fallbackworkflows/research.yml,workflows/deep-research.yml) untouched — they are already the source of truth with 9 and 13 deterministic steps respectivelyNet: -467 lines of duplicated logic removed. The Go engine now owns step sequencing; Claude handles thinking within each step.
Test plan
devkit workflow run research "test query"— engine executes full research pipelinedevkit workflow run deep-research "test query"— engine executes full ACH pipeline/devkit:deep-researchtab-completion still works viacommands/deep-research.md