Summary

Fixed the sync skill loader to produce consistently wrapped templates matching the async loader.

Problem

The sync loader was setting template to an empty string, while the async loader properly wrapped it with and tags. This caused skills loaded via config-handler.ts to have empty templates.

Solution

1. Create wrappedTemplate variable upfront with proper tags
2. Use wrappedTemplate for both definition.template and lazyContent.load()
3. Removed redundant file re-read in lazyContent.load()

Testing

• bun run typecheck passes
• bun run build succeeds

Summary by cubic

Fixes the sync skill loader so templates are wrapped with and , matching the async loader. Prevents empty templates when loading skills via config-handler.

• Bug Fixes

  • Use a single wrapped template for definition.template and lazyContent.load() to keep behavior consistent.

• Refactors

  • Remove redundant file read in lazyContent.load() by reusing the prepared wrapped template.

^{Written for commit c965ec6. Summary will update on new commits.}