feat(ambient): inject ambient-router skill at session start#84
Open
feat(ambient): inject ambient-router skill at session start#84
Conversation
Restructures session-start-memory.sh to not early-exit when no memory file exists. Ambient skill content is now injected directly into session context, eliminating the unreliable "Read the ambient-router skill" instruction in ambient-prompt.sh. Key changes: - session-start-memory.sh: builds context incrementally (memory section + ambient section), either can fire independently - ambient-prompt.sh: references "skill already in your session context" instead of requiring a Read tool call Fixes the critical gap where ambient mode was broken for projects without working memory enabled. Closes #79
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
session-start-memory.shto not early-exit when no memory file existsambient-router/SKILL.mdcontent directly into session context at startupambient-prompt.shpreamble to reference "skill already in your session context" instead of requiring a Read tool callProblem
ambient-prompt.shline 38 tells Claude to "Read the ambient-router skill" — this requires Claude to make a Read tool call, which it often skips, making ambient mode appear broken. Additionally, the session-start hook exits early if no.memory/WORKING-MEMORY.mdexists, meaning ambient injection would never fire for projects without working memory.Fix
additionalContextat session start, eliminating the need for a Read tool callTest plan
npm run buildpasses (40 skill copies, 20 agent copies)npm test— all 178 tests passCloses #79
Part of #78