fix: add interactive mode instruction to embedded skills

[pitzcarraldo]

Summary

Ensure embedded skills follow the Guided Interactive Workflow pattern in interactive mode by adding explicit instructions to getSkillPrompt. This fixes the issue where /event-tracking and other interactive skills skip confirmation/proposal steps and immediately proceed to implementation.

Details

Added INTERACTIVE_MODE_INSTRUCTION constant that enforces the Confirm → Propose → Validate → Implement → Verify workflow for embedded skills in interactive mode. The instruction is only applied when oneShot: false (interactive mode), while command mode behavior remains unchanged. This applies to all embedded skills: event-tracking, integration, user-management, personalization, and api-triggered-campaigns.

How to Validate

1. Run bun run dev to start interactive mode
2. Execute /event-tracking command
3. Verify AI asks for platform, goals, and other required inputs before proceeding
4. Test other embedded skills (/integration, /user-management) exhibit same behavior
5. Confirm clix install (command mode) still runs autonomously without asking questions
6. Run bun test to verify all tests pass

Pre-Merge Checklist

Code Quality

• Types check correctly
• Linter passes
• Tests pass (bun test src/lib/__tests__/skills.test.ts)

Commit Standards

• Commits follow Conventional Commits format

Summary by CodeRabbit

• New Features
  • Added interactive workflow mode that guides users through a structured Confirm → Propose → Validate → Implement → Verify pattern when interactive mode is enabled.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Added a new INTERACTIVE_MODE_INSTRUCTION constant describing a guided workflow pattern. Modified prompt construction to conditionally append either INTERACTIVE_MODE_INSTRUCTION (when oneShot is not set) or ONE_SHOT_INSTRUCTION (when oneShot is set) to skill prompts.

Changes

Cohort / File(s)	Summary
Interactive Mode Instructions src/lib/skills.ts	Added INTERACTIVE_MODE_INSTRUCTION constant with Confirm → Propose → Validate → Implement → Verify workflow pattern. Updated prompt construction logic to append mode-specific instructions based on oneShot parameter state.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: adding interactive mode instructions to embedded skills to enforce a guided workflow pattern.
Description check	✅ Passed	The description covers all required sections: clear summary, implementation details, validation steps, and completed code quality/commit standards checklist.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}