Skip to content

Point passive skill awareness at active runtime#62

Closed
mega123-art wants to merge 1 commit into
mainfrom
fix/passive-skill-awareness-paths
Closed

Point passive skill awareness at active runtime#62
mega123-art wants to merge 1 commit into
mainfrom
fix/passive-skill-awareness-paths

Conversation

@mega123-art

Copy link
Copy Markdown
Contributor

Summary

  • read installed skills from the active runtime directory instead of always Claude's dir
  • include the concrete Claude/Codex SKILL.md path in the managed awareness block
  • update the unit test to verify Codex reads ~/.codex/skills separately

Verification

  • pnpm --filter @iqlabs-official/agent-sdk exec tsc --noEmit
  • pnpm --filter @iqlabs-official/agent-sdk exec vitest run src/memory/skillsSection.spec.ts

Split out of #56 per review feedback.

zo-sol added a commit that referenced this pull request Jun 20, 2026
Build the memory skills block from the active engine's skills dir (claude vs
codex) and point the line at that runtime's SKILL.md path, so the agent only
reaches for skills the running engine can actually load.

Cherry-picked the core change from #62.

Co-authored-by: mega123-art <agrawalparth326@gmail.com>
@zo-sol

zo-sol commented Jun 20, 2026

Copy link
Copy Markdown
Collaborator

Thanks @mega123-art — great change. Making the skills block read the active runtime's dir (~/.codex/skills vs ~/.claude/skills) and pointing the memory line at the matching SKILL.md path is exactly right, so the agent only reaches for skills the running engine can load.

I cherry-picked the core of this onto main in c1a9f85 (with you as co-author), keeping your skillsSection.ts + spec as-is (specs pass). I took only the core file rather than merging the branch because this PR predates the market RPC refactor on main (3b6b601) and the dispose/re-equip work (#60) — a full merge would have silently reverted the attachMarketHandlers extraction (verified via a trial merge: attachMarketHandlers count → 0). The webview-side bits here are superseded by what's already on main.

Closing as cherry-picked into main. Really appreciate it — credit's on the commit. 🙏

@zo-sol zo-sol closed this Jun 20, 2026
@zo-sol zo-sol deleted the fix/passive-skill-awareness-paths branch June 20, 2026 11:56
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.

2 participants