Skip to content

chore: update skill creator#1431

Merged
Wendong-Fan merged 15 commits intomainfrom
update_skill-creator
Mar 18, 2026
Merged

chore: update skill creator#1431
Wendong-Fan merged 15 commits intomainfrom
update_skill-creator

Conversation

@fengju0213
Copy link
Collaborator

@fengju0213 fengju0213 commented Mar 4, 2026

Related Issue

#1429

Closes #

Description

Testing Evidence (REQUIRED)

  • I have included human-verified testing evidence in this PR.
  • This PR includes frontend/UI changes, and I attached screenshot(s) or screen recording(s).
  • No frontend/UI changes in this PR.

What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

Contribution Guidelines Acknowledgement

@fengju0213 fengju0213 linked an issue Mar 4, 2026 that may be closed by this pull request
Copy link
Contributor

@Wendong-Fan Wendong-Fan left a comment

Choose a reason for hiding this comment

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

i think we have same issues for other example skills, not just for the skill.md

@fengju0213
Copy link
Collaborator Author

i think we have same issues for other example skills, not just for the skill.md

will do a full check

@fengju0213
Copy link
Collaborator Author

updated cc @Wendong-Fan

@4pmtong
Copy link
Collaborator

4pmtong commented Mar 18, 2026

Hi @fengju0213 Can we keep the previous format for the skills docx, pdf, pptx, and xlsx? Currently there are a lot of unnecessary spaces and line breaks added, which wastes tokens.

@fengju0213
Copy link
Collaborator Author

Hi @fengju0213 Can we keep the previous format for the skills docx, pdf, pptx, and xlsx? Currently there are a lot of unnecessary spaces and line breaks added, which wastes tokens.

i think is because our CICD,let me check

@fengju0213
Copy link
Collaborator Author

Hi @fengju0213 Can we keep the previous format for the skills docx, pdf, pptx, and xlsx? Currently there are a lot of unnecessary spaces and line breaks added, which wastes tokens.

i think the format change is because cicd,i dont edited the format in this pr

Copy link
Collaborator

@4pmtong 4pmtong left a comment

Choose a reason for hiding this comment

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

LGTM

@4pmtong 4pmtong requested a review from Wendong-Fan March 18, 2026 10:05
@Wendong-Fan Wendong-Fan merged commit 14139d7 into main Mar 18, 2026
9 checks passed
@fengju0213 fengju0213 added this to the Sprint 19 milestone Mar 19, 2026
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.

[BUG] Example skills not matching with Eigent

3 participants