Implementation Plan: Rename sous-chef to admiral for L3 orchestrator terminology#2187
Merged
Trecek merged 4 commits intoMay 7, 2026
Conversation
Rename L3-specific sous-chef references to admiral in fleet dispatch and campaign prompts. The sous-chef SKILL.md and L1/L2 cook session references remain unchanged. - ADMIRAL_DISPATCH_SECTIONS (was SOUS_CHEF_FOOD_TRUCK_SECTIONS) - _build_admiral_dispatch_block() (was _build_food_truck_sous_chef_block()) - ADMIRAL DISCIPLINE header in L3 dispatch/campaign prompts - Updated corresponding tests and CLAUDE.md Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…in TestL3AdmiralDiscipline
Trecek
added a commit
that referenced
this pull request
May 8, 2026
…terminology (#2187) ## Summary Rename all "sous-chef" references in L3 orchestrator contexts (fleet dispatch, campaign dispatcher) to "admiral" terminology. The sous-chef SKILL.md file, its directory, and all L1/L2 cook session references remain unchanged. The rename affects: one constant in `_type_constants.py`, one builder function in `fleet/_prompts.py`, section headers in two L3 prompt builders, and corresponding tests. **Key finding from investigation:** The issue references `SOUS_CHEF_L3_SECTIONS` and `_build_l3_sous_chef_block()` — these do not exist. The actual symbols are `SOUS_CHEF_FOOD_TRUCK_SECTIONS` and `_build_food_truck_sous_chef_block()`. The plan uses the actual symbol names. ## Requirements - REQ-TERM-001: L3 orchestrator sessions (fleet dispatch, campaign) must use "admiral" terminology in their prompt sections, constants, and builder functions - REQ-TERM-002: L2 cook sessions must continue using "sous-chef" terminology unchanged - REQ-TERM-003: The `SOUS_CHEF_FOOD_TRUCK_SECTIONS` constant must be renamed to reflect admiral terminology, with `RETIRED_SKILL_NAMES` or equivalent migration handling if applicable - REQ-TERM-004: All tests referencing renamed constants/functions must be updated in the same commit ## Implementation Plan Plan file: `/home/talon/projects/autoskillit-runs/impl-20260507-120004-106551/.autoskillit/temp/make-plan/rename_sous_chef_to_admiral_plan_2026-05-07_120600.md` 🤖 Generated with [Claude Code](https://claude.com/claude-code) via AutoSkillit <!-- autoskillit:pipeline-signature steps=prepare_pr,run_arch_lenses,compose_pr,annotate_pr_diff,review_pr --> ## Token Usage Summary | Step | Model | count | uncached | output | cache_read | peak_ctx | turns | cache_write | time | |------|-------|-------|----------|--------|------------|----------|-------|-------------|------| | plan | claude-opus-4-6 | 1 | 85 | 20.1k | 1.7M | 74.8k | 110 | 61.6k | 9m 40s | | verify | claude-opus-4-6 | 1 | 50 | 7.9k | 521.9k | 48.9k | 58 | 35.9k | 4m 37s | | implement* | MiniMax-M2.7-highspeed | 1 | 4.1M | 22.1k | 3.1M | 56.8k | 238 | 72.7k | 13m 31s | | prepare_pr* | MiniMax-M2.7-highspeed | 1 | 69.9k | 4.1k | 213.6k | 35.0k | 22 | 51.0k | 1m 26s | | compose_pr* | MiniMax-M2.7-highspeed | 1 | 37.9k | 1.5k | 175.7k | 29.8k | 14 | 15.0k | 43s | | **Total** | | | 4.2M | 55.7k | 5.7M | 74.8k | | 236.2k | 29m 59s | \* *Step used a non-Anthropic provider; caching behavior may differ.* ## Token Efficiency | Step | LoC Changed | cache_read/LoC | cache_write/LoC | output/LoC | |------|-------------|----------------|-----------------|------------| | plan | 0 | — | — | — | | verify | 0 | — | — | — | | implement | 123 | 25092.2 | 591.3 | 179.5 | | prepare_pr | 0 | — | — | — | | compose_pr | 0 | — | — | — | | **Total** | **123** | 46656.3 | 1920.5 | 453.1 | ## Model Usage Breakdown | Model | steps | uncached | output | cache_read | cache_write | time | |-------|-------|----------|--------|------------|-------------|------| | claude-opus-4-6 | 2 | 135 | 28.1k | 2.3M | 97.5k | 14m 17s | | MiniMax-M2.7-highspeed | 3 | 4.2M | 27.6k | 3.5M | 138.7k | 15m 41s | --------- Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
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
Rename all "sous-chef" references in L3 orchestrator contexts (fleet dispatch, campaign dispatcher) to "admiral" terminology. The sous-chef SKILL.md file, its directory, and all L1/L2 cook session references remain unchanged. The rename affects: one constant in
_type_constants.py, one builder function infleet/_prompts.py, section headers in two L3 prompt builders, and corresponding tests.Key finding from investigation: The issue references
SOUS_CHEF_L3_SECTIONSand_build_l3_sous_chef_block()— these do not exist. The actual symbols areSOUS_CHEF_FOOD_TRUCK_SECTIONSand_build_food_truck_sous_chef_block(). The plan uses the actual symbol names.Requirements
SOUS_CHEF_FOOD_TRUCK_SECTIONSconstant must be renamed to reflect admiral terminology, withRETIRED_SKILL_NAMESor equivalent migration handling if applicableImplementation Plan
Plan file:
/home/talon/projects/autoskillit-runs/impl-20260507-120004-106551/.autoskillit/temp/make-plan/rename_sous_chef_to_admiral_plan_2026-05-07_120600.md🤖 Generated with Claude Code via AutoSkillit
Token Usage Summary
* Step used a non-Anthropic provider; caching behavior may differ.
Token Efficiency
Model Usage Breakdown