refactor the skills to reduce tokens and improve progresive disclousure patterns#47
refactor the skills to reduce tokens and improve progresive disclousure patterns#47Cesar-M-Diaz wants to merge 0 commit into
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
dobleuber
left a comment
There was a problem hiding this comment.
I think we have a conflict in the skills instructions
| - CPU profile: `node "<skill-dir>/wait.cjs" 35`. | ||
| - Heap sample: `node "<skill-dir>/wait.cjs" 30`. |
There was a problem hiding this comment.
The new wait step instructs agents to run node "<skill-dir>/wait.cjs" for CPU/heap asset generation, but this same skill still says to “Use ONLY N|Solid MCP tools” and “Do not use shell commands or bundled helper scripts for waits or downloads” in the guardrails. In practice, an agent following the stricter guardrail can refuse the required wait step, while an agent following step 4 violates the guardrail; this makes CPU/heap asset capture workflows unreliable. Please update the guardrails to allow the bundled wait helper, or switch the wait step back to the intended MCP wait tool.
583b49a to
b431bb5
Compare
b431bb5 to
b481c92
Compare
No description provided.