devflow-kit 1.2.0 → 1.3.0

package/CHANGELOG.md

@@ -5,7 +5,35 @@ All notable changes to DevFlow will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [Unreleased]
+## [1.3.0] - 2026-03-08
+
+### Added
+- **Skill shadowing** — `devflow skills shadow <name>` copies a skill for personal overrides
+  - `devflow skills unshadow <name>` restores the original
+  - `devflow skills list-shadowed` shows active overrides
+  - Shadowed skills are preserved during `devflow init` (not overwritten)
+  - Uninstall warns about remaining shadow files
+- **Cross-platform hook wrapper** — `run-hook` polyglot entry point for Windows compatibility
+  - Discovers bash on Windows (Git Bash, WSL, MSYS2) via standard paths
+  - All hook scripts renamed to drop `.sh` extension
+- **Ambient skill injection at session start** — `session-start-memory` hook injects `ambient-router` SKILL.md directly into context
+  - Eliminates the need for a Read tool call to load the ambient router
+  - Only activates when ambient mode is enabled
+- **Skill activation integration tests** — `vitest.integration.config.ts` + helpers for live classification tests
+  - Separate `npm run test:integration` for tests requiring `claude` CLI
+
+### Changed
+- **Ambient depth labels renamed** — STANDARD→GUIDED, ESCALATE→ELEVATE for clarity
+  - GUIDED: skills guide the response; ELEVATE: elevate to a full workflow
+- **Hook commands use `run-hook` dispatch** — Settings template and CLI now register hooks via `run-hook <name>` instead of direct `.sh` paths
+- **`devflow init` auto-upgrades hook format** — Removes old `.sh`-style hooks before re-adding, ensuring existing installs migrate seamlessly
+- **Skill descriptions audited** — All 12 review-only skills updated to trigger-format (`"This skill should be used when..."`)
+- **Skills architecture docs** — Added description rules section with good/bad examples
+- **`chmod` skipped on Windows** — `chmodRecursive` no longer runs on `win32` platform
+
+### Fixed
+- **Ambient preamble missing skill path** — Hook now tells Claude to `Read` skills from `~/.claude/skills/<name>/SKILL.md`
+- **Ambient `--status` hook path parsing** — Handles `run-hook <name>` format instead of assuming direct `.sh` path
 
 ---
 
@@ -48,7 +76,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 - **Ambient mode** — New `devflow-ambient` plugin with `/ambient` command for proportional quality enforcement
   - Intent classification (BUILD/DEBUG/REVIEW/PLAN/EXPLORE/CHAT) auto-loads relevant skills
-  - Three depth tiers: QUICK (zero overhead), STANDARD (2-3 skills), ESCALATE (nudge to workflows)
+  - Three depth tiers: QUICK (zero overhead), GUIDED (2-3 skills), ELEVATE (nudge to workflows)
   - Always-on mode via `devflow ambient --enable` or `devflow init --ambient`
   - New `ambient-router` skill for intent/depth classification
   - New `test-driven-development` skill (auto-activates for BUILD tasks)
@@ -787,6 +815,7 @@ devflow init
 
 ---
 
+[1.3.0]: https://github.com/dean0x/devflow/compare/v1.2.0...v1.3.0
 [1.2.0]: https://github.com/dean0x/devflow/compare/v1.1.0...v1.2.0
 [1.1.0]: https://github.com/dean0x/devflow/compare/v1.0.0...v1.1.0
 [1.0.0]: https://github.com/dean0x/devflow/compare/v0.9.0...v1.0.0

package/dist/cli.js

@@ -8,6 +8,7 @@ import { uninstallCommand } from './commands/uninstall.js';
 import { listCommand } from './commands/list.js';
 import { ambientCommand } from './commands/ambient.js';
 import { memoryCommand } from './commands/memory.js';
+import { skillsCommand } from './commands/skills.js';
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
 // Read version from package.json
@@ -25,6 +26,7 @@ program.addCommand(uninstallCommand);
 program.addCommand(listCommand);
 program.addCommand(ambientCommand);
 program.addCommand(memoryCommand);
+program.addCommand(skillsCommand);
 // Handle no command
 program.action(() => {
     program.help();

package/dist/commands/ambient.js

@@ -4,7 +4,7 @@ import * as path from 'path';
 import * as p from '@clack/prompts';
 import color from 'picocolors';
 import { getClaudeDirectory } from '../utils/paths.js';
-const AMBIENT_HOOK_MARKER = 'ambient-prompt.sh';
+const AMBIENT_HOOK_MARKER = 'ambient-prompt';
 /**
  * Add the ambient UserPromptSubmit hook to settings JSON.
  * Idempotent — returns unchanged JSON if hook already exists.
@@ -17,7 +17,7 @@ export function addAmbientHook(settingsJson, devflowDir) {
     if (!settings.hooks) {
         settings.hooks = {};
     }
-    const hookCommand = path.join(devflowDir, 'scripts', 'hooks', AMBIENT_HOOK_MARKER);
+    const hookCommand = path.join(devflowDir, 'scripts', 'hooks', 'run-hook') + ' ambient-prompt';
     const newEntry = {
         hooks: [
             {
@@ -103,8 +103,9 @@ export const ambientCommand = new Command('ambient')
         // Try to extract devflowDir from existing hooks (e.g., Stop hook path)
         const stopHook = settings.hooks?.Stop?.[0]?.hooks?.[0]?.command;
         if (stopHook) {
-            // e.g., /Users/dean/.devflow/scripts/hooks/stop-update-memory.sh → /Users/dean/.devflow
-            devflowDir = path.resolve(stopHook, '..', '..', '..');
+            // e.g., "run-hook stop-update-memory" or "/path/to/.devflow/scripts/hooks/run-hook stop-update-memory"
+            const hookBinary = stopHook.split(' ')[0];
+            devflowDir = path.resolve(hookBinary, '..', '..', '..');
         }
         else {
             devflowDir = path.join(process.env.HOME || '~', '.devflow');

package/dist/commands/init.js

@@ -379,9 +379,11 @@ export const initCommand = new Command('init')
         const settingsPath = path.join(claudeDir, 'settings.json');
         try {
             const content = await fs.readFile(settingsPath, 'utf-8');
+            // Always remove-then-add to upgrade hook format (e.g., .sh → run-hook)
+            const cleaned = removeMemoryHooks(content);
             const updated = memoryEnabled
-                ? addMemoryHooks(content, devflowDir)
-                : removeMemoryHooks(content);
+                ? addMemoryHooks(cleaned, devflowDir)
+                : cleaned;
             if (updated !== content) {
                 await fs.writeFile(settingsPath, updated, 'utf-8');
                 if (verbose) {

package/dist/commands/memory.js

@@ -9,9 +9,9 @@ import { createMemoryDir, migrateMemoryFiles } from '../utils/post-install.js';
  * Map of hook event type → filename marker for the 3 memory hooks.
  */
 const MEMORY_HOOK_CONFIG = {
-    Stop: 'stop-update-memory.sh',
-    SessionStart: 'session-start-memory.sh',
-    PreCompact: 'pre-compact-memory.sh',
+    Stop: 'stop-update-memory',
+    SessionStart: 'session-start-memory',
+    PreCompact: 'pre-compact-memory',
 };
 /**
  * Add all 3 memory hooks (Stop, SessionStart, PreCompact) to settings JSON.
@@ -30,7 +30,7 @@ export function addMemoryHooks(settingsJson, devflowDir) {
         const existing = settings.hooks[hookType] ?? [];
         const alreadyPresent = existing.some((matcher) => matcher.hooks.some((h) => h.command.includes(marker)));
         if (!alreadyPresent) {
-            const hookCommand = path.join(devflowDir, 'scripts', 'hooks', marker);
+            const hookCommand = path.join(devflowDir, 'scripts', 'hooks', 'run-hook') + ` ${marker}`;
             const newEntry = {
                 hooks: [
                     {

package/dist/commands/skills.d.ts

@@ -0,0 +1,11 @@
+import { Command } from 'commander';
+/**
+ * Check if a skill has a shadow (personal override).
+ */
+export declare function hasShadow(skillName: string, devflowDir?: string): Promise<boolean>;
+/**
+ * List all shadowed skill names.
+ */
+export declare function listShadowed(devflowDir?: string): Promise<string[]>;
+export declare const skillsCommand: Command;
+//# sourceMappingURL=skills.d.ts.map

package/dist/commands/skills.js

@@ -0,0 +1,116 @@
+import { Command } from 'commander';
+import { promises as fs } from 'fs';
+import * as path from 'path';
+import * as p from '@clack/prompts';
+import color from 'picocolors';
+import { getClaudeDirectory, getDevFlowDirectory } from '../utils/paths.js';
+import { getAllSkillNames } from '../plugins.js';
+import { copyDirectory } from '../utils/installer.js';
+/**
+ * Check if a directory exists.
+ */
+async function dirExists(dirPath) {
+    try {
+        const stat = await fs.stat(dirPath);
+        return stat.isDirectory();
+    }
+    catch {
+        return false;
+    }
+}
+/**
+ * Get the shadow directory for a skill.
+ */
+function getShadowDir(devflowDir, skillName) {
+    return path.join(devflowDir, 'skills', skillName);
+}
+/**
+ * Check if a skill has a shadow (personal override).
+ */
+export async function hasShadow(skillName, devflowDir) {
+    const dir = devflowDir ?? getDevFlowDirectory();
+    return dirExists(getShadowDir(dir, skillName));
+}
+/**
+ * List all shadowed skill names.
+ */
+export async function listShadowed(devflowDir) {
+    const dir = devflowDir ?? getDevFlowDirectory();
+    const shadowsRoot = path.join(dir, 'skills');
+    try {
+        const entries = await fs.readdir(shadowsRoot, { withFileTypes: true });
+        return entries.filter(e => e.isDirectory()).map(e => e.name);
+    }
+    catch {
+        return [];
+    }
+}
+export const skillsCommand = new Command('skills')
+    .description('Manage skill overrides (shadow/unshadow/list)')
+    .argument('<action>', 'Action: shadow, unshadow, or list-shadowed')
+    .argument('[name]', 'Skill name (required for shadow/unshadow)')
+    .action(async (action, name) => {
+    const devflowDir = getDevFlowDirectory();
+    const claudeDir = getClaudeDirectory();
+    const allSkills = getAllSkillNames();
+    if (action === 'shadow') {
+        if (!name) {
+            p.log.error('Skill name required. Usage: devflow skills shadow <name>');
+            p.log.info(`Available skills: ${allSkills.join(', ')}`);
+            process.exit(1);
+        }
+        if (!allSkills.includes(name)) {
+            p.log.error(`Unknown skill: ${name}`);
+            p.log.info(`Available skills: ${allSkills.join(', ')}`);
+            process.exit(1);
+        }
+        const installedSkillDir = path.join(claudeDir, 'skills', name);
+        if (!await dirExists(installedSkillDir)) {
+            p.log.error(`Skill not installed: ${name}. Run devflow init first.`);
+            process.exit(1);
+        }
+        const shadowDir = getShadowDir(devflowDir, name);
+        if (await dirExists(shadowDir)) {
+            p.log.info(`${name} is already shadowed`);
+            return;
+        }
+        // Create shadow directory and copy original as reference backup
+        await fs.mkdir(path.join(devflowDir, 'skills'), { recursive: true });
+        await copyDirectory(installedSkillDir, shadowDir);
+        p.log.success(`Shadowed ${color.cyan(name)}`);
+        p.log.info(`Edit ${color.dim(path.join(claudeDir, 'skills', name, 'SKILL.md'))} — it won't be overwritten on next init.`);
+    }
+    else if (action === 'unshadow') {
+        if (!name) {
+            p.log.error('Skill name required. Usage: devflow skills unshadow <name>');
+            process.exit(1);
+        }
+        const shadowDir = getShadowDir(devflowDir, name);
+        if (!await dirExists(shadowDir)) {
+            p.log.info(`${name} is not shadowed`);
+            return;
+        }
+        await fs.rm(shadowDir, { recursive: true, force: true });
+        p.log.success(`Unshadowed ${color.cyan(name)}`);
+        p.log.info('Run devflow init to restore DevFlow\'s version.');
+    }
+    else if (action === 'list-shadowed') {
+        const shadowed = await listShadowed(devflowDir);
+        if (shadowed.length === 0) {
+            p.log.info('No shadowed skills');
+            return;
+        }
+        p.log.info(`Shadowed skills (${shadowed.length}):`);
+        for (const skill of shadowed) {
+            const isKnown = allSkills.includes(skill);
+            const status = isKnown ? color.green('active') : color.yellow('unknown skill');
+            p.log.info(`  ${color.cyan(skill)} — ${status}`);
+        }
+    }
+    else {
+        p.log.error(`Unknown action: ${action}`);
+        p.log.info('Usage: devflow skills <shadow|unshadow|list-shadowed> [name]');
+        process.exit(1);
+    }
+});
+//# sourceMappingURL=skills.js.map

package/dist/commands/uninstall.js

@@ -5,12 +5,13 @@ import { fileURLToPath } from 'url';
 import { execSync } from 'child_process';
 import * as p from '@clack/prompts';
 import color from 'picocolors';
-import { getInstallationPaths, getClaudeDirectory, getManagedSettingsPath } from '../utils/paths.js';
+import { getInstallationPaths, getClaudeDirectory, getDevFlowDirectory, getManagedSettingsPath } from '../utils/paths.js';
 import { getGitRoot } from '../utils/git.js';
 import { isClaudeCliAvailable } from '../utils/cli.js';
 import { DEVFLOW_PLUGINS, getAllSkillNames, LEGACY_SKILL_NAMES } from '../plugins.js';
 import { removeAmbientHook } from './ambient.js';
 import { removeMemoryHooks } from './memory.js';
+import { listShadowed } from './skills.js';
 import { detectShell, getProfilePath } from '../utils/safe-delete.js';
 import { isAlreadyInstalled, removeFromProfile } from '../utils/safe-delete-install.js';
 import { removeManagedSettings } from '../utils/post-install.js';
@@ -403,6 +404,15 @@ export const uninstallCommand = new Command('uninstall')
             }
         }
     }
+    // Warn about personal skill overrides
+    if (!isSelectiveUninstall) {
+        const shadowed = await listShadowed();
+        if (shadowed.length > 0) {
+            const shadowPath = path.join(getDevFlowDirectory(), 'skills');
+            p.log.warn(`Personal skill overrides remain in ${shadowPath}: ${shadowed.join(', ')}`);
+            p.log.info(color.dim(`Remove manually or run: rm -rf ${shadowPath}`));
+        }
+    }
     const status = color.green('DevFlow uninstalled successfully');
     p.outro(`${status}${color.dim('  Reinstall: npx devflow-kit init')}`);
 });

package/dist/utils/installer.js

@@ -105,6 +105,14 @@ export async function installViaFileCopy(options) {
             }
         }
         for (const skill of allSkills) {
+            // Skip cleanup for shadowed skills — user has a personal override
+            const shadowDir = path.join(devflowDir, 'skills', skill);
+            try {
+                const stat = await fs.stat(shadowDir);
+                if (stat.isDirectory())
+                    continue;
+            }
+            catch { /* no shadow — proceed with cleanup */ }
             try {
                 await fs.rm(path.join(claudeDir, 'skills', skill), { recursive: true, force: true });
             }
@@ -148,13 +156,21 @@ export async function installViaFileCopy(options) {
             }
         }
         catch { /* no agents directory */ }
-        // Install skills (deduplicated)
+        // Install skills (deduplicated, respects shadows)
         const skillsSource = path.join(pluginSourceDir, 'skills');
         try {
             const skillDirs = await fs.readdir(skillsSource, { withFileTypes: true });
             for (const skillDir of skillDirs) {
                 if (skillDir.isDirectory()) {
                     if (skillsMap.get(skillDir.name) === plugin.name) {
+                        // Skip copy for shadowed skills — user has a personal override
+                        const shadowDir = path.join(devflowDir, 'skills', skillDir.name);
+                        try {
+                            const stat = await fs.stat(shadowDir);
+                            if (stat.isDirectory())
+                                continue;
+                        }
+                        catch { /* no shadow — proceed with copy */ }
                         const skillTarget = path.join(claudeDir, 'skills', skillDir.name);
                         await copyDirectory(path.join(skillsSource, skillDir.name), skillTarget);
                     }
@@ -169,7 +185,9 @@ export async function installViaFileCopy(options) {
     try {
         await fs.mkdir(scriptsTarget, { recursive: true });
         await copyDirectory(scriptsSource, scriptsTarget);
-        await chmodRecursive(scriptsTarget, 0o755);
+        if (process.platform !== 'win32') {
+            await chmodRecursive(scriptsTarget, 0o755);
+        }
     }
     catch { /* scripts may not exist */ }
     spinner.stop('Components installed via file copy');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devflow-kit",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "description": "Agentic Development Toolkit for Claude Code - Enhance AI-assisted development with intelligent commands and workflows",
   "type": "module",
   "bin": {
@@ -26,7 +26,8 @@
     "prepublishOnly": "npm run build",
     "version:bump": "npx tsx scripts/bump-version.ts",
     "test": "vitest run",
-    "test:watch": "vitest"
+    "test:watch": "vitest",
+    "test:integration": "vitest run --config vitest.integration.config.ts"
   },
   "keywords": [
     "claude",

package/plugins/devflow-accessibility/.claude-plugin/plugin.json

@@ -5,11 +5,18 @@
     "name": "DevFlow Contributors",
     "email": "dean@keren.dev"
   },
-  "version": "1.2.0",
+  "version": "1.3.0",
   "homepage": "https://github.com/dean0x/devflow",
   "repository": "https://github.com/dean0x/devflow",
   "license": "MIT",
-  "keywords": ["accessibility", "wcag", "aria", "a11y"],
+  "keywords": [
+    "accessibility",
+    "wcag",
+    "aria",
+    "a11y"
+  ],
   "agents": [],
-  "skills": ["accessibility"]
+  "skills": [
+    "accessibility"
+  ]
 }

package/plugins/devflow-ambient/.claude-plugin/plugin.json

@@ -1,7 +1,9 @@
 {
   "name": "devflow-ambient",
   "description": "Ambient mode — auto-loads relevant skills for every prompt",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "agents": [],
-  "skills": ["ambient-router"]
+  "skills": [
+    "ambient-router"
+  ]
 }

package/plugins/devflow-ambient/README.md

@@ -9,10 +9,10 @@ Ambient mode — auto-loads relevant skills based on each prompt, no explicit co
 Classify user intent and apply proportional skill enforcement to any prompt.
 
 ```bash
-/ambient add a login form          # BUILD/STANDARD — loads TDD + implementation-patterns
-/ambient fix the auth error        # DEBUG/STANDARD — loads test-patterns + core-patterns
+/ambient add a login form          # BUILD/GUIDED — loads TDD + implementation-patterns
+/ambient fix the auth error        # DEBUG/GUIDED — loads test-patterns + core-patterns
 /ambient where is the config?      # EXPLORE/QUICK — responds normally, zero overhead
-/ambient refactor the auth system  # BUILD/ESCALATE — suggests /implement
+/ambient refactor the auth system  # BUILD/ELEVATE — suggests /implement
 ```
 
 ## Always-On Mode
@@ -30,19 +30,19 @@ When enabled, a `UserPromptSubmit` hook injects a classification preamble before
 ## How It Works
 
 1. **Classify intent** — BUILD, DEBUG, REVIEW, PLAN, EXPLORE, or CHAT
-2. **Classify depth** — QUICK (zero overhead), STANDARD (2-3 skills), or ESCALATE (workflow nudge)
+2. **Classify depth** — QUICK (zero overhead), GUIDED (2-3 skills), or ELEVATE (workflow nudge)
 3. **Apply proportionally**:
    - QUICK: respond normally
-   - STANDARD: load relevant skills, enforce TDD for BUILD
-   - ESCALATE: respond + recommend full workflow command
+   - GUIDED: load relevant skills, enforce TDD for BUILD
+   - ELEVATE: respond + recommend full workflow command
 
 ## Depth Tiers
 
 | Depth | When | Overhead |
 |-------|------|----------|
 | QUICK | Chat, simple exploration, git/devops ops, single-word confirmations | ~0 tokens |
-| STANDARD | BUILD/DEBUG/REVIEW/PLAN, 1-5 file scope | ~500-1000 tokens (skill reads) |
-| ESCALATE | Multi-file, architectural, system-wide scope | ~0 extra tokens (nudge only) |
+| GUIDED | BUILD/DEBUG/REVIEW/PLAN, 1-5 file scope | ~500-1000 tokens (skill reads) |
+| ELEVATE | Multi-file, architectural, system-wide scope | ~0 extra tokens (nudge only) |
 
 ## Skills
 

package/plugins/devflow-ambient/commands/ambient.md

@@ -25,7 +25,7 @@ Read the `ambient-router` skill:
 Apply the ambient-router classification to `$ARGUMENTS`:
 
 1. **Intent:** BUILD | DEBUG | REVIEW | PLAN | EXPLORE | CHAT
-2. **Depth:** QUICK | STANDARD | ESCALATE
+2. **Depth:** QUICK | GUIDED | ELEVATE
 
 If no arguments provided, output:
 
@@ -37,10 +37,10 @@ Classify intent and auto-load relevant skills.
 Usage: /ambient <your prompt>
 
 Examples:
-  /ambient add a login form          → BUILD/STANDARD (loads TDD + implementation-patterns)
-  /ambient fix the auth error        → DEBUG/STANDARD (loads test-patterns + core-patterns)
+  /ambient add a login form          → BUILD/GUIDED (loads TDD + implementation-patterns)
+  /ambient fix the auth error        → DEBUG/GUIDED (loads test-patterns + core-patterns)
   /ambient where is the config?      → EXPLORE/QUICK (responds normally)
-  /ambient refactor the auth system  → BUILD/ESCALATE (suggests /implement)
+  /ambient refactor the auth system  → BUILD/ELEVATE (suggests /implement)
 
 Always-on: devflow ambient --enable
 ```
@@ -50,15 +50,15 @@ Then stop.
 ### Phase 3: State Classification
 
 - **QUICK:** Skip this phase entirely. Respond directly in Phase 4.
-- **STANDARD:** Output one line: `Ambient: {INTENT}/{DEPTH}. Loading: {skill1}, {skill2}.`
-- **ESCALATE:** Skip — recommendation happens in Phase 4.
+- **GUIDED:** Output one line: `Ambient: {INTENT}/{DEPTH}. Loading: {skill1}, {skill2}.`
+- **ELEVATE:** Skip — recommendation happens in Phase 4.
 
 ### Phase 4: Apply
 
 **QUICK:**
 Respond to the user's prompt normally. Zero skill loading. Zero overhead.
 
-**STANDARD:**
+**GUIDED:**
 Read the selected skills based on the ambient-router's skill selection matrix:
 
 | Intent | Primary Skills | Secondary (conditional) |
@@ -72,7 +72,7 @@ Read up to 3 skills from `~/.claude/skills/{name}/SKILL.md`. Apply their pattern
 
 For BUILD intent: enforce RED-GREEN-REFACTOR from test-driven-development. Write failing tests before production code.
 
-**ESCALATE:**
+**ELEVATE:**
 Respond to the user's prompt with your best effort, then append:
 
 > This task spans multiple files/systems. Consider `/implement` for full lifecycle management (exploration → planning → implementation → review).
@@ -84,11 +84,11 @@ Respond to the user's prompt with your best effort, then append:
 │
 ├─ Phase 1: Load ambient-router skill
 ├─ Phase 2: Classify intent + depth
-├─ Phase 3: State classification (STANDARD only)
+├─ Phase 3: State classification (GUIDED only)
 └─ Phase 4: Apply
    ├─ QUICK → respond directly
-   ├─ STANDARD → load 2-3 skills, apply patterns, respond
-   └─ ESCALATE → respond + workflow nudge
+   ├─ GUIDED → load 2-3 skills, apply patterns, respond
+   └─ ELEVATE → respond + workflow nudge
 ```
 
 ## Edge Cases
@@ -104,7 +104,7 @@ Respond to the user's prompt with your best effort, then append:
 ## Principles
 
 1. **No agents** — Ambient enhances the main session, never spawns subagents
-2. **Proportional** — QUICK gets zero overhead, STANDARD gets 2-3 skills, ESCALATE gets a nudge
-3. **Transparent** — State classification for STANDARD/ESCALATE, silent for QUICK
+2. **Proportional** — QUICK gets zero overhead, GUIDED gets 2-3 skills, ELEVATE gets a nudge
+3. **Transparent** — State classification for GUIDED/ELEVATE, silent for QUICK
 4. **Respectful** — Never over-classify; when in doubt, go one tier lower
-5. **TDD for BUILD** — STANDARD depth BUILD tasks enforce test-first workflow
+5. **TDD for BUILD** — GUIDED depth BUILD tasks enforce test-first workflow

package/plugins/devflow-ambient/skills/ambient-router/SKILL.md

@@ -36,7 +36,7 @@ Determine what the user is trying to do from their prompt.
 | **EXPLORE** | "what is", "where is", "find", "show me", "explain", "how does" | "where is the config?", "explain this function" |
 | **CHAT** | greetings, meta-questions, confirmations, short responses | "thanks", "yes", "what can you do?" |
 
-**Ambiguous prompts:** Default to the lowest-overhead classification. "Update the README" → BUILD/STANDARD. Git operations like "commit this" → QUICK.
+**Ambiguous prompts:** Default to the lowest-overhead classification. "Update the README" → BUILD/GUIDED. Git operations like "commit this" → QUICK.
 
 ## Step 2: Classify Depth
 
@@ -45,10 +45,10 @@ Determine how much enforcement the prompt warrants.
 | Depth | Criteria | Action |
 |-------|----------|--------|
 | **QUICK** | CHAT intent. EXPLORE with no analytical depth ("where is X?"). Git/devops operations (commit, push, merge, branch, pr, deploy, reinstall). Single-word continuations. | Respond normally. Zero overhead. Do not state classification. |
-| **STANDARD** | BUILD/DEBUG/REVIEW/PLAN intent (any word count). EXPLORE with analytical depth ("analyze our X", "discuss how Y works"). | Read and apply 2-3 relevant skills from the selection matrix below. State classification briefly. |
-| **ESCALATE** | Multi-file architectural change, system-wide scope, > 5 files. Detailed implementation plan (100+ words with plan structure). | Respond at best effort + recommend: "This looks like it would benefit from `/implement` for full lifecycle management." |
+| **GUIDED** | BUILD/DEBUG/REVIEW/PLAN intent (any word count). EXPLORE with analytical depth ("analyze our X", "discuss how Y works"). | Read and apply 2-3 relevant skills from the selection matrix below. State classification briefly. |
+| **ELEVATE** | Multi-file architectural change, system-wide scope, > 5 files. Detailed implementation plan (100+ words with plan structure). | Respond at best effort + recommend: "This looks like it would benefit from `/implement` for full lifecycle management." |
 
-## Step 3: Select Skills (STANDARD depth only)
+## Step 3: Select Skills (GUIDED depth only)
 
 Based on classified intent, read the following skills to inform your response.
 
@@ -65,19 +65,26 @@ See `references/skill-catalog.md` for the full skill-to-intent mapping with file
 
 ## Step 4: Apply
 
+<IMPORTANT>
+When classification is GUIDED or ELEVATE, skill application is NON-NEGOTIABLE.
+Do not rationalize skipping skills. Do not respond without loading them first.
+If test-driven-development is selected, you MUST write the failing test before ANY production code.
+</IMPORTANT>
+
 - **QUICK:** Respond directly. No preamble, no classification statement.
-- **STANDARD:** State classification briefly: `Ambient: BUILD/STANDARD. Loading: test-driven-development, implementation-patterns.` Then read the selected skills and apply their patterns to your response. For BUILD intent, enforce RED-GREEN-REFACTOR from test-driven-development.
-- **ESCALATE:** Respond with your best effort, then append: `> This task spans multiple files/systems. Consider \`/implement\` for full lifecycle (exploration → planning → implementation → review).`
+- **GUIDED:** State classification briefly: `Ambient: BUILD/GUIDED. Loading: test-driven-development, implementation-patterns.` Then read the selected skills and apply their patterns. No exceptions.
+- **ELEVATE:** Respond with your best effort, then append: `> This task spans multiple files/systems. Consider \`/implement\` for full lifecycle.`
 
 ---
 
 ## Transparency Rules
 
 1. **QUICK → silent.** No classification output.
-2. **STANDARD → brief statement.** One line: intent, depth, skills loaded.
-3. **ESCALATE → recommendation.** Best-effort response + workflow nudge.
+2. **GUIDED → brief statement + full skill enforcement.** One line: intent, depth, skills loaded. Then follow every skill requirement without shortcuts.
+3. **ELEVATE → recommendation.** Best-effort response + workflow nudge.
 4. **Never lie about classification.** If uncertain, say so.
 5. **Never over-classify.** When in doubt, go one tier lower.
+6. **Never under-apply.** Rationalization is the enemy of quality. If a skill requires a step, do the step.
 
 ## Edge Cases
 

package/plugins/devflow-ambient/skills/ambient-router/references/skill-catalog.md

@@ -4,7 +4,7 @@ Full mapping of DevFlow skills to ambient intents and file-type triggers. The am
 
 ## Skills Available for Ambient Loading
 
-These skills may be loaded during STANDARD-depth ambient routing.
+These skills may be loaded during GUIDED-depth ambient routing.
 
 ### BUILD Intent
 
@@ -65,4 +65,4 @@ These skills are loaded only by explicit DevFlow commands (primarily `/code-revi
 - **Maximum 3 skills** per ambient response (primary + up to 2 secondary)
 - **Primary skills** are always loaded for the classified intent
 - **Secondary skills** are loaded only when file patterns match conversation context
-- If more than 3 skills seem relevant, this is an ESCALATE signal
+- If more than 3 skills seem relevant, this is an ELEVATE signal

package/plugins/devflow-audit-claude/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "devflow-audit-claude",
   "description": "Audit CLAUDE.md files against Anthropic best practices",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "agents": [],
   "skills": []
 }

package/plugins/devflow-code-review/.claude-plugin/plugin.json

@@ -5,12 +5,22 @@
     "name": "DevFlow Contributors",
     "email": "dean@keren.dev"
   },
-  "version": "1.2.0",
+  "version": "1.3.0",
   "homepage": "https://github.com/dean0x/devflow",
   "repository": "https://github.com/dean0x/devflow",
   "license": "MIT",
-  "keywords": ["review", "code-quality", "security", "architecture", "pr-review"],
-  "agents": ["git", "reviewer", "synthesizer"],
+  "keywords": [
+    "review",
+    "code-quality",
+    "security",
+    "architecture",
+    "pr-review"
+  ],
+  "agents": [
+    "git",
+    "reviewer",
+    "synthesizer"
+  ],
   "skills": [
     "agent-teams",
     "architecture-patterns",

package/plugins/devflow-code-review/skills/architecture-patterns/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: architecture-patterns
-description: Architecture analysis patterns for code review. Detects SOLID violations, tight coupling, layering issues, and dependency direction problems. Loaded by Reviewer agent when focus=architecture.
+description: This skill should be used when reviewing code for SOLID violations, tight coupling, or layering issues.
 user-invocable: false
 allowed-tools: Read, Grep, Glob
 ---

package/plugins/devflow-code-review/skills/complexity-patterns/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: complexity-patterns
-description: Complexity analysis patterns for code review. Detects high cyclomatic complexity, deep nesting, long functions, and readability issues that hinder maintainability. Loaded by Reviewer agent when focus=complexity.
+description: This skill should be used when reviewing code for high cyclomatic complexity, deep nesting, or long functions.
 user-invocable: false
 allowed-tools: Read, Grep, Glob
 ---