fix: move skillPath declaration outside try block to fix scoping bug

The skillPath variable was declared inside the try block but referenced
in the catch block for error logging. Since const is block-scoped, this
would cause a ReferenceError when the error handler executes.

Moved skillPath declaration before the try block so it's accessible in
both try and catch scopes.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

src/services/worker-service.ts

@@ -146,12 +146,12 @@ export class WorkerService {
     // Instructions endpoint - loads SKILL.md sections on-demand for progressive instruction loading
     this.app.get('/api/instructions', async (req, res) => {
       const topic = (req.query.topic as string) || 'all';
+      // Read SKILL.md from plugin directory
+      // Path resolution: __dirname is build output directory (plugin/scripts/)
+      // SKILL.md is at plugin/skills/mem-search/SKILL.md
+      const skillPath = path.join(__dirname, '../skills/mem-search/SKILL.md');
 
       try {
-        // Read SKILL.md from plugin directory
-        // Path resolution: __dirname is build output directory (plugin/scripts/)
-        // SKILL.md is at plugin/skills/mem-search/SKILL.md
-        const skillPath = path.join(__dirname, '../skills/mem-search/SKILL.md');
         const fullContent = await fs.promises.readFile(skillPath, 'utf-8');
 
         // Extract section based on topic