fix(skill-loader): wrap template with skill-instruction tags in sync loader

[ElwinLiu]

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.}

[github-actions]

All contributors have signed the CLA. Thank you! ✅
_{Posted by the CLA Assistant Lite bot.}

[greptile-apps]

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

[cubic-dev-ai]

No issues found across 1 file

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

[ElwinLiu]

I have read the CLA Document and I hereby sign the CLA