Skip to content

Integration: collapsed PRs #319, #318, #316, #314, #315, #317, #320 into integration#323

Merged
Trecek merged 1 commit intointegrationfrom
pr-batch/pr-merge-20260310-163009
Mar 11, 2026
Merged

Integration: collapsed PRs #319, #318, #316, #314, #315, #317, #320 into integration#323
Trecek merged 1 commit intointegrationfrom
pr-batch/pr-merge-20260310-163009

Conversation

@Trecek
Copy link
Collaborator

@Trecek Trecek commented Mar 11, 2026

Integration Summary

Collapsed 7 PRs into pr-batch/pr-merge-20260310-163009 targeting integration.

Merged PRs

# Title Complexity Additions Deletions Overlaps
#319 Add Step 4.5 Historical Regression Check to dry-walkthrough skill simple +200 -0
#318 Implementation Plan: Pipeline Observability — Quota Guard Logging and Per-Step Elapsed Time simple +267 -23
#316 Release CI Automation — Version Bump, Branch Sync, and Release Infrastructure simple +471 -0
#314 docs: complete release documentation sprint simple +1344 -104
#315 Recipe remediation — source_dir resolution and cook command display simple +317 -71 #320
#317 Implementation Plan: PR Review Pipeline Gates — Fidelity Checks, CI Gating, and Review-First Enforcement simple +1057 -7 #320
#320 feat: add recipe composition with run_recipe tool and dev-sprint consumer needs_check +724 -19 #315, #317

Audit

Verdict: GO

Architecture Impact

Development Diagram

%%{init: {'flowchart': {'nodeSpacing': 50, 'rankSpacing': 60, 'curve': 'basis'}}}%%
flowchart TB
    %% CLASS DEFINITIONS %%
    classDef cli fill:#1a237e,stroke:#7986cb,stroke-width:2px,color:#fff;
    classDef stateNode fill:#004d40,stroke:#4db6ac,stroke-width:2px,color:#fff;
    classDef handler fill:#e65100,stroke:#ffb74d,stroke-width:2px,color:#fff;
    classDef phase fill:#6a1b9a,stroke:#ba68c8,stroke-width:2px,color:#fff;
    classDef newComponent fill:#2e7d32,stroke:#81c784,stroke-width:2px,color:#fff;
    classDef output fill:#00695c,stroke:#4db6ac,stroke-width:2px,color:#fff;
    classDef detector fill:#b71c1c,stroke:#ef5350,stroke-width:2px,color:#fff;
    classDef terminal fill:#1a237e,stroke:#7986cb,stroke-width:2px,color:#fff;

    subgraph Structure ["PROJECT STRUCTURE"]
        direction LR
        SRC["src/autoskillit/<br/>━━━━━━━━━━<br/>105 source files<br/>10 sub-packages"]
        TESTS["tests/<br/>━━━━━━━━━━<br/>170 test files<br/>Mirrors src layout"]
    end

    subgraph Build ["BUILD TOOLING"]
        direction LR
        PYPROJECT["● pyproject.toml<br/>━━━━━━━━━━<br/>hatchling backend<br/>uv package manager"]
        TASKFILE["Taskfile.yml<br/>━━━━━━━━━━<br/>task test-all<br/>task test-check<br/>task install-worktree"]
        UVLOCK["uv.lock<br/>━━━━━━━━━━<br/>Locked manifest"]
    end

    subgraph PreCommit ["PRE-COMMIT HOOKS"]
        direction TB
        RUFF_FMT["ruff-format<br/>━━━━━━━━━━<br/>Auto-format<br/>(writes source)"]
        RUFF_LINT["ruff check<br/>━━━━━━━━━━<br/>Lint --fix<br/>target: py311"]
        MYPY["mypy<br/>━━━━━━━━━━<br/>src/ type check<br/>--ignore-missing"]
        UVCHECK["uv lock --check<br/>━━━━━━━━━━<br/>Lockfile guard"]
        NOGEN["no-generated-configs<br/>━━━━━━━━━━<br/>Blocks hooks.json<br/>recipes/diagrams/"]
        GITLEAKS["gitleaks v8.30.0<br/>━━━━━━━━━━<br/>Secret scanning"]
    end

    subgraph Testing ["TEST FRAMEWORK"]
        direction LR
        PYTEST["pytest -n 4<br/>━━━━━━━━━━<br/>asyncio_mode=auto<br/>timeout=60s"]
        FIXTURES["conftest.py<br/>━━━━━━━━━━<br/>StatefulMockTester<br/>MockSubprocessRunner<br/>tool_ctx fixture"]
        IMPORTLINT["import-linter<br/>━━━━━━━━━━<br/>L0→L1→L2→L3<br/>Architecture gates"]
    end

    subgraph NewTests ["★ NEW TEST FILES"]
        direction LR
        TGATE["★ test_analyze_prs_gates.py<br/>━━━━━━━━━━<br/>PR gate analysis"]
        TFIDELITY["★ test_review_pr_fidelity.py<br/>━━━━━━━━━━<br/>Fidelity checks"]
        TRELEASE["★ test_release_workflows.py<br/>━━━━━━━━━━<br/>CI workflow tests"]
        TDRY["★ test_dry_walkthrough_contracts.py<br/>━━━━━━━━━━<br/>10 contract assertions"]
    end

    subgraph CI ["CI/CD WORKFLOWS"]
        direction TB
        TESTS_CI["tests.yml<br/>━━━━━━━━━━<br/>ubuntu + macos matrix<br/>preflight: uv lock --check<br/>uv sync + task test-all"]
        VBUMP["★ version-bump.yml<br/>━━━━━━━━━━<br/>integration→main merged<br/>MAJOR.MINOR.PATCH+1<br/>sync main→integration"]
        RELEASE["★ release.yml<br/>━━━━━━━━━━<br/>stable branch merge<br/>MAJOR.MINOR+1.0<br/>git tag + GitHub Release"]
    end

    subgraph EntryPoints ["ENTRY POINTS"]
        direction LR
        CLI_EP["autoskillit CLI<br/>━━━━━━━━━━<br/>autoskillit.cli:main<br/>● app.py modified"]
        INSTALL["★ install.sh<br/>━━━━━━━━━━<br/>End-user installer<br/>uv tool install<br/>autoskillit install"]
    end

    %% FLOW %%
    SRC --> PYPROJECT
    TESTS --> PYPROJECT
    PYPROJECT --> TASKFILE
    PYPROJECT --> UVLOCK

    TASKFILE --> RUFF_FMT
    RUFF_FMT --> RUFF_LINT
    RUFF_LINT --> MYPY
    MYPY --> UVCHECK
    UVCHECK --> NOGEN
    NOGEN --> GITLEAKS

    TASKFILE --> PYTEST
    PYTEST --> FIXTURES
    PYTEST --> IMPORTLINT
    PYTEST --> TGATE
    PYTEST --> TFIDELITY
    PYTEST --> TRELEASE
    PYTEST --> TDRY

    PYPROJECT --> TESTS_CI
    TESTS_CI --> VBUMP
    VBUMP --> RELEASE

    PYPROJECT --> CLI_EP
    INSTALL --> CLI_EP

    %% CLASS ASSIGNMENTS %%
    class SRC,TESTS cli;
    class PYPROJECT,TASKFILE,UVLOCK phase;
    class RUFF_FMT,RUFF_LINT,MYPY,UVCHECK,NOGEN,GITLEAKS detector;
    class PYTEST,FIXTURES,IMPORTLINT handler;
    class TGATE,TFIDELITY,TRELEASE,TDRY,VBUMP,RELEASE,INSTALL newComponent;
    class TESTS_CI stateNode;
    class CLI_EP output;
Loading

Process Flow Diagram

%%{init: {'flowchart': {'nodeSpacing': 40, 'rankSpacing': 50, 'curve': 'basis'}}}%%
flowchart TB
    %% CLASS DEFINITIONS %%
    classDef terminal fill:#1a237e,stroke:#7986cb,stroke-width:2px,color:#fff;
    classDef stateNode fill:#004d40,stroke:#4db6ac,stroke-width:2px,color:#fff;
    classDef handler fill:#e65100,stroke:#ffb74d,stroke-width:2px,color:#fff;
    classDef phase fill:#6a1b9a,stroke:#ba68c8,stroke-width:2px,color:#fff;
    classDef newComponent fill:#2e7d32,stroke:#81c784,stroke-width:2px,color:#fff;
    classDef detector fill:#b71c1c,stroke:#ef5350,stroke-width:2px,color:#fff;
    classDef integration fill:#c62828,stroke:#ef9a9a,stroke-width:2px,color:#fff;

    %% TERMINALS %%
    START([START])
    SUCCESS([SUCCESS])
    VALFAIL([VALIDATION ERROR])

    subgraph Val ["Load-Time Validation"]
        direction LR
        ValidateRecipe["validate_recipe<br/>━━━━━━━━━━<br/>recipe/_api.py"]
        RulesRecipe["★ rules_recipe.py<br/>━━━━━━━━━━<br/>unknown-sub-recipe rule"]
        AvailCtx["★ available_recipes<br/>━━━━━━━━━━<br/>ValidationContext field"]
        NameKnown{"sub-recipe<br/>name known?"}
    end

    subgraph Exec ["Runtime Execution"]
        direction TB
        ToolHandler["★ run_recipe<br/>━━━━━━━━━━<br/>server/tools_recipe.py"]
        FindRecipe["★ _find_recipe()<br/>━━━━━━━━━━<br/>server/helpers.py"]
        FoundDec{"recipe<br/>file found?"}
        RunSub["★ _run_subrecipe_session()<br/>━━━━━━━━━━<br/>server/helpers.py"]
        BuildPrompt["★ build_subrecipe_prompt()<br/>━━━━━━━━━━<br/>cli/_prompts.py"]
        BuildCmd["★ build_subrecipe_cmd()<br/>━━━━━━━━━━<br/>execution/commands.py"]
        SubSess["★ run_subrecipe_session()<br/>━━━━━━━━━━<br/>execution/headless.py"]
    end

    HeadlessProc["Headless Claude<br/>━━━━━━━━━━<br/>AUTOSKILLIT_KITCHEN_OPEN=1<br/>executes sub-recipe YAML"]
    ResultDec{"success: True?"}

    START -->|"recipe YAML loaded"| ValidateRecipe
    ValidateRecipe --> RulesRecipe
    RulesRecipe --> AvailCtx
    AvailCtx --> NameKnown
    NameKnown -->|"unknown name"| VALFAIL
    NameKnown -->|"valid"| ToolHandler
    ToolHandler --> FindRecipe
    FindRecipe --> FoundDec
    FoundDec -->|"not found"| SUCCESS
    FoundDec -->|"found"| RunSub
    RunSub --> BuildPrompt
    RunSub --> BuildCmd
    BuildPrompt -->|"prompt str"| SubSess
    BuildCmd -->|"CLI cmd + env"| SubSess
    SubSess --> HeadlessProc
    HeadlessProc --> ResultDec
    ResultDec -->|"yes"| SUCCESS
    ResultDec -->|"no"| SUCCESS

    %% CLASS ASSIGNMENTS %%
    class START,SUCCESS,VALFAIL terminal;
    class ValidateRecipe handler;
    class RulesRecipe detector;
    class AvailCtx,NameKnown,FoundDec stateNode;
    class ToolHandler,FindRecipe,RunSub,BuildPrompt,BuildCmd,SubSess newComponent;
    class HeadlessProc integration;
    class ResultDec stateNode;
Loading

Deployment Diagram

%%{init: {'flowchart': {'nodeSpacing': 50, 'rankSpacing': 60, 'curve': 'basis'}}}%%
flowchart TB
    %% CLASS DEFINITIONS %%
    classDef cli fill:#1a237e,stroke:#7986cb,stroke-width:2px,color:#fff;
    classDef stateNode fill:#004d40,stroke:#4db6ac,stroke-width:2px,color:#fff;
    classDef handler fill:#e65100,stroke:#ffb74d,stroke-width:2px,color:#fff;
    classDef phase fill:#6a1b9a,stroke:#ba68c8,stroke-width:2px,color:#fff;
    classDef newComponent fill:#2e7d32,stroke:#81c784,stroke-width:2px,color:#fff;
    classDef output fill:#00695c,stroke:#4db6ac,stroke-width:2px,color:#fff;
    classDef integration fill:#c62828,stroke:#ef9a9a,stroke-width:2px,color:#fff;
    classDef detector fill:#b71c1c,stroke:#ef5350,stroke-width:2px,color:#fff;

    subgraph DevMachine ["DEVELOPER MACHINE"]
        direction TB

        subgraph Bootstrap ["BOOTSTRAP (★ NEW)"]
            INSTALL_SH["★ install.sh<br/>━━━━━━━━━━<br/>curl from GitHub stable<br/>Python 3.11+ · uv · claude<br/>uv tool install"]
        end

        subgraph UVTool ["INSTALLED PACKAGE"]
            PKG["autoskillit package<br/>━━━━━━━━━━<br/>~/.local/share/uv/tools/<br/>autoskillit/lib/python3.x/<br/>site-packages/autoskillit/"]
            PLUGIN_CACHE["Plugin cache<br/>━━━━━━━━━━<br/>~/.claude/plugins/cache/<br/>autoskillit-local/"]
        end

        subgraph ClaudeCode ["CLAUDE CODE PROCESS"]
            CC["Claude Code IDE<br/>━━━━━━━━━━<br/>Reads hooks from<br/>~/.claude/settings.json<br/>.claude/settings.json"]
            MCP["● MCP Server (stdio)<br/>━━━━━━━━━━<br/>FastMCP · stdin/stdout<br/>12 ungated + 26 kitchen tools<br/>★ run_recipe · ★ get_ci_status"]
        end

        subgraph HeadlessSessions ["HEADLESS SUBPROCESS SESSIONS (pty)"]
            SKILL_SESS["Skill session<br/>━━━━━━━━━━<br/>claude --print prompt<br/>AUTOSKILLIT_HEADLESS=1<br/>KITCHEN_OPEN via open_kitchen"]
            SUBRECIPE["★ Sub-recipe session<br/>━━━━━━━━━━<br/>claude --print sous-chef-prompt<br/>AUTOSKILLIT_KITCHEN_OPEN=1<br/>NO HEADLESS flag<br/>build_subrecipe_cmd"]
        end

        subgraph LocalStorage ["LOCAL STORAGE"]
            SESSION_LOGS[("Session logs<br/>━━━━━━━━━━<br/>~/.local/share/autoskillit/<br/>logs/sessions/*.jsonl<br/>proc_trace · anomalies")]
            CRASH_TMP[("Crash traces<br/>━━━━━━━━━━<br/>/dev/shm/<br/>autoskillit_trace_pid.jsonl<br/>Linux tmpfs")]
            PROJ_STORE[("Project storage<br/>━━━━━━━━━━<br/>project/.autoskillit/<br/>config.yaml · .secrets.yaml<br/>temp/ · recipes/")]
            CLAUDE_LOGS[("Claude Code logs<br/>━━━━━━━━━━<br/>~/.claude/projects/<br/>encoded-cwd/<br/>session-id.jsonl")]
        end
    end

    subgraph GitHub ["GITHUB INFRASTRUCTURE"]
        direction TB
        GH_ACTIONS["GitHub Actions<br/>━━━━━━━━━━<br/>tests.yml: ubuntu + macos-15<br/>uv sync · task test-all"]
        VBUMP_WF["★ version-bump.yml<br/>━━━━━━━━━━<br/>integration→main merge<br/>patch bump + uv lock<br/>sync main→integration"]
        RELEASE_WF["★ release.yml<br/>━━━━━━━━━━<br/>stable branch merge<br/>minor bump + uv lock<br/>git tag + GitHub Release"]
        GH_REPO["GitHub Repository<br/>━━━━━━━━━━<br/>main · integration · stable<br/>Issues · PRs · Releases"]
        GH_API["GitHub API<br/>━━━━━━━━━━<br/>api.github.com<br/>Actions runs · CI jobs<br/>Issues · PR reviews"]
    end

    subgraph Anthropic ["EXTERNAL: ANTHROPIC"]
        ANT_API["Anthropic API<br/>━━━━━━━━━━<br/>api.anthropic.com<br/>/api/oauth/usage<br/>5-hour quota check"]
    end

    %% BOOTSTRAP FLOW %%
    INSTALL_SH -->|"uv tool install git@stable"| PKG
    PKG -->|"autoskillit install<br/>claude plugin install"| PLUGIN_CACHE

    %% CLAUDE CODE %%
    PLUGIN_CACHE -->|"plugin load"| CC
    CC -->|"spawns stdio"| MCP

    %% MCP → SESSIONS %%
    MCP -->|"run_skill: spawns pty"| SKILL_SESS
    MCP -->|"★ run_recipe: spawns pty<br/>KITCHEN_OPEN=1"| SUBRECIPE

    %% STORAGE WRITES %%
    SKILL_SESS -->|"writes diagnostics"| SESSION_LOGS
    SKILL_SESS -->|"crash: writes"| CRASH_TMP
    SUBRECIPE -->|"writes diagnostics"| SESSION_LOGS
    MCP -->|"reads/writes"| PROJ_STORE
    CC -->|"writes JSONL"| CLAUDE_LOGS

    %% CI/RELEASE %%
    GH_REPO -->|"push/PR event"| GH_ACTIONS
    GH_REPO -->|"integration→main merge"| VBUMP_WF
    GH_REPO -->|"★ PR→stable merge"| RELEASE_WF
    RELEASE_WF -->|"creates"| GH_REPO
    VBUMP_WF -->|"pushes commits"| GH_REPO

    %% EXTERNAL API CALLS %%
    MCP -->|"httpx HTTPS<br/>CI polling"| GH_API
    GH_ACTIONS -->|"gh CLI"| GH_API
    MCP -->|"httpx HTTPS<br/>quota check"| ANT_API
    SKILL_SESS -->|"Anthropic API<br/>inference"| ANT_API

    %% CLASS ASSIGNMENTS %%
    class INSTALL_SH,VBUMP_WF,RELEASE_WF,SUBRECIPE newComponent;
    class CC,SKILL_SESS cli;
    class MCP,GH_ACTIONS handler;
    class PKG,PLUGIN_CACHE phase;
    class SESSION_LOGS,CRASH_TMP,PROJ_STORE,CLAUDE_LOGS stateNode;
    class GH_REPO,GH_API,ANT_API integration;
Loading

Closes #307
Closes #302
Closes #298
Closes #297
Closes #300
Closes #303

🤖 Generated with Claude Code via AutoSkillit

Auto-generated conflict resolution branch. Closes after merge.

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
@Trecek Trecek force-pushed the pr-batch/pr-merge-20260310-163009 branch from 6ab9c9a to 32d2baa Compare March 11, 2026 01:03
@Trecek Trecek merged commit 44c447d into integration Mar 11, 2026
2 checks passed
@Trecek Trecek deleted the pr-batch/pr-merge-20260310-163009 branch March 11, 2026 01:47
Trecek added a commit that referenced this pull request Mar 11, 2026
…nto integration (#323)

## Integration Summary

Collapsed 7 PRs into `pr-batch/pr-merge-20260310-163009` targeting
`integration`.

## Merged PRs

| # | Title | Complexity | Additions | Deletions | Overlaps |
|---|-------|-----------|-----------|-----------|---------|
| #319 | Add Step 4.5 Historical Regression Check to dry-walkthrough
skill | simple | +200 | -0 | — |
| #318 | Implementation Plan: Pipeline Observability — Quota Guard
Logging and Per-Step Elapsed Time | simple | +267 | -23 | — |
| #316 | Release CI Automation — Version Bump, Branch Sync, and Release
Infrastructure | simple | +471 | -0 | — |
| #314 | docs: complete release documentation sprint | simple | +1344 |
-104 | — |
| #315 | Recipe remediation — source_dir resolution and cook command
display | simple | +317 | -71 | #320 |
| #317 | Implementation Plan: PR Review Pipeline Gates — Fidelity
Checks, CI Gating, and Review-First Enforcement | simple | +1057 | -7 |
#320 |
| #320 | feat: add recipe composition with run_recipe tool and
dev-sprint consumer | needs_check | +724 | -19 | #315, #317 |

## Audit

**Verdict:** GO

## Architecture Impact

### Development Diagram

```mermaid
%%{init: {'flowchart': {'nodeSpacing': 50, 'rankSpacing': 60, 'curve': 'basis'}}}%%
flowchart TB
    %% CLASS DEFINITIONS %%
    classDef cli fill:#1a237e,stroke:#7986cb,stroke-width:2px,color:#fff;
    classDef stateNode fill:#004d40,stroke:#4db6ac,stroke-width:2px,color:#fff;
    classDef handler fill:#e65100,stroke:#ffb74d,stroke-width:2px,color:#fff;
    classDef phase fill:#6a1b9a,stroke:#ba68c8,stroke-width:2px,color:#fff;
    classDef newComponent fill:#2e7d32,stroke:#81c784,stroke-width:2px,color:#fff;
    classDef output fill:#00695c,stroke:#4db6ac,stroke-width:2px,color:#fff;
    classDef detector fill:#b71c1c,stroke:#ef5350,stroke-width:2px,color:#fff;
    classDef terminal fill:#1a237e,stroke:#7986cb,stroke-width:2px,color:#fff;

    subgraph Structure ["PROJECT STRUCTURE"]
        direction LR
        SRC["src/autoskillit/<br/>━━━━━━━━━━<br/>105 source files<br/>10 sub-packages"]
        TESTS["tests/<br/>━━━━━━━━━━<br/>170 test files<br/>Mirrors src layout"]
    end

    subgraph Build ["BUILD TOOLING"]
        direction LR
        PYPROJECT["● pyproject.toml<br/>━━━━━━━━━━<br/>hatchling backend<br/>uv package manager"]
        TASKFILE["Taskfile.yml<br/>━━━━━━━━━━<br/>task test-all<br/>task test-check<br/>task install-worktree"]
        UVLOCK["uv.lock<br/>━━━━━━━━━━<br/>Locked manifest"]
    end

    subgraph PreCommit ["PRE-COMMIT HOOKS"]
        direction TB
        RUFF_FMT["ruff-format<br/>━━━━━━━━━━<br/>Auto-format<br/>(writes source)"]
        RUFF_LINT["ruff check<br/>━━━━━━━━━━<br/>Lint --fix<br/>target: py311"]
        MYPY["mypy<br/>━━━━━━━━━━<br/>src/ type check<br/>--ignore-missing"]
        UVCHECK["uv lock --check<br/>━━━━━━━━━━<br/>Lockfile guard"]
        NOGEN["no-generated-configs<br/>━━━━━━━━━━<br/>Blocks hooks.json<br/>recipes/diagrams/"]
        GITLEAKS["gitleaks v8.30.0<br/>━━━━━━━━━━<br/>Secret scanning"]
    end

    subgraph Testing ["TEST FRAMEWORK"]
        direction LR
        PYTEST["pytest -n 4<br/>━━━━━━━━━━<br/>asyncio_mode=auto<br/>timeout=60s"]
        FIXTURES["conftest.py<br/>━━━━━━━━━━<br/>StatefulMockTester<br/>MockSubprocessRunner<br/>tool_ctx fixture"]
        IMPORTLINT["import-linter<br/>━━━━━━━━━━<br/>L0→L1→L2→L3<br/>Architecture gates"]
    end

    subgraph NewTests ["★ NEW TEST FILES"]
        direction LR
        TGATE["★ test_analyze_prs_gates.py<br/>━━━━━━━━━━<br/>PR gate analysis"]
        TFIDELITY["★ test_review_pr_fidelity.py<br/>━━━━━━━━━━<br/>Fidelity checks"]
        TRELEASE["★ test_release_workflows.py<br/>━━━━━━━━━━<br/>CI workflow tests"]
        TDRY["★ test_dry_walkthrough_contracts.py<br/>━━━━━━━━━━<br/>10 contract assertions"]
    end

    subgraph CI ["CI/CD WORKFLOWS"]
        direction TB
        TESTS_CI["tests.yml<br/>━━━━━━━━━━<br/>ubuntu + macos matrix<br/>preflight: uv lock --check<br/>uv sync + task test-all"]
        VBUMP["★ version-bump.yml<br/>━━━━━━━━━━<br/>integration→main merged<br/>MAJOR.MINOR.PATCH+1<br/>sync main→integration"]
        RELEASE["★ release.yml<br/>━━━━━━━━━━<br/>stable branch merge<br/>MAJOR.MINOR+1.0<br/>git tag + GitHub Release"]
    end

    subgraph EntryPoints ["ENTRY POINTS"]
        direction LR
        CLI_EP["autoskillit CLI<br/>━━━━━━━━━━<br/>autoskillit.cli:main<br/>● app.py modified"]
        INSTALL["★ install.sh<br/>━━━━━━━━━━<br/>End-user installer<br/>uv tool install<br/>autoskillit install"]
    end

    %% FLOW %%
    SRC --> PYPROJECT
    TESTS --> PYPROJECT
    PYPROJECT --> TASKFILE
    PYPROJECT --> UVLOCK

    TASKFILE --> RUFF_FMT
    RUFF_FMT --> RUFF_LINT
    RUFF_LINT --> MYPY
    MYPY --> UVCHECK
    UVCHECK --> NOGEN
    NOGEN --> GITLEAKS

    TASKFILE --> PYTEST
    PYTEST --> FIXTURES
    PYTEST --> IMPORTLINT
    PYTEST --> TGATE
    PYTEST --> TFIDELITY
    PYTEST --> TRELEASE
    PYTEST --> TDRY

    PYPROJECT --> TESTS_CI
    TESTS_CI --> VBUMP
    VBUMP --> RELEASE

    PYPROJECT --> CLI_EP
    INSTALL --> CLI_EP

    %% CLASS ASSIGNMENTS %%
    class SRC,TESTS cli;
    class PYPROJECT,TASKFILE,UVLOCK phase;
    class RUFF_FMT,RUFF_LINT,MYPY,UVCHECK,NOGEN,GITLEAKS detector;
    class PYTEST,FIXTURES,IMPORTLINT handler;
    class TGATE,TFIDELITY,TRELEASE,TDRY,VBUMP,RELEASE,INSTALL newComponent;
    class TESTS_CI stateNode;
    class CLI_EP output;
```

### Process Flow Diagram

```mermaid
%%{init: {'flowchart': {'nodeSpacing': 40, 'rankSpacing': 50, 'curve': 'basis'}}}%%
flowchart TB
    %% CLASS DEFINITIONS %%
    classDef terminal fill:#1a237e,stroke:#7986cb,stroke-width:2px,color:#fff;
    classDef stateNode fill:#004d40,stroke:#4db6ac,stroke-width:2px,color:#fff;
    classDef handler fill:#e65100,stroke:#ffb74d,stroke-width:2px,color:#fff;
    classDef phase fill:#6a1b9a,stroke:#ba68c8,stroke-width:2px,color:#fff;
    classDef newComponent fill:#2e7d32,stroke:#81c784,stroke-width:2px,color:#fff;
    classDef detector fill:#b71c1c,stroke:#ef5350,stroke-width:2px,color:#fff;
    classDef integration fill:#c62828,stroke:#ef9a9a,stroke-width:2px,color:#fff;

    %% TERMINALS %%
    START([START])
    SUCCESS([SUCCESS])
    VALFAIL([VALIDATION ERROR])

    subgraph Val ["Load-Time Validation"]
        direction LR
        ValidateRecipe["validate_recipe<br/>━━━━━━━━━━<br/>recipe/_api.py"]
        RulesRecipe["★ rules_recipe.py<br/>━━━━━━━━━━<br/>unknown-sub-recipe rule"]
        AvailCtx["★ available_recipes<br/>━━━━━━━━━━<br/>ValidationContext field"]
        NameKnown{"sub-recipe<br/>name known?"}
    end

    subgraph Exec ["Runtime Execution"]
        direction TB
        ToolHandler["★ run_recipe<br/>━━━━━━━━━━<br/>server/tools_recipe.py"]
        FindRecipe["★ _find_recipe()<br/>━━━━━━━━━━<br/>server/helpers.py"]
        FoundDec{"recipe<br/>file found?"}
        RunSub["★ _run_subrecipe_session()<br/>━━━━━━━━━━<br/>server/helpers.py"]
        BuildPrompt["★ build_subrecipe_prompt()<br/>━━━━━━━━━━<br/>cli/_prompts.py"]
        BuildCmd["★ build_subrecipe_cmd()<br/>━━━━━━━━━━<br/>execution/commands.py"]
        SubSess["★ run_subrecipe_session()<br/>━━━━━━━━━━<br/>execution/headless.py"]
    end

    HeadlessProc["Headless Claude<br/>━━━━━━━━━━<br/>AUTOSKILLIT_KITCHEN_OPEN=1<br/>executes sub-recipe YAML"]
    ResultDec{"success: True?"}

    START -->|"recipe YAML loaded"| ValidateRecipe
    ValidateRecipe --> RulesRecipe
    RulesRecipe --> AvailCtx
    AvailCtx --> NameKnown
    NameKnown -->|"unknown name"| VALFAIL
    NameKnown -->|"valid"| ToolHandler
    ToolHandler --> FindRecipe
    FindRecipe --> FoundDec
    FoundDec -->|"not found"| SUCCESS
    FoundDec -->|"found"| RunSub
    RunSub --> BuildPrompt
    RunSub --> BuildCmd
    BuildPrompt -->|"prompt str"| SubSess
    BuildCmd -->|"CLI cmd + env"| SubSess
    SubSess --> HeadlessProc
    HeadlessProc --> ResultDec
    ResultDec -->|"yes"| SUCCESS
    ResultDec -->|"no"| SUCCESS

    %% CLASS ASSIGNMENTS %%
    class START,SUCCESS,VALFAIL terminal;
    class ValidateRecipe handler;
    class RulesRecipe detector;
    class AvailCtx,NameKnown,FoundDec stateNode;
    class ToolHandler,FindRecipe,RunSub,BuildPrompt,BuildCmd,SubSess newComponent;
    class HeadlessProc integration;
    class ResultDec stateNode;
```

### Deployment Diagram

```mermaid
%%{init: {'flowchart': {'nodeSpacing': 50, 'rankSpacing': 60, 'curve': 'basis'}}}%%
flowchart TB
    %% CLASS DEFINITIONS %%
    classDef cli fill:#1a237e,stroke:#7986cb,stroke-width:2px,color:#fff;
    classDef stateNode fill:#004d40,stroke:#4db6ac,stroke-width:2px,color:#fff;
    classDef handler fill:#e65100,stroke:#ffb74d,stroke-width:2px,color:#fff;
    classDef phase fill:#6a1b9a,stroke:#ba68c8,stroke-width:2px,color:#fff;
    classDef newComponent fill:#2e7d32,stroke:#81c784,stroke-width:2px,color:#fff;
    classDef output fill:#00695c,stroke:#4db6ac,stroke-width:2px,color:#fff;
    classDef integration fill:#c62828,stroke:#ef9a9a,stroke-width:2px,color:#fff;
    classDef detector fill:#b71c1c,stroke:#ef5350,stroke-width:2px,color:#fff;

    subgraph DevMachine ["DEVELOPER MACHINE"]
        direction TB

        subgraph Bootstrap ["BOOTSTRAP (★ NEW)"]
            INSTALL_SH["★ install.sh<br/>━━━━━━━━━━<br/>curl from GitHub stable<br/>Python 3.11+ · uv · claude<br/>uv tool install"]
        end

        subgraph UVTool ["INSTALLED PACKAGE"]
            PKG["autoskillit package<br/>━━━━━━━━━━<br/>~/.local/share/uv/tools/<br/>autoskillit/lib/python3.x/<br/>site-packages/autoskillit/"]
            PLUGIN_CACHE["Plugin cache<br/>━━━━━━━━━━<br/>~/.claude/plugins/cache/<br/>autoskillit-local/"]
        end

        subgraph ClaudeCode ["CLAUDE CODE PROCESS"]
            CC["Claude Code IDE<br/>━━━━━━━━━━<br/>Reads hooks from<br/>~/.claude/settings.json<br/>.claude/settings.json"]
            MCP["● MCP Server (stdio)<br/>━━━━━━━━━━<br/>FastMCP · stdin/stdout<br/>12 ungated + 26 kitchen tools<br/>★ run_recipe · ★ get_ci_status"]
        end

        subgraph HeadlessSessions ["HEADLESS SUBPROCESS SESSIONS (pty)"]
            SKILL_SESS["Skill session<br/>━━━━━━━━━━<br/>claude --print prompt<br/>AUTOSKILLIT_HEADLESS=1<br/>KITCHEN_OPEN via open_kitchen"]
            SUBRECIPE["★ Sub-recipe session<br/>━━━━━━━━━━<br/>claude --print sous-chef-prompt<br/>AUTOSKILLIT_KITCHEN_OPEN=1<br/>NO HEADLESS flag<br/>build_subrecipe_cmd"]
        end

        subgraph LocalStorage ["LOCAL STORAGE"]
            SESSION_LOGS[("Session logs<br/>━━━━━━━━━━<br/>~/.local/share/autoskillit/<br/>logs/sessions/*.jsonl<br/>proc_trace · anomalies")]
            CRASH_TMP[("Crash traces<br/>━━━━━━━━━━<br/>/dev/shm/<br/>autoskillit_trace_pid.jsonl<br/>Linux tmpfs")]
            PROJ_STORE[("Project storage<br/>━━━━━━━━━━<br/>project/.autoskillit/<br/>config.yaml · .secrets.yaml<br/>temp/ · recipes/")]
            CLAUDE_LOGS[("Claude Code logs<br/>━━━━━━━━━━<br/>~/.claude/projects/<br/>encoded-cwd/<br/>session-id.jsonl")]
        end
    end

    subgraph GitHub ["GITHUB INFRASTRUCTURE"]
        direction TB
        GH_ACTIONS["GitHub Actions<br/>━━━━━━━━━━<br/>tests.yml: ubuntu + macos-15<br/>uv sync · task test-all"]
        VBUMP_WF["★ version-bump.yml<br/>━━━━━━━━━━<br/>integration→main merge<br/>patch bump + uv lock<br/>sync main→integration"]
        RELEASE_WF["★ release.yml<br/>━━━━━━━━━━<br/>stable branch merge<br/>minor bump + uv lock<br/>git tag + GitHub Release"]
        GH_REPO["GitHub Repository<br/>━━━━━━━━━━<br/>main · integration · stable<br/>Issues · PRs · Releases"]
        GH_API["GitHub API<br/>━━━━━━━━━━<br/>api.github.com<br/>Actions runs · CI jobs<br/>Issues · PR reviews"]
    end

    subgraph Anthropic ["EXTERNAL: ANTHROPIC"]
        ANT_API["Anthropic API<br/>━━━━━━━━━━<br/>api.anthropic.com<br/>/api/oauth/usage<br/>5-hour quota check"]
    end

    %% BOOTSTRAP FLOW %%
    INSTALL_SH -->|"uv tool install git@stable"| PKG
    PKG -->|"autoskillit install<br/>claude plugin install"| PLUGIN_CACHE

    %% CLAUDE CODE %%
    PLUGIN_CACHE -->|"plugin load"| CC
    CC -->|"spawns stdio"| MCP

    %% MCP → SESSIONS %%
    MCP -->|"run_skill: spawns pty"| SKILL_SESS
    MCP -->|"★ run_recipe: spawns pty<br/>KITCHEN_OPEN=1"| SUBRECIPE

    %% STORAGE WRITES %%
    SKILL_SESS -->|"writes diagnostics"| SESSION_LOGS
    SKILL_SESS -->|"crash: writes"| CRASH_TMP
    SUBRECIPE -->|"writes diagnostics"| SESSION_LOGS
    MCP -->|"reads/writes"| PROJ_STORE
    CC -->|"writes JSONL"| CLAUDE_LOGS

    %% CI/RELEASE %%
    GH_REPO -->|"push/PR event"| GH_ACTIONS
    GH_REPO -->|"integration→main merge"| VBUMP_WF
    GH_REPO -->|"★ PR→stable merge"| RELEASE_WF
    RELEASE_WF -->|"creates"| GH_REPO
    VBUMP_WF -->|"pushes commits"| GH_REPO

    %% EXTERNAL API CALLS %%
    MCP -->|"httpx HTTPS<br/>CI polling"| GH_API
    GH_ACTIONS -->|"gh CLI"| GH_API
    MCP -->|"httpx HTTPS<br/>quota check"| ANT_API
    SKILL_SESS -->|"Anthropic API<br/>inference"| ANT_API

    %% CLASS ASSIGNMENTS %%
    class INSTALL_SH,VBUMP_WF,RELEASE_WF,SUBRECIPE newComponent;
    class CC,SKILL_SESS cli;
    class MCP,GH_ACTIONS handler;
    class PKG,PLUGIN_CACHE phase;
    class SESSION_LOGS,CRASH_TMP,PROJ_STORE,CLAUDE_LOGS stateNode;
    class GH_REPO,GH_API,ANT_API integration;
```

Closes #307
Closes #302
Closes #298
Closes #297
Closes #300
Closes #303

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

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Trecek added a commit that referenced this pull request Mar 15, 2026
…, Headless Isolation (#404)

## Summary

Integration rollup of **43 PRs** (#293#406) consolidating **62
commits** across **291 files** (+27,909 / −6,040 lines). This release
advances AutoSkillit from v0.2.0 to v0.3.1 with GitHub merge queue
integration, sub-recipe composition, a PostToolUse output reformatter,
headless session isolation guards, and comprehensive pipeline
observability — plus 24 new bundled skills, 3 new MCP tools, and 47 new
test files.

---

## Major Features

### GitHub Merge Queue Integration (#370, #362, #390)
- New `wait_for_merge_queue` MCP tool — polls a PR through GitHub's
merge queue until merged, ejected, or timed out (default 600s). Uses
REST + GraphQL APIs with stuck-queue detection and auto-merge
re-enrollment
- New `DefaultMergeQueueWatcher` L1 service (`execution/merge_queue.py`)
— never raises; all outcomes are structured results
- `parse_merge_queue_response()` pure function for GraphQL queue entry
parsing
- New `auto_merge` ingredient in `implementation.yaml` and
`remediation.yaml` — enrolls PRs in the merge queue after CI passes
- Full queue-mode path added to `merge-prs.yaml`: detect queue → enqueue
→ wait → handle ejections → re-enter
- `analyze-prs` skill gains Step 0.5 (merge queue detection) and Step
1.5 (CI/review eligibility filtering)

### Sub-Recipe Composition (#380)
- Recipe steps can now reference sub-recipes via `sub_recipe` + `gate`
fields — lazy-loaded and merged at validation time
- Composition engine in `recipe/_api.py`: `_merge_sub_recipe()` inlines
sub-recipe steps with safe name-prefixing and route remapping (`done` →
parent's `on_success`, `escalate` → parent's `on_failure`)
- `_build_active_recipe()` evaluates gate ingredients against
overrides/defaults; dual validation runs on both active and combined
recipes
- First sub-recipe: `sprint-prefix.yaml` — triage → plan → confirm →
dispatch workflow, gated by `sprint_mode` ingredient (hidden, default
false)
- Both `implementation.yaml` and `remediation.yaml` gain `sprint_entry`
placeholder step
- New semantic rules: `unknown-sub-recipe` (ERROR),
`circular-sub-recipe` (ERROR) with DFS cycle detection

### PostToolUse Output Reformatter (#293, #405)
- `pretty_output.py` — new 671-line PostToolUse hook that rewrites raw
MCP JSON responses to Markdown-KV before Claude consumes them (30–77%
token overhead reduction)
- Dedicated formatters for 11 high-traffic tools (`run_skill`,
`run_cmd`, `test_check`, `merge_worktree`, `get_token_summary`, etc.)
plus a generic KV formatter for remaining tools
- Pipeline vs. interactive mode detection via hook config file
- Unwraps Claude Code's `{"result": "<json-string>"}` envelope before
dispatching
- 1,516-line test file with 40+ behavioral tests

### Headless Session Isolation (#359, #393, #397, #405, #406)
- **Env isolation**: `build_sanitized_env()` strips
`AUTOSKILLIT_PRIVATE_ENV_VARS` from subprocess environments, preventing
`AUTOSKILLIT_HEADLESS=1` from leaking into test runners
- **CWD path contamination defense**: `_inject_cwd_anchor()` anchors all
relative paths to session CWD; `_validate_output_paths()` checks
structured output tokens against CWD prefix; `_scan_jsonl_write_paths()`
post-session scanner catches actual Write/Edit/Bash tool calls outside
CWD
- **Headless orchestration guard**: new PreToolUse hook blocks
`run_skill`/`run_cmd`/`run_python` when `AUTOSKILLIT_HEADLESS=1`,
enforcing Tier 1/Tier 2 nesting invariant
- **`_require_not_headless()` server-side guard**: blocks 10
orchestration-only tools from headless sessions at the handler layer
- **Unified error response contract**: `headless_error_result()`
produces consistent 9-field responses;
`_build_headless_error_response()` canonical builder for all failure
paths in `tools_integrations.py`

### Cook UX Overhaul (#375, #363)
- `open_kitchen` now accepts optional `name` + `overrides` — opens
kitchen AND loads recipe in a single call
- Pre-launch terminal preview with ANSI-colored flow diagram and
ingredients table via new `cli/_ansi.py` module
- `--dangerously-skip-permissions` warning banner with interactive
confirmation prompt
- Randomized session greetings from themed pools
- Orchestrator prompt rewritten: recipe YAML no longer injected via
`--append-system-prompt`; session calls `open_kitchen('{recipe_name}')`
as first action
- Conversational ingredient collection replaces mechanical per-field
prompting

---

## New MCP Tools

| Tool | Gate | Description |
|------|------|-------------|
| `wait_for_merge_queue` | Kitchen | Polls PR through GitHub merge queue
(REST + GraphQL) |
| `set_commit_status` | Kitchen | Posts GitHub Commit Status to a SHA
for review-first gating |
| `get_quota_events` | Ungated | Surfaces quota guard decisions from
`quota_events.jsonl` |

---

## Pipeline Observability (#318, #341)

- **`TelemetryFormatter`** (`pipeline/telemetry_fmt.py`) — single source
of truth for all telemetry rendering; replaces dual-formatter
anti-pattern. Four rendering modes: Markdown table, terminal table,
compact KV (for PostToolUse hook)
- `get_token_summary` and `get_timing_summary` gain `format` parameter
(`"json"` | `"table"`)
- `wall_clock_seconds` merged into token summary output — see duration
alongside token counts in one call
- **Telemetry clear marker**: `write_telemetry_clear_marker()` /
`read_telemetry_clear_marker()` prevent token accounting drift on MCP
server restart after `clear=True`
- **Quota event logging**: `quota_check.py` hook now writes structured
JSONL events (`cache_miss`, `parse_error`, `blocked`, `approved`) to
`quota_events.jsonl`

---

## CI Watcher & Remote Resolution Fixes (#395, #406)

- **`CIRunScope` value object** — carries `workflow` + `head_sha` scope;
replaces bare `head_sha` parameter across all CI watcher signatures
- **Workflow filter**: `wait_for_ci` and `get_ci_status` accept
`workflow` parameter (falls back to project-level `config.ci.workflow`),
preventing unrelated workflows (version bumps, labelers) from satisfying
CI checks
- **`FAILED_CONCLUSIONS` expanded**: `failure` → `{failure, timed_out,
startup_failure, cancelled}`
- **Canonical remote resolver** (`execution/remote_resolver.py`):
`resolve_remote_repo()` with `REMOTE_PRECEDENCE = (upstream, origin)` —
correctly resolves `owner/repo` after `clone_repo` sets `origin` to
`file://` isolation URL
- **Clone isolation fix**: `clone_repo` now always clones from remote
URL (never local path); sets `origin=file:///<clone>` for isolation and
`upstream=<real_url>` for push/CI operations

---

## PR Pipeline Gates (#317, #343)

- **`pipeline/pr_gates.py`**: `is_ci_passing()`, `is_review_passing()`,
`partition_prs()` — partitions PRs into
eligible/CI-blocked/review-blocked with human-readable reasons
- **`pipeline/fidelity.py`**: `extract_linked_issues()`
(Closes/Fixes/Resolves patterns), `is_valid_fidelity_finding()` schema
validation
- **`check_pr_mergeable`** now returns `mergeable_status` field
alongside boolean
- **`release_issue`** gains `target_branch` + `staged_label` parameters
for staged issue lifecycle on non-default branches (#392)

---

## Recipe System Changes

### Structural
- `RecipeIngredient.hidden` field — excluded from ingredients table
(used for internal flags like `sprint_mode`)
- `Recipe.experimental` flag parsed from YAML
- `_TERMINAL_TARGETS` moved to `schema.py` as single source of truth
- `format_ingredients_table()` with sorted display order (required →
auto-detect → flags → optional → constants)
- Diagram rendering engine (~670 lines) removed from `diagrams.py` —
rendering now handled by `/render-recipe` skill; format version bumped
to v7

### Recipe YAML Changes
- **Deleted**: `audit-and-fix.yaml`, `batch-implementation.yaml`,
`bugfix-loop.yaml`
- **Renamed**: `pr-merge-pipeline.yaml` → `merge-prs.yaml`
- **`implementation.yaml`**: merge queue steps,
`auto_merge`/`sprint_mode` ingredients, `base_branch` default → `""`
(auto-detect), CI workflow filter, `extract_pr_number` step
- **`remediation.yaml`**: `topic` → `task` rename, merge queue steps,
`dry_walkthrough` retries:3 with forward-only routing, `verify` → `test`
rename
- **`merge-prs.yaml`**: full queue-mode path, `open-integration-pr` step
(replaces `create-review-pr`), post-PR mergeability polling, review
cycle with `resolve-review` retries

### New Semantic Rules
- `missing-output-patterns` (WARNING) — flags `run_skill` steps without
`expected_output_patterns`
- `unknown-sub-recipe` (ERROR) — validates sub-recipe references exist
- `circular-sub-recipe` (ERROR) — DFS cycle detection
- `unknown-skill-command` (ERROR) — validates skill names against
bundled set
- `telemetry-before-open-pr` (WARNING) — ensures telemetry step precedes
`open-pr`

---

## New Skills (24)

### Architecture Lens Family (13)
`arch-lens-c4-container`, `arch-lens-concurrency`,
`arch-lens-data-lineage`, `arch-lens-deployment`,
`arch-lens-development`, `arch-lens-error-resilience`,
`arch-lens-module-dependency`, `arch-lens-operational`,
`arch-lens-process-flow`, `arch-lens-repository-access`,
`arch-lens-scenarios`, `arch-lens-security`, `arch-lens-state-lifecycle`

### Audit Family (5)
`audit-arch`, `audit-bugs`, `audit-cohesion`, `audit-defense-standards`,
`audit-tests`

### Planning & Diagramming (3)
`elaborate-phase`, `make-arch-diag`, `make-req`

### Bug/Guard Lifecycle (2)
`design-guards`, `verify-diag`

### Pipeline (1)
`open-integration-pr` — creates integration PRs with per-PR details,
arch-lens diagrams, carried-forward `Closes #N` references, and
auto-closes collapsed PRs

### Sprint Planning (1 — gated by sub-recipe)
`sprint-planner` — selects a focused, conflict-free sprint from a triage
manifest

---

## Skill Modifications (Highlights)

- **`analyze-prs`**: merge queue detection, CI/review eligibility
filtering, queue-mode ordering
- **`dry-walkthrough`**: Step 4.5 Historical Regression Check (git
history mining + GitHub issue cross-reference)
- **`review-pr`**: deterministic diff annotation via
`diff_annotator.py`, echo-primary-obligation step, post-completion
confirmation, degraded-mode narration
- **`collapse-issues`**: content fidelity enforcement — per-issue
`fetch_github_issue` calls, copy-mode body assembly (#388)
- **`prepare-issue`**: multi-keyword dedup search, numbered candidate
selection, extend-existing-issue flow
- **`resolve-review`**: GraphQL thread auto-resolution after addressing
findings (#379)
- **`resolve-merge-conflicts`**: conflict resolution decision report
with per-file log (#389)
- **Cross-skill**: output tokens migrated to `key = value` format;
code-index paths made generic with fallback notes; arch-lens references
fully qualified; anti-prose guards at loop boundaries

---

## CLI & Hooks

### New CLI Commands
- `autoskillit install` — plugin installation + cache refresh
- `autoskillit upgrade` — `.autoskillit/scripts/` →
`.autoskillit/recipes/` migration

### CLI Changes
- `doctor`: plugin-aware MCP check, PostToolUse hook scanning, `--fix`
flag removed
- `init`: GitHub repo prompt, `.secrets.yaml` template, plugin-aware
registration
- `chefs-hat`: pre-launch banner, `--dangerously-skip-permissions`
confirmation
- `recipes render`: repurposed from generator to viewer (delegates to
`/render-recipe`)
- `serve`: server import deferred to after `configure_logging()` to
prevent stdout corruption

### New Hooks
- `branch_protection_guard.py` (PreToolUse) — denies
`merge_worktree`/`push_to_remote` targeting protected branches
- `headless_orchestration_guard.py` (PreToolUse) — blocks orchestration
tools in headless sessions
- `pretty_output.py` (PostToolUse) — MCP JSON → Markdown-KV reformatter

### Hook Infrastructure
- `HookDef.event_type` field — registry now handles both PreToolUse and
PostToolUse
- `generate_hooks_json()` groups entries by event type
- `_evict_stale_autoskillit_hooks` and `sync_hooks_to_settings` made
event-type-agnostic

---

## Core & Config

### New Core Modules
- `core/branch_guard.py` — `is_protected_branch()` pure function
- `core/github_url.py` — `parse_github_repo()` +
`normalize_owner_repo()` canonical parsers

### Core Type Expansions
- `AUTOSKILLIT_PRIVATE_ENV_VARS` frozenset
- `WORKER_TOOLS` / `HEADLESS_BLOCKED_UNGATED_TOOLS` split from
`UNGATED_TOOLS`
- `TOOL_CATEGORIES` — categorized listing for `open_kitchen` response
- `CIRunScope` — immutable scope for CI watcher calls
- `MergeQueueWatcher` protocol
- `SkillResult.cli_subtype` + `write_path_warnings` fields
- `SubprocessRunner.env` parameter

### Config
- `safety.protected_branches`: `[main, integration, stable]`
- `github.staged_label`: `"staged"`
- `ci.workflow`: workflow filename filter (e.g., `"tests.yml"`)
- `branching.default_base_branch`: `"integration"` → `"main"`
- `ModelConfig.default`: `str | None` → `str = "sonnet"`

---

## Infrastructure & Release

### Version
- `0.2.0` → `0.3.1` across `pyproject.toml`, `plugin.json`, `uv.lock`
- FastMCP dependency: `>=3.0.2` → `>=3.1.1,<4.0` (#399)

### CI/CD Workflows
- **`version-bump.yml`** (new) — auto patch-bumps `main` on integration
PR merge, force-syncs integration branch one patch ahead
- **`release.yml`** (new) — minor version bump + GitHub Release on merge
to `stable`
- **`codeql.yml`** (new) — CodeQL analysis for `stable` PRs (Python +
Actions)
- **`tests.yml`** — `merge_group:` trigger added; multi-OS now only for
`stable`

### PyPI Readiness
- `pyproject.toml`: `readme`, `license`, `authors`, `keywords`,
`classifiers`, `project.urls`, `hatch.build.targets.sdist` inclusion
list

### readOnlyHint Parallel Execution Fix
- All MCP tools annotated `readOnlyHint=True` — enables Claude Code
parallel tool execution (~7x speedup). One deliberate exception:
`wait_for_merge_queue` uses `readOnlyHint=False` (actually mutates queue
state)

### Tool Response Exception Boundary
- `track_response_size` decorator catches unhandled exceptions and
serializes them as `{"success": false, "subtype": "tool_exception"}` —
prevents FastMCP opaque error wrapping

### SkillResult Subtype Normalization (#358)
- `_normalize_subtype()` gate eliminates dual-source contradiction
between CLI subtype and session outcome
- Class 2 upward: `SUCCEEDED + error_subtype → "success"` (drain-race
artifact)
- Class 1 downward: `non-SUCCEEDED + "success" → "empty_result"` /
`"missing_completion_marker"` / `"adjudicated_failure"`

---

## Test Coverage

**47 new test files** (+12,703 lines) covering:

| Area | Key Tests |
|------|-----------|
| Merge queue watcher state machine | `test_merge_queue.py` (226 lines)
|
| Clone isolation × CI resolution | `test_clone_ci_contract.py`,
`test_remote_resolver.py` |
| PostToolUse hook | `test_pretty_output.py` (1,516 lines, 40+ cases) |
| Branch protection + headless guards |
`test_branch_protection_guard.py`,
`test_headless_orchestration_guard.py` |
| Sub-recipe composition | 5 test files (schema, loading, validation,
sprint mode × 2) |
| Telemetry formatter | `test_telemetry_formatter.py` (281 lines) |
| PR pipeline gates | `test_analyze_prs_gates.py`,
`test_review_pr_fidelity.py` |
| Diff annotator | `test_diff_annotator.py` (242 lines) |
| Skill compliance | Output token format, genericization, loop-boundary
guards |
| Release workflows | Structural contracts for `version-bump.yml`,
`release.yml` |
| Issue content fidelity | Body-assembling skills must call
`fetch_github_issue` per-issue |
| CI watcher scope | `test_ci_params.py` — workflow_id query param
composition |

---

## Consolidated PRs

#293, #295, #314, #315, #316, #317, #318, #319, #323, #332, #336, #337,
#338, #339, #341, #343, #351, #358, #359, #360, #361, #362, #363, #366,
#368, #370, #375, #377, #378, #379, #380, #388, #389, #390, #391, #392,
#393, #395, #396, #397, #399, #405, #406

---

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

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant