claude-dev-env 8.88.7 → 8.90.0

package/bin/install.test.mjs

@@ -57,6 +57,7 @@ import {
 } from './install.mjs';
 import { EVER_SHIPPED_SKILL_NAMES } from './ever-shipped-skills.mjs';
 import {
+    DEEPEN_GUIDANCE_BLOCK_END,
     DEEPEN_GUIDANCE_BLOCK_START,
     PACKAGE_GUIDANCE_BLOCK_END,
     PACKAGE_GUIDANCE_BLOCK_START,
@@ -257,9 +258,9 @@ test('CORE_SKILLS ships ELI5 as the leaf presentation skill', () => {
     assert.equal(INSTALL_GROUPS.core.skills.includes('eli5'), true);
 });
 
-test('CORE_SKILLS ships deepen so the first-task skill installs with core', () => {
-    assert.ok(CORE_SKILLS.includes('deepen'), 'deepen must be in CORE_SKILLS');
-    assert.equal(INSTALL_GROUPS.core.skills.includes('deepen'), true);
+test('CORE_SKILLS leaves out the retired deepen skill', () => {
+    assert.equal(CORE_SKILLS.includes('deepen'), false);
+    assert.equal(INSTALL_GROUPS.core.skills.includes('deepen'), false);
 });
 
 test('CORE_SKILLS ships efficient-review so a core install includes the skill', () => {
@@ -1243,6 +1244,7 @@ test('retired hook registrations stay managed so reinstall removes them', () =>
         'blocking/artifact_look_gate.py',
         'blocking/session_title_stop_gate.py',
         'session/issue_tracker_session_starter.py',
+        'session/deepen_first_prompt.py',
     ]);
     const shippedHooks = JSON.parse(
         readFileSync(new URL('../hooks/hooks.json', import.meta.url), 'utf8')
@@ -3048,7 +3050,6 @@ test('a failed update preserves an existing skills lookup pointer', () => {
             CDE_INSTALL_DEEPSTACK: '0',
             CDE_INSTALL_USAGE_WRAPUP: '0',
             CDE_INSTALL_SUBAGENT_MODELS: '0',
-            CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
         };
         const argumentsForCoreInstall = [PSTACK_TEST_INSTALLER_PATH, '--only', 'core'];
         const firstRun = spawnSync(process.execPath, argumentsForCoreInstall, {
@@ -3103,7 +3104,6 @@ function runPstackInstaller(homeDirectory, extraArguments, environmentOverrides
             CDE_INSTALL_DEEPSTACK: '0',
             CDE_INSTALL_USAGE_WRAPUP: '0',
             CDE_INSTALL_SUBAGENT_MODELS: '0',
-            CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
             ...environmentOverrides,
         },
     });
@@ -3213,10 +3213,7 @@ test('a full install adds DeepStack for Claude, Codex, and Cursor', t => {
         hooks.hooks.sessionStart.filter(entry => entry.command.includes('session-start.sh')).length,
         1,
     );
-    assert.equal(
-        hooks.hooks.beforeSubmitPrompt.filter(entry => entry.command.includes('deepen_first_prompt.py')).length,
-        1,
-    );
+    assert.equal(JSON.stringify(hooks).includes('deepen_first_prompt.py'), false, 'no retired first-task hook');
     assert.equal(JSON.stringify(hooks).includes('"hooks":[{'), false, 'no Claude-style group entries');
     const sheet = readFileSync(join(sandbox.homeDirectory, '.codex', 'deepstack-models.md'), 'utf8');
     assert.match(sheet, /^session hook: on$/m);
@@ -3228,7 +3225,8 @@ test('a repeat install preserves Codex pstack configuration', t => {
     runPstackInstaller(sandbox.homeDirectory, [], sandbox.environment);
     writeFileSync(
         join(codexHome, 'AGENTS.md'),
-        `${SKILL_LOAD_BLOCK_START}\n${STALE_SKILL_LOAD_SENTENCE}\n${SKILL_LOAD_BLOCK_END}\nCustom Codex guidance\n`,
+        `${SKILL_LOAD_BLOCK_START}\n${STALE_SKILL_LOAD_SENTENCE}\n${SKILL_LOAD_BLOCK_END}\nCustom Codex guidance\n`
+        + `${DEEPEN_GUIDANCE_BLOCK_START}\nDeepen the first interactive user task.\n${DEEPEN_GUIDANCE_BLOCK_END}\n`,
     );
     writeFileSync(join(codexHome, 'pstack-models.md'), 'session hook: off\n');
 
@@ -3238,7 +3236,7 @@ test('a repeat install preserves Codex pstack configuration', t => {
     assert.ok(codexGuidance.startsWith('Custom Codex guidance\n'));
     assertOmitsSkillLoadSentence(codexGuidance);
     assert.ok(codexGuidance.includes(QUESTION_PRESENTATION_BLOCK_START));
-    assert.ok(codexGuidance.includes(DEEPEN_GUIDANCE_BLOCK_START));
+    assert.equal(codexGuidance.includes(DEEPEN_GUIDANCE_BLOCK_START), false);
     assert.equal(readFileSync(join(codexHome, 'pstack-models.md'), 'utf8'), 'session hook: off\n');
 });
 
@@ -3322,7 +3320,7 @@ test('a fresh full install writes the package guidance into Codex AGENTS.md', t
     assertHoldsPackageGuidance(codexGuidance);
     assertOmitsSkillLoadSentence(codexGuidance);
     assert.ok(codexGuidance.includes(QUESTION_PRESENTATION_BLOCK_START));
-    assert.ok(codexGuidance.includes(DEEPEN_GUIDANCE_BLOCK_START));
+    assert.equal(codexGuidance.includes(DEEPEN_GUIDANCE_BLOCK_START), false);
 });
 
 test('--no-pstack writes package and question guidance without the skill-load block', t => {
@@ -3405,7 +3403,7 @@ test('a pstack install after a --no-pstack install seeds the Codex model sheet',
     assertOmitsSkillLoadSentence(codexGuidance);
     assertHoldsPackageGuidance(codexGuidance);
     assert.ok(codexGuidance.includes(QUESTION_PRESENTATION_BLOCK_START));
-    assert.ok(codexGuidance.includes(DEEPEN_GUIDANCE_BLOCK_START));
+    assert.equal(codexGuidance.includes(DEEPEN_GUIDANCE_BLOCK_START), false);
 });
 
 const USAGE_WRAPUP_CLAUDE_COMMANDS = Object.freeze([
@@ -3483,18 +3481,16 @@ test('the help output names the usage-wrapup opt-out beside the pstack one', ()
 
     assert.equal(helpRun.status, 0, helpRun.stderr);
     assert.match(helpRun.stdout, /--no-pstack .*\n.*--no-deepstack .*CDE_INSTALL_DEEPSTACK=0.*\n.*--no-usage-wrapup .*CDE_INSTALL_USAGE_WRAPUP=0/);
-    assert.match(helpRun.stdout, /--no-deepen-first-prompt .*CDE_INSTALL_DEEPEN_FIRST_PROMPT=0/);
+    assert.doesNotMatch(helpRun.stdout, /--no-deepen-first-prompt/);
 });
 
 const REPOSITORY_PLUGIN_CLAUDE_COMMANDS = Object.freeze([
     'claude plugin uninstall usage-wrapup@claude-dev-env',
     'claude plugin uninstall subagent-models@claude-dev-env',
-    'claude plugin uninstall deepen-first-prompt@claude-dev-env',
     'claude plugin marketplace remove claude-dev-env',
-    'claude plugin marketplace add jl-cmd-projects/claude-dev-env --sparse .claude-plugin packages/usage-wrapup packages/subagent-models packages/deepen-first-prompt',
+    'claude plugin marketplace add jl-cmd-projects/claude-dev-env --sparse .claude-plugin packages/usage-wrapup packages/subagent-models',
     'claude plugin install usage-wrapup@claude-dev-env',
     'claude plugin install subagent-models@claude-dev-env',
-    'claude plugin install deepen-first-prompt@claude-dev-env',
 ]);
 
 test('should install subagent-models beside usage-wrapup with one marketplace add on a full install', t => {
@@ -3504,13 +3500,12 @@ test('should install subagent-models beside usage-wrapup with one marketplace ad
         ...sandbox.environment,
         CDE_INSTALL_USAGE_WRAPUP: '1',
         CDE_INSTALL_SUBAGENT_MODELS: '1',
-        CDE_INSTALL_DEEPEN_FIRST_PROMPT: '1',
     });
 
     assert.deepEqual(usageWrapupCommands(sandbox.recordedCommands()), REPOSITORY_PLUGIN_CLAUDE_COMMANDS);
     assert.match(installerOutput, /Usage-wrapup \(claude\): installed/);
     assert.match(installerOutput, /Subagent-models \(claude\): installed/);
-    assert.match(installerOutput, /Deepen-first-prompt \(claude\): installed/);
+    assert.doesNotMatch(installerOutput, /Deepen-first-prompt \(/);
 });
 
 for (const [caseName, extraArguments, environmentOverrides] of [
@@ -3534,27 +3529,6 @@ for (const [caseName, extraArguments, environmentOverrides] of [
     });
 }
 
-for (const [caseName, extraArguments, environmentOverrides] of [
-    ['--no-deepen-first-prompt', ['--no-deepen-first-prompt'], { CDE_INSTALL_DEEPEN_FIRST_PROMPT: '1' }],
-    ['CDE_INSTALL_DEEPEN_FIRST_PROMPT=0', [], { CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0' }],
-    ['an --only run', ['--only', 'core'], { CDE_INSTALL_DEEPEN_FIRST_PROMPT: '1' }],
-]) {
-    test(`should skip deepen-first-prompt on ${caseName}`, t => {
-        const sandbox = pstackPluginSandbox(t);
-
-        const installerOutput = runPstackInstaller(sandbox.homeDirectory, extraArguments, {
-            ...sandbox.environment,
-            ...environmentOverrides,
-        });
-
-        assert.equal(
-            sandbox.recordedCommands().some(eachCommand => eachCommand.includes('deepen-first-prompt')),
-            false,
-        );
-        assert.doesNotMatch(installerOutput, /Deepen-first-prompt \(/);
-    });
-}
-
 test('should name the subagent-models opt-out in the help output', () => {
     const helpRun = spawnSync(process.execPath, [PSTACK_TEST_INSTALLER_PATH, '--help'], {
         encoding: 'utf8',
@@ -3649,7 +3623,6 @@ function installWithGlobalPrefix(context, globalPrefix) {
             CDE_INSTALL_DEEPSTACK: '0',
             CDE_INSTALL_USAGE_WRAPUP: '0',
             CDE_INSTALL_SUBAGENT_MODELS: '0',
-            CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
             HOME: homeDirectory,
             USERPROFILE: homeDirectory,
             APPDATA: join(homeDirectory, 'AppData', 'Roaming'),
@@ -3706,7 +3679,6 @@ test('install adds the open-tasks Stop gate to every profile with its own settin
             CDE_INSTALL_DEEPSTACK: '0',
             CDE_INSTALL_USAGE_WRAPUP: '0',
             CDE_INSTALL_SUBAGENT_MODELS: '0',
-            CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
             HOME: homeDirectory,
             USERPROFILE: homeDirectory,
             CODEX_HOME: join(homeDirectory, '.codex'),
@@ -3749,7 +3721,6 @@ test('a profile settings.json that is not valid JSON is left alone and named und
             CDE_INSTALL_DEEPSTACK: '0',
             CDE_INSTALL_USAGE_WRAPUP: '0',
             CDE_INSTALL_SUBAGENT_MODELS: '0',
-            CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
             HOME: homeDirectory,
             USERPROFILE: homeDirectory,
             CODEX_HOME: join(homeDirectory, '.codex'),
@@ -3795,7 +3766,6 @@ test('uninstall stops before changing anything while a profile settings.json tha
             CDE_INSTALL_DEEPSTACK: '0',
             CDE_INSTALL_USAGE_WRAPUP: '0',
             CDE_INSTALL_SUBAGENT_MODELS: '0',
-            CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
             HOME: homeDirectory,
             USERPROFILE: homeDirectory,
             CODEX_HOME: join(homeDirectory, '.codex'),

package/bin/install.transaction.test.mjs

@@ -91,7 +91,6 @@ test('capture and restore recover settings, manifest, files, and hooksPath', ()
         CDE_INSTALL_DEEPSTACK: '0',
         CDE_INSTALL_USAGE_WRAPUP: '0',
         CDE_INSTALL_SUBAGENT_MODELS: '0',
-        CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
         HOME: box.root,
         USERPROFILE: box.root,
         GIT_CONFIG_GLOBAL: gitConfigPath,
@@ -158,7 +157,6 @@ test('capture and restore recover an additional host settings file', () => {
             CDE_INSTALL_DEEPSTACK: '0',
             CDE_INSTALL_USAGE_WRAPUP: '0',
             CDE_INSTALL_SUBAGENT_MODELS: '0',
-            CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
             HOME: box.root,
             USERPROFILE: box.root,
             GIT_CONFIG_GLOBAL: gitConfigPath,
@@ -198,7 +196,6 @@ test('runWithInstallTransaction restores prior state on injected fault', () => {
         CDE_INSTALL_DEEPSTACK: '0',
         CDE_INSTALL_USAGE_WRAPUP: '0',
         CDE_INSTALL_SUBAGENT_MODELS: '0',
-        CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
         HOME: box.root,
         USERPROFILE: box.root,
         GIT_CONFIG_GLOBAL: gitConfigPath,
@@ -257,7 +254,6 @@ test('runWithInstallTransaction commits and discards journal on success', () =>
             CDE_INSTALL_DEEPSTACK: '0',
             CDE_INSTALL_USAGE_WRAPUP: '0',
             CDE_INSTALL_SUBAGENT_MODELS: '0',
-            CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
             HOME: box.root,
             USERPROFILE: box.root,
             GIT_CONFIG_GLOBAL: gitConfigPath,
@@ -305,7 +301,6 @@ function runInstaller(homeDirectory, extraArguments, options = {}) {
         CDE_INSTALL_DEEPSTACK: '0',
         CDE_INSTALL_USAGE_WRAPUP: '0',
         CDE_INSTALL_SUBAGENT_MODELS: '0',
-        CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
         HOME: homeDirectory,
         USERPROFILE: homeDirectory,
         GIT_CONFIG_GLOBAL: join(homeDirectory, '.gitconfig'),
@@ -343,7 +338,6 @@ test('a later install fault removes newly seeded Codex pstack files', () => {
                 CDE_INSTALL_DEEPSTACK: '0',
                 CDE_INSTALL_USAGE_WRAPUP: '0',
                 CDE_INSTALL_SUBAGENT_MODELS: '0',
-                CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
                 CDE_CODEX_EXECUTABLE: codexCommandPath,
             },
         });
@@ -491,7 +485,6 @@ test('installer fault after_git_config restores prior core.hooksPath', () => {
                 CDE_INSTALL_DEEPSTACK: '0',
                 CDE_INSTALL_USAGE_WRAPUP: '0',
                 CDE_INSTALL_SUBAGENT_MODELS: '0',
-                CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
                 HOME: homeDirectory,
                 USERPROFILE: homeDirectory,
                 GIT_CONFIG_GLOBAL: gitConfigPath,
@@ -513,7 +506,6 @@ test('installer fault after_git_config restores prior core.hooksPath', () => {
                 CDE_INSTALL_DEEPSTACK: '0',
                 CDE_INSTALL_USAGE_WRAPUP: '0',
                 CDE_INSTALL_SUBAGENT_MODELS: '0',
-                CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
                 HOME: homeDirectory,
                 USERPROFILE: homeDirectory,
                 GIT_CONFIG_GLOBAL: gitConfigPath,

package/bin/install.uninstall-transaction.test.mjs

@@ -77,7 +77,6 @@ function runInstaller(homeDirectory, extraArguments, options = {}) {
         CDE_INSTALL_DEEPSTACK: '0',
         CDE_INSTALL_USAGE_WRAPUP: '0',
         CDE_INSTALL_SUBAGENT_MODELS: '0',
-        CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
         HOME: homeDirectory,
         USERPROFILE: homeDirectory,
         GIT_CONFIG_GLOBAL: join(homeDirectory, '.gitconfig'),
@@ -316,7 +315,6 @@ test('uninstall fault after_git_config restores prior core.hooksPath', () => {
                 CDE_INSTALL_DEEPSTACK: '0',
                 CDE_INSTALL_USAGE_WRAPUP: '0',
                 CDE_INSTALL_SUBAGENT_MODELS: '0',
-                CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
                 HOME: homeDirectory,
                 USERPROFILE: homeDirectory,
                 GIT_CONFIG_GLOBAL: gitConfigPath,

package/bin/install.update-keeps-hooks.test.mjs

@@ -28,7 +28,6 @@ function installerEnvironment(homeDirectory) {
         CDE_INSTALL_DEEPSTACK: '0',
         CDE_INSTALL_USAGE_WRAPUP: '0',
         CDE_INSTALL_SUBAGENT_MODELS: '0',
-        CDE_INSTALL_DEEPEN_FIRST_PROMPT: '0',
         HOME: homeDirectory,
         USERPROFILE: homeDirectory,
         GIT_CONFIG_GLOBAL: join(homeDirectory, '.gitconfig'),

package/docs/BDD_SCENARIO_QUALITY.md

@@ -40,7 +40,7 @@ Then I should be presented with the following hotels:
 
 ## Meaningful Actors
 
-Personas ground scenarios in realistic goals and context. Use light soap-opera personas when you need depth before full UX research: introduce names and roles as needed and deepen them across scenarios.
+Personas ground scenarios in realistic goals and context. Use light soap-opera personas when you need depth before full UX research: introduce names and roles as needed and develop them across scenarios.
 
 > Smart & Molak §7.6.5 describe personas as rich, realistic descriptions: each persona captures goals, abilities, and background information that ground the test scenario in a user context.
 

package/docs/BDD_TEST_LAYOUT.md

@@ -18,7 +18,7 @@ Dan North (2006): naming tests as sentences keeps focus on the **next behaviour*
 
 ## Soap-opera personas
 
-When you lack full personas, introduce **short-lived characters** with a name and role and deepen them as scenarios grow—like a serial drama adding cast over episodes (Smart & Molak §7.6.5). Embed **\[Name], the \[role]** in scenario titles or setup where the actor changes behavior.
+When you lack full personas, introduce **short-lived characters** with a name and role and develop them as scenarios grow—like a serial drama adding cast over episodes (Smart & Molak §7.6.5). Embed **\[Name], the \[role]** in scenario titles or setup where the actor changes behavior.
 
 ## Example (JavaScript-style)
 

package/docs/CODE_RULES.md

@@ -132,4 +132,4 @@ Open [details](code-rules/enforcement-surfaces.md) when adding a check.
 
 ## 11.7 A NEW INSTRUCTION NAMES THE TRIGGER THAT COVERS IT
 
-A new rule, hook, or skill names the trigger that covers it and records that trigger in `rules-archived/ARCHIVE-MANIFEST.md`. The first-task gate is `hooks/session/deepen_first_prompt.py`. DeepStack cites that gate. This criterion cites the gate. It does not restate the gate.
+A new rule, hook, or skill names the trigger that covers it and records that trigger in `rules-archived/ARCHIVE-MANIFEST.md`.

package/docs/rule-guides/deepstack-wording.md

@@ -4,9 +4,9 @@ Back to the [rules index](../../rules/index.md).
 
 ## In brief
 
-A deepened task uses the three DeepStack labels and puts `CDE_DEEPEN:v1` on line 1. The gate in `hooks/session/deepen_first_prompt.py` is the check. This guide cites that gate.
+A DeepStack task uses the three DeepStack labels. No hook checks them.
 
-**When this applies:** Writing a deepened task, a worker prompt, or a sentence that names the three DeepStack lines.
+**When this applies:** Writing a worker prompt or a sentence that names the three DeepStack lines.
 
 ## Labels
 
@@ -18,6 +18,4 @@ Say "the user" in public text.
 
 ## Where it is enforced
 
-| Surface | What runs |
-|---|---|
-| A first-task rewrite | `hooks/session/deepen_first_prompt.py` |
+Nowhere. The agent applies it.

package/docs/rule-guides/session-title.md

@@ -4,9 +4,9 @@ Back to the [rules index](../../rules/index.md).
 
 ## In brief
 
-Before each turn ends, set the title to `<emoji> <name>`. Use the red flag when the user must act, the check mark when the work is done, and the hourglass while work, CI, or a merge queue runs. The name has 25 characters or fewer, in sentence case, with concrete nouns and no IDs, dates, or branch names.
+Before each turn ends, set the title to `<emoji> <name>`. Use the red flag when the user must act, the construction sign when an outside dependency blocks the work, the check mark when the work is done, and the hourglass while work, CI, or a merge queue runs. The name has 25 characters or fewer, in sentence case, with concrete nouns and no IDs, dates, or branch names.
 
-**Enforcement:** `hooks/blocking/session_title_format_gate.py` denies a malformed title.
+**Enforcement:** `hooks/blocking/session_title_format_gate.py` denies a malformed title. `hooks/blocking/session_title_turn_gate.py` holds a turn that set no title.
 
 **When this applies:** The session has a tool whose name ends in `__set_session_title`, such as `mcp__claude-code-remote__set_session_title` in a cloud session or `mcp__ccd_session_mgmt__set_session_title` in the desktop app.
 
@@ -14,13 +14,14 @@ Before each turn ends, set the title to `<emoji> <name>`. Use the red flag when
 
 Before each turn ends, set the title to `<emoji> <name>`. Get the session id from `get_session` with no `session_id` when the tool asks for one.
 
-Set the title before the final reply, and set it in silence. The reply to the user carries no mention of the title, its emoji, or the reminder that asked for it.
+Set the title before the final reply, and set it in silence. The reply to the user carries no mention of the title, its emoji, or the reminder that asked for it. When the Stop gate holds the turn after the answer went out, call the title tool and close with one short sentence. Leave the answer as it stands.
 
 ## Status emoji
 
 | Emoji | Use it when |
 |---|---|
 | Red flag | The user must act: a question, an approval, or a step only the user can do. |
+| Construction sign | An outside dependency blocks the work and the user cannot clear it right now, such as a refused push or an unreachable host. |
 | Check mark | The work is done: merged, or nothing left. |
 | Hourglass | Work, CI, or a merge queue still runs and nothing waits on the user. |
 
@@ -30,4 +31,4 @@ The name has 25 characters or fewer and names the main change or outcome in conc
 
 ## Enforcement
 
-`hooks/blocking/session_title_format_gate.py` denies a malformed title.
+`hooks/blocking/session_title_format_gate.py` denies a malformed title. `hooks/blocking/session_title_turn_gate.py` is a Stop hook. It holds the end of a turn until a title call in that turn comes back without an error. It applies when the transcript shows the title tool. A held turn whose title call failed may end, so a failing title tool cannot hold it forever.