Skip to content

feat(Skill): Improve skill and fix patterns example#1526

Merged
Marcosld merged 10 commits into
masterfrom
WEB-2427
Apr 13, 2026
Merged

feat(Skill): Improve skill and fix patterns example#1526
Marcosld merged 10 commits into
masterfrom
WEB-2427

Conversation

@Marcosld

@Marcosld Marcosld commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

We found sometimes llms skip reading parts of the doc or skip important files altogether.

@Marcosld Marcosld self-assigned this Apr 8, 2026
@Marcosld Marcosld requested review from atabel, aweell and yceballost April 8, 2026 10:00
@Marcosld Marcosld changed the title WEB 2427 feat(chore): Improve skill and fix patterns example Apr 8, 2026
@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown

Size stats

master this branch diff
Total JS 16.1 MB 16.1 MB 0 B
JS without icons 2.01 MB 2.01 MB 0 B
Lib overhead 92.5 kB 92.5 kB 0 B
Lib overhead (gzip) 19.9 kB 19.9 kB 0 B

@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown

Accessibility report
✔️ No issues found

ℹ️ You can run this locally by executing yarn audit-accessibility.

@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown

Deploy preview for mistica-web ready!

✅ Preview
https://mistica-o11swbagu-flows-projects-65bb050e.vercel.app

Built with commit d8ff5e6.
This pull request is being automatically deployed with vercel-action

@Marcosld Marcosld changed the title feat(chore): Improve skill and fix patterns example feat(Skill): Improve skill and fix patterns example Apr 8, 2026
Comment thread doc/patterns.md Outdated
Co-authored-by: Ignacio Ceballos (Yayo) <yceballost@gmail.com>
@Marcosld Marcosld requested a review from yceballost April 9, 2026 07:40
Comment thread doc/llms.md Outdated
Comment thread skills/mistica-react/SKILL.md Outdated
and step-by-step instructions on which documentation files to read and in what order.

### Step 1: Read the main reference
> **Fallback**: If `node_modules/@telefonica/mistica` is not available, fetch the equivalent file from the

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
> **Fallback**: If `node_modules/@telefonica/mistica` is not available, fetch the equivalent file from the
> **Fallback**: If `node_modules/@telefonica/mistica/doc/llms.md` is not available, fetch the equivalent file from the

Although this could be dangerous, the LLM could be reading docs from a different mistica version. Maybe we should suggest to install dependencies (yarn/npm i) first before going to the repo

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, the step just before this one (Setup) suggests that the llm should install mistica first of all

@Marcosld Marcosld enabled auto-merge April 13, 2026 09:47
@Marcosld Marcosld added this pull request to the merge queue Apr 13, 2026
Merged via the queue into master with commit 522c305 Apr 13, 2026
11 checks passed
@Marcosld Marcosld deleted the WEB-2427 branch April 13, 2026 10:15
@tuentisre

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 16.59.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@tuentisre

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 16.59.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants